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;
_______________________________________________
Gnash-commit mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/gnash-commit
