Neil Conway <[EMAIL PROTECTED]> wrote:

> > --- 622,628 ----
> >     for (i = 0; i < (sizeof(DDLAFTERs) / sizeof(char *)); i++)
> >     {
> >             res = PQexec(con, DDLAFTERs[i]);
> > !           if (strncmp(DDLAFTERs[i], "drop", 4) && PQresultStatus(res) != 
> > PGRES_COMMAND_OK)
> >             {
> >                     fprintf(stderr, "%s", PQerrorMessage(con));
> >                     exit(1);
> 
> None of the DDL in DDLAFTERs begins with "drop", so ISTM the right fix 
> is to just remove the strncmp().

I certainly think so.

But this code seems to be an idiom,
"Check the result, but drop commands may be fail".
Drop commands might be added to DDLAFTERs some time, so we may leave it.

---
ITAGAKI Takahiro <[EMAIL PROTECTED]>
NTT Cyber Space Laboratories
Nippon Telegraph and Telephone Corporation.



---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to