sc/qa/extras/macros-test.cxx | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
New commits: commit a7d057acec0e871408d28c782dfed934df40f368 Author: Mike Kaganski <mike.kagan...@collabora.com> AuthorDate: Mon Dec 30 11:49:35 2019 +0100 Commit: Mike Kaganski <mike.kagan...@collabora.com> CommitDate: Mon Dec 30 13:01:32 2019 +0100 Use range-based for loop Change-Id: Ia7572b2f8a5bc901434a9564b69408643a3e9fd9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/85990 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com> diff --git a/sc/qa/extras/macros-test.cxx b/sc/qa/extras/macros-test.cxx index 328cbb150baf..046a0d131b0c 100644 --- a/sc/qa/extras/macros-test.cxx +++ b/sc/qa/extras/macros-test.cxx @@ -281,10 +281,10 @@ void ScMacrosTest::testVba() sTempDir += OUStringChar(SAL_PATHDELIMITER); OUString sTestFileName("My Test WorkBook.xls"); Sequence< uno::Any > aParams; - for ( size_t i=0; i<SAL_N_ELEMENTS( testInfo ); ++i ) + for (const auto& rTestInfo : testInfo) { OUString aFileName; - createFileURL(testInfo[i].sFileBaseName + "xls", aFileName); + createFileURL(rTestInfo.sFileBaseName + "xls", aFileName); uno::Reference< css::lang::XComponent > xComponent = loadFromDesktop(aFileName, "com.sun.star.sheet.SpreadsheetDocument"); OUString sMsg( "Failed to load " + aFileName ); CPPUNIT_ASSERT_MESSAGE( OUStringToOString( sMsg, RTL_TEXTENCODING_UTF8 ).getStr(), xComponent.is() ); @@ -297,7 +297,7 @@ void ScMacrosTest::testVba() Any aRet; Sequence< sal_Int16 > aOutParamIndex; Sequence< Any > aOutParam; - bool bWorkbooksHandling = testInfo[i].sFileBaseName == "Workbooks." && !sTempDir.isEmpty() ; + bool bWorkbooksHandling = rTestInfo.sFileBaseName == "Workbooks." && !sTempDir.isEmpty() ; if ( bWorkbooksHandling ) { @@ -309,15 +309,15 @@ void ScMacrosTest::testVba() SfxObjectShell* pFoundShell = SfxObjectShell::GetShellFromComponent(xComponent); CPPUNIT_ASSERT_MESSAGE("Failed to access document shell", pFoundShell); - SAL_INFO("sc.qa", "about to invoke vba test in " << aFileName << " with url " << testInfo[i].sMacroUrl); + SAL_INFO("sc.qa", "about to invoke vba test in " << aFileName << " with url " << rTestInfo.sMacroUrl); SfxObjectShell::CallXScript( - xComponent, testInfo[i].sMacroUrl, aParams, aRet, aOutParamIndex, + xComponent, rTestInfo.sMacroUrl, aParams, aRet, aOutParamIndex, aOutParam); OUString aStringRes; aRet >>= aStringRes; CPPUNIT_ASSERT_EQUAL_MESSAGE( - OUString("script reported failure in file " + testInfo[i].sFileBaseName + "xls") + OUString("script reported failure in file " + rTestInfo.sFileBaseName + "xls") .toUtf8() .getStr(), OUString("OK"), aStringRes); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits