On Mon, May 18, 2015 at 05:18:59PM +0300, Alberto Garcia wrote:
> On Mon, May 18, 2015 at 12:33:13PM +0100, Mark Hindley wrote:
> 
> > This patch at least fixes the environment variable so that 
> > JavaScriptCoreUseJIT=0 is honoured as a work-araound.
> 
> If this is not working properly then it should be fixed upstream.

Yes

> You're using 2.4.x, right?

Yes, packages from current jessie

> I anyway wonder, what's the problem with the current code? Doesn't the
> part inside #if USE(CF) || OS(UNIX) handle this already?

I think the current code returns from the line
 
 return kCFBooleanTrue == canUseJIT.get();

ie it returns the application default. The env variable never gets checked.

My patch just reorders and allows JavaScriptCoreUseJIT to override the app
default, if set.

Mark


-- 
To UNSUBSCRIBE, email to debian-qt-kde-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20150518142809.ga4...@hindley.org.uk

Reply via email to