Re: [PATCH][next] platform/x86: thinkpad_acpi: remove redundant assignment to variable i

From: Ilpo Järvinen
Date: Wed Jan 24 2024 - 06:02:24 EST


On Sat, 06 Jan 2024 15:47:40 +0000, Colin Ian King wrote:

> The variable i is being initialized with the value 0 that is never
> read, it is being re-assigned 0 again in a for-loop statement later
> on. The initialization is redundant and can be removed.
>
> The initialization of variable n can also be deferred after the
> sanity check on pointer n and the declaration of all the int variables
> can be combined as a final code clear-up.
>
> [...]


Thank you for your contribution, it has been applied to my local
review-ilpo branch. Note it will show up in the public
platform-drivers-x86/review-ilpo branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/1] platform/x86: thinkpad_acpi: remove redundant assignment to variable i
commit: 214509e5d61d294193b220f397418e76879f74c0

--
i.