Suppress compile warning, avoid possible problems with signed vs. unsigned

comparisons in recently-added CheckPointWarning code.

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index 53aa731e7196f74299de279643e5448c87b8237a..187f0191c94c104b4a2e6a53fd77617f998854f5 100644 (file)
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.297 2002/11/15 02:44:55 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.298 2002/11/18 00:40:46 tgl Exp $
  *
  * NOTES
  *
@@ -2338,12 +2338,17 @@ sigusr1_handler(SIGNAL_ARGS)
             *  segment files.  Other checkpoints could reduce
             *  the frequency of forced checkpoints.
             */
-           time_t now = time(NULL);
+           time_t  now = time(NULL);
 
-           if (now - LastSignalledCheckpoint < CheckPointWarning)
-               elog(LOG, "Checkpoint segments are being created too frequently (%d secs)\n
-               Consider increasing CHECKPOINT_SEGMENTS",
-               now - LastSignalledCheckpoint);
+           if (LastSignalledCheckpoint != 0)
+           {
+               int     elapsed_secs = now - LastSignalledCheckpoint;
+
+               if (elapsed_secs < CheckPointWarning)
+                   elog(LOG, "Checkpoint segments are being created too frequently (%d secs)"
+                        "\n\tConsider increasing CHECKPOINT_SEGMENTS",
+                        elapsed_secs);
+           }
            LastSignalledCheckpoint = now;
        }