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>

diff --git a/epan/ftypes/ftype-bytes.c b/epan/ftypes/ftype-bytes.c
index aad3cf3685046f873bc41ddb2e9628d5f3f7ca3d..f99993183e8402d039a9d9f24db820330537d91d 100644 (file)
--- 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 5195a72e32ec087398134d9002ed973e6a640256..c1e658371277161f6643e527accb55a7e5444f55 100644 (file)
--- 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 1a6aa921d4732c4926fb4fdd356c2050a85313f0..294f0208213638d852b6504d3473a178752a97c3 100644 (file)
--- 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 dbee9982bc36994e4f2103d5576ae98f1ea0e0b3..e6498349073e74b7649869540e722f2407efdd45 100644 (file)
--- 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 c2a2576ad6b89abcd0061ef441ee2fdcfcbfca4d..b8a1313d99f8d8ea3f5fe69252227f86b816e0c9 100644 (file)
--- 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 fd80cbbd95e4865382f0fa100004f95c25a81f44..531ac9399cd2ed8667f647a9ef2cf98e04c0aa7f 100644 (file)
--- 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 4395ce765c3c437a32877a66315bdca8419909bc..e4ebabc5f77bf570d64ec34be0d56f5adca3341b 100644 (file)
--- 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 5b10d9bc6fdf6ab0f0872a623d8c18812aadac49..810adc14bb62904a6b49ce5b7fb5fc5db132c1f4 100644 (file)
--- 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 97c1122aafa2990a8a3b28a58a87756028db90f6..d269e865b11996b0db96564c06d90ba4f715dd15 100644 (file)
--- 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 1be42345c5f5f4c8d609836a6603c06f41791e93..7ee01c474b47f31436f1b98716d5ffb8a49c1e9a 100644 (file)
--- 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 d533a553690a59de42d92f9dddd70d31fcf38cbb..cbdc372b6d3bd00691e2b7aaadd5ce9661643233 100644 (file)
--- 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 84fd02f9cd9f5e360686764e309f6e387ec01c87..6d1eb7b0d270f1f7f29940992ac70027eea94ada 100644 (file)
--- 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 4c490dcc7e334ef35456a45a2100e8d7c2c17e90..756e59ff4c4e57debcc4cc09d39e3dc64b0f191a 100644 (file)
--- 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 cd948ac4677955234aaa37597bb4ea49295a9f10..2953e9dfbc8469d65dccc553da94d3268d0f2e2c 100644 (file)
--- 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