[ http://issues.apache.org/jira/browse/ADFFACES-320?page=all ]
Kamran Kashanian updated ADFFACES-320:
--------------------------------------
Attachment: obfuscator.plugin2.patch
This patch implements a less conservative approach to stripping whitespaces
than the previous patch. While the previous patch left a single whitespace
character around *every* JS token in the file, this patch restricts this to
"unary" operators. In the grammar file unary operators include the following:
delete, void, typeof, ++, --, +, -, ~, and !.
> Obfuscation of addition statement where one of the items being added is an
> increment statement
> ----------------------------------------------------------------------------------------------
>
> Key: ADFFACES-320
> URL: http://issues.apache.org/jira/browse/ADFFACES-320
> Project: MyFaces ADF-Faces
> Issue Type: Bug
> Components: Build
> Environment: maven-javascript-plugin
> Reporter: Matt Cooper
> Priority: Minor
> Attachments: obfuscator.plugin.patch, obfuscator.plugin2.patch
>
>
> If you have JavaScript code like the following:
> var x = 1;
> var y = "" + ++x;
> It is a legal operation. However, if this code is run through the
> maven-javascript-plugin, the second line becomes something like this:
> var y=""+++x;
> This yields an invalid increment operand. Instead, the space between the "+"
> and the "++" must be preserved. I also suspect (but have not tested) this
> may be a problem with this code too:
> var a = 1;
> var b = a++ + "";
> I suspect the increment and the decrement tokens in the obfuscater just need
> to be tagged as space sensitive (both before and after)--so we don't remove
> the spaces when compacting the code.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
