Philippe Mathieu-Daudé <[email protected]> writes: > On 9/1/25 20:45, Alex Bennée wrote: >> Pierrick Bouvier <[email protected]> writes: >> >>> On 1/9/25 09:06, Alex Bennée wrote: >>>> This started as a clean-up to properly pass a Error handler to the >>>> gdbserver_start so we could do the right thing for command line and >>>> HMP invocations. >>>> Now that we have cleaned up foreach_device_config_or_exit() in >>>> earlier >>>> patches we can further simplify by it by passing &error_fatal instead >>>> of checking the return value. Having a return value is still useful >>>> for HMP though so tweak the return to use a simple bool instead. >>>> Signed-off-by: Alex Bennée <[email protected]> >>>> Acked-by: Ilya Leoshkevich <[email protected]> >>>> >>> >>> <snip> >>> >>>> -static int gdbserver_open_port(int port) >>>> +static int gdbserver_open_port(int port, Error **errp) >>> >>> Did you mean: >>> static bool gdbserver_open_port...? >> yes, yes I did. Will fix. > > This is returning a socket file descriptor, why bool?
Doh - misread - I thought this was gdbserver_start... let me check. > >> >>> >>> With that, >>> Reviewed-by: Pierrick Bouvier <[email protected]> >> -- Alex Bennée Virtualisation Tech Lead @ Linaro
