|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 1/1] xen/scripts: Fix regex syntax warnings with Python 3.12
CCing Luca Fancellu.
On Thursday, December 19, 2024 19:10 CET, Ariel Otilibili
<Ariel.Otilibili-Anieli@xxxxxxxxxx> wrote:
> Same fix than commit 826a9eb072 (tools: Fix regex syntax warnings with Python
> 3.12).
>
> It clears out the warning:
>
> ```
> $ xen/scripts/xen-analysis.py
> xen/scripts/xen_analysis/cppcheck_analysis.py:94: SyntaxWarning: invalid
> escape sequence '\*'
> comment_line_starts = re.match('^[ \t]*/\*.*$', line)
> ```
>
> The warning appears only the first time the command is run, then it
> disappears.
>
> Fixes: 02b26c02c7 (xen/scripts: add cppcheck tool to the xen-analysis.py
> script)
> Signed-off-by: Ariel Otilibili <Ariel.Otilibili-Anieli@xxxxxxxxxx>
> --
> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> Cc: Michal Orzel <michal.orzel@xxxxxxx>
> Cc: Jan Beulich <jbeulich@xxxxxxxx>
> Cc: Julien Grall <julien@xxxxxxx>
> Cc: "Roger Pau Monné" <roger.pau@xxxxxxxxxx>
> Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> ---
> xen/scripts/xen_analysis/cppcheck_analysis.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/xen/scripts/xen_analysis/cppcheck_analysis.py
> b/xen/scripts/xen_analysis/cppcheck_analysis.py
> index ce7bda91b6..532a14b0ad 100644
> --- a/xen/scripts/xen_analysis/cppcheck_analysis.py
> +++ b/xen/scripts/xen_analysis/cppcheck_analysis.py
> @@ -91,7 +91,7 @@ def __generate_suppression_list(out_file):
> line = header_content[tmp_line]
> # Matches a line with just optional spaces/tabs and
> the
> # start of a comment '/*'
> - comment_line_starts = re.match('^[ \t]*/\*.*$', line)
> + comment_line_starts = re.match(r'^[ \t]*/\*.*$',
> line)
> # Matches a line with text and the end of a comment
> '*/'
> comment_line_stops = re.match(r'^.*\*/$', line)
> if (not comment_section) and comment_line_starts:
> --
> 2.47.1
>
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |