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]