Hi Pádraig,
Pádraig Brady <[email protected]> writes:
> + /* Skip a single blank or NBSP between the number and suffix. */
> + mcel_t g = mcel_scanz (*endptr);
> + if (c32isblank (g.ch) || c32isnbspace (g.ch))
> + (*endptr) += g.len;
Looks good.
> if (**endptr == '\0')
> break; /* Treat as no suffix. */
>
> if (!valid_suffix (**endptr))
> - return SSE_INVALID_SUFFIX;
> + {
> + /* Trailing blanks are allowed. */
> + while (isblank (to_uchar (**endptr)))
> + (*endptr)++;
> + if (**endptr == '\0')
> + break;
> +
> + return SSE_INVALID_SUFFIX;
> + }
Any reason not to do the same here? This one only handles ' ' and '\t'.
Collin
