643772 - dom/plugins can't compile on mingw due to implicit function pointer to void* casts

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Jacek Caban]

Attachment: fix v1.0

It's the same problem as bug 606846, discovered after enabling IPC

Comment 1

[Jacek Caban]

Attachment: fix v1.1

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 2

[Benjamin Smedberg]

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

Comment 3

[Jacek Caban]

Attachment: fix v2.0

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.

Comment 4

[Jacek Caban]

Thanks for the review.

http://hg.mozilla.org/mozilla-central/rev/24966d916555