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(-)
