[PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Jeremy Huddleston
From: Choe Hwanjin choe.hwan...@gmail.com Because the focused Xic can be changed before sending sync reply. After focused Xic changed, the new Xic doesn't have NEED_SYNC_REPLY flag enabled, so libX11 doesn't send XIM_SYNC_REPLY packet. This patch adds sync reply flag to Xim and removes sync

Re: [PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Jeremy Huddleston
On Oct 11, 2011, at 18:34, Choe Hwanjin wrote: I've made my patch again against libX11 git master 22ba43d198dcca86c88eb15a56fc7d8fc47c422e. This patch already has changes that revert commit 1a1a42a3ca1dfaf42f1094936b71c140fc030fcb. I don't see any significant differences between your patch

Re: [PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Choe Hwanjin
I've made my patch again against libX11 git master 22ba43d198dcca86c88eb15a56fc7d8fc47c422e. This patch already has changes that revert commit 1a1a42a3ca1dfaf42f1094936b71c140fc030fcb. I don't see any significant differences between your patch and mine. And your patch have some problem. See