[Libreoffice-commits] core.git: sw/inc sw/source
Noel Grandin
noel.grandin at collabora.co.uk
Tue Apr 11 10:21:12 UTC 2017
sw/inc/AnnotationWin.hxx | 4 -
sw/source/uibase/docvw/AnnotationWin.cxx | 64 +++++++++++++++---------------
sw/source/uibase/docvw/AnnotationWin2.cxx | 28 ++++++-------
3 files changed, 46 insertions(+), 50 deletions(-)
New commits:
commit 8b472a6067cfef1588b4b6122e52fcee269d6bdf
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Mon Apr 10 15:43:33 2017 +0200
inline some trivial private methods
Change-Id: Ic97c4202495112ad4cd32c295516ae8611a73d10
Reviewed-on: https://gerrit.libreoffice.org/36381
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/sw/inc/AnnotationWin.hxx b/sw/inc/AnnotationWin.hxx
index 256441bc9dfb..6f660a27ccca 100644
--- a/sw/inc/AnnotationWin.hxx
+++ b/sw/inc/AnnotationWin.hxx
@@ -204,10 +204,6 @@ class SwAnnotationWin : public vcl::Window
DECL_LINK(ScrollHdl, ScrollBar*, void);
DECL_LINK(DeleteHdl, void*, void);
- SwView& DocView() { return mrView;}
- SwPostItMgr& Mgr() { return mrMgr; }
- Outliner* Engine() { return mpOutliner;}
-
sal_uInt32 CountFollowing();
SwAnnotationWin* GetTopReplyNote();
SvxLanguageItem GetLanguage();
diff --git a/sw/source/uibase/docvw/AnnotationWin.cxx b/sw/source/uibase/docvw/AnnotationWin.cxx
index 9f70393f637e..241de71f81cc 100644
--- a/sw/source/uibase/docvw/AnnotationWin.cxx
+++ b/sw/source/uibase/docvw/AnnotationWin.cxx
@@ -201,25 +201,25 @@ void SwAnnotationWin::SetPostItText()
//point .e.g. fdo#33599
mpField = static_cast<SwPostItField*>(mpFormatField->GetField());
OUString sNewText = mpField->GetPar2();
- bool bTextUnchanged = sNewText.equals(Engine()->GetEditEngine().GetText());
+ bool bTextUnchanged = sNewText.equals(mpOutliner->GetEditEngine().GetText());
ESelection aOrigSelection(GetOutlinerView()->GetEditView().GetSelection());
// get text from SwPostItField and insert into our textview
- Engine()->SetModifyHdl( Link<LinkParamNone*,void>() );
- Engine()->EnableUndo( false );
+ mpOutliner->SetModifyHdl( Link<LinkParamNone*,void>() );
+ mpOutliner->EnableUndo( false );
if( mpField->GetTextObject() )
- Engine()->SetText( *mpField->GetTextObject() );
+ mpOutliner->SetText( *mpField->GetTextObject() );
else
{
- Engine()->Clear();
+ mpOutliner->Clear();
GetOutlinerView()->SetAttribs(DefaultItem());
GetOutlinerView()->InsertText(sNewText);
}
- Engine()->ClearModifyFlag();
- Engine()->GetUndoManager().Clear();
- Engine()->EnableUndo( true );
- Engine()->SetModifyHdl( LINK( this, SwAnnotationWin, ModifyHdl ) );
+ mpOutliner->ClearModifyFlag();
+ mpOutliner->GetUndoManager().Clear();
+ mpOutliner->EnableUndo( true );
+ mpOutliner->SetModifyHdl( LINK( this, SwAnnotationWin, ModifyHdl ) );
if (bTextUnchanged)
GetOutlinerView()->GetEditView().SetSelection(aOrigSelection);
if (bCursorVisible)
@@ -229,17 +229,17 @@ void SwAnnotationWin::SetPostItText()
void SwAnnotationWin::UpdateData()
{
- if ( Engine()->IsModified() )
+ if ( mpOutliner->IsModified() )
{
IDocumentUndoRedo & rUndoRedo(
- DocView().GetDocShell()->GetDoc()->GetIDocumentUndoRedo());
+ mrView.GetDocShell()->GetDoc()->GetIDocumentUndoRedo());
std::unique_ptr<SwField> pOldField;
if (rUndoRedo.DoesUndo())
{
pOldField.reset(mpField->Copy());
}
- mpField->SetPar2(Engine()->GetEditEngine().GetText());
- mpField->SetTextObject(Engine()->CreateParaObject());
+ mpField->SetPar2(mpOutliner->GetEditEngine().GetText());
+ mpField->SetTextObject(mpOutliner->CreateParaObject());
if (rUndoRedo.DoesUndo())
{
SwTextField *const pTextField = mpFormatField->GetTextField();
@@ -249,18 +249,18 @@ void SwAnnotationWin::UpdateData()
new SwUndoFieldFromDoc(aPosition, *pOldField, *mpField, nullptr, true));
}
// so we get a new layout of notes (anchor position is still the same and we would otherwise not get one)
- Mgr().SetLayout();
+ mrMgr.SetLayout();
// #i98686# if we have several views, all notes should update their text
mpFormatField->Broadcast(SwFormatFieldHint( nullptr, SwFormatFieldHintWhich::CHANGED));
- DocView().GetDocShell()->SetModified();
+ mrView.GetDocShell()->SetModified();
}
- Engine()->ClearModifyFlag();
- Engine()->GetUndoManager().Clear();
+ mpOutliner->ClearModifyFlag();
+ mpOutliner->GetUndoManager().Clear();
}
void SwAnnotationWin::Delete()
{
- if (DocView().GetWrtShellPtr()->GotoField(*mpFormatField))
+ if (mrView.GetWrtShellPtr()->GotoField(*mpFormatField))
{
if ( mrMgr.GetActiveSidebarWin() == this)
{
@@ -274,21 +274,21 @@ void SwAnnotationWin::Delete()
}
// we delete the field directly, the Mgr cleans up the PostIt by listening
GrabFocusToDocument();
- DocView().GetWrtShellPtr()->ClearMark();
- DocView().GetWrtShellPtr()->DelRight();
+ mrView.GetWrtShellPtr()->ClearMark();
+ mrView.GetWrtShellPtr()->DelRight();
}
}
void SwAnnotationWin::GotoPos()
{
- DocView().GetDocShell()->GetWrtShell()->GotoField(*mpFormatField);
+ mrView.GetDocShell()->GetWrtShell()->GotoField(*mpFormatField);
}
sal_uInt32 SwAnnotationWin::MoveCaret()
{
// if this is an answer, do not skip over all following ones, but insert directly behind the current one
// but when just leaving a note, skip all following ones as well to continue typing
- return Mgr().IsAnswer()
+ return mrMgr.IsAnswer()
? 1
: 1 + CountFollowing();
}
@@ -361,7 +361,7 @@ void SwAnnotationWin::InitAnswer(OutlinerParaObject* pText)
return;
//collect our old meta data
- SwAnnotationWin* pWin = Mgr().GetNextPostIt(KEY_PAGEUP, this);
+ SwAnnotationWin* pWin = mrMgr.GetNextPostIt(KEY_PAGEUP, this);
const SvtSysLocale aSysLocale;
const LocaleDataWrapper& rLocalData = aSysLocale.GetLocaleData();
SwRewriter aRewriter;
@@ -381,7 +381,7 @@ void SwAnnotationWin::InitAnswer(OutlinerParaObject* pText)
GetOutlinerView()->InsertText("\"\n");
GetOutlinerView()->SetSelection(ESelection(0,0,EE_PARA_ALL,EE_TEXTPOS_ALL));
- SfxItemSet aAnswerSet( DocView().GetDocShell()->GetPool() );
+ SfxItemSet aAnswerSet( mrView.GetDocShell()->GetPool() );
aAnswerSet.Put(SvxFontHeightItem(200,80,EE_CHAR_FONTHEIGHT));
aAnswerSet.Put(SvxPostureItem(ITALIC_NORMAL,EE_CHAR_ITALIC));
GetOutlinerView()->SetAttribs(aAnswerSet);
@@ -392,16 +392,16 @@ void SwAnnotationWin::InitAnswer(OutlinerParaObject* pText)
GetOutlinerView()->SetAttribs(DefaultItem());
// lets insert an undo step so the initial text can be easily deleted
// but do not use UpdateData() directly, would set modified state again and reentrance into Mgr
- Engine()->SetModifyHdl( Link<LinkParamNone*,void>() );
+ mpOutliner->SetModifyHdl( Link<LinkParamNone*,void>() );
IDocumentUndoRedo & rUndoRedo(
- DocView().GetDocShell()->GetDoc()->GetIDocumentUndoRedo());
+ mrView.GetDocShell()->GetDoc()->GetIDocumentUndoRedo());
std::unique_ptr<SwField> pOldField;
if (rUndoRedo.DoesUndo())
{
pOldField.reset(mpField->Copy());
}
- mpField->SetPar2(Engine()->GetEditEngine().GetText());
- mpField->SetTextObject(Engine()->CreateParaObject());
+ mpField->SetPar2(mpOutliner->GetEditEngine().GetText());
+ mpField->SetTextObject(mpOutliner->CreateParaObject());
if (rUndoRedo.DoesUndo())
{
SwTextField *const pTextField = mpFormatField->GetTextField();
@@ -410,14 +410,14 @@ void SwAnnotationWin::InitAnswer(OutlinerParaObject* pText)
rUndoRedo.AppendUndo(
new SwUndoFieldFromDoc(aPosition, *pOldField, *mpField, nullptr, true));
}
- Engine()->SetModifyHdl( LINK( this, SwAnnotationWin, ModifyHdl ) );
- Engine()->ClearModifyFlag();
- Engine()->GetUndoManager().Clear();
+ mpOutliner->SetModifyHdl( LINK( this, SwAnnotationWin, ModifyHdl ) );
+ mpOutliner->ClearModifyFlag();
+ mpOutliner->GetUndoManager().Clear();
}
void SwAnnotationWin::UpdateText(const OUString& aText)
{
- Engine()->Clear();
+ mpOutliner->Clear();
GetOutlinerView()->InsertText(aText);
UpdateData();
}
diff --git a/sw/source/uibase/docvw/AnnotationWin2.cxx b/sw/source/uibase/docvw/AnnotationWin2.cxx
index 30d5648f652b..84abbb3026fb 100644
--- a/sw/source/uibase/docvw/AnnotationWin2.cxx
+++ b/sw/source/uibase/docvw/AnnotationWin2.cxx
@@ -540,7 +540,7 @@ void SwAnnotationWin::InitControls()
SetLanguage(GetLanguage());
GetOutlinerView()->StartSpeller();
SetPostItText();
- Engine()->CompleteOnlineSpelling();
+ mpOutliner->CompleteOnlineSpelling();
mpSidebarTextControl->Show();
mpMetadataAuthor->Show();
@@ -776,7 +776,7 @@ void SwAnnotationWin::SetPosAndSize()
const bool bTableCursorNeeded = pTextNode->FindTableBoxStartNode() != pContentNd->FindTableBoxStartNode();
if ( bTableCursorNeeded )
{
- SwShellTableCursor* pTableCursor = new SwShellTableCursor( DocView().GetWrtShell(), aStartPos );
+ SwShellTableCursor* pTableCursor = new SwShellTableCursor( mrView.GetWrtShell(), aStartPos );
pTableCursor->SetMark();
pTableCursor->GetMark()->nNode = *pTextNode;
pTableCursor->GetMark()->nContent.Assign( pTextNode, pTextAnnotationField->GetStart()+1 );
@@ -785,7 +785,7 @@ void SwAnnotationWin::SetPosAndSize()
}
else
{
- SwShellCursor* pCursor = new SwShellCursor( DocView().GetWrtShell(), aStartPos );
+ SwShellCursor* pCursor = new SwShellCursor( mrView.GetWrtShell(), aStartPos );
pCursor->SetMark();
pCursor->GetMark()->nNode = *pTextNode;
pCursor->GetMark()->nContent.Assign( pTextNode, pTextAnnotationField->GetStart()+1 );
@@ -834,7 +834,7 @@ void SwAnnotationWin::SetPosAndSize()
// This window is not a reply, then draw its range overlay.
mpTextRangeOverlay =
sw::overlay::OverlayRanges::CreateOverlayRange(
- DocView(),
+ mrView,
mColorAnchor,
maAnnotationTextRanges,
mpAnchor && mpAnchor->getLineSolid() );
@@ -1022,33 +1022,33 @@ void SwAnnotationWin::SetReadonly(bool bSet)
void SwAnnotationWin::SetLanguage(const SvxLanguageItem& rNewItem)
{
- Link<LinkParamNone*,void> aLink = Engine()->GetModifyHdl();
- Engine()->SetModifyHdl( Link<LinkParamNone*,void>() );
+ Link<LinkParamNone*,void> aLink = mpOutliner->GetModifyHdl();
+ mpOutliner->SetModifyHdl( Link<LinkParamNone*,void>() );
ESelection aOld = GetOutlinerView()->GetSelection();
- ESelection aNewSelection( 0, 0, Engine()->GetParagraphCount()-1, EE_TEXTPOS_ALL );
+ ESelection aNewSelection( 0, 0, mpOutliner->GetParagraphCount()-1, EE_TEXTPOS_ALL );
GetOutlinerView()->SetSelection( aNewSelection );
SfxItemSet aEditAttr(GetOutlinerView()->GetAttribs());
aEditAttr.Put(rNewItem);
GetOutlinerView()->SetAttribs( aEditAttr );
GetOutlinerView()->SetSelection(aOld);
- Engine()->SetModifyHdl( aLink );
+ mpOutliner->SetModifyHdl( aLink );
const SwViewOption* pVOpt = mrView.GetWrtShellPtr()->GetViewOptions();
- EEControlBits nCntrl = Engine()->GetControlWord();
+ EEControlBits nCntrl = mpOutliner->GetControlWord();
// turn off
nCntrl &= ~EEControlBits::ONLINESPELLING;
- Engine()->SetControlWord(nCntrl);
+ mpOutliner->SetControlWord(nCntrl);
//turn back on
if (pVOpt->IsOnlineSpell())
nCntrl |= EEControlBits::ONLINESPELLING;
else
nCntrl &= ~EEControlBits::ONLINESPELLING;
- Engine()->SetControlWord(nCntrl);
+ mpOutliner->SetControlWord(nCntrl);
- Engine()->CompleteOnlineSpelling();
+ mpOutliner->CompleteOnlineSpelling();
Invalidate();
}
@@ -1143,7 +1143,7 @@ void SwAnnotationWin::DeactivatePostIt()
if ( !Application::GetSettings().GetStyleSettings().GetHighContrastMode() )
GetOutlinerView()->SetBackgroundColor(COL_TRANSPARENT);
- if ( !IsProtected() && Engine()->GetEditEngine().GetText().isEmpty() )
+ if ( !IsProtected() && mpOutliner->GetEditEngine().GetText().isEmpty() )
{
mnEventId = Application::PostUserEvent( LINK( this, SwAnnotationWin, DeleteHdl), nullptr, true );
}
@@ -1175,7 +1175,7 @@ void SwAnnotationWin::ExecuteCommand(sal_uInt16 nSlot)
{
// if this note is empty, it will be deleted once losing the focus, so no reply, but only a new note
// will be created
- if (!Engine()->GetEditEngine().GetText().isEmpty())
+ if (!mpOutliner->GetEditEngine().GetText().isEmpty())
{
OutlinerParaObject* pPara = new OutlinerParaObject(*GetOutlinerView()->GetEditView().CreateTextObject());
mrMgr.RegisterAnswer(pPara);
More information about the Libreoffice-commits
mailing list