search

na0607 / jgrasstools

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

Add Nabla #20

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
add nabla

Original issue reported on code.google.com by daniele.andreis on 2 Aug 2010 at 9:40

Attachments:

GoogleCodeExporter commented 8 years ago 
The class doesn't follow the conventions.
In a quick look I found things like:
"Raster elevationIter"

and also the javadocs are not up to date.

Daniele, please be a bit more focused on these things.

Won't apply that patch as it is now.

Also, why use and extra variable DOUBLE_NO_VALUE and do
private static final double DOUBLE_NO_VALUE = JGTConstants.doubleNovalue;
when you can statically import JGTContstans and use doubleNovalue?

Original comment by andrea.a...@gmail.com on 2 Aug 2010 at 1:05