[ http://issues.apache.org/jira/browse/MYFACES-559?page=all ]
     
Martin Marinschek closed MYFACES-559:
-------------------------------------

    Fix Version: Nightly Build
     Resolution: Fixed

Thanks to Timo Thomas for this suggestion + patch! very good improvement for 
jsValueSet.

> HtmlJsValueSet: smaller JavaScript code size, nested maps and collections, 
> correct string escaping
> --------------------------------------------------------------------------------------------------
>
>          Key: MYFACES-559
>          URL: http://issues.apache.org/jira/browse/MYFACES-559
>      Project: MyFaces
>         Type: Improvement
>   Components: Tomahawk
>     Versions: 1.0.9m9
>     Reporter: Timo Thomas
>     Assignee: Martin Marinschek
>     Priority: Minor
>      Fix For: Nightly Build
>  Attachments: jsvalueset.diff
>
> Note: patch file following
> - smaller generated Javascript code size:
> The current version initializes array elements in separate statements, 
> whereas my version initializes them in the array constructor, e.g.
>   current version:
>   var a = new Array();
>   a[0] = 'foo';
>   a[1] = 'bar';
>   ...
>   new version:
>   var a = new Array('foo','bar',...);
> This is important for big arrays containing (textual) small values in 
> small-bandwidth environments and has no drawbacks.
> - support for nested collections and maps:
> Collections of collections, maps of collections, maps of maps of collections, 
> etc. are supported. The nesting level is not limited (but will be, in 
> practice, limited by stack size or the javascript interpreter). 
> - correct JavaScript string escaping:
> The new version uses StringEscapeUtils.escapeJavaScript() from the Jakarta 
> Commons Lang library for generating JavaScript string constants.

-- 
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

Reply via email to