[patch][cla] Dojo Compressor should support JScript conditional compilation

[[email protected]…]

Currently, Dojo's Compressor strips JScript conditional compilation expressions, but it would be very useful if it didn't.

Documentation for the conditional compilation feature is here (for the relevant version of JScript): ​http://msdn.microsoft.com/library/en-us/script56/html/a843de4e-3aae-43cd-ad64-477dd00814a2.asp

and the newer documentation for JScript.NET is here: ​http://msdn.microsoft.com/library/en-us/jscript7/html/jsConditionalCompilation.asp

[alex]

The changes to fix this will require adding comment data back into the parse stream. Currently, they are discarded and don't appear in the token stream in the interest of saving space.

[alex]

punt!

[guest]

bash script to work around Ticket #395

[guest]

I have written a small bash script that transforms the conditional compilation directives into expressions that are not stripped, compresses the script and retransforms the compilation directives into their old form. it can be found under (see also ​http://fforw.de/post/Dojo_Javascript_Compression_and_IE_conditional_compilation/ )

[alex]

Jesse, why did you move this back? Do you think you'll hit it for 0.4.1?

[jkuhnert]

Oops! I did actually investigate this one and finally re-remembered that rhino doesn't parse out comments at all. This one is going to be a real pain.

[Adam Peller]

bump

[Adam Peller]

please reopen if there is any plan to fix this