On Mon, Feb 2, 2009 at 05:42, Christopher Hegarty - Sun Microsystems
Ireland <christopher.hega...@sun.com> wrote:
> +static int isAsciiSpace(char c) {
> +  return (((c) == '\t') || ((c) == '\r')  || ((c) == '\b') ||
> +          ((c) == '\n') || ((c) == ' ') );
> +}
> +

A very minor comment -
the parens around (c) are a very good idea in macro definitions,
but they are overkill in a function definition.

Reply via email to