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