[PATCH] test: ipa: rkisp1-utils: Fix coding style for template arguments

Paul Elder paul.elder at ideasonboard.com
Sun Jun 2 18:48:57 CEST 2024


On Sun, Jun 02, 2024 at 02:57:40AM +0300, Laurent Pinchart wrote:
> 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: Paul Elder <paul.elder 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


More information about the libcamera-devel mailing list