Skip to content

Conversation

@ednolan
Copy link
Member

@ednolan ednolan commented Oct 25, 2025

Remove unnecessary clearing of to_increment_, clean up logic, remove separate overloads for advance_one(), remove unnecessary else clause for if constexpr in operator==

@ednolan ednolan requested a review from camio as a code owner October 25, 2025 17:56
@coveralls
Copy link

coveralls commented Oct 25, 2025

Coverage Status

coverage: 98.856% (-0.03%) from 98.886%
when pulling 99e2bba on enolan_operatorplus2
into 8b971fb on main.

@ednolan ednolan force-pushed the enolan_operatorplus2 branch 3 times, most recently from fd33c96 to 553ffd3 Compare October 25, 2025 18:07
Remove unnecessary clearing of to_increment_, clean up logic, remove
separate overloads for advance_one(), remove unnecessary else clause
for if constexpr in operator==
@ednolan ednolan force-pushed the enolan_operatorplus2 branch from 553ffd3 to 99e2bba Compare October 25, 2025 18:34
@ednolan ednolan changed the title Further simplify operator++ Further simplify operator++, operator--, operator== Oct 25, 2025
@ednolan ednolan merged commit f7af12b into main Oct 25, 2025
37 checks passed
@ednolan ednolan deleted the enolan_operatorplus2 branch October 25, 2025 19:17
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

Successfully merging this pull request may close these issues.

3 participants