[Tarantool-patches] [PATCH luajit 2/4] sysprof: fix typo in the comment
Sergey Kaplun
skaplun at tarantool.org
Mon Feb 10 17:51:32 MSK 2025
Hi, Sergey!
Thanks for the patch!
LGTM, with a few nits below regarding the commit message.
On 04.02.25, Sergey Bronnikov wrote:
> The patch fixes inconsistency in comment and source code below
> the comment. The typo was added in the initial commit
> 2593a9f83072 ("misc: introduce Lua API for new sampling profiler").
Please use the full commit hash.
I know that this is enough too, but:
a) It would be consistent with other commit messages.
b) There is less probability of the collision between hashes.
Maybe it is worth to mention "Follows up tarantool/tarantool#781" (the
sysprof ticket) if you want (I am not sure that this is needed, since it
is only the comment).
> ---
> src/lib_misc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/lib_misc.c b/src/lib_misc.c
> index ca1d1c75..5b7a4b62 100644
> --- a/src/lib_misc.c
> +++ b/src/lib_misc.c
<snipped>
> --
> 2.34.1
>
--
Best regards,
Sergey Kaplun
More information about the Tarantool-patches
mailing list