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

<xiosbase>: Non-conforming extension ios_base::hexfloat #3296

Closed
frederick-vs-ja opened this issue Dec 18, 2022 · 1 comment · Fixed by #4345
Closed

<xiosbase>: Non-conforming extension ios_base::hexfloat #3296

frederick-vs-ja opened this issue Dec 18, 2022 · 1 comment · Fixed by #4345
Labels
bug Something isn't working fixed Something works now, yay!

Comments

@frederick-vs-ja
Copy link
Contributor

Describe the bug

MSVC STL provides ios_base::hexfloat, which is probably non-conforming.

static constexpr _Fmtflags hexfloat = static_cast<_Fmtflags>(0x3000); // added with TR1 (not in C++11)

And the comment seems wrong as ios_base::hexfloat is not a part of TR1. It was stated in the final draft of TR1 (WG21-N1836) that:

[Note: Adding the format flag hexfloat to class ios_base cannot be done without invading namespace std, so only the named manipulator is provided. Note also that the more obvious use of ios_base::hex to specify hexadecimal floating-point format would change the meaning of existing well defined programs. C++2003 gives no meaning to the combination of fixed and scientific. —end note]

Should we remove it (maybe in vNext) or make it conforming with an LWG issue or a paper?

@strega-nil-ms
Copy link
Contributor

Yeah this is certainly a vNext kind of thing.

@strega-nil-ms strega-nil-ms added vNext Breaks binary compatibility bug Something isn't working labels Dec 20, 2022
@StephanTLavavej StephanTLavavej added fixed Something works now, yay! and removed vNext Breaks binary compatibility labels Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed Something works now, yay!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants