To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=108161
                 Issue #|108161
                 Summary|[cppuhelper] WeakReferenceHelper::operator= performanc
                        |e
               Component|udk
                 Version|DEV300m67
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|code
             Assigned to|sb
             Reported by|mst





------- Additional comments from [email protected] Fri Jan  8 17:30:56 +0000 
2010 -------
cppuhelper/weak.hxx:

WeakReferenceHelper has 2 assignment operators, one copy assignment, and one for
assigning a uno::Reference.

the latter is implemented by constructing a temporary WeakReferenceHelper from
the uno::Reference, then assigning that via the other operator=.

callgrind says that constructing and destructing the temporary copy is twice as
expensive as the actual assignment.

it probably would be better to have a fast assignment operator for
uno::Reference, and have the copy assignment operator invoke that instead of the
other way around.

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