Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


Hi Alexander,

On Sun, Nov 7, 2010 at 12:55 PM, Alexander O. Anisimov
<alenyashka@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
Time: 2007.02.27

What the heck is "buy wuy" that was added here? No clue, but visiting
http://qa.openoffice.org/issues/show_bug.cgi?id=71445
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
strings (#i71445)

So I did not know to remove them or
not.

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?

ciao
Christian

Context


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.