[immodule-qt] New unified patch has been released as stable

[Daisuke Kameda]

On Saturday 04 September 2004 09:00, YamaKen wrote:
> Hi all,
>
> I've released new unified patch as stable. Distributors, please
> try it. But since some APIs and specifications are still
> altering in development, be careful about version mismatch
> between the patch and optional plugins. See "immodule plugins
> for unified patches" in the download page to select proper one.
>
> Try qt-x11-immodule-unified-qt3.3.3-20040819.diff.gz from the
> download page.
>
> http://immodule-qt.freedesktop.org/Software/ImmoduleQtDownload

Thank you for your work, YamaKen.

I have read the patch. The patch is sufficiently stable.
But some things about which we should discuss also remain.



- The default input method have to be able to be set up by qtconfig.

I agree with the present solution, which is using QT_IM_MODULE and 
QT_IM_SWITCHER. However, we should also leave the method of 
setting up by qtconfig. There are two reasons in it.

First reason is that not all users can set up the default input method 
using the setting tool of distribution. Second reason is that environment 
variable is difficult to understand for the beginners of desktop.

So, I propose introducing the check box written "set up default input 
method" into qtconfig. And if the check box is checked, setting up 
DefaultInputMethod and DefaultInputMethodSwitcher in qtrc is enabled. 
Moreover, the setup of qtrc overwrite an environment variable.



- It will be better to include QXIMInputContext and QMultiInputContext in 
libqt.

Now, all sublass of QInputContext corresponding to input method is 
served as plugin. Therefore, XIM and imsw-multi may be unable 
to be used when the package of plugin is another and not installed.

XIM(QXIMInputContext) and imsw-multi(QMultiInputContext)
is default in case environment variable isn't setted. So, 

As mentioned above, I consider that it is better to include at least 
these in libqt. And, it will be better to display an error message, 
if the instance of QInputContext was not generated.



- The files in $QTDIR/src/input/ should be moved to $QTDIR/src/inputmethods/.

The code about input method exists in two places, $QTDIR/src/input/ and 
$QTDIR/src/inputmethods/. I think that this is difficult to understand.

We should choose either. Considering compatibility with Qt/Embedded, 
I think it good to choose $QTDIR/src/inputmethods. 



What do you these my opinion?


Regards,
-- 
Daisuke Kameda <kaminmat at cc.rim.or.jp>