Re: [PATCHES] BUG #2221: Bad delimiters allowed in COPY ...

On Sun, Jan 29, 2006 at 10:20:47PM -0500, Neil Conway wrote:
> On Sun, 2006-01-29 at 17:03 -0800, David Fetter wrote:
> > Another followup, this time with the comment done right.
>
> +       /* Disallow the forbidden_delimiter strings */
> +       if (strcspn(cstate->delim, BADCHARS) != 1)
> +               elog(ERROR, "COPY delimiter cannot be %#02x",
> +                        *cstate->delim);
> +
>
> The comment is still wrong: referencing "forbidden_delimiter" makes
> it sound like there is something named forbidden_delimiter, but
> there is not (at least in the patch as submitted).
>
> The patch should also use ereport rather than elog, because this
> error message might reasonably be encountered by the user.

Patch with BADCHARS attached :)

Cheers,
D
--
David Fetter david@fetter.org http://fetter.org/
phone: +1 415 235 3778

Remember to vote!