> 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 -- devel@xxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx