Wireshark-commits: [Wireshark-commits] master 4dad151: sip: remove some unnecessary if (tree) check
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 13 Jun 2016 21:13:25 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4dad15150490c651f1c741f0caed442df46d543c
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4dad151 by Martin Kaiser (wireshark@xxxxxxxxx):

    sip: remove some unnecessary if (tree) checks
    
    which shouldn't have any impact on performance
    
    we should not remove the chcks around blocks where
    vb_format_text() is called
    
    Change-Id: Idc191cc70e7458225bbc5dfececb3c88c83ab5f1
    Reviewed-on: https://code.wireshark.org/review/15879
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  bcbcad1   Move inclusion of CmakeListsCustom to the end of the file in order to have all defines in place.
    adds  4dad151   sip: remove some unnecessary if (tree) checks


Summary of changes:
 epan/dissectors/packet-sip.c |   24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)