Re: [LTP] [PATCH v9 5/5] Add documentation about syscalls.h generator

Petr Vorel
2024-11-04T08:17:21Z

Hi Andrea,

Reviewed-by: Petr Vorel <pvorel@suse.cz>

LGTM, thanks!

> diff --git a/doc/developers/test_case_tutorial.rst b/doc/developers/test_case_tutorial.rst
> index be9a0ea8bac1d690b4ed4879d110f87c3f2a381a..97f385e5853a9041ac3e5ded7fb99e8355d84742 100644
> --- a/doc/developers/test_case_tutorial.rst
> +++ b/doc/developers/test_case_tutorial.rst
> @@ -302,24 +302,27 @@ for a distribution's C library version to be older than its kernel or it may use
> cut down C library in comparison to the GNU one. So we must call ``statx()``
> using the general ``syscall()`` interface.

> -The LTP contains a library for dealing with the ``syscall`` interface, which is
> +LTP contains a library for dealing with the ``syscall`` interface, which is
very nit: This was added by Richie in 7324ee19de. I also think that the "The"
article should not be there, but I'm surprised that he would make a make a mistake
(native speaker).

Kind regards,
Petr