-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix dirtyness propagation optimization to work fine with tweaks #2048
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn’t it be more appropriate to move this business logic out of the template, to the helper definitions or even further? Templates are harder to maintain.
ecb2e50
to
49a3e0d
Compare
If I understand #2047 correctly, the following is happening:
but the code generated in xod/packages/xod-arduino/platform/program.tpl.cpp Lines 149 to 150 in faf25a8
(and maybe in some other place?) does not respect that. I think that forcing generation of |
49a3e0d
to
9fc2298
Compare
@nkrkv @evgenykochetkov check it out again, please :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM👍🏻
9fc2298
to
7f279f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
It fixes #2047
The bug was introduced in the 0.33.0 right in the commit 0880f41 😬