Hello Alexander,
Its nice to see those patches. But, it will be nice to see/use the
tools/scripts directly? Especially if they can be hooked to git (like
the current white space/tabs removal ones)?
@lubos: Alexander is good with scripts, so as a starter he wanted to do
something like
"http://cgit.freedesktop.org/libreoffice/core/commit/?id=59cefd339d0c3fcde5b614d7f142a957397672d9"
. So, yes, I had asked him to give it a shot. I guess, the patch is good
as well?
Thanks!
Muthu Subramanian
On 01/26/2012 10:03 PM, Alexander Bergmann wrote:
Hi Muthu,
as I announced yesterday, I started a search & replace task. But - as
life goes - this is not wanted? I thought, you encouraged me to do this
sort of thing... See:
http://lists.freedesktop.org/archives/libreoffice/2012-January/024616.html
and
http://lists.freedesktop.org/archives/libreoffice/2012-January/024624.html
The patch should replace SAL_THROW( () ) by SAL_THROW(()) and THROWS( ()
) by THROWS(()). Actually at the moment both versions of SAL_THROW and
THROWS are used in the LibreOffice code base.
Best regards,
Alexander
Context
- Re: [Libreoffice] LibreOffice Code Cleanup Patch · Muthu Subramanian K
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.