warning: cast to pointer from integer about c11threads HOT 6 CLOSED

Even the existing attempt to appease the compiler, with the double-cast, seems pointless and ugly. Adding even more cruft just isn't worth it. I'd say better pass that flag to disable the warning.

Leaving this bug report open, just in case I think about it some more and reconsider. But this is probably a won't-fix.

from c11threads.

Conversion between pointers and integers are implementation-defined

I should create a pull request.

from c11threads.

Actually, there is no compliant solution for that kind of code:

unsigned int *g(void) {
  unsigned int *ptr = 0xdeadbeef;
  /* ... */
  return ptr;
} 

I'm thinking about using _Thread_local to statically store (per-thread) that integer. The downside is that this is C11, although gnu99 will probably have that implemented long-time ago.

from c11threads.

Musl has a seemingly working solution:

#include <threads.h>
#include <pthread.h>
#include <stdint.h>

_Noreturn void thrd_exit(int result)
{
	__pthread_exit((void*)(intptr_t)result);
}

#include <stdint.h>
#include <threads.h>
#include <pthread.h>

int thrd_join(thrd_t t, int *res)
{
        void *pthread_res;
        __pthread_join(t, &pthread_res);
        if (res) *res = (int)(intptr_t)pthread_res;
        return thrd_success;
}

The plus side is that intptr_t is C99.

from c11threads.

Yes conversions are implementation defined, but every implementation defines them as "if the bits you put into it fit, you'll get the same bits out of it".

Going through intptr_t is the correct choice to silence the compiler.

from c11threads.

Thanks a lot!

from c11threads.