Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210824085947.224062-1-nghialm78@gmail.com>
In-Reply-To: <20210824085947.224062-1-nghialm78@gmail.com>
From:   Lukas Bulwahn <lukas.bulwahn@gmail.com>
Date:   Tue, 24 Aug 2021 16:37:19 +0200
Message-ID: <CAKXUXMzbJRHyUpFLGk8SBJHrkmkwq4VtfXmgvZtUBJGiED8qxw@mail.gmail.com>
Subject: Re: [PATCH] perf tools: Add missing newline at the end of file
To:     Nghia Le <nghialm78@gmail.com>
Cc:     Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>, acme@kernel.org,
        mark.rutland@arm.com, alexander.shishkin@linux.intel.com,
        jolsa@redhat.com, Namhyung Kim <namhyung@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-perf-users@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, Aug 24, 2021 at 11:00 AM Nghia Le <nghialm78@gmail.com> wrote:
>
> Add missing newline at the end of file parse-sublevel-options.h.
> Thus removing relevant warning reported by checkpatch.
>
> Signed-off-by: Nghia Le <nghialm78@gmail.com>

Reviewed-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>

This addresses the only instance of a "no newline at the end of the
file" throughout the kernel tree.

This is part of a larger effort to identify all rules that checkpatch
warns about and that are never false positives (compared to the many
rules in checkpatch that are heuristics or are known to be ignored by
the current development community and hence, cause many false
positives). We could use such rules to be enforced in the submission
process.

Lukas
Lukas
> ---
>  tools/perf/util/parse-sublevel-options.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/util/parse-sublevel-options.h b/tools/perf/util/parse-sublevel-options.h
> index 9b9efcc2aaad..578b18ef03bb 100644
> --- a/tools/perf/util/parse-sublevel-options.h
> +++ b/tools/perf/util/parse-sublevel-options.h
> @@ -8,4 +8,4 @@ struct sublevel_option {
>
>  int perf_parse_sublevel_options(const char *str, struct sublevel_option *opts);
>
> -#endif
> \ No newline at end of file
> +#endif
> --
> 2.25.1
>