Code review comment for lp:~nmarshall23/do-plugins/spellcheck-plugin

Revision history for this message
Nicholas Marshall (nmarshall23) wrote :

>How many dictionaries are supported?
I don't know, you can install support for individual Dictionaries, I will
check.

>Making an Item out of each dictionary, and using those as dynamic modifier
items?
I like this idea and will start working on it.. However if there are a lot
of Dictionaries installed by default, then I see no other option but to
write a config widget.

About libenchantnet, I didn't write these bindings. They were originally
intended for use on MS windows. I found them while completing writing my
own, it seemed strange that there wasn't mono bindings for some kind of
spell check.. So I am sure they are quirky. All I did was change one line so
that it loaded the right shared library. And then run the nunit tests to see
if that was all that was required.

>You seem to have done a very good job integrating this with our build
system, thanks!
I spend a week, learning Ubuntu Packaging and autoconfig. I figured it was
my job to package libenchantnet if I really wanted to see it used.

I still have plenty of questions on how is the best way of packaging mono
applications.

Nicholas Marshall

PS: gee, that was funny, guy writes spell checking code then forgets to
check spelling..

On 8/13/09, Chris S. <email address hidden> wrote:
>
> How many dictionaries are supported? Instead of a config widget, what
> about making an Item out of each dictinoary, and using those as
> dynamic modifier items?
> --
>
> https://code.launchpad.net/~nmarshall23/do-plugins/spellcheck-plugin/+merge/10048
> You are the owner of lp:~nmarshall23/do-plugins/spellcheck-plugin.
>

--
И

« Back to merge proposal