summaryrefslogtreecommitdiff
path: root/epan/ftypes
diff options
context:
space:
mode:
authorMartin Kaiser <wireshark@kaiser.cx>2017-03-19 13:08:06 +0100
committerMichael Mann <mmann78@netscape.net>2017-03-19 13:47:57 +0000
commit5bb8f1a2418863debe2b935c38a769c8e5adbf53 (patch)
tree9623d9c53f06dbb2a4a6c58afa962a2fe2f02752 /epan/ftypes
parente66b05fa9da469bc2d6e7127977bf93d2b477a0a (diff)
downloadwireshark-5bb8f1a2418863debe2b935c38a769c8e5adbf53.tar.gz
ftypes: move get_value_sinteger64 into the union
Delete get_value_sinteger64 from struct _ftype_t, make it part of the get_value union. Change-Id: I0113f70ab0aadd1aa655466e896e3acce6c8faeb Reviewed-on: https://code.wireshark.org/review/20617 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>
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.c29
-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.c1
-rw-r--r--epan/ftypes/ftype-protocol.c1
-rw-r--r--epan/ftypes/ftype-string.c4
-rw-r--r--epan/ftypes/ftype-time.c2
-rw-r--r--epan/ftypes/ftypes-int.h2
-rw-r--r--epan/ftypes/ftypes.c8
14 files changed, 11 insertions, 53 deletions
diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c
index aad3cf3685..f99993183e 100644
--- a/epan/ftypes/ftype-bytes.c
+++ b/epan/ftypes/ftype-bytes.c
@@ -744,7 +744,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -779,7 +778,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -814,7 +812,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_integer */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -849,7 +846,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_integer */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -884,7 +880,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -919,7 +914,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -954,7 +948,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -989,7 +982,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1024,7 +1016,6 @@ ftype_register_bytes(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c
index 5195a72e32..c1e6583712 100644
--- a/epan/ftypes/ftype-double.c
+++ b/epan/ftypes/ftype-double.c
@@ -177,7 +177,6 @@ ftype_register_double(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -212,7 +211,6 @@ ftype_register_double(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c
index 1a6aa921d4..294f020821 100644
--- a/epan/ftypes/ftype-guid.c
+++ b/epan/ftypes/ftype-guid.c
@@ -142,7 +142,6 @@ ftype_register_guid(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c
index dbee9982bc..e649834907 100644
--- a/epan/ftypes/ftype-ieee-11073-float.c
+++ b/epan/ftypes/ftype-ieee-11073-float.c
@@ -1445,7 +1445,6 @@ Example: 114 is 0x0072
sfloat_ieee_11073_value_get, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
sfloat_ieee_11073_cmp_eq,
sfloat_ieee_11073_cmp_ne,
@@ -1507,7 +1506,6 @@ Example: 36.4 is 0xFF00016C
float_ieee_11073_value_get, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
float_ieee_11073_cmp_eq,
float_ieee_11073_cmp_ne,
diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c
index c2a2576ad6..b8a1313d99 100644
--- a/epan/ftypes/ftype-integer.c
+++ b/epan/ftypes/ftype-integer.c
@@ -1033,7 +1033,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1067,7 +1066,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1101,7 +1099,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1135,7 +1132,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1169,7 +1165,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_integer */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1203,7 +1198,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1237,7 +1231,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1271,7 +1264,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1305,7 +1297,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1339,7 +1330,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1373,7 +1363,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1407,7 +1396,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_integer */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1441,7 +1429,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
get_sinteger, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1469,13 +1456,12 @@ ftype_register_integers(void)
integer64_repr_len, /* len_string_repr */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
- { NULL }, /* union get_value */
+ { .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- get_sinteger64, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1503,13 +1489,12 @@ ftype_register_integers(void)
integer64_repr_len, /* len_string_repr */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
- { NULL }, /* union get_value */
+ { .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- get_sinteger64, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1537,13 +1522,12 @@ ftype_register_integers(void)
integer64_repr_len, /* len_string_repr */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
- { NULL }, /* union get_value */
+ { .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- get_sinteger64, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1571,13 +1555,12 @@ ftype_register_integers(void)
integer64_repr_len, /* len_string_repr */
{ .set_value_sinteger64 = set_sinteger64 }, /* union set_value */
- { NULL }, /* union get_value */
+ { .get_value_sinteger64 = get_sinteger64 }, /* union get_value */
NULL, /* get_value_ptr */
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- get_sinteger64, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
@@ -1611,7 +1594,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
bool_eq, /* cmp_eq */
bool_ne, /* cmp_ne */
@@ -1646,7 +1628,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1681,7 +1662,6 @@ ftype_register_integers(void)
get_uinteger, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -1716,7 +1696,6 @@ ftype_register_integers(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
get_uinteger64, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq64,
cmp_ne64,
diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c
index fd80cbbd95..531ac9399c 100644
--- a/epan/ftypes/ftype-ipv4.c
+++ b/epan/ftypes/ftype-ipv4.c
@@ -192,7 +192,6 @@ ftype_register_ipv4(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c
index 4395ce765c..e4ebabc5f7 100644
--- a/epan/ftypes/ftype-ipv6.c
+++ b/epan/ftypes/ftype-ipv6.c
@@ -242,7 +242,6 @@ ftype_register_ipv6(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c
index 5b10d9bc6f..810adc14bb 100644
--- a/epan/ftypes/ftype-none.c
+++ b/epan/ftypes/ftype-none.c
@@ -46,7 +46,6 @@ ftype_register_none(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
NULL, /* cmp_eq */
NULL, /* cmp_ne */
diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c
index 97c1122aaf..d269e865b1 100644
--- a/epan/ftypes/ftype-pcre.c
+++ b/epan/ftypes/ftype-pcre.c
@@ -170,7 +170,6 @@ ftype_register_pcre(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
NULL, /* cmp_eq */
NULL, /* cmp_ne */
diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c
index 1be42345c5..7ee01c474b 100644
--- a/epan/ftypes/ftype-protocol.c
+++ b/epan/ftypes/ftype-protocol.c
@@ -470,7 +470,6 @@ ftype_register_tvbuff(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c
index d533a55369..cbdc372b6d 100644
--- a/epan/ftypes/ftype-string.c
+++ b/epan/ftypes/ftype-string.c
@@ -247,7 +247,6 @@ ftype_register_string(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -281,7 +280,6 @@ ftype_register_string(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -315,7 +313,6 @@ ftype_register_string(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -349,7 +346,6 @@ ftype_register_string(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c
index 84fd02f9cd..6d1eb7b0d2 100644
--- a/epan/ftypes/ftype-time.c
+++ b/epan/ftypes/ftype-time.c
@@ -402,7 +402,6 @@ ftype_register_time(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
@@ -436,7 +435,6 @@ ftype_register_time(void)
NULL, /* get_value_uinteger */
NULL, /* get_value_sinteger */
NULL, /* get_value_uinteger64 */
- NULL, /* get_value_sinteger64 */
cmp_eq,
cmp_ne,
diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h
index 4c490dcc7e..756e59ff4c 100644
--- a/epan/ftypes/ftypes-int.h
+++ b/epan/ftypes/ftypes-int.h
@@ -105,6 +105,7 @@ struct _ftype_t {
} set_value;
union {
+ FvalueGetSignedInteger64Func get_value_sinteger64;
FvalueGetFloatingFunc get_value_floating;
} get_value;
@@ -113,7 +114,6 @@ struct _ftype_t {
FvalueGetUnsignedIntegerFunc get_value_uinteger;
FvalueGetSignedIntegerFunc get_value_sinteger;
FvalueGetUnsignedInteger64Func get_value_uinteger64;
- FvalueGetSignedInteger64Func get_value_sinteger64;
FvalueCmp cmp_eq;
FvalueCmp cmp_ne;
diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c
index cd948ac467..2953e9dfbc 100644
--- a/epan/ftypes/ftypes.c
+++ b/epan/ftypes/ftypes.c
@@ -647,8 +647,12 @@ fvalue_get_uinteger64(fvalue_t *fv)
gint64
fvalue_get_sinteger64(fvalue_t *fv)
{
- g_assert(fv->ftype->get_value_sinteger64);
- return fv->ftype->get_value_sinteger64(fv);
+ 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->get_value.get_value_sinteger64);
+ return fv->ftype->get_value.get_value_sinteger64(fv);
}
double
generated by cgit v1.2.1 (git 2.18.0) at 2024-05-19 02:00:34 +0000