[PATCH 4/4] msctf: Implement stubbed out TF_CreateLangBarMgr [Try 2]

Aric Stewart aric at codeweavers.com
Mon Feb 15 08:34:04 CST 2010


Hi,

  I am really happy that you are helping with the msctf stuff I started. 
Very cool!

   2 quick things.

1) typedef struct tagACLMulti,   don't call it tagACLMulti.  I am sure 
this is from copying from threadmgr.c where it is a copy/paste thing 
from when I started that framework.  It has always bugged me (though not 
enough to submit a patch just to change it), but it would be nice not to 
see it propagated even more.


2)  you add pITfLangBarMgr to the ActivatedTextService, but dont do 
anything with it.  Does the Activated TextService need to have a 
reference to the ITfLangBarMgr? My quick read though does not seem to 
reveal that. When should it get set and initialized and where would it 
be used?

thanks!
-aric



More information about the wine-devel mailing list