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

ftypes: move get_value_sinteger into the union 

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

Change-Id: I3127252cafc62389ce426639992f1d59f7ac9731
Reviewed-on: https://code.wireshark.org/review/20637
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>
This commit is contained in:
Martin Kaiser 2017-03-19 20:32:20 +01:00 committed by Michael Mann
parent 2a3473820d
commit e94bd9b4ee
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

@ -742,7 +742,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -775,7 +774,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -808,7 +806,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -841,7 +838,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -874,7 +870,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -907,7 +902,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -940,7 +934,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -973,7 +966,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1006,7 +998,6 @@ ftype_register_bytes(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -175,7 +175,6 @@ ftype_register_double(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -208,7 +207,6 @@ ftype_register_double(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -140,7 +140,6 @@ ftype_register_guid(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -1443,7 +1443,6 @@ Example: 114 is 0x0072
NULL, /* get_value_ptr */
sfloat_ieee_11073_value_get, /* get_value_uinteger */
NULL, /* get_value_sinteger */
sfloat_ieee_11073_cmp_eq,
sfloat_ieee_11073_cmp_ne,
@ -1503,7 +1502,6 @@ Example: 36.4 is 0xFF00016C
NULL, /* get_value_ptr */
float_ieee_11073_value_get, /* get_value_uinteger */
NULL, /* get_value_sinteger */
float_ieee_11073_cmp_eq,
float_ieee_11073_cmp_ne,

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

@ -1031,7 +1031,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1063,7 +1062,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1095,7 +1093,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1127,7 +1124,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1159,7 +1155,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1191,7 +1186,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1223,7 +1217,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1255,7 +1248,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1287,7 +1279,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1315,11 +1306,10 @@ ftype_register_integers(void)
integer_repr_len, /* len_string_repr */
{ .set_value_sinteger = set_sinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1347,11 +1337,10 @@ ftype_register_integers(void)
integer_repr_len, /* len_string_repr */
{ .set_value_sinteger = set_sinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1379,11 +1368,10 @@ ftype_register_integers(void)
integer_repr_len, /* len_string_repr */
{ .set_value_sinteger = set_sinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_integer */
cmp_eq,
cmp_ne,
@ -1411,11 +1399,10 @@ ftype_register_integers(void)
integer_repr_len, /* len_string_repr */
{ .set_value_sinteger = set_sinteger }, /* union set_value */
{ NULL }, /* union get_value */
{ .get_value_sinteger = get_sinteger }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1447,7 +1434,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1479,7 +1465,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1511,7 +1496,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1543,7 +1527,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,
@ -1575,7 +1558,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
bool_eq, /* cmp_eq */
bool_ne, /* cmp_ne */
@ -1608,7 +1590,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1641,7 +1622,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -1674,7 +1654,6 @@ ftype_register_integers(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq64,
cmp_ne64,

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

@ -190,7 +190,6 @@ ftype_register_ipv4(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -240,7 +240,6 @@ ftype_register_ipv6(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -44,7 +44,6 @@ ftype_register_none(void)
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* cmp_eq */
NULL, /* cmp_ne */

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

@ -168,7 +168,6 @@ ftype_register_pcre(void)
gregex_fvalue_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* cmp_eq */
NULL, /* cmp_ne */

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

@ -468,7 +468,6 @@ ftype_register_tvbuff(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -245,7 +245,6 @@ ftype_register_string(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -277,7 +276,6 @@ ftype_register_string(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -309,7 +307,6 @@ ftype_register_string(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -341,7 +338,6 @@ ftype_register_string(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

@ -400,7 +400,6 @@ ftype_register_time(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,
@ -432,7 +431,6 @@ ftype_register_time(void)
value_get, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
cmp_eq,
cmp_ne,

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

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

8
epan/ftypes/ftypes.c
View File

@ -633,8 +633,12 @@ fvalue_get_uinteger(fvalue_t *fv)
gint32
fvalue_get_sinteger(fvalue_t *fv)
{
g_assert(fv->ftype->get_value_sinteger);
return fv->ftype->get_value_sinteger(fv);
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->get_value.get_value_sinteger);
return fv->ftype->get_value.get_value_sinteger(fv);
}
guint64