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


On Tuesday 13 of March 2012, Olivier Hallot wrote:

Are we going to remove  RTL_CONSTASCII_USTRINGPARAM from all the code?
There a 33599(*) lines in whole code...

 That is the plan, but I still want to give it a little more time in case 
somebody runs into a problem. I just massively broke the build on mac on 
Monday with my OString changes because the ancient gcc we use there had a 
problem, and I wasn't very happy with the prospect of reverting even what 
I've committed only so far. But eventually yes, although it's probably not 
going to be all (e.g. the tools String class still uses it and should be 
instead converted to rtl strings, and there may be few more cases where the 
macro makes sense).

Can it be automated?

 Not with something as simple as grep&sed I think, because of what I said 
above. But I've just found out about the clang compiler and switched to it, 
and it seems one of its nice features should be the ability to analyze and 
rewrite code, in which case a change like this or e.g. the 'str.length() == 
0' -> 'str.isEmpty()' change could be done reliably by writting some clang 
code. I still need to actually check that.

-- 
 Lubos Lunak
 l.lunak@suse.cz

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.