On Thu, 15 May 2025 at 14:59, Daniel P. Berrangé <berra...@redhat.com> wrote:
>
> The ACPI test data check needs to analyse a list of all files in a
> commit, so can use the new hook for processing the file list.
>
> Signed-off-by: Daniel P. Berrangé <berra...@redhat.com>
> ---
>  scripts/checkpatch.pl | 61 ++++++++++++++++++++-----------------------
>  1 file changed, 29 insertions(+), 32 deletions(-)



> @@ -1770,9 +1769,7 @@ sub process {
>                      $line =~ /^rename (?:from|to) [\w\/\.\-]+\s*$/ ||
>                      ($line =~ 
> /\{\s*([\w\/\.\-]*)\s*\=\>\s*([\w\/\.\-]*)\s*\}/ &&
>                       (defined($1) || defined($2)))) &&
> -                      !(($realfile ne '') &&
> -                        defined($acpi_testexpected) &&
> -                        ($realfile eq $acpi_testexpected))) {
> +                   $realfile !~ m#^tests/data/acpi/#) {

Is the indentation off on this line?

>                         $reported_maintainer_file = 1;
>                         WARN("added, moved or deleted file(s), does 
> MAINTAINERS need updating?\n" . $herecurr);
>                 }
> --

Otherwise

Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>

thanks
-- PMM

Reply via email to