Bug 304933 - [validation] [syntax] false warning reported about missing semicolon before the closing }
Summary: [validation] [syntax] false warning reported about missing semicolon before t...
Status: NEW
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: Future   Edit
Assignee: Chris Jaun CLA
QA Contact: Chris Jaun CLA
URL:
Whiteboard:
Keywords: plan
Depends on:
Blocks:
 
Reported: 2010-03-07 04:42 EST by Matěj Cepl CLA
Modified: 2014-05-29 13:53 EDT (History)
2 users (show)

See Also:

Attachments
screenshot of the wrong behavior (203.16 KB, image/png)
2010-03-07 04:44 EST, Matěj Cepl CLA 		no flags Details
example javascript program presenting the problem (69.44 KB, text/plain)
2010-03-13 03:45 EST, Matěj Cepl CLA 		no flags Details
screenshot of incorrect warning (129.18 KB, image/png)
2010-09-25 14:29 EDT, Matěj Cepl CLA 		no flags Details
file where the error happens (3.32 KB, text/plain)
2010-09-25 14:30 EDT, Matěj Cepl CLA 		no flags Details
View All 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 Matěj Cepl CLA 2010-03-07 04:42:45 EST 
In this code

    var serialForm = {
        dataOut: "",
        name: form.name,
        method: form.method,
        acceptCharset: form.acceptCharset,
        action: form.action, // TODO shouldn't we get a non-relative URL?
        enctype: form.enctype,
        cookie: this.dok.cookie,
        autocomplete: form.getAttribute("autocomplete"),
        bugNo: this.bugNo
    };

 Eclipse claims there is a missing semicolon before the closing }
Comment 1 Matěj Cepl CLA 2010-03-07 04:44:34 EST 
Created attachment 161237 [details]
screenshot of the wrong behavior
Comment 2 Chris Jaun CLA 2010-03-12 15:09:05 EST 
I am having trouble reproducing this with the code snippet you included.

What WTP build are you using?

Are you able to reproduce with just this block?

    var serialForm = {
        dataOut: "",
        name: form.name,
        method: form.method,
        acceptCharset: form.acceptCharset,
        action: form.action, // TODO shouldn't we get a non-relative URL?
        enctype: form.enctype,
        cookie: this.dok.cookie,
        autocomplete: form.getAttribute("autocomplete"),
        bugNo: this.bugNo
    };

I know your image includes more code. Maybe we need a bit more context to reproduce.
Comment 3 Matěj Cepl CLA 2010-03-13 03:45:18 EST 
Created attachment 161956 [details]
example javascript program presenting the problem
Comment 4 Matěj Cepl CLA 2010-03-13 03:45:34 EST 
OK, I am not able to reproduce the problem when I make a new file with just your snippet. Complete script which presents the problem is attached.
Comment 5 Chris Jaun CLA 2010-03-15 09:22:11 EDT 
Thank you for the updated information and example.
Comment 6 Matěj Cepl CLA 2010-09-25 14:29:15 EDT 
Created attachment 179572 [details]
screenshot of incorrect warning

Still present in

Version: 1.1.2.v200908101420-77-FGDCcNBDjBXMoBbFb
Build id: 20100211202452
Comment 7 Matěj Cepl CLA 2010-09-25 14:30:47 EDT 
Created attachment 179573 [details]
file where the error happens