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

[over.ics.rank] p3.2.1 "Proper subsequence" vs. "subsequence" #5160

Open
xmh0511 opened this issue Dec 21, 2021 · 1 comment
Open

[over.ics.rank] p3.2.1 "Proper subsequence" vs. "subsequence" #5160

xmh0511 opened this issue Dec 21, 2021 · 1 comment

Comments

@xmh0511
Copy link
Contributor

xmh0511 commented Dec 21, 2021

[over.ics.rank] p3.2.1

S1 is a proper subsequence of S2 (comparing the conversion sequences in the canonical form defined by [over.ics.scs], excluding any Lvalue Transformation; the identity conversion sequence is considered to be a subsequence of any non-identity conversion sequence) or, if not that,

It seems "proper subsequence" has a stricter meaning than "subsequence". The intent of this provision is to say that [over.ics.rank] p3.2.1 will be applied to if S1 is an identity conversion sequence while S2 is a non-identity conversion sequence. However, according to the sentence in the parenthesis: the identity conversion sequence is considered to be a subsequence of any non-identity conversion sequence. It did not say it's a "proper subsequence".

Should we change the definition to that

S1 is a proper subsequence of S2 (comparing the conversion sequences in the canonical form defined by [over.ics.scs], excluding any Lvalue Transformation; the identity conversion sequence is considered to be a proper subsequence of any non-identity conversion sequence) or, if not that,

@frederick-vs-ja
Copy link
Contributor

"X is a proper subsequence of Y" subsumes both "X is a subsequence of Y" and "X is not Y".

I think we should say

the identity conversion sequence is considered to be a subsequence of any non-identity conversion sequence

to avoid verbosity.

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

No branches or pull requests

2 participants