Dan Hipschman wrote:
diff --git a/tools/widl/widltypes.h b/tools/widl/widltypes.h index d39def3..cf0f246 100644 --- a/tools/widl/widltypes.h +++ b/tools/widl/widltypes.h @@ -207,6 +207,7 @@ struct _type_t { func_t *funcs; /* interfaces and modules */ var_t *fields; /* interfaces, structures and enumerations */ ifref_t *ifaces; /* coclasses */
- type_t *orig; /* dup'd types */ int ignore, is_const, sign; int defined, written, user_types_registered; int typelib_idx;
You add a new field to type_t, yet no code uses it. Will it have some future purpose?
On Tue, Aug 29, 2006 at 01:46:52PM +0100, Robert Shearman wrote:
@@ -207,6 +207,7 @@ struct _type_t { func_t *funcs; /* interfaces and modules */ var_t *fields; /* interfaces, structures and enumerations */ ifref_t *ifaces; /* coclasses */
- type_t *orig; /* dup'd types */
int ignore, is_const, sign; int defined, written, user_types_registered; int typelib_idx;
You add a new field to type_t, yet no code uses it. Will it have some future purpose?
It takes the place of ref for typedefs. I.e., it refers to the typedef'd type. It's used in write_typedef, check_for_user_types, encode_type, and add_typedef_typeinfo in the third patch of my last series. It's true no code used it in this earlier patch (except duptype, which only initialized it).
-
Dan Hipschman -
Robert Shearman