From 8fc6280f5e907699928ce4dc80175dabdf8fb2fa Mon Sep 17 00:00:00 2001 From: stevenhorsman Date: Fri, 13 Feb 2026 17:08:08 +0000 Subject: [PATCH] log-parser: Use time.IsZero() to check Using time.IsZero() to check for uninitialised times is clearer Signed-off-by: stevenhorsman --- src/tools/log-parser/logentry.go | 2 +- src/tools/log-parser/parse_test.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/tools/log-parser/logentry.go b/src/tools/log-parser/logentry.go index 4c3f0e95d6..254e1d772c 100644 --- a/src/tools/log-parser/logentry.go +++ b/src/tools/log-parser/logentry.go @@ -129,7 +129,7 @@ func (le LogEntry) Check(ignoreMissingFields bool) error { return fmt.Errorf("missing line number: %+v", le) } - if le.Time == (time.Time{}) { + if le.Time.IsZero() { return fmt.Errorf("missing timestamp: %+v", le) } diff --git a/src/tools/log-parser/parse_test.go b/src/tools/log-parser/parse_test.go index ef82510e21..ab399b670e 100644 --- a/src/tools/log-parser/parse_test.go +++ b/src/tools/log-parser/parse_test.go @@ -294,7 +294,7 @@ func TestParseTime(t *testing.T) { } for i, d := range data { - if d.timeString != "" && d.t == (time.Time{}) { + if d.timeString != "" && d.t.IsZero() { t, err := time.Parse(time.RFC3339Nano, d.timeString) assert.NoError(err) d.t = t