osmocom
/
wireshark
11
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

ftypes: move get_value_uinteger into the union 

Delete get_value_uinteger from struct _ftype_t, make it
part of the get_value union.

Change-Id: I4a6c8341676c442e2bf8ae3b8f771b72161d133c
Reviewed-on: https://code.wireshark.org/review/20640
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-19 20:57:43 +01:00 committed by Michael Mann
parent 01b460fb77
commit c258b6b056
14 changed files with 21 additions and 58 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -741,7 +741,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -773,7 +772,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -805,7 +803,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -837,7 +834,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -869,7 +865,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -901,7 +896,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -933,7 +927,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -965,7 +958,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -997,7 +989,6 @@ ftype_register_bytes(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -174,7 +174,6 @@ ftype_register_double(void)
{ .get_value_floating = value_get_floating }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -206,7 +205,6 @@ ftype_register_double(void)
{ .get_value_floating = value_get_floating }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -139,7 +139,6 @@ ftype_register_guid(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -1439,10 +1439,9 @@ Example: 114 is 0x0072
sfloat_ieee_11073_val_repr_len, /* len_string_repr */
{ .set_value_uinteger = sfloat_ieee_11073_value_set }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = sfloat_ieee_11073_value_get }, /* union get_value */
NULL, /* get_value_ptr */
sfloat_ieee_11073_value_get, /* get_value_uinteger */
sfloat_ieee_11073_cmp_eq,
sfloat_ieee_11073_cmp_ne,
@ -1498,10 +1497,9 @@ Example: 36.4 is 0xFF00016C
float_ieee_11073_val_repr_len, /* len_string_repr */
{ .set_value_uinteger = float_ieee_11073_value_set }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = float_ieee_11073_value_get }, /* union get_value */
NULL, /* get_value_ptr */
float_ieee_11073_value_get, /* get_value_uinteger */
float_ieee_11073_cmp_eq,
float_ieee_11073_cmp_ne,

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

@ -1027,10 +1027,9 @@ ftype_register_integers(void)
char_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1058,10 +1057,9 @@ ftype_register_integers(void)
uinteger_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1089,10 +1087,9 @@ ftype_register_integers(void)
uinteger_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -1120,10 +1117,9 @@ ftype_register_integers(void)
uinteger_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -1151,10 +1147,9 @@ ftype_register_integers(void)
uinteger_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -1185,7 +1180,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1216,7 +1210,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1247,7 +1240,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1278,7 +1270,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1309,7 +1300,6 @@ ftype_register_integers(void)
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1340,7 +1330,6 @@ ftype_register_integers(void)
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1371,7 +1360,6 @@ ftype_register_integers(void)
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1402,7 +1390,6 @@ ftype_register_integers(void)
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1433,7 +1420,6 @@ ftype_register_integers(void)
{ .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1464,7 +1450,6 @@ ftype_register_integers(void)
{ .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1495,7 +1480,6 @@ ftype_register_integers(void)
{ .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1526,7 +1510,6 @@ ftype_register_integers(void)
{ .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,
@ -1557,7 +1540,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
bool_eq, /* cmp_eq */
bool_ne, /* cmp_ne */
@ -1586,10 +1568,9 @@ ftype_register_integers(void)
ipxnet_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1618,10 +1599,9 @@ ftype_register_integers(void)
uinteger_repr_len, /* len_string_repr */
{ .set_value_uinteger = set_uinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_uinteger = get_uinteger }, /* union get_value */
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -1653,7 +1633,6 @@ ftype_register_integers(void)
{ .get_value_uinteger64 = get_uinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq64,
cmp_ne64,

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

@ -189,7 +189,6 @@ ftype_register_ipv4(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -239,7 +239,6 @@ ftype_register_ipv6(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -43,7 +43,6 @@ ftype_register_none(void)
{ NULL }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* cmp_eq */
NULL, /* cmp_ne */

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

@ -167,7 +167,6 @@ ftype_register_pcre(void)
{ NULL }, /* union get_value */
gregex_fvalue_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* cmp_eq */
NULL, /* cmp_ne */

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

@ -467,7 +467,6 @@ ftype_register_tvbuff(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -244,7 +244,6 @@ ftype_register_string(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -275,7 +274,6 @@ ftype_register_string(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -306,7 +304,6 @@ ftype_register_string(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -337,7 +334,6 @@ ftype_register_string(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -399,7 +399,6 @@ ftype_register_time(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,
@ -430,7 +429,6 @@ ftype_register_time(void)
{ NULL }, /* union get_value */
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
cmp_eq,
cmp_ne,

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

@ -105,6 +105,7 @@ struct _ftype_t {
} set_value;
union {
FvalueGetUnsignedIntegerFunc get_value_uinteger;
FvalueGetSignedIntegerFunc get_value_sinteger;
FvalueGetUnsignedInteger64Func get_value_uinteger64;
FvalueGetSignedInteger64Func get_value_sinteger64;
@ -113,7 +114,6 @@ struct _ftype_t {
/* could be union */
FvalueGetFunc get_value_ptr;
FvalueGetUnsignedIntegerFunc get_value_uinteger;
FvalueCmp cmp_eq;
FvalueCmp cmp_ne;

13
epan/ftypes/ftypes.c
View File

@ -626,8 +626,17 @@ fvalue_get(fvalue_t *fv)
guint32
fvalue_get_uinteger(fvalue_t *fv)
{
g_assert(fv->ftype->get_value_uinteger);
return fv->ftype->get_value_uinteger(fv);
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);
g_assert(fv->ftype->get_value.get_value_uinteger);
return fv->ftype->get_value.get_value_uinteger(fv);
}
gint32