[libcamera-devel] [PATCH 9/9] test: timer: Test start() with absolute deadline
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Mon Oct 7 05:20:08 CEST 2019
Hi Jacopo,
On Sun, Oct 06, 2019 at 09:37:28PM +0200, Jacopo Mondi wrote:
> On Sun, Oct 06, 2019 at 08:32:26AM +0300, Laurent Pinchart wrote:
> > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > ---
> > test/timer.cpp | 19 +++++++++++++++++++
> > 1 file changed, 19 insertions(+)
> >
> > diff --git a/test/timer.cpp b/test/timer.cpp
> > index 5ff94dbbdeb0..93ce26a0a9a3 100644
> > --- a/test/timer.cpp
> > +++ b/test/timer.cpp
> > @@ -35,6 +35,15 @@ public:
> > Timer::start(msec);
> > }
> >
> > + void start(std::chrono::steady_clock::time_point deadline)
> > + {
> > + count_ = 0;
> > + start_ = std::chrono::steady_clock::now();
> > + expiration_ = std::chrono::steady_clock::time_point();
> > +
> > + Timer::start(deadline);
> > + }
> > +
> > int jitter()
> > {
> > std::chrono::steady_clock::duration duration = expiration_ - deadline();
> > @@ -127,6 +136,16 @@ protected:
> > return TestFail;
> > }
> >
> > + /* Timer with absolute deadline. */
> > + timer.start(std::chrono::steady_clock::now() + std::chrono::milliseconds(200));
> > +
> > + dispatcher->processEvents();
> > +
> > + if (timer.isRunning() || timer.jitter() > 50) {
> > + cout << "Absolute timer test failed" << endl;
>
> "Absolute timer" sounds funny.
I'll rename that to "Absolute deadline".
> Otherwise
> Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>
>
> > + return TestFail;
> > + }
> > +
> > /* Two timers. */
> > timer.start(1000);
> > timer2.start(300);
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list