[Rd] missing extern in GraphicsBase.h

Paul Murrell paul at stat.auckland.ac.nz
Thu Feb 15 22:21:32 CET 2018


I have committed the suggested "extern" patch.

Could you please confirm that this fixes the issue on Solaris (and 
anything else you can test) ?



On 16/02/18 03:39, dmitrii.pasechnik at maths.ox.ac.uk wrote:
> Dear all,
> in src/include/GraphicsBase.h one has a declaration
> int baseRegisterIndex;
> the same as in src/main/devices.c
> which causes problems on Solaris, see bug #17385,
> and other platforms with "unusual" linkers, see bug #16633.
> By right, global variables like baseRegisterIndex are to be
> declared just once, and not in a header file, but in a *.c file.
> Then, to use them elsewhere in the code, one declares them as
> extern in the header.
> (as proposed on #17385)
> Otherwise one has an undefined behaviour,
> some linkers might silently prepend extern, some not...
> May I humbly request attention to this bug
> (which is classified as UNCONFIRNMED---and indeed it needs an extra
> effort to reproduce the error on, say, Linux --- but it really is an obvious C
> bug, which will rear its ugly head sooner or later again)
> Thanks,
> Dmitrii
> ______________________________________________
> R-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel

Dr Paul Murrell
Department of Statistics
The University of Auckland
Private Bag 92019
New Zealand
64 9 3737599 x85392
paul at stat.auckland.ac.nz

More information about the R-devel mailing list