[Libreoffice-commits] core.git: tools/CppunitTest_tools_test.mk tools/qa tools/source
Caolán McNamara
caolanm at redhat.com
Wed May 9 11:16:03 UTC 2018
tools/CppunitTest_tools_test.mk | 1
tools/qa/cppunit/test_time.cxx | 41 ++++++++++++++++++++++++++++++++++++++++
tools/source/datetime/ttime.cxx | 2 -
3 files changed, 43 insertions(+), 1 deletion(-)
New commits:
commit d8e90fdd90f299445509d93c905446f879bf8c73
Author: Caolán McNamara <caolanm at redhat.com>
Date: Tue May 8 21:36:18 2018 +0100
there are 3600000 seconds in an hour
regression from...
commit 9830fd36dbdb72c79703b0c61efc027fba793c5a
Date: Sun Mar 17 08:36:26 2013 +0100
date/time IDL datatypes incompatible change
Change-Id: I2f4b64a73b5529ba190acc678d907761bb568bbf
Reviewed-on: https://gerrit.libreoffice.org/54009
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/tools/CppunitTest_tools_test.mk b/tools/CppunitTest_tools_test.mk
index 7c7b9eab6184..d3e2de881053 100644
--- a/tools/CppunitTest_tools_test.mk
+++ b/tools/CppunitTest_tools_test.mk
@@ -16,6 +16,7 @@ $(eval $(call gb_CppunitTest_use_external,tools_test,boost_headers))
$(eval $(call gb_CppunitTest_add_exception_objects,tools_test, \
tools/qa/cppunit/test_bigint \
tools/qa/cppunit/test_date \
+ tools/qa/cppunit/test_time \
tools/qa/cppunit/test_fract \
tools/qa/cppunit/test_inetmime \
tools/qa/cppunit/test_pathutils \
diff --git a/tools/qa/cppunit/test_time.cxx b/tools/qa/cppunit/test_time.cxx
new file mode 100644
index 000000000000..f1915b71baa1
--- /dev/null
+++ b/tools/qa/cppunit/test_time.cxx
@@ -0,0 +1,41 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include <cppunit/TestFixture.h>
+#include <cppunit/extensions/HelperMacros.h>
+
+#include <tools/time.hxx>
+
+namespace tools
+{
+class TimeTest : public CppUnit::TestFixture
+{
+public:
+ void testTime();
+
+ CPPUNIT_TEST_SUITE(TimeTest);
+ CPPUNIT_TEST(testTime);
+ CPPUNIT_TEST_SUITE_END();
+};
+
+void TimeTest::testTime()
+{
+ Time aOrigTime(1, 56, 10);
+ auto nMS = aOrigTime.GetMSFromTime();
+
+ Time aNewTime(0);
+ aNewTime.MakeTimeFromMS(nMS);
+
+ CPPUNIT_ASSERT(bool(aOrigTime == aNewTime));
+}
+
+CPPUNIT_TEST_SUITE_REGISTRATION(TimeTest);
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */
diff --git a/tools/source/datetime/ttime.cxx b/tools/source/datetime/ttime.cxx
index 231a096e09dc..a8b3a5d52b58 100644
--- a/tools/source/datetime/ttime.cxx
+++ b/tools/source/datetime/ttime.cxx
@@ -239,7 +239,7 @@ sal_Int32 tools::Time::GetMSFromTime() const
( nNanoSec/1000000 +
nSec * 1000 +
nMin * 60000 +
- nHour * 360000 );
+ nHour * 3600000 );
}
void tools::Time::MakeTimeFromMS( sal_Int32 nMS )
More information about the Libreoffice-commits
mailing list