From 9a3ebcb54dc266d8905d97003bef44d4d82d646b Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Fri, 12 Aug 2005 08:22:47 +0000 Subject: [PATCH] (long_time_expected_width): Don't test for failed localtime. That cannot happen when the result date's year is in range. Add an assertion instead. --- src/ls.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/ls.c b/src/ls.c index dc01eaeba7..561d7fe526 100644 --- a/src/ls.c +++ b/src/ls.c @@ -3068,13 +3068,15 @@ long_time_expected_width (void) struct tm const *tm = localtime (&epoch); char buf[TIME_STAMP_LEN_MAXIMUM + 1]; - if (tm) - { - size_t len = - nstrftime (buf, sizeof buf, long_time_format[0], tm, 0, 0); - if (len != 0) - width = mbsnwidth (buf, len, 0); - } + /* The above use of localtime cannot fail. */ + assert (tm != NULL); + + { + size_t len = + nstrftime (buf, sizeof buf, long_time_format[0], tm, 0, 0); + if (len != 0) + width = mbsnwidth (buf, len, 0); + } if (width < 0) width = 0; -- 2.47.3