[webkit-dev] [webkit-changes] [38187] trunk/JavaScriptCore

[George Staikos]

Shouldn't that be PLATFORM(LINUX) instead of PLATFORM(GTK)?  Seems  
that PLATFORM(GTK) could be wrong in some cases, and not having, for  
example PLATFORM(QT), could be also wrong.

On 7-Nov-08, at 3:47 AM, alp at webkit.org wrote:
> --- trunk/JavaScriptCore/wtf/TCSystemAlloc.cpp	2008-11-06 18:42:24  
> UTC (rev 38186)
> +++ trunk/JavaScriptCore/wtf/TCSystemAlloc.cpp	2008-11-06 18:47:44  
> UTC (rev 38187)
> @@ -51,6 +51,14 @@
>  #include "TCSpinLock.h"
>  #include "UnusedParam.h"
>
> +#if HAVE(MMAP)
> +static const int cProtFlags = PROT_READ | PROT_WRITE
> +#if ENABLE(CTI) && PLATFORM(GTK)
> +                              | PROT_EXEC
> +#endif
> +                              ;
> +#endif

--
George Staikos
Torch Mobile Inc.
http://www.torchmobile.com/

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20081111/71203819/attachment.html>