[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - sc/qa
Tor Lillqvist
tml at collabora.com
Mon Oct 19 05:02:39 PDT 2015
sc/qa/unit/opencl-test.cxx | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
New commits:
commit 2438f18b4286f6444b03a69df5f11bea80308bd4
Author: Tor Lillqvist <tml at collabora.com>
Date: Thu Oct 15 15:40:00 2015 +0300
WaE: variable set but not used, in NDEBUG build
Seems that people prefer this to be CPPUNIT_ASSERT() instead of
assert().
My first thinking was that there is a slight difference in meaning;
that assert() checks that the code (or, in this case, data) in the
test *itself* is as expected while CPPUNIT_ASSERT() checks that the
functionality *somewhere else* works as expected. But whatever.
Change-Id: I93f7f8cf2b3aac9e8fc35baf2dcbad1a4003773f
diff --git a/sc/qa/unit/opencl-test.cxx b/sc/qa/unit/opencl-test.cxx
index 7848627..e53857c 100644
--- a/sc/qa/unit/opencl-test.cxx
+++ b/sc/qa/unit/opencl-test.cxx
@@ -768,14 +768,14 @@ void ScOpenCLTest::testSystematic()
}
}
- assert(nAVertBegin != 0);
- assert(nBVertBegin != 0);
- assert(nAVertEnd > nAVertBegin + 100);
- assert(nBVertEnd > nBVertBegin + 100);
- assert((nAVertEnd-nAVertBegin) == (nBVertEnd-nBVertBegin));
- assert(nAHorEnd > 10);
- assert(nBHorEnd > 10);
- assert(nAHorEnd == nBHorEnd);
+ CPPUNIT_ASSERT(nAVertBegin != 0);
+ CPPUNIT_ASSERT(nBVertBegin != 0);
+ CPPUNIT_ASSERT(nAVertEnd > nAVertBegin + 100);
+ CPPUNIT_ASSERT(nBVertEnd > nBVertBegin + 100);
+ CPPUNIT_ASSERT((nAVertEnd-nAVertBegin) == (nBVertEnd-nBVertBegin));
+ CPPUNIT_ASSERT(nAHorEnd > 10);
+ CPPUNIT_ASSERT(nBHorEnd > 10);
+ CPPUNIT_ASSERT(nAHorEnd == nBHorEnd);
for (SCROW i = nAVertBegin; i < nAVertEnd; ++i)
{
More information about the Libreoffice-commits
mailing list