From patchwork Tue Apr 1 13:19:39 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= X-Patchwork-Id: 23098 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id C91C8C3213 for ; Tue, 1 Apr 2025 13:19:54 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 08BB668993; Tue, 1 Apr 2025 15:19:50 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="wNTsAZWX"; dkim-atps=neutral Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 0FF9A68987 for ; Tue, 1 Apr 2025 15:19:44 +0200 (CEST) Received: from pb-laptop.local (185.221.143.221.nat.pool.zt.hu [185.221.143.221]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id D6EA78DB for ; Tue, 1 Apr 2025 15:17:51 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1743513472; bh=mwW6x9TBMmlyGxuMnq6tMVV/YoQQ+Jb8V2Sov7+p4Zc=; h=From:To:Subject:Date:In-Reply-To:References:From; b=wNTsAZWXFzrdfxiJSQjnpN8OuE0+FMqWSzzN81TZacFDqyP0wHopuud6IQNqO83My Cwu1J+XCvn9YuBVRLa8XKFVfvu9JhKFUmkMbrAFMokcn22IMmPMOE3t53onRuHGa7E ZnNLYnHIIVYeNFaQXX/s4i9hlV0U2XzC2q7L9hXc= From: =?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= To: libcamera-devel@lists.libcamera.org Subject: [PATCH v1 4/4] libcamera: controls: Simplify ControlValue::{ControlValue, get, set} Date: Tue, 1 Apr 2025 15:19:39 +0200 Message-ID: <20250401131939.749583-5-barnabas.pocze@ideasonboard.com> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250401131939.749583-1-barnabas.pocze@ideasonboard.com> References: <20250401131939.749583-1-barnabas.pocze@ideasonboard.com> MIME-Version: 1.0 X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Whether or not the control has an array type can be determined from the static information in `detail::control_type`, so use that and `if constexpr` to deduplicate the constructor, get, and set functions. Signed-off-by: Barnabás Pőcze --- include/libcamera/controls.h | 75 ++++++++++-------------------------- 1 file changed, 20 insertions(+), 55 deletions(-) diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h index 85c724ec1..22b5e3d96 100644 --- a/include/libcamera/controls.h +++ b/include/libcamera/controls.h @@ -136,28 +136,14 @@ public: ControlValue(); #ifndef __DOXYGEN__ - template::value && - details::control_type::value && - !std::is_same>::value, - std::nullptr_t> = nullptr> - ControlValue(const T &value) - : type_(ControlTypeNone), numElements_(0) - { - set(details::control_type>::value, false, - &value, 1, sizeof(T)); - } - - template::value || - std::is_same>::value, - std::nullptr_t> = nullptr> + template::value)>> #else template #endif ControlValue(const T &value) - : type_(ControlTypeNone), numElements_(0) + : type_(ControlTypeNone), isArray_(false), numElements_(0) { - set(details::control_type>::value, true, - value.data(), value.size(), sizeof(typename T::value_type)); + set(value); } ~ControlValue(); @@ -180,54 +166,33 @@ public: return !(*this == other); } -#ifndef __DOXYGEN__ - template::value && - !std::is_same>::value, - std::nullptr_t> = nullptr> - T get() const - { - assert(type_ == details::control_type>::value); - assert(!isArray_); - - return *reinterpret_cast(data().data()); - } - - template::value || - std::is_same>::value, - std::nullptr_t> = nullptr> -#else template -#endif T get() const { - assert(type_ == details::control_type>::value); - assert(isArray_); + using TypeInfo = details::control_type>; - using V = typename T::value_type; - const V *value = reinterpret_cast(data().data()); - return T{ value, numElements_ }; - } + assert(type_ == TypeInfo::value); + assert(isArray_ == (TypeInfo::size > 0)); -#ifndef __DOXYGEN__ - template::value && - !std::is_same>::value, - std::nullptr_t> = nullptr> - void set(const T &value) - { - set(details::control_type>::value, false, - reinterpret_cast(&value), 1, sizeof(T)); + if constexpr (TypeInfo::size > 0) + return T(reinterpret_cast(data().data()), numElements_); + else + return *reinterpret_cast(data().data()); } - template::value || - std::is_same>::value, - std::nullptr_t> = nullptr> -#else template -#endif void set(const T &value) { - set(details::control_type>::value, true, - value.data(), value.size(), sizeof(typename T::value_type)); + using TypeInfo = details::control_type>; + constexpr auto type = TypeInfo::value; + + if constexpr (TypeInfo::size > 0) { + static_assert(std::is_trivially_copyable_v); + set(type, true, value.data(), value.size(), sizeof(typename T::value_type)); + } else { + static_assert(std::is_trivially_copyable_v); + set(type, false, reinterpret_cast(&value), 1, sizeof(T)); + } } void reserve(ControlType type, bool isArray = false,