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

Unified Diff: runtime/vm/regexp_ast.cc

Issue 754383002: Revert "Integrate the Irregexp Regular Expression Engine." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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
« no previous file with comments | « runtime/vm/regexp_ast.h ('k') | runtime/vm/regexp_parser.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/regexp_ast.cc
diff --git a/runtime/vm/regexp_ast.cc b/runtime/vm/regexp_ast.cc
index 9296170dfd00efd6bd5b8c60ce0c4c188d997fee..c36b890e86e358606851f81d601de255d28ee0d7 100644
--- a/runtime/vm/regexp_ast.cc
+++ b/runtime/vm/regexp_ast.cc
@@ -4,8 +4,7 @@
#include "vm/regexp_ast.h"
-#include "platform/utils.h"
-#include "vm/os.h"
+// SNIP
namespace dart {
@@ -20,7 +19,7 @@ FOR_EACH_REG_EXP_TREE_TYPE(MAKE_ACCEPT)
RegExp##Name* RegExpTree::As##Name() { \
return NULL; \
} \
- bool RegExpTree::Is##Name() const { return false; }
+ bool RegExpTree::Is##Name() { return false; }
FOR_EACH_REG_EXP_TREE_TYPE(MAKE_TYPE_CASE)
#undef MAKE_TYPE_CASE
@@ -28,59 +27,59 @@ FOR_EACH_REG_EXP_TREE_TYPE(MAKE_TYPE_CASE)
RegExp##Name* RegExp##Name::As##Name() { \
return this; \
} \
- bool RegExp##Name::Is##Name() const { return true; }
+ bool RegExp##Name::Is##Name() { return true; }
FOR_EACH_REG_EXP_TREE_TYPE(MAKE_TYPE_CASE)
#undef MAKE_TYPE_CASE
-static Interval ListCaptureRegisters(ZoneGrowableArray<RegExpTree*>* children) {
+static Interval ListCaptureRegisters(ZoneList<RegExpTree*>* children) {
Interval result = Interval::Empty();
- for (intptr_t i = 0; i < children->length(); i++)
- result = result.Union(children->At(i)->CaptureRegisters());
+ for (int i = 0; i < children->length(); i++)
+ result = result.Union(children->at(i)->CaptureRegisters());
return result;
}
-Interval RegExpAlternative::CaptureRegisters() const {
+Interval RegExpAlternative::CaptureRegisters() {
return ListCaptureRegisters(nodes());
}
-Interval RegExpDisjunction::CaptureRegisters() const {
+Interval RegExpDisjunction::CaptureRegisters() {
return ListCaptureRegisters(alternatives());
}
-Interval RegExpLookahead::CaptureRegisters() const {
+Interval RegExpLookahead::CaptureRegisters() {
return body()->CaptureRegisters();
}
-Interval RegExpCapture::CaptureRegisters() const {
+Interval RegExpCapture::CaptureRegisters() {
Interval self(StartRegister(index()), EndRegister(index()));
return self.Union(body()->CaptureRegisters());
}
-Interval RegExpQuantifier::CaptureRegisters() const {
+Interval RegExpQuantifier::CaptureRegisters() {
return body()->CaptureRegisters();
}
-bool RegExpAssertion::IsAnchoredAtStart() const {
+bool RegExpAssertion::IsAnchoredAtStart() {
return assertion_type() == RegExpAssertion::START_OF_INPUT;
}
-bool RegExpAssertion::IsAnchoredAtEnd() const {
+bool RegExpAssertion::IsAnchoredAtEnd() {
return assertion_type() == RegExpAssertion::END_OF_INPUT;
}
-bool RegExpAlternative::IsAnchoredAtStart() const {
- ZoneGrowableArray<RegExpTree*>* nodes = this->nodes();
- for (intptr_t i = 0; i < nodes->length(); i++) {
- RegExpTree* node = nodes->At(i);
+bool RegExpAlternative::IsAnchoredAtStart() {
+ ZoneList<RegExpTree*>* nodes = this->nodes();
+ for (int i = 0; i < nodes->length(); i++) {
+ RegExpTree* node = nodes->at(i);
if (node->IsAnchoredAtStart()) { return true; }
if (node->max_match() > 0) { return false; }
}
@@ -88,10 +87,10 @@ bool RegExpAlternative::IsAnchoredAtStart() const {
}
-bool RegExpAlternative::IsAnchoredAtEnd() const {
- ZoneGrowableArray<RegExpTree*>* nodes = this->nodes();
- for (intptr_t i = nodes->length() - 1; i >= 0; i--) {
- RegExpTree* node = nodes->At(i);
+bool RegExpAlternative::IsAnchoredAtEnd() {
+ ZoneList<RegExpTree*>* nodes = this->nodes();
+ for (int i = nodes->length() - 1; i >= 0; i--) {
+ RegExpTree* node = nodes->at(i);
if (node->IsAnchoredAtEnd()) { return true; }
if (node->max_match() > 0) { return false; }
}
@@ -99,37 +98,37 @@ bool RegExpAlternative::IsAnchoredAtEnd() const {
}
-bool RegExpDisjunction::IsAnchoredAtStart() const {
- ZoneGrowableArray<RegExpTree*>* alternatives = this->alternatives();
- for (intptr_t i = 0; i < alternatives->length(); i++) {
- if (!alternatives->At(i)->IsAnchoredAtStart())
+bool RegExpDisjunction::IsAnchoredAtStart() {
+ ZoneList<RegExpTree*>* alternatives = this->alternatives();
+ for (int i = 0; i < alternatives->length(); i++) {
+ if (!alternatives->at(i)->IsAnchoredAtStart())
return false;
}
return true;
}
-bool RegExpDisjunction::IsAnchoredAtEnd() const {
- ZoneGrowableArray<RegExpTree*>* alternatives = this->alternatives();
- for (intptr_t i = 0; i < alternatives->length(); i++) {
- if (!alternatives->At(i)->IsAnchoredAtEnd())
+bool RegExpDisjunction::IsAnchoredAtEnd() {
+ ZoneList<RegExpTree*>* alternatives = this->alternatives();
+ for (int i = 0; i < alternatives->length(); i++) {
+ if (!alternatives->at(i)->IsAnchoredAtEnd())
return false;
}
return true;
}
-bool RegExpLookahead::IsAnchoredAtStart() const {
+bool RegExpLookahead::IsAnchoredAtStart() {
return is_positive() && body()->IsAnchoredAtStart();
}
-bool RegExpCapture::IsAnchoredAtStart() const {
+bool RegExpCapture::IsAnchoredAtStart() {
return body()->IsAnchoredAtStart();
}
-bool RegExpCapture::IsAnchoredAtEnd() const {
+bool RegExpCapture::IsAnchoredAtEnd() {
return body()->IsAnchoredAtEnd();
}
@@ -139,43 +138,46 @@ bool RegExpCapture::IsAnchoredAtEnd() const {
// in as many cases as possible, to make it more difficult for incorrect
// parses to look as correct ones which is likely if the input and
// output formats are alike.
-class RegExpUnparser : public RegExpVisitor {
+class RegExpUnparser FINAL : public RegExpVisitor {
public:
+ RegExpUnparser(OStream& os, Zone* zone) : os_(os), zone_(zone) {}
void VisitCharacterRange(CharacterRange that);
#define MAKE_CASE(Name) virtual void* Visit##Name(RegExp##Name*, \
- void* data);
+ void* data) OVERRIDE;
FOR_EACH_REG_EXP_TREE_TYPE(MAKE_CASE)
#undef MAKE_CASE
+ private:
+ OStream& os_;
+ Zone* zone_;
};
void* RegExpUnparser::VisitDisjunction(RegExpDisjunction* that, void* data) {
- OS::Print("(|");
- for (intptr_t i = 0; i < that->alternatives()->length(); i++) {
- OS::Print(" ");
- (*that->alternatives())[i]->Accept(this, data);
+ os_ << "(|";
+ for (int i = 0; i < that->alternatives()->length(); i++) {
+ os_ << " ";
+ that->alternatives()->at(i)->Accept(this, data);
}
- OS::Print(")");
+ os_ << ")";
return NULL;
}
void* RegExpUnparser::VisitAlternative(RegExpAlternative* that, void* data) {
- OS::Print("(:");
- for (intptr_t i = 0; i < that->nodes()->length(); i++) {
- OS::Print(" ");
- (*that->nodes())[i]->Accept(this, data);
+ os_ << "(:";
+ for (int i = 0; i < that->nodes()->length(); i++) {
+ os_ << " ";
+ that->nodes()->at(i)->Accept(this, data);
}
- OS::Print(")");
+ os_ << ")";
return NULL;
}
void RegExpUnparser::VisitCharacterRange(CharacterRange that) {
- PrintUtf16(that.from());
+ os_ << AsUC16(that.from());
if (!that.IsSingleton()) {
- OS::Print("-");
- PrintUtf16(that.to());
+ os_ << "-" << AsUC16(that.to());
}
}
@@ -183,13 +185,13 @@ void RegExpUnparser::VisitCharacterRange(CharacterRange that) {
void* RegExpUnparser::VisitCharacterClass(RegExpCharacterClass* that,
void* data) {
- if (that->is_negated()) OS::Print("^");
- OS::Print("[");
- for (intptr_t i = 0; i < that->ranges()->length(); i++) {
- if (i > 0) OS::Print(" ");
- VisitCharacterRange((*that->ranges())[i]);
+ if (that->is_negated()) os_ << "^";
+ os_ << "[";
+ for (int i = 0; i < that->ranges(zone_)->length(); i++) {
+ if (i > 0) os_ << " ";
+ VisitCharacterRange(that->ranges(zone_)->at(i));
}
- OS::Print("]");
+ os_ << "]";
return NULL;
}
@@ -197,22 +199,22 @@ void* RegExpUnparser::VisitCharacterClass(RegExpCharacterClass* that,
void* RegExpUnparser::VisitAssertion(RegExpAssertion* that, void* data) {
switch (that->assertion_type()) {
case RegExpAssertion::START_OF_INPUT:
- OS::Print("@^i");
+ os_ << "@^i";
break;
case RegExpAssertion::END_OF_INPUT:
- OS::Print("@$i");
+ os_ << "@$i";
break;
case RegExpAssertion::START_OF_LINE:
- OS::Print("@^l");
+ os_ << "@^l";
break;
case RegExpAssertion::END_OF_LINE:
- OS::Print("@$l");
+ os_ << "@$l";
break;
case RegExpAssertion::BOUNDARY:
- OS::Print("@b");
+ os_ << "@b";
break;
case RegExpAssertion::NON_BOUNDARY:
- OS::Print("@B");
+ os_ << "@B";
break;
}
return NULL;
@@ -220,96 +222,96 @@ void* RegExpUnparser::VisitAssertion(RegExpAssertion* that, void* data) {
void* RegExpUnparser::VisitAtom(RegExpAtom* that, void* data) {
- OS::Print("'");
- ZoneGrowableArray<uint16_t>* chardata = that->data();
- for (intptr_t i = 0; i < chardata->length(); i++) {
- PrintUtf16(chardata->At(i));
+ os_ << "'";
+ Vector<const uc16> chardata = that->data();
+ for (int i = 0; i < chardata.length(); i++) {
+ os_ << AsUC16(chardata[i]);
}
- OS::Print("'");
+ os_ << "'";
return NULL;
}
void* RegExpUnparser::VisitText(RegExpText* that, void* data) {
if (that->elements()->length() == 1) {
- (*that->elements())[0].tree()->Accept(this, data);
+ that->elements()->at(0).tree()->Accept(this, data);
} else {
- OS::Print("(!");
- for (intptr_t i = 0; i < that->elements()->length(); i++) {
- OS::Print(" ");
- (*that->elements())[i].tree()->Accept(this, data);
+ os_ << "(!";
+ for (int i = 0; i < that->elements()->length(); i++) {
+ os_ << " ";
+ that->elements()->at(i).tree()->Accept(this, data);
}
- OS::Print(")");
+ os_ << ")";
}
return NULL;
}
void* RegExpUnparser::VisitQuantifier(RegExpQuantifier* that, void* data) {
- OS::Print("(# %" Pd " ", that->min());
+ os_ << "(# " << that->min() << " ";
if (that->max() == RegExpTree::kInfinity) {
- OS::Print("- ");
+ os_ << "- ";
} else {
- OS::Print("%" Pd " ", that->max());
+ os_ << that->max() << " ";
}
- OS::Print(that->is_greedy() ? "g " : that->is_possessive() ? "p " : "n ");
+ os_ << (that->is_greedy() ? "g " : that->is_possessive() ? "p " : "n ");
that->body()->Accept(this, data);
- OS::Print(")");
+ os_ << ")";
return NULL;
}
void* RegExpUnparser::VisitCapture(RegExpCapture* that, void* data) {
- OS::Print("(^ ");
+ os_ << "(^ ";
that->body()->Accept(this, data);
- OS::Print(")");
+ os_ << ")";
return NULL;
}
void* RegExpUnparser::VisitLookahead(RegExpLookahead* that, void* data) {
- OS::Print("(-> %s", (that->is_positive() ? "+ " : "- "));
+ os_ << "(-> " << (that->is_positive() ? "+ " : "- ");
that->body()->Accept(this, data);
- OS::Print(")");
+ os_ << ")";
return NULL;
}
void* RegExpUnparser::VisitBackReference(RegExpBackReference* that,
- void*) {
- OS::Print("(<- %" Pd ")", that->index());
+ void* data) {
+ os_ << "(<- " << that->index() << ")";
return NULL;
}
-void* RegExpUnparser::VisitEmpty(RegExpEmpty*, void*) {
- OS::Print("%%");
+void* RegExpUnparser::VisitEmpty(RegExpEmpty* that, void* data) {
+ os_ << '%';
return NULL;
}
-void RegExpTree::Print() {
- RegExpUnparser unparser;
+OStream& RegExpTree::Print(OStream& os, Zone* zone) { // NOLINT
+ RegExpUnparser unparser(os, zone);
Accept(&unparser, NULL);
+ return os;
}
-RegExpDisjunction::RegExpDisjunction(
- ZoneGrowableArray<RegExpTree*>* alternatives)
+RegExpDisjunction::RegExpDisjunction(ZoneList<RegExpTree*>* alternatives)
: alternatives_(alternatives) {
- ASSERT(alternatives->length() > 1);
- RegExpTree* first_alternative = alternatives->At(0);
+ DCHECK(alternatives->length() > 1);
+ RegExpTree* first_alternative = alternatives->at(0);
min_match_ = first_alternative->min_match();
max_match_ = first_alternative->max_match();
- for (intptr_t i = 1; i < alternatives->length(); i++) {
- RegExpTree* alternative = alternatives->At(i);
- min_match_ = Utils::Minimum(min_match_, alternative->min_match());
- max_match_ = Utils::Maximum(max_match_, alternative->max_match());
+ for (int i = 1; i < alternatives->length(); i++) {
+ RegExpTree* alternative = alternatives->at(i);
+ min_match_ = Min(min_match_, alternative->min_match());
+ max_match_ = Max(max_match_, alternative->max_match());
}
}
-static intptr_t IncreaseBy(intptr_t previous, intptr_t increase) {
+static int IncreaseBy(int previous, int increase) {
if (RegExpTree::kInfinity - previous < increase) {
return RegExpTree::kInfinity;
} else {
@@ -317,16 +319,16 @@ static intptr_t IncreaseBy(intptr_t previous, intptr_t increase) {
}
}
-RegExpAlternative::RegExpAlternative(ZoneGrowableArray<RegExpTree*>* nodes)
+RegExpAlternative::RegExpAlternative(ZoneList<RegExpTree*>* nodes)
: nodes_(nodes) {
- ASSERT(nodes->length() > 1);
+ DCHECK(nodes->length() > 1);
min_match_ = 0;
max_match_ = 0;
- for (intptr_t i = 0; i < nodes->length(); i++) {
- RegExpTree* node = nodes->At(i);
- intptr_t node_min_match = node->min_match();
+ for (int i = 0; i < nodes->length(); i++) {
+ RegExpTree* node = nodes->at(i);
+ int node_min_match = node->min_match();
min_match_ = IncreaseBy(min_match_, node_min_match);
- intptr_t node_max_match = node->max_match();
+ int node_max_match = node->max_match();
max_match_ = IncreaseBy(max_match_, node_max_match);
}
}
« no previous file with comments | « runtime/vm/regexp_ast.h ('k') | runtime/vm/regexp_parser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698