Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFA] Remove unused declaration in top.c
@ 2001-05-17  7:51 Eli Zaretskii
  2001-05-17  8:02 ` Elena Zannoni
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2001-05-17  7:51 UTC (permalink / raw)
  To: gdb-patches

I'm asking for approval of the following patch which removes an unused 
definition of struct user_args variable in top.c.  AFAICS, this is a 
relic left behind when cli-* files were separated from top.c.  The reason 
I payed attention to it was that the Irix linker printed a warning about 
duplicate definition (one in top.c, the other in cli-script.c).

Okay to commit?  (It looks like an obvious fix, but...)

2001-05-17  Eli Zaretskii  <eliz@is.elta.co.il>

	* top.c (user_args): Remove unused declaration.


--- gdb/top.c~	Thu Apr  5 18:59:30 2001
+++ gdb/top.c	Thu May 17 17:36:24 2001
@@ -224,21 +224,6 @@
 /* Level of control structure.  */
 static int control_level;
 
-/* Structure for arguments to user defined functions.  */
-#define MAXUSERARGS 10
-struct user_args
-  {
-    struct user_args *next;
-    struct
-      {
-	char *arg;
-	int len;
-      }
-    a[MAXUSERARGS];
-    int count;
-  }
- *user_args;
-
 /* Signal to catch ^Z typed while reading a command: SIGTSTP or SIGCONT.  */
 
 #ifndef STOP_SIGNAL


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFA] Remove unused declaration in top.c
  2001-05-17  7:51 [RFA] Remove unused declaration in top.c Eli Zaretskii
@ 2001-05-17  8:02 ` Elena Zannoni
  2001-05-17  8:06   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Elena Zannoni @ 2001-05-17  8:02 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: gdb-patches

Eli Zaretskii writes:
 > I'm asking for approval of the following patch which removes an unused 
 > definition of struct user_args variable in top.c.  AFAICS, this is a 
 > relic left behind when cli-* files were separated from top.c.  The reason 
 > I payed attention to it was that the Irix linker printed a warning about 
 > duplicate definition (one in top.c, the other in cli-script.c).
 > 
 > Okay to commit?  (It looks like an obvious fix, but...)
 > 

Yes, thanks!

Elena



 > 2001-05-17  Eli Zaretskii  <eliz@is.elta.co.il>
 > 
 > 	* top.c (user_args): Remove unused declaration.
 > 
 > 
 > --- gdb/top.c~	Thu Apr  5 18:59:30 2001
 > +++ gdb/top.c	Thu May 17 17:36:24 2001
 > @@ -224,21 +224,6 @@
 >  /* Level of control structure.  */
 >  static int control_level;
 >  
 > -/* Structure for arguments to user defined functions.  */
 > -#define MAXUSERARGS 10
 > -struct user_args
 > -  {
 > -    struct user_args *next;
 > -    struct
 > -      {
 > -	char *arg;
 > -	int len;
 > -      }
 > -    a[MAXUSERARGS];
 > -    int count;
 > -  }
 > - *user_args;
 > -
 >  /* Signal to catch ^Z typed while reading a command: SIGTSTP or SIGCONT.  */
 >  
 >  #ifndef STOP_SIGNAL
 > 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFA] Remove unused declaration in top.c
  2001-05-17  8:02 ` Elena Zannoni
@ 2001-05-17  8:06   ` Eli Zaretskii
  0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2001-05-17  8:06 UTC (permalink / raw)
  To: Elena Zannoni; +Cc: gdb-patches

On Thu, 17 May 2001, Elena Zannoni wrote:

> Eli Zaretskii writes:
>  > I'm asking for approval of the following patch which removes an unused 
>  > definition of struct user_args variable in top.c.  AFAICS, this is a 
>  > relic left behind when cli-* files were separated from top.c.  The reason 
>  > I payed attention to it was that the Irix linker printed a warning about 
>  > duplicate definition (one in top.c, the other in cli-script.c).
>  > 
>  > Okay to commit?  (It looks like an obvious fix, but...)
>  > 
> 
> Yes, thanks!

Commited.  Thanks for the fast response.

> 
> Elena
> 
> 
> 
>  > 2001-05-17  Eli Zaretskii  <eliz@is.elta.co.il>
>  > 
>  > 	* top.c (user_args): Remove unused declaration.
>  > 
>  > 
>  > --- gdb/top.c~	Thu Apr  5 18:59:30 2001
>  > +++ gdb/top.c	Thu May 17 17:36:24 2001
>  > @@ -224,21 +224,6 @@
>  >  /* Level of control structure.  */
>  >  static int control_level;
>  >  
>  > -/* Structure for arguments to user defined functions.  */
>  > -#define MAXUSERARGS 10
>  > -struct user_args
>  > -  {
>  > -    struct user_args *next;
>  > -    struct
>  > -      {
>  > -	char *arg;
>  > -	int len;
>  > -      }
>  > -    a[MAXUSERARGS];
>  > -    int count;
>  > -  }
>  > - *user_args;
>  > -
>  >  /* Signal to catch ^Z typed while reading a command: SIGTSTP or SIGCONT.  */
>  >  
>  >  #ifndef STOP_SIGNAL
>  > 
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2001-05-17  8:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-05-17  7:51 [RFA] Remove unused declaration in top.c Eli Zaretskii
2001-05-17  8:02 ` Elena Zannoni
2001-05-17  8:06   ` Eli Zaretskii

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox