On Sun, May 15, 2016 at 10:27:13PM -0400, David T. Lewis wrote:
> 
> If I understand correctly, here is what we need to do:
> 
> 1) Apply the changes of Mr Hachisuka to the VM.
>  - The diff is in the earlier email from Yoshiki:
>       http://lists.squeakfoundation.org/pipermail/squeak-dev/2016-May/189599.html
>       http://www2.asu.ac.jp/hachi/v3/scratch14ime.html
>  - Test updated VM to make sure the changes do not cause problems. Make sure
>    it works without problems for Squeak trunk and for Scratch.
>  - Apply the update in SVN for both oscog and trunk. I can help here.
>  - Test the VM. Tim, I need help on this. I do not currently have a working
>    Cog build environment (Unbuntu/autotools/gcc version issues). Can you
>    apply the patch from Yoshiki's email and see if your resulting Cog VM
>    continues to work for you as expected on Pi with Scratch?
>  - Yoshiki, we should give credit to Mr Hachisuka for his contribution. Can
>    you please give his full name for the commit notice? I assume the patch
>    is MIT licensed.
> 

Hi Yoshiki,

I think that I now understand the VM changes for sqUnixX11.c, and I have applied
the changes of Mr Hachisuka and committed them to the SVN trunk for interpreter
VM. If these changes are correct, then I will also make a patch and commit them
to the oscog branch for Cog/Spur VMs.

I arranged the code such that the multibyte recordPendingKeys() logic is used
if and only if the -compositioninput option is set on the command line or the
equivalent SQUEAK_COMPOSITIONINPUT environment variable is set. I believe that
this is the intended usage, but please let me know if I did not understand.

If you are able to compile an interpreter VM (http://wiki.squeak.org/squeak/6354),
then please let me know if these changes work as you expect. If you cannot easily
make an interpreter VM, then I can commit the changes to the Cog/Spur branch,
but I would prefer to test them first if possible.

I also found that many of the module command line options are not being passed
correctly from the VM to the modules. The -compositionsinput option was one
of these, so I fixed it in this update (and I will check if for Cog/Spur when
we apply the patch there). But there are others that do not work, so if you
find other VM options for multilingual support that do not work as expected,
this may be the reason.

Dave