From: Daniel Berlin <dberlin@dberlin.org>
To: Joel Brobecker <brobecker@gnat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] Add new language: "unsupported"
Date: Wed, 07 May 2003 15:01:00 -0000 [thread overview]
Message-ID: <D2DAD462-809C-11D7-BC31-000A95A34564@dberlin.org> (raw)
In-Reply-To: <20030507014231.GA3156@gnat.com>
On Tuesday, May 6, 2003, at 09:42 PM, Joel Brobecker wrote:
> As promised to Elena, this is a followup on:
>
> http://sources.redhat.com/ml/gdb-patches/2003-04/msg00209.html
>
> Basically, I added a new "unsupported" language which support is
> minimalistic (identical to what we do with the "asm" language).
> I didn't fancy "partial":
>
> (gdb) show lang
> Current language: auto; currently partial
>
> I felt like it could confuse the user to think that "partial" is the
> name of a real language :-). But I'm not such a big fan of
> "unsupported"
> either, so all suggestions are welcome.
>
> This new language will first be used by the dwarf2 reader, for objects
> which language is currently not supported. What it does, at the moment,
> is use the "unknown" language, which makes a lot of the GDB commands
> fall flat. Like so, when debugging an Ada program:
>
> (gdb) list foo.adb:1
> internal error - unimplemented function
> unk_lang_create_fundamental_type called.
> (gdb) quit
>
> With the attached patch, and also the little patch to dwarf2read.c
> (attached too, will be submitted later, after this one is agreed on),
> GDB behaves in a much more friendly way:
>
> (gdb) list foo.adb:1
> 1 procedure Foo is
> 2 A : Integer := 1;
> 3 begin
> 4 A := A + 1;
> 5 end Foo;
> (gdb) b foo.adb:4
> Breakpoint 1 at 0x8049769: file foo.adb, line 4.
> (gdb) run
> Starting program: /lek.a/brobecke/ada_example/foo
>
> Breakpoint 1, _ada_foo () at foo.adb:4
> 4 A := A + 1;
> Current language: auto; currently unsupported
> (gdb) p a
> $1 = 1
It should probably warn (I guess it has to be in the reader, since we
have no hook for when you switch to a language, right?) when it is
switching to the unsupported language.
Something like "Warning: The current language is unsupported by GDB.
Name lookups and printouts may not look like what you expect as a
result".
That way, nobody files bugs because the unsupported language does what
C would do, when they expect what <insert other language here> would do.
next prev parent reply other threads:[~2003-05-07 15:01 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-07 1:42 Joel Brobecker
2003-05-07 11:01 ` Eli Zaretskii
2003-05-07 14:21 ` Elena Zannoni
2003-05-07 14:24 ` Daniel Jacobowitz
2003-05-08 17:37 ` Joel Brobecker
2003-05-07 15:01 ` Daniel Berlin [this message]
2003-05-07 17:27 ` Elena Zannoni
2003-05-07 18:10 ` Joel Brobecker
2003-05-08 17:55 ` Joel Brobecker
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=D2DAD462-809C-11D7-BC31-000A95A34564@dberlin.org \
--to=dberlin@dberlin.org \
--cc=brobecker@gnat.com \
--cc=gdb-patches@sources.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