diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-03-12 19:15:19 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-03-13 03:06:31 +0000 |
commit | 9e82cffeea10fce62cdfc49cec3b9e3794ac04bf (patch) | |
tree | b7ca3b6784efb5477acbf0ed16194dcf9c2c733d | |
parent | c70927c0e11a251aec14bc3bb391cd18cacba6db (diff) | |
download | wireshark-9e82cffeea10fce62cdfc49cec3b9e3794ac04bf.tar.gz |
ftypes: move set_value_uinteger into the union
Delete set_value_uinteger from struct _ftype_t, make it part of the
set_value union.
Change-Id: I26de15be8ea0ede76236a8f8a4958bfd97ad9d52
Reviewed-on: https://code.wireshark.org/review/20521
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
-rw-r--r-- | epan/ftypes/ftype-bytes.c | 9 | ||||
-rw-r--r-- | epan/ftypes/ftype-double.c | 2 | ||||
-rw-r--r-- | epan/ftypes/ftype-guid.c | 1 | ||||
-rw-r--r-- | epan/ftypes/ftype-ieee-11073-float.c | 6 | ||||
-rw-r--r-- | epan/ftypes/ftype-integer.c | 35 | ||||
-rw-r--r-- | epan/ftypes/ftype-ipv4.c | 3 | ||||
-rw-r--r-- | epan/ftypes/ftype-ipv6.c | 1 | ||||
-rw-r--r-- | epan/ftypes/ftype-none.c | 1 | ||||
-rw-r--r-- | epan/ftypes/ftype-pcre.c | 1 | ||||
-rw-r--r-- | epan/ftypes/ftype-protocol.c | 1 | ||||
-rw-r--r-- | epan/ftypes/ftype-string.c | 4 | ||||
-rw-r--r-- | epan/ftypes/ftype-time.c | 2 | ||||
-rw-r--r-- | epan/ftypes/ftypes-int.h | 2 | ||||
-rw-r--r-- | epan/ftypes/ftypes.c | 14 |
14 files changed, 23 insertions, 59 deletions
diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c index 3140945ba4..05df2e9a42 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_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -780,7 +779,6 @@ ftype_register_bytes(void) { .set_value_byte_array = bytes_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -821,7 +819,6 @@ ftype_register_bytes(void) { .set_value_bytes = ax25_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_integer */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -862,7 +859,6 @@ ftype_register_bytes(void) { .set_value_bytes = vines_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_integer */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -903,7 +899,6 @@ ftype_register_bytes(void) { .set_value_bytes = ether_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -944,7 +939,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -985,7 +979,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1026,7 +1019,6 @@ ftype_register_bytes(void) { .set_value_byte_array = system_id_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1067,7 +1059,6 @@ ftype_register_bytes(void) { .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c index 01dde67149..7e5f01cfd8 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_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -213,7 +212,6 @@ ftype_register_double(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c index 9d11aa1574..7a1a007c48 100644 --- a/epan/ftypes/ftype-guid.c +++ b/epan/ftypes/ftype-guid.c @@ -137,7 +137,6 @@ ftype_register_guid(void) { .set_value_guid = guid_fvalue_set_guid }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c index 7400c302b6..3949324f00 100644 --- a/epan/ftypes/ftype-ieee-11073-float.c +++ b/epan/ftypes/ftype-ieee-11073-float.c @@ -1438,9 +1438,8 @@ Example: 114 is 0x0072 sfloat_ieee_11073_val_to_repr, /* val_to_string_repr */ sfloat_ieee_11073_val_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = sfloat_ieee_11073_value_set }, /* union set_value */ - sfloat_ieee_11073_value_set, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1506,9 +1505,8 @@ Example: 36.4 is 0xFF00016C float_ieee_11073_val_to_repr, /* val_to_string_repr */ float_ieee_11073_val_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = float_ieee_11073_value_set }, /* union set_value */ - float_ieee_11073_value_set, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c index 17aaebb8d7..844311ba22 100644 --- a/epan/ftypes/ftype-integer.c +++ b/epan/ftypes/ftype-integer.c @@ -1026,9 +1026,8 @@ ftype_register_integers(void) char_to_repr, /* val_to_string_repr */ char_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1066,9 +1065,8 @@ ftype_register_integers(void) uinteger_to_repr, /* val_to_string_repr */ uinteger_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1106,9 +1104,8 @@ ftype_register_integers(void) uinteger_to_repr, /* val_to_string_repr */ uinteger_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1146,9 +1143,8 @@ ftype_register_integers(void) uinteger_to_repr, /* val_to_string_repr */ uinteger_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1186,9 +1182,8 @@ ftype_register_integers(void) uinteger_to_repr, /* val_to_string_repr */ uinteger_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1228,7 +1223,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1268,7 +1262,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1308,7 +1301,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1348,7 +1340,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1388,7 +1379,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1428,7 +1418,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1468,7 +1457,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1508,7 +1496,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1548,7 +1535,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ @@ -1588,7 +1574,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ @@ -1628,7 +1613,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ @@ -1668,7 +1652,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ @@ -1708,7 +1691,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1747,9 +1729,8 @@ ftype_register_integers(void) ipxnet_to_repr, /* val_to_string_repr */ ipxnet_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* get_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1788,9 +1769,8 @@ ftype_register_integers(void) uinteger_to_repr, /* val_to_string_repr */ uinteger_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -1831,7 +1811,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c index 7611c9f279..34091a81e0 100644 --- a/epan/ftypes/ftype-ipv4.c +++ b/epan/ftypes/ftype-ipv4.c @@ -185,9 +185,8 @@ ftype_register_ipv4(void) val_to_repr, /* val_to_string_repr */ val_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_uinteger = set_uinteger }, /* union set_value */ - set_uinteger, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c index 1ef45755fe..d38836923e 100644 --- a/epan/ftypes/ftype-ipv6.c +++ b/epan/ftypes/ftype-ipv6.c @@ -237,7 +237,6 @@ ftype_register_ipv6(void) { .set_value_bytes = ipv6_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c index c8201b94b7..436fc0b3ed 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_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c index 637d1a753f..308baa6fc0 100644 --- a/epan/ftypes/ftype-pcre.c +++ b/epan/ftypes/ftype-pcre.c @@ -165,7 +165,6 @@ ftype_register_pcre(void) { .set_value_string = gregex_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c index f0f25e9085..d67ff3132e 100644 --- a/epan/ftypes/ftype-protocol.c +++ b/epan/ftypes/ftype-protocol.c @@ -465,7 +465,6 @@ ftype_register_tvbuff(void) { .set_value_protocol = value_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c index 0929d0d3f9..a4454c63f3 100644 --- a/epan/ftypes/ftype-string.c +++ b/epan/ftypes/ftype-string.c @@ -242,7 +242,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -282,7 +281,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -322,7 +320,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -362,7 +359,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c index 6a615a8e68..86f4001919 100644 --- a/epan/ftypes/ftype-time.c +++ b/epan/ftypes/ftype-time.c @@ -397,7 +397,6 @@ ftype_register_time(void) { .set_value_time = time_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ @@ -437,7 +436,6 @@ ftype_register_time(void) { .set_value_time = time_fvalue_set }, /* union set_value */ - NULL, /* set_value_uinteger */ NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h index d5883c74bd..1846b9952e 100644 --- a/epan/ftypes/ftypes-int.h +++ b/epan/ftypes/ftypes-int.h @@ -97,10 +97,10 @@ struct _ftype_t { FvalueSetTimeFunc set_value_time; FvalueSetStringFunc set_value_string; FvalueSetProtocolFunc set_value_protocol; + FvalueSetUnsignedIntegerFunc set_value_uinteger; } set_value; /* could be union */ - FvalueSetUnsignedIntegerFunc set_value_uinteger; FvalueSetSignedIntegerFunc set_value_sinteger; FvalueSetUnsignedInteger64Func set_value_uinteger64; FvalueSetSignedInteger64Func set_value_sinteger64; diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c index 5117849758..9a7b872b42 100644 --- a/epan/ftypes/ftypes.c +++ b/epan/ftypes/ftypes.c @@ -557,8 +557,18 @@ fvalue_set_protocol(fvalue_t *fv, tvbuff_t *value, const gchar *name) void fvalue_set_uinteger(fvalue_t *fv, guint32 value) { - g_assert(fv->ftype->set_value_uinteger); - fv->ftype->set_value_uinteger(fv, value); + g_assert(fv->ftype->ftype == FT_IEEE_11073_SFLOAT || + fv->ftype->ftype == FT_IEEE_11073_FLOAT || + fv->ftype->ftype == FT_CHAR || + fv->ftype->ftype == FT_UINT8 || + fv->ftype->ftype == FT_UINT16 || + fv->ftype->ftype == FT_UINT24 || + fv->ftype->ftype == FT_UINT32 || + fv->ftype->ftype == FT_IPXNET || + fv->ftype->ftype == FT_FRAMENUM || + fv->ftype->ftype == FT_IPv4); + g_assert(fv->ftype->set_value.set_value_uinteger); + fv->ftype->set_value.set_value_uinteger(fv, value); } void |