The envelope class (used here for the bbox variable) has a very specific
definition for isNull:
Envelope.isNull() - un initialised or envelope of an empty geometry
For more information check the javadocs:
http://www.vividsolutions.com/jts/javadoc/com/vividsolutions/jts/geom/Envelope.html
Jody Garnett
On Wed, Dec 4, 2013 at 4:59 PM, cai yuanlong <caiyuanl...@126.com> wrote:
> Dear develepers,
> I recently studied source code of geotools and found a mistake: class
> org.geotools.data.shapefile.ShapefileFeatureSource, line 237: } else
> if (getDataStore().isIndexed() && !bbox.isNull(). If bbox is null, it will
> throw an NullPointerException. So I think it should be } else if
> (getDataStore().isIndexed() && bbox != null && !bbox.isNull().
>
> sincerely
> cai yuanlong
>
>
>
> --
> View this message in context:
> http://osgeo-org.1560.x6.nabble.com/Mistake-found-in-GeoTools-tp5092553.html
> Sent from the geotools-gt2-users mailing list archive at Nabble.com.
>
>
> ------------------------------------------------------------------------------
> Sponsored by Intel(R) XDK
> Develop, test and display web and hybrid apps with a single code base.
> Download it for free now!
>
> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
> _______________________________________________
> GeoTools-GT2-Users mailing list
> GeoTools-GT2-Users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
>
------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT
organizations don't have a clear picture of how application performance
affects their revenue. With AppDynamics, you get 100% visibility into your
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-GT2-Users mailing list
GeoTools-GT2-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
