On 11/05/2018 09:59 PM, Juan Quintela wrote:
Fei Li <f...@suse.com> wrote:
Always call migrate_set_error() to set the error state without relying
on whether multifd_save_cleanup() succeeds. As the passed &local_err
is never used in multifd_save_cleanup(), remove it.
Error is not used, you are right.
But then just change the prototype to:
void multifd_save_cleanup(void);
??
With that change,
Reviewed-by: Juan Quintela <quint...@redhat.com>
Thanks for the review!
Have changed that function to be void, besides, correct the previous
erroneous
function name in the patch title:
s/migratr_set_error/multifd_save_cleanup/g.
