Re: [Gnash-commit] gnash ChangeLog backend/render_handler.h libgeo...

[strk]

On Thu, Mar 01, 2007 at 11:06:32AM +0000, Udo Giacomozzi wrote:

>    virtual bool bounds_in_clipping_area(const InvalidatedRanges& ranges) {
> -     for (int rno=0; rno<ranges.size(); rno++) 
> +     for (unsigned int rno=0; rno<ranges.size(); rno++) 
>               if (bounds_in_clipping_area(ranges.getRange(rno)))
>                       return true;

This was the reason to provide a 'visitor' model for SnappingRanges2d, hiding
the underlying container type. This use uses a random access iterator when
it doesn't even need to. For a source-level compatibility with future container
switch we could expose the iterator interface, or provide the visitor pattern.

Yest another possibility (the cleanset) would be to do everything inside 
SnappingTRanges,
as I see _clipbounds are basically a SnappingRange2d now (right?)

--strk;