13 Mar
2019
13 Mar
'19
10:29 a.m.
On Wed, 2019-03-13 at 10:12 +0100, Peter Krempa wrote:
Actually I don't see the reason for anything in this patch. Each function is called exactly once, so splitting it out does not make much sense without any further justification.
The need to reset data->ret to 0 is not enough of a justification.
Better code organization: instead of having a bunch of initialization and cleanup steps sprinkled throughout main(), we give them their own functions and keep main() simple, clean and focused. -- Andrea Bolognani / Red Hat / Virtualization