public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@cygnus.com>
To: Paul Hilfinger <hilfingr@gnat.com>
Cc: Christopher Faylor <cgf@redhat.com>,
	Danny Smith <danny_r_smith_2001@yahoo.co.nz>,
	gdb@sources.redhat.com, mingw-users@lists.sourceforge,
	dewar@gnat.com
Subject: Re: [Mingw-users] Re: _WIN32?
Date: Thu, 03 May 2001 13:41:00 -0000	[thread overview]
Message-ID: <3AF1C25B.5040304@cygnus.com> (raw)
In-Reply-To: <20010503082444.40BDEF28A1@nile.gnat.com>

> ACT, Inc. is also committed to producing or assisting in a MINGW32
> port of GDB5, and contributing the necessary patches.  However, I
> don't have a timetable for this yet.
> 
> In any case, again this argues against removing the _WIN32 conditionals.

I'm not so sure.  It depends on why/how the tests are there. For instance:


#if defined(_WIN32) || defined(__MSDOS__) || defined(__CYGWIN__):

Remember, GDB is moving away from OS specific tests and towards 
autoconf/feature based tests.  Things like this should be replaced by 
something like WITH_REALLY_TRICKY_FILESYSTEM_SUPPORT test (and 
eventually changed to a runtime switch?).


main.c:
   /* The default command loop.
      The WIN32 Gui calls this main to set up gdb's state, and
      has its own command loop. */
#if !defined _WIN32 || defined __GNUC__
   /* GUIs generally have their own command loop, mainloop, or
      whatever.  This is a good place to gain control because many
      error conditions will end up here via longjmp(). */
The _WIN32 test here is probably for wingdb (and old, long dead, MS 
Windows based GDB) and not for mingw.  There are plenty of other 
examples of the same thing.

For reference, there are ~38 uses of _WIN32 in the sources, ~18 don't 
appear to be accompanied by #ifdef MSDOS, CYGWIN or GO32. It is those 18 
that I think should be deleted.

	Andrew



  reply	other threads:[~2001-05-03 13:41 UTC|newest]

Thread overview: 121+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200105010009.RAA12115@tully.CS.Berkeley.EDU>
2001-05-03  1:24 ` Paul Hilfinger
2001-05-03 13:41   ` Andrew Cagney [this message]
2001-05-03 14:15     ` Danny Smith
2001-05-03 15:24       ` Andrew Cagney
2001-05-03 15:54         ` Christopher Faylor
2001-05-03 16:11           ` Andrew Cagney
2001-05-04  2:17             ` Eli Zaretskii
2001-05-04  2:19         ` Eli Zaretskii
2001-05-04  8:45           ` Christopher Faylor
2001-05-04 10:18             ` Eli Zaretskii
2001-05-04 12:04               ` Christopher Faylor
2001-05-07  9:08         ` Eli Zaretskii
2001-05-08  4:38           ` DOS/Windows-specific code (was: _WIN32?) Eli Zaretskii
2001-05-08  4:40           ` DOS/Windows-specific code: cli-cmds.c Eli Zaretskii
2001-05-08  5:07             ` Registers are not available in Cygwin leonp
2001-05-08  7:16             ` DOS/Windows-specific code: cli-cmds.c Christopher Faylor
2001-05-08 17:51               ` Andrew Cagney
2001-05-08 17:53                 ` Christopher Faylor
2001-05-09  3:02                 ` Eli Zaretskii
2001-05-10  8:33                   ` Andrew Cagney
2001-05-08 17:56             ` Andrew Cagney
2001-05-09  3:03               ` Eli Zaretskii
2001-05-09  9:10                 ` Andrew Cagney
2001-05-08  4:42           ` DOS/Windows-specific code: exec.c and symfile.c Eli Zaretskii
2001-05-08  7:43             ` Elena Zannoni
2001-05-08 17:58             ` Andrew Cagney
2001-05-09  3:03               ` Eli Zaretskii
2001-05-09  7:07                 ` Christopher Faylor
2001-05-08  4:43           ` DOS/Windows-specific code: inflow.c Eli Zaretskii
2001-05-08  7:54             ` Christopher Faylor
2001-05-08 11:15               ` Eli Zaretskii
2001-05-09 13:10                 ` Christopher Faylor
2001-05-08  4:46           ` DOS/Windows-specific code: main.c Eli Zaretskii
2001-05-08  7:58             ` Christopher Faylor
2001-05-08  8:17               ` Keith Seitz
2001-05-08  8:19                 ` Christopher Faylor
2001-05-08  4:46           ` DOS/Windows-specific code: maint.c Eli Zaretskii
2001-05-08 23:40             ` Kevin Buettner
2001-05-09  3:22               ` Eli Zaretskii
2001-05-10  8:33             ` Andrew Cagney
2001-05-10  8:53               ` Eli Zaretskii
2001-05-08  4:47           ` DOS/Windows-specific code: source.c Eli Zaretskii
2001-05-08  9:30             ` DJ Delorie
2001-05-08 11:18               ` Christopher Faylor
2001-05-08 12:23                 ` Eli Zaretskii
2001-05-08 14:00                   ` DJ Delorie
2001-05-09  3:05                     ` Eli Zaretskii
2001-05-09  7:08                       ` Christopher Faylor
2001-05-08 11:14             ` Christopher Faylor
2001-05-08 12:25               ` Eli Zaretskii
2001-05-08 12:33                 ` Christopher Faylor
2001-05-09  3:05                   ` Eli Zaretskii
2001-05-08  4:48           ` DOS/Windows-specific code: sparcl-tdep.c Eli Zaretskii
2001-05-08 23:47             ` Kevin Buettner
2001-05-09  3:14               ` Eli Zaretskii
2001-05-10  8:33               ` Andrew Cagney
2001-05-08  4:48           ` DOS/Windows-specific code: terminal.h Eli Zaretskii
2001-05-08 11:18             ` Christopher Faylor
2001-05-08  4:50           ` DOS/Windows-specific code: top.c Eli Zaretskii
2001-05-08  7:46             ` Elena Zannoni
2001-05-08  4:51           ` DOS/Windows-specific code: utils.c Eli Zaretskii
2001-05-08 11:26             ` Christopher Faylor
2001-05-08  4:52           ` DOS/Windows-specific code: sh-tdep.c and sh3-rom.c Eli Zaretskii
2001-05-08  7:49             ` Elena Zannoni
2001-05-08 11:17               ` Eli Zaretskii
2001-05-08 11:40                 ` Elena Zannoni
2001-05-08 12:36                   ` Christopher Faylor
2001-05-08 12:35               ` Christopher Faylor
2001-05-08 18:17                 ` Andrew Cagney
2001-05-08 19:35                   ` Christopher Faylor
2001-05-08 23:55             ` Kevin Buettner
2001-05-09  3:20               ` Eli Zaretskii
2001-05-09  7:49                 ` Elena Zannoni
2001-05-08  4:52           ` DOS/Windows-specific code: ser-tcp.c Eli Zaretskii
2001-05-08  5:56             ` Russ Allbery
2001-05-08  6:01             ` Christopher Faylor
2001-05-08  8:09               ` Eli Zaretskii
2001-05-08  8:20                 ` Christopher Faylor
2001-05-08 10:30                   ` Eli Zaretskii
2001-05-08 11:03                     ` Christopher Faylor
2001-05-08 23:53             ` Kevin Buettner
2001-05-09  6:59               ` DOS/Windows-specific code: ser-tcp.c [need ruling from Andrew] Christopher Faylor
2001-05-09  9:29                 ` Andrew Cagney
2001-05-10  8:32             ` DOS/Windows-specific code: ser-tcp.c Andrew Cagney
2001-05-08  4:53           ` DOS/Windows-specific code: values.c Eli Zaretskii
2001-05-08 23:58             ` Kevin Buettner
2001-05-09  7:01               ` Christopher Faylor
2001-05-10  8:33               ` Andrew Cagney
2001-05-08  4:54           ` DOS/Windows-specific code in sim/ Eli Zaretskii
2001-05-08  9:30             ` DJ Delorie
2001-05-08 11:04               ` Eli Zaretskii
2001-05-10  8:33               ` Andrew Cagney
2001-05-08  4:55           ` DOS/Windows-specific code: all the rest Eli Zaretskii
2001-05-08  5:12             ` leonp
2001-05-08  7:59               ` Christopher Faylor
2001-05-08  9:45             ` DJ Delorie
2001-05-08 10:53               ` Eli Zaretskii
2001-05-08 12:08                 ` DJ Delorie
2001-05-07  9:09         ` [RFA] Remove OS-specific defines (was: _WIN32?) Eli Zaretskii
2001-05-07 10:39           ` Kevin Buettner
2001-05-07 11:23             ` Eli Zaretskii
2001-05-07 11:37               ` Andrew Cagney
2001-05-03 23:58     ` [Mingw-users] Re: _WIN32? Paul Hilfinger
2001-05-04  9:04       ` Andrew Cagney
2001-05-04  0:20     ` Eli Zaretskii
2001-05-05 15:57 Danny Smith
2001-05-05 18:54 ` Christopher Faylor
2001-05-05 23:03   ` Eli Zaretskii
2001-05-05 23:02 ` Eli Zaretskii
2001-05-06  1:52   ` Danny Smith
  -- strict thread matches above, loose matches on Subject: below --
2001-04-30 10:15 _WIN32? Christopher Faylor
2001-04-30 16:19 ` [Mingw-users] _WIN32? Danny Smith
2001-04-30 17:03   ` Christopher Faylor
2001-04-30 17:29     ` Danny Smith
2001-05-01  7:03     ` Kai Ruottu
2001-05-01  7:44       ` Christopher Faylor
2001-03-21 15:59 MACROS in gdb ??? Aditya Chugh
2001-03-21 15:59 ` J.T. Conklin
2001-03-21 15:59 ` Eli Zaretskii
     [not found]   ` <eliz@is.elta.co.il>
2001-03-21 15:59     ` Kevin Buettner
2001-03-21 15:59       ` Eli Zaretskii
2001-03-21 15:59 ` Daniel Berlin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3AF1C25B.5040304@cygnus.com \
    --to=ac131313@cygnus.com \
    --cc=cgf@redhat.com \
    --cc=danny_r_smith_2001@yahoo.co.nz \
    --cc=dewar@gnat.com \
    --cc=gdb@sources.redhat.com \
    --cc=hilfingr@gnat.com \
    --cc=mingw-users@lists.sourceforge \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).