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]

PATCH: Obsolete comment in ada-lang.c


A comment in ada-lang.c was complaining about the use of TYPE_ARG_TYPES,
which I smote out of existence last week.  Pointed out by Elena; committed
as obvious.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer
2002-06-19  Daniel Jacobowitz  <drow@mvista.com>

        * ada-lang.c (fill_in_ada_prototype): Update comment.

Index: ada-lang.c
===================================================================
RCS file: /cvs/src/src/gdb/ada-lang.c,v
retrieving revision 1.1
diff -u -p -r1.1 ada-lang.c
--- ada-lang.c	4 Jun 2002 15:28:47 -0000	1.1
+++ ada-lang.c	19 Jun 2002 16:52:51 -0000
@@ -4280,15 +4280,7 @@ ada_add_block_symbols (block, name, name
 				/* Function Types */
 
 /* Assuming that SYM is the symbol for a function, fill in its type 
-   with prototype information, if it is not already there.  
-   
-   Why is there provision in struct type for BOTH an array of argument
-   types (TYPE_ARG_TYPES) and for an array of typed fields, whose
-   comment suggests it may also represent argument types?  I presume
-   this is some attempt to save space.  The problem is that argument
-   names in Ada are significant.  Therefore, for Ada we use the
-   (apparently older) TYPE_FIELD_* stuff to store argument types. */
-
+   with prototype information, if it is not already there.  */
 
 static void
 fill_in_ada_prototype (func)

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