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.

Sign up for GitHub

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

Jump to bottom

[ratio.ratio] Order of ratio members #67

Closed
sdutoit opened this issue Apr 15, 2013 · 0 comments
Closed

[ratio.ratio] Order of ratio members #67

sdutoit opened this issue Apr 15, 2013 · 0 comments
Labels
tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent.
Milestone
2013 Post-Bristol...

Comments

@sdutoit
Copy link
Contributor

sdutoit commented Apr 15, 2013

Via cxxeditor@:

In 20.10.3 Class template ratio [ratio.ratio] we declare:

namespace std {
 template <intmax_t N, intmax_t D = 1>
 class ratio {
   public:
     typedef ratio<num, den> type;
     static constexpr intmax_t num;
     static constexpr intmax_t den;
 };
}

It would be clearer as:

class ratio {
  public:
    static constexpr intmax_t num;
    static constexpr intmax_t den;
    typedef ratio<num, den> type;
};
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent.
Projects
None yet
Milestone
2013 Post-Bristol Mailing Draft
Development

No branches or pull requests
1 participant
@sdutoit