This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Objective-C language support.


Andrew Cagney wrote:
> 
> > -       error ("no args, no `this'");
> >> +       error ("no args, no %s", name);
> 
> Michael,
> 
> This changes GDB's behavior.  Instead of printing:
> 
>         no args, no `this'
> it will print
>         no args, no this
> 
> I don't think GDB's output should be changed in this way.
> (I'm suprized that running the testsuite didn't detect this).

Hi Andrew, 

I corrected this with the patch below.  I did not add a new test,
because I cannot find a way to invoke this error message.  ;-/

Michael
2002-10-23  Michael Snyder  <msnyder@redhat.com>

	* valops.c (value_of_local): Restore quotes in error message.

Index: valops.c
===================================================================
RCS file: /cvs/src/src/gdb/valops.c,v
retrieving revision 1.76
diff -p -r1.76 valops.c
*** valops.c	14 Oct 2002 02:02:42 -0000	1.76
--- valops.c	23 Oct 2002 19:21:12 -0000
*************** value_of_local (const char *name, int co
*** 3340,3346 ****
    if (i <= 0)
      {
        if (complain)
! 	error ("no args, no %s", name);
        else
  	return 0;
      }
--- 3340,3346 ----
    if (i <= 0)
      {
        if (complain)
! 	error ("no args, no '%s'", name);
        else
  	return 0;
      }

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]