From: Edwin C. <com...@gm...> - 2010-03-02 21:00:22
|
Hi Dave, I will be off radar for a while, since my wife gave birth to a wonderful daugther two days ago. http://www.headspingames.com/bente/familieportret.jpg I thought redraw was a property of HttpRequestLayer and it's subclasses. It was on Layer before for convenience (I thought) and maybe we should put it there. Sounds ok. Greetings, Edwin On 1 March 2010 23:46, Dave Koberstein <da...@ko...> wrote: > Hi Edwin, > > I haven't pushed this commit out yet as I'd like to check with you first. > > Layer.redraw is supported on all layers so I added a redraw method in class > Layer. Also, this method has a return boolean of redrawn so adding that to > LayerImpl required changing the return type of HTTPRequestLayer.redrawn from > void to boolean. If anyone is using this it could create a problem with the > signature change. > > Most of the layer subclasses don't support an argument of force. So I > implemented the Layer.redraw with no argument and then passed through true > to LayerImpl.redraw so that we wouldn't need to change or add a method to > LayerImpl. > > Does that sound ok? > > Davek > |