Wireshark-commits: [Wireshark-commits] master d60ce0b: no need for if(tree) display the opcode firs

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 5 Aug 2014 19:51:56 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d60ce0b991a16b972493a3d97e3c06a542e35397
Submitter: Martin Kaiser (wireshark@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

d60ce0b by Martin Kaiser (wireshark@xxxxxxxxx):

    no need for if(tree)
    display the opcode first, followed by the file name
    
    Change-Id: Ic5dd7409414d2f748c326c59a4b850dc2244da1f
    Reviewed-on: https://code.wireshark.org/review/3446
    Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
    Tested-by: Martin Kaiser <wireshark@xxxxxxxxx>
    

Actions performed:

    from  c5ca59f   delete unnecessary return;
    adds  d60ce0b   no need for if(tree) display the opcode first, followed by the file name


Summary of changes:
 epan/dissectors/packet-tftp.c |   36 ++++++++++++++++--------------------
 1 file changed, 16 insertions(+), 20 deletions(-)