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

[dcl.fct.spec] mistakenly used as reference for 'inline' #1458

Closed
Eelis opened this issue Feb 11, 2017 · 1 comment
Closed

[dcl.fct.spec] mistakenly used as reference for 'inline' #1458

Eelis opened this issue Feb 11, 2017 · 1 comment
Assignees
@jensmaurer

Comments

@Eelis
Copy link
Contributor

Eelis commented Feb 11, 2017

17.5.5.4p1 reads:

It is unspecified whether any non-member functions in the C++ standard library are defined as inline (7.1.2).

And 17.5.5.5p1 reads:

It is unspecified whether any member functions in the C++ standard library are defined as inline (7.1.2).

But 7.1.2 is [dcl.fct.spec] "Function specifiers", which only talks about "virtual" and "explicit", not about "inline", which isn't a function-specifier.
@cpplearner
Copy link
Contributor

9.2.1p1:

A member function may be defined (8.4) in its class definition, in which case it is an inline member
function (7.1.2)

9.2.1p3:

[ Note: There can be at most one definition of a non-inline member function in a program. There may be more than one inline member function definition in a program. See 3.2 and 7.1.2. — end note ]

And 7.1.6[dcl.inline]p2 (!):

An implementation is not required to perform this inline substitution at the point of call; however, even if this inline substitution is omitted, the other rules for inline functions defined by 7.1.2 shall still be respected.

@Eelis Eelis changed the title [member.functions, global.functions] Wrong reference for 'inline' [dlc.fct.spec] mistakenly used as reference for 'inline' Feb 12, 2017
@Eelis Eelis changed the title [dlc.fct.spec] mistakenly used as reference for 'inline' [dcl.fct.spec] mistakenly used as reference for 'inline' Feb 12, 2017
jensmaurer added a commit to jensmaurer/draft that referenced this issue Feb 14, 2017
@jensmaurer
Fix cross-reference for 'inline'.
46777f8 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
@jensmaurer jensmaurer mentioned this issue Feb 14, 2017
Merged
@jensmaurer jensmaurer self-assigned this Feb 14, 2017
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 15, 2017
@jensmaurer @Eelis
Fix cross-reference for 'inline'.
bdf3640 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 15, 2017
@jensmaurer @Eelis
Fix cross-reference for 'inline'.
d4d4eb6 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 18, 2017
@jensmaurer @Eelis
Fix cross-reference for 'inline'.
a11269d 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 18, 2017
@jensmaurer @Eelis
Fix cross-reference for 'inline'.
779b929 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
Eelis pushed a commit to Eelis/draft that referenced this issue Feb 19, 2017
@jensmaurer @Eelis
Fix cross-reference for 'inline'.
6017ab6 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes cplusplus#1458.
tkoeppe pushed a commit that referenced this issue Feb 23, 2017
@jensmaurer @tkoeppe
Fix cross-reference for 'inline'. (#1464)
0f1e91e 
The 'inline' specifier is defined in [dcl.inline];
adjust cross-references still pointing to [dcl.fct.spec].

Fixes #1458.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Eelis @cpplearner @jensmaurer