|
|
Simplify constructors translation, improve code style.
This CL stops generating and using "_impl" methods with constructor implementation.
This is actually required only locally, when there are other statements aside of redirecting constructor invocation.
So, this may be done later, during constructors optimizing in ConstructorSemanticProcessor.
For now we just transform placeholder of redirecting constructor invocation into the actual initializer.
Also, this CL adds more optimizations into ConstructorSemanticProcessor, such as:
1. removing empty constructor body;
2. enum constructor tweaks.
Plus new snapshot of analyzer_experimental with these changes in translation applied.
R=brianwilkerson@google.com
BUG=
Committed: https://code.google.com/p/dart/source/detail?r=24701
Total comments: 1
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+1116 lines, -1228 lines) |
Patch |
 |
M |
editor/util/plugins/com.google.dart.java2dart/resources/java_io.dart
|
View
|
|
1 chunk |
+5 lines, -1 line |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart/src/com/google/dart/java2dart/Context.java
|
View
|
|
9 chunks |
+35 lines, -44 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart/src/com/google/dart/java2dart/SyntaxTranslator.java
|
View
|
|
15 chunks |
+75 lines, -151 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart/src/com/google/dart/java2dart/engine/MainEngine.java
|
View
|
|
2 chunks |
+0 lines, -2 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart/src/com/google/dart/java2dart/processor/ConstructorSemanticProcessor.java
|
View
|
|
2 chunks |
+150 lines, -33 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart_test/src/com/google/dart/java2dart/SemanticTest.java
|
View
|
|
18 chunks |
+22 lines, -97 lines |
1 comment
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart_test/src/com/google/dart/java2dart/SyntaxTranslatorTest.java
|
View
|
|
3 chunks |
+4 lines, -14 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart_test/src/com/google/dart/java2dart/processor/ConstructorSemanticProcessorTest.java
|
View
|
|
2 chunks |
+39 lines, -16 lines |
0 comments
|
Download
|
 |
M |
editor/util/plugins/com.google.dart.java2dart_test/src/com/google/dart/java2dart/processor/ObjectSemanticProcessorTest.java
|
View
|
|
1 chunk |
+4 lines, -20 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/ast.dart
|
View
|
|
13 chunks |
+20 lines, -41 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/constant.dart
|
View
|
|
2 chunks |
+0 lines, -6 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/element.dart
|
View
|
|
54 chunks |
+55 lines, -187 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/engine.dart
|
View
|
|
56 chunks |
+238 lines, -208 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/error.dart
|
View
|
|
8 chunks |
+70 lines, -70 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/html.dart
|
View
|
|
3 chunks |
+6 lines, -15 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/instrumentation.dart
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/java_io.dart
|
View
|
|
1 chunk |
+5 lines, -1 line |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/parser.dart
|
View
|
|
3 chunks |
+15 lines, -13 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/resolver.dart
|
View
|
|
21 chunks |
+123 lines, -81 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/scanner.dart
|
View
|
|
4 chunks |
+10 lines, -34 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/source.dart
|
View
|
|
3 chunks |
+5 lines, -14 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/lib/src/generated/source_io.dart
|
View
|
|
4 chunks |
+9 lines, -25 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/test/generated/ast_test.dart
|
View
|
|
2 chunks |
+2 lines, -4 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/test/generated/parser_test.dart
|
View
|
|
5 chunks |
+23 lines, -1 line |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/test/generated/resolver_test.dart
|
View
|
|
8 chunks |
+179 lines, -130 lines |
0 comments
|
Download
|
 |
M |
pkg/analyzer_experimental/test/generated/test_support.dart
|
View
|
|
1 chunk |
+4 lines, -11 lines |
0 comments
|
Download
|
 |
M |
tests/standalone/standalone.status
|
View
|
|
1 chunk |
+17 lines, -7 lines |
0 comments
|
Download
|
Total messages: 3 (0 generated)
|