Bug 47816

Summary: Remove some unnecessary lines of code from Parser.cpp
Product: WebKit Reporter: Peter Varga <pvarga>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, commit-queue, ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
proposed patch none

Peter Varga
Reported 2010-10-18 05:50:55 PDT
Looks like the parser/Parser.cpp includes two unused headers in JavaScriptCore: - wtf/HashSet.h - wtf/Vector.h The explicit decleration of jscyyparse(void*) function should be removed as well because we don't use a generated parser any more.
Attachments
proposed patch (1.23 KB, patch)
2010-10-18 05:54 PDT, Peter Varga 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Peter Varga
Comment 1 2010-10-18 05:54:12 PDT
Created attachment 71027 [details] proposed patch
Andras Becsi
Comment 2 2010-10-18 05:56:56 PDT
Comment on attachment 71027 [details] proposed patch LGTM. Thanks, Peter.
Andreas Kling
Comment 3 2010-10-18 06:02:47 PDT
Comment on attachment 71027 [details] proposed patch LGTM, too.
WebKit Commit Bot
Comment 4 2010-10-18 06:20:42 PDT
Comment on attachment 71027 [details] proposed patch Clearing flags on attachment: 71027 Committed r69957: <http://trac.webkit.org/changeset/69957>
WebKit Commit Bot
Comment 5 2010-10-18 06:20:47 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.