[Spice-devel] 'fix a memory leak in qxl_screen_init'

2014-04-03 Thread longguang.yue
thanks uxa-memory-leak.patch Description: Binary data ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] [Spice-commits] 2 commits - gtk/spice-gtk-session.c gtk/spice-gtk-session.h gtk/spice-widget.c

2014-04-03 Thread longguang.yue
have you decided ? i do not get your code from git. where is the code you judge whether need to sync or not? according to your code, i think there are at least two times to sync lock status, first , your code of state-changed, the another is key_press_and_release. i think we just ensure t

Re: [Spice-devel] 0001-a patch for syncing keyboard lock status

2014-03-27 Thread longguang.yue
eyboard_lock_modifiers), display); > spice_channel_connect(channel); > sync_keyboard_lock_modifiers(display); > return; > > >However, I still need to look into the details a bit more to see if this is >really the right way to handle this particular bug. > >Jonathon > &

Re: [Spice-devel] 0001-a patch for syncing keyboard lock status

2014-03-27 Thread longguang.yue
2014-03-26 at 23:10 +0800, bigclouds wrote: > > >> From cdfcb3083825836e69f3e8d9ef18323117e43015 Mon Sep 17 00:00:00 2001 >> From: "longguang.yue" >> Date: Wed, 26 Mar 2014 22:28:55 +0800 >> Subject: [PATCH] there is only few oppotunity (one or two) to sync keyblo

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2014-03-26 Thread longguang.yue
please forgive my ignore. jwhite you are right, i post a new patch as you suggest. thanks At 2013-11-19 20:21:35,"Marian Krcmarik" wrote: >ping.. > >"bigclouds", >are you willing to finish the patch and address Jeremy's comment? It's imo >worthy to fix and It would be nice If you can f

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2014-03-26 Thread longguang.yue
please forgive my ignore. jwhite you are right, i post a new patch as you suggest. thanks At 2013-11-19 20:21:35,"Marian Krcmarik" wrote: >ping.. > >"bigclouds", >are you willing to finish the patch and address Jeremy's comment? It's imo >worthy to fix and It would be nice If you can fi