[Libreoffice] [Partly pushed] Easy Hack completed - All bogus comments removed
julio.hoffimann at gmail.com
Fri Apr 15 09:36:29 PDT 2011
Ok Cedric, no problem. I will fix these problems.
Thank you for return.
2011/4/15 Cedric Bosdonnat <cedric.bosdonnat.ooo at free.fr>
> Hi Julio,
> Many thanks for your patches nice patches. I think we'll manage to
> eradicate loads of useless comments this way ;)
> I pushed both patches in libs-gui and libs-core after some manual edits,
> but it takes too much time to manually adjust the patch for sw. Could
> you please adjust your regexps to fix the following problems:
> * comments indentation is reduced to 4 spaces in all cases: you should
> keep the number of existing leading spaces to avoid breaking the
> * #b[0-9]\+# comments should be purely removed: we don't need them as
> they refer to some Oracle internal bug database. Please pay attention
> here ase you may have #b...# and #i....# references on the same line
> * you often have some comment after the bug number, please keep them.
> Some could be removed, but not all.
> * While at it, could you remove the lines with comments like '// <--'
> as those are the "closing" comments for the other silly SCM-like
> Don't forget to check your patch after running your regexps as you may
> have other unexpected cases.
> On Thu, 2011-04-14 at 19:26 -0300, Júlio Hoffimann wrote:
> > I forget to mention...
> > This patch is under the LGPLv3+ / MPL license.
> > 2011/4/14 Júlio Hoffimann <julio.hoffimann at gmail.com>
> > ... and the last and big one. ;-)
> > https://bugs.freedesktop.org/show_bug.cgi?id=36234
> > Regards,
> > Júlio.
> > _______________________________________________
> > LibreOffice mailing list
> > LibreOffice at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/libreoffice
> Cédric Bosdonnat
> LibreOffice hacker
> OOo Eclipse Integration developer
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the LibreOffice