ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-dev: Re: [Wireshark-dev] About Dead Store in clang Analysis

From: Chris Maynard <chris.maynard@xxxxxxxxx>
Date: Thu, 24 Mar 2011 20:57:42 +0000 (UTC)
Alexis La Goutte <alexis.lagoutte@...> writes:

> What it the official solution to fix this ?
> 1) Remove the code 
> 2) Ignore this warning 
> 3) Comment the code  add a 
>  /*     offset += 1;  Remove Clang Dead increment */

Or ... convert the dissector to a "new-style" dissector where the number of
bytes it consumes is returned, and in that case the offset should be incremented
but will need to be returned from any functions so proper offset (and thus the
proper number of bytes consumed) can be maintained and eventually returned from
the dissector.