Opened 3 years ago

Closed 3 years ago

#11652 closed Bugs (fixed)

Checking __cpp_lib_integer_sequence

Reported by: Mikhail Strelnikov <mikhail.strelnikov@…> Owned by: olli
Milestone: To Be Determined Component: context
Version: Boost 1.59.0 Severity: Cosmetic
Keywords: Cc:


In this line (boost/context/detail/config.hpp):

! defined(cpp_lib_integer_sequence) && cpp_lib_integer_sequence < 201304

If cpp_lib_integer_sequence is not defined, then it is zero and checking against 201304 is redundant - 0 is less than 201304.

If cpp_lib_integer_sequence is defined, then comparinson with 201304 is not evaluated because of short-circuiting.

I think "! defined(cpp_lib_integer_sequence)" should be removed.

Change History (4)

comment:1 Changed 3 years ago by Mikhail Strelnikov <mikhail.strelnikov@…>

Severity: ProblemCosmetic
Version: Boost 1.57.0Boost 1.59.0

comment:2 Changed 3 years ago by Mikhail Strelnikov <mikhail.strelnikov@…>

Component: Nonecoroutine
Owner: set to olli

comment:3 Changed 3 years ago by Mikhail Strelnikov <mikhail.strelnikov@…>

And, btw, this check won't work for clang+libc++ because of this bug

comment:4 Changed 3 years ago by olli

Component: coroutinecontext
Resolution: fixed
Status: newclosed

fixed - in a little bit other way

Note: See TracTickets for help on using tickets.