On Apr 21, 2008, at 12:22, Vincent Hennebert wrote:

My reaction was not so much triggered by this (indeed) minor issue, as
by the fact that once again I was being answered “feel free to improve” when asking for cleanup/documentation, and I got tired of it. Each time, they were small issues that were not a big deal in themselves, but when
combined together make the code IMO unnecessarily difficult to
understand. We are dealing with a very complex piece of software, so it
looks even more important to me to keep the code as clean and readable
as possible. Actually I consider this to be my duty whenever I commit
something, and not spending the little time this requires for that
eludes me completely.

FWIW: I think you do have a major point here. If it really concerns documentation, it's /always/ better if this is kept up-to-date by the dev that committed the changes in the first place. Someone else is much more likely to make errors in interpretation. Only the creator knows precisely what a certain part of the code is meant to do.

Cleanup is maybe more subjective than documentation (*), but that said, some simple rules of thumb would probably not hurt here.

In this last case, however, I got the impression (as Jeremias, I presume) that it was not about documentation or cleanup. I initially read it more as a suggestion --What would you think if...? As such, Jeremias' reaction was very understandable to me, and it surprised me to see it escalate further.

(*) as in: My apartment is clean enough for me, but I know of people that would get a heart-attack if they entered here... ;-)


Cheers

Andreas

Reply via email to