summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-10-12 02:10:28 +0300
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-10-13 20:37:32 +0300
commita1772d44e8655484f47baefa92c8d6ed15603fee (patch)
treec17ebe98b54a1f3f321ffcb541e7f1926ae2bf5b
parent273b87c781554b33b641a9aea017a80570749b11 (diff)
libcamera: controls: Default ControlList validator argument to nullptr
The ControlList constructor takes a validator pointer that can be null. Set its default value to nullptr to simplify code in users of ControlList. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Jacopo Mondi <jacopo@jmondi.org> Tested-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
-rw-r--r--include/libcamera/controls.h2
-rw-r--r--src/ipa/rkisp1/rkisp1.cpp2
-rw-r--r--src/libcamera/request.cpp2
3 files changed, 3 insertions, 3 deletions
diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h
index 342251c2..12a13aac 100644
--- a/include/libcamera/controls.h
+++ b/include/libcamera/controls.h
@@ -126,7 +126,7 @@ private:
using ControlListMap = std::unordered_map<const ControlId *, ControlValue>;
public:
- ControlList(ControlValidator *validator);
+ ControlList(ControlValidator *validator = nullptr);
using iterator = ControlListMap::iterator;
using const_iterator = ControlListMap::const_iterator;
diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp
index 02419e25..80138f19 100644
--- a/src/ipa/rkisp1/rkisp1.cpp
+++ b/src/ipa/rkisp1/rkisp1.cpp
@@ -220,7 +220,7 @@ void IPARkISP1::setControls(unsigned int frame)
void IPARkISP1::metadataReady(unsigned int frame, unsigned int aeState)
{
- ControlList ctrls(nullptr);
+ ControlList ctrls;
if (aeState)
ctrls.set(controls::AeLocked, aeState == 2);
diff --git a/src/libcamera/request.cpp b/src/libcamera/request.cpp
index 23d3ab6f..e800f144 100644
--- a/src/libcamera/request.cpp
+++ b/src/libcamera/request.cpp
@@ -69,7 +69,7 @@ Request::Request(Camera *camera, uint64_t cookie)
/**
* \todo: Add a validator for metadata controls.
*/
- metadata_ = new ControlList(nullptr);
+ metadata_ = new ControlList();
}
Request::~Request()