Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: Adam Fedor <fedor@doc.com>
Cc: Andrew Cagney <ac131313@redhat.com>,
	David Carlton <carlton@math.stanford.edu>,
	GDB Patches <gdb-patches@sources.redhat.com>
Subject: Re: [RFA] Compile objc-lang.c, objc-exp.tab.c [1/5]
Date: Tue, 01 Apr 2003 21:38:00 -0000	[thread overview]
Message-ID: <20030401213821.GA12857@nevyn.them.org> (raw)
In-Reply-To: <3E8910F5.2090507@doc.com>

On Mon, Mar 31, 2003 at 09:09:25PM -0700, Adam Fedor wrote:
> 
> 
> Andrew Cagney wrote:
> >
> >
> >Hmm, good catch.  Any reason for the unknown language demangler to not 
> >just do c++ demangling?
> >
> 
> OK. Here I've add java_demangle and now ukn_lang_demangle uses 
> cplus_demangle.

I think you've satisfied all my concerns.  Since Andrew and David both
liked this patch, it's OK to check in now.  Please fix one thing for
me: there are two switch statements below from which you remove all
cases except for default.  Please remove the switches entirely.

> @@ -153,13 +153,9 @@ maintenance_demangle (char *args, int fr
>      {
>        switch (current_language->la_language)
>  	{
> -	case language_objc:
> -	  /* Commented out until ObjC handling is enabled. */
> -	  /* demangled = objc_demangle (args); */
> -	  /* break; */
> -	case language_cplus:
>  	default:
> -	  demangled = cplus_demangle (args, DMGL_ANSI | DMGL_PARAMS);
> +	  demangled = language_demangle (current_language, args, 
> +					 DMGL_ANSI | DMGL_PARAMS);
>  	  break;
>  	}
>        if (demangled != NULL)

> Index: utils.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/utils.c,v
> retrieving revision 1.98
> diff -u -p -r1.98 utils.c
> --- utils.c	26 Feb 2003 14:35:19 -0000	1.98
> +++ utils.c	1 Apr 2003 04:01:30 -0000
> @@ -2300,18 +2300,9 @@ fprintf_symbol_filtered (struct ui_file 
>  	{
>  	  switch (lang)
>  	    {
> -	    case language_cplus:
> -	      demangled = cplus_demangle (name, arg_mode);
> -	      break;
> -	    case language_java:
> -	      demangled = cplus_demangle (name, arg_mode | DMGL_JAVA);
> -	      break;
> -	    case language_objc:
> -	      /* Commented out until ObjC handling is enabled.  */
> -	      /*demangled = objc_demangle (name); */
> -	      /*break; */
>  	    default:
> -	      demangled = NULL;
> +	      demangled = language_demangle (language_def (lang), 
> +					     name, arg_mode);
>  	      break;
>  	    }
>  	  fputs_filtered (demangled ? demangled : name, stream);


-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer


  parent reply	other threads:[~2003-04-01 21:38 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-03 22:06 Adam Fedor
2003-02-19 19:02 ` Elena Zannoni
2003-02-19 20:27   ` Adam Fedor
2003-02-19 21:11     ` Elena Zannoni
2003-03-20 21:27 ` Andrew Cagney
2003-03-20 21:39   ` Daniel Jacobowitz
2003-03-20 22:13     ` Andrew Cagney
2003-03-20 22:19       ` Daniel Jacobowitz
2003-03-31  2:23         ` Adam Fedor
2003-03-31 22:39           ` Andrew Cagney
2003-03-31 22:53           ` David Carlton
2003-03-31 23:03             ` David Carlton
2003-03-31 23:15             ` Adam Fedor
2003-04-01  0:31               ` Andrew Cagney
2003-04-01  0:45                 ` David Carlton
2003-04-01  4:09                 ` Adam Fedor
2003-04-01 21:31                   ` David Carlton
2003-04-01 21:38                   ` Daniel Jacobowitz [this message]
2003-03-20 22:22     ` David Ayers
2003-03-20 21:35 ` Andrew Cagney
2003-03-24 17:46   ` Adam Fedor
2003-03-24 18:25     ` Andrew Cagney
2003-03-25  2:23       ` Adam Fedor

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=20030401213821.GA12857@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=ac131313@redhat.com \
    --cc=carlton@math.stanford.edu \
    --cc=fedor@doc.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