[Bf-codereview] Node linking operator supports multiple links at once (issue 6206070)
brechtvanlommel at gmail.com
brechtvanlommel at gmail.com
Tue May 15 14:34:49 CEST 2012
LGTM.
http://codereview.appspot.com/6206070/diff/1/source/blender/editors/space_node/node_edit.c
File source/blender/editors/space_node/node_edit.c (right):
http://codereview.appspot.com/6206070/diff/1/source/blender/editors/space_node/node_edit.c#newcode2479
source/blender/editors/space_node/node_edit.c:2479: /* XXX links->data
pointers are either held by the tree or freed already */
Does this "XXX" indicate that something needs to be fixed here?
Otherwise I'd leave it out.
http://codereview.appspot.com/6206070/
More information about the Bf-codereview
mailing list