Joe Buck <[EMAIL PROTECTED]> writes: > On Sat, Jul 29, 2006 at 07:33:03PM -0400, Simon Boulet wrote: >> After a couple hours debugging code, I figured our an if() somewhere >> had a trailing ; like this: >> >> if (memcmp(p, COMMUNITY, strlen(COMMUNITY)) != 0); >> continue; /* failed */ >> >> The code above will always reach "continue" even when memcmp() == 0. >> >> I was surprised to see gcc doesn't report anything, I would expect a >> "statement has no effect" warning. > > But it is common to have an empty action on a condition. You'll often > see code like > > if (condition) > /* nothing */;
Having an empty action on a condition without an else clause does not make any sense. Andreas. -- Andreas Schwab, SuSE Labs, [EMAIL PROTECTED] SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany PGP key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."
