Re: usbutils 0.82 release

[Xiaofan Chen <xiaofanc@gmail.com>]

On Fri, May 8, 2009 at 2:40 PM, Xiaofan Chen <xiaofanc@gmail.com> wrote:
> On Fri, May 8, 2009 at 12:01 PM, Greg KH <greg@kroah.com> wrote:
>> On Thu, May 07, 2009 at 07:51:23PM +0800, Xiaofan Chen wrote:
>>> Not sure if this is a real problem or not. There is a compiler warning
>>> for gcc 4.4.0.
>>>
>>> gcc -DHAVE_CONFIG_H -I.   -DDATADIR=\"/usr/share\"   -g -O2 -MT
>>> lsusb-names.o -MD -MP -MF .deps/lsusb-names.Tpo -c -o lsusb-names.o
>>> `test -f 'names.c' || echo './'`names.c
>>> names.c: In function ‘names_init’:
>>> names.c:803: warning: assignment makes pointer from integer without a cast
>>
>> Care to provide a patch for this?
>
> To be honest, I do not know the reason of the warning. It looks fine to me
> but I am not a real programmer.
>
> In fact, I tried version 0.73 and there is no warning there and the file name.c
> is the same between 0.73 and 0.81/0.82.
>

I still do not know the exact reason but it seems to be related to the
generated config.h file or zlib.

If I use the file config.h from 0.73, there will be no warning. It is
longer than the one generated in 0.82 or git.

And if I undefine HAVE_LIBZ in names.c, there will be no warning.

The warning is not only in gcc 4.4.0, but also in gcc 4.3.3.

-- 
Xiaofan http://mcuee.blogspot.com