Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1022)

Unified Diff: src/jsregexp.h

Issue 9638: More automaton translation (Closed)
Patch Set: Created 12 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/jsregexp.h
diff --git a/src/jsregexp.h b/src/jsregexp.h
index 331a84d894d2f41400cc946f11179ebbd210df61..cf358beb16e0179c3015bf8704d2dfefb83facbf 100644
--- a/src/jsregexp.h
+++ b/src/jsregexp.h
@@ -136,9 +136,6 @@ class RegExpImpl {
};
-template <typename Char> class RegExpNode;
-
-
class CharacterRange {
public:
// For compatibility with the CHECK_OK macro
@@ -318,13 +315,18 @@ class DispatchTable {
};
+struct RegExpParseResult {
+ RegExpTree* tree;
+ bool has_character_escapes;
+ Handle<String> error;
+ int capture_count;
+};
Lasse Reichstein 2008/11/06 16:46:44 What!?! No third out-parameter? ;) Good call.
+
+
class RegExpEngine: public AllStatic {
public:
- template <typename Char>
- static RegExpNode<Char>* Compile(RegExpTree* regexp);
-
- template <typename Char>
- static bool Execute(RegExpNode<Char>* start, Vector<Char> input);
+ static RegExpNode* Compile(RegExpParseResult* input);
+ static void DotPrint(const char* label, RegExpNode* node);
};

Powered by Google App Engine
This is Rietveld 408576698