Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 335265

Summary:

Optional Semi-Colon warning should be controllable

Product:

[WebTools] JSDT

Reporter:

Paul Beusterien <paul.beusterien>

Component:

General

Assignee:

Nitin Dahyabhai <thatnitind>

Status:

RESOLVED FIXED

QA Contact:

Nitin Dahyabhai <thatnitind>

Severity:

enhancement

Priority:

CC:

cmjaun

Version:

3.3

Flags:

thatnitind: review?

Target Milestone:

3.3

Hardware:

OS:

All

Whiteboard:

Attachments:

Description	Flags
Makes semicolon problem less painful until broader fix available	none

Description Paul Beusterien

2011-01-24 18:45:01 EST

Unlike the other Potential programming problems, the semi-colon warning occurs even when semantic validation is disabled.

this is troubling because of
1. the various semicolon bugs - 324673, 326842, 304933
2. the prevalence of these in common javaScript libraries
3. the unintuitive workaround - 
A. Turn on semantic validation
B.  Find and ignore Semi-colon warning
C. Disable semantic validation

Comment 1 Chris Jaun

2011-02-07 16:45:55 EST

We're working on cleaning up semantic validation in 3.2.4.

Comment 2 Paul Beusterien

2011-02-20 11:23:29 EST

Created attachment 189363 [details]
Makes semicolon problem less painful until broader fix available

Make default be ignore optional semicolons.

Comment 3 Nitin Dahyabhai

2011-06-06 11:54:14 EDT

Semi-colons are a syntax problem, not a semantic problem.  While allowed in the language, they're ambiguous and generally discouraged.  Changing summary and severity.

Comment 4 Nitin Dahyabhai

2011-06-30 17:34:40 EDT

Done.