Re: [PATCH v3 7/9] winedbg: Define table for GDB qXfer command handlers.

[Rémi Bernon]

On 11/16/21 17:51, Jinoh Kang wrote:
> Replace the series of sscanf in packet_query with a dedicated qXfer
> request parser function and a lookup into a predefined handler table.
> Signed-off-by: Jinoh Kang
>
> programs/winedbg/gdbproxy.c | 218 ++++++++++++++++++++++++++----------
>   1 file changed, 160 insertions(+), 58 deletions(-)

To be honest I find the sscanf version much more readable, especially as 
gdb protocol is not always easy to follow, and the number of possible 
qxfer variants (3) doesn't really justify to use a dispatch table imho.

Is this change solving anything in particular?