Re: [PATCH] perf test: fix spelling mistake "leadking" -> "leaking"

From: Colin Ian King
Date: Wed Apr 17 2019 - 07:09:31 EST


On 17/04/2019 12:07, Mukesh Ojha wrote:
>
> On 4/17/2019 4:25 PM, Colin King wrote:
>> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>>
>> There are a couple of spelling mistakes in test assert messages. Fix
>> them.
>>
>> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> Well, how are you shooting these mistakes one after the other?

I've got a spelling checker and I'm working down a list of 65948
potential mistakes. I'm doing this between kernel build + debug cycles
while I'm waiting for kernels to build and tests to complete.



>
> Reviewed-by: Mukesh Ojha <mojha@xxxxxxxxxxxxxx>
>
> Cheers,
> -Mukesh
>
>> ---
>> Â tools/perf/tests/dso-data.c | 4 ++--
>> Â 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
>> index 7f6c52021e41..946ab4b63acd 100644
>> --- a/tools/perf/tests/dso-data.c
>> +++ b/tools/perf/tests/dso-data.c
>> @@ -304,7 +304,7 @@ int test__dso_data_cache(struct test *test
>> __maybe_unused, int subtest __maybe_u
>> ÂÂÂÂÂ /* Make sure we did not leak any file descriptor. */
>> ÂÂÂÂÂ nr_end = open_files_cnt();
>> ÂÂÂÂÂ pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> -ÂÂÂ TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
>> +ÂÂÂ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
>> ÂÂÂÂÂ return 0;
>> Â }
>> Â @@ -380,6 +380,6 @@ int test__dso_data_reopen(struct test *test
>> __maybe_unused, int subtest __maybe_
>> ÂÂÂÂÂ /* Make sure we did not leak any file descriptor. */
>> ÂÂÂÂÂ nr_end = open_files_cnt();
>> ÂÂÂÂÂ pr_debug("nr start %ld, nr stop %ld\n", nr, nr_end);
>> -ÂÂÂ TEST_ASSERT_VAL("failed leadking files", nr == nr_end);
>> +ÂÂÂ TEST_ASSERT_VAL("failed leaking files", nr == nr_end);
>> ÂÂÂÂÂ return 0;
>> Â }