toddfarmer / arrow-migration

0 stars 1 forks source link

[GLib] Resolve include dependency #1459

Closed toddfarmer closed 7 years ago

toddfarmer commented 7 years ago

Note: This issue was originally created as ARROW-1528. Please see the migration documentation for further details.

Original Issue Description:

* arrow-glib/field.h depends on arrow-glib/data-type.h * arrow-glib/data-type.h depends on arrow-glib/field.h

It causes the following warnings:

../arrow-glib/field.h:47:37: warning: redefinition of typedef 'GArrowField' is a C11 feature [-Wtypedef-redefinition]

typedef struct _GArrowField         GArrowField;

                                    ^

../arrow-glib/data-type.h:29:37: note: previous definition is here

typedef struct _GArrowField         GArrowField;

                                    ^
toddfarmer commented 7 years ago

Note: Comment by Wes McKinney (wesm): Issue resolved by pull request 1091 https://github.com/apache/arrow/pull/1091