To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73908


User mmeeks changed the following:

                What    |Old value                 |New value
================================================================================
                  Status|RESOLVED                  |REOPENED
--------------------------------------------------------------------------------
              Resolution|FIXED                     |
--------------------------------------------------------------------------------




------- Additional comments from [EMAIL PROTECTED] Wed Feb  7 12:26:25 +0000 
2007 -------
Hokay - I'm backing this out of the CWS - for several reasons:

* it doesn't work as is:
   + We really need a macro (eg.) RTL_USTRING_LITERAL(a) that will
     include the OUString() creation, then use that in hot-spots.

* it doesn't save that much memory:
   + much of the string duplication I've measured is removed by using
     'intern' anyway, worse using it will increase the size of the
     binaries by more than the heap we will save, since at a given time
     most strings are not instantiated on the heap.

* shlib unloading (normal lifecycle problem)
   + if we pass references to .rodata static strings around the app,
     then we can never unload that library again, ergo we should do
     this sparingly where we can prove there is a useful win I think.

And of course, I don't want the problems to block the 'intern' feature getting
integrated :-)

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