loplugin:unusedfields in sw part2
Change-Id: I7404fcac486f54fc0e310f401d8546c4aacfede2
Reviewed-on: https://gerrit.libreoffice.org/39040
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
diff --git a/sw/source/ui/dbui/mmoutputtypepage.cxx b/sw/source/ui/dbui/mmoutputtypepage.cxx
index 6b54448..03eaa93 100644
--- a/sw/source/ui/dbui/mmoutputtypepage.cxx
+++ b/sw/source/ui/dbui/mmoutputtypepage.cxx
@@ -269,7 +269,6 @@ SwSendMailDialog::SwSendMailDialog(vcl::Window *pParent, SwMailMergeConfigItem&
m_pStatus = VclPtr<SvSimpleTable>::Create(*m_pContainer);
m_pStatusHB = &(m_pStatus->GetTheHeaderBar());
m_nStatusHeight = m_pContainer->get_height_request();
OUString sTask(SwResId(ST_TASK));
OUString sStatus(SwResId(ST_STATUS));
diff --git a/sw/source/ui/dbui/mmresultdialogs.cxx b/sw/source/ui/dbui/mmresultdialogs.cxx
index 186dc92..3a1f5b0 100644
--- a/sw/source/ui/dbui/mmresultdialogs.cxx
+++ b/sw/source/ui/dbui/mmresultdialogs.cxx
@@ -822,7 +822,6 @@ IMPL_LINK(SwMMResultPrintDialog, PrintHdl_Impl, Button*, pButton, void)
if(nEnd > documentCount)
nEnd = documentCount;
}
xConfigItem->SetPrintRange((sal_uInt16)nBegin, (sal_uInt16)nEnd);
OUString sPages(OUString::number(documentStartPageNumber(xConfigItem.get(), nBegin)));
sPages += " - ";
diff --git a/sw/source/ui/envelp/label1.cxx b/sw/source/ui/envelp/label1.cxx
index 31cf691..eee6e23 100644
--- a/sw/source/ui/envelp/label1.cxx
+++ b/sw/source/ui/envelp/label1.cxx
@@ -99,16 +99,13 @@ SwLabDlg::SwLabDlg(vcl::Window* pParent, const SfxItemSet& rSet,
, aTypeIds(50, 10)
, m_pRecs(new SwLabRecs)
, m_bLabel(bLabel)
, m_nFormatId(0)
, m_nOptionsId(0)
, m_nLabelId(0)
, m_nCardsId(0)
, m_nBusinessId(0)
, m_nPrivateId(0)
{
WaitObject aWait( pParent );
m_nFormatId = AddTabPage("format", SwLabFormatPage::Create, nullptr);
AddTabPage("format", SwLabFormatPage::Create, nullptr);
m_nOptionsId = AddTabPage("options", SwLabPrtPage::Create, nullptr);
m_nCardsId = AddTabPage("cards", SwVisitingCardPage::Create, nullptr);
m_sBusinessCardDlg = GetPageText(m_nCardsId);
@@ -126,8 +123,8 @@ SwLabDlg::SwLabDlg(vcl::Window* pParent, const SfxItemSet& rSet,
RemoveTabPage("labels");
RemoveTabPage("cards");
m_nLabelId = AddTabPage("medium", SwLabPage::Create, nullptr);
m_nBusinessId = AddTabPage("business", SwBusinessDataPage::Create, nullptr );
m_nPrivateId = AddTabPage("private", SwPrivateDataPage::Create, nullptr);
AddTabPage("business", SwBusinessDataPage::Create, nullptr );
AddTabPage("private", SwPrivateDataPage::Create, nullptr);
SetText(m_sBusinessCardDlg);
}
// Read user label from writer.cfg
diff --git a/sw/source/ui/frmdlg/frmdlg.cxx b/sw/source/ui/frmdlg/frmdlg.cxx
index 5294a92..19d81ef 100644
--- a/sw/source/ui/frmdlg/frmdlg.cxx
+++ b/sw/source/ui/frmdlg/frmdlg.cxx
@@ -64,9 +64,6 @@ SwFrameDlg::SwFrameDlg( SfxViewFrame* pViewFrame,
, m_nStdId(0)
, m_nAddId(0)
, m_nWrapId(0)
, m_nUrlId(0)
, m_nPictureId(0)
, m_nCropId(0)
, m_nColumnId(0)
//, m_nBackgroundId(0)
, m_nAreaId(0)
@@ -87,11 +84,11 @@ SwFrameDlg::SwFrameDlg( SfxViewFrame* pViewFrame,
m_nStdId = AddTabPage("type", SwFramePage::Create, nullptr);
m_nAddId = AddTabPage("options", SwFrameAddPage::Create, nullptr);
m_nWrapId = AddTabPage("wrap", SwWrapTabPage::Create, nullptr);
m_nUrlId = AddTabPage("hyperlink", SwFrameURLPage::Create, nullptr);
AddTabPage("hyperlink", SwFrameURLPage::Create, nullptr);
if (m_sDlgType == "PictureDialog")
{
m_nPictureId = AddTabPage("picture", SwGrfExtPage::Create, nullptr);
m_nCropId = AddTabPage("crop", RID_SVXPAGE_GRFCROP);
AddTabPage("picture", SwGrfExtPage::Create, nullptr);
AddTabPage("crop", RID_SVXPAGE_GRFCROP);
}
if (m_sDlgType == "FrameDialog")
{
diff --git a/sw/source/ui/inc/mmresultdialogs.hxx b/sw/source/ui/inc/mmresultdialogs.hxx
index e8280c1..e64e03b 100644
--- a/sw/source/ui/inc/mmresultdialogs.hxx
+++ b/sw/source/ui/inc/mmresultdialogs.hxx
@@ -191,7 +191,6 @@ class SwSendMailDialog : public ModelessDialog //SfxModalDialog
SwSendMailDialog_Impl* m_pImpl;
SwMailMergeConfigItem* m_pConfigItem;
sal_Int32 m_nStatusHeight;
sal_Int32 m_nSendCount;
sal_Int32 m_nErrorCount;
diff --git a/sw/source/uibase/dbui/dbmgr.cxx b/sw/source/uibase/dbui/dbmgr.cxx
index 393a1df..d6671ae 100644
--- a/sw/source/uibase/dbui/dbmgr.cxx
+++ b/sw/source/uibase/dbui/dbmgr.cxx
@@ -3163,8 +3163,6 @@ std::shared_ptr<SwMailMergeConfigItem> SwDBManager::PerformMailMerge(SwView* pVi
aMergeDesc.bCreateSingleFile = true;
rSh.GetDBManager()->Merge(aMergeDesc);
xConfigItem->SetMergeDone();
return xConfigItem;
}
diff --git a/sw/source/uibase/dbui/mmconfigitem.cxx b/sw/source/uibase/dbui/mmconfigitem.cxx
index 86e9567..ca0559b 100644
--- a/sw/source/uibase/dbui/mmconfigitem.cxx
+++ b/sw/source/uibase/dbui/mmconfigitem.cxx
@@ -703,11 +703,8 @@ void SwMailMergeConfigItem_Impl::SetCurrentGreeting(
SwMailMergeConfigItem::SwMailMergeConfigItem() :
m_pImpl(new SwMailMergeConfigItem_Impl),
m_bAddressInserted(false),
m_bMergeDone(false),
m_bGreetingInserted(false),
m_nGreetingMoves(0),
m_nStartPrint(0),
m_nEndPrint(0),
m_pSourceView(nullptr),
m_pTargetView(nullptr)
{
@@ -1534,7 +1531,6 @@ void SwMailMergeConfigItem::SetInServerPassword(const OUString& rPassword)
void SwMailMergeConfigItem::DocumentReloaded()
{
m_bMergeDone = false;
m_bGreetingInserted = false;
m_bAddressInserted = false;
m_rAddressBlockFrame.clear();
@@ -1602,7 +1598,6 @@ void SwMailMergeConfigItem::SetTargetView(SwView* pView)
if(!m_pTargetView)
{
m_pImpl->m_aMergeInfos.clear();
m_bMergeDone = false;
}
}
diff --git a/sw/source/uibase/dialog/SwSpellDialogChildWindow.cxx b/sw/source/uibase/dialog/SwSpellDialogChildWindow.cxx
index 52aeec2..32f651c 100644
--- a/sw/source/uibase/dialog/SwSpellDialogChildWindow.cxx
+++ b/sw/source/uibase/dialog/SwSpellDialogChildWindow.cxx
@@ -60,10 +60,6 @@ using namespace ::com::sun::star::beans;
SFX_IMPL_CHILDWINDOW_WITHID(SwSpellDialogChildWindow, FN_SPELL_GRAMMAR_DIALOG)
#define SPELL_START_BODY 0 // body text area
#define SPELL_START_OTHER 1 // frame, footnote, header, footer
#define SPELL_START_DRAWTEXT 2 // started in a draw text object
struct SpellState
{
bool m_bInitialCall;
@@ -71,7 +67,6 @@ struct SpellState
bool m_bLostFocus;
// restart and progress information
sal_uInt16 m_SpellStartPosition;
bool m_bBodySpelled; // body already spelled
bool m_bOtherSpelled; // frames, footnotes, headers and footers spelled
bool m_bStartedInOther; // started the spelling inside of the _other_ area
@@ -101,7 +96,6 @@ struct SpellState
m_bInitialCall(true),
m_bLockFocus(false),
m_bLostFocus(false),
m_SpellStartPosition(SPELL_START_BODY),
m_bBodySpelled(false),
m_bOtherSpelled(false),
m_bStartedInOther(false),
@@ -226,7 +220,6 @@ svx::SpellPortions SwSpellDialogChildWindow::GetNextWrongSentence(bool bRecheck)
}
// determine if the selection is outside of the body text
bOtherText = !(pWrtShell->GetFrameType(nullptr,true) & FrameTypeFlags::BODY);
m_pSpellState->m_SpellStartPosition = bOtherText ? SPELL_START_OTHER : SPELL_START_BODY;
if(bOtherText)
{
m_pSpellState->pOtherCursor.reset( new SwPaM(*pWrtShell->GetCursor()->GetPoint()) );
@@ -250,7 +243,6 @@ svx::SpellPortions SwSpellDialogChildWindow::GetNextWrongSentence(bool bRecheck)
else
{
SdrView* pSdrView = pWrtShell->GetDrawView();
m_pSpellState->m_SpellStartPosition = SPELL_START_DRAWTEXT;
m_pSpellState->m_pStartDrawing = pSdrView->GetMarkedObjectList().GetMark(0)->GetMarkedSdrObj();
OutlinerView* pOLV = pSdrView->GetTextEditOutlinerView();
// start checking at the top of the drawing object
diff --git a/sw/source/uibase/docvw/AnchorOverlayObject.cxx b/sw/source/uibase/docvw/AnchorOverlayObject.cxx
index 116cc14..c7f1ee9 100644
--- a/sw/source/uibase/docvw/AnchorOverlayObject.cxx
+++ b/sw/source/uibase/docvw/AnchorOverlayObject.cxx
@@ -238,7 +238,6 @@ AnchorOverlayObject::AnchorOverlayObject( const basegfx::B2DPoint& rBasePos,
, maTriangle()
, maLine()
, maLineTop()
, mHeight(0)
, mAnchorState(AnchorState::All)
, mbLineSolid(false)
{
diff --git a/sw/source/uibase/docvw/AnchorOverlayObject.hxx b/sw/source/uibase/docvw/AnchorOverlayObject.hxx
index cc15504..9fcac87 100644
--- a/sw/source/uibase/docvw/AnchorOverlayObject.hxx
+++ b/sw/source/uibase/docvw/AnchorOverlayObject.hxx
@@ -73,8 +73,6 @@ class AnchorOverlayObject: public sdr::overlay::OverlayObjectWithBasePosition
void setLineSolid( const bool bNew );
bool getLineSolid() const { return mbLineSolid; }
void SetHeight( const unsigned long aHeight ) { mHeight = aHeight; };
void SetAnchorState( const AnchorState aState );
AnchorState GetAnchorState() const { return mAnchorState; }
@@ -104,7 +102,6 @@ class AnchorOverlayObject: public sdr::overlay::OverlayObjectWithBasePosition
basegfx::B2DPolygon maTriangle;
basegfx::B2DPolygon maLine;
basegfx::B2DPolygon maLineTop;
unsigned long mHeight;
AnchorState mAnchorState;
bool mbLineSolid : 1;
diff --git a/sw/source/uibase/docvw/AnnotationWin2.cxx b/sw/source/uibase/docvw/AnnotationWin2.cxx
index e4360ab..0738fcc 100644
--- a/sw/source/uibase/docvw/AnnotationWin2.cxx
+++ b/sw/source/uibase/docvw/AnnotationWin2.cxx
@@ -685,7 +685,6 @@ void SwAnnotationWin::SetPosAndSize()
basegfx::B2DPoint( mPageBorder ,mAnchorRect.Bottom()+2*15),
basegfx::B2DPoint( aLineStart.X(),aLineStart.Y()),
basegfx::B2DPoint( aLineEnd.X(),aLineEnd.Y()));
mpAnchor->SetHeight(mAnchorRect.Height());
}
else
{
@@ -697,7 +696,6 @@ void SwAnnotationWin::SetPosAndSize()
mColorAnchor );
if ( mpAnchor )
{
mpAnchor->SetHeight(mAnchorRect.Height());
mpAnchor->setVisible(true);
mpAnchor->SetAnchorState(AnchorState::Tri);
if (HasChildPathFocus())
diff --git a/sw/source/uibase/inc/frmdlg.hxx b/sw/source/uibase/inc/frmdlg.hxx
index b31931d..e31abe7 100644
--- a/sw/source/uibase/inc/frmdlg.hxx
+++ b/sw/source/uibase/inc/frmdlg.hxx
@@ -37,9 +37,6 @@ class SwFrameDlg : public SfxTabDialog
sal_uInt16 m_nStdId;
sal_uInt16 m_nAddId;
sal_uInt16 m_nWrapId;
sal_uInt16 m_nUrlId;
sal_uInt16 m_nPictureId;
sal_uInt16 m_nCropId;
sal_uInt16 m_nColumnId;
//sal_uInt16 m_nBackgroundId;
sal_uInt16 m_nAreaId;
diff --git a/sw/source/uibase/inc/inputwin.hxx b/sw/source/uibase/inc/inputwin.hxx
index c83301f..0470787 100644
--- a/sw/source/uibase/inc/inputwin.hxx
+++ b/sw/source/uibase/inc/inputwin.hxx
@@ -55,7 +55,6 @@ friend class InputEdit;
OUString aAktTableName, sOldFormula;
bool bFirst : 1; // initialisations at first call
bool bActive : 1; // for hide/show when switching documents
bool bIsTable : 1;
bool bDelSel : 1;
bool m_bDoesUndo : 1;
diff --git a/sw/source/uibase/inc/label.hxx b/sw/source/uibase/inc/label.hxx
index dc1ddba..055deac 100644
--- a/sw/source/uibase/inc/label.hxx
+++ b/sw/source/uibase/inc/label.hxx
@@ -42,12 +42,9 @@ class SwLabDlg : public SfxTabDialog
OUString aLstGroup;
OUString m_sBusinessCardDlg;
bool m_bLabel;
sal_uInt16 m_nFormatId;
sal_uInt16 m_nOptionsId;
sal_uInt16 m_nLabelId;
sal_uInt16 m_nCardsId;
sal_uInt16 m_nBusinessId;
sal_uInt16 m_nPrivateId;
void ReplaceGroup_( const OUString &rMake );
virtual void PageCreated( sal_uInt16 nId, SfxTabPage &rPage ) override;
diff --git a/sw/source/uibase/inc/mmconfigitem.hxx b/sw/source/uibase/inc/mmconfigitem.hxx
index b16f280..84b78fa 100644
--- a/sw/source/uibase/inc/mmconfigitem.hxx
+++ b/sw/source/uibase/inc/mmconfigitem.hxx
@@ -54,16 +54,12 @@ class SW_DLLPUBLIC SwMailMergeConfigItem
std::unique_ptr<SwMailMergeConfigItem_Impl> m_pImpl;
//session information - not stored in configuration
bool m_bAddressInserted;
bool m_bMergeDone;
bool m_bGreetingInserted;
sal_Int32 m_nGreetingMoves;
OUString m_rAddressBlockFrame;
std::set<sal_Int32> m_aExcludedRecords;
css::uno::Reference<css::view::XSelectionChangeListener> m_xDBChangedListener;
sal_uInt16 m_nStartPrint;
sal_uInt16 m_nEndPrint;
OUString m_sSelectedPrinter;
SwView* m_pSourceView;
@@ -226,9 +222,6 @@ public:
void MoveGreeting( sal_Int32 nMove) { m_nGreetingMoves += nMove;}
sal_Int32 GetGreetingMoves() const { return m_nGreetingMoves;}
void SetMergeDone( )
{ m_bMergeDone = true; }
// new source document - reset some flags
void DocumentReloaded();
@@ -240,9 +233,6 @@ public:
SwDocMergeInfo& GetDocumentMergeInfo(sal_uInt32 nDocument);
sal_uInt32 GetMergedDocumentCount();
void SetPrintRange( sal_uInt16 nStartDocument, sal_uInt16 nEndDocument)
{ m_nStartPrint = nStartDocument; m_nEndPrint = nEndDocument; }
const OUString& GetSelectedPrinter() const
{ return m_sSelectedPrinter; }
void SetSelectedPrinter(const OUString& rSet)
diff --git a/sw/source/uibase/ribbar/inputwin.cxx b/sw/source/uibase/ribbar/inputwin.cxx
index 6bf68a3..36df87ea 100644
--- a/sw/source/uibase/ribbar/inputwin.cxx
+++ b/sw/source/uibase/ribbar/inputwin.cxx
@@ -72,7 +72,7 @@ SwInputWindow::SwInputWindow(vcl::Window* pParent, SfxDispatcher* pDispatcher)
, m_bCallUndo(false)
{
bFirst = true;
bActive = bIsTable = bDelSel = false;
bIsTable = bDelSel = false;
aEdit->SetSizePixel(aEdit->CalcMinimumSize());
aPos->SetSizePixel(aPos->LogicToPixel(Size(45, 11), MapMode(MapUnit::MapAppFont)));