[PATCH] Remove unused and probably incorrect macro TYPE_FN_FIELDS

[PATCH] Remove unused and probably incorrect macro TYPE_FN_FIELDS

[Siva Chandra]

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

Probably an obvious commit, but I might not be aware of any historical
decision around this.

Patch is attached. Tested by building on x86_64 GNU/Linux.

2014-04-16  Siva Chandra Reddy  <sivachandra@google.com>

        * gdbtypes.h: Remove definition of the macro TYPE_FN_FIELDS.

[-- Attachment #2: remove_type_fn_fields.txt --]
[-- Type: text/plain, Size: 663 bytes --]

diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
index e91ab97..e63d255 100644
--- a/gdb/gdbtypes.h
+++ b/gdb/gdbtypes.h
@@ -1205,7 +1205,6 @@ extern void allocate_gnat_aux_type (struct type *);
 #define TYPE_VPTR_BASETYPE(thistype) TYPE_MAIN_TYPE(thistype)->vptr_basetype
 #define TYPE_DOMAIN_TYPE(thistype) TYPE_MAIN_TYPE(thistype)->vptr_basetype
 #define TYPE_VPTR_FIELDNO(thistype) TYPE_MAIN_TYPE(thistype)->vptr_fieldno
-#define TYPE_FN_FIELDS(thistype) TYPE_CPLUS_SPECIFIC(thistype)->fn_fields
 #define TYPE_NFN_FIELDS(thistype) TYPE_CPLUS_SPECIFIC(thistype)->nfn_fields
 #define TYPE_SPECIFIC_FIELD(thistype) \
   TYPE_MAIN_TYPE(thistype)->type_specific_field

Re: [PATCH] Remove unused and probably incorrect macro TYPE_FN_FIELDS

[Joel Brobecker]

> 2014-04-16  Siva Chandra Reddy  <sivachandra@google.com>
> 
>         * gdbtypes.h: Remove definition of the macro TYPE_FN_FIELDS.

I'm not aware of the history either, but it's really easy to put back
should there be any reason to, so go ahead and push.

Thanks!

> diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h
> index e91ab97..e63d255 100644
> --- a/gdb/gdbtypes.h
> +++ b/gdb/gdbtypes.h
> @@ -1205,7 +1205,6 @@ extern void allocate_gnat_aux_type (struct type *);
>  #define TYPE_VPTR_BASETYPE(thistype) TYPE_MAIN_TYPE(thistype)->vptr_basetype
>  #define TYPE_DOMAIN_TYPE(thistype) TYPE_MAIN_TYPE(thistype)->vptr_basetype
>  #define TYPE_VPTR_FIELDNO(thistype) TYPE_MAIN_TYPE(thistype)->vptr_fieldno
> -#define TYPE_FN_FIELDS(thistype) TYPE_CPLUS_SPECIFIC(thistype)->fn_fields
>  #define TYPE_NFN_FIELDS(thistype) TYPE_CPLUS_SPECIFIC(thistype)->nfn_fields
>  #define TYPE_SPECIFIC_FIELD(thistype) \
>    TYPE_MAIN_TYPE(thistype)->type_specific_field


-- 
Joel

Re: [PATCH] Remove unused and probably incorrect macro TYPE_FN_FIELDS

[Siva Chandra]

On Wed, Apr 16, 2014 at 8:36 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> 2014-04-16  Siva Chandra Reddy  <sivachandra@google.com>
>>
>>         * gdbtypes.h: Remove definition of the macro TYPE_FN_FIELDS.
>
> I'm not aware of the history either, but it's really easy to put back
> should there be any reason to, so go ahead and push.

Done, but with two pushes as I missed adding the ChangeLog entry in
the first push:
8000c58e455259951daa4370631aa616b367d1a7 and
7a23c5494be60600d34527f5adeaa2a34b85fdb6