[haiku-commits] Re: r35905 - in haiku/trunk: headers/os/interface headers/private/interface src/kits/interface src/servers/app src/servers/app/drawing ...

  • From: Stephan Assmus <superstippi@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 18 Mar 2010 21:28:09 +0100

On 2010-03-18 at 19:01:00 [+0100], Stefano Ceccherini 
<stefano.ceccherini@xxxxxxxxx> wrote:
> 2010/3/18  <superstippi@xxxxxx>:
> 
> >  * Wrapped iterating the BShape into try/catch blocks in ShapeIterator. 
> >  But
> >   I really don't understand the purpose of the class in the first place.
> 
> I think it's useful in case you implement your own way to render
> BShapes. We used it in ServerPictures, for example.

Yes, it should be only used in ServerPicture, since it's a local class in 
that file. But it's main purpose seems to be to iterate an existing BShape 
to copy all it's data, and then draw this shape with the appropriate view 
to screen transformation. So it seems to be an expensive way to memcpy() 
the shape data and apply the transformation on the point data. But since 
the transformation is no longer necessary, I believe it can be thrown out.

Best regards,
-Stephan

Other related posts: