NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

libcamera: src: ipa: raspberrypi: agc: Fix uninitialised members in Agc constructor

Browse source 
Use memset in the constructor for embedded structures, it is tidier
and initialises everything. We use the initialiser list for other
members.

Signed-off-by: David Plowman <david.plowman@raspberrypi.com>
Reviewed-by: Naushir Patuck <naush@raspberrypi.com>
Acked-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
David Plowman 2020-11-23 07:38:02 +00:00 • committed by Kieran Bingham
parent 2fb5b298da
commit 393f05a9a8
1 changed files with 8 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

17
src/ipa/raspberrypi/controller/rpi/agc.cpp
View file

@ -153,16 +153,15 @@ void AgcConfig::Read(boost::property_tree::ptree const &params)
Agc::Agc(Controller *controller)
: AgcAlgorithm(controller), metering_mode_(nullptr),
exposure_mode_(nullptr), constraint_mode_(nullptr),
frame_count_(0), lock_count_(0)
frame_count_(0), lock_count_(0),
ev_(1.0), flicker_period_(0.0),
fixed_shutter_(0), fixed_analogue_gain_(0.0)
{
ev_ = status_.ev = 1.0;
flicker_period_ = status_.flicker_period = 0.0;
fixed_shutter_ = status_.fixed_shutter = 0;
fixed_analogue_gain_ = status_.fixed_analogue_gain = 0.0;
// set to zero initially, so we can tell it's not been calculated
status_.total_exposure_value = 0.0;
status_.target_exposure_value = 0.0;
status_.locked = false;
memset(&awb_, 0, sizeof(awb_));
// Setting status_.total_exposure_value_ to zero initially tells us
// it's not been calculated yet (i.e. Process hasn't yet run).
memset(&status_, 0, sizeof(status_));
status_.ev = ev_;
}
char const *Agc::Name() const