On Mon, 2012-01-16 at 11:09 +0100, Patrick Ohly wrote: > On Mo, 2012-01-16 at 10:24 +0100, Marcel Holtmann wrote: > > Also this topic > > has been raised before and I need confirmation for a GCC guru to confirm > > that this does exactly the same all all platforms. > > Now that you mentioned it I found the previous patch: > http://article.gmane.org/gmane.linux.bluez.kernel/20276/match=invalid > +conversion+void+bt_get_le64+anonymous+struct > > Note that my patch is different. It keeps the "struct > __attribute__((packed))" magic and merely changes the (void *) typecast > to something that works in C++ and C. How do we move forward with this? More and more distros are picking up the broken header file. In MeeGo and Tizen we applied the patch that I had suggested. Milan, do you know what Fedora is doing? Tino, what about Debian? Roran, you had it in Ubuntu Precise Pangolin. Care to file a bug there, if there isn't one already? Would it be more acceptable for upstream to put the modified macros into an ifdef so that they are really only used in C++, keeping the code for C as it is now? I'll send such a patch immediately. -- Best Regards, Patrick Ohly The content of this message is my personal opinion only and although I am an employee of Intel, the statements I make here in no way represent Intel's position on the issue, nor am I authorized to speak on behalf of Intel on this matter. -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
