Skip to content

silent extern error (PR#898)

1 message · Brian Ripley

Original
Brian Ripley
#
On Sat, 7 Apr 2001 bellis@hsph.harvard.edu wrote:

            

      
      
      
    

        
Sorry, but as __MAIN__ (not MAIN) is only defined in one C module internal
to R, why is it an error?  It would be an error if there ever was an
intention that module was to be in C++, but there is not.  The #ifdef
__cplusplus constructs are only there to allow the header files to be used
in C++ addons, and defining __MAIN__ is *not* part of the API.

In short, those two #ifdef's are mutually exclusive, so their order is
immaterial.  How can one order be an error?

            

      
      
      
    

        
However, *that* would be an error.  _MSC_VER may well be defined during the
actual compilation of R, and __MAIN__ is only defined in the compilation of
main.c.  So compilation would fail under VC++ (and other compilers which
define _MSC_VER).

(It is also possible for _MSC_VER to be defined but not have a value, so is
the test what you really meant?)

If we intended to make compilation for other C compilers easier we would
need to add lines like that to lots of the header files, and for more than
one compiler system.  However, we don't particularly want to make the use
of Arith.h under VC++ easier, as VC++ does not handle IEEE arithmetic
correctly!   In short, using Arith.h in addons compiled with VC++ is
likely to give incorrect results.

(We know from past experience that R can be compiled with VC++ (6.0sp3, to
be precise) but then fails make check, giving incorrect results in the
arithmetic tests and elsewhere.)