Patch 4 only removes structuring
"//-----------------------" lines. I'm not sure if this is wanted. Maybe
someone of the experts can comment on this?
I would say yes, definitely. Vertical space is a non-renewable resource, as I think Michael so
nicely put it.
Such comments have no value if used between more or less all methods in a source file.
Removing them should be done exactly as you say, in a diff / patch by itself that does nothing
else, so that it is trivial to confirm that it indeed does nothing else.
They have value if they (or similar comments) if given some semantic meaning, for instance if used
to indicate parts of a source file that are differently licensed, cop-pasted from some other
source, or something like that. Of course, in such a case only if accompanied by a textual
explanation. Like "// ----- From here to the next // ----- the code is copied from libfoo version
4.2 which is licensed under a MIT/X11 license."
--tml
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.