On 2010/12/02 06:08:12, Shawn.P.Huang wrote: > On 2010/12/02 05:25:00, Daiki Ueno wrote: > > LGTM. ...
13 years, 4 months ago
(2010-12-02 07:36:34 UTC)
#3
On 2010/12/02 06:08:12, Shawn.P.Huang wrote:
> On 2010/12/02 05:25:00, Daiki Ueno wrote:
>
> LGTM. BTW, Do you think should check the return value of the scan function?
Yeah uploaded a new patch to check the errors. Thanks for the review.
Issue 3392041: Use ibus_m17n_scan_engine_name() to simplify the code.
(Closed)
Created 13 years, 4 months ago by Daiki Ueno
Modified 13 years, 4 months ago
Reviewers: shawn.p.huang_gmail.com, Peng
Base URL: git@github.com:ibus/ibus-m17n.git@master
Comments: 0