Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad error reporting. #1846

Closed
floitschG opened this issue Feb 24, 2012 · 2 comments
Closed

Bad error reporting. #1846

floitschG opened this issue Feb 24, 2012 · 2 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.

Comments

@floitschG
Copy link
Contributor

When a syntax error is detected the VM prints a list of library-imports first.
Since the error-buffer seems to be limited to some arbitrary limit the real error is often not shown.
Example:
===
$ cat bug1.dart

library("bug1");

import("bug2.dart");

$ cat bug2.dart

library("bug2");

import("bug3.dart");

$ cat bug3.dart

library("bug3");

abstract class A { }
$ out/Release_ia32/dart bug1.dart
'/tmp/bug1.dart': Error: line 2 pos 1: library handler failed: '/tmp/bug2.dart': Error: line 2 pos 1: library handler failed: '/tmp/bug3.dart': Error: line 3 pos 1: unexpected token 'abstract'
abstract class A { }
^

import("bug3.dart");

^

import("bug2.dart");

^
===

@ghost
Copy link

ghost commented Feb 24, 2012

Added Accepted label.

@ghost
Copy link

ghost commented Feb 29, 2012

Fixed in r4773


Added Fixed label.

@floitschG floitschG added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. labels Feb 29, 2012
@floitschG floitschG assigned ghost Feb 29, 2012
copybara-service bot pushed a commit that referenced this issue Jan 19, 2023
Revisions updated by `dart tools/rev_sdk_deps.dart`.

dartdoc (https://github.com/dart-lang/dartdoc/compare/a99abd4..ed56883):
  ed56883d  2023-01-17  Parker Lougheed  Fix, cleanup, and simplify CSS (#3302)

http (https://github.com/dart-lang/http/compare/c955c7e..57c53b0):
  57c53b0  2023-01-17  Brian Quinlan  Fix a NPE that occurs when an error occurs before a response is received. (#852)

protobuf (https://github.com/dart-lang/protobuf/compare/dd04535..cc0f287):
  cc0f287  2023-01-18  Kevin Moore  Run non-null-safe tests with Dart 2 (#791)

test (https://github.com/dart-lang/test/compare/43fd928..19582a8):
  19582a8d  2023-01-19  Nate Bosch  Avoid empty `::group::` in Github reporter (#1855)
  fc4ea341  2023-01-19  Nate Bosch  Fix a test to use an still invalid URI (#1856)
  bd96213a  2023-01-18  Nate Bosch  Improve output for collections and long strings (#1852)
  9f7de8a2  2023-01-17  Nate Bosch  Remove operator extensions (#1847)
  f790feb0  2023-01-17  Nate Bosch  Add Condition abstraction (#1846)

webdriver (https://github.com/google/webdriver.dart/compare/b8c7bb5..00c887d):
  00c887d  2023-01-17  Devon Carew  rev to 3.0.2 in preparation for publishing (#268)

Change-Id: I2cbbcdce4fa510cd2458aa05dd45eed01e840c9c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/279329
Auto-Submit: Devon Carew <devoncarew@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
Projects
None yet
Development

No branches or pull requests

1 participant