[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c73ead608b7d2b9b3dedb8d7786950f761f344f2
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c73ead6 by Guy Harris (guy@xxxxxxxxxxxx):

    Clean up a warning.
    
    offset has to be volatile, as it's used in a loop that involves the
    setjmp/longjmp-based TRY mechanism.
    
    Instead of passing pointers to the offset to routines that dissect
    headers, have the routines take the offset as an argument and return the
    updated offset, to avoid having to mark said pointers as pointing to a
    volatile variable.
    
    Update comments while we're at it.
    
    Change-Id: I3058a4e6a736c234ad7508521c9fe9da358b6096
    Reviewed-on: https://code.wireshark.org/review/27109
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  993b47e   remove redundant break statement
    adds  c73ead6   Clean up a warning.


Summary of changes:
 epan/dissectors/packet-bluecom.c | 96 ++++++++++++++++++++--------------------
 1 file changed, 49 insertions(+), 47 deletions(-)