Opened 11 years ago

Closed 11 years ago

#5453 closed defect (wontfix)

Shrinksafe barfs on "debugger" command in a file

Reported by: owen Owned by: alex
Priority: high Milestone: 1.3
Component: ShrinkSafe Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

It appears that ShrinkSafe? will fail if the "debugger" browser command is found within a JS file. (tested against version at: ​http://shrinksafe.dojotoolkit.org/ ).

Attachments (1)

shrinksafe_bug.js​ (197 bytes) - added by owen 11 years ago.
File to be shrunk that shows the problem. Output is simply "null" rather than the compressed version of the file.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by owen

Attachment: shrinksafe_bug.js​ added

File to be shrunk that shows the problem. Output is simply "null" rather than the compressed version of the file.

comment:1 Changed 11 years ago by dylan

Milestone: → 1.3

comment:2 Changed 11 years ago by Adam Peller

Resolution: → wontfix
Status: new → closed

the current rev of Rhino throws an error when seeing the debugger keyword, which I think is acceptable, given that it probably should be flagged for a release, and a clear error message is far superior to just returning "null". Users could always re-insert debugging in release images as necessary, though debugging there should be a last resort.

Note: See TracTickets for help on using tickets.