Re: PR 53889: Add __gthread_recursive_mutex_destroy

2012-10-02 Thread Jonathan Wakely
On 2 October 2012 07:02, Ian Lance Taylor wrote: > > The libgcc part of this is OK. Thanks, I've committed it and closed the PR.

Re: PR 53889: Add __gthread_recursive_mutex_destroy

2012-10-02 Thread Jakub Jelinek
On Mon, Oct 01, 2012 at 11:02:27PM -0700, Ian Lance Taylor wrote: > On Mon, Oct 1, 2012 at 5:46 PM, Jonathan Wakely wrote: > > > > static inline int > > __gthread_recursive_mutex_destroy (__gthread_recursive_mutex_t > >* UNUSED(__mutex)) > > { > > return 0; >

Re: PR 53889: Add __gthread_recursive_mutex_destroy

2012-10-01 Thread Ian Lance Taylor
On Mon, Oct 1, 2012 at 5:46 PM, Jonathan Wakely wrote: > > static inline int > __gthread_recursive_mutex_destroy (__gthread_recursive_mutex_t >* UNUSED(__mutex)) > { > return 0; > } > > Is that indentation right? (the asterisk is in the same column as the > p

Re: PR 53889: Add __gthread_recursive_mutex_destroy

2012-10-01 Thread Jonathan Wakely
On 1 October 2012 20:22, Ian Lance Taylor wrote: > On Sun, Sep 30, 2012 at 11:41 AM, Jonathan Wakely > wrote: >> There is no __gthread_recursive_mutex_destroy function in the gthreads API. >> >> Trying to use __gthread_mutex_destroy fails to compile on platforms >> where the mutex >> types are di

Re: PR 53889: Add __gthread_recursive_mutex_destroy

2012-10-01 Thread Ian Lance Taylor
On Sun, Sep 30, 2012 at 11:41 AM, Jonathan Wakely wrote: > There is no __gthread_recursive_mutex_destroy function in the gthreads API. > > Trying to use __gthread_mutex_destroy fails to compile on platforms > where the mutex > types are different. To avoid resource leaks libstdc++ needs to hack >

PR 53889: Add __gthread_recursive_mutex_destroy

2012-09-30 Thread Jonathan Wakely
There is no __gthread_recursive_mutex_destroy function in the gthreads API. Trying to use __gthread_mutex_destroy fails to compile on platforms where the mutex types are different. To avoid resource leaks libstdc++ needs to hack around the missing function with overloaded functions and SFINAE tric