[PATCH v2 2/2] utils: checkstyle: Remove style checker for python pep8

[Laurent Pinchart]

Hi Stefan,

Thank you for the patch.

On Tue, Sep 03, 2024 at 06:10:55PM +0200, Stefan Klug wrote:
> The issues detected and fixed by autopep8 are the same as the ones
> detected by pycodestyle. As the formatter runs unconditionally we can
> remove the checker.
> 
> Signed-off-by: Stefan Klug <stefan.klug at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  utils/checkstyle.py | 33 ---------------------------------
>  1 file changed, 33 deletions(-)
> 
> diff --git a/utils/checkstyle.py b/utils/checkstyle.py
> index 1ee211c3bb9b..ab89c0a14fab 100755
> --- a/utils/checkstyle.py
> +++ b/utils/checkstyle.py
> @@ -709,39 +709,6 @@ class MesonChecker(StyleChecker):
>          return issues
>  
>  
> -class Pep8Checker(StyleChecker):
> -    patterns = ('*.py',)
> -    results_regex = re.compile(r'stdin:([0-9]+):([0-9]+)(.*)')
> -
> -    def __init__(self, content):
> -        super().__init__()
> -        self.__content = content
> -
> -    def check(self, line_numbers):
> -        issues = []
> -        data = ''.join(self.__content).encode('utf-8')
> -
> -        try:
> -            ret = subprocess.run(['pycodestyle', '--ignore=E501', '-'],
> -                                 input=data, stdout=subprocess.PIPE)
> -        except FileNotFoundError:
> -            issues.append(StyleIssue(0, None, None, 'Please install pycodestyle to validate python additions'))
> -            return issues
> -
> -        results = ret.stdout.decode('utf-8').splitlines()
> -        for item in results:
> -            search = re.search(Pep8Checker.results_regex, item)
> -            line_number = int(search.group(1))
> -            position = int(search.group(2))
> -            msg = search.group(3)
> -
> -            if line_number in line_numbers:
> -                line = self.__content[line_number - 1]
> -                issues.append(StyleIssue(line_number, None, line, msg))
> -
> -        return issues
> -
> -
>  class ShellChecker(StyleChecker):
>      patterns = ('*.sh',)
>      results_line_regex = re.compile(r'In - line ([0-9]+):')

-- 
Regards,

Laurent Pinchart