[GitHub] [qpid-dispatch] kgiusti commented on pull request #760: DISPATCH-1679: Add sys_thread_self()

[GitBox <gi...@apache.org>]

kgiusti commented on pull request #760:
URL: https://github.com/apache/qpid-dispatch/pull/760#issuecomment-642861207


   > N.B. I am responsible for the death of previous incarnation of `sys_thread_self` in [3db6223](https://github.com/apache/qpid-dispatch/commit/3db622312ecaf6b872af01c258f4687fdb5d9720). The problem with previous implementation was that it assumed `pthread_self()` returns (typedef'd alias of) `long`, which is not always true. It's not true on macOS. Nobody was using that function then, so it quietly died.
   
   I'm hoping the new sys_thread_self() call will work on macOS as it avoids exporting the pthread_t type.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org