Ignore this.

On Fri, Aug 7, 2009 at 12:59 PM, <[email protected]> wrote:

> Reviewers: Lex, scottb,
>
> Description:
> The globalTemp used for prototype setup ends up being assigned a large
> identifier in some obfuscated programs. In Showcase, it is atleast 3
> characters. Reserving the "_" for this variable leads to  a 3%
> compressed savings, and 6% uncompressed savings.
>
>
> Please review this at http://gwt-code-reviews.appspot.com/57805
>
> Affected files:
>  dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
>
>
> Index: dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
> ===================================================================
> --- dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
> (revision 5895)
> +++ dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
> (working copy)
> @@ -500,8 +500,8 @@
>     private final JsName prototype = objectScope.declareName("prototype");
>
>     {
> +      prototype.setObfuscatable(false);
>       globalTemp.setObfuscatable(false);
> -      prototype.setObfuscatable(false);
>     }
>
>     public GenerateJavaScriptVisitor() {
>
>
>

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to