On Wed, May 18, 2011 at 10:29 AM, Glynn Clements <gl...@gclements.plus.com> wrote: > > Markus Metz wrote: > >> >> The NULL value of the reclass map is not recognized as NULL value, but >> >> instead treated as a valid number. >> >> >> >> This seems to be a bug in the GRASS library, specifically in >> >> G_get_null_value_row() and has probably been around since GRASS 6.3. >> > >> > AFAICT, it has been around as long as G_get_null_value_row() has >> > existed. G_get_null_value_row() returns the raw data, without any >> > reclass applied. >> > >> > As it stands, this function probably isn't of much use. >> >> Remove? > > No, fix (as per r46323). > Done in r46345, r46346.
> If you only care about whether a cell is null or non-null, reading and > decoding the actual data is needless overhead. > > If you need to read the data anyhow, then use that rather than reading > the null data a second time (the raw data is cached, but conversion > and masking will be re-done). > > r.out.gdal was the latter case, so it shouldn't have been using that > function. OGSF is also the latter case, and should be fixed. > OGSF fixed in r46347 for trunk. This is an enhancement, no bug because [Rast|G]_get_null_value_row is fixed, so the OGSF change does not need to be backported. Markus M _______________________________________________ grass-user mailing list grass-user@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-user