Re: [patch] remove extraneous check_typedef () call in c_val_print ()

[Jason Molenda <jmolenda@apple.com>]

[-- Attachment #1: Type: text/plain, Size: 543 bytes --]

On Friday, August 31, 2001, at 02:39 PM, Jason Molenda wrote:

> Hello, my name is Jason Molenda, here is my first patch submission to 
> GDB.  I've heard that my employer has a copyright assignment on file 
> already, so I expect those details are all in order.  If I've missed 
> some bit of documentation about how to submit a patch properly, please 
> let me know.
>

Including the patch would be a good first step, I suppose.  BTW I did 
run the gdb testsuite with this patch and there are no additional 
regressions.  Here's the patch:


[-- Attachment #2: pa --]
[-- Type: text/x-diff, Size: 665 bytes --]

2001-08-31  Jason Molenda  (jmolenda@apple.com)

	* c-valprint.c (c_val_print):  Second call to check_typedef ()
	is no longer necessary.

Index: c-valprint.c
===================================================================
RCS file: /cvs/src/src/gdb/c-valprint.c,v
retrieving revision 1.9
diff -u -p -r1.9 c-valprint.c
--- c-valprint.c	2001/04/27 00:19:09	1.9
+++ c-valprint.c	2001/08/31 21:29:04
@@ -168,7 +168,6 @@ c_val_print (struct type *type, char *va
 	{
 	  addr = unpack_pointer (type, valaddr + embedded_offset);
 	print_unpacked_pointer:
-	  elttype = check_typedef (TYPE_TARGET_TYPE (type));
 
 	  if (TYPE_CODE (elttype) == TYPE_CODE_FUNC)
 	    {