Commit Graph

578 Commits

Author SHA1 Message Date
scheglov@google.com e5ffdfa735 'Inline Method' refactoring
Support for methods/functions.
Support for bodies with several statements.

R=messick@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10823372

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10866 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-17 00:48:51 +00:00
brianwilkerson@google.com a16f04fac2 Rename DartMetadata in preparation for supporting real metadata
Review URL: https://chromiumcodereview.appspot.com//10829362

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10862 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 20:47:18 +00:00
brianwilkerson@google.com a7e2599cfe Fix for issue 3550
Review URL: https://chromiumcodereview.appspot.com//10857041

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10856 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 20:06:29 +00:00
brianwilkerson@google.com 5eb5f42a24 Support new optional parameter syntax (issue 4289)
Review URL: https://chromiumcodereview.appspot.com//10837286

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10832 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 16:38:40 +00:00
scheglov@google.com a7fec28e05 Issue 4518. Use inferred types to check if type of argumetn is assignable to the type of parameter
http://code.google.com/p/dart/issues/detail?id=4518

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10837282

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10825 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 14:38:25 +00:00
scheglov@google.com 4b42faaa5f Issue 4394. Consider type arguments when build super types list
http://code.google.com/p/dart/issues/detail?id=4394

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10830349

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10815 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 13:32:17 +00:00
scheglov@google.com 4a498a6981 Issue 4523. Analyzer should require two type arguments for map literals
http://code.google.com/p/dart/issues/detail?id=4523

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10836281

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10814 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 13:30:55 +00:00
ager@google.com 37ae98110a Reapply "Sharing of sources for corelib date implementation between VM
and dart2js".

BUG=

Review URL: https://chromiumcodereview.appspot.com//10834353

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10799 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-16 08:00:02 +00:00
scheglov@google.com 121ef9fdaa Issue 4048. Support for revised library/import syntax
http://code.google.com/p/dart/issues/detail?id=4048

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10832321

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10775 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-15 22:39:39 +00:00
brianwilkerson@google.com 53b93f854b Support unary minus operator (issue 3767)
Review URL: https://chromiumcodereview.appspot.com//10834334

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10745 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-15 15:50:46 +00:00
scheglov@google.com 2d94944c8d Issue 4460. Mark inferred type variable in for-in loop as inferred
http://code.google.com/p/dart/issues/detail?id=4460

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10827339

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10691 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 23:32:29 +00:00
brianwilkerson@google.com ee09a19368 Finalize support for new getter syntax, per issue 3607
Review URL: https://chromiumcodereview.appspot.com//10836252

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10682 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 22:03:30 +00:00
brianwilkerson@google.com d29d114054 Fix bugs in parser and update tests to use new directive syntax. Note that the changes in TypeAnalyzer exist to keep the tests running and are probably something you need to fix.
Review URL: https://chromiumcodereview.appspot.com//10825346

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10678 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 20:47:42 +00:00
scheglov@google.com a872467e43 Issue 4383. Calling field without setter should generate problem
http://code.google.com/p/dart/issues/detail?id=4383

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10837237

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10649 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 14:25:22 +00:00
scheglov@google.com 5488dbbb0f Issue 4294. Const constructor cannot call methods
http://code.google.com/p/dart/issues/detail?id=4294

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10825335

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10647 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 14:01:56 +00:00
scheglov@google.com b056277534 Issue 3562. Analyzer should not warn about shadowing
http://code.google.com/p/dart/issues/detail?id=3562

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10855140

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10646 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-14 14:00:00 +00:00
brianwilkerson@google.com 7692794a4a Initial parser changes for the new directive syntax
Review URL: https://chromiumcodereview.appspot.com//10832274

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10599 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-13 20:01:43 +00:00
scheglov@google.com b97168a038 Issue 4410. Infer types using 'assert' statement
http://code.google.com/p/dart/issues/detail?id=4410

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10829292

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10579 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-13 14:56:40 +00:00
scheglov@google.com d10c9e8a2c Issue 4347. Report error if ++/-- is applied to final variable
http://code.google.com/p/dart/issues/detail?id=4374

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10854106

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10578 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-13 14:55:08 +00:00
scheglov@google.com 20b1510e1d Fix for typo
R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10832265

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10577 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-13 14:46:49 +00:00
scheglov@google.com 929f158657 Issue 3800. Calling an undefined constructor via new is a warning, not an error
http://code.google.com/p/dart/issues/detail?id=3800

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10829288

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10531 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-10 21:04:47 +00:00
brianwilkerson@google.com c395b13644 Fix for issue 4429 - remove #resource directive
Review URL: https://chromiumcodereview.appspot.com//10827286

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10530 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-10 21:03:36 +00:00
scheglov@google.com 89b6b69e9f Issue 3084. Generate warning, not error, for 'Unknown.unknown'
http://code.google.com/p/dart/issues/detail?id=3084

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10826249

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10524 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-10 18:33:07 +00:00
brianwilkerson@google.com 28b3a4b574 Fix for issue 3756 - support new catch syntax
Review URL: https://chromiumcodereview.appspot.com//10854097

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10521 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-10 18:18:09 +00:00
scheglov@google.com c3e678447d Issue 3078. Compiler error converted to warning in section 7.5.2: Factories
http://code.google.com/p/dart/issues/detail?id=3078

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10824256

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10520 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-10 18:10:39 +00:00
scheglov@google.com c6849c48ca Initial implementation of 'Create Method' quick fix
- only top-level functions are supported;
 - argument suggestions are not bound to the linked nodes, just first suggestion is used.

2. Tweak for resolved AST access

R=messick@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10834222

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10468 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-09 23:02:35 +00:00
scheglov@google.com cb9721599f Fix for inferred type warnings pref page
Finishes http://codereview.chromium.org/10829255/ fix.

R=devoncarew@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10837194

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10467 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-09 22:53:44 +00:00
scheglov@google.com 1977779c20 Issue 4400. Fix for using top-level field in instance field initializer
http://code.google.com/p/dart/issues/detail?id=4400

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10854030

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10353 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-07 20:23:12 +00:00
scheglov@google.com 09c4bbafe9 'Split && condition' quick assist.
And clean ups, simplifications for other places.

R=messick@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10828169

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10295 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-06 16:38:53 +00:00
scheglov@google.com c0e823e0f7 Don't infer non-final field types
We can not analyze universe and prove that field is not reassigned to have value of othe type.

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10825190

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10260 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-03 22:10:39 +00:00
devoncarew@google.com eec0cd7855 Fixed several compiler warnings.
Review URL: https://chromiumcodereview.appspot.com//10832144

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10256 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-03 22:01:40 +00:00
scheglov@google.com 48d5420f6c Issue 4354. Fix for NPE when switch/case has no expression
http://code.google.com/p/dart/issues/detail?id=4354

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10825185

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10243 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-03 19:36:52 +00:00
scheglov@google.com 0b01ba645c Issue 4347. Fix for 'as' type cast parsing and nodes positions
http://code.google.com/p/dart/issues/detail?id=4347

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10823164

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10241 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-03 18:56:52 +00:00
scheglov@google.com 099e23eb11 Issue 3847. Allow non-constant instance variable initializers
http://code.google.com/p/dart/issues/detail?id=3847

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10830145

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10233 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-03 14:41:00 +00:00
danrubel@google.com c450181f31 translate uris pointing to non-existing dart libraries
Review URL: https://chromiumcodereview.appspot.com//10843040

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10153 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-02 03:28:34 +00:00
scheglov@google.com cec0b6cf01 Issue 4315. Fix for type of cascade assignment
http://code.google.com/p/dart/issues/detail?id=4315

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10827127

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10150 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-02 00:00:21 +00:00
scheglov@google.com 9874031222 Issue 4300. Support for 'external' keyword
http://code.google.com/p/dart/issues/detail?id=4300

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10823127

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10144 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-01 22:09:49 +00:00
brianwilkerson@google.com 709a44a4f3 Fix for issue 4242
Review URL: https://chromiumcodereview.appspot.com//10834113

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10137 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-01 19:01:37 +00:00
scheglov@google.com a4bc575bf2 Issue 4238. Infer Element subclass from query() parameter
http://code.google.com/p/dart/issues/detail?id=4238

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10825135

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10128 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-01 18:13:12 +00:00
keertip@google.com 1380d64e60 set package root after creating configuration, since defualt compiler configuration is used by analysis server too.
Review URL: http://codereview.chromium.org//10821119

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10095 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-31 22:43:20 +00:00
scheglov@google.com 50d8814ed6 Issue 4262. Support for --version in dart_analyzer
http://code.google.com/p/dart/issues/detail?id=4262

Also fix for using "dartc" in usage text.
http://code.google.com/p/dart/issues/detail?id=4261

R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10825112

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10084 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-31 19:39:14 +00:00
keertip@google.com 68cae7ebba fix resolving import uri's for packages
Review URL: http://codereview.chromium.org//10823060

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10077 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-31 18:07:24 +00:00
brianwilkerson@google.com 8723c22ee4 Fix for issue 2439
Review URL: https://chromiumcodereview.appspot.com//10823090

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10058 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-31 14:47:32 +00:00
brianwilkerson@google.com f3dc7258c8 Fix for issue 4109
Review URL: https://chromiumcodereview.appspot.com//10823085

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10024 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-30 19:46:52 +00:00
brianwilkerson@google.com 08e1f8c63b Bug fix for issue 4256
Review URL: https://chromiumcodereview.appspot.com//10825079

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10019 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-30 16:50:51 +00:00
scheglov@google.com df6f881967 Issue 4090. For-in loop variable should not be visible in iterable expression.
R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10826055

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10002 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-28 15:20:40 +00:00
scheglov@google.com 72b2199a3a Issue 4072. Generate warning when private instance method is called
R=brianwilkerson@google.com
BUG=

Review URL: https://chromiumcodereview.appspot.com//10829064

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10001 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-28 00:52:24 +00:00
danrubel@google.com 900fa10dcd Restore DartCompiler method to fix the build (TBR)
Review URL: https://chromiumcodereview.appspot.com//10826053

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9993 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-27 21:04:20 +00:00
danrubel@google.com 5bdca22f3b Replace parsed and resolved maps with cache interface
Review URL: https://chromiumcodereview.appspot.com//10826047

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9991 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-27 20:03:13 +00:00
keertip@google.com e41a49c9a6 fix support for --package-root command line option to analyzer
Review URL: https://chromiumcodereview.appspot.com//10825054

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9981 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-27 16:15:29 +00:00