[Bug libstdc++/104866] New: this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[Bug libstdc++/104866] New: this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[dv at vollmann dot ch]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

            Bug ID: 104866
           Summary: this_thread_sleep.h doesn't compile if
                    _GLIBCXX_NO_SLEEP is defined
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dv at vollmann dot ch
                CC: dv at vollmann dot ch
  Target Milestone: ---
              Host: x86_64-linux-gnu
            Target: avr
             Build: x86_64-linux-gnu

Created attachment 52596
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52596&action=edit
Patch to fix the problem

this_thread_sleep.h opens the namespace std::this_thread before checking for
the define but then closes it inside the #ifndef

[Bug libstdc++/104866] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2022-03-10
     Ever confirmed|0                           |1

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |11.2.1
   Target Milestone|---                         |12.0
      Known to fail|                            |12.0
           Priority|P3                          |P1
            Summary|this_thread_sleep.h doesn't |[12 Regression]
                   |compile if                  |this_thread_sleep.h doesn't
                   |_GLIBCXX_NO_SLEEP is        |compile if
                   |defined                     |_GLIBCXX_NO_SLEEP is
                   |                            |defined

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jonathan Wakely <redi@gcc.gnu.org>:

https://gcc.gnu.org/g:b5417a0ba7e26bec2abf05cad6c6ef840a9be41c

commit r12-7603-gb5417a0ba7e26bec2abf05cad6c6ef840a9be41c
Author: Detlef Vollmann <dv@vollmann.ch>
Date:   Thu Mar 10 16:57:25 2022 +0000

    libstdc++: Move closing brace outside #endif [PR104866]

    libstdc++-v3/ChangeLog:

            PR libstdc++/104866
            * include/bits/this_thread_sleep.h: Fix order of #endif and
            closing brace of namespace.

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Thanks!

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[dv at vollmann dot ch]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

--- Comment #3 from dv at vollmann dot ch ---
Thanks.
Tested for AVR, works :-)

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Jonathan Wakely
<redi@gcc.gnu.org>:

https://gcc.gnu.org/g:502fd5575bfe0793ef4dc90dd714755e9878f308

commit r11-10497-g502fd5575bfe0793ef4dc90dd714755e9878f308
Author: Detlef Vollmann <dv@vollmann.ch>
Date:   Thu Mar 10 16:57:25 2022 +0000

    libstdc++: Move closing brace outside #endif [PR104866]

    libstdc++-v3/ChangeLog:

            PR libstdc++/104866
            * include/bits/this_thread_sleep.h: Fix order of #endif and
            closing brace of namespace.

    (cherry picked from commit b5417a0ba7e26bec2abf05cad6c6ef840a9be41c)

[Bug libstdc++/104866] [12 Regression] this_thread_sleep.h doesn't compile if _GLIBCXX_NO_SLEEP is defined

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104866

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |11.1.0
      Known to work|11.2.1                      |10.4.0
   Target Milestone|12.0                        |11.4

--- Comment #5 from Jonathan Wakely <redi at gcc dot gnu.org> ---
I incorrectly marked this as Known-to-work: 11.2.1 but the bug was present
since 11.1.0

It's fixed for 11.4.0 now.