Normalization of patch signatures.

lib/compiler/implementation/lib/isolate_patch.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // Patch file for the dart:isolate library.
 
 #import("dart:uri");
 
 /**
  * Called by the compiler to support switching
@@ skipping 76 matching lines @@
 // the change to break the isolate abstraction.
 _Manager get _globalState() native "return \$globalState;";
 set _globalState(_Manager val) native "\$globalState = val;";
 
 void _fillStatics(context) native @"""
   $globals = context.isolateStatics;
   $static_init();
 """;
 
 ReceivePort _lazyPort;
-patch ReceivePort get port {
+patch ReceivePort get port() {

[Mads Ager (google), 2012/09/17 05:52:56]

Can this go the other way instead? That is, can we remove the '()' in
lib/isolate/ instead?

[Lasse Reichstein Nielsen, 2012/09/17 11:32:43]

I think we can, and should.

Remove ALL the getter parens!

[Johnni Winther, 2012/09/19 09:03:30]

I'll do that after the patch refactoring.

[ahe, 2012/09/19 09:39:13]

I'm sorry, I don't think that is the right order. By adding these parentheses,
you're regressing on the work Kasper has been doing to remove parentheses.

   if (_lazyPort === null) {
     _lazyPort = new ReceivePort();
   }
   return _lazyPort;
 }
 
 patch SendPort spawnFunction(void topLevelFunction()) {
   final name = _IsolateNatives._getJSFunctionName(topLevelFunction);
   if (name == null) {
     throw new UnsupportedOperationException(
@@ skipping 1153 matching lines @@
       _window.clearTimeout(_handle);
     } else {
       _window.clearInterval(_handle);
     }
   }
 }
 
 Timer _timerFactory(int millis, void callback(Timer timer), bool repeating) =>
   repeating ? new _Timer.repeating(millis, callback)
             : new _Timer(millis, callback);