]> git.ipfire.org Git - thirdparty/ntp.git/commitdiff
add test cases to reveal bug#2100
authorJuergen Perlinger <perlinger@ntp.org>
Fri, 30 Dec 2011 11:12:49 +0000 (12:12 +0100)
committerJuergen Perlinger <perlinger@ntp.org>
Fri, 30 Dec 2011 11:12:49 +0000 (12:12 +0100)
cleanup of test code

bk: 4efd9cb1yFcrkr1WV4JPnWOvj9KYmA

tests/libntp/timespecops.cpp
tests/libntp/timestructs.cpp
tests/libntp/timestructs.h
tests/libntp/timevalops.cpp

index 008349298719fcd73e26d6c12dc9cb08c9488daf..5af8c934fbcaa17dc0dfd1e784ac3a4e1c51cb1d 100644 (file)
@@ -1,14 +1,21 @@
 #include "libntptest.h"
+#include "timestructs.h"
 
 extern "C" {
+#include <math.h>
 #include "timespecops.h"
 }
 
 #include <string>
 #include <sstream>
 
+using namespace timeStruct;
+
 class timespecTest : public libntptest {
 protected:
+       static u_int32 my_tick_to_tsf(u_int32 ticks);
+       static u_int32 my_tsf_to_tick(u_int32 tsf);
+
        static const long NANOSECONDS;
        // that's it...
        struct lfpfracdata {
@@ -17,7 +24,41 @@ protected:
        };
        static const lfpfracdata fdata[];
 };
+
+u_int32
+timespecTest::my_tick_to_tsf(
+       u_int32 ticks
+       )
+{
+       // convert nanoseconds to l_fp fractional units, using double
+       // precision float calculations or, if available, 64bit integer
+       // arithmetic. This should give the precise fraction, rounded to
+       // the nearest representation.
+#if SIZEOF_LONG >= 8
+       return u_int32(((u_long(ticks) << 32) + 500000000) / 1000000000);
+#else
+       return u_int32(double(ticks) * 4.294967296 + 0.5);
+#endif
+       // And before you ask: if ticks >= 1000000000, the result is
+       // truncated nonsense, so don't use it out-of-bounds.
+}
+
+u_int32
+timespecTest::my_tsf_to_tick(
+       u_int32 tsf
+       )
+{
+       // Inverse operation: converts fraction to microseconds.
+#if SIZEOF_LONG >= 8
+       return u_int32((u_long(tsf) * 1000000000 + 0x80000000) >> 32);
+#else
+       return u_int32(double(tsf) / 4.294967296 + 0.5);
+#endif
+       // Beware: The result might be 10^9 due to rounding!
+}
+
 const long timespecTest::NANOSECONDS = 1000000000;
+
 const timespecTest::lfpfracdata timespecTest::fdata [] = {
                {         0, 0x00000000 }, {   2218896, 0x00916ae6 },
                {  16408100, 0x0433523d }, { 125000000, 0x20000000 },
@@ -35,80 +76,47 @@ const timespecTest::lfpfracdata timespecTest::fdata [] = {
                { 956805507, 0xf4f134a9 }, { 982570733, 0xfb89c16c }
 };
 
-class TSPEC {
-public:
-       struct timespec V;
-
-       TSPEC()
-               { ZERO(V); }
-       TSPEC(time_t hi, long lo)
-               { V.tv_sec = hi; V.tv_nsec = lo; }
-       bool operator == (const TSPEC& rhs) const
-               { return timespec_cmp(&V, &rhs.V) == 0; }
-       bool valid() const
-               { return timespec_isnormal(&V); }
-       operator struct timespec* () 
-               { return &V; }
-       operator struct timespec& ()
-               { return V;     }
-       TSPEC& operator = (const TSPEC& rhs)
-               { V = rhs.V; return *this; }
-       TSPEC& operator = (const struct timespec& rhs)
-               { V = rhs; return *this; }
-};
-
-static std::ostream&
-operator << (std::ostream& os, const TSPEC& val)
-{
-       os << timespec_tostr(&val.V);
-       return os;
-}
 
+// ---------------------------------------------------------------------
+// test support stuff -- part 1
+// ---------------------------------------------------------------------
 
-class LFP {
-public:
-       l_fp V;
-
-       LFP()
-               { ZERO(V); }
-       LFP(u_int32 hi, u_int32 lo)
-               { V.l_ui = hi; V.l_uf = lo; }
-       bool operator == (const LFP& rhs) const
-               { return L_ISEQU(&V, &rhs.V); }
-       operator l_fp* () 
-               { return &V; }
-       operator l_fp& ()
-               { return V; }
-       LFP& operator = (const LFP& rhs)
-               { V = rhs.V; return *this; }
-       LFP& operator = (const l_fp& rhs)
-               { V = rhs; return *this; }
-};
+TEST_F(timespecTest, Helpers1) {
+       timespec_wrap x;
 
-std::ostream&
-operator << (std::ostream& os, const LFP &val)
-{
-       os << ulfptoa(&val.V, 10);
-       return os;
+       for (x.V.tv_sec = -2; x.V.tv_sec < 3; x.V.tv_sec++)
+       {
+               x.V.tv_nsec = -1;
+               ASSERT_FALSE(x.valid());
+               x.V.tv_nsec = 0;
+               ASSERT_TRUE(x.valid());
+               x.V.tv_nsec = 999999999;
+               ASSERT_TRUE(x.valid());
+               x.V.tv_nsec = 1000000000;
+               ASSERT_FALSE(x.valid());
+       }
 }
 
-
-// ---------------------------------------------------------------------
-// test support stuff
-// ---------------------------------------------------------------------
+//----------------------------------------------------------------------
+// test normalisation
+//----------------------------------------------------------------------
 
 TEST_F(timespecTest, Normalise) {
        for (long ns = -2000000000; ns <= 2000000000; ns += 10000000) {
-               TSPEC x(0, ns);
+               timespec_wrap x(0, ns);
                timespec_norm(x);
                ASSERT_TRUE(x.valid());
        }
 }
 
+//----------------------------------------------------------------------
+// test classification
+//----------------------------------------------------------------------
+
 TEST_F(timespecTest, SignNoFrac) {
        // sign test, no fraction
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i, 0);
+               timespec_wrap a(i, 0);
                int E = (i > 0) - (i < 0);
                int r = timespec_test(a);
                ASSERT_EQ(E, r);
@@ -118,24 +126,24 @@ TEST_F(timespecTest, SignNoFrac) {
 TEST_F(timespecTest, SignWithFrac) {
        // sign test, with fraction
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i, 10);
+               timespec_wrap a(i, 10);
                int E = (i >= 0) - (i < 0);
                int r = timespec_test(a);
-
                ASSERT_EQ(E, r);
        }
 }
 
+//----------------------------------------------------------------------
 // test compare
+//----------------------------------------------------------------------
 TEST_F(timespecTest, CmpFracEQ) {
        // fractions are equal
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a( i , 200);
-                       TSPEC b( j , 200);
+                       timespec_wrap a( i , 200);
+                       timespec_wrap b( j , 200);
                        int   E = (i > j) - (i < j);
                        int   r = timespec_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
@@ -144,11 +152,10 @@ TEST_F(timespecTest, CmpFracGT) {
        // fraction a bigger fraction b
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a(i, 999999800);
-                       TSPEC b(j, 200);
+                       timespec_wrap a(i, 999999800);
+                       timespec_wrap b(j, 200);
                        int   E = (i >= j) - (i < j);
                        int   r = timespec_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
@@ -157,24 +164,25 @@ TEST_F(timespecTest, CmpFracLT) {
        // fraction a less fraction b
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a(i, 200);
-                       TSPEC b(j, 999999800);
+                       timespec_wrap a(i, 200);
+                       timespec_wrap b(j, 999999800);
                        int   E = (i > j) - (i <= j);
                        int   r = timespec_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
 
+//----------------------------------------------------------------------
 // Test addition (sum)
+//----------------------------------------------------------------------
+
 TEST_F(timespecTest, AddFullNorm) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a(i, 200);
-                       TSPEC b(j, 400);
-                       TSPEC E(i + j, 200 + 400);
-                       TSPEC c;
-
+                       timespec_wrap a(i, 200);
+                       timespec_wrap b(j, 400);
+                       timespec_wrap E(i + j, 200 + 400);
+                       timespec_wrap c;
                        timespec_add(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -183,11 +191,10 @@ TEST_F(timespecTest, AddFullNorm) {
 TEST_F(timespecTest, AddFullOflow1) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a(i, 200);
-                       TSPEC b(j, 999999900);
-                       TSPEC E(i + j + 1, 100);
-                       TSPEC c;
-
+                       timespec_wrap a(i, 200);
+                       timespec_wrap b(j, 999999900);
+                       timespec_wrap E(i + j + 1, 100);
+                       timespec_wrap c;
                        timespec_add(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -195,10 +202,9 @@ TEST_F(timespecTest, AddFullOflow1) {
 
 TEST_F(timespecTest, AddNsecNorm) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i, 200);
-               TSPEC E(i, 600);
-               TSPEC c;
-
+               timespec_wrap a(i, 200);
+               timespec_wrap E(i, 600);
+               timespec_wrap c;
                timespec_addns(c, a, 400);
                ASSERT_EQ(E, c);
        }
@@ -206,24 +212,25 @@ TEST_F(timespecTest, AddNsecNorm) {
 
 TEST_F(timespecTest, AddNsecOflow1) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i, 200);
-               TSPEC E(i + 1, 100);
-               TSPEC c;
-
+               timespec_wrap a(i, 200);
+               timespec_wrap E(i + 1, 100);
+               timespec_wrap c;
                timespec_addns(c, a, NANOSECONDS - 100);
                ASSERT_EQ(E, c);
        }
 }
 
+//----------------------------------------------------------------------
 // test subtraction (difference)
+//----------------------------------------------------------------------
+
 TEST_F(timespecTest, SubFullNorm) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a( i , 600);
-                       TSPEC b( j , 400);
-                       TSPEC E(i-j, 200);
-                       TSPEC c;
-
+                       timespec_wrap a( i , 600);
+                       timespec_wrap b( j , 400);
+                       timespec_wrap E(i-j, 200);
+                       timespec_wrap c;
                        timespec_sub(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -232,11 +239,10 @@ TEST_F(timespecTest, SubFullNorm) {
 TEST_F(timespecTest, SubFullOflow) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
-                       TSPEC a(  i  , 100);
-                       TSPEC b(  j  , 999999900);
-                       TSPEC E(i-j-1, 200);
-                       TSPEC c;
-
+                       timespec_wrap a(  i  , 100);
+                       timespec_wrap b(  j  , 999999900);
+                       timespec_wrap E(i-j-1, 200);
+                       timespec_wrap c;
                        timespec_sub(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -244,10 +250,9 @@ TEST_F(timespecTest, SubFullOflow) {
 
 TEST_F(timespecTest, SubNsecNorm) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i, 600);
-               TSPEC E(i, 200);
-               TSPEC c;
-
+               timespec_wrap a(i, 600);
+               timespec_wrap E(i, 200);
+               timespec_wrap c;
                timespec_subns(c, a, 400);
                ASSERT_EQ(E, c);
        }
@@ -255,35 +260,38 @@ TEST_F(timespecTest, SubNsecNorm) {
 
 TEST_F(timespecTest, SubNsecOflow) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a( i , 100);
-               TSPEC E(i-1, 200);
-               TSPEC c;
-
+               timespec_wrap a( i , 100);
+               timespec_wrap E(i-1, 200);
+               timespec_wrap c;
                timespec_subns(c, a, NANOSECONDS - 100);
                ASSERT_EQ(E, c);
        }
 }
 
+//----------------------------------------------------------------------
 // test negation
+//----------------------------------------------------------------------
+
 TEST_F(timespecTest, Neg) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a( i , 100);
-               TSPEC b;
-               TSPEC c;
-
+               timespec_wrap a( i , 100);
+               timespec_wrap b;
+               timespec_wrap c;
                timespec_neg(b, a);
                timespec_add(c, a, b);
                ASSERT_EQ(0, timespec_test(c));
        }
 }
 
+//----------------------------------------------------------------------
 // test abs value
+//----------------------------------------------------------------------
+
 TEST_F(timespecTest, AbsNoFrac) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i , 0);
-               TSPEC b;
+               timespec_wrap a(i , 0);
+               timespec_wrap b;
                int   c;
-
                c = timespec_abs(b, a);
                ASSERT_EQ((i < 0), c);
                ASSERT_EQ((i != 0), timespec_test(b));
@@ -292,23 +300,62 @@ TEST_F(timespecTest, AbsNoFrac) {
 
 TEST_F(timespecTest, AbsWithFrac) {
        for (int i = -4; i <= 4; ++i) {
-               TSPEC a(i , 100);
-               TSPEC b;
-               int   c;
-
+               timespec_wrap a(i , 100);
+               timespec_wrap b;
+               int           c;
                c = timespec_abs(b, a);
                ASSERT_EQ((i < 0), c);
                ASSERT_EQ(1, timespec_test(b));
        }
 }
 
+// ---------------------------------------------------------------------
+// test support stuff -- part 2
+// ---------------------------------------------------------------------
+
+TEST_F(timespecTest, Helpers2) {
+       AssertTimespecClose isClose(0, 2);
+       timespec_wrap x, y;
+
+       for (x.V.tv_sec = -2; x.V.tv_sec < 3; x.V.tv_sec++)
+               for (x.V.tv_nsec = 1;
+                    x.V.tv_nsec < 1000000000;
+                    x.V.tv_nsec += 499999999)
+               {
+                       for (long i = -4; i < 5; i++) {
+                               y = x;
+                               y.V.tv_nsec += i;
+                               if (i >= -2 && i <= 2)
+                                       ASSERT_PRED_FORMAT2(isClose, x,y);
+                               else
+                                       ASSERT_PRED_FORMAT2(!isClose, x,y);
+                       }
+               }
+}
+
+// global predicate instances we're using here
+static AssertFpClose FpClose(0, 1);
+static AssertTimespecClose TimespecClose(0, 2);
+
+//----------------------------------------------------------------------
 // conversion to l_fp
+//----------------------------------------------------------------------
+
+TEST_F(timespecTest, ToLFPbittest) {
+       for (u_int32 i = 0; i < 1000000000; i+=1000) {
+               timespec_wrap a(1, i);
+               l_fp_wrap     E(1, my_tick_to_tsf(i));
+               l_fp_wrap     r;
+               timespec_reltolfp(r, a);
+               ASSERT_PRED_FORMAT2(FpClose, E, r);
+       }
+}
+
 TEST_F(timespecTest, ToLFPrelPos) {
        for (int i = 0; i < COUNTOF(fdata); i++) {
-               TSPEC a(1, fdata[i].nsec);
-               LFP   E(1, fdata[i].frac);
-               LFP   r;
-
+               timespec_wrap a(1, fdata[i].nsec);
+               l_fp_wrap     E(1, fdata[i].frac);
+               l_fp_wrap     r;
                timespec_reltolfp(r, a);
                ASSERT_EQ(E, r);
        }
@@ -316,10 +363,9 @@ TEST_F(timespecTest, ToLFPrelPos) {
 
 TEST_F(timespecTest, ToLFPrelNeg) {
        for (int i = 0; i < COUNTOF(fdata); i++) {
-               TSPEC a(-1, fdata[i].nsec);
-               LFP   E(~0, fdata[i].frac);
-               LFP   r;
-
+               timespec_wrap a(-1, fdata[i].nsec);
+               l_fp_wrap     E(~0, fdata[i].frac);
+               l_fp_wrap     r;
                timespec_reltolfp(r, a);
                ASSERT_EQ(E, r);
        }
@@ -327,15 +373,69 @@ TEST_F(timespecTest, ToLFPrelNeg) {
 
 TEST_F(timespecTest, ToLFPabs) {
        for (int i = 0; i < COUNTOF(fdata); i++) {
-               TSPEC a(1, fdata[i].nsec);
-               LFP   E(1 + JAN_1970, fdata[i].frac);
-               LFP   r;
-
+               timespec_wrap a(1, fdata[i].nsec);
+               l_fp_wrap     E(1 + JAN_1970, fdata[i].frac);
+               l_fp_wrap     r;
                timespec_abstolfp(r, a);
                ASSERT_EQ(E, r);
        }
 }
 
+//----------------------------------------------------------------------
+// conversion from l_fp
+//----------------------------------------------------------------------
+TEST_F(timespecTest, FromLFPbittest) {
+       // Not *exactly* a bittest, because 2**32 tests would take a
+       // really long time even on very fast machines! So we do test
+       // every 1000 fractional units.
+       for (u_int32 tsf = 0; tsf < ~u_int32(1000); tsf += 1000) {
+               timespec_wrap E(1, my_tsf_to_tick(tsf));
+               l_fp_wrap     a(1, tsf);
+               timespec_wrap r;
+               timespec_relfromlfp(r, a);
+               // The conversion might be off by one nanosecond when
+               // comparing to calculated value.
+               ASSERT_PRED_FORMAT2(TimespecClose, E, r);
+       }
+}
+
+TEST_F(timespecTest, FromLFPrelPos) {
+       for (int i = 0; i < COUNTOF(fdata); i++) {
+               l_fp_wrap     a(1, fdata[i].frac);
+               timespec_wrap E(1, fdata[i].nsec);
+               timespec_wrap r;
+               timespec_relfromlfp(r, a);
+               ASSERT_PRED_FORMAT2(TimespecClose, E, r);
+       }
+}
+
+TEST_F(timespecTest, FromLFPrelNeg) {
+       for (int i = 0; i < COUNTOF(fdata); i++) {
+               l_fp_wrap     a(~0, fdata[i].frac);
+               timespec_wrap E(-1, fdata[i].nsec);
+               timespec_wrap r;
+               timespec_relfromlfp(r, a);
+               ASSERT_PRED_FORMAT2(TimespecClose, E, r);
+       }
+}
+
+
+// nsec -> frac -> nsec roundtrip, using a prime start and increment
+TEST_F(timespecTest, LFProundtrip) {
+       for (int32_t t = -1; t < 2; ++t)
+               for (u_int32 i = 4999; i < 1000000000; i+=10007) {
+                       timespec_wrap E(t, i);
+                       l_fp_wrap     a;
+                       timespec_wrap r;
+                       timespec_reltolfp(a, E);
+                       timespec_relfromlfp(r, a);
+                       ASSERT_EQ(E, r);
+               }
+}
+
+//----------------------------------------------------------------------
+// string formatting
+//----------------------------------------------------------------------
 
 TEST_F(timespecTest, ToString) {
        static const struct {
@@ -353,10 +453,9 @@ TEST_F(timespecTest, ToString) {
                {-1,-1, "-1.000000001" },
        };
        for (int i = 0; i < COUNTOF(data); i++) {
-               TSPEC a(data[i].sec, data[i].nsec);
+               timespec_wrap a(data[i].sec, data[i].nsec);
                std::string E(data[i].repr);
                std::string r(timespec_tostr(a));
-
                ASSERT_EQ(E, r);
        }
 }
index 28f0668dbbce71501f8d531e216e3ef12bc229f6..69fc874075e154577cc462429a2fdba4007d534e 100644 (file)
@@ -23,7 +23,7 @@ operator << (std::ostream& os, const timeStruct::l_fp_wrap& val)
           << std::setfill('0') << std::setw(8) << val.V.l_uf
           << std::dec;
        // human-readable format
-       os << '[' << dolfptoa(val.V.l_ui, val.V.l_uf, 0, 10, 0) << ']';
+       os << '[' << lfptoa(&val.V, 10) << ']';
        return os;
 }
 
@@ -124,4 +124,37 @@ AssertTimevalClose::operator()(
            << "\nare not close; diff=" << timeval_wrap(diff);
 }
 
+// Implementation of the timespec closeness predicate
+
+AssertTimespecClose::AssertTimespecClose(
+       time_t hi,
+       int32  lo
+       )
+{
+       limit.tv_sec = hi;
+       limit.tv_nsec = lo;
+}
+
+::testing::AssertionResult
+AssertTimespecClose::operator()(
+       const char* m_expr,
+       const char* n_expr,
+       const struct timespec & m,
+       const struct timespec & n
+       )
+{
+       struct timespec diff;
+
+       timespec_sub(&diff, &m, &n);
+       timespec_abs(&diff, &diff);
+       if (timespec_cmp(&limit, &diff) >= 0)
+               return ::testing::AssertionSuccess();
+
+       return ::testing::AssertionFailure()
+           << m_expr << " which is " << timespec_wrap(m)
+           << "\nand\n"
+           << n_expr << " which is " << timespec_wrap(n)
+           << "\nare not close; diff=" << timespec_wrap(diff);
+}
+
 } // namespace timeStruct
index 14e2c44d45ee04ef619f722f95e1349e07ccc5f1..1e9c2b22cc61136c37dc3716b01a0c4a28c2b221 100644 (file)
@@ -57,6 +57,8 @@ public:
                { V.tv_sec = hi; V.tv_usec = lo; }
        timeval_wrap(const struct timeval & rhs)
                { V = rhs; }
+       timeval_wrap(const timeval_wrap & rhs)
+               { V = rhs.V; }
        bool operator == (const timeval_wrap& rhs) const
                { return V.tv_sec == rhs.V.tv_sec &&
                         V.tv_usec == rhs.V.tv_usec ; }
@@ -83,6 +85,8 @@ public:
                { V.tv_sec = hi; V.tv_nsec = lo; }
        timespec_wrap(const struct timespec & rhs)
                { V = rhs; }
+       timespec_wrap(const timespec_wrap & rhs)
+               { V = rhs.V; }
        bool operator == (const timespec_wrap& rhs) const
                { return V.tv_sec == rhs.V.tv_sec &&
                         V.tv_nsec == rhs.V.tv_nsec ; }
@@ -143,6 +147,29 @@ public:
 };
 
 
+// timespec closeness testing predicate
+//
+// CAVEAT: This class uses the timevalops functions
+// - timespec_sub
+// - timespec_abs
+// - timespec_cmp
+//
+// See the equivalent timeval helper.
+class AssertTimespecClose {
+private:
+       struct timespec limit;
+
+public:
+       // note: (hi,lo) should be a positive normalised timespec;
+       // the constructor does not normalise the values!
+       AssertTimespecClose(time_t hi, int32 lo);
+
+       ::testing::AssertionResult
+       operator()(const char* m_expr, const char* n_expr,
+                  const struct timespec & m, const struct timespec & n);
+};
+
+
 } // namespace timeStruct
 
 // since googletest wants to string format items, we declare the
index cb43123afd35efbb5ee56cce54f7b82dcdde7178..0bf2248e654f9802ce623f0f3431da8db4fecb0f 100644 (file)
@@ -13,7 +13,6 @@ using namespace timeStruct;
 
 class timevalTest : public libntptest {
 protected:
-
        static u_int32 my_tick_to_tsf(u_int32 ticks);
        static u_int32 my_tsf_to_tick(u_int32 tsf);
 
@@ -36,9 +35,9 @@ timevalTest::my_tick_to_tsf(
        // arithmetic. This should give the precise fraction, rounded to
        // the nearest representation.
 #if SIZEOF_LONG >= 8
-       return (u_int32)((((u_long)ticks << 32) + 500000) / 1000000);
+       return u_int32(((u_long(ticks) << 32) + 500000) / 1000000);
 #else
-       return (u_int32)floor((double)ticks * 4294.967296 + 0.5);
+       return u_int32(double(ticks) * 4294.967296 + 0.5);
 #endif
        // And before you ask: if ticks >= 1000000, the result is
        // truncated nonsense, so don't use it out-of-bounds.
@@ -51,15 +50,15 @@ timevalTest::my_tsf_to_tick(
 {
        // Inverse operation: converts fraction to microseconds.
 #if SIZEOF_LONG >= 8
-       return (u_int32)(((u_long)tsf * 1000000 + 0x80000000) >> 32);
+       return u_int32((u_long(tsf) * 1000000 + 0x80000000) >> 32);
 #else
-       return (u_int32)floor((double)tsf / 4294.967296 + 0.5);
+       return u_int32(double(tsf) / 4294.967296 + 0.5);
 #endif
        // Beware: The result might be 10^6 due to rounding!
 }
 
-
 const long timevalTest::MICROSECONDS = 1000000;
+
 const timevalTest::lfpfracdata timevalTest::fdata [] = {
        {      0, 0x00000000 }, {   7478, 0x01ea1405 },
        {  22077, 0x05a6d699 }, { 125000, 0x20000000 },
@@ -78,32 +77,48 @@ const timevalTest::lfpfracdata timevalTest::fdata [] = {
 };
 
 
-// and the global predicate instances we're using here
+// ---------------------------------------------------------------------
+// test support stuff - part1
+// ---------------------------------------------------------------------
 
-static AssertFpClose FpClose(0, 1);
+TEST_F(timevalTest, Helpers1) {
+       timeval_wrap x;
 
-static AssertTimevalClose TimevalClose(0, 1);
+       for (x.V.tv_sec = -2; x.V.tv_sec < 3; x.V.tv_sec++)
+       {
+               x.V.tv_usec = -1;
+               ASSERT_FALSE(x.valid());
+               x.V.tv_usec = 0;
+               ASSERT_TRUE(x.valid());
+               x.V.tv_usec = 999999;
+               ASSERT_TRUE(x.valid());
+               x.V.tv_usec = 1000000;
+               ASSERT_FALSE(x.valid());
+       }
+}
 
-// ---------------------------------------------------------------------
-// test support stuff
-// ---------------------------------------------------------------------
+//----------------------------------------------------------------------
+// test normalisation
+//----------------------------------------------------------------------
 
 TEST_F(timevalTest, Normalise) {
        for (long ns = -2000000000; ns <= 2000000000; ns += 10000000) {
                timeval_wrap x(0, ns);
-
                timeval_norm(x);
                ASSERT_TRUE(x.valid());
        }
 }
 
+//----------------------------------------------------------------------
+// test classification
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, SignNoFrac) {
        // sign test, no fraction
        for (int i = -4; i <= 4; ++i) {
                timeval_wrap a(i, 0);
                int          E = (i > 0) - (i < 0);
                int          r = timeval_test(a);
-
                ASSERT_EQ(E, r);
        }
 }
@@ -114,12 +129,13 @@ TEST_F(timevalTest, SignWithFrac) {
                timeval_wrap a(i, 10);
                int          E = (i >= 0) - (i < 0);
                int          r = timeval_test(a);
-
                ASSERT_EQ(E, r);
        }
 }
 
+//----------------------------------------------------------------------
 // test compare
+//----------------------------------------------------------------------
 TEST_F(timevalTest, CmpFracEQ) {
        // fractions are equal
        for (int i = -4; i <= 4; ++i)
@@ -128,7 +144,6 @@ TEST_F(timevalTest, CmpFracEQ) {
                        timeval_wrap b(j, 200);
                        int          E = (i > j) - (i < j);
                        int          r = timeval_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
@@ -141,7 +156,6 @@ TEST_F(timevalTest, CmpFracGT) {
                        timeval_wrap b( j , 200);
                        int          E = (i >= j) - (i < j);
                        int          r = timeval_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
@@ -154,12 +168,14 @@ TEST_F(timevalTest, CmpFracLT) {
                        timeval_wrap b(j, 999800);
                        int          E = (i > j) - (i <= j);
                        int          r = timeval_cmp(a, b);
-
                        ASSERT_EQ(E, r);
                }
 }
 
+//----------------------------------------------------------------------
 // Test addition (sum)
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, AddFullNorm) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
@@ -167,7 +183,6 @@ TEST_F(timevalTest, AddFullNorm) {
                        timeval_wrap b(j, 400);
                        timeval_wrap E(i + j, 200 + 400);
                        timeval_wrap c;
-
                        timeval_add(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -190,7 +205,6 @@ TEST_F(timevalTest, AddUsecNorm) {
                timeval_wrap a(i, 200);
                timeval_wrap E(i, 600);
                timeval_wrap c;
-
                timeval_addus(c, a, 400);
                ASSERT_EQ(E, c);
        }
@@ -201,13 +215,15 @@ TEST_F(timevalTest, AddUsecOflow1) {
                timeval_wrap a(i, 200);
                timeval_wrap E(i + 1, 100);
                timeval_wrap c;
-
                timeval_addus(c, a, MICROSECONDS - 100);
                ASSERT_EQ(E, c);
        }
 }
 
+//----------------------------------------------------------------------
 // test subtraction (difference)
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, SubFullNorm) {
        for (int i = -4; i <= 4; ++i)
                for (int j = -4; j <= 4; ++j) {
@@ -215,7 +231,6 @@ TEST_F(timevalTest, SubFullNorm) {
                        timeval_wrap b(j, 400);
                        timeval_wrap E(i - j, 600 - 400);
                        timeval_wrap c;
-
                        timeval_sub(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -228,7 +243,6 @@ TEST_F(timevalTest, SubFullOflow) {
                        timeval_wrap b(j, 999900);
                        timeval_wrap E(i - j - 1, 200);
                        timeval_wrap c;
-
                        timeval_sub(c, a, b);
                        ASSERT_EQ(E, c);
                }
@@ -239,7 +253,6 @@ TEST_F(timevalTest, SubUsecNorm) {
                timeval_wrap a(i, 600);
                timeval_wrap E(i, 200);
                timeval_wrap c;
-
                timeval_subus(c, a, 600 - 200);
                ASSERT_EQ(E, c);
        }
@@ -250,32 +263,35 @@ TEST_F(timevalTest, SubUsecOflow) {
                timeval_wrap a(i, 100);
                timeval_wrap E(i - 1, 200);
                timeval_wrap c;
-
                timeval_subus(c, a, MICROSECONDS - 100);
                ASSERT_EQ(E, c);
        }
 }
 
+//----------------------------------------------------------------------
 // test negation
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, Neg) {
        for (int i = -4; i <= 4; ++i) {
                timeval_wrap a(i, 100);
                timeval_wrap b;
                timeval_wrap c;
-
                timeval_neg(b, a);
                timeval_add(c, a, b);
                ASSERT_EQ(0, timeval_test(c));
        }
 }
 
+//----------------------------------------------------------------------
 // test abs value
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, AbsNoFrac) {
        for (int i = -4; i <= 4; ++i) {
                timeval_wrap a(i, 0);
                timeval_wrap b;
                int          c;
-
                c = timeval_abs(b, a);
                ASSERT_EQ((i < 0), c);
                ASSERT_EQ((i != 0), timeval_test(b));
@@ -287,20 +303,49 @@ TEST_F(timevalTest, AbsWithFrac) {
                timeval_wrap a(i, 100);
                timeval_wrap b;
                int          c;
-
                c = timeval_abs(b, a);
                ASSERT_EQ((i < 0), c);
                ASSERT_EQ(1, timeval_test(b));
        }
 }
 
+// ---------------------------------------------------------------------
+// test support stuff -- part 2
+// ---------------------------------------------------------------------
+
+TEST_F(timevalTest, Helpers2) {
+       AssertTimevalClose isClose(0, 2);
+       timeval_wrap x, y;
+
+       for (x.V.tv_sec = -2; x.V.tv_sec < 3; x.V.tv_sec++)
+               for (x.V.tv_usec = 1;
+                    x.V.tv_usec < 1000000;
+                    x.V.tv_usec += 499999)
+               {
+                       for (long i = -4; i < 5; i++) {
+                               y = x;
+                               y.V.tv_usec += i;
+                               if (i >= -2 && i <= 2)
+                                       ASSERT_PRED_FORMAT2(isClose, x,y);
+                               else
+                                       ASSERT_PRED_FORMAT2(!isClose, x,y);
+                       }
+               }
+}
+
+// and the global predicate instances we're using here
+static AssertFpClose FpClose(0, 1);
+static AssertTimevalClose TimevalClose(0, 1);
+
+//----------------------------------------------------------------------
 // conversion to l_fp
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, ToLFPbittest) {
        for (u_int32 i = 0; i < 1000000; i++) {
                timeval_wrap a(1, i);
                l_fp_wrap    E(1, my_tick_to_tsf(i));
                l_fp_wrap    r;
-
                timeval_reltolfp(r, a);
                ASSERT_PRED_FORMAT2(FpClose, E, r);
        }
@@ -311,7 +356,6 @@ TEST_F(timevalTest, ToLFPrelPos) {
                timeval_wrap a(1, fdata[i].usec);
                l_fp_wrap    E(1, fdata[i].frac);
                l_fp_wrap    r;
-
                timeval_reltolfp(r, a);
                ASSERT_PRED_FORMAT2(FpClose, E, r);
        }
@@ -322,7 +366,6 @@ TEST_F(timevalTest, ToLFPrelNeg) {
                timeval_wrap a(-1, fdata[i].usec);
                l_fp_wrap    E(~0, fdata[i].frac);
                l_fp_wrap    r;
-
                timeval_reltolfp(r, a);
                ASSERT_PRED_FORMAT2(FpClose, E, r);
        }
@@ -333,13 +376,15 @@ TEST_F(timevalTest, ToLFPabs) {
                timeval_wrap a(1, fdata[i].usec);
                l_fp_wrap    E(1 + JAN_1970, fdata[i].frac);
                l_fp_wrap    r;
-
                timeval_abstolfp(r, a);
                ASSERT_PRED_FORMAT2(FpClose, E, r);
        }
 }
 
+//----------------------------------------------------------------------
 // conversion from l_fp
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, FromLFPbittest) {
        // Not *exactly* a bittest, because 2**32 tests would take a
        // really long time even on very fast machines! So we do test
@@ -348,30 +393,50 @@ TEST_F(timevalTest, FromLFPbittest) {
                timeval_wrap E(1, my_tsf_to_tick(tsf));
                l_fp_wrap    a(1, tsf);
                timeval_wrap r;
-
                timeval_relfromlfp(r, a);
-
                // The conversion might be off by one microsecond when
-               // comparing to calculated value; the table-driven
-               // conversion does not use all bits from the FP fraction
-               // and this truncation will affect the rounding.
+               // comparing to calculated value.
                ASSERT_PRED_FORMAT2(TimevalClose, E, r);
        }
 }
 
-// usec -> frac -> usec roundtrip
-TEST_F(timevalTest, LFProundtrip) {
-       for (u_int32 i = 0; i < 1000000; i++) {
-               timeval_wrap E(1, i);
-               l_fp_wrap    a;
+TEST_F(timevalTest, FromLFPrelPos) {
+       for (int i = 0; i < COUNTOF(fdata); i++) {
+               l_fp_wrap    a(1, fdata[i].frac);
+               timeval_wrap E(1, fdata[i].usec);
                timeval_wrap r;
+               timeval_relfromlfp(r, a);
+               ASSERT_PRED_FORMAT2(TimevalClose, E, r);
+       }
+}
 
-               timeval_reltolfp(a, E);
+TEST_F(timevalTest, FromLFPrelNeg) {
+       for (int i = 0; i < COUNTOF(fdata); i++) {
+               l_fp_wrap    a(~0, fdata[i].frac);
+               timeval_wrap E(-1, fdata[i].usec);
+               timeval_wrap r;
                timeval_relfromlfp(r, a);
-               ASSERT_EQ(E, r);
+               ASSERT_PRED_FORMAT2(TimevalClose, E, r);
        }
 }
 
+// usec -> frac -> usec roundtrip, using a prime start and increment
+TEST_F(timevalTest, LFProundtrip) {
+       for (int32_t t = -1; t < 2; ++t)
+               for (u_int32 i = 5; i < 1000000; i+=11) {
+                       timeval_wrap E(t, i);
+                       l_fp_wrap    a;
+                       timeval_wrap r;
+                       timeval_reltolfp(a, E);
+                       timeval_relfromlfp(r, a);
+                       ASSERT_EQ(E, r);
+               }
+}
+
+//----------------------------------------------------------------------
+// string formatting
+//----------------------------------------------------------------------
+
 TEST_F(timevalTest, ToString) {
        static const struct {
                time_t       sec;
@@ -391,7 +456,6 @@ TEST_F(timevalTest, ToString) {
                timeval_wrap a(data[i].sec, data[i].usec);
                std::string  E(data[i].repr);
                std::string  r(timeval_tostr(a));
-
                ASSERT_EQ(E, r);
        }
 }