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


On 01/07/2015 11:37 AM, Bjoern Michaelsen wrote:
Personal opinion: The double bookkeeping of log areas is bad, and people ignore
it because its a direct violation of DRY. IMHO if that plugin is active, it
should just add the missing log areas. Whoever is caring about consistency in
the log areas then simply needs to look at that file once in a while and tweak
existing log areas into a consistent state. Consistency is not something that
can be achieved by forcing over 200 people to do edits to one file(*), it needs to
be maintained by 1-2 people with a good context on what is there and what the
customs are.

Not sure about the double-bookkeeping and DRY-ness arguments. (For amusement, there's even a real, printed book out there, touting itself as a guidebook on contemporary C, that recommends using string literals instead of enums; you can't make that up.)

Regarding the remainder, my perspective is somewhat more relaxed: Log area inconsistencies become a problem when one wants to filter on them. But at least as long as the logging is only enabled in (developers') debug builds, a developer who wants to do filtering and is bitten by some inconsistency can just clean up the relevant code. So as long as the proliferation of log areas does not grow too much out of control, I think we can happily live with the status quo.

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.