Patch dbus-binding-tool

keith preston keithpre at gmail.com
Thu Aug 24 07:03:45 PDT 2006


> 1. The patch is reversed, right?  I mean, it's meant to add the cast
> rather than removing it?


Yes, the patch add the cast in


2. This style of casting is obsolete in C++.  Modern compilers, at very
> strict settings, will issue warnings for it.  At extremely strict
> settings, such as the ones I normally use, it will even be an error.  So
> if this code is not meant ever to be compiled as C, you may want to use
> reinterpret_cast instead.  Of course that in turn won't wash with truly
> antiquated compilers...
>
>
I didn't know that this style of casting was obsolete.  My basic problem is
that the file that the binding tools generated can not be included in a c++
file because the code it generates is not c++ compliant.   This can be fixed
in two way.   Either add the cast to make the code compile under c++ or have
the binding tools generate two files.  A .c file that contains the code and
a .h file that just include the prototypes and can be included in a c++
file.

fyi  I am just using standard g++ to compile

Keith
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freedesktop.org/archives/dbus/attachments/20060824/fd1371a9/attachment.html


More information about the dbus mailing list