Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: David Taylor <taylor@candd.org>
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: ac131313@cygnus.com, danny_r_smith_2001@yahoo.co.nz,
	gdb@sources.redhat.com, mingw-users@lists.sourceforge,
	gdb-patches@sources.redhat.com
Subject: Re: [RFA] Remove OS-specific defines (was: _WIN32?)
Date: Mon, 07 May 2001 11:36:00 -0000	[thread overview]
Message-ID: <200105071835.OAA02864@houston.candd.org> (raw)

    Date: Mon, 7 May 2001 19:10:59 +0300 (IDT)
    From: Eli Zaretskii <eliz@is.elta.co.il>

    I'm seeking approval for the following patches.  They remove all the
    DOS- and Windows-specific #ifdef's like "#ifdef _WIN32" and use the
    portable macros from filenames.h instead.  In addition, they convert
    code which used macros private to GDB, such as SLASH_P and ROOTED_P,
    to use the macros from filenames.h instead.

I was surprised that you sent this to gdb rather than gdb-patches.  I
added gdb-patches to the cc list.

    2001-05-07  Eli Zaretskii  <eliz@is.elta.co.il>

	    * source.c (mod_path, openp): Use HAVE_DOS_BASED_FILE_SYSTEM
	    instead of system-specific define's like _WIN32 and __MSDOS__.
	    Use IS_DIR_SEPARATOR and IS_ABSOLUTE_PATH instead of SLASH_P and
	    ROOTED_P.
	    (top-level): #include "filenames.h".

	    * solib.c (solib_open): Use IS_DIR_SEPARATOR and IS_ABSOLUTE_PATH
	    instead of SLASH_CHAR, ROOTED_P and SLASH_P.
	    (top-level): #include "filenames.h".

	    * defs.h (SLASH_P, SLASH_CHAR, ROOTED_P): Remove definitions.
	    (SLASH_STRING): Define only for _WIN32.

The defs.h part of this patch is approved.

	    * completer.c: Use HAVE_DOS_BASED_FILE_SYSTEM instead of
	    __MSDOS_.

	    * cli/cli-cmds.c (cd_command): Use IS_DIR_SEPARATOR and
	    IS_ABSOLUTE_PATH instead of SLASH_P and ROOTED_P.  Replace
	    system-specific ifdefs with HAVE_DOS_BASED_FILE_SYSTEM.
	    (top-level): #include "filenames.h".


             reply	other threads:[~2001-05-07 11:36 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-05-07 11:36 David Taylor [this message]
2001-05-08  0:12 ` Eli Zaretskii
     [not found] <3AF6EB58.1020904@cygnus.com>
2001-05-10  3:31 ` Eli Zaretskii
     [not found] <20010503211502.21716.qmail@web6401.mail.yahoo.com>
     [not found] ` <3AF1DAA0.3060702@cygnus.com>
     [not found]   ` <200105071610.TAA24144@is.elta.co.il>
2001-05-13  1:29     ` Eli Zaretskii
2001-05-13  7:09       ` Fernando Nasser
2001-05-13 23:34         ` Eli Zaretskii
2001-05-14 13:18           ` Andrew Cagney
2001-06-04  0:46             ` Eli Zaretskii
     [not found]     ` <1010507173855.ZM18890@ocotillo.lan>
2001-05-07 11:24       ` Eli Zaretskii
2001-06-10  6:28       ` Eli Zaretskii
2001-06-12 17:14         ` Christopher Faylor
2001-06-13  2:08           ` Eli Zaretskii

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=200105071835.OAA02864@houston.candd.org \
    --to=taylor@candd.org \
    --cc=ac131313@cygnus.com \
    --cc=danny_r_smith_2001@yahoo.co.nz \
    --cc=eliz@is.elta.co.il \
    --cc=gdb-patches@sources.redhat.com \
    --cc=gdb@sources.redhat.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