[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