Joe Conway <[EMAIL PROTECTED]> writes:
> Here is my counter-proposal to Bruce's dblink patch. Any comments?

Minor coding suggestion: to me it seems messy to do

> +     int        *openCursorCount = NULL;
> +     bool       *newXactForCursor = NULL;

> !             openCursorCount = &pconn->openCursorCount;
> !             newXactForCursor = &pconn->newXactForCursor;

> !             *newXactForCursor = TRUE;

This looks a bit cluttered already, and would get more so if you need to
add more fields to a remoteConn.  Plus it confuses the reader (at least
this reader) who is left wondering if you intend that those variables
might sometimes point to something other than two fields of the same
remoteConn.  I think it would be shorter and clearer to write

        remoteConn  *remconn = NULL;
        remconn = rconn;
        remconn->newXactForCursor = TRUE;

Also, you might be able to combine this variable with the existing
rconn local variable and thus simplify the code even more.

> Is it too late to apply this for 8.1? I tend to agree with calling this 
> a bugfix.

I think it's reasonable to fix now, yes.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
       subscribe-nomail command to [EMAIL PROTECTED] so that your
       message can get through to the mailing list cleanly

Reply via email to