Re: c++/4095: Variable misspelled in gcc/cp/rtti.c

Re: c++/4095: Variable misspelled in gcc/cp/rtti.c

[rodrigc]

Synopsis: Variable misspelled in gcc/cp/rtti.c

State-Changed-From-To: open->closed
State-Changed-By: rodrigc
State-Changed-When: Sat Sep 29 11:47:30 2001
State-Changed-Why:
    Fixed

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view&pr=4095&database=gcc

Re: c++/4095: Variable misspelled in gcc/cp/rtti.c

[Zack Weinberg]

The following reply was made to PR c++/4095; it has been noted by GNATS.

From: Zack Weinberg <zackw@panix.com>
To: Mark Mitchell <mark@codesourcery.com>,
	Andreas Jaeger <aj@suse.de>
Cc: gcc-gnats@gcc.gnu.org, gcc-patches@gcc.gnu.org
Subject: Re: c++/4095: Variable misspelled in gcc/cp/rtti.c
Date: Fri, 24 Aug 2001 15:48:05 -0400

 > I think Zack is working on a patch to allow you to use the same
 > set of macros with both stdarg and varargs, which will be nice
 > and tidy.
 
 It's finished and checked in.  The macros are VA_OPEN, VA_CLOSE,
 and VA_FIXEDARG, defined in ansidecl.h.  Look there for documentation,
 and in cpperror.c for example uses.
 
 zw

Re: c++/4095: Variable misspelled in gcc/cp/rtti.c

[Mark Mitchell]

The following reply was made to PR c++/4095; it has been noted by GNATS.

From: Mark Mitchell <mark@codesourcery.com>
To: Andreas Jaeger <aj@suse.de>, "rkl@connect.org.uk" <rkl@connect.org.uk>
Cc: "gcc-gnats@gcc.gnu.org" <gcc-gnats@gcc.gnu.org>,
   "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: Re: c++/4095: Variable misspelled in gcc/cp/rtti.c
Date: Fri, 24 Aug 2001 09:36:50 -0700

 > Here's a patch.  Ok to commit to the trunk?
 >
 
 Yes.
 
 I think Zack is working on a patch to allow you to use the same
 set of macros with both stdarg and varargs, which will be nice
 and tidy.
 
 -- 
 Mark Mitchell                mark@codesourcery.com
 CodeSourcery, LLC            http://www.codesourcery.com

Re: c++/4095: Variable misspelled in gcc/cp/rtti.c

[Andreas Jaeger]

The following reply was made to PR c++/4095; it has been noted by GNATS.

From: Andreas Jaeger <aj@suse.de>
To: rkl@connect.org.uk
Cc: gcc-gnats@gcc.gnu.org, gcc-patches@gcc.gnu.org
Subject: Re: c++/4095: Variable misspelled in gcc/cp/rtti.c
Date: Fri, 24 Aug 2001 06:52:00 +0200

 rkl@connect.org.uk writes:
 
 >>Number:         4095
 >>Category:       c++
 >>Synopsis:       Variable misspelled in gcc/cp/rtti.c
 
 [...]
 > >Description:
 > In line 1293 (create_pseudo_type_info() function), the
 > first param of va_arg() is spelled "app", which doesn't
 > exist as a variable.
 > >How-To-Repeat:
 > Compile without ANSI_PROTOTYPES defined - I did this by
 > building with a K&R C compiler.
 > >Fix:
 > Line 1293 should read:
 >
 >   ident = va_arg (ap, int);
 
 Here's a patch.  Ok to commit to the trunk?
 
 Or should we remove this ANSI_PROTOTYPES stuff since cp will be
 compiled with GCC?
 
 Andreas
 
 2001-08-24  Andreas Jaeger  <aj@suse.de>, rkl@connect.org.uk
 
 	* cp/rtti.c (VPARAMS): Fix parameter.
 
 ============================================================
 Index: gcc/cp/rtti.c
 --- gcc/cp/rtti.c	2001/08/14 23:51:27	1.121
 +++ gcc/cp/rtti.c	2001/08/24 04:51:35
 @@ -1170,7 +1170,7 @@
    VA_START (ap, ident);
  #ifndef ANSI_PROTOTYPES
    real_name = va_arg (ap, char const *);
 -  ident = va_arg (app, int);
 +  ident = va_arg (ap, int);
  #endif
  
    /* Generate the pseudo type name. */
 
 -- 
  Andreas Jaeger
   SuSE Labs aj@suse.de
    private aj@arthur.inka.de
     http://www.suse.de/~aj

c++/4095: Variable misspelled in gcc/cp/rtti.c

[rkl]

>Number:         4095
>Category:       c++
>Synopsis:       Variable misspelled in gcc/cp/rtti.c
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          mistaken
>Submitter-Id:   net
>Arrival-Date:   Thu Aug 23 07:36:01 PDT 2001
>Closed-Date:
>Last-Modified:
>Originator:     rkl@connect.org.uk
>Release:        3.0.1
>Organization:
>Environment:
All platforms
>Description:
In line 1293 (create_pseudo_type_info() function), the
first param of va_arg() is spelled "app", which doesn't
exist as a variable.
>How-To-Repeat:
Compile without ANSI_PROTOTYPES defined - I did this by
building with a K&R C compiler.
>Fix:
Line 1293 should read:

  ident = va_arg (ap, int);
>Release-Note:
>Audit-Trail:
>Unformatted: