Remove the (long deprecated) proto_tree_add_*_hidden() functions
authormorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 24 Jul 2008 15:35:29 +0000 (15:35 +0000)
committermorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Thu, 24 Jul 2008 15:35:29 +0000 (15:35 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@25820 f5534014-38df-0310-8fa8-9805f1628bb7

doc/README.developer
epan/proto.c
epan/proto.h
tools/checkAPIs.pl

index 85aab0ef647f6f9373b4e2bf37fe7f9886980e95..2b00cfd3d0477f7172538ab30e071f1c2b286918 100644 (file)
@@ -1802,9 +1802,6 @@ protocol or field labels to the proto_tree:
        proto_item*
        proto_tree_add_item(tree, id, tvb, start, length, little_endian);
 
-       proto_item*
-       proto_tree_add_item_hidden(tree, id, tvb, start, length, little_endian);
-
        proto_item*
        proto_tree_add_none_format(tree, id, tvb, start, length, format, ...);
 
@@ -1815,9 +1812,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_bytes(tree, id, tvb, start, length, start_ptr);
 
-       proto_item *
-       proto_tree_add_bytes_hidden(tree, id, tvb, start, length, start_ptr);
-
        proto_item *
        proto_tree_add_bytes_format(tree, id, tvb, start, length, start_ptr,
            format, ...);
@@ -1829,9 +1823,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_time(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_time_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_time_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -1843,9 +1834,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_ipxnet(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_ipxnet_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_ipxnet_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1857,9 +1845,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_ipv4(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_ipv4_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_ipv4_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1871,9 +1856,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_ipv6(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_ipv6_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_ipv6_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -1885,9 +1867,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_ether(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_ether_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_ether_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -1899,9 +1878,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_string(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_string_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_string_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -1913,9 +1889,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_boolean(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_boolean_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_boolean_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1927,9 +1900,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_float(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_float_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_float_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1941,9 +1911,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_double(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_double_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_double_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1955,9 +1922,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_uint(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_uint_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_uint_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -1980,9 +1944,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_int(tree, id, tvb, start, length, value);
 
-       proto_item *
-       proto_tree_add_int_hidden(tree, id, tvb, start, length, value);
-
        proto_item *
        proto_tree_add_int_format(tree, id, tvb, start, length, value,
            format, ...);
@@ -2011,9 +1972,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_guid(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_guid_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_guid_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -2025,9 +1983,6 @@ protocol or field labels to the proto_tree:
        proto_item *
        proto_tree_add_oid(tree, id, tvb, start, length, value_ptr);
 
-       proto_item *
-       proto_tree_add_oid_hidden(tree, id, tvb, start, length, value_ptr);
-
        proto_item *
        proto_tree_add_oid_format(tree, id, tvb, start, length, value_ptr,
            format, ...);
@@ -2112,68 +2067,6 @@ Subarea Nodes. The user does not have to shift the value of the FID to
 the high nibble of the byte ("sna.th.fid == 0xf0") as was necessary
 in the past.
 
-proto_tree_add_item_hidden()
-----------------------------
-proto_tree_add_item_hidden is used to add fields and values to a tree,
-but not show them on a GUI tree.
-
-NOTE that creating hidden fields is actually quite a bad idea from a UI design
-perspective because the user (someone who did not write nor has ever seen the
-code) has no way of knowing that hidden fields are there to be filtered on
-thus defeating the whole purpose of putting them there.  A Better Way might
-be to add the fields (that might otherwise be hidden) to a subtree where they
-won't be seen unless the user opens the subtree--but they can be found if the
-user wants.
-
-NOTE, too, that all of the proto_tree_add_*_hidden() APIs are deprecated:
-instead of using them, add the item using proto_tree_add_item() and then
-make it hidden using PROTO_ITEM_SET_HIDDEN().
-
-One use for hidden fields (which would be better implemented using visible
-fields in a subtree) follows: The caller may want a value to be
-included in a tree so that the packet can be filtered on this field, but
-the representation of that field in the tree is not appropriate.  An
-example is the token-ring routing information field (RIF).  The best way
-to show the RIF in a GUI is by a sequence of ring and bridge numbers.
-Rings are 3-digit hex numbers, and bridges are single hex digits:
-
-       RIF: 001-A-013-9-C0F-B-555
-
-In the case of RIF, the programmer should use a field with no value and
-use proto_tree_add_none_format() to build the above representation. The
-programmer can then add the ring and bridge values, one-by-one, with
-proto_tree_add_item_hidden() so that the user can then filter on or
-search for a particular ring or bridge. Here's a skeleton of how the
-programmer might code this.
-
-       char *rif;
-       rif = create_rif_string(...);
-
-       proto_tree_add_none_format(tree, hf_tr_rif_label, ..., "RIF: %s", rif);
-
-       for(i = 0; i < num_rings; i++) {
-               proto_tree_add_item_hidden(tree, hf_tr_rif_ring, ..., FALSE);
-       }
-       for(i = 0; i < num_rings - 1; i++) {
-               proto_tree_add_item_hidden(tree, hf_tr_rif_bridge, ..., FALSE);
-       }
-
-The logical tree has these items:
-
-       hf_tr_rif_label, text="RIF: 001-A-013-9-C0F-B-555", value = NONE
-       hf_tr_rif_ring,  hidden, value=0x001
-       hf_tr_rif_bridge, hidden, value=0xA
-       hf_tr_rif_ring,  hidden, value=0x013
-       hf_tr_rif_bridge, hidden, value=0x9
-       hf_tr_rif_ring,  hidden, value=0xC0F
-       hf_tr_rif_bridge, hidden, value=0xB
-       hf_tr_rif_ring,  hidden, value=0x555
-
-GUI or print code will not display the hidden fields, but a display
-filter or "packet grep" routine will still see the values. The possible
-filter is then possible:
-
-       tr.rif_ring eq 0x013
 
 proto_tree_add_protocol_format()
 --------------------------------
@@ -2278,25 +2171,6 @@ e_guid_t structure.
 For proto_tree_add_oid(), the 'value_ptr' argument is a pointer to an
 ASN.1 Object Identifier.
 
-proto_tree_add_bytes_hidden()
-proto_tree_add_time_hidden()
-proto_tree_add_ipxnet_hidden()
-proto_tree_add_ipv4_hidden()
-proto_tree_add_ipv6_hidden()
-proto_tree_add_ether_hidden()
-proto_tree_add_string_hidden()
-proto_tree_add_boolean_hidden()
-proto_tree_add_float_hidden()
-proto_tree_add_double_hidden()
-proto_tree_add_uint_hidden()
-proto_tree_add_int_hidden()
-proto_tree_add_guid_hidden()
-proto_tree_add_oid_hidden()
-----------------------------
-These routines add fields and values to a tree, but don't show them in the GUI
-tree.  They are used for the same reason that proto_tree_add_item_hidden() is
-used (and they should not be used for the same reasons).
-
 proto_tree_add_bytes_format()
 proto_tree_add_time_format()
 proto_tree_add_ipxnet_format()
@@ -2456,6 +2330,71 @@ Example: (from the scsi dissector)
 
 Which provides very pretty dissection of this one byte bitmask.
 
+PROTO_ITEM_SET_HIDDEN()
+-----------------------
+PROTO_ITEM_SET_HIDDEN is used to hide fields, which have already been added
+to the tree, from being visible in the displayed tree.
+
+NOTE that creating hidden fields is actually quite a bad idea from a UI design
+perspective because the user (someone who did not write nor has ever seen the
+code) has no way of knowing that hidden fields are there to be filtered on
+thus defeating the whole purpose of putting them there.  A Better Way might
+be to add the fields (that might otherwise be hidden) to a subtree where they
+won't be seen unless the user opens the subtree--but they can be found if the
+user wants.
+
+One use for hidden fields (which would be better implemented using visible
+fields in a subtree) follows: The caller may want a value to be
+included in a tree so that the packet can be filtered on this field, but
+the representation of that field in the tree is not appropriate.  An
+example is the token-ring routing information field (RIF).  The best way
+to show the RIF in a GUI is by a sequence of ring and bridge numbers.
+Rings are 3-digit hex numbers, and bridges are single hex digits:
+
+       RIF: 001-A-013-9-C0F-B-555
+
+In the case of RIF, the programmer should use a field with no value and
+use proto_tree_add_none_format() to build the above representation. The
+programmer can then add the ring and bridge values, one-by-one, with
+proto_tree_add_item() and hide them with PROTO_ITEM_SET_HIDDEN() so that the
+user can then filter on or search for a particular ring or bridge. Here's a
+skeleton of how the programmer might code this.
+
+       char *rif;
+       rif = create_rif_string(...);
+
+       proto_tree_add_none_format(tree, hf_tr_rif_label, ..., "RIF: %s", rif);
+
+       for(i = 0; i < num_rings; i++) {
+               proto_item *pi;
+
+               pi = proto_tree_add_item(tree, hf_tr_rif_ring, ..., FALSE);
+               PROTO_ITEM_SET_HIDDEN(pi);
+       }
+       for(i = 0; i < num_rings - 1; i++) {
+               proto_item *pi;
+
+               pi = proto_tree_add_item(tree, hf_tr_rif_bridge, ..., FALSE);
+               PROTO_ITEM_SET_HIDDEN(pi);
+       }
+
+The logical tree has these items:
+
+       hf_tr_rif_label, text="RIF: 001-A-013-9-C0F-B-555", value = NONE
+       hf_tr_rif_ring,  hidden, value=0x001
+       hf_tr_rif_bridge, hidden, value=0xA
+       hf_tr_rif_ring,  hidden, value=0x013
+       hf_tr_rif_bridge, hidden, value=0x9
+       hf_tr_rif_ring,  hidden, value=0xC0F
+       hf_tr_rif_bridge, hidden, value=0xB
+       hf_tr_rif_ring,  hidden, value=0x555
+
+GUI or print code will not display the hidden fields, but a display
+filter or "packet grep" routine will still see the values. The possible
+filter is then possible:
+
+       tr.rif_ring eq 0x013
+
 
 1.7 Utility routines.
 
index 88bb3f34301c02926dede47aa420606043035646..bc78efac9f12fd596a113e589aa8b6b363298913 100644 (file)
@@ -1347,22 +1347,6 @@ proto_tree_add_item(proto_tree *tree, int hfindex, tvbuff_t *tvb,
            length, little_endian);
 }
 
-proto_item *
-proto_tree_add_item_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb,
-    gint start, gint length, gboolean little_endian)
-{
-       proto_item      *pi;
-
-       pi = proto_tree_add_item(tree, hfindex, tvb, start, length, little_endian);
-       if (pi == NULL)
-               return(NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
-
 /* Add a FT_NONE to a proto_tree */
 proto_item *
 proto_tree_add_none_format(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
@@ -1472,21 +1456,6 @@ proto_tree_add_bytes(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        return pi;
 }
 
-proto_item *
-proto_tree_add_bytes_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-               gint length, const guint8 *start_ptr)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_bytes(tree, hfindex, tvb, start, length, start_ptr);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_bytes_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const guint8 *start_ptr,
@@ -1569,21 +1538,6 @@ proto_tree_add_time(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gi
        return pi;
 }
 
-proto_item *
-proto_tree_add_time_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               nstime_t *value_ptr)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_time(tree, hfindex, tvb, start, length, value_ptr);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_time_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, nstime_t *value_ptr,
@@ -1661,21 +1615,6 @@ proto_tree_add_ipxnet(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        return pi;
 }
 
-proto_item *
-proto_tree_add_ipxnet_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               guint32 value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_ipxnet(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_ipxnet_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, guint32 value, const char *format, ...)
@@ -1742,21 +1681,6 @@ proto_tree_add_ipv4(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gi
        return pi;
 }
 
-proto_item *
-proto_tree_add_ipv4_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               guint32 value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_ipv4(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_ipv4_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, guint32 value, const char *format, ...)
@@ -1825,21 +1749,6 @@ proto_tree_add_ipv6(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gi
        return pi;
 }
 
-proto_item *
-proto_tree_add_ipv6_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               const guint8* value_ptr)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_ipv6(tree, hfindex, tvb, start, length, value_ptr);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_ipv6_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const guint8* value_ptr,
@@ -1914,21 +1823,6 @@ proto_tree_add_guid(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gi
        return pi;
 }
 
-proto_item *
-proto_tree_add_guid_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               const e_guid_t *value_ptr)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_guid(tree, hfindex, tvb, start, length, value_ptr);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_guid_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const e_guid_t *value_ptr,
@@ -2008,21 +1902,6 @@ proto_tree_add_oid(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gin
        return pi;
 }
 
-proto_item *
-proto_tree_add_oid_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               const guint8* value_ptr)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_oid(tree, hfindex, tvb, start, length, value_ptr);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_oid_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const guint8* value_ptr,
@@ -2154,21 +2033,6 @@ proto_tree_add_string(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        return pi;
 }
 
-proto_item *
-proto_tree_add_string_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-               gint length, const char* value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_string(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_string_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const char* value, const char *format,
@@ -2310,21 +2174,6 @@ proto_tree_add_ether(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, g
        return pi;
 }
 
-proto_item *
-proto_tree_add_ether_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               const guint8* value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_ether(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_ether_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, const guint8* value,
@@ -2399,21 +2248,6 @@ proto_tree_add_boolean(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        return pi;
 }
 
-proto_item *
-proto_tree_add_boolean_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               guint32 value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_boolean(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_boolean_format_value(proto_tree *tree, int hfindex,
                tvbuff_t *tvb, gint start, gint length, guint32 value,
@@ -2481,21 +2315,6 @@ proto_tree_add_float(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, g
        return pi;
 }
 
-proto_item *
-proto_tree_add_float_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               float value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_float(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_float_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, float value, const char *format, ...)
@@ -2564,21 +2383,6 @@ proto_tree_add_double(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        return pi;
 }
 
-proto_item *
-proto_tree_add_double_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               double value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_double(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_double_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, double value, const char *format, ...)
@@ -2657,21 +2461,6 @@ proto_tree_add_uint(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gi
        return pi;
 }
 
-proto_item *
-proto_tree_add_uint_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               guint32 value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_uint(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_uint_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, guint32 value, const char *format, ...)
@@ -2835,21 +2624,6 @@ proto_tree_add_int(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gin
        return pi;
 }
 
-proto_item *
-proto_tree_add_int_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start, gint length,
-               gint32 value)
-{
-       proto_item              *pi;
-
-       pi = proto_tree_add_int(tree, hfindex, tvb, start, length, value);
-       if (pi == NULL)
-               return (NULL);
-
-       PROTO_ITEM_SET_HIDDEN(pi);
-
-       return pi;
-}
-
 proto_item *
 proto_tree_add_int_format_value(proto_tree *tree, int hfindex, tvbuff_t *tvb,
                gint start, gint length, gint32 value, const char *format, ...)
index 44aff8e10459750a09469b4a2de7ec9aa43e16f6..9ec473df33e8f95551c6a76e5a85017bf3cb46dc 100644 (file)
@@ -501,12 +501,6 @@ extern proto_item *
 proto_tree_add_item(proto_tree *tree, int hfindex, tvbuff_t *tvb,
     gint start, gint length, gboolean little_endian);
 
-/** Add a hidden item to a proto_tree.
- @deprecated use proto_tree_add_item() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_item_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb,
-    gint start, gint length, gboolean little_endian);
-
 /** Add a text-only node to a proto_tree.
  @param tree the tree to append this item to
  @param tvb the tv buffer of the current data
@@ -573,12 +567,6 @@ extern proto_item *
 proto_tree_add_bytes(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const guint8* start_ptr);
 
-/** Add a hidden FT_BYTES to a proto_tree.
- @deprecated use proto_tree_add_bytes() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_bytes_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const guint8* start_ptr);
-
 /** Add a formatted FT_BYTES to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -623,12 +611,6 @@ extern proto_item *
 proto_tree_add_time(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, nstime_t* value_ptr);
 
-/** Add a hidden FT_ABSOLUTE_TIME or FT_RELATIVE_TIME to a proto_tree.
- @deprecated use proto_tree_add_time() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_time_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, nstime_t* value_ptr);
-
 /** Add a formatted FT_ABSOLUTE_TIME or FT_RELATIVE_TIME to a proto_tree, with
     the format generating the string for the value and with the field name
     being included automatically.
@@ -674,12 +656,6 @@ extern proto_item *
 proto_tree_add_ipxnet(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, guint32 value);
 
-/** Add a hidden FT_IPXNET to a proto_tree.
- @deprecated use proto_tree_add_ipxnet() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_ipxnet_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, guint32 value);
-
 /** Add a formatted FT_IPXNET to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -724,12 +700,6 @@ extern proto_item *
 proto_tree_add_ipv4(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, guint32 value);
 
-/** Add a hidden FT_IPv4 to a proto_tree.
- @deprecated use proto_tree_add_ipv4() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_ipv4_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, guint32 value);
-
 /** Add a formatted FT_IPv4 to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -774,12 +744,6 @@ extern proto_item *
 proto_tree_add_ipv6(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const guint8* value_ptr);
 
-/** Add a hidden FT_IPv6 to a proto_tree.
- @deprecated use proto_tree_add_ipv6() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_ipv6_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const guint8* value_ptr);
-
 /** Add a formatted FT_IPv6 to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -824,12 +788,6 @@ extern proto_item *
 proto_tree_add_ether(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const guint8* value);
 
-/** Add a hidden FT_ETHER to a proto_tree.
- @deprecated use proto_tree_add_ether() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_ether_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const guint8* value);
-
 /** Add a formatted FT_ETHER to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -874,12 +832,6 @@ extern proto_item *
 proto_tree_add_guid(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const e_guid_t *value_ptr);
 
-/** Add a hidden FT_GUID to a proto_tree.
- @deprecated use proto_tree_add_guid() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_guid_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const e_guid_t *value_ptr);
-
 /** Add a formatted FT_GUID to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -924,12 +876,6 @@ extern proto_item *
 proto_tree_add_oid(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const guint8* value_ptr);
 
-/** Add a hidden FT_OID to a proto_tree.
- @deprecated use proto_tree_add_oid() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_oid_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const guint8* value_ptr);
-
 /** Add a formatted FT_OID to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -974,12 +920,6 @@ extern proto_item *
 proto_tree_add_string(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, const char* value);
 
-/** Add a hidden FT_STRING to a proto_tree.
- @deprecated use proto_tree_add_string() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_string_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, const char* value);
-
 /** Add a formatted FT_STRING to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -1024,12 +964,6 @@ extern proto_item *
 proto_tree_add_boolean(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, guint32 value);
 
-/** Add a hidden FT_BOOLEAN to a proto_tree.
- @deprecated use proto_tree_add_boolean() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_boolean_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, guint32 value);
-
 /** Add a formatted FT_BOOLEAN to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -1074,12 +1008,6 @@ extern proto_item *
 proto_tree_add_float(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, float value);
 
-/** Add a hidden FT_FLOAT to a proto_tree.
- @deprecated use proto_tree_add_float() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_float_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, float value);
-
 /** Add a formatted FT_FLOAT to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -1124,12 +1052,6 @@ extern proto_item *
 proto_tree_add_double(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, double value);
 
-/** Add a hidden FT_DOUBLE to a proto_tree.
- @deprecated use proto_tree_add_double() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_double_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, double value);
-
 /** Add a formatted FT_DOUBLE to a proto_tree, with the format generating
     the string for the value and with the field name being included
     automatically.
@@ -1174,12 +1096,6 @@ extern proto_item *
 proto_tree_add_uint(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, guint32 value);
 
-/** Add a hidden FT_UINT8, FT_UINT16, FT_UINT24 or FT_UINT32 to a proto_tree.
- @deprecated use proto_tree_add_uint() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_uint_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, guint32 value);
-
 /** Add a formatted FT_UINT8, FT_UINT16, FT_UINT24 or FT_UINT32 to a proto_tree,
     with the format generating the string for the value and with the field
     name being included automatically.
@@ -1269,12 +1185,6 @@ extern proto_item *
 proto_tree_add_int(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
        gint length, gint32 value);
 
-/** Add a hidden FT_INT8, FT_INT16, FT_INT24 or FT_INT32 to a proto_tree.
- @deprecated use proto_tree_add_int() and a subsequent call to PROTO_ITEM_SET_HIDDEN() instead */
-extern proto_item *
-proto_tree_add_int_hidden(proto_tree *tree, int hfindex, tvbuff_t *tvb, gint start,
-       gint length, gint32 value);
-
 /** Add a formatted FT_INT8, FT_INT16, FT_INT24 or FT_INT32 to a proto_tree,
     with the format generating the string for the value and with the field
     name being included automatically.
index 65a15402bfe4ac4279650d0a1f600d28a1272de8..bc668490d613946a35199ca78391526ef804e38a 100755 (executable)
@@ -134,22 +134,6 @@ my %APIs = (
                # Use strerror() and report messages in whatever
                # fashion is appropriate for the code in question.
                'perror',
-               # Use PROTO_ITEM_SET_HIDDEN instead of these:
-               'proto_tree_add_item_hidden',
-               'proto_tree_add_bytes_hidden',
-               'proto_tree_add_time_hidden',
-               'proto_tree_add_ipxnet_hidden',
-               'proto_tree_add_ipv4_hidden',
-               'proto_tree_add_ipv6_hidden',
-               'proto_tree_add_ether_hidden',
-               'proto_tree_add_guid_hidden',
-               'proto_tree_add_oid_hidden',
-               'proto_tree_add_string_hidden',
-               'proto_tree_add_boolean_hidden',
-               'proto_tree_add_float_hidden',
-               'proto_tree_add_double_hidden',
-               'proto_tree_add_uint_hidden',
-               'proto_tree_add_int_hidden',
                ]},
 
        # APIs that make the program exit. Dissectors shouldn't call these