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

ftypes: move set_value_time into the union 

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

Change-Id: Ieee3b083d52afdd608e31e8ca75b50016fec8a69
Reviewed-on: https://code.wireshark.org/review/20442
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: Peter Wu <peter@lekensteyn.nl>
This commit is contained in:
Martin Kaiser 2017-03-07 22:43:04 +01:00 committed by Peter Wu
parent ae9ccbeb72
commit abcaf3e370
14 changed files with 6 additions and 51 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_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -783,7 +782,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = bytes_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -827,7 +825,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ax25_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -871,7 +868,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = vines_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -915,7 +911,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ether_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -959,7 +954,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1003,7 +997,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1047,7 +1040,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = system_id_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1091,7 +1083,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -172,7 +172,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -216,7 +215,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -1440,7 +1440,6 @@ Example: 114 is 0x0072
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
sfloat_ieee_11073_value_set, /* set_value_uinteger */
@ -1511,7 +1510,6 @@ Example: 36.4 is 0xFF00016C
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
float_ieee_11073_value_set, /* set_value_uinteger */

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

@ -1028,7 +1028,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1071,7 +1070,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1114,7 +1112,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1157,7 +1154,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1200,7 +1196,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1243,7 +1238,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1286,7 +1280,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1329,7 +1322,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1372,7 +1364,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1415,7 +1406,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1458,7 +1448,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1501,7 +1490,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1544,7 +1532,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1587,7 +1574,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1630,7 +1616,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1673,7 +1658,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1716,7 +1700,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1759,7 +1742,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -1803,7 +1785,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1847,7 +1828,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
@ -1891,7 +1871,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -187,7 +187,6 @@ ftype_register_ipv4(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */

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_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -41,7 +41,6 @@ ftype_register_none(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -165,7 +165,6 @@ ftype_register_pcre(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
gregex_fvalue_set, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -465,7 +465,6 @@ ftype_register_tvbuff(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
NULL, /* set_value_string */
value_set, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -242,7 +242,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -285,7 +284,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -328,7 +326,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -371,7 +368,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -395,9 +395,8 @@ ftype_register_time(void)
absolute_val_to_repr, /* val_to_string_repr */
absolute_val_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_time = time_fvalue_set }, /* union set_value */
time_fvalue_set, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
@ -438,9 +437,8 @@ ftype_register_time(void)
relative_val_to_repr, /* val_to_string_repr */
relative_val_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_time = time_fvalue_set }, /* union set_value */
time_fvalue_set, /* set_value_time */
NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */

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

@ -94,10 +94,10 @@ struct _ftype_t {
FvalueSetByteArrayFunc set_value_byte_array;
FvalueSetBytesFunc set_value_bytes;
FvalueSetGuidFunc set_value_guid;
FvalueSetTimeFunc set_value_time;
} set_value;
/* could be union */
FvalueSetTimeFunc set_value_time;
FvalueSetStringFunc set_value_string;
FvalueSetProtocolFunc set_value_protocol;
FvalueSetUnsignedIntegerFunc set_value_uinteger;

5
epan/ftypes/ftypes.c
View File

@ -531,8 +531,9 @@ fvalue_set_guid(fvalue_t *fv, const e_guid_t *value)
void
fvalue_set_time(fvalue_t *fv, const nstime_t *value)
{
g_assert(fv->ftype->set_value_time);
fv->ftype->set_value_time(fv, value);
g_assert(IS_FT_TIME(fv->ftype->ftype));
g_assert(fv->ftype->set_value.set_value_time);
fv->ftype->set_value.set_value_time(fv, value);
}
void