On Thu, 2016-04-14 at 16:50 -0500, Jonathon Jongsma wrote:
> From: Christophe Fergeau
>
> RedCharDevice implementation had to callback into reds.c in order to let
> it know a char device was being destroyed. Now that RedCharDevice is a
> gobject, a weak reference can be used instead allowing to r
On Thu, 2016-04-14 at 16:50 -0500, Jonathon Jongsma wrote:
> From: Christophe Fergeau
>
> There was an extra RedCharDeviceMsgToClientItem type whose only
> purpose was to manage a linked list of items to send. GQueue has the
> same purpose as this type in addition to being generic. As the length
On Fri, 2016-04-15 at 06:18 -0400, Frediano Ziglio wrote:
> >
> > From: Christophe Fergeau
> >
> > It's more natural to do things this way with glib, and this allows to
> > remove some coupling between Reds and RedCharDeviceVDIPort. Before this
> > commit, the RedCharDeviceVDIPort has to provide
On Thu, 2016-04-14 at 17:56 +0100, Frediano Ziglio wrote:
> LZ image decompression was broken for 16 bpp:
> - stride was computed not computed correctly (as width*4). This caused
> also a buffer underflow;
FYI: extra "computed" here. Remove the first one.
> - stride in pixman is always multipl
No need to re-implement refcounting in this subclass. However, I needed
to add a new 'dcc' member to UpgradeItem to be able to unref properly.
---
Changes:
- take a reference on dcc in UpgradeItem
server/dcc.c | 18 +-
server/display-channel.c | 4 +---
server/displ
>
> Since the base class now implements refcounting, there's no need to
> re-invent it here.
> ---
> server/dcc-send.c| 4 ++--
> server/dcc.c | 19 +--
> server/dcc.h | 3 +--
> server/display-channel.c | 2 +-
> 4 files changed, 9 insertions(+)
On Fri, 2016-04-15 at 05:58 -0400, Frediano Ziglio wrote:
> >
> > Since the base class now implements refcounting, there's no need to
> > re-invent it here.
> > ---
> > server/dcc-send.c| 4 ++--
> > server/dcc.c | 19 +--
> > server/dcc.h | 3 +--
We may get stream reports after the stream has been closed, and even, in
extreme cases, after a new stream has started reusing the same stream
id. Neither case should cause the server to crash.
So restore dcc_handle_stream_report() to how it was before being moved
to dcc.c.
Signed-off-by: Francois
Hi, I like the approach a lot. It seems quite clear and nice to me,
but ...
On So, 2016-03-12 at 15:32 +0100, Victor Toso wrote:
> Hi! I've rebased this series and pushed to my remote branch in
> freedesktop [0] [1]. I'll try to clarify the ideia for working on this
> and if it does not get any po
Will it be possible to print all existing subdomains? Or would it be
hard to add?
On So, 2016-03-12 at 15:32 +0100, Victor Toso wrote:
> "audio" : spice-audio.c spice-gstaudio.c spice-pulse.c
> "base" : channel-base.c
> "channel": spice-channel.c
> "coroutine" : coroutine_gthread.c
Hi,
On Thu, 2016-04-14 at 17:56 +0100, Frediano Ziglio wrote:
> LZ image decompression was broken for 16 bpp:
> - stride was computed not computed correctly (as width*4). This
> caused
> also a buffer underflow;
> - stride in pixman is always multiple of 4 bytes (so for 16 bpp is
> ALIGN(width
This version contains style change:
- remove goto/label;
- remove useless default switch.
Takao Fujiwara (3):
Send Zenkaku_Hankaku key in JP keyboard
Send key release event for some keys in JP keyboard
Send Hangul key in KR keyboard
src/spice-widget.c | 99 +
From: Takao Fujiwara
With the previous fix, WM_KEYDOWN of Alt+Zenkaku_Hankaku (VK_KANJI) can
be sent with spice-gtk but Alt+Zenkaku_Hankaku does not send the WM_KEYUP
event in Windows and it caused Linux desktop freeze with unlimited key
press events.
The proposed fix is to send the key release
From: Takao Fujiwara
Korean keyboard assigns Hangul key on the position of Right Alt and
Left Alt and Hangul keys have the different scancodes but MapVirtualKey()
returned the same scancode and could not use Hangul key on Linux desktop.
The fix is to send the right scancode of VK_HANGUL.
---
sr
From: Takao Fujiwara
Zenkaku_Hankaku key has the different virtual-key codes between WM_KEYDOWN
and WM_KEYUP and MapVirtualKey() cannot get the scancode from virtual-key
code of WM_KEYDOWN (VK_DBE_DBCSCHAR) and spice-gtk didn't send the key
press events and caused the desktop freeze with unlimite
On Thu, 2016-04-14 at 16:50 -0500, Jonathon Jongsma wrote:
> From: Christophe Fergeau
>
> A user-defined callback is called when the refcount drops to 0.
>
> Reference counting is manually coded for several classes deriving
> from
> PipeItem, so this change will help to share this code, and allo
> From: Christophe Fergeau
>
> We can get 'reds' from the RedCharDevice accessible from the
> SpiceCharDeviceInstance passed as an argument.
> ---
> server/reds.c | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/server/reds.c b/server/reds.c
> index 7e5626a..3c95
>
> From: Christophe Fergeau
>
> It's more natural to do things this way with glib, and this allows to
> remove some coupling between Reds and RedCharDeviceVDIPort. Before this
> commit, the RedCharDeviceVDIPort has to provide a on_free_self_token()
> because Reds needs to know about it. With a
>
> From: Christophe Fergeau
>
> Now that client messages are always RedPipeItem, we don't need virtual
> functions to know how to ref/unref them.
> ---
> server/char-device.c | 26 --
> server/char-device.h | 6 ++
> server/reds.c| 14 --
> serv
>
> From: Christophe Fergeau
>
> Now that all derived classes use a type deriving from PipeItem for their
> RedCharDeviceMsgToClient, we can make this explicit in the
> RedCharDeviceClass vfuncs, and remove the RedCharDeviceMsgToClient
> typedef.
> ---
> server/char-device.c | 18 +-
>
> ---
> server/dcc.c | 25 ++---
> 1 file changed, 10 insertions(+), 15 deletions(-)
>
> diff --git a/server/dcc.c b/server/dcc.c
> index b58a86a..1cd9f1a 100644
> --- a/server/dcc.c
> +++ b/server/dcc.c
> @@ -510,6 +510,12 @@ void dcc_stream_agent_clip(DisplayChannelClient
>
> No need to re-implement refcounting in this subclass. However, I needed
> to add a new 'dcc' member to UpgradeItem to be able to unref properly.
> ---
> server/dcc.c | 18 +-
> server/display-channel.c | 4 +---
> server/display-channel.h | 2 +-
> server/stream.
>
> PipeItem already implements refcounting. Use it.
> ---
> server/dcc.c | 10 +++---
> server/display-channel.c | 4 +---
> server/stream.c | 30 +++---
> server/stream.h | 6 +-
> 4 files changed, 20 insertions(+), 30 deletions(-)
>
> Since PipeItem already implements refcounting, there's no need to
> re-implement it here.
> ---
> server/dcc.c | 26 +-
> server/dcc.h | 3 ---
> server/display-channel.c | 6 ++
> 3 files changed, 11 insertions(+), 24 deletions(-)
>
> di
>
> Since the base class now implements refcounting, there's no need to
> re-invent it here.
> ---
> server/dcc-send.c| 4 ++--
> server/dcc.c | 19 +--
> server/dcc.h | 3 +--
> server/display-channel.c | 2 +-
> 4 files changed, 9 insertions(+)
>
> From: Christophe Fergeau
>
> This allows to reuse pipe_item_{ref, unref} rather than
> reimplementing them in spicevmc.c
> ---
> server/spicevmc.c | 27 ++-
> 1 file changed, 6 insertions(+), 21 deletions(-)
>
> diff --git a/server/spicevmc.c b/server/spicevmc.c
> i
>
> From: Christophe Fergeau
>
> This allows to reuse pipe_item_{ref, unref} rather than
> reimplementing them in smartcard.c
> ---
> server/smartcard.c | 35 ---
> 1 file changed, 12 insertions(+), 23 deletions(-)
>
> diff --git a/server/smartcard.c b/server/sm
>
> From: Christophe Fergeau
>
> Since PipeItem is already refcounted, this allows to remove various
> layers of ref/unref helpers from reds.c, and use the generic
> pipe_item_{ref, unref} instead.
> ---
> server/reds.c | 74
> ++-
> 1 fi
I copy the same note on https://bugzilla.redhat.com/show_bug.cgi?id=1311858 for
reference:
The ImmDisableIME patch is not necessary only if you have the grab of the
keyboard.
This as the windows hook which is installed when you have the keyboard grabbed
receives the messages before IME change t
I confirmed v3 works fine.
On 04/14/16 19:52, Frediano Ziglio-san wrote:
> From: Takao Fujiwara
>
> Zenkaku_Hankaku key has the different virtual-key codes between WM_KEYDOWN
> and WM_KEYUP and MapVirtualKey() cannot get the scancode from virtual-key
> code of WM_KEYDOWN (VK_DBE_DBCSCHAR) and sp
It seems my debug log of libgdk-win32-2.0-0.dll caused a conflicted IO and SEGV.
After I deleted my debug log, I no longer encounter SEGV so now
ImmDisableIME.patch is not necessary.
Thanks,
Fujiwara
On 04/12/16 20:08, Takao Fujiwara-san wrote:
I mean ImmDisableIME.patch should be used as a wo
Korean keyboard assigns Hangul key on the position of Right Alt and
Left Alt and Hangul keys have the different scancodes but MapVirtualKey()
returned the same scancode and could not use Hangul key on Linux desktop.
The fix is to send the right scancode of VK_HANGUL.
---
src/spice-widget.c | 10 +
With the previous fix, WM_KEYDOWN of Alt+Zenkaku_Hankaku (VK_KANJI) can
be sent with spice-gtk but Alt+Zenkaku_Hankaku does not send the WM_KEYUP
event in Windows and it caused Linux desktop freeze with unlimited key
press events.
The proposed fix is to send the key release event in spice-gtk.
VK
Acked-by: Christophe Fergeau
On Thu, Apr 14, 2016 at 06:01:24PM +0200, Pavel Grunt wrote:
> ---
> src/channel-main.c | 5 +
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/src/channel-main.c b/src/channel-main.c
> index a7770fa..604f333 100644
> --- a/src/channel-main.c
>
Acked-by: Christophe Fergeau
On Thu, Apr 14, 2016 at 06:01:23PM +0200, Pavel Grunt wrote:
> Avoid repeating the same code and having a SpiceSession variable
> defined when is not really needed.
> ---
> src/channel-main.c | 17 +
> 1 file changed, 9 insertions(+), 8 deletions(-)
>
Hi guys,
I have used the spice long time, now I want to compare the spice
protocol to others, such as ICA、PCoIP,but I have not a tool or method to
test the performance of spice, could you help me?
Thanks in advance.
Ji lie quan
___
36 matches
Mail list logo