On 12/04/2009 08:50 PM, Andre Poenitz wrote:
On Fri, Dec 04, 2009 at 04:03:44AM +0100, [email protected] wrote:
This is an abuse of updateLabels(), in a way, but updateLabels() long
ago became the general recurse-through-the-Buffer routine, and to
implement the sort of thing I want to do here in validate(), say, much
of the code in updateLabels()---in particular, the counter-update
code---would have to be duplicated. So I believe this is the best, and
easiest, way to go.
I am not really a fan of "random" boolean arguments, with default value
even tacked onto "unrelated" functions. Is this really necessary?
I don't know if it's absolutely necessary, but it seems a bad idea to do
a bunch of calculations every time through updateLabels() that we only
need to do when we are exporting. I thought about other solutions, but
this was the only one that made sense in the end.
I've tried to clean up the argument structures a bit.
rh
