summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2021-04-07 16:52:46 +0300
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2021-04-12 22:16:52 +0300
commit9b2544d3df481bc9dd0795fb645b544ba6484216 (patch)
treefff75ad9c40d91ec37c618758145d4f44fd31873
parent210b81104730d32c47a689e3217f58f344c50f8c (diff)
libcamera: bound_method: Fix type of pack for void methods
The BoundMethodPack used by the void BoundMethodArgs variant incorrectly specified the template argument as void * instead of void. This causes no functional problem, but results in space for an unused void * return value being reserved. Fix it. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Sebastian Fricke <sebastian.fricke@posteo.net> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--include/libcamera/bound_method.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libcamera/bound_method.h b/include/libcamera/bound_method.h
index 95a95653..f216e3b5 100644
--- a/include/libcamera/bound_method.h
+++ b/include/libcamera/bound_method.h
@@ -174,7 +174,7 @@ template<typename T, typename... Args>
class BoundMethodMember<T, void, Args...> : public BoundMethodArgs<void, Args...>
{
public:
- using PackType = typename BoundMethodArgs<void *, Args...>::PackType;
+ using PackType = typename BoundMethodArgs<void, Args...>::PackType;
BoundMethodMember(T *obj, Object *object, void (T::*func)(Args...),
ConnectionType type = ConnectionTypeAuto)