Re: [Spice-devel] gitlab

2016-09-30 Thread Fabiano Fidêncio
On Fri, Sep 30, 2016 at 1:48 PM, Victor Toso wrote: > Hi again! > > On Wed, Jun 22, 2016 at 04:48:39PM +0200, Victor Toso wrote: >> Hi all, >> >> I created a spice group in gitlab [0] mirroring the repository from >> freedesktop which should be updated every hour. >> >> [0] https://gitlab.com/grou

[Spice-devel] [spice-gtk v1 1/2] tests: fix some compiler warnings

2016-09-30 Thread victortoso
From: Victor Toso file-transfer.c: In function ‘transfer_on_init_async_cb_agent_cancel’: file-transfer.c:316:19: warning: unused variable ‘cancellable’ [-Wunused-variable] GCancellable *cancellable; ^~~ pipe.c: In function ‘test_pipe_readblock’: pipe.c:91:12: warnin

[Spice-devel] [spice-gtk v1 0/2] tests: fix warnings and set name prefix

2016-09-30 Thread victortoso
From: Victor Toso Hi, First patch fixes a few harmless compiler warnings. I put it in a single patch as I did not see a strong reason to split. Second patch sets the test binaries to have 'test-' as prefix. Make check result: PASS: test-util PASS: test-coroutine PASS: test-session PASS: test-s

[Spice-devel] [spice-gtk v1 2/2] tests: set binaries to have 'test' name prefix

2016-09-30 Thread victortoso
From: Victor Toso To follow test-spice-uri and test-file-transfer standard. --- tests/Makefile.am | 26 +- tests/usb-acl-helper.c | 2 +- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/tests/Makefile.am b/tests/Makefile.am index 6d9cfeb..3a0188d 10

[Spice-devel] [PATCH spice-server] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Split lines with more than 100 characters Signed-off-by: Frediano Ziglio --- server/cursor-channel-client.c| 6 -- server/cursor-channel-client.h| 15 ++- server/dcc.c | 6 -- server/dcc.h | 15 ++- server/dummy-

[Spice-devel] [PATCH spice-server v2 7/7] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Remove old arrays if present. Signed-off-by: Frediano Ziglio --- server/red-channel-client.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/server/red-channel-client.c b/server/red-channel-client.c index 5b209d8..546d58d 100644 --- a/server/red-channel-client.c +++ b/server/red-channel-cl

[Spice-devel] [PATCH spice-server v2 3/7] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Reduce diff. Remove some style changes. Use declarations instead of moving functions. Signed-off-by: Frediano Ziglio --- server/red-channel-client.c | 171 +++--- server/red-channel.h | 2 +- server/reds.h | 1 - server/s

[Spice-devel] [PATCH spice-server v2 6/7] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Remove base from SmartCardChannelClientPrivate. Public have already the base. Signed-off-by: Frediano Ziglio --- server/smartcard-channel-client.c | 1 - 1 file changed, 1 deletion(-) diff --git a/server/smartcard-channel-client.c b/server/smartcard-channel-client.c index afd0899..beda2fc 1006

[Spice-devel] [PATCH spice-server v2 5/7] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Style changes. New code should respect style. Signed-off-by: Frediano Ziglio --- server/smartcard-channel-client.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/server/smartcard-channel-client.c b/server/smartcard-channel-client.c index 76b8eed..afd0899 100644 --- a/

[Spice-devel] [PATCH spice-server v2 1/7] Split RedCharDeviceSmartcard and SmartCardChannelClient

2016-09-30 Thread Frediano Ziglio
--- server/Makefile.am| 2 + server/smartcard-channel-client.c | 300 +++ server/smartcard-channel-client.h | 96 ++ server/smartcard.c| 360 ++ server/smartcard.h| 21 +++ 5

[Spice-devel] [PATCH spice-server v2 4/7] fixup! Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Initialise id field in DisplayChannelClient. This line was removed. Signed-off-by: Frediano Ziglio --- server/dcc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/server/dcc.c b/server/dcc.c index bb85861..08866c9 100644 --- a/server/dcc.c +++ b/server/dcc.c @@ -521,6 +521,7 @@ DisplayChann

[Spice-devel] [PATCH spice-server v2 2/7] Convert RedChannelClient hierarchy to GObject

2016-09-30 Thread Frediano Ziglio
Convert the RedChannelClient heirarchy into GObjects. Since the existing constructors could fail and return NULL, I inherited the base channel client from GInitable, which introduces a dependency on gio. When using private structs with GObject, there's a maximum size of (I think) 64k, which was ex

[Spice-devel] [PATCH spice-server v2 0/7] Update proposed refactory merges

2016-09-30 Thread Frediano Ziglio
This patchset contains my split of the patches and some proposed changes to the second big patch. Frediano Ziglio (7): Split RedCharDeviceSmartcard and SmartCardChannelClient Convert RedChannelClient hierarchy to GObject fixup! Convert RedChannelClient hierarchy to GObject fixup! Convert R

Re: [Spice-devel] [PATCH] Cache field to make code easier

2016-09-30 Thread Victor Toso
Hi, On Thu, Sep 29, 2016 at 09:46:04AM +0100, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio why not? :) Acked-by: Victor Toso > --- > server/display-channel.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/server/display-channel.c b/server/disp

Re: [Spice-devel] gitlab

2016-09-30 Thread Victor Toso
Hi again! On Wed, Jun 22, 2016 at 04:48:39PM +0200, Victor Toso wrote: > Hi all, > > I created a spice group in gitlab [0] mirroring the repository from > freedesktop which should be updated every hour. > > [0] https://gitlab.com/groups/spice https://gitlab.com/spice works too. > > But I would l

Re: [Spice-devel] [PATCH v2 0/4] Protocol file syntax documentation

2016-09-30 Thread Pavel Grunt
On Fri, 2016-09-30 at 07:02 -0400, Frediano Ziglio wrote: > > > > Hi Frediano! > > > > Thank you for looking into it > > > > imho it is confusing to document spice-protocol which is defined > > in > > spice-common in spice server git repository. > > > > ^ that sentence is strange > > > > > A

Re: [Spice-devel] [PATCH v2 0/4] Protocol file syntax documentation

2016-09-30 Thread Victor Toso
Hi, On Fri, Sep 30, 2016 at 07:08:55AM -0400, Frediano Ziglio wrote: > > > > Hi, > > > > On Fri, Sep 30, 2016 at 08:44:53AM +0200, Pavel Grunt wrote: > > > Hi Frediano! > > > > > > Thank you for looking into it > > > > > > imho it is confusing to document spice-protocol which is defined in > > >

Re: [Spice-devel] [PATCH v2 0/4] Protocol file syntax documentation

2016-09-30 Thread Frediano Ziglio
> > Hi, > > On Fri, Sep 30, 2016 at 08:44:53AM +0200, Pavel Grunt wrote: > > Hi Frediano! > > > > Thank you for looking into it > > > > imho it is confusing to document spice-protocol which is defined in > > spice-common in spice server git repository. > > > > ^ that sentence is strange > > > > I

Re: [Spice-devel] [PATCH v2 0/4] Protocol file syntax documentation

2016-09-30 Thread Victor Toso
Hi, On Fri, Sep 30, 2016 at 08:44:53AM +0200, Pavel Grunt wrote: > Hi Frediano! > > Thank you for looking into it > > imho it is confusing to document spice-protocol which is defined in > spice-common in spice server git repository. > > ^ that sentence is strange > > I know the documentation files

Re: [Spice-devel] [PATCH v2 0/4] Protocol file syntax documentation

2016-09-30 Thread Frediano Ziglio
> > Hi Frediano! > > Thank you for looking into it > > imho it is confusing to document spice-protocol which is defined in > spice-common in spice server git repository. > > ^ that sentence is strange > All protocol files (spice.proto and python code) are in spice-common, not in spice-protoco

Re: [Spice-devel] [PATCH 2/2] SPICE port basic implementation

2016-09-30 Thread Pavel Grunt
Hi Oliver, I tested it (in FF) and it works nicely, I just have minor comments: in the console: SPICE port com.spice.fc.1 event data: Uint8Array { 0=0} SPICE port com.spice.fc.1 message text: CHILI SPICE port com.spice.fc.1 event data: Uint8Array { 0=1} 1) what is the Uint8Array - it has just one