-
-
Notifications
You must be signed in to change notification settings - Fork 732
Update volatile.adoc: Add explanation to examples #712
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
Conversation
This change also adds a visual separator between the two code blocks so it is clear that they do not belong together.
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.
Hi @Granjow. Thanks so much for your pull request. I have made a few suggestions for how it might be improved. Please let me know what you (and any other interested parties) think.
Language/Variables/Variable Scope & Qualifiers/volatile.adoc
Outdated
Show resolved
Hide resolved
Language/Variables/Variable Scope & Qualifiers/volatile.adoc
Outdated
Show resolved
Hide resolved
Language/Variables/Variable Scope & Qualifiers/volatile.adoc
Outdated
Show resolved
Hide resolved
Hi @per1234, thanks a lot for your review and sorry for the bad quality of the MR. I was too much in a rush. Will improve it, but it is going to take a few days :) |
@per1234 Any comments on the updated docs? |
@per1234 ping. Can this MR be merged? Any open issues? |
Guys, this PR is more than 1.5 years old. Would really appreciate to get this forward a bit. @kengdahl could you help me on that maybe? |
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.
Suggested some changes in the loop
.
For the rest the changes looks good.
Language/Variables/Variable Scope & Qualifiers/volatile.adoc
Outdated
Show resolved
Hide resolved
Language/Variables/Variable Scope & Qualifiers/volatile.adoc
Outdated
Show resolved
Hide resolved
Thanks for merging! |
This change also adds a visual separator between the two code blocks so it is clear that they do not belong together.