[PATCH 2/2] test: vboot: Using variable 'old_dtb' before assignment

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Mon Dec 11 18:55:10 CET 2023


On 11.12.23 18:52, Simon Glass wrote:
> On Sat, 9 Dec 2023 at 11:54, Heinrich Schuchardt
> <heinrich.schuchardt at canonical.com> wrote:
>>
>> old_dtb can only be assumed initialized in the finally block
>> if it is assigned a value before the try statement.
>>
>> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>> ---
>>   test/py/tests/test_vboot.py | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> I suggest mentioning the motivation for the patch in your commit
> message. E.g. does it cause a pylint failure?

You only get a pylint error with current pylint, not with the one used 
by our CI.

Best regards

Heinrich


More information about the U-Boot mailing list