Return std::optional instead of using bool* argument

Change-Id: I47f03411a6677f22291bea4dc06920ab9052349a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138399
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

1 files changed, 6 insertions, 8 deletions

diff --git a/test/source/lokcallback.cxx b/test/source/lokcallback.cxx
index 323ba4af1f10..c938e1e699a8 100644
--- a/test/source/lokcallback.cxx
+++ b/test/source/lokcallback.cxx
@@ -151,10 +151,9 @@ void TestLokCallbackWrapper::flushLOKData()
 
     for (int type : updatedTypes)
     {
-        bool ignore = false;
-        OString payload = viewShell->getLOKPayload(type, m_viewId, &ignore);
-        if (!ignore)
-            libreOfficeKitViewCallback(type, payload.getStr());
+        std::optional<OString> payload = viewShell->getLOKPayload(type, m_viewId);
+        if (payload)
+            libreOfficeKitViewCallback(type, payload->getStr());
     }
     for (const PerViewIdData& data : updatedTypesPerViewId)
     {
@@ -162,10 +161,9 @@ void TestLokCallbackWrapper::flushLOKData()
             return shell->GetViewShellId().get() == data.sourceViewId;
         });
         assert(viewShell != nullptr);
-        bool ignore = false;
-        OString payload = viewShell->getLOKPayload(data.type, data.viewId, &ignore);
-        if (!ignore)
-            libreOfficeKitViewCallbackWithViewId(data.type, payload.getStr(), data.viewId);
+        std::optional<OString> payload = viewShell->getLOKPayload(data.type, data.viewId);
+        if (payload)
+            libreOfficeKitViewCallbackWithViewId(data.type, payload->getStr(), data.viewId);
     }
 }