[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#191737: acknowledged by developer (Re: Bug#191737: xinerama.h is missing extern C { for C++ compiling)



> On Sat, May 03, 2003 at 08:35:21PM +1000, Ken Foskey wrote:
> > The Xinerama.h file is missing the following construct from the top and
> > bottom of the file.  This allows it to be safely used in C++ programs,=20
> > this is a build bug for the very latest OpenOffice.org.  This is also
> > the case on RedHat, Suse, etc so it is definitely an upstream problem.
> >=20
> > #ifdef __cplusplus
> > extern "C" {
> > #endif
> >=20
> > #ifdef __cplusplus
> > };
> > #endif
> 
> Uh, how is this X11's fault?
> 
> extern "C" {
> #include <Xinerama.h>
> }
> 
> Closing this spurious bug. Xinerama is a C library, so you should take
> the appropriate cautions when interfacing with it, not the other way
> around.

C++ programmers cannot know whether a library is C only of C++ in every
instance.  The only error we get is a obscure link message of a missing
_Z16XineramaIsActiveP9_XDisplay

Not very easy to debug, easy to fix the include so that people do not
get tripped up though.  Also SUSE have included a fix at source which is
why the problem is random.

http://ftp.suselinux.hu/suse/i386/supplementary/X/XFree86/XFree86-4.2.0-SuSE/CHANGES

Where do I go to raise the problem direct at source?

-- 
Thanks
KenF
OpenOffice.org developer





Reply to: