[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [49713] trunk/blender/source/blender: Renamed the mx/my coordinates in SpaceNode to more descriptive 'cursor'.
Lukas Toenne
lukas.toenne at googlemail.com
Wed Aug 8 19:37:02 CEST 2012
Revision: 49713
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=49713
Author: lukastoenne
Date: 2012-08-08 17:37:02 +0000 (Wed, 08 Aug 2012)
Log Message:
-----------
Renamed the mx/my coordinates in SpaceNode to more descriptive 'cursor'. This follows the naming in the View3D space, since this vector is used for essentially the same purpose (adding nodes at a specific location), although not used for any transform origin and not displayed in the editor.
Note: While renaming DNA properties is usually dangerous and can break files, in this case it is possible because the mx/my values are only used at runtime.
Modified Paths:
--------------
trunk/blender/source/blender/editors/space_node/drawnode.c
trunk/blender/source/blender/editors/space_node/node_add.c
trunk/blender/source/blender/editors/space_node/node_draw.c
trunk/blender/source/blender/editors/space_node/node_edit.c
trunk/blender/source/blender/editors/space_node/node_header.c
trunk/blender/source/blender/editors/space_node/node_relationships.c
trunk/blender/source/blender/editors/space_node/node_select.c
trunk/blender/source/blender/editors/space_node/space_node.c
trunk/blender/source/blender/makesdna/DNA_space_types.h
trunk/blender/source/blender/makesrna/intern/rna_space.c
Modified: trunk/blender/source/blender/editors/space_node/drawnode.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/drawnode.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/drawnode.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -3185,8 +3185,7 @@
}
else {
if (snode == NULL) return 0;
- vec[0][0] = snode->mx;
- vec[0][1] = snode->my;
+ copy_v2_v2(vec[0], snode->cursor);
fromreroute = 0;
}
if (link->tosock) {
@@ -3196,8 +3195,7 @@
}
else {
if (snode == NULL) return 0;
- vec[3][0] = snode->mx;
- vec[3][1] = snode->my;
+ copy_v2_v2(vec[3], snode->cursor);
toreroute = 0;
}
Modified: trunk/blender/source/blender/editors/space_node/node_add.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_add.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_add.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -273,7 +273,7 @@
ED_preview_kill_jobs(C);
- node = node_add_node(snode, bmain, scene, &ntemp, snode->mx, snode->my);
+ node = node_add_node(snode, bmain, scene, &ntemp, snode->cursor[0], snode->cursor[1]);
if (!node) {
BKE_report(op->reports, RPT_WARNING, "Could not add an image node");
@@ -296,7 +296,7 @@
/* convert mouse coordinates to v2d space */
UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1],
- &snode->mx, &snode->my);
+ &snode->cursor[0], &snode->cursor[1]);
if (RNA_struct_property_is_set(op->ptr, "filepath") || RNA_struct_property_is_set(op->ptr, "name"))
return node_add_file_exec(C, op);
Modified: trunk/blender/source/blender/editors/space_node/node_draw.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_draw.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_draw.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -989,11 +989,11 @@
else {
/* check nodes front to back */
for (node = ntree->nodes.last; node; node = node->prev) {
- if (BLI_in_rctf(&node->totr, snode->mx, snode->my))
+ if (BLI_in_rctf(&node->totr, snode->cursor[0], snode->cursor[1]))
break; /* first hit on node stops */
}
if (node) {
- int dir = node->typeinfo->resize_area_func(node, snode->mx, snode->my);
+ int dir = node->typeinfo->resize_area_func(node, snode->cursor[0], snode->cursor[1]);
cursor = node_get_resize_cursor(dir);
}
}
Modified: trunk/blender/source/blender/editors/space_node/node_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_edit.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_edit.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -797,8 +797,8 @@
NodeSizeWidget *nsw = MEM_callocN(sizeof(NodeSizeWidget), "size widget op data");
op->customdata = nsw;
- nsw->mxstart = snode->mx;
- nsw->mystart = snode->my;
+ nsw->mxstart = snode->cursor[0];
+ nsw->mystart = snode->cursor[1];
/* store old */
nsw->oldlocx = node->locx;
@@ -936,8 +936,8 @@
if (node) {
/* convert mouse coordinates to v2d space */
UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1],
- &snode->mx, &snode->my);
- dir = node->typeinfo->resize_area_func(node, snode->mx, snode->my);
+ &snode->cursor[0], &snode->cursor[1]);
+ dir = node->typeinfo->resize_area_func(node, snode->cursor[0], snode->cursor[1]);
if (dir != 0) {
node_resize_init(C, op, event, node, dir);
return OPERATOR_RUNNING_MODAL;
@@ -1051,10 +1051,10 @@
/* check if we click in a socket */
for (node = snode->edittree->nodes.first; node; node = node->next) {
- rect.xmin = snode->mx - (NODE_SOCKSIZE + 4);
- rect.ymin = snode->my - (NODE_SOCKSIZE + 4);
- rect.xmax = snode->mx + (NODE_SOCKSIZE + 4);
- rect.ymax = snode->my + (NODE_SOCKSIZE + 4);
+ rect.xmin = snode->cursor[0] - (NODE_SOCKSIZE + 4);
+ rect.ymin = snode->cursor[1] - (NODE_SOCKSIZE + 4);
+ rect.xmax = snode->cursor[0] + (NODE_SOCKSIZE + 4);
+ rect.ymax = snode->cursor[1] + (NODE_SOCKSIZE + 4);
if (!(node->flag & NODE_HIDDEN)) {
/* extra padding inside and out - allow dragging on the text areas too */
@@ -2052,8 +2052,8 @@
/* place nodes around the mouse cursor. child nodes locations are relative to parent */
if (!new_node->parent) {
- new_node->locx += snode->mx - centerx - gnode_x;
- new_node->locy += snode->my - centery - gnode_y;
+ new_node->locx += snode->cursor[0] - centerx - gnode_x;
+ new_node->locy += snode->cursor[1] - centery - gnode_y;
}
}
@@ -2076,7 +2076,7 @@
SpaceNode *snode = CTX_wm_space_node(C);
/* convert mouse coordinates to v2d space */
- UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1], &snode->mx, &snode->my);
+ UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1], &snode->cursor[0], &snode->cursor[1]);
return node_clipboard_paste_exec(C, op);
}
Modified: trunk/blender/source/blender/editors/space_node/node_header.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_header.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_header.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -74,7 +74,7 @@
int y = win->eventstate->y - ar->winrct.ymin;
if (y < 60) y += 60;
- UI_view2d_region_to_view(&ar->v2d, x, y, &snode->mx, &snode->my);
+ UI_view2d_region_to_view(&ar->v2d, x, y, &snode->cursor[0], &snode->cursor[1]);
}
}
@@ -84,7 +84,7 @@
else node->flag &= ~NODE_TEST;
}
- /* node= */ node_add_node(snode, bmain, scene, ntemp, snode->mx, snode->my);
+ /* node= */ node_add_node(snode, bmain, scene, ntemp, snode->cursor[0], snode->cursor[1]);
/* select previous selection before autoconnect */
for (node = snode->edittree->nodes.first; node; node = node->next) {
Modified: trunk/blender/source/blender/editors/space_node/node_relationships.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_relationships.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_relationships.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -437,10 +437,10 @@
{
bNode *gnode = node_tree_get_editgroup(snode->nodetree);
if (gnode) {
- return (snode->mx < gnode->totr.xmin ||
- snode->mx >= gnode->totr.xmax ||
- snode->my < gnode->totr.ymin ||
- snode->my >= gnode->totr.ymax);
+ return (snode->cursor[0] < gnode->totr.xmin ||
+ snode->cursor[0] >= gnode->totr.xmax ||
+ snode->cursor[1] < gnode->totr.ymin ||
+ snode->cursor[1] >= gnode->totr.ymax);
}
return 0;
}
@@ -462,7 +462,7 @@
in_out = nldrag->in_out;
UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1],
- &snode->mx, &snode->my);
+ &snode->cursor[0], &snode->cursor[1]);
switch (event->type) {
case MOUSEMOVE:
@@ -708,7 +708,7 @@
int detach = RNA_boolean_get(op->ptr, "detach");
UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1],
- &snode->mx, &snode->my);
+ &snode->cursor[0], &snode->cursor[1]);
ED_preview_kill_jobs(C);
@@ -1139,7 +1139,7 @@
/* skip selected, those are the nodes we want to attach */
if ((frame->type != NODE_FRAME) || (frame->flag & NODE_SELECT))
continue;
- if (BLI_in_rctf(&frame->totr, snode->mx, snode->my))
+ if (BLI_in_rctf(&frame->totr, snode->cursor[0], snode->cursor[1]))
break;
}
if (frame) {
@@ -1185,7 +1185,7 @@
SpaceNode *snode = CTX_wm_space_node(C);
/* convert mouse coordinates to v2d space */
- UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1], &snode->mx, &snode->my);
+ UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1], &snode->cursor[0], &snode->cursor[1]);
return node_attach_exec(C, op);
}
Modified: trunk/blender/source/blender/editors/space_node/node_select.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/node_select.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/node_select.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -330,8 +330,8 @@
/* get mouse coordinates in view2d space */
UI_view2d_region_to_view(&ar->v2d, mval[0], mval[1], &mx, &my);
/* node_find_indicated_socket uses snode->mx/my */
- snode->mx = mx;
- snode->my = my;
+ snode->cursor[0] = mx;
+ snode->cursor[1] = my;
if (extend) {
/* first do socket selection, these generally overlap with nodes.
Modified: trunk/blender/source/blender/editors/space_node/space_node.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/space_node.c 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/editors/space_node/space_node.c 2012-08-08 17:37:02 UTC (rev 49713)
@@ -337,7 +337,7 @@
/* convert mouse coordinates to v2d space */
UI_view2d_region_to_view(&ar->v2d, win->eventstate->x - ar->winrct.xmin, win->eventstate->y - ar->winrct.ymin,
- &snode->mx, &snode->my);
+ &snode->cursor[0], &snode->cursor[1]);
node_set_cursor(win, snode);
}
Modified: trunk/blender/source/blender/makesdna/DNA_space_types.h
===================================================================
--- trunk/blender/source/blender/makesdna/DNA_space_types.h 2012-08-08 17:26:13 UTC (rev 49712)
+++ trunk/blender/source/blender/makesdna/DNA_space_types.h 2012-08-08 17:37:02 UTC (rev 49713)
@@ -879,7 +879,7 @@
float xof, yof; /* offset for drawing the backdrop */
float zoom; /* zoom for backdrop */
- float mx, my; /* mousepos for drawing socketless link */
+ float cursor[2]; /* mouse pos for drawing socketless link and adding nodes */
struct bNodeTree *nodetree, *edittree;
int treetype; /* treetype: as same nodetree->type */
Modified: trunk/blender/source/blender/makesrna/intern/rna_space.c
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list