Don't guard col_set_str (COL_INFO/COL_PROTOCOL) with col_check

[obnox/wireshark/wip.git] / epan / dissectors / packet-quake3.c

diff --git a/epan/dissectors/packet-quake3.c b/epan/dissectors/packet-quake3.c
index b3f60d5b1017987947627cbf14b576fc3b7e2676..89c244196edbe6cea3b303d619775c20e2766bdf 100644 (file)
--- a/epan/dissectors/packet-quake3.c
+++ b/epan/dissectors/packet-quake3.c
@@ -485,8 +485,7 @@ dissect_quake3(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 
        direction = DIR_UNKNOWN;
 
-       if (check_col(pinfo->cinfo, COL_PROTOCOL))
-               col_set_str(pinfo->cinfo, COL_PROTOCOL, "QUAKE3");
+       col_set_str(pinfo->cinfo, COL_PROTOCOL, "QUAKE3");
 
        if (tree) {
                quake3_item = proto_tree_add_item(tree, proto_quake3,
@@ -506,9 +505,7 @@ dissect_quake3(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
        }
 
        if (tvb_get_ntohl(tvb, 0) == 0xffffffff) {
-               if (check_col(pinfo->cinfo, COL_INFO)) {
-                       col_set_str(pinfo->cinfo, COL_INFO, "Connectionless ");
-               }
+               col_set_str(pinfo->cinfo, COL_INFO, "Connectionless ");
                if (quake3_tree)
                        proto_tree_add_uint_format(quake3_tree,
                                hf_quake3_connectionless,
@@ -518,9 +515,7 @@ dissect_quake3(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
                        tvb, pinfo, quake3_tree, &direction);
        }
        else {
-               if (check_col(pinfo->cinfo, COL_INFO)) {
-                       col_set_str(pinfo->cinfo, COL_INFO, "Game ");
-               }
+               col_set_str(pinfo->cinfo, COL_INFO, "Game ");
                if (quake3_tree)
                        proto_tree_add_uint_format(quake3_tree,
                                hf_quake3_game,
@@ -582,31 +577,31 @@ proto_register_quake3(void)
        static hf_register_info hf[] = {
                { &hf_quake3_direction,
                        { "Direction", "quake3.direction",
-                       FT_NONE, BASE_DEC, NULL, 0x0,
+                       FT_NONE, BASE_NONE, NULL, 0x0,
                        "Packet Direction", HFILL }},
                { &hf_quake3_connectionless,
                        { "Connectionless", "quake3.connectionless",
                        FT_UINT32, BASE_DEC, NULL, 0x0,
-                       "Connectionless", HFILL }},
+                       NULL, HFILL }},
                { &hf_quake3_game,
                        { "Game", "quake3.game",
                        FT_UINT32, BASE_DEC, NULL, 0x0,
-                       "Game", HFILL }},
+                       NULL, HFILL }},
                { &hf_quake3_connectionless_marker,
                        { "Marker", "quake3.connectionless.marker",
                        FT_UINT32, BASE_HEX, NULL, 0x0,
-                       "Marker", HFILL }},
+                       NULL, HFILL }},
                { &hf_quake3_connectionless_text,
                        { "Text", "quake3.connectionless.text",
-                       FT_STRING, BASE_DEC, NULL, 0x0,
-                       "Text", HFILL }},
+                       FT_STRING, BASE_NONE, NULL, 0x0,
+                       NULL, HFILL }},
                { &hf_quake3_connectionless_command,
                        { "Command", "quake3.connectionless.command",
-                       FT_STRING, BASE_DEC, NULL, 0x0,
-                       "Command", HFILL }},
+                       FT_STRING, BASE_NONE, NULL, 0x0,
+                       NULL, HFILL }},
                { &hf_quake3_server_addr,
                        { "Server Address", "quake3.server.addr",
-                       FT_IPv4, BASE_DEC, NULL, 0x0,
+                       FT_IPv4, BASE_NONE, NULL, 0x0,
                        "Server IP Address", HFILL }},
                { &hf_quake3_server_port,
                        { "Server Port", "quake3.server.port",
@@ -618,7 +613,7 @@ proto_register_quake3(void)
                        "Sequence number of the current packet", HFILL }},
                { &hf_quake3_game_rel1,
                        { "Reliable", "quake3.game.rel1",
-                       FT_BOOLEAN, BASE_DEC, NULL, 0x0,
+                       FT_BOOLEAN, BASE_NONE, NULL, 0x0,
                        "Packet is reliable and may be retransmitted", HFILL }},
                { &hf_quake3_game_seq2,
                        { "Sequence Number", "quake3.game.seq2",
@@ -626,7 +621,7 @@ proto_register_quake3(void)
                        "Sequence number of the last received packet", HFILL }},
                { &hf_quake3_game_rel2,
                        { "Reliable", "quake3.game.rel2",
-                       FT_BOOLEAN, BASE_DEC, NULL, 0x0,
+                       FT_BOOLEAN, BASE_NONE, NULL, 0x0,
                        "Packet was reliable and may be retransmitted", HFILL }},
                { &hf_quake3_game_qport,
                        { "QPort", "quake3.game.qport",