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

[compatibility] Add compatibility notices for pp-number #713

Merged
merged 1 commit into from Jun 8, 2016

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Apr 21, 2016

Addresses #680.


\begin{codeblock}
#define F(a) b ## a
int b0p = F(0p0+0); // ill-formed; equivalent to ``\tcode{int b0p = b0p + 0;}\!'' in C++ 2003
Copy link
Contributor

Choose a reason for hiding this comment

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

Is F(0p+0) meant?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@timsong-cpp: Yes :-)

@zygoloid
Copy link
Member

We generally only list differences against other languages in one subclause of Annex C.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented May 31, 2016

@zygoloid: So where should this go: break from C++14, or from C++03?

@tkoeppe
Copy link
Contributor Author

tkoeppe commented May 31, 2016

@zygoloid: Now there's only a single compatibility note against C++14, with the both examples.

Valid \CppXI code may fail to compile or produce different results in
this International Standard. Specifically, character sequences like \tcode{0p+0}
and \tcode{0e1_p+0} are three separate tokens each in \CppXIV, but one single token
in this International Standard.
Copy link
Contributor

Choose a reason for hiding this comment

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

"Valid \CppXI code"...."in \CppXIV"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well spotted, thanks! Fixed.

@tkoeppe tkoeppe force-pushed the annexhex branch 2 times, most recently from d75230e to ba7598b Compare June 8, 2016 09:52
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Jun 8, 2016

Updated, thanks @timsong-cpp. New screenshot:

image

@zygoloid: PTAL

@zygoloid zygoloid merged commit a167f5c into cplusplus:master Jun 8, 2016
tkoeppe added a commit to google/cxx-std-draft that referenced this pull request Jun 9, 2016
FrankHB pushed a commit to FrankHB/draft that referenced this pull request Jul 9, 2016
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