Re: [PATCH v2] Sphinx error fixed for inline literal end-string by changing $type_constant2 in kernel-doc script to include "*" unicode character in highlights_rst.

From: Jonathan Corbet
Date: Wed May 01 2024 - 14:57:18 EST


Utkarsh Tripathi <utripathi2002@xxxxxxxxx> writes:

> The kernel-doc script uses the $type_constant2 variable to match
> expressions used to find embedded type information. The current
> implementation of $type_constant2 does not include the "*" unicode
> character, which is used to highlight inline literals in the
> documentation. This causes a Sphinx error when the inline literal
> end-string is used in the documentation.

So I need to look a bit further at the actual change, but I do have a
couple of comments on the patch itself. First, the text above is a
reasonable description of the problem, as a changelog should have. That
said, the subject line could be a bit shorter and to the point.

This text below:

> This commit follows the pattern of the commit
> 8aaf297a0dd6 ("docs: scripts: kernel-doc: accept bitwise negation like ~@var")
> and takes inspiration from the following commit
> 69fc23efc7e5 ("kernel-doc: Add unary operator * to $type_param_ref").
>
> Thanks Akira, for your suggestions, I have made the required changes.
> I am fairly new to the kernel community, so if I am making
> any mistakes while making patches and replying to mails,
> please let me know, it will be very helpful.

..doesn't belong in the changelog. If you put comments like this below
the "---" line, then the maintainer won't have to edit them out when
applying the patch.

> Signed-off-by: Utkarsh Tripathi <utripathi2002@xxxxxxxxx>
> Reviewed-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> Suggested-by: Akira Yokosawa <akiyks@xxxxxxxxx>

Did Akira offer you that Reviewed-by tag? I haven't seen it (which
doesn't mean it didn't happen). If it was not explicitly given to you,
though, you cannot put it here.

Thanks,

jon