[Libreoffice] [PATCH] Comment cleanup writer

Cedric Bosdonnat cedric.bosdonnat.ooo at free.fr
Tue Mar 8 00:51:07 PST 2011


Hi Muthu, David,

On Mon, 2011-03-07 at 21:46 +0530, Muthu Subramanian K wrote:
> Hi David,
> 
> Thank you for you patch.
> The general rule is to keep the "i####" kind of numbers - these are
> issuezilla ids. But, in this case I would prefer to go with you to
> remove most of these - since, in this patch context, the ids seem to
> have been added to every line.
> 
> On the other hand, I would prefer another person reviewing before I push
> these changes - cedric?

I would keep the #i91601# before one FN_UNO_LIST_ID to show it has been
added to fix that bug, but remove the other instances. IMHO quite a lot
of #i123456# comments would be lost: these need to be kept in at least
some strategic places.

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr



More information about the LibreOffice mailing list