> On 1/4/24 01:16, Artem Chernyshev wrote:
> 
> It doesn't really fix anything. The code is not broken.
> 
> 

True, just mentioned commit which I addressed in my patch

> 
> No. This changes semantics. Prior to your change, if
> qemuMonitorJSONParseCPUx86FeatureWord() failed then 'return NULL' path
> would be taken. After your change the failure is ignored and ...
> 
> 
> ... this return path is taken.
> 
> Michal

Yes, my mistake. Should I send v2 or you already fix that?

Thank you,
Artem
_______________________________________________
Devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to