Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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.


  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