Commit Graph

103 Commits

Author SHA1 Message Date
scheglov@google.com d9605fe9ef Issue 8051. Make extended exit codes optional.
http://code.google.com/p/dart/issues/detail?id=8051

Unfortunately testing framework expects exactly 0 and 1.
So, I've made 0/1/2 exit codes optional.

Phil, FYI

R=devoncarew@google.com,pquitslund@google.com
BUG=

Review URL: https://codereview.chromium.org//12045088

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@17629 260f80e4-7a28-3924-810f-c04153c831b5
2013-01-25 04:11:12 +00:00
scheglov@google.com be31b47690 Issue 8051. dart_analyzer return codes on errors
http://code.google.com/p/dart/issues/detail?id=8051

R=devoncarew@google.com,whesse@google.com
BUG=

Review URL: https://codereview.chromium.org//12041051

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@17622 260f80e4-7a28-3924-810f-c04153c831b5
2013-01-25 00:47:48 +00:00
scheglov@google.com b806f5f3f9 Issue 7884. Use UTF-8 for output.
http://code.google.com/p/dart/issues/detail?id=7884

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//12036045

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@17475 260f80e4-7a28-3924-810f-c04153c831b5
2013-01-23 17:31:47 +00:00
scheglov@google.com 0abc92dae5 Tweak to support 'dart.core' as core library name.
Needed to commit again https://codereview.chromium.org/11419032/

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11615031

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@16278 260f80e4-7a28-3924-810f-c04153c831b5
2012-12-18 19:11:20 +00:00
scheglov@google.com 694b17e439 Issue 6971. Fixes for using built-in identifier as import prefix.
http://code.google.com/p/dart/issues/detail?id=6971

Only first part of issue.
I don't agree that code like this is valid:
   abstract<dynamic> v;

I think "abstract" here is still type annotation, so error should be reported.

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11549028

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@16086 260f80e4-7a28-3924-810f-c04153c831b5
2012-12-13 01:16:41 +00:00
scheglov@google.com b7cbab1a19 Enabled reporting problems in some libraries.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11464004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@15802 260f80e4-7a28-3924-810f-c04153c831b5
2012-12-06 19:14:50 +00:00
scheglov@google.com 17d16e41d7 Triage and fixes for co19 library tests.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11416307

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@15643 260f80e4-7a28-3924-810f-c04153c831b5
2012-12-03 17:30:21 +00:00
scheglov@google.com f90ff506ad Issue 6859. Report wrong URI at correct location.
http://code.google.com/p/dart/issues/detail?id=6859

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11280196

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@15425 260f80e4-7a28-3924-810f-c04153c831b5
2012-11-27 23:31:46 +00:00
danrubel@google.com da4d1c5c7e cleanup unused code
Review URL: https://codereview.chromium.org//11369214

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14888 260f80e4-7a28-3924-810f-c04153c831b5
2012-11-14 13:11:11 +00:00
scheglov@google.com 5a8befa2d8 Get real result for batch exacution.
Not just exit code 1 always.

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11366213

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14865 260f80e4-7a28-3924-810f-c04153c831b5
2012-11-13 21:04:14 +00:00
ajohnsen@google.com 970cc5ba68 Move JSSyntaxRegExp to core as a private member. This removes the last refrences to dart:coreimpl.
After this cleanup, RegExp no longer have a const constructor. Use 'new
RegExp(...)' from now on.

BUG=

Review URL: https://codereview.chromium.org//11365196

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14838 260f80e4-7a28-3924-810f-c04153c831b5
2012-11-13 10:41:37 +00:00
scheglov@google.com 303a6202cd Issue 5722. Better 'missing part of' error message. Quick Fix for it.
http://code.google.com/p/dart/issues/detail?id=5722

R=messick@google.com
BUG=

Review URL: https://codereview.chromium.org//11364020

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14378 260f80e4-7a28-3924-810f-c04153c831b5
2012-11-01 00:35:28 +00:00
scheglov@google.com 2e574c8c14 Issue 5887. Parts must start with 'part of'.
Second attempt.
Now only DartC changes, and status updates.
So, should not break anyone else.

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//11273064

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14217 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-29 19:17:28 +00:00
devoncarew@google.com 6ee84d5e7f Removed a -d32 flag to dart_analyzer (fix for http://code.google.com/p/dart/issues/detail?id=5836)
Quoted input to tr in dart_analyzer.sh (http://code.google.com/p/dart/issues/detail?id=6051)
Changed dart_analyzer --version to report the new version format (not the old build revision format).
Review URL: https://codereview.chromium.org//11234069

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@14028 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-24 16:53:46 +00:00
ahe@google.com 6254729761 Revert "Parts must start with 'part of'" and "Attempt to fix VM build"
This reverts r13961 and r13963.

Review URL: https://codereview.chromium.org//11233061

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@13967 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-23 14:24:04 +00:00
scheglov@google.com 50b5b6e1a8 Parts must start with 'part of'
git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@13961 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-23 13:20:50 +00:00
scheglov@google.com da10523b14 Issue 6077. warn when importing dart:mirrors
http://code.google.com/p/dart/issues/detail?id=6077

R=danrubel@google.com
BUG=

Review URL: https://codereview.chromium.org//11225007

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@13845 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-19 23:11:29 +00:00
scheglov@google.com 8dbff1e87c Ignore 'dart-ext:' scheme, allow 'native' in such files
R=danrubel@google.com
BUG=

Review URL: https://codereview.chromium.org//11088009

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@13371 260f80e4-7a28-3924-810f-c04153c831b5
2012-10-08 20:15:24 +00:00
scheglov@google.com c095d071e9 Issue 5052. Analyzer should treat assert as a keyword
http://code.google.com/p/dart/issues/detail?id=5052

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//10949015

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12566 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-19 15:30:12 +00:00
karlklose@google.com d90d5b1698 Reapply 'Add runtimeType() to Object which returns canonicalized instances of Type'.
This CL fixes a bug in https://chromiumcodereview.appspot.com/10908142/ which generated unneeded type information leading to runtime regression.

Original committed in 12491 and reverted in

R=ngeoffray@google.com

Review URL: https://codereview.chromium.org//10941031

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12540 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-19 08:56:06 +00:00
ricow@google.com 5bcc7e0092 Revert r12491
This is causing checked mode failures on the buildbot.
Review URL: https://codereview.chromium.org//10949003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12500 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-18 14:40:15 +00:00
karlklose@google.com 59e0304206 Add runtimeType() to Object which returns canonicalized instances of Type.
BUG=3903

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12491 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-18 12:12:07 +00:00
scheglov@google.com 03ccc4df50 Issue 4988. Fixe for 'assert' statement vs function resolving
http://code.google.com/p/dart/issues/detail?id=4988

R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12071 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-07 22:09:49 +00:00
scheglov@google.com 93f4f29df1 Support for new 'export' directive in resolver
R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11984 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-06 20:28:44 +00:00
ajohnsen@google.com 3cf5d6531d Move core and coreimpl from corelib/ to lib/core and lib/coreimpl.
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11606 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-30 11:45:40 +00:00
scheglov@google.com 6ef7aa818d Issue 3904. Allow using class literal in expressions
http://code.google.com/p/dart/issues/detail?id=3904

R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11440 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-28 13:16:21 +00:00
scheglov@google.com 305f5adaa7 Issue 4539. Tweaks for old/new spec compatibility
Temporary disables check for underscore in named optional parameters.
There are not way to know if user means old or new syntax.

Also makes "final" and "const" local variables a constant expressions.

R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11117 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-22 00:11:38 +00:00
keertip@google.com 0573730e02 fix build failure, rename refactor take 2
Review URL: http://codereview.chromium.org//10837369

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11104 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-21 22:22:49 +00:00
keertip@google.com 784d765bdd rename refactor SystemLibraryManager -> PackagelibraryManager SdkLibraryManager -> SystemLibraryManager
Review URL: http://codereview.chromium.org//10827457

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11101 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-21 21:36:44 +00:00
brianwilkerson@google.com cdaf30bda9 Stop reporting an error in parts that contain a part of directive
Review URL: https://chromiumcodereview.appspot.com//10827453

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11092 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-21 20:19:10 +00:00
scheglov@google.com cc287b555b Issue 4632. Resolve all units before using next phase.
http://code.google.com/p/dart/issues/detail?id=4632

R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@11085 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-21 18:55:08 +00:00
scheglov@google.com acca1aaf61 Issue 4543. Use complete Resolver before constants analyzer
http://code.google.com/p/dart/issues/detail?id=4543

This allows us to use resolved Elements for all nodes, without duplicating Resolver functionality.

R=brianwilkerson@google.com
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@10902 260f80e4-7a28-3924-810f-c04153c831b5
2012-08-17 15:58:11 +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
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
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
scheglov@google.com a25ed30356 Check for conflicts between new function/method and other elements.
R=messick@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9812 260f80e4-7a28-3924-810f-c04153c831b5
2012-07-23 14:33:54 +00:00
scheglov@google.com 90d27a764a Issue 3839. Warn when 'dart:io' and 'dart:html' are used in single library
http://code.google.com/p/dart/issues/detail?id=3839

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9133 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-26 20:06:12 +00:00
scheglov@google.com 6ed95757fe Issue 3752. Support for @override annotations (as structured doc comments)
http://code.google.com/p/dart/issues/detail?id=3752

Now we always record comments and bind them to nodes.
We record DartMetadata for each DartDeclaration with flags for @deprecated and @override.
In this CL we analyze @override and report error if method does not actually override anything.

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9129 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-26 19:47:48 +00:00
danrubel@google.com e63836db88 Fix NPE when DartC analyze library fails
Review URL: https://chromiumcodereview.appspot.com//10665020

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@9066 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-25 16:51:19 +00:00
scheglov@google.com a01d245a87 Issue 3530. Import elements into separate import namespace
http://code.google.com/p/dart/issues/detail?id=3530

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8696 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-14 22:55:41 +00:00
scheglov@google.com c1e44ea184 Issue 3550. Initial understanding "const".
http://code.google.com/p/dart/issues/detail?id=3550

To complete "const" support http://code.google.com/p/dart/issues/detail?id=3553 should be done.

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8670 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-14 17:00:45 +00:00
scheglov@google.com 462b64cb7f Attempt to fix build
git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8565 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-12 19:07:00 +00:00
scheglov@google.com 405b72c292 Issue 3532. Report error if one unit included twice
http://code.google.com/p/dart/issues/detail?id=3532

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8562 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-12 18:38:51 +00:00
scheglov@google.com f50560b8ad Issue 3531. Don't allow using built-in identifier as import prefix
http://code.google.com/p/dart/issues/detail?id=3531

And couple other built-in/reserved issues.

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8558 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-12 18:05:20 +00:00
scheglov@google.com b5142cdb20 Issue 3268. Support for replacing 'assert' statement with 'assert' function
http://code.google.com/p/dart/issues/detail?id=3268

R=brianwilkerson@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8290 260f80e4-7a28-3924-810f-c04153c831b5
2012-06-05 16:08:09 +00:00
scheglov@google.com 3eb17a6ca6 Issue 3155. Better error report for directives in sourced unit
http://code.google.com/p/dart/issues/detail?id=3155

R=danrubel@google.com
BUG=
TEST=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@8078 260f80e4-7a28-3924-810f-c04153c831b5
2012-05-29 20:29:58 +00:00
danrubel@google.com 5b2e8281a9 Cache unresolved units separately from libraries
Review URL: https://chromiumcodereview.appspot.com//10389074

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@7527 260f80e4-7a28-3924-810f-c04153c831b5
2012-05-11 01:24:17 +00:00