Re: Re: patch:[update0703] enchant_dict_hyphenation_chenxiajian

Great.
btw, make sure to commit the patches to your abiword branch and your
enchant branch as well.

Regards,
Pradeeban.

On Mon, Jul 4, 2011 at 8:18 PM, chenxiajian1985
<chenxiajian1985@gmail.com> wrote:
> hi, Pradeeban.
>
> Sorry for that.
> the following codes is just for test (or debug). Because I want to watch some result while running dll.
> and the dll is not easy for debug.  I can watch output information in Trace or debug window
> "c://1.txt" is just for output information.
>  //std::ofstream myfile("c://1.txt",ios::app);
>
>
> I have removed the codes in my project. After some modification, I will send you the patch.
> another modification: I have changed all the path to relative path. thanks for your suggestion.
>
>
>
> chen xiajian
>
>
>
>
> 发件人： Kathiravelu Pradeeban
> 发送时间： 2011-07-04  19:00:46
> 收件人： chenxiajian1985
> 抄送： abiword-dev
> 主题： Re: patch:[update0703] enchant_dict_hyphenation_chenxiajian
>
> Chen,
> In src/ispell/Hyphenator.cpp, you are building a pattern from the file provided.
> +   std::ofstream myfile("c://1.txt",ios::app);
> +   myfile<<"result0£º"<<result<<endl;
> +   myfile.close();
> I can see some weird characters, above.
> Make sure the relative path is given for the files. Hope you are not
> hard-coding the file location as above to a fixed absolute location.
> What happens if the file is missing? Is there any performance impact?
> Regards,
> Pradeeban.
> 2011/7/3 chenxiajian1985 <chenxiajian1985@gmail.com>:
>> Hi
>> the attachment is the newest patch file that I make some changes about my last patch. Thanks.
>>
>> the changes:
>> 1 in my hyphenation branch, I add some code to call the function from enchant
>>   and some comments in lineBreaker.h(cpp) and textRun.h(cpp)
>> 2 some tests in hyphenation
>>
>> todo:
>> 1 Fully support hyphenation in Abiword
>> 2 Support more language besides En_us
>> 3 Some tests in Linux(Unix)
>>
>>
>> Best Regards~
>> chenxiajian
> --
> Kathiravelu Pradeeban.
> Software Engineer.
> WSO2 Inc.
> Blog: [Llovizna] http://kkpradeeban.blogspot.com/
>

-- 
Kathiravelu Pradeeban.
Software Engineer.
WSO2 Inc.
Blog: [Llovizna] http://kkpradeeban.blogspot.com/