Bug 36009 - Unused functions in yarr interpreter
Summary: Unused functions in yarr interpreter
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-11 04:13 PST by Peter Varga
Modified: 2010-03-12 07:28 PST (History)
4 users (show)

See Also:


Attachments
proposed patch (1.96 KB, patch)
2010-03-11 04:13 PST, Peter Varga
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Varga 2010-03-11 04:13:35 PST
Created attachment 50485 [details]
proposed patch

I have found two functions in RegexInterpreter.cpp: 
tryConsumeCharacter() and tryConsumeCharacterClass() which are never called.

If these functions are reserved for future using the tryConsumeCharacter 
is deprecated, because the ignoreCase checking is
unnecessary in this phase of regex matching.
Comment 1 WebKit Commit Bot 2010-03-12 07:28:32 PST
Comment on attachment 50485 [details]
proposed patch

Clearing flags on attachment: 50485

Committed r55900: <http://trac.webkit.org/changeset/55900>
Comment 2 WebKit Commit Bot 2010-03-12 07:28:37 PST
All reviewed patches have been landed.  Closing bug.