diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-29 08:58:11 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-29 15:12:26 +0100 |
commit | 41eeaace84b45c803fff3ebd5ab981f0ad09393b (patch) | |
tree | 0a0290bcc852507efa8241bd0853ebdd84ec9cbf /tools/qa/cppunit/test_config.cxx | |
parent | a2f7678171618d958e3c387718cd389bea63eaeb (diff) |
loplugin:oncevar
Change-Id: Iba892694acb378887a1d15ab59104c55f591f0bd
Reviewed-on: https://gerrit.libreoffice.org/62498
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'tools/qa/cppunit/test_config.cxx')
-rw-r--r-- | tools/qa/cppunit/test_config.cxx | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/tools/qa/cppunit/test_config.cxx b/tools/qa/cppunit/test_config.cxx index aaf599b8fa5e..4f392a3c7b50 100644 --- a/tools/qa/cppunit/test_config.cxx +++ b/tools/qa/cppunit/test_config.cxx @@ -76,9 +76,8 @@ public: CPPUNIT_ASSERT(!aConfig.HasGroup("TestGroup")); CPPUNIT_ASSERT_EQUAL(OString("TestGroup2"), aConfig.GetGroupName(0)); - sal_uInt16 nExpected = 1; sal_uInt16 nActual = aConfig.GetGroupCount(); - CPPUNIT_ASSERT_EQUAL(nExpected, nActual); + CPPUNIT_ASSERT_EQUAL(sal_uInt16(1), nActual); } osl::File::copy(maOriginalConfigFile, maConfigFile); @@ -90,9 +89,8 @@ public: aConfig.DeleteGroup(OString("NonExistentTestGroup")); CPPUNIT_ASSERT_EQUAL(OString("TestGroup"), aConfig.GetGroupName(0)); - sal_uInt16 nExpected = 2; sal_uInt16 nActual = aConfig.GetGroupCount(); - CPPUNIT_ASSERT_EQUAL(nExpected, nActual); + CPPUNIT_ASSERT_EQUAL(sal_uInt16(2), nActual); } osl::File::copy(maOriginalConfigFile, maConfigFile); @@ -101,9 +99,8 @@ public: void testGetGroupCount() { Config aConfig(maConfigFile); - sal_uInt16 nExpected = 2; sal_uInt16 nActual = aConfig.GetGroupCount(); - CPPUNIT_ASSERT_EQUAL(nExpected, nActual); + CPPUNIT_ASSERT_EQUAL(sal_uInt16(2), nActual); } void testReadKey() |