Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-16 Thread Christophe de Dinechin
Sorry for cutting out a lot of context, but this email appeared as HTML in my inbox. > On 16 Nov 2017, at 14:34, Frediano Ziglio wrote: > > Let me clarify one point. I agree the options interface lacks some feature. > Changing > them dynamically will be necessary. > On the other hand the inte

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-16 Thread Frediano Ziglio
> > On 15 Nov 2017, at 13:55, Frediano Ziglio < fzig...@redhat.com > wrote: > > > > From: Christophe de Dinechin < dinec...@redhat.com > > > > > > > Several functional objectives: > > > > > > 1. Be able to share some common options, e.g. fps > > > > > > 2. Prepare for the possibility to u

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-16 Thread Christophe de Dinechin
> On 15 Nov 2017, at 13:55, Frediano Ziglio wrote: > >> >> From: Christophe de Dinechin >> >> Several functional objectives: >> >> 1. Be able to share some common options, e.g. fps >> 2. Prepare for the possibility to update options on the fly >> >> Also get rid of the null-terminated C++

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-15 Thread Frediano Ziglio
> > From: Christophe de Dinechin > > Several functional objectives: > > 1. Be able to share some common options, e.g. fps > 2. Prepare for the possibility to update options on the fly > > Also get rid of the null-terminated C++ vector > > Signed-off-by: Christophe de Dinechin > --- > includ

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-15 Thread Christophe de Dinechin
> On 14 Nov 2017, at 16:53, Christophe de Dinechin > wrote: > > > >> On 14 Nov 2017, at 16:41, Frediano Ziglio > > wrote: >> >>> >>> From: Christophe de Dinechin >> > >>> >>> Several functional objectives: >>> >>> 1. Be able to share

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-14 Thread Christophe de Dinechin
> On 14 Nov 2017, at 16:41, Frediano Ziglio wrote: > >> >> From: Christophe de Dinechin > > >> >> Several functional objectives: >> >> 1. Be able to share some common options, e.g. fps >> 2. Prepare for the possibility to update options on the fly >> >> Also get

Re: [Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-14 Thread Frediano Ziglio
> > From: Christophe de Dinechin > > Several functional objectives: > > 1. Be able to share some common options, e.g. fps > 2. Prepare for the possibility to update options on the fly > > Also get rid of the null-terminated C++ vector > > Signed-off-by: Christophe de Dinechin This change is

[Spice-devel] [PATCH spice-streaming-agent 2/3] Rework option handling

2017-11-14 Thread Christophe de Dinechin
From: Christophe de Dinechin Several functional objectives: 1. Be able to share some common options, e.g. fps 2. Prepare for the possibility to update options on the fly Also get rid of the null-terminated C++ vector Signed-off-by: Christophe de Dinechin --- include/spice-streaming-agent/plu