2003-01-31 Andrew Cagney <ac131313@redhat.com>

	* objc-exp.y, c-exp.y, f-exp.y: Remove PTR casts.
	* utils.c: Update comments documenting legitimate uses of PTR.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 78f652162da5d0e42b495b6213cf7f21e78bddcc..0f3fc6fe9cb7b27630af881105b0cff186bfdd4d 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,8 @@
 2003-01-31  Andrew Cagney  <ac131313@redhat.com>
 
+       * objc-exp.y, c-exp.y, f-exp.y: Remove PTR casts.
+       * utils.c: Update comments documenting legitimate uses of PTR.
+       
        * utils.c: Re-indent.
 
        * config/djgpp/fnchange.lst: Delete nindy files.

diff --git a/gdb/c-exp.y b/gdb/c-exp.y
index 9df0aee428ffe8af76ab93153a88a26e097ec0e3..d47588ec59ed67d2b9389ef497af19ec1f967f33 100644 (file)
--- a/gdb/c-exp.y
+++ b/gdb/c-exp.y
@@ -781,7 +781,7 @@ array_mod:  '[' ']'
 func_mod:      '(' ')'
                        { $$ = 0; }
        |       '(' nonempty_typelist ')'
-                       { free ((PTR)$2); $$ = 0; }
+                       { free ($2); $$ = 0; }
        ;
 
 /* We used to try to recognize more pointer to member types here, but

diff --git a/gdb/f-exp.y b/gdb/f-exp.y
index 56fd70af954871ff8f1b5143e53b2132be0e937e..69662310e1de9f4084860ea259e965f13fd650c7 100644 (file)
--- a/gdb/f-exp.y
+++ b/gdb/f-exp.y
@@ -564,7 +564,7 @@ direct_abs_decl: '(' abs_decl ')'
 func_mod:      '(' ')'
                        { $$ = 0; }
        |       '(' nonempty_typelist ')'
-                       { free ((PTR)$2); $$ = 0; }
+                       { free ($2); $$ = 0; }
        ;
 
 typebase  /* Implements (approximately): (type-qualifier)* type-specifier */

diff --git a/gdb/objc-exp.y b/gdb/objc-exp.y
index 78f207df15f990aaca56c277a92b3ee8f2fdadd0..1a7c5f002a8c80b11006793e13c2ee98b28dae10 100644 (file)
--- a/gdb/objc-exp.y
+++ b/gdb/objc-exp.y
@@ -842,7 +842,7 @@ array_mod:  '[' ']'
 func_mod:      '(' ')'
                        { $$ = 0; }
        |       '(' nonempty_typelist ')'
-                       { free ((PTR)$2); $$ = 0; }
+                       { free ($2); $$ = 0; }
        ;
 
 /* We used to try to recognize more pointer to member types here, but

diff --git a/gdb/utils.c b/gdb/utils.c
index 9b047bc8993be2e0850abeb206dd00422677cdc4..e7353bf5d066e721ba13981a6827d52be28cbc6d 100644 (file)
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -66,10 +66,10 @@
 #endif
 
 #ifdef NEED_DECLARATION_MALLOC
-extern PTR malloc ();
+extern PTR malloc ();          /* OK: PTR */
 #endif
 #ifdef NEED_DECLARATION_REALLOC
-extern PTR realloc ();
+extern PTR realloc ();         /* OK: PTR */
 #endif
 #ifdef NEED_DECLARATION_FREE
 extern void free ();
@@ -955,9 +955,6 @@ request_quit (int signo)
 
 #if !defined (USE_MMALLOC)
 
-/* NOTE: These must use PTR so that their definition matches the
-   declaration found in "mmalloc.h". */
-
 static void *
 mmalloc (void *md, size_t size)
 {
@@ -1150,19 +1147,19 @@ xmfree (void *md, void *ptr)
 /* NOTE: These are declared using PTR to ensure consistency with
    "libiberty.h".  xfree() is GDB local.  */
 
-PTR
+PTR                            /* OK: PTR */
 xmalloc (size_t size)
 {
   return xmmalloc (NULL, size);
 }
 
-PTR
-xrealloc (PTR ptr, size_t size)
+PTR                            /* OK: PTR */
+xrealloc (PTR ptr, size_t size)        /* OK: PTR */
 {
   return xmrealloc (NULL, ptr, size);
 }
 
-PTR
+PTR                            /* OK: PTR */
 xcalloc (size_t number, size_t size)
 {
   return xmcalloc (NULL, number, size);