[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