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

Side by Side Diff: compiler/javatests/com/google/dart/compiler/end2end/inc/myother1.dart

Issue 8231031: Check for compile-time constants in DartCompiler (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Renamed CompileTimeConstTest to CTConst2Test Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5
5 class Other1 { 6 class Other1 {
ngeoffray 2011/10/14 09:26:56 ditto.
zundel 2011/10/14 10:04:21 I updated the comment, the old code violates the l
ngeoffray 2011/10/14 10:33:42 Oh, OK. So the final Function FN = ... was acciden
6 // The existence of this final triggers b/5078969.
7 static final Function FN = () { return 42; };
8 7
9 Other1() { } 8 // Static variables must be initialized with compile time constants.
9
10 static Function FN;
11
12 Other1() {
13
14 if (FN == null) {
15 FN = () { return 42; };
16 }
17 }
10 void newMethod() { } 18 void newMethod() { }
11 } 19 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698