On 13. Sep 2025, at 15:11, Klara Modin wrote: > On 2025-09-13 14:15:14 +0200, Thorsten Blum wrote: >> The goto label 'successful_load' isn't really necessary. Set 'res = 1' >> immediately and let 'goto done' handle the rest. >> >> No functional changes. >> >> Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx> >> --- >> ... > > This is now the only place where res will be set to 1. Sorry my bad. > This does not seem correct? After this patch res is not updated to 1 anymore > if execution reaches here without taking another goto, i.e. the return > value is changed by this patch. Thanks, I agree that it doesn't make sense. I only intended to delete the label, not the 'res = 1'. Please ignore this patch (unless someone thinks removing the goto label is worth a v2) and sorry for the noise. Thanks, Thorsten
