git: 4a0b57b0b4cb - main - fifolog: Trim some dead code and unused variables.

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Tue, 20 Jun 2023 21:04:07 UTC
The branch main has been updated by jhb:

URL: https://cgit.FreeBSD.org/src/commit/?id=4a0b57b0b4cbf7eb848927cf3f493163c3e76142

commit 4a0b57b0b4cbf7eb848927cf3f493163c3e76142
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2023-06-20 21:03:35 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2023-06-20 21:03:35 +0000

    fifolog: Trim some dead code and unused variables.
    
    The gmt_ptr and gmt variables are not used, so the call to gmtime can
    be removed entirely.  In addition, there isn't a need to call
    localtime twice.
    
    Reported by:    GCC -Wunused-but-set-variable
    Reviewed by:    phk, emaste
    Differential Revision:  https://reviews.freebsd.org/D40656
---
 usr.sbin/fifolog/lib/getdate.y | 14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/usr.sbin/fifolog/lib/getdate.y b/usr.sbin/fifolog/lib/getdate.y
index 8e63e8112993..57f3cb1d46fb 100644
--- a/usr.sbin/fifolog/lib/getdate.y
+++ b/usr.sbin/fifolog/lib/getdate.y
@@ -818,30 +818,18 @@ yylex(void)
 time_t
 get_date(char *p)
 {
-    struct tm		*tm, gmt;
+    struct tm		*tm;
     time_t		Start;
     time_t		tod;
     time_t nowtime;
-    struct tm *gmt_ptr;
 
     yyInput = p;
 
     (void)time (&nowtime);
 
-    gmt_ptr = gmtime (&nowtime);
-    if (gmt_ptr != NULL)
-    {
-	/* Make a copy, in case localtime modifies *tm (I think
-	   that comment now applies to *gmt_ptr, but I am too
-	   lazy to dig into how gmtime and locatime allocate the
-	   structures they return pointers to).  */
-	gmt = *gmt_ptr;
-    }
-
     if (! (tm = localtime (&nowtime)))
 	return -1;
 
-    tm = localtime(&nowtime);
     yyYear = tm->tm_year + 1900;
     yyMonth = tm->tm_mon + 1;
     yyDay = tm->tm_mday;