osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

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>
This commit is contained in:
Martin Kaiser 2017-03-12 19:49:42 +01:00 committed by Michael Mann
parent 5d217530a1
commit a6d69084dc
14 changed files with 11 additions and 53 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
epan/ftypes/ftype-bytes.c
View File

@ -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 */

2
epan/ftypes/ftype-double.c
View File

@ -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 */

1
epan/ftypes/ftype-guid.c
View File

@ -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 */

2
epan/ftypes/ftype-ieee-11073-float.c
View File

@ -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 */

29
epan/ftypes/ftype-integer.c
View File

@ -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 */

1
epan/ftypes/ftype-ipv4.c
View File

@ -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 */

1
epan/ftypes/ftype-ipv6.c
View File

@ -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 */

1
epan/ftypes/ftype-none.c
View File

@ -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 */

1
epan/ftypes/ftype-pcre.c
View File

@ -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 */

1
epan/ftypes/ftype-protocol.c
View File

@ -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 */

4
epan/ftypes/ftype-string.c
View File

@ -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 */

2
epan/ftypes/ftype-time.c
View File

@ -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 */

2
epan/ftypes/ftypes-int.h
View File

@ -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;

8
epan/ftypes/ftypes.c
View File

@ -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