From: Syd Polk <spolk@redhat.com>
To: tromey@cygnus.com, Insight List <insight@sourceware.cygnus.com>
Subject: Re: Patch: fixlet in gdbtk-cmds.c
Date: Tue, 28 Nov 2000 12:55:00 -0000 [thread overview]
Message-ID: <4.2.0.58.20001128125809.01a7f100@pop.cygnus.com> (raw)
In-Reply-To: <87zoik9bu1.fsf@creche.cygnus.com>
Approved, although I would have preferred using Tcl_WrongNumArgs. That
would give a slightly different error message, however, and that might
break some tests.
At 11:43 AM 11/28/00 -0700, Tom Tromey wrote:
>I found a buglet in gdbtk-cmds.c. This patch fixes it. Ok to commit?
>
>2000-11-28 Tom Tromey <tromey@cygnus.com>
>
> * gdbtk-cmds.c (gdb_clear_file): Return error if wrong number of
> arguments given.
>
>Tom
>
>Index: generic/gdbtk-cmds.c
>===================================================================
>RCS file: /cvs/src/src/gdb/gdbtk/generic/gdbtk-cmds.c,v
>retrieving revision 1.12
>diff -u -r1.12 gdbtk-cmds.c
>--- gdbtk-cmds.c 2000/07/25 20:41:07 1.12
>+++ gdbtk-cmds.c 2000/11/28 18:33:19
>@@ -580,8 +606,11 @@
> Tcl_Obj *CONST objv[];
> {
> if (objc != 1)
>- Tcl_SetStringObj (result_ptr->obj_ptr,
>- "Wrong number of args, none are allowed.", -1);
>+ {
>+ Tcl_SetStringObj (result_ptr->obj_ptr,
>+ "Wrong number of args, none are allowed.", -1);
>+ return TCL_ERROR;
>+ }
>
> if (inferior_pid != 0 && target_has_execution)
> {
Syd Polk spolk@redhat.com
Engineering Manager +1 415 777 9810 x 241
Red Hat, Inc.
next prev parent reply other threads:[~2000-11-28 12:55 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-11-28 10:34 Tom Tromey
2000-11-28 12:55 ` Syd Polk [this message]
2000-11-28 13:46 ` Tom Tromey
2000-11-28 14:47 ` Syd Polk
2000-11-28 17:32 ` Tom Tromey
2000-11-28 23:51 ` Syd Polk
2000-11-29 13:48 ` Tom Tromey
2000-11-29 18:41 ` Syd Polk
2000-11-29 20:07 ` Fernando Nasser
2000-11-30 9:09 ` Tom Tromey
2000-11-30 9:57 ` Fernando Nasser
[not found] <Syd>
[not found] ` <Polk's>
[not found] ` <message>
[not found] ` <of>
[not found] ` <"Wed,>
[not found] ` <"Tue,>
[not found] ` <"28>
[not found] ` <"Mon,>
[not found] ` <01>
[not found] ` <May>
[not found] ` <2000>
[not found] ` <09:07:18>
[not found] ` <-0700>
2000-04-29 7:21 ` make check in itcl Andreas Jaeger
2000-05-01 9:05 ` Syd Polk
2000-05-01 9:27 ` Andreas Jaeger
2000-05-01 10:15 ` Syd Polk
2000-05-01 10:58 ` Andreas Jaeger
2000-05-01 11:21 ` Syd Polk
2000-05-03 10:37 ` Syd Polk
2000-05-03 12:05 ` Andreas Jaeger
2000-05-03 12:37 ` Syd Polk
2000-05-04 2:34 ` Andrew Cagney
[not found] ` <09:17:35>
[not found] ` <-0800>
2000-11-21 16:03 ` rename to insight Tom Tromey
[not found] ` <mailpost.974851789.4627@postal.sibyte.com>
2000-11-22 9:42 ` Chris G. Demetriou
2000-11-22 10:29 ` Tom Tromey
2000-11-28 9:18 ` Chris G. Demetriou
2000-11-28 9:34 ` Tom Tromey
2000-11-28 12:52 ` Syd Polk
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=4.2.0.58.20001128125809.01a7f100@pop.cygnus.com \
--to=spolk@redhat.com \
--cc=insight@sourceware.cygnus.com \
--cc=tromey@cygnus.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).