> I had a look at the patch! I thought it was a GUII patch but what you did
> is just moving some functions around! IMO this is not good it just makes
> spellchecker.C more complicated. If you want to remove those functions
> from BufferView, then IMO it's better to move them inside a new file
> (could be just a namespace-class) and not to put them into spellchecker.C.

I was intending to make the spellchecker guii but then thought that cleaning 
up the SR stuff should be done first.

But perhaps you are right and I should do the spellchecker first and wait 
with SR.

I will do the following: 

1.

frontends/xforms/FormSpell.[Ch]
frontends/xforms/form_spell.[Ch]
frontends/xforms/forms/form_spell.fd

2.

frontends/controllers/ControlSpell.[Ch]

where the controller uses the present spellcheck implementation. I 
will put the bufferview functions from the patch in the controller.

A next step would then be to

1. rewrite SR
2. rewrite the spellcheck implementation
3. rewrite controller

Do you agree?

gr.ed.

Reply via email to