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

Unified Diff: pkg/analyzer/lib/src/generated/html.dart

Issue 667463004: Code clean-up (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 2 months 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 | « pkg/analyzer/lib/src/generated/engine.dart ('k') | pkg/analyzer/lib/src/generated/java_core.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/lib/src/generated/html.dart
diff --git a/pkg/analyzer/lib/src/generated/html.dart b/pkg/analyzer/lib/src/generated/html.dart
index 3984124ea95d6f7787cb817f141f56b526386669..334dfd5d5a990ad2cb853bace92982988e19b5e6 100644
--- a/pkg/analyzer/lib/src/generated/html.dart
+++ b/pkg/analyzer/lib/src/generated/html.dart
@@ -341,17 +341,17 @@ abstract class AbstractScanner {
}
class ExpressionVisitor_HtmlUnitUtils_getExpression extends ExpressionVisitor {
- int offset = 0;
+ final int offset;
- List<Expression> result;
+ Expression result;
- ExpressionVisitor_HtmlUnitUtils_getExpression(this.offset, this.result) : super();
+ ExpressionVisitor_HtmlUnitUtils_getExpression(this.offset);
@override
void visitExpression(Expression expression) {
Expression at = HtmlUnitUtils._getExpressionAt(expression, offset);
if (at != null) {
- result[0] = at;
+ result = at;
throw new HtmlUnitUtils_FoundExpressionError();
}
}
@@ -383,7 +383,7 @@ class HtmlParser extends XmlParser {
/**
* A set containing the names of tags that do not have a closing tag.
*/
- static Set<String> SELF_CLOSING = new HashSet<String>.from(JavaArrays.asList(<String> [
+ static Set<String> SELF_CLOSING = new HashSet<String>.from(<String>[
"area",
"base",
"basefont",
@@ -396,7 +396,7 @@ class HtmlParser extends XmlParser {
"link",
"meta",
"param",
- "!"]));
+ "!"]);
/**
* Given the contents of an embedded expression that occurs at the given offset, parse it as a
@@ -628,11 +628,12 @@ class HtmlUnitUtils {
if (htmlUnit == null) {
return null;
}
- List<XmlAttributeNode> result = [null];
+ RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode visitor
+ = new RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode(offset);
try {
- htmlUnit.accept(new RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode(offset, result));
+ htmlUnit.accept(visitor);
} on HtmlUnitUtils_FoundAttributeNodeError catch (e) {
- return result[0];
+ return visitor.result;
}
return null;
}
@@ -678,11 +679,12 @@ class HtmlUnitUtils {
if (htmlUnit == null) {
return null;
}
- List<XmlTagNode> result = [null];
+ RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode visitor
+ = new RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode(offset);
try {
- htmlUnit.accept(new RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode(offset, result));
+ htmlUnit.accept(visitor);
} on HtmlUnitUtils_FoundTagNodeError catch (e) {
- return result[0];
+ return visitor.result;
}
return null;
}
@@ -695,12 +697,13 @@ class HtmlUnitUtils {
if (htmlUnit == null) {
return null;
}
- List<Expression> result = [null];
+ ExpressionVisitor_HtmlUnitUtils_getExpression visitor
+ = new ExpressionVisitor_HtmlUnitUtils_getExpression(offset);
try {
// TODO(scheglov) this code is very Angular specific
- htmlUnit.accept(new ExpressionVisitor_HtmlUnitUtils_getExpression(offset, result));
+ htmlUnit.accept(visitor);
} on HtmlUnitUtils_FoundExpressionError catch (e) {
- return result[0];
+ return visitor.result;
}
return null;
}
@@ -819,17 +822,17 @@ class RecursiveXmlVisitor<R> implements XmlVisitor<R> {
}
class RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode extends RecursiveXmlVisitor<Object> {
- int offset = 0;
+ final int offset;
- List<XmlAttributeNode> result;
+ XmlAttributeNode result;
- RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode(this.offset, this.result) : super();
+ RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode(this.offset);
@override
Object visitXmlAttributeNode(XmlAttributeNode node) {
Token nameToken = node.nameToken;
if (nameToken.offset <= offset && offset <= nameToken.end) {
- result[0] = node;
+ result = node;
throw new HtmlUnitUtils_FoundAttributeNodeError();
}
return super.visitXmlAttributeNode(node);
@@ -837,16 +840,16 @@ class RecursiveXmlVisitor_HtmlUnitUtils_getAttributeNode extends RecursiveXmlVis
}
class RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode extends RecursiveXmlVisitor<Object> {
- int offset = 0;
+ final int offset;
- List<XmlTagNode> result;
+ XmlTagNode result;
- RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode(this.offset, this.result) : super();
+ RecursiveXmlVisitor_HtmlUnitUtils_getEnclosingTagNode(this.offset);
@override
Object visitXmlTagNode(XmlTagNode node) {
if (node.offset <= offset && offset < node.end) {
- result[0] = node;
+ result = node;
super.visitXmlTagNode(node);
throw new HtmlUnitUtils_FoundTagNodeError();
}
« no previous file with comments | « pkg/analyzer/lib/src/generated/engine.dart ('k') | pkg/analyzer/lib/src/generated/java_core.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698