ftypes: move set_value_sinteger into the union

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

Change-Id: I084c4c9beba9978b538d359ebaac21391ebd7f82
Reviewed-on: https://code.wireshark.org/review/20523
Petri-Dish: Michael Mann <mmann78@netscape.net>
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 05df2e9a427e854b112dfe350d6928ff1ed18da4..eacf3bacd07f1452710c8ed113ae7fdd70be4ad6 100644 (file)
--- 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_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -779,7 +778,6 @@ ftype_register_bytes(void)
 
                { .set_value_byte_array = bytes_fvalue_set },   /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -819,7 +817,6 @@ ftype_register_bytes(void)
 
                { .set_value_bytes = ax25_fvalue_set }, /* union set_value */
 
-               NULL,                           /* set_value_integer */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -859,7 +856,6 @@ ftype_register_bytes(void)
 
                { .set_value_bytes = vines_fvalue_set },        /* union set_value */
 
-               NULL,                           /* set_value_integer */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -899,7 +895,6 @@ ftype_register_bytes(void)
 
                { .set_value_bytes = ether_fvalue_set },        /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -939,7 +934,6 @@ ftype_register_bytes(void)
 
                { .set_value_byte_array = oid_fvalue_set },     /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -979,7 +973,6 @@ ftype_register_bytes(void)
 
                { .set_value_byte_array = oid_fvalue_set },     /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1019,7 +1012,6 @@ ftype_register_bytes(void)
 
                { .set_value_byte_array = system_id_fvalue_set }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1059,7 +1051,6 @@ ftype_register_bytes(void)
 
                { .set_value_bytes = fcwwn_fvalue_set },        /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-double.c b/epan/ftypes/ftype-double.c
index 7e5f01cfd86adee5790185350e260472a38a43b7..18a324f5ca6e2758b6937a70544a23665d3ad431 100644 (file)
--- 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_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                double_fvalue_set_floating,     /* set_value_floating */
@@ -212,7 +211,6 @@ ftype_register_double(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                double_fvalue_set_floating,     /* set_value_floating */

diff --git a/epan/ftypes/ftype-guid.c b/epan/ftypes/ftype-guid.c
index 7a1a007c48c3bded30137d07032fe06b55eac18a..a3c835b56f138a2cdb5a7f02d44a79d65c6827e8 100644 (file)
--- 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_sinteger */
         NULL,                /* set_value_uinteger64 */
         NULL,                /* set_value_sinteger64 */
         NULL,                /* set_value_floating */

diff --git a/epan/ftypes/ftype-ieee-11073-float.c b/epan/ftypes/ftype-ieee-11073-float.c
index 3949324f00a9e4f358475b922da2edc0ef660534..ee863c115d2815fbf518601a1d12a86b7cbef6b7 100644 (file)
--- a/epan/ftypes/ftype-ieee-11073-float.c
+++ b/epan/ftypes/ftype-ieee-11073-float.c
@@ -1440,7 +1440,6 @@ Example: 114 is 0x0072
 
         { .set_value_uinteger = sfloat_ieee_11073_value_set }, /* union set_value */
 
-        NULL,                                 /* set_value_sinteger */
         NULL,                                 /* set_value_uinteger64 */
         NULL,                                 /* set_value_sinteger64 */
         NULL,                                 /* set_value_floating */
@@ -1507,7 +1506,6 @@ Example: 36.4 is 0xFF00016C
 
         { .set_value_uinteger = float_ieee_11073_value_set }, /* union set_value */
 
-        NULL,                                /* set_value_sinteger */
         NULL,                                /* set_value_uinteger64 */
         NULL,                                /* set_value_sinteger64 */
         NULL,                                /* set_value_floating */

diff --git a/epan/ftypes/ftype-integer.c b/epan/ftypes/ftype-integer.c
index 844311ba229d0b765d3732402ae05cadc8a28fb3..f1d4015f94be64979e225497ee177d37b2dec6a5 100644 (file)
--- a/epan/ftypes/ftype-integer.c
+++ b/epan/ftypes/ftype-integer.c
@@ -1028,7 +1028,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1067,7 +1066,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1106,7 +1104,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1145,7 +1142,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1184,7 +1180,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1223,7 +1218,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1262,7 +1256,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1301,7 +1294,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1340,7 +1332,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1377,9 +1368,8 @@ ftype_register_integers(void)
                integer_to_repr,                /* val_to_string_repr */
                integer_repr_len,               /* len_string_repr */
 
-               { NULL },                       /* union set_value */
+               { .set_value_sinteger = set_sinteger }, /* union set_value */
 
-               set_sinteger,                   /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1416,9 +1406,8 @@ ftype_register_integers(void)
                integer_to_repr,                /* val_to_string_repr */
                integer_repr_len,               /* len_string_repr */
 
-               { NULL },                       /* union set_value */
+               { .set_value_sinteger = set_sinteger }, /* union set_value */
 
-               set_sinteger,                   /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1455,9 +1444,8 @@ ftype_register_integers(void)
                integer_to_repr,                /* val_to_string_repr */
                integer_repr_len,               /* len_string_repr */
 
-               { NULL },                       /* union set_value */
+               { .set_value_sinteger = set_sinteger }, /* union set_value */
 
-               set_sinteger,                   /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1494,9 +1482,8 @@ ftype_register_integers(void)
                integer_to_repr,                /* val_to_string_repr */
                integer_repr_len,               /* len_string_repr */
 
-               { NULL },                       /* union set_value */
+               { .set_value_sinteger = set_sinteger }, /* union set_value */
 
-               set_sinteger,                   /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1535,7 +1522,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                set_sinteger64,                 /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1574,7 +1560,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                set_sinteger64,                 /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1613,7 +1598,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                set_sinteger64,                 /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1652,7 +1636,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                set_sinteger64,                 /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1691,7 +1674,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1731,7 +1713,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* get_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1771,7 +1752,6 @@ ftype_register_integers(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -1811,7 +1791,6 @@ ftype_register_integers(void)
 
                { NULL },                       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                set_uinteger64,                 /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-ipv4.c b/epan/ftypes/ftype-ipv4.c
index 34091a81e028add74fcfbefe0bc9bd70e2c73864..596b3a58a4dd79aa56de435fafbef38f195e8580 100644 (file)
--- a/epan/ftypes/ftype-ipv4.c
+++ b/epan/ftypes/ftype-ipv4.c
@@ -187,7 +187,6 @@ ftype_register_ipv4(void)
 
                { .set_value_uinteger = set_uinteger }, /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-ipv6.c b/epan/ftypes/ftype-ipv6.c
index d38836923e690b5bfbc8039643ce7ac18317dcc0..2c0da9ee79f3a554f437595ee75ff9aa58fbce60 100644 (file)
--- 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_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-none.c b/epan/ftypes/ftype-none.c
index 436fc0b3edf3048b58040d987df58f55280b713f..007fe4eb1e40e4698c4982f64746cef60c4ad2eb 100644 (file)
--- 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_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-pcre.c b/epan/ftypes/ftype-pcre.c
index 308baa6fc035c30b703390c2e4f2a6980e0d82f0..d0a52406db016b8226bbc2b9d9a522984848c955 100644 (file)
--- a/epan/ftypes/ftype-pcre.c
+++ b/epan/ftypes/ftype-pcre.c
@@ -165,7 +165,6 @@ ftype_register_pcre(void)
 
         { .set_value_string = gregex_fvalue_set }, /* union set_value */
 
-        NULL,               /* set_value_sinteger */
         NULL,               /* set_value_uinteger64 */
         NULL,               /* set_value_sinteger64 */
         NULL,               /* set_value_floating */

diff --git a/epan/ftypes/ftype-protocol.c b/epan/ftypes/ftype-protocol.c
index d67ff3132e17ea5a85d1d49dab19701dbcb0c271..843bb7f6aa075eb71748bf440103188a3bdafbbd 100644 (file)
--- a/epan/ftypes/ftype-protocol.c
+++ b/epan/ftypes/ftype-protocol.c
@@ -465,7 +465,6 @@ ftype_register_tvbuff(void)
 
                { .set_value_protocol = value_set },    /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-string.c b/epan/ftypes/ftype-string.c
index a4454c63f310355b99c984a9efee59155eaa12b0..631275fd42978cccdb19787e3c7cd589bd660534 100644 (file)
--- a/epan/ftypes/ftype-string.c
+++ b/epan/ftypes/ftype-string.c
@@ -242,7 +242,6 @@ ftype_register_string(void)
 
                { .set_value_string = string_fvalue_set_string },       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -281,7 +280,6 @@ ftype_register_string(void)
 
                { .set_value_string = string_fvalue_set_string },       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -320,7 +318,6 @@ ftype_register_string(void)
 
                { .set_value_string = string_fvalue_set_string },       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -359,7 +356,6 @@ ftype_register_string(void)
 
                { .set_value_string = string_fvalue_set_string },       /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftype-time.c b/epan/ftypes/ftype-time.c
index 86f4001919a939262005eb8ac4d471610ca96305..9bb8ed1cd307c177bcbc3d79cfdd8803334b117b 100644 (file)
--- 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_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */
@@ -436,7 +435,6 @@ ftype_register_time(void)
 
                { .set_value_time = time_fvalue_set },  /* union set_value */
 
-               NULL,                           /* set_value_sinteger */
                NULL,                           /* set_value_uinteger64 */
                NULL,                           /* set_value_sinteger64 */
                NULL,                           /* set_value_floating */

diff --git a/epan/ftypes/ftypes-int.h b/epan/ftypes/ftypes-int.h
index 1846b9952e3fa124f064ca7f676bd2d37464239c..961926f6fa416c0ea239c23ec3c49f6a12622c53 100644 (file)
--- a/epan/ftypes/ftypes-int.h
+++ b/epan/ftypes/ftypes-int.h
@@ -98,10 +98,10 @@ struct _ftype_t {
                FvalueSetStringFunc     set_value_string;
                FvalueSetProtocolFunc   set_value_protocol;
                FvalueSetUnsignedIntegerFunc    set_value_uinteger;
+               FvalueSetSignedIntegerFunc      set_value_sinteger;
        } set_value;
 
        /* could be union */
-       FvalueSetSignedIntegerFunc              set_value_sinteger;
        FvalueSetUnsignedInteger64Func  set_value_uinteger64;
        FvalueSetSignedInteger64Func            set_value_sinteger64;
        FvalueSetFloatingFunc   set_value_floating;

diff --git a/epan/ftypes/ftypes.c b/epan/ftypes/ftypes.c
index 9a7b872b42dfddbd04210e15600764d0ccdad6b9..eff5ff7e22db7172aae9d057ab2489e58010906a 100644 (file)
--- a/epan/ftypes/ftypes.c
+++ b/epan/ftypes/ftypes.c
@@ -574,8 +574,12 @@ fvalue_set_uinteger(fvalue_t *fv, guint32 value)
 void
 fvalue_set_sinteger(fvalue_t *fv, gint32 value)
 {
-       g_assert(fv->ftype->set_value_sinteger);
-       fv->ftype->set_value_sinteger(fv, value);
+       g_assert(fv->ftype->ftype == FT_INT8 ||
+                       fv->ftype->ftype == FT_INT16 ||
+                       fv->ftype->ftype == FT_INT24 ||
+                       fv->ftype->ftype == FT_INT32);
+       g_assert(fv->ftype->set_value.set_value_sinteger);
+       fv->ftype->set_value.set_value_sinteger(fv, value);
 }
 
 void