Proper handling of built-in identifiers

tests/language/src/NamingTest.dart

Index: tests/language/src/NamingTest.dart
===================================================================
--- tests/language/src/NamingTest.dart	(revision 3272)
+++ tests/language/src/NamingTest.dart	(working copy)
@@ -182,10 +182,6 @@
     return () { return x + 1; };
   }

[siva, 2012/01/13 18:48:20]

I guess you removed this because it is a duplicate definition, Shouldn't we move
it into a negative test?

[hausner, 2012/01/13 19:22:47]

Yes, the test was just wrong.

 
-  negate(x) {
-    return () { return x + 2; };
-  }
-
   operator[] (x) {
     return () { return x + 3; };
   }
@@ -195,7 +191,7 @@
     Expect.equals(1, (h.f_)());
     var f = -h;
     Expect.equals(4, f());
-    Expect.equals(6, h.negate(4)());
+    Expect.equals(4, h.negate()());
     Expect.equals(7, h[4]());
   }
 }
@@ -518,21 +514,8 @@
   }

[siva, 2012/01/13 18:48:20]

Ditto comment.

[hausner, 2012/01/13 19:22:47]

Ditto answer.

 }
 
-// Ensure we don't have false positivesj.
-class Naming3Test {
-  Naming3Test() { }
-  operator negate() { }
-  negate() => 777;
-
-  static void main(args) {
-    var a = new Naming3Test();
-    Expect.equals(777, a.negate());
-  }
-}
-
 main() {
   NamingTest.testMain();
   Naming1Test.main(null);
   Naming2Test.main(null);
-  Naming3Test.main(null);
 }