[Libreoffice] Replacing ByteString with rtl::OString

Caolán McNamara caolanm at redhat.com
Wed Jul 13 03:01:40 PDT 2011


On Tue, 2011-07-12 at 21:18 +0200, Chr. Rossmanith wrote:
> Hi,
> 
> next problem: there is no IsNumericAscii() for O(U)String but for 
> ByteString. Should O(U)String get this method as well?

Don't know if its really widely used enough to warrant expanding the
O[U]String API ? You could perhaps re-implement it in terms of a OString
and move it to be a helper method in comphelper, i.e.
comphelper/inc/comphelper/string.hxx which is the home of useful string
utilities which aren't great fits for the public api, e.g. Sébastien's
cool decimalStringToNumber which knows about non-standard unicode
numerals and can do basically a toInt32 on those.

As an aside, see https://bugs.freedesktop.org/show_bug.cgi?id=34779 and
commit
http://cgit.freedesktop.org/libreoffice/writer/commit/?id=65d3573c5afd7fd132b30c41a240d1d2d04c8527
for the kind of goodness that flows out of ByteString->OString and
UniString->OUString overhauls.

C.



More information about the LibreOffice mailing list