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 t...@openoffice.org Fri Jan 23 09:31:22 +0000
2009 -------
Debugging further into this the difference is quite the other way around as
expected before. In m39 the text for the notes seems to be already loaded at
that time (the first call to the speller) where it is not done in m15.
The lines where the difference occurs is in svx/source/editeng/impedit4.cxx in
the function ImpEditEngine::Spell( EditView* pEditView, sal_Bool bMultipleDoc )
The respective lines are:
sal_Bool bIsStart = sal_False;
if ( bMultipleDoc )
bIsStart = sal_True; // Immer von Vorne bzw. von hinten...
else if ( ( CreateEPaM( aEditDoc.GetStartPaM() ) == pSpellInfo->aSpellStart
) )
bIsStart = sal_True;
In m15 pSpellInfo->aSpellStart is 0/0 and thus in the next line bIsStart will
become true. Both of these values are needed for the function
EditSpellWrapper::SpellStart in edtspell.cxx to set the SpellInfo to values
that later on (in a different function) results in the decision that nothing
needs to be done right now. The values will be set in these lines:
if ( !IsStartDone() )
{
pSpellInfo->bSpellToEnd = sal_True;
pSpellInfo->aSpellTo = pImpEE->CreateEPaM(
pImpEE->GetEditDoc().GetEndPaM() );
}
else
{
pSpellInfo->bSpellToEnd = sal_False;
pSpellInfo->aSpellTo = pSpellInfo->aSpellStart;
pEditView->GetImpEditView()->SetEditSelection(
pImpEE->GetEditDoc().GetEndPaM() );
}
To prove my point:
If in m39 you go back to code lines in impedit4.cxx and set a breakpoint right
after those 5 lines got executed and then set the values to
pSpellInfo->aSpellStart = 0/0
and
bIsStart = sal_True
then even in m39 the problem will not occur.
I have not further debugged where in m39 pSpellInfo->aSpellStart gets its value
of 0/9 since I'm quite confident that it is the case because the notes text are
already loaded.
Thus it seems the reason is that in m15
InitControls();
which calls mpOutlinerView->StartSpeller()
SetPostItText();
Thus SetPostItText was called AFTER StartSpeller() while in m39 it is the other
way around:
SetPostItText();
SetLanguage(GetLanguage());
View()->StartSpeller();
Now SetPostItText() is called before StartSpeller().
---------------------------------------------------------------------
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: issues-unsubscr...@sw.openoffice.org
For additional commands, e-mail: issues-h...@sw.openoffice.org
---------------------------------------------------------------------
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org
