Re: [PATCH] bpftool: Fix the wrong format specifier

From: Quentin Monnet
Date: Thu Nov 21 2024 - 10:36:07 EST


2024-11-21 13:43 UTC+0100 ~ Alexis Lothoré <alexis.lothore@xxxxxxxxxxx>
> Hi Liujing,
>
> On 11/21/24 13:17, liujing wrote:
>> The type of lines is unsigned int, so the correct format specifier should be
>> %u instead of %d.
>>
>> Signed-off-by: liujing <liujing@xxxxxxxxxxxxxxxxxxxx>
>>
>> V1 -> V2: Fixed two other wrong type outputs about lines
>
> This commit changelog line should not appear in the commit message. This is
> still useful for review though, so you can keep it in the commit by using a ---
> separator (see other patches on the ML), which will make it automatically
> trimmed when the maintainers will apply the patch.

Also next time please remember to tag your patch as "v2" in the email
subject.

Patch looks good, thank you!

Reviewed-by: Quentin Monnet <qmo@xxxxxxxxxx>