[libcamera-devel] [PATCH 0/5] General fixes

Kieran Bingham kieran.bingham at ideasonboard.com
Thu Jan 17 21:20:38 CET 2019


Heya,

Here's a few miscellaneous patches I've collected this week while
developing.

The first three patches tackle the Timer class, after I hit an issue
where timers longer than 4.29 seconds fail, due to the internals
wrapping at 32 bits.

The first patch ensures that all member variables are initialised as
this was generating a warning in my IDE static analyser.

Then we add a test which fails, and follow that by a patch to introduce
the fix allowing the test to pass.

Patch 4 fixes up some warnings that our checkstyle.py script complained
about in the log class, and I caught a /really/ trivial spelling error
in event_dispatch_poll.



Kieran Bingham (5):
  test: timer: Initialise all variables
  test: timer: Add a 32 bit wraparound test
  libcamera: timer: Fix 32 bit wrap
  libcamera: log: Fix coding style warnings
  libcamera: event_dispatcher_poll: Fix trivial spelling

 src/libcamera/event_dispatcher_poll.cpp |  2 +-
 src/libcamera/include/log.h             |  6 +++---
 src/libcamera/log.cpp                   |  2 +-
 src/libcamera/timer.cpp                 |  3 ++-
 test/timer.cpp                          | 16 +++++++++++++++-
 5 files changed, 22 insertions(+), 7 deletions(-)

-- 
2.17.1



More information about the libcamera-devel mailing list