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
[language.support, utilities] Condense description of exception classes #1867
Conversation
Note that this currently has a lame sentence "The constructors are specified in \ref{exception}." to point readers to the paragraph that says "all standard library classes T deriving from std::exception have these constructors". I've also removed the NTBS blurb from the derived classes; it's now implied by std::exception::what(). Maybe we can get rid of the lame sentence and just add a comment in the class synopsis, saying something like "// see also \ref{exception}. Suggestions welcome. Oh, and before I forget: This is borderline editorial; for @zygoloid to decide. |
I would prefer a comment inside the class definition pointing at [exception] rather than wording in a section that a casual reader may well not look at. Just using the defaulted special members here seems very reasonable to me. I think it's fine that we now give these exception subclasses move operations, but a second opinion on that would be good (@jwakely?). |
Updated. The fun part is that |
Fixed the "logic_error now has a default constructor" issue. We might want to have LWG ack the changes. |
@jwakely: Ping |
e3dbfe2
to
1a21a65
Compare
Rebased. |
Rebased. @jwakely, ping? |
@jwakely, ping^2. |
Fixes #748.