Bug 49328

Summary:

remove remaining reference to ENABLE_JSC_MULTIPLE_THREADS

Product:

WebKit

Reporter:

Siddharth Mathur <s.mathur>

Component:

JavaScriptCore

Assignee:

Siddharth Mathur <s.mathur>

Status:

RESOLVED INVALID

Severity:

Trivial

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

All

Attachments:

Description	Flags
remove flag which doesn't appear to be needed anymore	ggaren: review-

Siddharth Mathur

Reported 2010-11-10 10:33:54 PST

ENABLE_JSC_MULTIPLE_THREADS isn't used in the code anymore except for its declaration in wtf\Platform.h. I noticed this while going through old Bug 26838.

Attachments
remove flag which doesn't appear to be needed anymore (1.05 KB, patch) 2010-11-10 10:57 PST, Siddharth Mathur	ggaren: review-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Siddharth Mathur

Comment 1 2010-11-10 10:57:13 PST

Created attachment 73508 [details] remove flag which doesn't appear to be needed anymore

Geoffrey Garen

Comment 2 2010-11-10 11:04:49 PST

Comment on attachment 73508 [details] remove flag which doesn't appear to be needed anymore According to Xcode's Find in Project feature, there are 35 occurrences of "ENABLE(JSC_MULTIPLE_THREADS)" in JavaScriptCore.

Siddharth Mathur

Comment 3 2010-11-10 11:09:47 PST

Duh, sorry. PEBKAC!

Note You need to log in before you can comment on or make changes to this bug.