[GRASS-dev] [GRASS GIS] #2820: v.surf.idw results seem seriously wrong and don't match r.surf.idw results

GRASS GIS trac at osgeo.org
Sat Dec 19 07:42:38 PST 2015


#2820: v.surf.idw results seem seriously wrong and don't match r.surf.idw results
---------------------+-------------------------------------------------
  Reporter:  lrntct  |      Owner:  grass-dev@…
      Type:  defect  |     Status:  new
  Priority:  normal  |  Milestone:  7.0.3
 Component:  Vector  |    Version:  7.0.1
Resolution:          |   Keywords:  v.surf.idw r.surf.idw interpolation
       CPU:  x86-64  |   Platform:  Linux
---------------------+-------------------------------------------------

Comment (by mlennert):

 Replying to [comment:21 wenzeslaus]:
 > Replying to [comment:19 pkelly]:
 > > and more in keeping with the antiquated style of the existing C code!
 >
 > Sorry, but using the same logic we should also introduce some cryptic
 names and perhaps memory leaks. What we should value in relation to code
 is readability and reusability. What is means in this particular case
 might be debatable, but in general we should definitively prefer
 ''readable code'' rather than ''antiquated style''.

 I agree with you in principle, although when you have a module written in
 one style, it might be a good idea to not mix too much with another, as
 that might actually confuse and cause less readability.

 And in this particular case, I do find the static declaration within the
 function more easily understandable and readable than the pointer magic
 spread over all the code.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2820#comment:22>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list