Re: [Qemu-devel] [6736] Sparse fixes: NULL use, header order, ANSI prototypes, static

[Anthony Liguori <anthony@codemonkey.ws>]

Blue Swirl wrote:
> Revision: 6736
>           http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6736
> Author:   blueswir1
> Date:     2009-03-07 15:32:56 +0000 (Sat, 07 Mar 2009)
> Log Message:
> -----------
> Sparse fixes: NULL use, header order, ANSI prototypes, static
>
> Fix Sparse warnings:
>  * use NULL instead of plain 0
>  * rearrange header include order to avoid redefining types accidentally
>  * ANSIfy SLIRP
>  * avoid "restrict" keyword
>  * add static
>   

This broke the Windows build.  The reason is that you moved all system 
headers to be included before any QEMU headers.  While I agree with this 
in principle, Windows headers are sloppy and don't include their 
dependencies.  Right now, we rely on qemu-common.h being included in the 
top in order to get #include <windows.h> every where.

To complicate matters further, we define some special versioning magic 
to get windows IPv6 support working.  I don't want to add that full blob 
to every .c file so it makes sense to use a qemu include file (like 
qemu-common.h).  That defeats that purpose of your refactoring though so 
I wanted to see what you thought about it.

Regards,

Anthony Liguori