Wireshark-commits: [Wireshark-commits] master 0af8d7c: Some cppcheck fixes.
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: Sat, 21 Mar 2020 21:28:23 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0af8d7c091d5eca352474a71fd0423b3cebbc5d3
Submitter: "Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

0af8d7c by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):

    Some cppcheck fixes.
    
    - UDPCP: fix test for Sync frame detection
    - Avoid a couple of redundant assignemnts
    
    Change-Id: Ib69a3ff4f0e802ddbbbd29eb2a803372c41a7ce4
    Reviewed-on: https://code.wireshark.org/review/36526
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    

Actions performed:

    from  1796abb   cirrus-ci: update freebsd 12.0 to 12.1.
     add  0af8d7c   Some cppcheck fixes.


Summary of changes:
 epan/dissectors/packet-catapult-dct2000.c | 2 +-
 epan/dissectors/packet-opensafety.c       | 2 +-
 epan/dissectors/packet-rlc-lte.c          | 2 +-
 epan/dissectors/packet-rlc-nr.c           | 2 +-
 epan/dissectors/packet-udpcp.c            | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)