Re: [R-pkg-devel] Package valgrind problem I can't solve: Direction?

2017-10-29 Thread Iñaki Úcar
2017-10-30 0:54 GMT+01:00 Peter Dunn :
> Thanks for this direction.
>
> But I am still confused, as the problem persists after attending to any
> possible issue: That piece of identified code only has four variables, and I
> can account for them all.
>
> The two variables its and aimrerr are now explicitly initialised. So they
> cannot be the problem.
>
> The variables maxit and relerr are inputs to the subroutine... and both are
> explicitly initialised in the only subroutine which calls smallp. So I can’t
> see how that can be a problem. (Clearly they cannot be initialised in the
> subroutine smallp or their input values are overwritten.)

Maybe the location is wrong. What about the "result" variable in the
"smallp" subroutine? Shouldn't it be initialised?

> Any more idea for where I should be looking, or how I can progress this?
> Using the flags below when gfortran is called, with -O0 –g included,
> produced no errors or warnings at all.

Given that the issue disappears depending on the optimisation level,
maybe it is a false error.

Iñaki

__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel

Re: [R-pkg-devel] -Wlong-long

2017-10-29 Thread Uwe Ligges



On 29.10.2017 19:25, Tim Keitt wrote:

Anyone have a fix for:

Found the following significant warnings:
   ../windows/libpq-9.5.2/include/pg_config_ext.h:8:28: warning: ISO
C++ 1998 does not support 'long long' [-Wlong-long]


? I tried prepending __extension__ to the definition and adding CXX_STD =
CXX11 to Makevars. This is external code from postgres that is used to
build libpq.



See Writing R Extesions on how to ask for C++11

Best,
Uwe Ligges




THK


http://www.keittlab.org/

[[alternative HTML version deleted]]

__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel



__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel


[R-pkg-devel] -Wlong-long

2017-10-29 Thread Tim Keitt
Anyone have a fix for:

Found the following significant warnings:
  ../windows/libpq-9.5.2/include/pg_config_ext.h:8:28: warning: ISO
C++ 1998 does not support 'long long' [-Wlong-long]


? I tried prepending __extension__ to the definition and adding CXX_STD =
CXX11 to Makevars. This is external code from postgres that is used to
build libpq.

THK


http://www.keittlab.org/

[[alternative HTML version deleted]]

__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel