#if FORCE_SYSTEM_MALLOC #include <stdlib.h> #if !OS(WINDOWS) #include <pthread.h> #else #include "windows.h" #endif Not sure if <stdlib.h> is needed, but <pthread.h> or "windows.h" is definitely unnecessary.
Created attachment 46150 [details] the patch
style-queue ran check-webkit-style on attachment 46150 [details] without any errors.
Comment on attachment 46150 [details] the patch OK. Which platforms use FORCE_SYSTEM_MALLOC at the moment?
Comment on attachment 46150 [details] the patch It's clear that the <stdlib.h> include is not needed because FastMalloc.h includes it. There is no use of pthread down here and there is an include of pthread.h above for the case where we need it. But how did you determine the windows.h include was not needed? I'll take your word for it.
(In reply to comment #4) > (From update of attachment 46150 [details]) > It's clear that the <stdlib.h> include is not needed because FastMalloc.h > includes it. There is no use of pthread down here and there is an include of > pthread.h above for the case where we need it. But how did you determine the > windows.h include was not needed? I'll take your word for it. I cannot see any #if OS(WINOS) or something similar inside #ifdef FORCE_SYSTEM_MALLOC block. Also, I successfully made FastMalloc build for win32 with forcing FORCE_SYSTEM_MALLOC=1 :) Will commit it soon and monitor the buildbots.
landed @ r53008 bug closed