Return-Path: MIME-Version: 1.0 Sender: rohan16garg@gmail.com In-Reply-To: References: <1326702341.3360.107.camel@pohly-mobl1.fritz.box> <1326705845.6454.274.camel@aeonflux> <1326708597.3360.133.camel@pohly-mobl1.fritz.box> <1328439363.32199.20.camel@pohly-mobl1.fritz.box> <1328439464.32199.22.camel@pohly-mobl1.fritz.box> <1328441601.321.1.camel@pohly-mobl1.fritz.box> From: Rohan Garg Date: Sun, 5 Feb 2012 17:53:24 +0530 Message-ID: Subject: Re: [PATCH] bluetooth.h: fix compile issue in C++ with ifdef To: Patrick Ohly Cc: Marcel Holtmann , linux-bluetooth@vger.kernel.org, Milan Crha , ying.an.deng@intel.com, ulf.hofemeier@intel.com, ning.w.wang@intel.com, Tino Keitel Content-Type: text/plain; charset=UTF-8 List-ID: Seems like your patch causes a build failiure : http://paste.ubuntu.com/829971/ Best Rohan Garg On Sun, Feb 5, 2012 at 5:03 PM, Rohan Garg wrote: > Ah, understood :) > > Best > Rohan Garg > > > > On Sun, Feb 5, 2012 at 5:03 PM, Patrick Ohly wrote: >> On Sun, 2012-02-05 at 16:37 +0530, Rohan Garg wrote: >>> I've not gone through the entire patch, but if the problem is merely >>> the fact that one cannot convert a void* to anything in C++, why not >>> use a static_cast in syncevolution wherever the struct is used? >> >> The problem is not in the way how SyncEvolution uses the structs. The >> problem is in the header files; it also occurs in any other C++ app >> including the bluetooth.h file. >> >> -- >> 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. >> >>