search

bilaldursun1 / nettopologysuite

Automatically exported from code.google.com/p/nettopologysuite
0 stars 0 forks source link

Polygon constructor should take ILinearRing, not LinearRing #47

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
This one should be really easy to "fix" (in Polygon.cs). 

Constructors:
- public Polygon(LinearRing shell)
- public Polygon(LinearRing shell, GeometryFactory factory)
should be: 
- public Polygon(ILinearRing shell)
- public Polygon(ILinearRing shell, GeometryFactory factory)

Original issue reported on code.google.com by skipperkongen on 24 Jul 2009 at 9:58

GoogleCodeExporter commented 9 years ago 
This is of course not a defect, but an enhancement. Sorry for setting wrong 
type :)

Original comment by skipperkongen on 24 Jul 2009 at 9:59

GoogleCodeExporter commented 9 years ago 
fixed with changeset 450
actually ow methods are like this:
public Polygon(ILinearRing shell)
public Polygon(ILinearRing shell, IGeometryFactory factory)

thanks for the segnalation

Original comment by diegogu...@gmail.com on 25 Jul 2009 at 9:02