Skip to content
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

Delete the notionally redundant 'update' when describing actions upon ... #627

Merged
merged 1 commit into from Mar 7, 2016

Conversation

faisalv
Copy link
Contributor

@faisalv faisalv commented Feb 27, 2016

... independent memory locations. The definition of 'access' includes modifying a location per 1.3.1/access, so we don't need to state 'update and access' we should be able to just state 'access'

(p.s. My first pull request - please let me know if I'm not doing this right or I should be doing something better. Thank you!)

…t memory locations. The definition of 'access' includes modifying a location per 1.3.1/access.
zygoloid added a commit that referenced this pull request Mar 7, 2016
[intro.memory] Delete the notionally redundant 'update' when describing actions upon independent memory locations.
@zygoloid zygoloid merged commit 7e1ed4b into cplusplus:master Mar 7, 2016
@zygoloid
Copy link
Member

zygoloid commented Mar 7, 2016

Regarding your ps: we aim for editorial commits to the draft to start with the section name they're modifying, for instance "[intro.memory] Delete the [...]". No big deal if you forget, I can do it as part of the merge, but it saves me a couple of seconds.

@faisalv faisalv deleted the fv branch March 8, 2016 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants