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

Unified Diff: compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java

Issue 10855140: Issue 3562. Analyzer should not warn about shadowing (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 4 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
Index: compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
diff --git a/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java b/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
index 6bfad9c40bbebbdb1becd3cef36f907b5e663d07..390c60a8ad1453441812b9f7f5a4a290f6641377 100644
--- a/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
+++ b/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
@@ -1192,7 +1192,6 @@ public class ResolverTest extends ResolverTestCase {
" new Foo<Param>();",
" }",
"}"),
- ResolverErrorCode.DUPLICATE_LOCAL_VARIABLE_WARNING,
TypeErrorCode.NOT_A_TYPE);
}
@@ -1221,16 +1220,6 @@ public class ResolverTest extends ResolverTestCase {
errEx(TypeErrorCode.TYPE_VARIABLE_IN_STATIC_CONTEXT, 4, 18, 1));
}
- public void testTypeVariableShadowsClass() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "class T {}",
- "class A<T> {", // type var T shadows class T
- " static foo() { new T(); }", // should resolve to class T
- "}"),
- ResolverErrorCode.DUPLICATE_TYPE_VARIABLE_WARNING);
- }
-
public void testConstClass() {
resolveAndTest(Joiner.on("\n").join(
"class Object {}",

Powered by Google App Engine
This is Rietveld 408576698