[Libreoffice-commits] core.git: include/unotest unotest/source

[Mike Kaganski (via logerrit)]

 include/unotest/bootstrapfixturebase.hxx    |    2 +-
 unotest/source/cpp/bootstrapfixturebase.cxx |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit a182e9ce5e0da3cc98603ed27819405812d7fab1
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Wed Mar 31 09:52:24 2021 +0300
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Wed Mar 31 09:50:24 2021 +0200

    This should be a static method
    
    Change-Id: I70b725d42a91151e0c1f2f1fedc4be18433a7cf4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113397
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/include/unotest/bootstrapfixturebase.hxx b/include/unotest/bootstrapfixturebase.hxx
index ac0e1166f8a6..11650df8c4d5 100644
--- a/include/unotest/bootstrapfixturebase.hxx
+++ b/include/unotest/bootstrapfixturebase.hxx
@@ -66,7 +66,7 @@ public:
   virtual void setUp() override;
   virtual void tearDown() override;
 
-  bool isWindowsRDP() const;
+  static bool isWindowsRDP();
 };
 
 }
diff --git a/unotest/source/cpp/bootstrapfixturebase.cxx b/unotest/source/cpp/bootstrapfixturebase.cxx
index 5eb5b91dcaf0..a8affd272674 100644
--- a/unotest/source/cpp/bootstrapfixturebase.cxx
+++ b/unotest/source/cpp/bootstrapfixturebase.cxx
@@ -36,10 +36,10 @@ void test::BootstrapFixtureBase::setUp()
 
 void test::BootstrapFixtureBase::tearDown() { StarBASIC::DetachAllDocBasicItems(); }
 
-bool test::BootstrapFixtureBase::isWindowsRDP() const
+bool test::BootstrapFixtureBase::isWindowsRDP()
 {
 #if defined _WIN32
-    return GetSystemMetrics(SM_REMOTESESSION);
+    return GetSystemMetrics(SM_REMOTESESSION) != 0;
 #else
     return false;
 #endif