Remove useless variable

Per Jeff Janes

diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
index 95cf5f0b7852aa700f4261000a486ddb4ef20a33..bdd67dd97f822e9f52d27bfb1f512a7c23ee99ee 100644 (file)
--- a/src/backend/postmaster/pgstat.c
+++ b/src/backend/postmaster/pgstat.c
@@ -4386,15 +4386,19 @@ static void
 pgstat_recv_inquiry(PgStat_MsgInquiry *msg, int len)
 {
    slist_iter  iter;
-   bool        found = false;
    DBWriteRequest *newreq;
    PgStat_StatDBEntry *dbentry;
 
    elog(DEBUG2, "received inquiry for %d", msg->databaseid);
 
    /*
-    * Find the last write request for this DB (found=true in that case).
-    * Plain linear search, not really worth doing any magic here (probably).
+    * Find the last write request for this DB.  If it's older than the
+    * request's cutoff time, update it; otherwise there's nothing to do.
+    *
+    * Note that if a request is found, we return early and skip the below
+    * check for clock skew.  This is okay, since the only way for a DB request
+    * to be present in the list is that we have been here since the last write
+    * round.
     */
    slist_foreach(iter, &last_statrequests)
    {
@@ -4405,7 +4409,6 @@ pgstat_recv_inquiry(PgStat_MsgInquiry *msg, int len)
 
        if (msg->cutoff_time > req->request_time)
            req->request_time = msg->cutoff_time;
-       found = true;
        return;
    }