[libcamera-devel] [PATCH 1/8] utils: checkstyle.py: Drop arguments to super() when possible

Niklas Söderlund niklas.soderlund at ragnatech.se
Sun Dec 27 11:24:36 CET 2020


Hi Laurent,

Thanks for your work.

On 2020-12-24 14:28:48 +0200, Laurent Pinchart wrote:
> The super() call is a shortcut syntax for super(__class__, <first arg>).
> Drom the arguments when they match the default.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  utils/checkstyle.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/utils/checkstyle.py b/utils/checkstyle.py
> index 7225cac47a4e..c04bf3850dcd 100755
> --- a/utils/checkstyle.py
> +++ b/utils/checkstyle.py
> @@ -198,7 +198,7 @@ _style_checkers = []
>  
>  class StyleCheckerRegistry(type):
>      def __new__(cls, clsname, bases, attrs):
> -        newclass = super(StyleCheckerRegistry, cls).__new__(cls, clsname, bases, attrs)
> +        newclass = super().__new__(cls, clsname, bases, attrs)
>          if clsname != 'StyleChecker':
>              _style_checkers.append(newclass)
>          return newclass
> @@ -387,7 +387,7 @@ _formatters = []
>  
>  class FormatterRegistry(type):
>      def __new__(cls, clsname, bases, attrs):
> -        newclass = super(FormatterRegistry, cls).__new__(cls, clsname, bases, attrs)
> +        newclass = super().__new__(cls, clsname, bases, attrs)
>          if clsname != 'Formatter':
>              _formatters.append(newclass)
>          return newclass
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list