From da78a59744619b2b4e082eb6aa42a1e5b22628ce Mon Sep 17 00:00:00 2001 From: Martin Kaiser Date: Tue, 7 Mar 2017 22:16:02 +0100 Subject: [PATCH] ftypes: move set_value_bytes into the union Delete set_value_bytes from struct _ftype_t, make it part of the set_value union. Change-Id: I3ee6a9f68b269ee083794947816e13ea6c1b9d29 Reviewed-on: https://code.wireshark.org/review/20440 Petri-Dish: Martin Kaiser Tested-by: Petri Dish Buildbot Reviewed-by: Martin Kaiser --- epan/ftypes/ftype-bytes.c | 17 ++++------------- epan/ftypes/ftype-double.c | 2 -- epan/ftypes/ftype-guid.c | 1 - epan/ftypes/ftype-ieee-11073-float.c | 2 -- epan/ftypes/ftype-integer.c | 21 --------------------- epan/ftypes/ftype-ipv4.c | 1 - epan/ftypes/ftype-ipv6.c | 3 +-- epan/ftypes/ftype-none.c | 1 - epan/ftypes/ftype-pcre.c | 1 - epan/ftypes/ftype-protocol.c | 1 - epan/ftypes/ftype-string.c | 4 ---- epan/ftypes/ftype-time.c | 2 -- epan/ftypes/ftypes-int.h | 2 +- epan/ftypes/ftypes.c | 9 +++++++-- 14 files changed, 13 insertions(+), 54 deletions(-) diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c index 3f15f737ff..63be839583 100644 --- a/epan/ftypes/ftype-bytes.c +++ b/epan/ftypes/ftype-bytes.c @@ -739,7 +739,6 @@ ftype_register_bytes(void) { .set_value_byte_array = bytes_fvalue_set }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -785,7 +784,6 @@ ftype_register_bytes(void) { .set_value_byte_array = bytes_fvalue_set }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -829,9 +827,8 @@ ftype_register_bytes(void) bytes_to_repr, /* val_to_string_repr */ bytes_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_bytes = ax25_fvalue_set }, /* union set_value */ - ax25_fvalue_set, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -875,9 +872,8 @@ ftype_register_bytes(void) bytes_to_repr, /* val_to_string_repr */ bytes_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_bytes = vines_fvalue_set }, /* union set_value */ - vines_fvalue_set, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -921,9 +917,8 @@ ftype_register_bytes(void) bytes_to_repr, /* val_to_string_repr */ bytes_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_bytes = ether_fvalue_set }, /* union set_value */ - ether_fvalue_set, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -969,7 +964,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1015,7 +1009,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1061,7 +1054,6 @@ ftype_register_bytes(void) { .set_value_byte_array = system_id_fvalue_set }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1105,9 +1097,8 @@ ftype_register_bytes(void) bytes_to_repr, /* val_to_string_repr */ bytes_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */ - fcwwn_fvalue_set, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c index 998777ed3a..1fb5bafe84 100644 --- a/epan/ftypes/ftype-double.c +++ b/epan/ftypes/ftype-double.c @@ -172,7 +172,6 @@ ftype_register_double(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -218,7 +217,6 @@ ftype_register_double(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c index a8dd20b6ec..92a7829599 100644 --- a/epan/ftypes/ftype-guid.c +++ b/epan/ftypes/ftype-guid.c @@ -137,7 +137,6 @@ ftype_register_guid(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ guid_fvalue_set_guid, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c index 55ea8ee025..2ebafe7283 100644 --- a/epan/ftypes/ftype-ieee-11073-float.c +++ b/epan/ftypes/ftype-ieee-11073-float.c @@ -1440,7 +1440,6 @@ Example: 114 is 0x0072 { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1513,7 +1512,6 @@ Example: 36.4 is 0xFF00016C { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c index 1d39a7fd9b..597b8dd399 100644 --- a/epan/ftypes/ftype-integer.c +++ b/epan/ftypes/ftype-integer.c @@ -1028,7 +1028,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1073,7 +1072,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1118,7 +1116,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1163,7 +1160,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1208,7 +1204,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1253,7 +1248,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1298,7 +1292,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1343,7 +1336,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1388,7 +1380,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1433,7 +1424,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1478,7 +1468,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1523,7 +1512,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1568,7 +1556,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1613,7 +1600,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1658,7 +1644,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1703,7 +1688,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1748,7 +1732,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1793,7 +1776,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1839,7 +1821,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1885,7 +1866,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ @@ -1931,7 +1911,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c index bcea7d24e6..d937f48549 100644 --- a/epan/ftypes/ftype-ipv4.c +++ b/epan/ftypes/ftype-ipv4.c @@ -187,7 +187,6 @@ ftype_register_ipv4(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c index 8c2846584a..f849df7db1 100644 --- a/epan/ftypes/ftype-ipv6.c +++ b/epan/ftypes/ftype-ipv6.c @@ -235,9 +235,8 @@ ftype_register_ipv6(void) ipv6_to_repr, /* val_to_string_repr */ ipv6_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_bytes = ipv6_fvalue_set }, /* union set_value */ - ipv6_fvalue_set, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c index c863350b31..a69f75699d 100644 --- a/epan/ftypes/ftype-none.c +++ b/epan/ftypes/ftype-none.c @@ -41,7 +41,6 @@ ftype_register_none(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c index a479dcabb8..f711615302 100644 --- a/epan/ftypes/ftype-pcre.c +++ b/epan/ftypes/ftype-pcre.c @@ -165,7 +165,6 @@ ftype_register_pcre(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ gregex_fvalue_set, /* set_value_string */ diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c index abbe86e793..81b25ad567 100644 --- a/epan/ftypes/ftype-protocol.c +++ b/epan/ftypes/ftype-protocol.c @@ -465,7 +465,6 @@ ftype_register_tvbuff(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c index d9ae82c3ba..4f2d54d774 100644 --- a/epan/ftypes/ftype-string.c +++ b/epan/ftypes/ftype-string.c @@ -242,7 +242,6 @@ ftype_register_string(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ string_fvalue_set_string, /* set_value_string */ @@ -287,7 +286,6 @@ ftype_register_string(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ string_fvalue_set_string, /* set_value_string */ @@ -332,7 +330,6 @@ ftype_register_string(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ string_fvalue_set_string, /* set_value_string */ @@ -377,7 +374,6 @@ ftype_register_string(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ NULL, /* set_value_time */ string_fvalue_set_string, /* set_value_string */ diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c index e75e90dfe6..54486851b3 100644 --- a/epan/ftypes/ftype-time.c +++ b/epan/ftypes/ftype-time.c @@ -397,7 +397,6 @@ ftype_register_time(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ time_fvalue_set, /* set_value_time */ NULL, /* set_value_string */ @@ -442,7 +441,6 @@ ftype_register_time(void) { NULL }, /* union set_value */ - NULL, /* set_value_bytes */ NULL, /* set_value_guid */ time_fvalue_set, /* set_value_time */ NULL, /* set_value_string */ diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h index 81293616e9..1009a36d12 100644 --- a/epan/ftypes/ftypes-int.h +++ b/epan/ftypes/ftypes-int.h @@ -92,10 +92,10 @@ struct _ftype_t { union { FvalueSetByteArrayFunc set_value_byte_array; + FvalueSetBytesFunc set_value_bytes; } set_value; /* could be union */ - FvalueSetBytesFunc set_value_bytes; FvalueSetGuidFunc set_value_guid; FvalueSetTimeFunc set_value_time; FvalueSetStringFunc set_value_string; diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c index 6bdc466460..2f7583e40f 100644 --- a/epan/ftypes/ftypes.c +++ b/epan/ftypes/ftypes.c @@ -511,8 +511,13 @@ fvalue_set_byte_array(fvalue_t *fv, GByteArray *value) void fvalue_set_bytes(fvalue_t *fv, const guint8 *value) { - g_assert(fv->ftype->set_value_bytes); - fv->ftype->set_value_bytes(fv, value); + g_assert(fv->ftype->ftype == FT_AX25 || + fv->ftype->ftype == FT_VINES || + fv->ftype->ftype == FT_ETHER || + fv->ftype->ftype == FT_FCWWN || + fv->ftype->ftype == FT_IPv6); + g_assert(fv->ftype->set_value.set_value_bytes); + fv->ftype->set_value.set_value_bytes(fv, value); } void