On Aug 17, 2008, at 4:59 PM, Maximilian Albert wrote:
Jon A. Cruz schrieb:
Actually what happened is that the eraser code was copied from the dyna-draw code. Then as we go that is being refactored, with common code going into a common base class and the specific code being retuned and left different in each of the two original classes
So yes, you did spot a bit of duplication that needs refactoring. However it is an area that I'm working on incrementally, so you won't have to do direct cleanup yourself.
Awesome. Sorry for the hassle, and apologies if I came across a bit overzealous. I certainly didn't want to mess with someone else's work-in-progress. Just stumbled across this and thought I'd ask.
Oh, no problems at all.
For things such as this, just asking is a very good step. Sometimes the area is in the middle of updating, but then some other times it has been left and needs someone else to pick up fixing it.
Sending a quick note to check is probably the best way to know how to proceed.