Wireshark-commits: [Wireshark-commits] master f5340b2: Split init routine into init/cleanup for mor
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 3 Jul 2015 23:36:17 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f5340b2a07a0a5f6830a091283d3170195203b82
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f5340b2 by Peter Wu (peter@xxxxxxxxxxxxx):

    Split init routine into init/cleanup for more dissectors
    
    This patch converts some dissectors using g_hash_table_foreach_remove.
    
     - 9p: drop no-op free func.
     - nfs: use g_hash_table_new_full such that a destructor function can
       be used. Drop NULL check since g_free can handle these just fine.
     - nlm: use g_hash_table_new_full such that a destructor function can
       be used. Simplify "matched" destruction by replacing the wrapper by
       a direct g_free call.
    
    Change-Id: I455e7f0ad4e47e70dae05af7233fdcdebf583f9f
    Reviewed-on: https://code.wireshark.org/review/9224
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  126e02c   Call reassembly_table_destroy and move g_hash_table_destroy
    adds  f5340b2   Split init routine into init/cleanup for more dissectors


Summary of changes:
 epan/dissectors/packet-9p.c  |   17 +++++-------
 epan/dissectors/packet-nfs.c |   61 +++++++++++++++---------------------------
 epan/dissectors/packet-nlm.c |   42 +++++++++++------------------
 3 files changed, 44 insertions(+), 76 deletions(-)