13 Jan
2022
13 Jan
'22
1:45 p.m.
On Thu, Jan 13, 2022 at 06:47:26PM +0530, Ani Sinha wrote:
I preferred your previous style to this one.
Below is cleaner IMO: we don't repeat code, and the flow is much clearer.
If you look at other functions for example qemuBuildVirtioDevProps() etc they follow the previous style. I think it's fine to return NULL from multiple points. In any case if the maintainers are fine with it , I'm ok. Style is trivial.
Sure, either is fine with me too, if needed. regards john