diff options
author | Michael Mann <mmann78@netscape.net> | 2016-12-16 20:06:11 -0500 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-12-31 07:31:42 +0000 |
commit | f4b0abc7296bbb431e64e31f85b24c29196c2ae4 (patch) | |
tree | 68394f5fdfa1987900f0b079d0ecfd34003e8a5a /epan/dissectors/packet-dpnss-link.c | |
parent | 13964595ad09e5d1115f6c5cb604cded27f9f55d (diff) |
Dissectors don't need a journey of self discovery.
They already know who they are when they register themselves. Saving the
handle then to avoid finding it later.
Not sure if this will increase unnecessary register_dissector functions
(instead of using create_dissector_handle in proto_reg_handoff function)
when other dissectors copy/paste, but it should make startup time
a few microseconds better.
Change-Id: I3839be791b32b84887ac51a6a65fb5733e9f1f43
Reviewed-on: https://code.wireshark.org/review/19481
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-dpnss-link.c')
-rw-r--r-- | epan/dissectors/packet-dpnss-link.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/epan/dissectors/packet-dpnss-link.c b/epan/dissectors/packet-dpnss-link.c index 3d811bedc5..31bb20e356 100644 --- a/epan/dissectors/packet-dpnss-link.c +++ b/epan/dissectors/packet-dpnss-link.c @@ -92,6 +92,8 @@ static const value_string dpnss_link_frameType_vals[] = { static int ett_dpnss_link = -1; +static dissector_handle_t dpnss_link_handle; + /* Code to actually dissect the packets */ static int dissect_dpnss_link(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_) @@ -206,7 +208,7 @@ proto_register_dpnss_link(void) /* Register the protocol name and description */ proto_dpnss_link = proto_register_protocol("Digital Private Signalling System No 1 Link Layer", "DPNSS Link", "dpnss_link"); - register_dissector("dpnss_link", dissect_dpnss_link, proto_dpnss_link); + dpnss_link_handle = register_dissector("dpnss_link", dissect_dpnss_link, proto_dpnss_link); /* Required function calls to register the header fields and subtrees used */ proto_register_field_array(proto_dpnss_link, hf, array_length(hf)); @@ -216,9 +218,6 @@ proto_register_dpnss_link(void) void proto_reg_handoff_dpnss_link(void) { - dissector_handle_t dpnss_link_handle; - - dpnss_link_handle = find_dissector("dpnss_link"); dissector_add_uint("wtap_encap", WTAP_ENCAP_DPNSS, dpnss_link_handle); dpnss_handle = find_dissector_add_dependency("dpnss", proto_dpnss_link); |