On Wed, 2016-10-12 at 06:55 -0400, Frediano Ziglio wrote:
> >
> >
> > Prepare for GObject port
>
> Nacked
>
> >
> > ---
> > server/inputs-channel.c | 68
> > ++---
> > 1 file changed, 36 insertions(+), 32 deletions(-)
> >
> > diff --git a/server/i
Hi
- Original Message -
> Hi,
>
> I plan to merge both uri parsers (one is in spice-session, one in spice-uri).
> These patches add more to tests to avoid regressions.
>
> v2 per Victor review splits IPv6 tests and checks for expected warnings
> v1: https://lists.freedesktop.org/archives
- Original Message -
> Hi
>
> - Original Message -
> > Hi,
> >
> > I plan to merge both uri parsers (one is in spice-session, one in
> > spice-uri).
> > These patches add more to tests to avoid regressions.
> >
> > v2 per Victor review splits IPv6 tests and checks for expected
---
tests/session.c | 157 +---
1 file changed, 105 insertions(+), 52 deletions(-)
diff --git a/tests/session.c b/tests/session.c
index 494e92f..09856b3 100644
--- a/tests/session.c
+++ b/tests/session.c
@@ -1,5 +1,16 @@
#include
+typedef st
Acked-by: Victor Toso
---
tests/session.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/tests/session.c b/tests/session.c
index 00076f2..a6276c2 100644
--- a/tests/session.c
+++ b/tests/session.c
@@ -24,6 +24,12 @@ static void test_session_uri(void)
{ NULL, "5930",
---
tests/session.c | 45 +
1 file changed, 41 insertions(+), 4 deletions(-)
diff --git a/tests/session.c b/tests/session.c
index a6276c2..74d32f2 100644
--- a/tests/session.c
+++ b/tests/session.c
@@ -8,47 +8,81 @@ static void test_session_uri(void)
---
tests/session.c | 126 +++-
1 file changed, 124 insertions(+), 2 deletions(-)
diff --git a/tests/session.c b/tests/session.c
index 74d32f2..494e92f 100644
--- a/tests/session.c
+++ b/tests/session.c
@@ -1,6 +1,127 @@
#include
-static voi
Hi,
I plan to merge both uri parsers (one is in spice-session, one in spice-uri).
These patches add more to tests to avoid regressions.
v2 per Victor review splits IPv6 tests and checks for expected warnings
v1: https://lists.freedesktop.org/archives/spice-devel/2016-May/029261.html
Thanks,
Pav
On Wed, 2016-10-12 at 10:13 -0400, Frediano Ziglio wrote:
> >
> > On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote:
> > > >
> > > >
> > > > When MainChannelClient was split to a separate file, the
> > > > responsibility
> > > > for incrementing this field was supposed to belong to the
>
>
> On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote:
> > >
> > >
> > > When MainChannelClient was split to a separate file, the
> > > responsibility
> > > for incrementing this field was supposed to belong to the
> > > MainChannel
> > > function (main_channel_connect_semi_seamless()), b
On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote:
> >
> >
> > When MainChannelClient was split to a separate file, the
> > responsibility
> > for incrementing this field was supposed to belong to the
> > MainChannel
> > function (main_channel_connect_semi_seamless()), but by mistake it
>
>
> When MainChannelClient was split to a separate file, the responsibility
> for incrementing this field was supposed to belong to the MainChannel
> function (main_channel_connect_semi_seamless()), but by mistake it was
> incremented both there and in the client function
> (main_channel_client_co
>
> Prepare for GObject port
Nacked
> ---
> server/inputs-channel.c | 68
> ++---
> 1 file changed, 36 insertions(+), 32 deletions(-)
>
> diff --git a/server/inputs-channel.c b/server/inputs-channel.c
> index 85ca155..dfb7ba6 100644
> --- a/server/i
>
> In preparation for converting RedChannel to GObject, switch to using
> RED_CHANNEL()-type macros for casting. For now they just do a regular
> cast, but it helps reduce the size of the GObject patch to make it
> easier to review.
> ---
> server/common-graphics-channel.c | 2 +-
> server/curs
Do not include red-worker.h if not necessary.
Signed-off-by: Frediano Ziglio
---
server/common-graphics-channel.h | 2 +-
server/cursor-channel.c | 1 +
server/cursor-channel.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
Beside this small patch, which basically move
> On Tue, 2016-10-11 at 09:55 -0400, Frediano Ziglio wrote:
> > >
> > > > > diff --git a/server/cursor-channel.h b/server/cursor-channel.h
> > > > > index a3ddaa3..8b3bc17 100644
> > > > > --- a/server/cursor-channel.h
> > > > > +++ b/server/cursor-channel.h
> > > > > @@ -19,6 +19,17 @@
> > > > >
16 matches
Mail list logo
