The problem is that uintptr_t is defined in qcmstypes.h to PRUptrdiff. On x86 uintptr_t is usually defined to unsigned int, but PRUptrdiff is unsigned long, and thus there is a conflict. Also it seems like uintptr_t is expected to not defined at this point by system headers, but it's not guaranteed and it is already typedefed in my case. I've fixed it by checking for standard guards, that are __intptr_t_defined for GCC (including mingw and it's not a typo, it shouldn't be __uintptr_t_defined) and _UINTPTR_T_DEFINED (the guard for msvc). The attached patch fixes compilation for me as well as makes code less system headers sensitive.

Comment on attachment 473539 [details] [diff] [review] fix v1.0 We these guards in place it seems like we'd be able to remove the defined(__FreeBSD__). Why don't you take that part out and if it breaks we can always add it back.

Thanks for the review. I've attached a patch with __FreeBSD__ check removed.

Comment on attachment 473654 [details] [diff] [review] fix v1.1 This can go in post-b7.