Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Michael Elizabeth Chastain <mec@shout.net>
To: msnyder@redhat.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] nuke CONST_PTR
Date: Wed, 13 Feb 2002 09:06:00 -0000	[thread overview]
Message-ID: <200202131706.g1DH6k124943@duracef.shout.net> (raw)

Hi Michael,

ms> Your reasoning seems good, but your testing doesn't.
ms> Seems to me you need to test this when building with Microsoft C.

I think I already addressed this point:

mec> However, in gdb 4.18, gdb 5.0, gdb 5.1, and gdb 5.1, c-lang.c has also
mec> contained this line:
mec> 
mec>   struct type **const (cplus_builtin_types[]) =
mec> 
mec> So this form has been in gdb source code for four releases already
mec> without drawing complaint.

Also, the CONST_PTR macro has been unconditionally "struct type **const"
since 5.1, which admittedly is not much time.

ms> OTOH, do we ever build with Microsoft C any more?
ms> Is there any reason to support it?  Cygwin and Djgcc
ms> are both self-hosting, aren't they?

I don't know about Microsoft C.  I do know that I build Cygwin gdb
using Cygwin gcc.  I don't know about djgpp.

ms> So the discussion (if there still is one) is (I think)
ms> "do we still support building with Microsoft C?"

I don't know.

The second half of the question is "and does Microsoft C still have
a problem with struct type **const".

If we do support building with Microsoft C, and if Microsoft C has
a problem with "struct type **const", then gdb 4.18, 5.0, 5.1, and
5.1.1 are all broken in one spot.  Maybe that is not a good reason for
breaking it in other spots.  But one instance of "struct type **const"
has been in released versions of gdb for 34 months now and I don't see
complaints coming in.

Michael C


             reply	other threads:[~2002-02-13 17:06 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-13  9:06 Michael Elizabeth Chastain [this message]
2002-02-13 11:43 ` Eli Zaretskii
  -- strict thread matches above, loose matches on Subject: below --
2002-02-13 10:49 Michael Elizabeth Chastain
2002-02-13  6:44 Michael Elizabeth Chastain
2002-02-13  7:19 ` Kevin Buettner
2002-02-13  8:15 ` Michael Snyder
2002-02-13  9:30   ` Andrew Cagney
2002-02-13 10:04     ` Michael Snyder
2002-02-13 10:56       ` Christopher Faylor
2002-02-13 11:05       ` Andrew Cagney
2002-02-13 11:08         ` Christopher Faylor

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=200202131706.g1DH6k124943@duracef.shout.net \
    --to=mec@shout.net \
    --cc=gdb-patches@sources.redhat.com \
    --cc=msnyder@redhat.com \
    /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