ACE_TAO icon indicating copy to clipboard operation
ACE_TAO copied to clipboard

Test and Potential Fix for Issue #363 Native Windows Condition Variables do not set errno to ETIME

Open mathersm opened this issue 7 years ago • 1 comments

  • Added a really simple test for the return value and errno combination expected when a wait on a condition variable times out. Project name is Native_Condition_Variable_Test.
  • Added new test onto the list for automatic testing via run_test.lst
  • Added an additional case to the ACE_FAIL_RETURN macro to mutate ERROR_TIMEOUT into ETIME.
  • Added an additional check at the end of the implementation of ACE_Condition<T>::wait(const ACE_Time_Value*) to mutate WIN32 errors into errno (as expected).

It might be worth considering revisiting the MSVC configuration files to enable ACE_HAS_WTHREADS_CONDITION_VARIABLE (with appropriate include statements) for Windows Vista/Server2008 and beyond.

mathersm avatar Feb 08 '17 11:02 mathersm

FYI, because ACE_HAS_WTHREADS_CONDITION_VARIABLE is not defined by any of the CI or regular scoreboard builds none will test the core changes and the support for this

jwillemsen avatar Feb 08 '17 11:02 jwillemsen