[PATCH] test: ipa: rkisp1-utils: Fix coding style for template arguments
Kieran Bingham
kieran.bingham at ideasonboard.com
Mon Jun 3 10:53:38 CEST 2024
Quoting Laurent Pinchart (2024-06-02 00:57:40)
> The coding style names template arguments using CamelCase with an
> uppercase initial letter. Fix the template arguments in the rkisp1-utils
> test.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> ---
> test/ipa/rkisp1/rkisp1-utils.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/ipa/rkisp1/rkisp1-utils.cpp b/test/ipa/rkisp1/rkisp1-utils.cpp
> index 9fff0cdcef15..e48f8d3622d2 100644
> --- a/test/ipa/rkisp1/rkisp1-utils.cpp
> +++ b/test/ipa/rkisp1/rkisp1-utils.cpp
> @@ -21,10 +21,10 @@ using namespace ipa::rkisp1;
> class RkISP1UtilsTest : public Test
> {
> protected:
> - template<unsigned int intPrec, unsigned fracPrec, typename T>
> + template<unsigned int IntPrec, unsigned FracPrec, typename T>
> int testSingleFixedPoint(double input, T expected)
> {
> - T ret = utils::floatingToFixedPoint<intPrec, fracPrec, T>(input);
> + T ret = utils::floatingToFixedPoint<IntPrec, FracPrec, T>(input);
> if (ret != expected) {
> cerr << "Expected " << input << " to convert to "
> << expected << ", got " << ret << std::endl;
> @@ -35,7 +35,7 @@ protected:
> * The precision check is fairly arbitrary but is based on what
> * the rkisp1 is capable of in the crosstalk module.
> */
> - double f = utils::fixedToFloatingPoint<intPrec, fracPrec, double>(ret);
> + double f = utils::fixedToFloatingPoint<IntPrec, FracPrec, double>(ret);
> if (std::abs(f - input) > 0.005) {
> cerr << "Reverse conversion expected " << ret
> << " to convert to " << input
>
> base-commit: aad5837d70f8d3dc19f62f88f1434e3c9cf5ffbe
> --
> Regards,
>
> Laurent Pinchart
>
More information about the libcamera-devel
mailing list