[PATCH] winsock include fixes

Daniel Jacobowitz dan@codesourcery.com
Fri Mar 26 14:17:00 GMT 2010


On Fri, Mar 26, 2010 at 12:54:10PM +0200, Ozkan Sezer wrote:
> Hi: Here is a patch fixing the winsock include problem
> which appeared after the recent winsock header work in
> mingw-w64-headers.

This patch isn't OK, because it fixes an inclusion order problem by
moving things around without a comment.  Everywhere else the
"gdb_XXX.h" headers are included in a group, last.  So someone's going
to clean up these files and reintroduce the problem.

It sounds like part of the problem is the inclusion of winsock2.h in
gdb_select.h, and windows.h in serial.h.  Should we include both
headers in both files to avoid the problem?

-- 
Daniel Jacobowitz
CodeSourcery



More information about the Gdb-patches mailing list