<div dir="ltr"><div><div><div><div>Hi,<br></div>Sorry for all the trouble.<br><br>Sho 
on irc asked me that he want to test the ibus 1.5 support easier so I 
get it merged for kde4. Unfortunately there's no further 
kdeplamsa-addons for kde4 and I didn't finish this before 4.14, so it's
 also either leave it broken in kde4 or add a usable version.<br><br></div>This
 change doesn't break old things but fix a never supported version for 
ibus, and the old support is also kept in a different directory. So 
there is no actual code deletion, and doesn't change the behavior for 
old version of ibus 1.4.<br><br></div><div>The line number looks huge for following reason:</div>1. The ibus upstream changed quite a 
lot and made that ibus support must parse a gtk key string. It's not 
possible to use gtk for just a keystring parse so I was forced to copy 
some gdk key string parsing code here. In this part of code there is a hardcoded key symbol to string table which is unfortunately huge.<br></div><div>2. The old supporting code is also kept but moved, and new code is also based on those code.<br></div><div><br></div><div>For i18n, there is no new string should be introduced.<br></div><div><br></div><div>In summary, this change doesn't break on old version, and introduce a fix of supoort for new version which is never supported before.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 3:52 PM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Weng, I have been just told you just merged your<br>
  origin/xuetianweng/kimpanel-ibus-1.5<br>
into<br>
  KDE/4.14<br>
in the kdeplasma-addons repo<br>
<br>
That brings<br>
  27 files changed, 10677 insertions(+), 1424 deletions(-)<br>
<br>
Has this been discussed anywhere? Because honestly this seems quite a big<br>
change for a stable branch.<br>
<br>
Cheers,<br>
  Albert<br>
</blockquote></div><br></div>