[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ #3209

cjdb · 2019-09-02T18:46:06Z

The boolean-expressions in [iterators] should be explicitly converted to
bool. This commit fixes a few strays that weren't boolified.

The boolean-expressions in [iterators] should be explicitly converted to `bool`. This commit fixes a few strays that weren't boolified.

jwakely · 2019-09-02T21:49:14Z

Is this correction already implied by a paper approved in Cologne? Otherwise, this is arguably not editorial and so would be better handled as an LWG issue.

jensmaurer · 2019-09-03T13:55:27Z

References to prior art (papers, LWG issues) would be good. Otherwise, this sounds non-editorial.

cjdb changed the title ~~s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/~~ [iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ Sep 2, 2019

[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/

2b9939d

The boolean-expressions in [iterators] should be explicitly converted to `bool`. This commit fixes a few strays that weren't boolified.

jensmaurer added the lwg Issue must be reviewed by LWG. label Sep 3, 2019

wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 15, 2021

tkoeppe closed this Jun 19, 2021

tkoeppe deleted the branch cplusplus:master June 19, 2021 15:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ #3209

[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ #3209

cjdb commented Sep 2, 2019

jwakely commented Sep 2, 2019

jensmaurer commented Sep 3, 2019

[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ #3209

[iterators] s/a == b/bool(a == b)/ and s/++a == ++b/bool(++a == ++b)/ #3209

Conversation

cjdb commented Sep 2, 2019

jwakely commented Sep 2, 2019

jensmaurer commented Sep 3, 2019