>> We will modify the the if in the when code like this:
>>
>> @@ -22,7 +22,7 @@ if (id == NULL || ...) { ... return ...; }
>> ... when != put_device(&id->dev)
…
>> - when != if (id) { ... put_device(&id->dev) ... }
>> + when != if (...) { ... put_device(&id->dev) ... }
>
> This looks ok.
I have got another different software development opinion also for
such SmPL code in the adjusted line.
I find this specific source code search variant irrelevant
because the shown reference release function should be found
by the first SmPL when specification already.
Would you like to determine generally if the desired function call
is present at all?
Thus I do not see a need (or requirement) for a duplicate search attempt.
Regards,
Markus
