dosper7 / google-code-prettify

Automatically exported from code.google.com/p/google-code-prettify
Apache License 2.0
0 stars 0 forks source link

Syntax error loading vhdl addon (due to comma) #112

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Add the following to prettify_test.html:
<script src="../src/lang-vhdl.js" type="text/javascript"
 onerror="alert('Error: failed to load ' + this.src)"></script>

2. (Download lang-vhdl.js and put in src directory)
3. Open page in IE
(Please include HTML, not just your source code)

What is the expected output?  What do you see instead?
The expected output is the script being imported correctly. Instead I get
an error:
Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0;
.NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.4506.2152; .NET CLR
3.5.30729; .NET4.0C; .NET4.0E; InfoPath.3; MS-RTC LM 8)
Timestamp: Mon, 29 Mar 2010 18:12:53 UTC

Message: '3' is null or not an object
Line: 809
Char: 9
Code: 0
URI: file:///C:/Documents%20and%20Settings/mjdiehl/Desktop/pp2/src/prettify.js

What version are you using?  On what browser?
I am using the 3 Dec 2009 version in IE. The source comes from the trunk.

Please provide any additional information below.
In the lang-vhdl.js file, line 15 says:
         [PR.PR_PLAIN, /^[\t\n\r \xA0]+/, null, '\t\n\r \xA0'],
if you remove the comma (since there is nothing in the list after this line):
         [PR.PR_PLAIN, /^[\t\n\r \xA0]+/, null, '\t\n\r \xA0']

Then there is no error and it loads fine. Please remove the comma (my
firewalls prevent me from doing a turnin myself)

Original issue reported on code.google.com by St33lD1...@gmail.com on 29 Mar 2010 at 6:16

GoogleCodeExporter commented 8 years ago
Thanks for the thorough bug report.

Fixed at 98 : http://code.google.com/p/google-code-prettify/source/detail?r=98

Original comment by mikesamuel@gmail.com on 31 Mar 2010 at 2:31