
27 Oct
2012
27 Oct
'12
12:32 p.m.
Hi
Afer a quick code inspection I would say: Looks good! The only thing that must be checked carefully is the new meaning of the isClosed() method. I am quite sure that there are some places at least in the multipolygon code that relies on isClosed() == true => getPoints().get(0).equals(getPoints().get(getPoints().size()-1))
Yes agreed. The first thing I'm going to do is find out how common it is that isClosed() != isClosedOld(). Perhaps the multi-poly code can make good use of isComplete()? ..Steve