blender
/
blender
132
398
Fork 577
Code Issues 6.2k Pull Requests 743 Projects 19 Wiki Activity

Cleanup: nodeToView & nodeFromView usage #105469

Closed
Iliya Katushenock wants to merge 5 commits from mod_moder:cleanup_no_to_view_usage into main
pull from: mod_moder:cleanup_no_to_view_usage
merge into: blender:main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Conversation 1 Commits 5 Files Changed 1 +9 -24
1 changed files with 9 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
source/blender/editors/transform/transform_convert_node.cc
View File

@ -45,19 +45,8 @@ static void create_transform_data_for_node(TransData &td,
const float dpi_fac)
{
float locx, locy;
/* account for parents (nested nodes) */
if (node.parent) {
nodeToView(node.parent,
node.locx + roundf(node.offsetx),
node.locy + roundf(node.offsety),
&locx,
&locy);
}
else {
locx = node.locx + roundf(node.offsetx);
locy = node.locy + roundf(node.offsety);
}
/* Account for parents (nested nodes). */
nodeToView(&node, roundf(node.offsetx), roundf(node.offsety), &locx, &locy);
/* use top-left corner as the transform origin for nodes */
/* Weirdo - but the node system is a mix of free 2d elements and DPI sensitive UI. */
@ -246,18 +235,14 @@ static void flushTransNodes(TransInfo *t)
loc[0] /= dpi_fac;
loc[1] /= dpi_fac;
/* account for parents (nested nodes) */
if (node->parent) {
nodeFromView(node->parent,
loc[0] - roundf(node->offsetx),
loc[1] - roundf(node->offsety),
&node->locx,
&node->locy);
}
else {
node->locx = loc[0] - roundf(node->offsetx);
node->locy = loc[1] - roundf(node->offsety);
loc[0] -= roundf(node->offsetx);
loc[1] -= roundf(node->offsety);
if (const bNode *parent_space = node->parent) {
mod_moder marked this conversation as resolved Outdated
Hans Goudey commented 2023-03-05 23:18:05 +01:00
Outdated
Review
Copy Link

The second parent_space != nullptr is unnecessary here in C++ as far as I know

The second `parent_space != nullptr` is unnecessary here in C++ as far as I know
Iliya Katushenock commented 2023-03-06 11:11:04 +01:00
Outdated
Review
Copy Link

I do this for greater clarity, but yes, we can consider it redundant

I do this for greater clarity, but yes, we can consider it redundant
nodeFromView(parent_space, loc[0], loc[1], &loc[0], &loc[1]);
}
node->locx = loc[0];
node->locy = loc[1];
}
/* handle intersection with noodles */