Re: [Spice-devel] [PATCH 10/14] Make CCD::load_api throw exception by its own

2016-09-01 Thread Christophe Fergeau
On Thu, Sep 01, 2016 at 10:58:39AM -0400, Frediano Ziglio wrote: > > > > s/by its own/on its own/ > > > > On Thu, Sep 01, 2016 at 01:00:52PM +0100, Frediano Ziglio wrote: > > > Avoid to use two way to handle errors. > > > > "two different ways" > > > > Acked-by: Christophe Fergeau > > > > >

Re: [Spice-devel] [PATCH 10/14] Make CCD::load_api throw exception by its own

2016-09-01 Thread Frediano Ziglio
> > s/by its own/on its own/ > > On Thu, Sep 01, 2016 at 01:00:52PM +0100, Frediano Ziglio wrote: > > Avoid to use two way to handle errors. > > "two different ways" > > Acked-by: Christophe Fergeau > Fixed these typos after sending v3. If I understood you removed the ack in a later comment

Re: [Spice-devel] [PATCH 10/14] Make CCD::load_api throw exception by its own

2016-09-01 Thread Christophe Fergeau
s/by its own/on its own/ On Thu, Sep 01, 2016 at 01:00:52PM +0100, Frediano Ziglio wrote: > Avoid to use two way to handle errors. "two different ways" Acked-by: Christophe Fergeau > > Signed-off-by: Frediano Ziglio > --- > vdagent/display_configuration.cpp | 19 +++ > vdage

[Spice-devel] [PATCH 10/14] Make CCD::load_api throw exception by its own

2016-09-01 Thread Frediano Ziglio
Avoid to use two way to handle errors. Signed-off-by: Frediano Ziglio --- vdagent/display_configuration.cpp | 19 +++ vdagent/display_configuration.h | 2 +- 2 files changed, 8 insertions(+), 13 deletions(-) diff --git a/vdagent/display_configuration.cpp b/vdagent/display_co