Re: [Qemu-devel] [PATCH] clean build: Add bt-host.h

[andrzej zaborowski <balrogg@gmail.com>]

2009/3/8 Jan Kiszka <jan.kiszka@web.de>:
> Aurelien Jarno wrote:
>> On Sun, Mar 08, 2009 at 03:56:12PM +0100, Jan Kiszka wrote:
>>> Jan Kiszka wrote:
>>>> Silence compiler warning by providing a prototype in the CONFIG_BLUEZ
>>>> case (hw/bt.h provides it otherwise).
>>>>
>>>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>>>> ---
>>>>
>>>>  bt-host.c |    2 ++
>>>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/bt-host.c b/bt-host.c
>>>> index 07679f6..066757a 100644
>>>> --- a/bt-host.c
>>>> +++ b/bt-host.c
>>>> @@ -31,6 +31,8 @@
>>>>  #  include <bluetooth/bluetooth.h>
>>>>  #  include <bluetooth/hci.h>
>>>>  #  include <bluetooth/hci_lib.h>
>>>> +/* Silence compiler warning */
>>>> +struct HCIInfo *bt_host_hci(const char *id);
>>>>  # else
>>>>  #  include "hw/bt.h"
>>>>  #  define HCI_MAX_FRAME_SIZE       1028
>>> Thanks for applying the other patches, but this tiny one always seems to
>>> be ignored - for unknown reasons. :)
>>>
>>
>> On my side, I consider that a hack, not a fix. We should make sure the
>> correct header is included so that this function is declared.
>>
>
> As always: no comment means no progress. This one better?

Is it?
struct HCIInfo is defined in net.h so maybe this one belongs there too.

I'd much rather turn off the unhelpful warnings, some of them caused
developers to add code that is both more hacky and slower.  I also
think we could save many header files like the hw/virtio-*.h, at this
granularity they don't help with compile times which was the reason
vl.h was split.

Regards