You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by sa...@apache.org on 2009/03/08 21:54:02 UTC

svn commit: r751522 - in /incubator/click/trunk/click: documentation/docs/roadmap-changes.html framework/src/org/apache/click/control/AbstractContainer.java

Author: sabob
Date: Sun Mar  8 20:54:01 2009
New Revision: 751522

URL: http://svn.apache.org/viewvc?rev=751522&view=rev
Log:
performance improvement for HTML imports

Modified:
    incubator/click/trunk/click/documentation/docs/roadmap-changes.html
    incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractContainer.java

Modified: incubator/click/trunk/click/documentation/docs/roadmap-changes.html
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/documentation/docs/roadmap-changes.html?rev=751522&r1=751521&r2=751522&view=diff
==============================================================================
--- incubator/click/trunk/click/documentation/docs/roadmap-changes.html (original)
+++ incubator/click/trunk/click/documentation/docs/roadmap-changes.html Sun Mar  8 20:54:01 2009
@@ -106,7 +106,11 @@
       or 1.5 respectively.
     </div>
     <ul style="padding: 0em; margin-left:0em;margin-bottom: 2em">
-       <li class="change">
+      <li class="change">
+          Improved performance in situations where many controls are added to
+          a page by decreasing the buffer size used for including HTML imports.
+      </li>
+      <li class="change">
           <a class="external" target="_blank" href="http://code.google.com/p/click-calendar/">Click Calendar</a>
           version 1.0.1 has been released which fixes a memory leak in the calendar popup
           [<a target='_blank' href="https://issues.apache.org/click/browse/CLK-499">499</a>].

Modified: incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractContainer.java
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractContainer.java?rev=751522&r1=751521&r2=751522&view=diff
==============================================================================
--- incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractContainer.java (original)
+++ incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractContainer.java Sun Mar  8 20:54:01 2009
@@ -282,7 +282,7 @@
      */
     public String getHtmlImports() {
         if (hasControls()) {
-            HtmlStringBuffer buffer = new HtmlStringBuffer(512);
+            HtmlStringBuffer buffer = new HtmlStringBuffer(0);
             for (int i = 0, size = getControls().size(); i < size; i++) {
                 Control control = (Control) getControls().get(i);
                 String htmlImports = control.getHtmlImports();