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

ftypes: move set_value_sinteger64 into the union 

Delete set_value_sinteger64 from struct _ftype_t, make it
part of the set_value union.

Change-Id: I76668f0a6455c5d75c703cd4f440757601246bdb
Reviewed-on: https://code.wireshark.org/review/20551
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-14 23:00:36 +01:00 committed by Michael Mann
parent 42d410b8e3
commit 5f4f62bde0
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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -777,7 +776,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = bytes_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -815,7 +813,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ax25_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -853,7 +850,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = vines_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -891,7 +887,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ether_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -929,7 +924,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -967,7 +961,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -1005,7 +998,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = system_id_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -1043,7 +1035,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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

@ -172,7 +172,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_sinteger64 */
double_fvalue_set_floating, /* set_value_floating */
NULL, /* get_value */
@ -210,7 +209,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_sinteger64 */
double_fvalue_set_floating, /* set_value_floating */
NULL, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1505,7 +1504,6 @@ Example: 36.4 is 0xFF00016C
{ .set_value_uinteger = float_ieee_11073_value_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1065,7 +1064,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1102,7 +1100,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1139,7 +1136,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1176,7 +1172,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1213,7 +1208,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1250,7 +1244,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1287,7 +1280,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1324,7 +1316,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1361,7 +1352,6 @@ ftype_register_integers(void)
{ .set_value_sinteger = set_sinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1398,7 +1388,6 @@ ftype_register_integers(void)
{ .set_value_sinteger = set_sinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1435,7 +1424,6 @@ ftype_register_integers(void)
{ .set_value_sinteger = set_sinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1472,7 +1460,6 @@ ftype_register_integers(void)
{ .set_value_sinteger = set_sinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1507,9 +1494,8 @@ ftype_register_integers(void)
integer64_to_repr, /* val_to_string_repr */
integer64_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
set_sinteger64, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1544,9 +1530,8 @@ ftype_register_integers(void)
integer64_to_repr, /* val_to_string_repr */
integer64_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
set_sinteger64, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1581,9 +1566,8 @@ ftype_register_integers(void)
integer64_to_repr, /* val_to_string_repr */
integer64_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
set_sinteger64, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1618,9 +1602,8 @@ ftype_register_integers(void)
integer64_to_repr, /* val_to_string_repr */
integer64_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
set_sinteger64, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1657,7 +1640,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1695,7 +1677,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1733,7 +1714,6 @@ ftype_register_integers(void)
{ .set_value_uinteger = set_uinteger }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */
@ -1771,7 +1751,6 @@ ftype_register_integers(void)
{ .set_value_uinteger64 = set_uinteger64 }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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

@ -41,7 +41,6 @@ ftype_register_none(void)
{ NULL }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
NULL, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
gregex_fvalue_get, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -279,7 +278,6 @@ ftype_register_string(void)
{ .set_value_string = string_fvalue_set_string }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -316,7 +314,6 @@ ftype_register_string(void)
{ .set_value_string = string_fvalue_set_string }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -353,7 +350,6 @@ ftype_register_string(void)
{ .set_value_string = string_fvalue_set_string }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */
@ -434,7 +433,6 @@ ftype_register_time(void)
{ .set_value_time = time_fvalue_set }, /* union set_value */
NULL, /* set_value_sinteger64 */
NULL, /* set_value_floating */
value_get, /* get_value */

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

@ -100,10 +100,10 @@ struct _ftype_t {
FvalueSetUnsignedIntegerFunc set_value_uinteger;
FvalueSetSignedIntegerFunc set_value_sinteger;
FvalueSetUnsignedInteger64Func set_value_uinteger64;
FvalueSetSignedInteger64Func set_value_sinteger64;
} set_value;
/* could be union */
FvalueSetSignedInteger64Func set_value_sinteger64;
FvalueSetFloatingFunc set_value_floating;
/* could be union */

8
epan/ftypes/ftypes.c
View File

@ -598,8 +598,12 @@ fvalue_set_uinteger64(fvalue_t *fv, guint64 value)
void
fvalue_set_sinteger64(fvalue_t *fv, gint64 value)
{
g_assert(fv->ftype->set_value_sinteger64);
fv->ftype->set_value_sinteger64(fv, value);
g_assert(fv->ftype->ftype == FT_INT40 ||
fv->ftype->ftype == FT_INT48 ||
fv->ftype->ftype == FT_INT56 ||
fv->ftype->ftype == FT_INT64);
g_assert(fv->ftype->set_value.set_value_sinteger64);
fv->ftype->set_value.set_value_sinteger64(fv, value);
}
void