summaryrefslogtreecommitdiff
path: root/epan/ftypes
diff options
context:
space:
mode:
authorMartin Kaiser <wireshark@kaiser.cx>2017-03-09 22:56:07 +0100
committerMichael Mann <mmann78@netscape.net>2017-03-10 02:35:00 +0000
commit3bcda66971e7eff1bca5155207cb035e185dc6d0 (patch)
treedd8597bd612641d74220f7f93bd413c898ebada8 /epan/ftypes
parente00390842333e6548f365b266dd66bc8710e33a9 (diff)
downloadwireshark-3bcda66971e7eff1bca5155207cb035e185dc6d0.tar.gz
ftypes: move set_value_string into the union
Delete set_value_string from struct _ftype_t, make it part of the set_value union. Change-Id: Ia3a57245330ed0d158c8c3e4552ec5d80752fe0f Reviewed-on: https://code.wireshark.org/review/20474 Reviewed-by: Peter Wu <peter@lekensteyn.nl> Petri-Dish: Peter Wu <peter@lekensteyn.nl> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/ftypes')
-rw-r--r--epan/ftypes/ftype-bytes.c9
-rw-r--r--epan/ftypes/ftype-double.c2
-rw-r--r--epan/ftypes/ftype-guid.c1
-rw-r--r--epan/ftypes/ftype-ieee-11073-float.c2
-rw-r--r--epan/ftypes/ftype-integer.c21
-rw-r--r--epan/ftypes/ftype-ipv4.c1
-rw-r--r--epan/ftypes/ftype-ipv6.c1
-rw-r--r--epan/ftypes/ftype-none.c1
-rw-r--r--epan/ftypes/ftype-pcre.c3
-rw-r--r--epan/ftypes/ftype-protocol.c1
-rw-r--r--epan/ftypes/ftype-string.c12
-rw-r--r--epan/ftypes/ftype-time.c2
-rw-r--r--epan/ftypes/ftypes-int.h2
-rw-r--r--epan/ftypes/ftypes.c7
14 files changed, 11 insertions, 54 deletions
diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c
index 92057c32a2..817a1d064c 100644
--- a/epan/ftypes/ftype-bytes.c
+++ b/epan/ftypes/ftype-bytes.c
@@ -739,7 +739,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = bytes_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -782,7 +781,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = bytes_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -825,7 +823,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ax25_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_integer */
@@ -868,7 +865,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = vines_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_integer */
@@ -911,7 +907,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = ether_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -954,7 +949,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -997,7 +991,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = oid_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1040,7 +1033,6 @@ ftype_register_bytes(void)
{ .set_value_byte_array = system_id_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1083,7 +1075,6 @@ ftype_register_bytes(void)
{ .set_value_bytes = fcwwn_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c
index 3f816eb103..dd7bb4cf89 100644
--- a/epan/ftypes/ftype-double.c
+++ b/epan/ftypes/ftype-double.c
@@ -172,7 +172,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -215,7 +214,6 @@ ftype_register_double(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c
index 28b72ed02f..959ed8e9fc 100644
--- a/epan/ftypes/ftype-guid.c
+++ b/epan/ftypes/ftype-guid.c
@@ -137,7 +137,6 @@ ftype_register_guid(void)
{ .set_value_guid = guid_fvalue_set_guid }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c
index 62f30fd907..15480ea828 100644
--- a/epan/ftypes/ftype-ieee-11073-float.c
+++ b/epan/ftypes/ftype-ieee-11073-float.c
@@ -1440,7 +1440,6 @@ Example: 114 is 0x0072
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
sfloat_ieee_11073_value_set, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1510,7 +1509,6 @@ Example: 36.4 is 0xFF00016C
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
float_ieee_11073_value_set, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c
index ea47eeeb7c..782aa0193d 100644
--- a/epan/ftypes/ftype-integer.c
+++ b/epan/ftypes/ftype-integer.c
@@ -1028,7 +1028,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1070,7 +1069,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1112,7 +1110,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1154,7 +1151,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1196,7 +1192,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1238,7 +1233,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1280,7 +1274,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1322,7 +1315,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1364,7 +1356,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1406,7 +1397,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
set_sinteger, /* set_value_sinteger */
@@ -1448,7 +1438,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
set_sinteger, /* set_value_sinteger */
@@ -1490,7 +1479,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
set_sinteger, /* set_value_sinteger */
@@ -1532,7 +1520,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
set_sinteger, /* set_value_sinteger */
@@ -1574,7 +1561,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1616,7 +1602,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1658,7 +1643,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1700,7 +1684,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1742,7 +1725,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1785,7 +1767,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* get_value_sinteger */
@@ -1828,7 +1809,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -1871,7 +1851,6 @@ ftype_register_integers(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c
index 67ff88e5ba..7761e0c105 100644
--- a/epan/ftypes/ftype-ipv4.c
+++ b/epan/ftypes/ftype-ipv4.c
@@ -187,7 +187,6 @@ ftype_register_ipv4(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
set_uinteger, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c
index 521754bfae..98eb03fbd5 100644
--- a/epan/ftypes/ftype-ipv6.c
+++ b/epan/ftypes/ftype-ipv6.c
@@ -237,7 +237,6 @@ ftype_register_ipv6(void)
{ .set_value_bytes = ipv6_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c
index 6dfbfa0f44..320a8069b7 100644
--- a/epan/ftypes/ftype-none.c
+++ b/epan/ftypes/ftype-none.c
@@ -41,7 +41,6 @@ ftype_register_none(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c
index a4beee0d1a..35d0e48b88 100644
--- a/epan/ftypes/ftype-pcre.c
+++ b/epan/ftypes/ftype-pcre.c
@@ -163,9 +163,8 @@ ftype_register_pcre(void)
gregex_to_repr, /* val_to_string_repr */
gregex_repr_len, /* len_string_repr */
- { NULL }, /* union set_value */
+ { .set_value_string = gregex_fvalue_set }, /* union set_value */
- gregex_fvalue_set, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c
index d9135eb749..64aa09a830 100644
--- a/epan/ftypes/ftype-protocol.c
+++ b/epan/ftypes/ftype-protocol.c
@@ -465,7 +465,6 @@ ftype_register_tvbuff(void)
{ NULL }, /* union set_value */
- NULL, /* set_value_string */
value_set, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c
index dad4e3ad71..d0fec6145f 100644
--- a/epan/ftypes/ftype-string.c
+++ b/epan/ftypes/ftype-string.c
@@ -240,9 +240,8 @@ ftype_register_string(void)
string_to_repr, /* val_to_string_repr */
string_repr_len, /* len_string_repr */
- { NULL }, /* union set_value */
+ { .set_value_string = string_fvalue_set_string }, /* union set_value */
- string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -282,9 +281,8 @@ ftype_register_string(void)
string_to_repr, /* val_to_string_repr */
string_repr_len, /* len_string_repr */
- { NULL }, /* union set_value */
+ { .set_value_string = string_fvalue_set_string }, /* union set_value */
- string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -324,9 +322,8 @@ ftype_register_string(void)
string_to_repr, /* val_to_string_repr */
string_repr_len, /* len_string_repr */
- { NULL }, /* union set_value */
+ { .set_value_string = string_fvalue_set_string }, /* union set_value */
- string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -366,9 +363,8 @@ ftype_register_string(void)
string_to_repr, /* val_to_string_repr */
string_repr_len, /* len_string_repr */
- { NULL }, /* union set_value */
+ { .set_value_string = string_fvalue_set_string }, /* union set_value */
- string_fvalue_set_string, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c
index 0f23e76fb1..4c11f579d3 100644
--- a/epan/ftypes/ftype-time.c
+++ b/epan/ftypes/ftype-time.c
@@ -397,7 +397,6 @@ ftype_register_time(void)
{ .set_value_time = time_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
@@ -439,7 +438,6 @@ ftype_register_time(void)
{ .set_value_time = time_fvalue_set }, /* union set_value */
- NULL, /* set_value_string */
NULL, /* set_value_protocol */
NULL, /* set_value_uinteger */
NULL, /* set_value_sinteger */
diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h
index 8ba8286dc2..bf65e9b0da 100644
--- a/epan/ftypes/ftypes-int.h
+++ b/epan/ftypes/ftypes-int.h
@@ -95,10 +95,10 @@ struct _ftype_t {
FvalueSetBytesFunc set_value_bytes;
FvalueSetGuidFunc set_value_guid;
FvalueSetTimeFunc set_value_time;
+ FvalueSetStringFunc set_value_string;
} set_value;
/* could be union */
- FvalueSetStringFunc set_value_string;
FvalueSetProtocolFunc set_value_protocol;
FvalueSetUnsignedIntegerFunc set_value_uinteger;
FvalueSetSignedIntegerFunc set_value_sinteger;
diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c
index 04d74ca92d..21ae24cc04 100644
--- a/epan/ftypes/ftypes.c
+++ b/epan/ftypes/ftypes.c
@@ -539,8 +539,11 @@ fvalue_set_time(fvalue_t *fv, const nstime_t *value)
void
fvalue_set_string(fvalue_t *fv, const gchar *value)
{
- g_assert(fv->ftype->set_value_string);
- fv->ftype->set_value_string(fv, value);
+ g_assert(IS_FT_STRING(fv->ftype->ftype) ||
+ fv->ftype->ftype == FT_UINT_STRING ||
+ fv->ftype->ftype == FT_PCRE);
+ g_assert(fv->ftype->set_value.set_value_string);
+ fv->ftype->set_value.set_value_string(fv, value);
}
void
generated by cgit v1.2.1 (git 2.18.0) at 2024-05-19 00:43:40 +0000