Re: [boost] [thread] catch(...) removed

From: Pedro Lamarão (pedro.lamarao_at_[hidden])
Date: 2006-07-06 13:04:00

Peter Dimov escreveu:

> Index: thread.cpp
> ===================================================================
> RCS file: /cvsroot/boost/boost/libs/thread/src/thread.cpp,v
> retrieving revision 1.21
> diff -u -r1.21 thread.cpp
> --- thread.cpp  14 Jul 2005 15:46:50 -0000      1.21
> +++ thread.cpp  6 Jul 2006 13:39:48 -0000
> @@ -105,7 +105,7 @@
>          static OSStatus thread_proxy(void* param)
>  #endif
>      {
> -        try
> +//        try
>          {
>              thread_param* p = static_cast<thread_param*>(param);
>              boost::function0<void> threadfunc = p->m_threadfunc;
> @@ -115,12 +115,12 @@
>              on_thread_exit();
>  #endif
>          }
> -        catch (...)
> -        {
> -#if defined(BOOST_HAS_WINTHREADS)
> -            on_thread_exit();
> -#endif
> -        }
> +//        catch (...)
> +//        {
> +//#if defined(BOOST_HAS_WINTHREADS)
> +//            on_thread_exit();
> +//#endif
> +//        }
>  #if defined(BOOST_HAS_MPTASKS)
>          ::boost::detail::thread_cleanup();
>  #endif

Shouldn't there be some sentry object then, with that "on_thread_exit()" 
thing called by the destructor?

--
  Pedro Lamarão