diff options
| author | Justin Luth <justin.luth@collabora.com> | 2023-12-19 10:14:29 -0500 |
|---|---|---|
| committer | Justin Luth <jluth@mail.com> | 2024-03-13 22:38:36 +0100 |
| commit | 80219819e068dcf53059e2026be2a6f4151d6fc6 (patch) | |
| tree | b99bca43a4d79ea3e882c4875211015925226c91 /sw/qa/extras/rtfimport | |
| parent | aba13d6b4b2d586b356e7b98a23a7796e30c37ec (diff) | |
address nit: use xmlDocUniquePtr, not const auto&
Miklos "nitted" me that I should NOT use const auto&
in this context. Obviously anything I do
is simply copy/paste, so lets make sure that
this "bad example" is removed from all unit tests.
Change-Id: I9b76aeafc5848637e8ebd523a7a550789bd991c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160993
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
Diffstat (limited to 'sw/qa/extras/rtfimport')
| -rw-r--r-- | sw/qa/extras/rtfimport/rtfimport.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sw/qa/extras/rtfimport/rtfimport.cxx b/sw/qa/extras/rtfimport/rtfimport.cxx index aa6365a40f0e..f1f1085e80b7 100644 --- a/sw/qa/extras/rtfimport/rtfimport.cxx +++ b/sw/qa/extras/rtfimport/rtfimport.cxx @@ -741,7 +741,7 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf116265) CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(0), getProperty<sal_Int32>(getParagraph(2), "ParaFirstLineIndent")); - const auto& pLayout = parseLayoutDump(); + xmlDocUniquePtr pLayout = parseLayoutDump(); // Ensure that there is a tabstop in the pseudo-numbering (numbering::NONE followed by tabstop) assertXPath(pLayout, "//SwFixPortion"_ostr, 1); } @@ -1481,7 +1481,7 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf78506) CPPUNIT_ASSERT(rProp.Value.get<OUString>().isEmpty()); } - const auto& pLayout = parseLayoutDump(); + xmlDocUniquePtr pLayout = parseLayoutDump(); // Ensure that there is a tabstop in the pseudo-numbering (numbering::NONE followed by tabstop) assertXPath(pLayout, "//SwFixPortion"_ostr, 1); } @@ -1608,7 +1608,7 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf153196) { createSwDoc("tdf153196.rtf"); - const auto& pLayout = parseLayoutDump(); + xmlDocUniquePtr pLayout = parseLayoutDump(); CPPUNIT_ASSERT_EQUAL(4, getPages()); |