Revision cc8edaf7e8e8eff2fc61804b5f00f0d32db7887d authored by Rene Brun on 01 July 2004, 20:02:52 UTC, committed by Rene Brun on 01 July 2004, 20:02:52 UTC
I attached the modifications that are needed in gcc3strm.cxx to compile with gcc 3.4.0.
There are 2 problems.
One is that I could not find any trace of
ostream& operator<< (ostream&,const streampos&);
in the gcc 3.3.1 or the gcc 3.4.0 header files.
However it is in cint/lib/gcc3strm/iostrm.h and of course in gcc3strm.cxx
The second problem is more fundamental. In gcc 3.4.0, streamoff is defined
as (include/c++/3.4.0/bits/postypes.h)
#ifdef _GLIBCXX_HAVE_INT64_T
typedef int64_t streamoff;
#else
typedef long long streamoff;
#endif
in turn int64_t is platform dependent. In particular, on my node int64_t
is typedef to an int. In gcc 3.3.1, streamoff was a long.
This leads to the compilation error:
cint/src/gcc3strm.cxx: In function `int G__G__stream_6_4_0(G__value*, const char*, G__param*, int)':
cint/src/gcc3strm.cxx:149: error: 'const class std::fpos<mbstate_t>' has no member named 'operator long int'
As a quick fix, I simply replaced the implementatio of the wrapper so
that it does not do an explicit call to the operator long:
const fpos<mbstate_t>*pos = (const fpos<mbstate_t>*)(G__getstructoffset());
G__letint(result7,108,(long)(*pos));
instead of
G__letint(result7,108,(long)((const fpos<mbstate_t>*)(G__getstructoffset()))->operator long());
Of course, this is fundamentally wrong, since on a 64 bit platform, this will truncate the long long
into a long.
I am not sure what would be good solution (theoritically, it would require to make sure that the
dictionary for fpos<mbstate_t> is different (long vs long long) on the appropriate platforms).
git-svn-id: http://root.cern.ch/svn/root/trunk@9367 27541ba8-7e3a-0410-8455-c3a389f83636
1 parent 023bf10
TPoint.h
/* @(#)root/base:$Name: $:$Id: TPoint.h,v 1.2 2002/05/03 14:30:41 brun Exp $ */
/*************************************************************************
* Copyright (C) 1995-2000, Rene Brun and Fons Rademakers. *
* All rights reserved. *
* *
* For the licensing terms see $ROOTSYS/LICENSE. *
* For the list of contributors see $ROOTSYS/README/CREDITS. *
*************************************************************************/
#ifndef ROOT_TPoint
#define ROOT_TPoint
//////////////////////////////////////////////////////////////////////////
// //
// TPoint //
// //
// TPoint implements a 2D screen (device) point (see also TPoints). //
// //
// Don't add in dictionary since that will add a virtual table pointer //
// and that will destroy the data layout of an array of TPoint's which //
// should match the layout of an array of XPoint's (so no extra copying //
// needs to be done in the X11 drawing routines). //
// //
//////////////////////////////////////////////////////////////////////////
#ifndef ROOT_Gtypes
#include "Gtypes.h"
#endif
class TPoint {
public: // for easy access
#if !defined(WIN32) && !defined(G__WIN32)
SCoord_t fX; //X device coordinate
SCoord_t fY; //Y device coordinate
#else
long fX; //X device coordinate
long fY; //Y device coordinate
#endif
public:
TPoint() : fX(0), fY(0) { }
TPoint(SCoord_t xy) : fX(xy), fY(xy) { }
TPoint(SCoord_t x, SCoord_t y) : fX(x), fY(y) { }
~TPoint() { }
SCoord_t GetX() const { return (SCoord_t)fX; }
SCoord_t GetY() const { return (SCoord_t)fY; }
void SetX(SCoord_t x) { fX = x; }
void SetY(SCoord_t y) { fY = y; }
};
#endif
Computing file changes ...
