Re: [Spice-devel] [PATCH spice-streaming-agent 1/6] Be more restrictive about error handling

2018-02-19 Thread Uri Lublin
On 02/19/2018 05:52 PM, Frediano Ziglio wrote: There's no much point in ignoring the error if these errors cause the communication to be out of sync. Ignoring them just change the state to indetermidate. Hi Frediano, There is a typo in the last word. I would be more specific, mentioning those

Re: [Spice-devel] [PATCH spice-streaming-agent 1/6] Be more restrictive about error handling

2018-02-19 Thread Frediano Ziglio
> > On Mon, 2018-02-19 at 15:52 +, Frediano Ziglio wrote: > > There's no much point in ignoring the error if these errors cause the > > communication to be out of sync. > > Ignoring them just change the state to indetermidate. > > > > Signed-off-by: Frediano Ziglio > > --- > > src/spice-str

Re: [Spice-devel] [PATCH spice-streaming-agent 1/6] Be more restrictive about error handling

2018-02-19 Thread Lukáš Hrázký
On Mon, 2018-02-19 at 15:52 +, Frediano Ziglio wrote: > There's no much point in ignoring the error if these errors cause the > communication to be out of sync. > Ignoring them just change the state to indetermidate. > > Signed-off-by: Frediano Ziglio > --- > src/spice-streaming-agent.cpp |

[Spice-devel] [PATCH spice-streaming-agent 1/6] Be more restrictive about error handling

2018-02-19 Thread Frediano Ziglio
There's no much point in ignoring the error if these errors cause the communication to be out of sync. Ignoring them just change the state to indetermidate. Signed-off-by: Frediano Ziglio --- src/spice-streaming-agent.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/