Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358273 - Severe parse errors not displayed
Summary: Severe parse errors not displayed
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Editor (show other bugs)
Version: 0.3   Edit
Hardware: PC All
: P3 major (vote)
Target Milestone: 0.3 M2   Edit
Assignee: Mark Macdonald CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-20 13:01 EDT by Mark Macdonald CLA
Modified: 2011-09-20 17:58 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Macdonald CLA 2011-09-20 13:01:22 EDT 
If your Javascript file contains an error that causes JSLint to give up parsing completely, you will not see any error messages in the UI.

1. Create a new JavaScript file
2. Enter some gibberish and save. For example:
> %^&!*foo
3. A TypeError is thrown by the Orion UI (see below), but no error markers are produced.

TypeError
  arguments: Array[2]
    0:  "substring"
    1: undefined
  message: "—"
  stack: "—"
  type: "non_object_property_call"

AnnotationFactory.showProblems() at editorFeatures.js:113
exports.setUpEditor.mOutliner.Outliner.parent() at setup.js:453
EventTarget.dispatchEvent() at serviceregistry.js:185
internalRegistry.dispatchEvent() at serviceregistry.js:266
ServiceRegistration.dispatchEvent() at serviceregistry.js:94
ProblemService._setProblems() at problems.js:31
Service.addEventListener() at serviceregistry.js:127
eclipse.SyntaxChecker.SyntaxChecker.checkSyntax() at syntaxchecker.js:65
notify() at Deferred.js:171
dojo.Deferred.then.promise.then() at Deferred.js:269
eclipse.SyntaxChecker.SyntaxChecker.checkSyntax() at syntaxchecker.js:64
...
Comment 1 Mark Macdonald CLA 2011-09-20 17:58:19 EDT 
There was a problem with some code in editorFeatures.js that worked around a quirk in jslint's data format.
Moved it to jslintPlugin and fixed the error.

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=13d756a16a565abbd600e3c75ce81b7cd09865d3