Wireshark-commits: [Wireshark-commits] master cc77d6c: There's no need for all these variables to b

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 25 Oct 2015 11:21:19 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cc77d6c66f0c0c32bda005bacbac686a158b95d0
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cc77d6c by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    There's no need for all these variables to be volatile.
    
    I suppose there must have been TRYs and CATCHes in here years ago. And/or
    there was just too much cutting-and-pasting going on here.
    
    Also reindent some col_ calls: they weren't moved left when check_col() was
    taken out.
    
    Change-Id: I0b01e008395be5b638a0470645d247dc706ca93f
    Reviewed-on: https://code.wireshark.org/review/11254
    Petri-Dish: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  93c647e   [nfs] don't THROW() an exception from a dissector
    adds  cc77d6c   There's no need for all these variables to be volatile.


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