Closed Bug 643772 Opened 10 years ago Closed 10 years ago
dom/plugins can't compile on mingw due to implicit function pointer to void* casts
It's the same problem as bug 606846, discovered after enabling IPC
I've updated this patch to fix also win64 implementation (instead of filing separated bug). The problem is that cast from pointer to integer that loses precision is an error on GCC. The fix is to cast to uintptr_t instead and use implicit cast on its result. It's a similar problem to bug 611986
Comment on attachment 521577 [details] [diff] [review] fix v1.1 I would be happier if this were reinterpret_cast<void*>(value). r=me with that change
Attachment #521577 - Flags: review?(benjamin) → review+
Thanks for review. reinterpret_cast is problematic, because GCC considers this case too illegal to allow it and reports an error. It's, however, possible to avoid even more casts if we change nsWindowsDllInterceptor.h to use intptr_t as an argument instead of void*. It's casted to intptr_t later anyway and on callers side, casts are still unavoidable, but reinterpret_cast works fine now.
Attachment #526480 - Flags: review?(benjamin)
Thanks for the review. http://hg.mozilla.org/mozilla-central/rev/24966d916555
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.