[GRASS-dev] Re: [GRASS GIS] #392: backport G_is_c_null_value() to devbr6

[GRASS GIS]

#392: backport G_is_c_null_value() to devbr6
----------------------+-----------------------------------------------------
  Reporter:  hamish   |       Owner:  grass-dev at lists.osgeo.org
      Type:  task     |      Status:  new                      
  Priority:  major    |   Milestone:  6.4.0                    
 Component:  default  |     Version:  svn-develbranch6         
Resolution:           |    Keywords:                           
  Platform:  All      |         Cpu:  All                      
----------------------+-----------------------------------------------------
Comment (by hamish):

 Replying to [comment:2 glynn]:
 > I suggest completely replacing null_val.c with the 7.0 version,
 > and editing gisdefs.h to match. Either that, or leave it as-is.

 In that case, for the rc1 release I'd say leave it as is. It's too core a
 fn to mess with so close to release time.

 For rc2, to completely replace null_val.c or not? I am still a bit unsure-
 is there actually a bug in the current devbr6 version or is the idea to
 keep the methods in sync to ease future maintenance?

 Forward compatibility with grass7 raster maps is of course a good thing,
 if only on the binary data level (ie even if dir struct changes). Or are
 the updates helpful for r.external null cells?

 note that r33717 from truck was already backported in r33752.
 (Make G_is_[fd]_null_value() check for any NaN, not just all-ones)


 ?
 Hamish

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/392#comment:3>
GRASS GIS <http://grass.osgeo.org>