diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-03-12 19:49:42 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-03-13 11:39:43 +0000 |
commit | a6d69084dc18da09d7020b9bfda74cdfea963cad (patch) | |
tree | 12620122f9d45df776624cbe6beaf14360419103 /epan/ftypes | |
parent | 5d217530a171a666ba5530675785e9463f2e20f6 (diff) | |
download | wireshark-a6d69084dc18da09d7020b9bfda74cdfea963cad.tar.gz |
ftypes: move set_value_sinteger into the union
Delete set_value_sinteger from struct _ftype_t, make it part of the
set_value union.
Change-Id: I084c4c9beba9978b538d359ebaac21391ebd7f82
Reviewed-on: https://code.wireshark.org/review/20523
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/ftypes')
-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 | 2 | ||||
-rw-r--r-- | epan/ftypes/ftype-integer.c | 29 | ||||
-rw-r--r-- | epan/ftypes/ftype-ipv4.c | 1 | ||||
-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 | 8 |
14 files changed, 11 insertions, 53 deletions
diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c index 05df2e9a42..eacf3bacd0 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -779,7 +778,6 @@ ftype_register_bytes(void) { .set_value_byte_array = bytes_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -819,7 +817,6 @@ ftype_register_bytes(void) { .set_value_bytes = ax25_fvalue_set }, /* union set_value */ - NULL, /* set_value_integer */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -859,7 +856,6 @@ ftype_register_bytes(void) { .set_value_bytes = vines_fvalue_set }, /* union set_value */ - NULL, /* set_value_integer */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -899,7 +895,6 @@ ftype_register_bytes(void) { .set_value_bytes = ether_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -939,7 +934,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -979,7 +973,6 @@ ftype_register_bytes(void) { .set_value_byte_array = oid_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1019,7 +1012,6 @@ ftype_register_bytes(void) { .set_value_byte_array = system_id_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1059,7 +1051,6 @@ ftype_register_bytes(void) { .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c index 7e5f01cfd8..18a324f5ca 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ double_fvalue_set_floating, /* set_value_floating */ @@ -212,7 +211,6 @@ ftype_register_double(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ double_fvalue_set_floating, /* set_value_floating */ diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c index 7a1a007c48..a3c835b56f 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c index 3949324f00..ee863c115d 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 { .set_value_uinteger = sfloat_ieee_11073_value_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1507,7 +1506,6 @@ Example: 36.4 is 0xFF00016C { .set_value_uinteger = float_ieee_11073_value_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c index 844311ba22..f1d4015f94 100644 --- a/epan/ftypes/ftype-integer.c +++ b/epan/ftypes/ftype-integer.c @@ -1028,7 +1028,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1067,7 +1066,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1106,7 +1104,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1145,7 +1142,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1184,7 +1180,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1223,7 +1218,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1262,7 +1256,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1301,7 +1294,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1340,7 +1332,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1377,9 +1368,8 @@ ftype_register_integers(void) integer_to_repr, /* val_to_string_repr */ integer_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_sinteger = set_sinteger }, /* union set_value */ - set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1416,9 +1406,8 @@ ftype_register_integers(void) integer_to_repr, /* val_to_string_repr */ integer_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_sinteger = set_sinteger }, /* union set_value */ - set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1455,9 +1444,8 @@ ftype_register_integers(void) integer_to_repr, /* val_to_string_repr */ integer_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_sinteger = set_sinteger }, /* union set_value */ - set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1494,9 +1482,8 @@ ftype_register_integers(void) integer_to_repr, /* val_to_string_repr */ integer_repr_len, /* len_string_repr */ - { NULL }, /* union set_value */ + { .set_value_sinteger = set_sinteger }, /* union set_value */ - set_sinteger, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1535,7 +1522,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1574,7 +1560,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1613,7 +1598,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1652,7 +1636,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ set_sinteger64, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1691,7 +1674,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1731,7 +1713,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* get_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1771,7 +1752,6 @@ ftype_register_integers(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -1811,7 +1791,6 @@ ftype_register_integers(void) { NULL }, /* union set_value */ - NULL, /* set_value_sinteger */ set_uinteger64, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c index 34091a81e0..596b3a58a4 100644 --- a/epan/ftypes/ftype-ipv4.c +++ b/epan/ftypes/ftype-ipv4.c @@ -187,7 +187,6 @@ ftype_register_ipv4(void) { .set_value_uinteger = set_uinteger }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c index d38836923e..2c0da9ee79 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c index 436fc0b3ed..007fe4eb1e 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c index 308baa6fc0..d0a52406db 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c index d67ff3132e..843bb7f6aa 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c index a4454c63f3..631275fd42 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -281,7 +280,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -320,7 +318,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -359,7 +356,6 @@ ftype_register_string(void) { .set_value_string = string_fvalue_set_string }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c index 86f4001919..9bb8ed1cd3 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_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ @@ -436,7 +435,6 @@ ftype_register_time(void) { .set_value_time = time_fvalue_set }, /* union set_value */ - NULL, /* set_value_sinteger */ NULL, /* set_value_uinteger64 */ NULL, /* set_value_sinteger64 */ NULL, /* set_value_floating */ diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h index 1846b9952e..961926f6fa 100644 --- a/epan/ftypes/ftypes-int.h +++ b/epan/ftypes/ftypes-int.h @@ -98,10 +98,10 @@ struct _ftype_t { FvalueSetStringFunc set_value_string; FvalueSetProtocolFunc set_value_protocol; FvalueSetUnsignedIntegerFunc set_value_uinteger; + FvalueSetSignedIntegerFunc set_value_sinteger; } set_value; /* could be union */ - FvalueSetSignedIntegerFunc set_value_sinteger; FvalueSetUnsignedInteger64Func set_value_uinteger64; FvalueSetSignedInteger64Func set_value_sinteger64; FvalueSetFloatingFunc set_value_floating; diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c index 9a7b872b42..eff5ff7e22 100644 --- a/epan/ftypes/ftypes.c +++ b/epan/ftypes/ftypes.c @@ -574,8 +574,12 @@ fvalue_set_uinteger(fvalue_t *fv, guint32 value) void fvalue_set_sinteger(fvalue_t *fv, gint32 value) { - g_assert(fv->ftype->set_value_sinteger); - fv->ftype->set_value_sinteger(fv, value); + g_assert(fv->ftype->ftype == FT_INT8 || + fv->ftype->ftype == FT_INT16 || + fv->ftype->ftype == FT_INT24 || + fv->ftype->ftype == FT_INT32); + g_assert(fv->ftype->set_value.set_value_sinteger); + fv->ftype->set_value.set_value_sinteger(fv, value); } void |