Hi Collin,

> The attached patch feels obvious, but I will leave it for review
> anyways. Maybe someone more creative than I will prefer to name the
> function something else.

The function name is OK.

The patch looks good, except that here:

int st_isapipe (struct stat const *const st)

the second 'const' is redundant. Better omit it, to avoid confusion.

Bruno




Reply via email to