tdf#107792 vcl/win/gdi: reduce number of return paths

Change-Id: I3e7f00a10e76933424149a931042a0611ede5d47
Reviewed-on: https://gerrit.libreoffice.org/65036
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

vcl/win/gdi/gdiimpl.cxx

@@ -1361,17 +1361,15 @@ void WinSalGraphicsImpl::SetLineColor(Color nColor)
 HPEN WinSalGraphicsImpl::SearchStockPen(COLORREF nPenColor)
 {
     // Only screen, because printer has problems, when we use stock objects.
-    if (mrParent.isPrinter())
+    if (!mrParent.isPrinter())
     {
-        return nullptr;
-    }
-
-    const SalData* pSalData = GetSalData();
+        const SalData* pSalData = GetSalData();
 
-    for (sal_uInt16 i = 0; i < pSalData->mnStockPenCount; i++)
-    {
-        if (nPenColor == pSalData->maStockPenColorAry[i])
-            return pSalData->mhStockPenAry[i];
+        for (sal_uInt16 i = 0; i < pSalData->mnStockPenCount; i++)
+        {
+            if (nPenColor == pSalData->maStockPenColorAry[i])
+                return pSalData->mhStockPenAry[i];
+        }
     }
 
     return nullptr;