diff options
author | Luboš Luňák <l.lunak@collabora.com> | 2022-03-03 10:57:21 +0100 |
---|---|---|
committer | Luboš Luňák <l.lunak@collabora.com> | 2022-03-03 22:37:35 +0100 |
commit | f92e15bc09def64a718b52812a9cb39e43fb8b5b (patch) | |
tree | 5448da27a291b2bd240f033facd0fb81322e5e15 /sc/qa/unit/mark_test.cxx | |
parent | dd6ff950cbc570a11100a7cce69d51577a26bbd7 (diff) |
improve ScMark* classes a bit
Bin pointless empty destructors, make trivial functions inline,
return value by simply returning it.
Change-Id: Ia71e73262802bbe6b022ca4bafb2b958ffdf39f5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130915
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'sc/qa/unit/mark_test.cxx')
-rw-r--r-- | sc/qa/unit/mark_test.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sc/qa/unit/mark_test.cxx b/sc/qa/unit/mark_test.cxx index 91b7af0e54b3..309a62143aff 100644 --- a/sc/qa/unit/mark_test.cxx +++ b/sc/qa/unit/mark_test.cxx @@ -155,9 +155,7 @@ void Test::testSimpleMark( const ScRange& rRange, const ScRange& rSelectionCover CPPUNIT_ASSERT( aMark.IsMarked() ); CPPUNIT_ASSERT( !aMark.IsMultiMarked() ); - ScRange aRangeResult; - aMark.GetMarkArea( aRangeResult ); - CPPUNIT_ASSERT_EQUAL( rRange, aRangeResult ); + CPPUNIT_ASSERT_EQUAL( rRange, aMark.GetMarkArea() ); SCROW nMidRow = ( rRange.aStart.Row() + rRange.aEnd.Row() ) / 2; SCCOL nMidCol = ( rRange.aStart.Col() + rRange.aEnd.Col() ) / 2; |