This if statement smells weird. Can anyone explain? #13881
Closed
krushia
started this conversation in
Code contribution discussions
Replies: 2 comments
-
Probably a mistake introduced when the curly braces were added to the single-line conditional. Obviously only the first return is needed. Thanks for the catch -- feel free to submit a cleanup PR. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Thanks. Somehow confused my little brain more than it should have. I'm reviewing the whole file, will try to get a PR in when done. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
FreeCAD/src/App/Document.cpp
Lines 1075 to 1078 in c1bc3b6
Beta Was this translation helpful? Give feedback.
All reactions