In mingw32 w32api and runtime, the powf function is _CRT_INLINE, but in debug build -O0, compiler fails to inline it and then there is problem of linking. The following patch makes the use of the proper overloade C++ pow (which is actually using compiler builtins with mingw). It makes the code even a bit more C++ like and does not break anything.
Created attachment 30355 [details] use a proper C++ overloaded pow instead of C-style powf
Comment on attachment 30355 [details] use a proper C++ overloaded pow instead of C-style powf Needs ChangeLog. Needs testing on all the major platforms that WebKit supports. Generally seems strange to do this here when the rest of the project takes the other approach.
Created attachment 30368 [details] Clean proper standard-compliant patch with a proper clean ChangeLog entry
I also submitted also a bug report against the mingw toolchain https://sourceforge.net/tracker/?func=detail&aid=2792082&group_id=2435&atid=102435
Created attachment 30375 [details] This patch even has the proper e-mail address and spaces instead of tabs
Comment on attachment 30375 [details] This patch even has the proper e-mail address and spaces instead of tabs Looks good.
(In reply to comment #6) > (From update of attachment 30375 [details] [review]) > Looks good. > Committed as r44035.
After various attempts to make this compile and work in Mac, Windows (MSVC) and Linux we ended up reverting the patch in r44039. Reopening the bug.
Comment on attachment 30375 [details] This patch even has the proper e-mail address and spaces instead of tabs Marking r- since this was rolled out. (Thus removing it from the commit queue)
I did not see this problem anymore since I changed the runtime implementation I build against