Changes between Initial Version and Version 5 of Ticket #3990
- Timestamp:
- Aug 9, 2007, 8:21:05 AM (12 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #3990
-
Property
Status
changed from
new
toassigned
-
Property
Summary
changed from
custom_rhino.diff - Invalid parser
to[patch]custom_rhino.diff - Invalid parser
-
Property
Milestone
changed from
to
0.9
- Property Owner changed from James Burke to alex
-
Property
Severity
changed from
blocker
tocritical
-
Property
Status
changed from
-
Ticket #3990 – Description
initial v5 1 1 When trying to build the following code: 2 2 3 {{{ 3 4 cb = function(){ 4 5 this.hideAction(); … … 7 8 } 8 9 }.createDelegate(this); 10 }}} 9 11 10 12 The dojo optimizer (Shrinksafe) will replace the last line with: 13 {{{ 14 };.createDelegate(this); 15 }}} 11 16 12 };.createDelegate(this); 13 17 {{{ 14 18 cb = function(){ 15 19 this.hideAction(); … … 18 22 } 19 23 };.createDelegate(this); 24 }}} 20 25 21 26 This ends up being invalid javascript. After going through rhino, here's the needed fix: 22 27 28 {{{ 23 29 > case FUNCTION_END: 24 30 > if ( … … 30 36 > (getNext(encodedSource, length, i+1) != Token.DOT) && 31 37 > (getNext(encodedSource, length, i+1) != Token.COLON) && 32 38 }}} 33 39 34 40 So adding the line: 41 {{{ 35 42 > (getNext(encodedSource, length, i+1) != Token.DOT) && 43 }}}