#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>
_______________________________________________
grass-dev mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/grass-dev
