Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


On Mon, 2010-12-13 at 23:52 +0900, Takeshi Abe wrote:
So I amend my patch as attached

Looks good to me, pushed, thanks for this. Nice to see some UniStrings
and ByteStrings bite the dust.

As an aside, there is plenty of code that started off with char*, got
changed to "String" which was then the 8bit version, then "UniString"
was introduced to get Unicode support, and the original "String" renamed
"ByteString" and "UniString" aliased to String.

Which all means that every now and then you might even find some bugs
where the old pointer logic of e.g. + 2 at the end of a char* got badly
mangled during all the transitions into a String into something like
String+=String::CreateFromInt32(2) !


which still allows the users to pass
rather cooler option -psn_is_not_PlayStation_Network than just -psn to
soffice, both resulting in the same effect ;)

Yeah, it might be a bug in the first place, but best to do an exact
like-for-like change just in case changing from one string type to the
other gets a bad reputation for silenty changing behaviour :-)

It sounds great, so I created a table at the end of
http://wiki.documentfoundation.org/Development/String_Classes
which now contains only some functions but will be added more.

That's great work. I always get somewhat confused when swapping from one
to the other and end up digging into the UniString impl to see what it
*really* does.

C.


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.