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

[utility,iterator] Apply p0174r2 deprecating vestigial library compon… #771

Closed

Conversation

AlisdairM
Copy link
Contributor

…ents

Moves text (with minor tweaks, per p0174r2) to Annex D
for deprecation, and updates the indices accordingly.

A drive-by fix in allocator so that operator==/!= in
the synopsis use the same parameter names in the
definition.

Moved another example using std::iterator into the deprecated clause,
as it is the appropriate location, but was missed in the original paper.

…ents

Moves text (with minor tweaks, per p0174r2) to Annex D
for deprecation, and updates the indices accordingly.

A drive-by fix in allocator so that operator==/!= in
the synopsis use the same parameter names in the
definition.
\indexlibrary{\idxcode{allocate}!\idxcode{allocator}}
\indexlibrary{\idxcode{allocator}!\idxcode{allocate}}
\begin{itemdecl}
T* allocate(size_type n, void* hint);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The synopsis above has T* allocate(size_t, const void* hint);.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - the synopsis is correct and I need to fix this.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 27, 2016

@AlisdairM: Did you check with @zygoloid? Usually formal motions aren't applied as editorial pull requests, but rather, the editor has his own mechanism for creating and applying the changes directly. I'm sure the help is appreciated, but maybe double-check first, because until now motion applications have never been handled via pull requests.

@AlisdairM
Copy link
Contributor Author

I mentioned in Oulu I would put up pull requests for my 3 library motions, hopefully for the end of Monday. No other scheme was suggested, although I am happy to take guidance. Trying to simplify everyone's work at a busy time, not make more process, so just point me to whatever will work most easily.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 27, 2016

OK, that's fine, if you've mentioned this to Richard already and Dawn knows to not duplicate the work, then I'm sure the help is well appreciated. :-)

@zygoloid, @burblebee: ping

@AlisdairM AlisdairM closed this Jun 27, 2016
@AlisdairM AlisdairM deleted the apply_p0174r2_deprecations branch June 27, 2016 19:23
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

3 participants