> 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.