Wireshark-commits: [Wireshark-commits] master 885ac0c: More cppcheck fixes, using top-of-tree cppch

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 12 Jun 2020 19:01:47 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=885ac0c2790b366f848604b69770e0a1c339fd08
Submitter: "Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

885ac0c by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):

    More cppcheck fixes, using top-of-tree cppcheck.
    
    Mostly redundant tests, but for bthci dissectors,
    avoid using a key that points to  out-of-scope variables.
    
    Change-Id: I2785690b8487d45c4e2daff0b6e03cd8747ed410
    Reviewed-on: https://code.wireshark.org/review/37467
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    

Actions performed:

    from  231663a   802.11: Add BSS membership selector to supported(and extended) rates Tag
     add  885ac0c   More cppcheck fixes, using top-of-tree cppcheck.


Summary of changes:
 epan/dissectors/packet-bthci_acl.c | 4 ++--
 epan/dissectors/packet-bthci_sco.c | 4 ++--
 epan/dissectors/packet-isobus-vt.c | 4 +---
 epan/dissectors/packet-multipart.c | 5 +----
 epan/dissectors/packet-obex.c      | 6 ++++--
 epan/dissectors/packet-rsvp.c      | 2 +-
 6 files changed, 11 insertions(+), 14 deletions(-)