gdb: delete a couple of declarations from f-lang.h
authorAndrew Burgess <andrew.burgess@embecosm.com>
Fri, 13 Nov 2020 11:03:05 +0000 (11:03 +0000)
committerAndrew Burgess <andrew.burgess@embecosm.com>
Fri, 13 Nov 2020 13:51:09 +0000 (13:51 +0000)
One declaration in f-lang.h is for a function that doesn't even exist,
another is for a function that is only used within f-lang.c.

One declaration is deleted, the other function I make static in
f-lang.c.

gdb/ChangeLog:

* f-lang.c (fortran_argument_convert): Add declaration.  Add
header comment, taken from f-lang.h.  Make static.
* f-lang.h (f77_get_dynamic_array_length): Delete declaration.
(fortran_argument_convert): Delete declaration.

gdb/ChangeLog
gdb/f-lang.c
gdb/f-lang.h

index 4ab418396f641fa6d618494eaf453c6488c28970..2d46ac9dac3eab38dd64f97c2f7fdfad03e1aac1 100644 (file)
@@ -1,3 +1,10 @@
+2020-11-13  Andrew Burgess  <andrew.burgess@embecosm.com>
+
+       * f-lang.c (fortran_argument_convert): Add declaration.  Add
+       header comment, taken from f-lang.h.  Make static.
+       * f-lang.h (f77_get_dynamic_array_length): Delete declaration.
+       (fortran_argument_convert): Delete declaration.
+
 2020-11-12  Andrew Burgess  <andrew.burgess@embecosm.com>
 
        * ada-exp.y (find_primitive_type): Make parameter const.
index 75cc05e2a71fc44cd6d91008c02e2f0bc6b5f3dd..ec5d6d181cf507690779b36da53ed4da8399fa21 100644 (file)
@@ -41,6 +41,9 @@
 
 /* Local functions */
 
+static struct value *fortran_argument_convert (struct value *value,
+                                              bool is_artificial);
+
 /* Return the encoding that should be used for the character type
    TYPE.  */
 
@@ -844,9 +847,21 @@ _initialize_f_language ()
   f_type_data = gdbarch_data_register_post_init (build_fortran_types);
 }
 
-/* See f-lang.h.  */
+/* Ensures that function argument VALUE is in the appropriate form to
+   pass to a Fortran function.  Returns a possibly new value that should
+   be used instead of VALUE.
+
+   When IS_ARTIFICIAL is true this indicates an artificial argument,
+   e.g. hidden string lengths which the GNU Fortran argument passing
+   convention specifies as being passed by value.
 
-struct value *
+   When IS_ARTIFICIAL is false, the argument is passed by pointer.  If the
+   value is already in target memory then return a value that is a pointer
+   to VALUE.  If VALUE is not in memory (e.g. an integer literal), allocate
+   space in the target, copy VALUE in, and return a pointer to the in
+   memory copy.  */
+
+static struct value *
 fortran_argument_convert (struct value *value, bool is_artificial)
 {
   if (!is_artificial)
index e59fdef1b19c8286a03931e191511e182dd1c363..8e693eb25e046ed4299e61cad1f8b407805f67d9 100644 (file)
@@ -312,8 +312,6 @@ extern LONGEST f77_get_upperbound (struct type *);
 
 extern LONGEST f77_get_lowerbound (struct type *);
 
-extern void f77_get_dynamic_array_length (struct type *);
-
 extern int calc_f77_array_dims (struct type *);
 
 
@@ -341,23 +339,6 @@ struct builtin_f_type
 /* Return the Fortran type table for the specified architecture.  */
 extern const struct builtin_f_type *builtin_f_type (struct gdbarch *gdbarch);
 
-/* Ensures that function argument VALUE is in the appropriate form to
-   pass to a Fortran function.  Returns a possibly new value that should
-   be used instead of VALUE.
-
-   When IS_ARTIFICIAL is true this indicates an artificial argument,
-   e.g. hidden string lengths which the GNU Fortran argument passing
-   convention specifies as being passed by value.
-
-   When IS_ARTIFICIAL is false, the argument is passed by pointer.  If the
-   value is already in target memory then return a value that is a pointer
-   to VALUE.  If VALUE is not in memory (e.g. an integer literal), allocate
-   space in the target, copy VALUE in, and return a pointer to the in
-   memory copy.  */
-
-extern struct value *fortran_argument_convert (struct value *value,
-                                              bool is_artificial);
-
 /* Ensures that function argument TYPE is appropriate to inform the debugger
    that ARG should be passed as a pointer.  Returns the potentially updated
    argument type.