[sane-devel] Fixing sequencing issues (was Re: Proposed Upgrade (pu) branches on GitLab)
paddy-hack at member.fsf.org
Mon Sep 28 09:59:55 UTC 2015
Johannes Meixner writes:
> On Sep 25 21:02 Olaf Meeuwissen wrote (excerpt):
>> In the
>> mean time, I think we should any code that is flagged as -Wunsequenced
>> (by clang) or -Wsequence-point (by gcc).
> I assume the meaning is clear but there is an omission.
> What exactly should be done with such code?
Oops, I forgot a verb there. I meant to say that "we should *fix* any
code [...]", where fixing is different from just making the warning go
With the sequencing issues, we can only speculate at what the intended
behaviour is, so we can't really fix things. That notwithstanding, I
think we can make a decent, educated guess at a fix for both cases and
make the warning go away :-|
I'll have a stab at preparing patches for both.
Hope this helps,
Olaf Meeuwissen, LPIC-2 FSF Associate Member since 2004-01-27
Support Free Software Support the Free Software Foundation
More information about the sane-devel