Re: [PATCH] scripts/show_delta: add __main__ judgement before main code

[Hu Haowen]

On 2023/10/14 18:15, Miguel Ojeda wrote:
> On Fri, Oct 13, 2023 at 3:29 PM Hu Haowen <2023002089@xxxxxxxxxxxxxxxx> wrote:
> > When doing Python programming it is a nice convention to insert the if
> > statement `if __name__ == "__main__":` before any main code that does
> > actual functionalities to ensure the code will be executed only as a
> > script rather than as an imported module. Hence attach the missing
> > judgement to show_delta.
> >
> > Signed-off-by: Hu Haowen <2023002089@xxxxxxxxxxxxxxxx>
> I am not sure why I was Cc'd, but the patch looks fine :)


Sorry for troubling you. I have just executed the get_maintainer script
to get the maintainer for the file I'm making the patch toward, with
the result that this file is not maintained by anyone. Consequently I
immediately ran again on the scripts directory, and obtained adequate
information on major commiters of scripts, adding them to the Cc list.

After all, it's nice of you to have ensured the quality of my patch. :)


> Reviewed-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
>
> Cheers,
> Miguel