Yes, except I would argue that such work is best left on the HEAD rather than applied to a maintenance branch until necessitated by bug reports on that branch, or at least the knowledge that such a bug definitely exists and can be reproduced on that branch. I'm all for improving the code base. We just need to excercise caution changing code on maintenance branches unnecessarily. The HEAD is another matter, IMO go nuts! -Casey On Monday 16 December 2002 12:29 pm, Jeremy Hylton wrote:
The nice thing about fixing this bug is that the code continues to be improved. The last round of beautification made the code substantially more readable. This was a direct result of trying to track down a reference count problem. The problem proved to be elsewhere, but it was nearly impossible for me to inspect the old coptimizations.c code to figure out if the problem was there.
If we make the code easier to read, we reduce the burden we all share more keeping the software working. Inscrutable code hides subtle bugs and encourages work arounds that make the whole system harder to understand.
Jeremy