[Libreoffice] LibreOffice Code Cleanup Patch

Muthu Subramanian K sumuthu at suse.com
Thu Jan 26 20:49:32 PST 2012


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
> 



More information about the LibreOffice mailing list