Re: [Qemu-devel] [PATCH 5/5] nbd/server: structurize option reply sending

2018-01-10 Thread Eric Blake
On 11/22/2017 04:02 PM, Eric Blake wrote: > On 11/22/2017 04:19 AM, Vladimir Sementsov-Ogievskiy wrote: >> Signed-off-by: Vladimir Sementsov-Ogievskiy >> --- >> nbd/server.c | 40 +--- >> 1 file changed, 13 insertions(+), 27 deletions(-) >> > > Nice diffstat;

Re: [Qemu-devel] [PATCH 5/5] nbd/server: structurize option reply sending

2017-11-22 Thread Eric Blake
On 11/22/2017 04:19 AM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > nbd/server.c | 40 +--- > 1 file changed, 13 insertions(+), 27 deletions(-) > Nice diffstat; shows that this one probably makes sense (even if it

[Qemu-devel] [PATCH 5/5] nbd/server: structurize option reply sending

2017-11-22 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- nbd/server.c | 40 +--- 1 file changed, 13 insertions(+), 27 deletions(-) diff --git a/nbd/server.c b/nbd/server.c index 79b937d88f..975fe8efe9 100644 --- a/nbd/server.c +++ b/nbd/server.c @@ -152,43 +152,29 @@ s