To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=98275
------- Additional comments from [email protected] Fri Jan 23 13:09:55 +0000 2009 ------- The odd thing (to people not knowing the context) is that the speller is called before the text is set. Thus a reference to the issue would be fine. Or maybe if StartSpell woul be named InitSpeller. StartSpeller seems to imply that in course of that call spell checking will be done. And if that were true it would look strange to add the text after the spell checking was already started. Thus either add a reference to the issue or rename the functions like StartSpeller -> InitSpeller and CompleteOnlineSpelling -> DoOnlineSpelling Choose whichever one you think is more appropriate. The 2nd thing that looks a strange is that the object where spelling is started is different from the one where it is ended (if that is the case). Ideally, if those two function above do actually serve some start/stop logic for a single specific task, one would expect them to be implemented in the very same object. But since I do not know what they exactly do I cannot really comment on this detail. --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
