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

ftypes: move set_value_bytes into the union 

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

Change-Id: I3ee6a9f68b269ee083794947816e13ea6c1b9d29
Reviewed-on: https://code.wireshark.org/review/20440
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
This commit is contained in:
Martin Kaiser 2017-03-07 22:16:02 +01:00
parent af8361a90e
commit da78a59744
14 changed files with 13 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
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_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -785,7 +784,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = bytes_fvalue_set }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -829,9 +827,8 @@ ftype_register_bytes(void)
bytes_to_repr, /* val_to_string_repr */
bytes_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_bytes = ax25_fvalue_set }, /* union set_value */
ax25_fvalue_set, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -875,9 +872,8 @@ ftype_register_bytes(void)
bytes_to_repr, /* val_to_string_repr */
bytes_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_bytes = vines_fvalue_set }, /* union set_value */
vines_fvalue_set, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -921,9 +917,8 @@ ftype_register_bytes(void)
bytes_to_repr, /* val_to_string_repr */
bytes_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_bytes = ether_fvalue_set }, /* union set_value */
ether_fvalue_set, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -969,7 +964,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1015,7 +1009,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1061,7 +1054,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = system_id_fvalue_set }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1105,9 +1097,8 @@ ftype_register_bytes(void)
bytes_to_repr, /* val_to_string_repr */
bytes_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */
fcwwn_fvalue_set, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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

@ -172,7 +172,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -218,7 +217,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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

@ -137,7 +137,6 @@ ftype_register_guid(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
guid_fvalue_set_guid, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1513,7 +1512,6 @@ Example: 36.4 is 0xFF00016C
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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

@ -1028,7 +1028,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1073,7 +1072,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1118,7 +1116,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1163,7 +1160,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1208,7 +1204,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1253,7 +1248,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1298,7 +1292,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1343,7 +1336,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1388,7 +1380,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1433,7 +1424,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1478,7 +1468,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1523,7 +1512,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1568,7 +1556,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1613,7 +1600,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1658,7 +1644,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1703,7 +1688,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1748,7 +1732,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1793,7 +1776,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1839,7 +1821,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1885,7 +1866,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */
@ -1931,7 +1911,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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

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

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

@ -235,9 +235,8 @@ ftype_register_ipv6(void)
ipv6_to_repr, /* val_to_string_repr */
ipv6_repr_len, /* len_string_repr */
{ NULL }, /* union set_value */
{ .set_value_bytes = ipv6_fvalue_set }, /* union set_value */
ipv6_fvalue_set, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
NULL, /* set_value_string */

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

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

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

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

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

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

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

@ -242,7 +242,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
@ -287,7 +286,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
@ -332,7 +330,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */
@ -377,7 +374,6 @@ ftype_register_string(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
NULL, /* set_value_time */
string_fvalue_set_string, /* set_value_string */

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

@ -397,7 +397,6 @@ ftype_register_time(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
time_fvalue_set, /* set_value_time */
NULL, /* set_value_string */
@ -442,7 +441,6 @@ ftype_register_time(void)
{ NULL }, /* union set_value */
NULL, /* set_value_bytes */
NULL, /* set_value_guid */
time_fvalue_set, /* set_value_time */
NULL, /* set_value_string */

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

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

9
epan/ftypes/ftypes.c
View File

@ -511,8 +511,13 @@ fvalue_set_byte_array(fvalue_t *fv, GByteArray *value)
void
fvalue_set_bytes(fvalue_t *fv, const guint8 *value)
{
g_assert(fv->ftype->set_value_bytes);
fv->ftype->set_value_bytes(fv, value);
g_assert(fv->ftype->ftype == FT_AX25 ||
fv->ftype->ftype == FT_VINES ||
fv->ftype->ftype == FT_ETHER ||
fv->ftype->ftype == FT_FCWWN ||
fv->ftype->ftype == FT_IPv6);
g_assert(fv->ftype->set_value.set_value_bytes);
fv->ftype->set_value.set_value_bytes(fv, value);
}
void