Wireshark-bugs: [Wireshark-bugs] [Bug 5858] some 'set but not used' compilation fixes

Date: Tue, 26 Apr 2011 13:49:32 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5858

--- Comment #31 from Yaniv Kaul <mykaul@xxxxxxxxx> 2011-04-26 13:49:32 PDT ---
(In reply to comment #28)
> (From update of attachment 6221 [details])
> The whole point of the exercise is to call
> seq->func(tree, tvb, hoffset, actx)-hoffset;
> As that will dissect the content of the "EQUENCE OF"

Since 'count' was not used, I removed it. That left me with:
seq->func(imp_tag, next_tvb, 0, actx, tree, *seq->p_id)-hoffset;

which when compiled, gave me:
packet-ber.c: In function 'dissect_ber_sq_of':
packet-ber.c:3843:58: error: value computed is not used [-Werror=unused-value]

So I changed it to:
seq->func(imp_tag, next_tvb, 0, actx, tree, *seq->p_id);

Hope it's better?


BTW: I also wonder what the next line's comment means:
        cnt++; /* rubbish*/

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.