diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2017-04-20 13:54:01 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2017-04-20 13:54:01 +0200 |
| commit | 5a155bb75b62f02c8ed16cdafd492b625ce208c0 (patch) | |
| tree | 774b8a229ee2d3880d6d01095fe33e7dedb43bfd /sal/osl/unx/thread.cxx | |
| parent | b2764a9b66d1bab85256e5f9c3577b994448ce4a (diff) | |
Simplify PTHREAD_VALUE
The distinction between MACOSX and other had been introduced with
ed8110e101db5a8ea4dde43a3ec38fe40e6fe7e2 "Avoid reinterpret_cast to same type"
to work around a GCC 4.6 bug that is fixed since GCC 4.7.
Change-Id: I7948721d311de2622c1e08e2ee17b5ccc304b51b
Diffstat (limited to 'sal/osl/unx/thread.cxx')
| -rw-r--r-- | sal/osl/unx/thread.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sal/osl/unx/thread.cxx b/sal/osl/unx/thread.cxx index 34bf38cde607..e829b91781c6 100644 --- a/sal/osl/unx/thread.cxx +++ b/sal/osl/unx/thread.cxx @@ -56,7 +56,8 @@ * (2) 'oslThreadIdentifier' and '{insert|remove|lookup}ThreadId()' * - cannot reliably be applied to 'alien' threads; * - memory leak for 'alien' thread 'HashEntry's; - * - use 'PTHREAD_VALUE(pthread_t)' as identifier instead (?) + * - use 'reinterpret_cast<unsigned long>(pthread_t)' as identifier + * instead (?) * - if yes, change 'oslThreadIdentifier' to 'intptr_t' or similar * (3) 'oslSigAlarmHandler()' (#71232#) * - [Under Solaris we get SIGALRM in e.g. pthread_join which terminates @@ -552,7 +553,7 @@ void SAL_CALL osl_setThreadName(char const * name) { /* osl_getThreadIdentifier @@@ see TODO @@@ */ /*****************************************************************************/ -#define HASHID(x) (PTHREAD_VALUE(x) % HashSize) +#define HASHID(x) (reinterpret_cast<unsigned long>(x) % HashSize) struct HashEntry { |