squashme: address review comments

Signed-off-by: Asad Sajjad Ahmed <[email protected]>
varnishcache · Nov 8, 2024 · 7b34199 · 7b34199
1 parent 3e53d5b
commit 7b34199
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/bin/varnishd/cache/cache_conn_pool.c b/bin/varnishd/cache/cache_conn_pool.c
@@ -137,13 +137,13 @@ PFD_Fd(struct pfd *p)
 vtim_dur
 PFD_Age(const struct pfd *p)
 {
-	vtim_mono now;
+	vtim_mono t_mono;
 
 	CHECK_OBJ_NOTNULL(p, PFD_MAGIC);
-	now = VTIM_mono();
-	assert(now >= p->created);
+	t_mono = VTIM_mono();
+	assert(t_mono >= p->created);
 
-	return (now - p->created);
+	return (t_mono - p->created);
 }
 
 uint64_t

diff --git a/bin/varnishd/cache/cache_conn_pool.h b/bin/varnishd/cache/cache_conn_pool.h
@@ -45,7 +45,7 @@ struct pfd;
 
 unsigned PFD_State(const struct pfd *);
 int *PFD_Fd(struct pfd *);
-vtim_real PFD_Age(const struct pfd *);
+vtim_dur PFD_Age(const struct pfd *);
 uint64_t PFD_Reused(const struct pfd *);
 void PFD_LocalName(const struct pfd *, char *, unsigned, char *, unsigned);
 void PFD_RemoteName(const struct pfd *, char *, unsigned, char *, unsigned);

diff --git a/include/tbl/vsl_tags.h b/include/tbl/vsl_tags.h
@@ -138,9 +138,9 @@ SLTM(SessClose, 0, "Client connection closed",
 SLTM(BackendOpen, 0, "Backend connection opened",
 	"Logged when a new backend connection is opened.\n\n"
 	"The format is::\n\n"
-	"\t%d %s %s %s %s %s %s [ %.6f %ld ]\n"
-	"\t|  |  |  |  |  |  |    |    |\n"
-	"\t|  |  |  |  |  |  |    |    +- Connection reuses\n"
+	"\t%d %s %s %s %s %s %s [ %f %ld ]\n"
+	"\t|  |  |  |  |  |  |    |  |\n"
+	"\t|  |  |  |  |  |  |    |  +--- Connection reuses\n"
 	"\t|  |  |  |  |  |  |    +------ Connection age\n"
 	"\t|  |  |  |  |  |  +----------- \"connect\" or \"reuse\"\n"
 	"\t|  |  |  |  |  +-------------- Local port\n"