Tested x86_64-pc-linux-gnu, committed to trunk.
Jonathan Wakely writes: > On 10/12/19 22:38 -0800, Thomas Rodgers wrote: >>User-agent: mu4e 1.3.4; emacs 26.2 >> * include/std/condition_variable >> (condition_variable_any::wait_on(_Lock&, stop_token, _Predicate): Rename >> to match current draft standard. >> (condition_variable_any::wait_on_until(_Lock&, stop_token, >> const chrono::time_point<>&, _Predicate): Likewise. >> (condition_variable_any::wait_on_for(_Lock&, stop_token, >> const chrono::duration<>&, _Predicate(: Likewise. > > The closing paren here is an opening one. OK for trunk with that > fixed. > > Optional tweaks ... > > Since the names wait_on, wait_on_until and wait_on_for are > unambiguous, you could just list them without parameters i.e. > > * include/std/condition_variable (condition_variable_any::wait_on) > (condition_variable_any::wait_on_until) > (condition_variable_any::wait_on_for): Rename to match current > draft standard. > >> * testsuite/30_threads/condition_variable_any/stop_token/wait_on.cc >> (main): > > We don't generally both to say which functions were modified in tests, > so the (main) isn't needed here, but is harmless. > >> Adjust tests to account for renamed methods.