Re: Support of gdb for Windows 64 native systems

[Eli Zaretskii <eliz@gnu.org>]

> Date: Fri, 12 Oct 2007 09:11:32 -0700
> From: Joel Brobecker <brobecker@adacore.com>
> Cc: Daniel Jacobowitz <drow@false.org>, Kai.Tietz@onevision.com,
> 	gdb@sourceware.org
> 
> > Actually, wouldn't it be better to separate the two completely?  That
> > should avoid a lot of ugly ifdefs, and make each native backend much
> > cleaner, I think.
> 
> The issue with that is that we'll end up duplicating a bit of code.

So what? the duplicates will never be linked into the same build.  We
already have duplicate code in targets that are alike, one more cannot
hurt.

In my experience, mixing two different targets is asking for trouble
in the long run.

> In our merge, I counted 5 instances of "ifdef/ifndef __MINGW32__

You need to count "ifdef __CYGWIN__" as well.

> in total, all of them in win32-nat.c:
>   - One to define MAXPATHLEN: Should really be done in a proper way,
>     so should go away

I don't see this one in the current CVS; am I missing something?

> The rest seems to be in i386-win32-tdep.c which is a separate file.

I don't see this file, either, so I cannot comment on that.