Chromium Code Reviews Chromium Code Reviews
Issue 1411243003:
Upgrade some tests to use annotations instead of 'clever' confuse() function (Closed)
Base URL: https://github.com/dart-lang/sdk.git@master| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file | 1 // Copyright (c) 2015, 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 // Regression test for http://dartbug.com/23404/ | 5 // Regression test for http://dartbug.com/23404/ |
| 6 import 'package:expect/expect.dart'; | 6 import 'package:expect/expect.dart'; |
| 7 | 7 |
| 8 // Dart2js crashed when the global metadata had escaped characters. That | 8 // Dart2js crashed when the global metadata had escaped characters. That |
| 9 // happens, for example, when tearing off a function that uses a default | 9 // happens, for example, when tearing off a function that uses a default |
| 10 // argument containing escape characters. | 10 // argument containing escape characters. |
| 11 foo([a='\u00a0']) => a; | 11 foo([a='\u00a0']) => a; |
| 12 bar() => ''; | 12 bar() => ''; |
| 13 | 13 |
| 14 @NoInline | 14 @NoInline() |
|
sra1
2015/10/20 17:04:37
We don't get a warning for this problem since a ty
Siggi Cherem (dart-lang)
2015/10/20 19:48:33
ouch, maybe we should handle in our compiler both
| |
| 15 @AssumeDynamic | 15 @AssumeDynamic() |
| 16 confuse(x) => x; | 16 confuse(x) => x; |
| 17 | 17 |
| 18 main() { | 18 main() { |
| 19 Expect.equals('\u00a0', confuse(foo)()); | 19 Expect.equals('\u00a0', confuse(foo)()); |
| 20 Expect.equals('', confuse(bar)()); | 20 Expect.equals('', confuse(bar)()); |
| 21 } | 21 } |
| OLD | NEW |
