To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73086
                 Issue #|73086
                 Summary|make local variable lookup faster
               Component|framework
                 Version|OOo 2.2
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|PATCH
                Priority|P3
            Subcomponent|scripting
             Assigned to|npower
             Reported by|npower





------- Additional comments from [EMAIL PROTECTED] Wed Jan  3 08:48:17 -0800 
2007 -------
looking up a local variable is quit slow as already illustrated by the document
attached to issue 69913

here's a little patch that speeds up that test (
http://www.openoffice.org/nonav/issues/showattachment.cgi/39396/TimerTest.ods )

from 130 secs to 90 secs.

basically SbxArray::Find is pretty slow ( well slow when its run thousands and
thousands of times in succession ) mainly it's slow because there is the
iteration serially through the array and the associated hash creation for the
name and the tolower ( or is it toupper? ) etc ) 

this is all uncessary in the case of a local ( and possibly other scenarios in
FindElement ( we should look further ) ) because we do the lookup as follows

opcode => name => index of the SbxArray

we can make this more efficient by creating a map of the opcode => index thus
directly accessing the variable in the SbxArray ( refLocals ) by index ( if it
has been found previously )

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to