davidmarklafleur / ijp

Automatically exported from code.google.com/p/ijp
0 stars 0 forks source link

Remove comma in line 138 #5

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Go here http://closure-compiler.appspot.com/home
1. Set "Google Closure Compiler" with "simple" optimations
2. Paste file ijp-0.6.js to the text area and start "compile"

What is the expected output? What do you see instead?

The compiler runs successfully. Instead you see an output similar to this:

"JSC_TRAILING_COMMA: Parse error. IE8 (and below) will parse trailing commas in 
array and object literals incorrectly. If you are targeting newer versions of 
JS, set the appropriate language_in option. at line 105 character 0
klass:this.getValue('CLASS',vtodoString), //This property defines the access ...
^"

What version of the product are you using? On what operating system?

ijp-0.6.js / Using Mac Os X

Please provide any additional information below.

Just remove in file ijp-0.6.js in line 138 the comma at the end of the line 
because the js object is closed after this and this does not like the Internet 
Explorer. Anyway the project is not for the "Internet Explorer" but "Google 
Closure Compiler" complains as well...

Original issue reported on code.google.com by michi.po...@gmail.com on 9 Oct 2012 at 8:14

GoogleCodeExporter commented 9 years ago
Trailing commas also exist at line 66 & 98.

Original comment by laps...@gmail.com on 10 Nov 2013 at 7:35

GoogleCodeExporter commented 9 years ago
Commas removed from lines 66 98 and 138

Original comment by lec...@gmail.com on 4 Jul 2014 at 10:55