This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] winsock include fixes


On Fri, Mar 26, 2010 at 4:17 PM, Daniel Jacobowitz <dan@codesourcery.com> wrote:
> 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
>

That may be a solution, too: Inclusion of winsock2.h
should be harmless in all cases when USE_WIN32API
is defined.

--
Ozkan


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]