[GRASS-dev] Backport r.li memory allocation fixes?

[Vaclav Petras]

On Thu, Oct 16, 2014 at 3:31 AM, Markus Metz <markus.metz.giswork at gmail.com>
wrote:

> On Wed, Oct 15, 2014 at 7:52 AM, Markus Neteler <neteler at osgeo.org> wrote:
> > On Wed, Oct 15, 2014 at 3:35 AM, Vaclav Petras <wenzeslaus at gmail.com>
> wrote:
> >> Hi Markus and others,
> >>
> >> I would do a backport or r61812 but I'm not sure if it works correctly.
> It
> >> seems to work and I got the same results as with the old version but I
> >> remember that you (Markus) had some doubts at one point. Now I actually
> see
> >> that you said you will backport it, so if you have your plan disregard
> this
> >> message.
> >
> > Please backport it.
>
> ... after removing the C++ style comment in raster/r.li/r.li.daemon/avl.c.
>
> Done. It was double wrong, it was just the old code which I forgot to
delete.

Anyway, I'm not sure if there is some reason not to use C++ comments since
we mostly relay on POSIX capable compiler (which will probably handle //,
just a guess).

http://trac.osgeo.org/grass/changeset/62280


> Markus M
>
>
> > It should probably also go into G6 which is almost
> > identical.
> >
> >> I also wanted to use the things bellow to write a test but I don't know
> how.
> >
> > Would this script give some ideas?
> > raster/r.li/r.li.testing_nc_asc.sh
> >
> > Markus
> > _______________________________________________
> > grass-dev mailing list
> > grass-dev at lists.osgeo.org
> > http://lists.osgeo.org/mailman/listinfo/grass-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20141017/a525601f/attachment-0001.html>