You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bcel-dev@jakarta.apache.org by bu...@apache.org on 2005/02/11 08:39:31 UTC

DO NOT REPLY [Bug 33513] New: - Unnecessary string object allocation in ConstantPoolGen

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33513>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33513

           Summary: Unnecessary string object allocation in ConstantPoolGen
           Product: BCEL
           Version: unspecified
          Platform: PC
               URL: http://www.helsinki.fi/~pohjalai/BCEL-patch.diff
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Main
        AssignedTo: bcel-dev@jakarta.apache.org
        ReportedBy: ppohja@iki.fi


I was writing software that uses the BCEL library. Having performance
problems, I found out that generic/ConstantPoolGen.java does unnecessary
StringBuffer expansion: in two cases the length of the resulting string is
known beforehand, but the created StringBuffer is not given a starting
length for the internal buffer.

With the attached patch applied, the running time of my application
decreased from 1 min 45 seconds to 1 min 30 seconds as the generation of
new constant pools was faster.

(However, by rewriting my algorithm to do less constant pool
instantiations, the running time dropped to 25 seconds).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: bcel-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: bcel-dev-help@jakarta.apache.org