Martin Vermeer wrote:
> OK. 1 is just verifying if you can remove it, 2 is tricky but should
> be possible (thorough testing!) 3: go right ahead if you think it is
> The Right Thing. 4. ... testing needed. I suspect colour lookup
> table refresh/caching (is there still such a thing?)

Patch attached addresses points 1-3 and results in MUCH simpler code 
although it shaves the total number of lines down by 30 only.

I'll hold off applying it to give you some chance to test it out 
yourself. Seems polite ;-)

-- 
Angus

Attachment: branches.diff.bz2
Description: BZip2 compressed data

Reply via email to