Re: [Qemu-devel] [PATCH v2 33/35] blockdev: Convert drive_new() to Error

2018-10-16 Thread Eric Blake
On 10/15/18 9:48 AM, Max Reitz wrote: On 15.10.18 13:53, Markus Armbruster wrote: Calling error_report() from within a function that takes an Error ** argument is suspicious. drive_new() calls error_report() even though it can run within drive_init_func(), which takes an Error ** argument. driv

Re: [Qemu-devel] [PATCH v2 33/35] blockdev: Convert drive_new() to Error

2018-10-15 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > On 15/10/2018 16:48, Max Reitz wrote: >> On 15.10.18 13:53, Markus Armbruster wrote: >>> Calling error_report() from within a function that takes an Error ** >>> argument is suspicious. drive_new() calls error_report() even though >>> it can run within drive_init

Re: [Qemu-devel] [PATCH v2 33/35] blockdev: Convert drive_new() to Error

2018-10-15 Thread Philippe Mathieu-Daudé
On 15/10/2018 16:48, Max Reitz wrote: > On 15.10.18 13:53, Markus Armbruster wrote: >> Calling error_report() from within a function that takes an Error ** >> argument is suspicious. drive_new() calls error_report() even though >> it can run within drive_init_func(), which takes an Error ** argume

Re: [Qemu-devel] [PATCH v2 33/35] blockdev: Convert drive_new() to Error

2018-10-15 Thread Max Reitz
On 15.10.18 13:53, Markus Armbruster wrote: > Calling error_report() from within a function that takes an Error ** > argument is suspicious. drive_new() calls error_report() even though > it can run within drive_init_func(), which takes an Error ** argument. > drive_init_func()'s caller main(), vi

[Qemu-devel] [PATCH v2 33/35] blockdev: Convert drive_new() to Error

2018-10-15 Thread Markus Armbruster
Calling error_report() from within a function that takes an Error ** argument is suspicious. drive_new() calls error_report() even though it can run within drive_init_func(), which takes an Error ** argument. drive_init_func()'s caller main(), via qemu_opts_foreach(), is fine with it, but clean it