Mention that resetting the timeout may be wrong on select retry.

diff --git a/src/interfaces/libpq/fe-misc.c b/src/interfaces/libpq/fe-misc.c
index 32b197bf6b6527e797b0a697e09e3cc887587747..7b5a2eeb15208c04c118f52743f8067d584b8e95 100644 (file)
--- a/src/interfaces/libpq/fe-misc.c
+++ b/src/interfaces/libpq/fe-misc.c
@@ -25,7 +25,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-misc.c,v 1.82 2002/10/14 17:33:08 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-misc.c,v 1.83 2002/10/14 18:11:17 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -824,7 +824,12 @@ retry5:
                {
                        /*
                         *      select() may modify timeout argument on some platforms so
-                        *      use copy
+                        *      use copy.
+                        *      XXX Do we really want to do that?  If select() returns
+                        *      the number of seconds remaining, we are resetting
+                        *      the timeout to its original value.  This will yeild
+                        *      incorrect timings when select() is interrupted.
+                        *      bjm 2002-10-14
                         */
                        tmp_timeout = *timeout;
                        ptmp_timeout = &tmp_timeout;