Opened 7 years ago
Closed 7 years ago
#11652 closed Bugs (fixed)
Checking __cpp_lib_integer_sequence
| Reported by: | Owned by: | olli | |
|---|---|---|---|
| Milestone: | To Be Determined | Component: | context |
| Version: | Boost 1.59.0 | Severity: | Cosmetic |
| Keywords: | Cc: |
Description
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 by , 7 years ago
| Severity: | Problem → Cosmetic |
|---|---|
| Version: | Boost 1.57.0 → Boost 1.59.0 |
comment:2 by , 7 years ago
| Component: | None → coroutine |
|---|---|
| Owner: | set to |
comment:3 by , 7 years ago
comment:4 by , 7 years ago
| Component: | coroutine → context |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
fixed - in a little bit other way
Note:
See TracTickets
for help on using tickets.
And, btw, this check won't work for clang+libc++ because of this bug https://llvm.org/bugs/show_bug.cgi?id=18584