>>>>> "Johannes" == Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:
Dscho> I would prefer something like this:
My taste would favor:
static int has_special(const char *p)
{
for (; *p; p++)
if (isspecial(*p))
return 1;
return 0;
}
as it underlines the intent (loop over "p" characters and stop no
later than the end of the string) while avoiding using side effects in
the body to increment the pointer. This habit comes from Ada, where
loop indices are considered read-only in the loop body.
It also eases further extensions such as
static int has_special(const char *p)
{
for (; *p; p++)
if (isspecial(*p) || isveryspecial(*p))
return 1;
return 0;
}
without having to move the "++" somewhere else.
Dscho> but that is probably a matter of taste.
Agreed.
Sam
--
Samuel Tardieu -- sam@xxxxxxxxxxx -- http://www.rfc1149.net/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
