From d39f1da6e384b9540d2d163d7a7a84371e231c11 Mon Sep 17 00:00:00 2001
From: Edward Welbourne <edward.welbourne@qt.io>
Date: Wed, 12 Oct 2022 19:55:03 +0200
Subject: Simplify a list-removal operation in QTestLog's
 handleIgnoredMessage()

When list->next is nullptr, setting ignoreResultList to list->next is
the same as setting it to nullptr.

Change-Id: If328ce31db4344cf0136b7a827be871eb89e35b5
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
---
 src/testlib/qtestlog.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'src/testlib/qtestlog.cpp')

diff --git a/src/testlib/qtestlog.cpp b/src/testlib/qtestlog.cpp
index 44480cbc04..3ec695679c 100644
--- a/src/testlib/qtestlog.cpp
+++ b/src/testlib/qtestlog.cpp
@@ -164,10 +164,8 @@ namespace QTest {
                 // remove the item from the list
                 if (last)
                     last->next = list->next;
-                else if (list->next)
-                    ignoreResultList = list->next;
                 else
-                    ignoreResultList = nullptr;
+                    ignoreResultList = list->next;
 
                 delete list;
                 return true;
-- 
cgit v1.2.3