Remove rounding on printing time.
1.999999999 was rendering as 1.1000000 seconds instead of 2.000000 or
1.999999 seconds. Wrapping the big digit is not worth it at this
level, so let's drop that feature.
Change-Id: I46a9b75049332d530e7c3c5ccc4cca30430d9572
diff --git a/aos/common/logging/implementations_test.cc b/aos/common/logging/implementations_test.cc
index aade0d1..9663fd2 100644
--- a/aos/common/logging/implementations_test.cc
+++ b/aos/common/logging/implementations_test.cc
@@ -211,10 +211,10 @@
// Make sure rounding works correctly.
ASSERT_EQ(18, snprintf(buffer, sizeof(buffer), AOS_TIME_FORMAT,
AOS_TIME_ARGS(2, 999)));
- EXPECT_EQ("0000000002.000001s", ::std::string(buffer));
+ EXPECT_EQ("0000000002.000000s", ::std::string(buffer));
ASSERT_EQ(18, snprintf(buffer, sizeof(buffer), AOS_TIME_FORMAT,
AOS_TIME_ARGS(2, 500)));
- EXPECT_EQ("0000000002.000001s", ::std::string(buffer));
+ EXPECT_EQ("0000000002.000000s", ::std::string(buffer));
ASSERT_EQ(18, snprintf(buffer, sizeof(buffer), AOS_TIME_FORMAT,
AOS_TIME_ARGS(2, 499)));
EXPECT_EQ("0000000002.000000s", ::std::string(buffer));
@@ -223,6 +223,11 @@
ASSERT_EQ(18, snprintf(buffer, sizeof(buffer), AOS_TIME_FORMAT,
AOS_TIME_ARGS(1, 995000000)));
EXPECT_EQ("0000000001.995000s", ::std::string(buffer));
+
+ // This used to result in "0000000001.099500s".
+ ASSERT_EQ(18, snprintf(buffer, sizeof(buffer), AOS_TIME_FORMAT,
+ AOS_TIME_ARGS(1, 999999999)));
+ EXPECT_EQ("0000000001.999999s", ::std::string(buffer));
}
TEST(LoggingPrintFormatTest, Base) {