diff options
author | Ton Roosendaal <ton@blender.org> | 2009-05-20 20:55:28 +0400 |
---|---|---|
committer | Ton Roosendaal <ton@blender.org> | 2009-05-20 20:55:28 +0400 |
commit | d5a4a57e5c2303607a98fd4c6a0cd6fb67339bca (patch) | |
tree | 403c125995aa9decb91900f4c740bfe3efb58ec8 /source/blender | |
parent | 34aaa18af2d6de656ead73c5de02fe8f0ed4eb49 (diff) |
2.5
Node editor:
- Crash could occur on ALT+LMB remove link lines (read free'd mem)
- Button labels were handled as full buttons, disabling node usage
quite a lot, like drag, select, or create links.
(Caused by making all node labels buttons)
Brecht: old UI_NO_HILITE can be depricated, it was for bad overdraw
issues. I guess it's OK to not handle Label buttons at all...
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/editors/interface/interface_handlers.c | 2 | ||||
-rw-r--r-- | source/blender/editors/space_node/node_edit.c | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/source/blender/editors/interface/interface_handlers.c b/source/blender/editors/interface/interface_handlers.c index 65752423dce..ea5eb354b54 100644 --- a/source/blender/editors/interface/interface_handlers.c +++ b/source/blender/editors/interface/interface_handlers.c @@ -2843,6 +2843,8 @@ static uiBut *ui_but_find_mouse_over(ARegion *ar, int x, int y) for(but=block->buttons.first; but; but= but->next) { if(but->flag & UI_NO_HILITE) continue; + if(but->type==LABEL) + continue; if(ui_but_contains_pt(but, mx, my)) /* give precedence to already activated buttons */ diff --git a/source/blender/editors/space_node/node_edit.c b/source/blender/editors/space_node/node_edit.c index 5ee7326cd2f..56d7a649eb1 100644 --- a/source/blender/editors/space_node/node_edit.c +++ b/source/blender/editors/space_node/node_edit.c @@ -2118,7 +2118,7 @@ static int cut_links_exec(bContext *C, wmOperator *op) if(i>1) { bNodeLink *link, *next; - for(link= snode->edittree->links.first; link; link= link->next) { + for(link= snode->edittree->links.first; link; link= next) { next= link->next; if(cut_links_intersect(link, mcoords, i)) { |