[Libreoffice] [PATCH] Remove bogus comments in components
lohmaier+libreoffice at googlemail.com
Sun Nov 7 04:48:59 PST 2010
On Sun, Nov 7, 2010 at 12:55 PM, Alexander O. Anisimov
<alenyashka at gmail.com> wrote:
> I've remove all bogus comments in components. Patches are in attachment. I
> was attached two patches because I could not understand the difference
> between "// --> OD" and "// --> PB".
Thanks for the patches, the OD and PB just are username nicks, so
those can be removed, but please don't remove the corresponding
issuezilla bugentries, like #i<number# or just i<number>, those refer
to the OpenOffice.org bugtracker (issuezilla), and that info sometimes
is very helpful.
Eg this comment
//-->Calculate dynamical width of controls, add buy wuy for i71445
What the heck is "buy wuy" that was added here? No clue, but visiting
helps - the "add buy wuy" means "contributed by (Yan) Wu to fix issue 71445
So it should be changed to something like
// calculate dynamic width of controls, to not cut-off translated
> So I did not know to remove them or
The date-stamp and the committer's nick: Yes, remove, but don't stip
the issue-number and an explanatory comment that follows those tags.
So apart from the "buy wuy" one, patch #1 is OK, thanks for that
Also I don't remove the //CHINA001 begin ... //CHINA001 end just now,
since could be that the hole parts is kind of obsolete, so leave in
the markers to have a second look - the removal of the
already-commented out stuff is OK though.
One minor thing: you changed encoding of a comment - but I'll
translate that one right away
// muß gespeichert werden ? -> // need to save?
More information about the LibreOffice