connectivity/source/drivers/mork/MResultSet.cxx | 3 connectivity/source/drivers/mork/MResultSet.hxx | 1 cui/source/inc/numpages.hxx | 6 - cui/source/tabpages/numpages.cxx | 4 - dbaccess/source/ui/inc/querycontroller.hxx | 1 dbaccess/source/ui/querydesign/querycontroller.cxx | 4 - filter/source/xsltdialog/xmlfiltercommon.hxx | 1 filter/source/xsltdialog/xmlfiltersettingsdialog.cxx | 1 forms/source/xforms/xformsevent.cxx | 6 - forms/source/xforms/xformsevent.hxx | 3 framework/inc/uielement/statusbarmanager.hxx | 1 framework/source/uiconfiguration/uiconfigurationmanager.cxx | 3 framework/source/uielement/statusbarmanager.cxx | 12 +-- sc/source/core/data/table3.cxx | 44 +++--------- sc/source/filter/html/htmlexp.cxx | 21 ----- sc/source/filter/html/htmlexp2.cxx | 6 - sc/source/filter/inc/htmlexp.hxx | 6 - svtools/source/contnr/imivctl.hxx | 1 svtools/source/contnr/imivctl1.cxx | 2 sw/source/ui/dbui/mmresultdialogs.cxx | 8 -- sw/source/ui/inc/mmresultdialogs.hxx | 2 writerfilter/source/dmapper/GraphicImport.cxx | 19 ----- writerfilter/source/rtftok/rtfdocumentimpl.cxx | 2 writerfilter/source/rtftok/rtfdocumentimpl.hxx | 1 24 files changed, 37 insertions(+), 121 deletions(-)
New commits: commit e0aed1459513be5e08fab9de06848df5dc9d0b5f Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 16:02:43 2017 +0200 unused m_sNoSubjectST/m_sDefaultAttachmentST in SwMMResultEmailDialog ever since commit ba0796b331247b27886384377c480e413c3ee83d Date: Wed Dec 23 17:43:14 2015 +0100 mailmerge: Killed the 'Save, print or send' page. Change-Id: Id77cd6beb3eaa13ead848495d9f8661c83f60975 diff --git a/sw/source/ui/dbui/mmresultdialogs.cxx b/sw/source/ui/dbui/mmresultdialogs.cxx index a53affbf99d2..c4000892c7bc 100644 --- a/sw/source/ui/dbui/mmresultdialogs.cxx +++ b/sw/source/ui/dbui/mmresultdialogs.cxx @@ -434,9 +434,7 @@ void SwMMResultEmailDialog::FillInEmailSettings() if (m_pAttachmentED->GetText().isEmpty()) { - OUString sAttach(m_sDefaultAttachmentST); - sAttach += "."; - sAttach += lcl_GetExtensionForDocType( + OUString sAttach = "." + lcl_GetExtensionForDocType( reinterpret_cast<sal_uLong>(m_pSendAsLB->GetSelectEntryData())); m_pAttachmentED->SetText(sAttach); @@ -1012,10 +1010,10 @@ IMPL_LINK(SwMMResultEmailDialog, SendDocumentsHdl_Impl, Button*, pButton, void) ScopedVclPtrInstance<SwSendQueryBox_Impl> aQuery(pButton, "SubjectDialog", "modules/swriter/ui/subjectdialog.ui"); aQuery->SetIsEmptyTextAllowed(true); - aQuery->SetValue(m_sNoSubjectST); + aQuery->SetValue(""); if(RET_OK == aQuery->Execute()) { - if(aQuery->GetValue() != m_sNoSubjectST) + if(!aQuery->GetValue().isEmpty()) m_pSubjectED->SetText(aQuery->GetValue()); } else diff --git a/sw/source/ui/inc/mmresultdialogs.hxx b/sw/source/ui/inc/mmresultdialogs.hxx index 4593af32c80a..77c71ca8a394 100644 --- a/sw/source/ui/inc/mmresultdialogs.hxx +++ b/sw/source/ui/inc/mmresultdialogs.hxx @@ -125,8 +125,6 @@ class SwMMResultEmailDialog : public SfxModalDialog VclPtr<Button> m_pOKButton; - OUString m_sDefaultAttachmentST; - OUString m_sNoSubjectST; OUString m_sConfigureMail; OUString m_sCC; commit e159f8b128a9fd1eda8e499e12304aaef97bd7e0 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 15:52:32 2017 +0200 unused maFilterService in filter_info_impl since it was added in commit d0d152d4e3e99c41dd71187eca917470c4d64adf Date: Fri Apr 4 15:34:02 2003 +0000 INTEGRATION: CWS xmlfilter01 (1.1.2); FILE ADDED Change-Id: I69bf6cad4e3d637c065d967a2b5915312cc23e93 diff --git a/filter/source/xsltdialog/xmlfiltercommon.hxx b/filter/source/xsltdialog/xmlfiltercommon.hxx index 4089993bb905..3d87a3e6bce3 100644 --- a/filter/source/xsltdialog/xmlfiltercommon.hxx +++ b/filter/source/xsltdialog/xmlfiltercommon.hxx @@ -42,7 +42,6 @@ public: OUString maFilterName; OUString maType; OUString maDocumentService; - OUString maFilterService; OUString maInterfaceName; OUString maComment; OUString maExtension; diff --git a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx index 60045efce674..b8de50691605 100644 --- a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx +++ b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx @@ -1511,7 +1511,6 @@ bool filter_info_impl::operator==( const filter_info_impl& r ) const return maFilterName == r.maFilterName && maType == r.maType && maDocumentService == r.maDocumentService && - maFilterService == r.maFilterService && maInterfaceName == r.maInterfaceName && maComment == r.maComment && maExtension == r.maExtension && commit 31d7894d8b0a08fca8bf8b08c2c0296b48b64f5f Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 15:50:14 2017 +0200 unused aDDLastRectPos in SvxIconChoiceCtrl_Impl since commit bf2373daf50595c02741f63d941fda0462535d62 Date: Tue Jul 26 23:32:40 2011 +0200 callcatcher: remove unused methods Change-Id: I2eef2bdcd40d4b4b1da8cf7ca95b0194ea99216f diff --git a/svtools/source/contnr/imivctl.hxx b/svtools/source/contnr/imivctl.hxx index 1684d2290836..5b8c816eb1f8 100644 --- a/svtools/source/contnr/imivctl.hxx +++ b/svtools/source/contnr/imivctl.hxx @@ -167,7 +167,6 @@ class SvxIconChoiceCtrl_Impl Size aImageSize; Size aDefaultTextSize; Size aOutputSize; // Pixel - Point aDDLastRectPos; VclPtr<SvtIconChoiceCtrl> pView; IcnCursor_Impl* pImpCursor; IcnGridMap_Impl* pGridMap; diff --git a/svtools/source/contnr/imivctl1.cxx b/svtools/source/contnr/imivctl1.cxx index c1d96f4e8cc2..424b080ff213 100644 --- a/svtools/source/contnr/imivctl1.cxx +++ b/svtools/source/contnr/imivctl1.cxx @@ -2024,7 +2024,7 @@ void SvxIconChoiceCtrl_Impl::HideDDIcon() { Size aSize( pDDDev->GetOutputSizePixel() ); // restore pView - pView->DrawOutDev( aDDLastRectPos, aSize, Point(), aSize, *pDDDev ); + pView->DrawOutDev( Point(), aSize, Point(), aSize, *pDDDev ); } pDDBufDev = pDDDev; pDDDev = nullptr; commit dff3e4887d1aa99801fdc70cd6253fb490617407 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:35:06 2017 +0200 unused fields in XFormsEventConcrete ever since commit 6297bf4e24bd48b52083ca711ac9930f60bf85c8 Date: Wed Mar 23 10:39:28 2005 +0000 INTEGRATION: CWS eforms4 (1.1.2); FILE ADDED Change-Id: Ic4915a9b31c3a5f0222da0a4fc8557b2cbe6078e diff --git a/forms/source/xforms/xformsevent.cxx b/forms/source/xforms/xformsevent.cxx index d65b5b0e73ca..e51beb8c52ab 100644 --- a/forms/source/xforms/xformsevent.cxx +++ b/forms/source/xforms/xformsevent.cxx @@ -40,12 +40,12 @@ OUString SAL_CALL XFormsEventConcrete::getType() css::uno::Reference< css::xml::dom::events::XEventTarget > SAL_CALL XFormsEventConcrete::getTarget() { - return m_target; + return css::uno::Reference< css::xml::dom::events::XEventTarget >(); } css::uno::Reference< css::xml::dom::events::XEventTarget > SAL_CALL XFormsEventConcrete::getCurrentTarget() { - return m_currentTarget; + return css::uno::Reference< css::xml::dom::events::XEventTarget >(); } css::xml::dom::events::PhaseType SAL_CALL XFormsEventConcrete::getEventPhase() @@ -65,7 +65,7 @@ sal_Bool SAL_CALL XFormsEventConcrete::getCancelable() css::util::Time SAL_CALL XFormsEventConcrete::getTimeStamp() { - return m_time; + return css::util::Time(); } void SAL_CALL XFormsEventConcrete::stopPropagation() diff --git a/forms/source/xforms/xformsevent.hxx b/forms/source/xforms/xformsevent.hxx index db163295dc88..d9d03d385a8a 100644 --- a/forms/source/xforms/xformsevent.hxx +++ b/forms/source/xforms/xformsevent.hxx @@ -63,11 +63,8 @@ class XFormsEventConcrete : public cppu::WeakImplHelper< XFormsEvent > { protected: OUString m_eventType; - css::uno::Reference< css::xml::dom::events::XEventTarget > m_target; - css::uno::Reference< css::xml::dom::events::XEventTarget > m_currentTarget; bool m_bubbles; bool m_cancelable; - css::util::Time m_time; }; } } } } commit 4174b5b08b461db489f681c1366e5c01b86013ca Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:28:22 2017 +0200 unused m_aModuleIdentifier in StatusBarManager Change-Id: I1f8260fd148b694337ea0440eb2edde4753b34c1 diff --git a/framework/inc/uielement/statusbarmanager.hxx b/framework/inc/uielement/statusbarmanager.hxx index bbafb5f55f43..e2c4966c091b 100644 --- a/framework/inc/uielement/statusbarmanager.hxx +++ b/framework/inc/uielement/statusbarmanager.hxx @@ -95,7 +95,6 @@ class StatusBarManager final: public ::cppu::WeakImplHelper< m_bFrameActionRegistered : 1, m_bUpdateControllers : 1; VclPtr<StatusBar> m_pStatusBar; - OUString m_aModuleIdentifier; css::uno::Reference< css::frame::XFrame > m_xFrame; StatusBarControllerMap m_aControllerMap; osl::Mutex m_mutex; diff --git a/framework/source/uielement/statusbarmanager.cxx b/framework/source/uielement/statusbarmanager.cxx index 176f0d5b33a3..9caa46cc539d 100644 --- a/framework/source/uielement/statusbarmanager.cxx +++ b/framework/source/uielement/statusbarmanager.cxx @@ -307,7 +307,7 @@ void StatusBarManager::CreateControllers() aPropVector.push_back( uno::makeAny( aPropValue ) ); aPropValue.Name = "ModuleIdentifier"; - aPropValue.Value <<= m_aModuleIdentifier; + aPropValue.Value <<= OUString(); aPropVector.push_back( uno::makeAny( aPropValue ) ); aPropValue.Name = "Frame"; @@ -336,7 +336,7 @@ void StatusBarManager::CreateControllers() // 1) UNO Statusbar controllers, registered in Controllers.xcu if ( m_xStatusbarControllerFactory.is() && - m_xStatusbarControllerFactory->hasController( aCommandURL, m_aModuleIdentifier )) + m_xStatusbarControllerFactory->hasController( aCommandURL, "" )) { xController.set(m_xStatusbarControllerFactory->createInstanceWithArgumentsAndContext( aCommandURL, aArgs, m_xContext ), @@ -447,7 +447,7 @@ void StatusBarManager::FillStatusBar( const uno::Reference< container::XIndexAcc if (( nType == css::ui::ItemType::DEFAULT ) && !aCommandURL.isEmpty() ) { - OUString aString( vcl::CommandInfoProvider::GetLabelForCommand(aCommandURL, m_aModuleIdentifier)); + OUString aString( vcl::CommandInfoProvider::GetLabelForCommand(aCommandURL, "")); StatusBarItemBits nItemBits( impl_convertItemStyleToItemBits( nStyle )); m_pStatusBar->InsertItem( nId, nWidth, nItemBits, nOffset ); @@ -474,7 +474,7 @@ void StatusBarManager::FillStatusBar( const uno::Reference< container::XIndexAcc for ( sal_uInt32 i = 0; i < nCount; i++ ) { MergeStatusbarInstruction &rInstruction = aMergeInstructions[i]; - if ( !StatusbarMerger::IsCorrectContext( rInstruction.aMergeContext, m_aModuleIdentifier ) ) + if ( !StatusbarMerger::IsCorrectContext( rInstruction.aMergeContext, "" ) ) continue; AddonStatusbarItemContainer aItems; @@ -486,7 +486,7 @@ void StatusBarManager::FillStatusBar( const uno::Reference< container::XIndexAcc StatusbarMerger::ProcessMergeOperation( m_pStatusBar, nRefPos, nItemId, - m_aModuleIdentifier, + "", rInstruction.aMergeCommand, rInstruction.aMergeCommandParameter, aItems ); @@ -495,7 +495,7 @@ void StatusBarManager::FillStatusBar( const uno::Reference< container::XIndexAcc { StatusbarMerger::ProcessMergeFallback( m_pStatusBar, nItemId, - m_aModuleIdentifier, + "", rInstruction.aMergeCommand, rInstruction.aMergeCommandParameter, aItems ); commit b92068bbe2a907b6e1fa9637ee5740f6daed75a8 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:26:43 2017 +0200 unused m_aModuleIdentifier in UIConfigurationManager Change-Id: Idd4ced156201e81ee818d4b6b69e6c4d254f3387 diff --git a/framework/source/uiconfiguration/uiconfigurationmanager.cxx b/framework/source/uiconfiguration/uiconfigurationmanager.cxx index 17a585b210a3..dcfd7227bc98 100644 --- a/framework/source/uiconfiguration/uiconfigurationmanager.cxx +++ b/framework/source/uiconfiguration/uiconfigurationmanager.cxx @@ -188,7 +188,6 @@ private: OUString m_aXMLPostfix; OUString m_aPropUIName; OUString m_aPropResourceURL; - OUString m_aModuleIdentifier; css::uno::Reference< css::uno::XComponentContext > m_xContext; osl::Mutex m_mutex; cppu::OMultiTypeInterfaceContainerHelper m_aListenerContainer; /// container for ALL Listener @@ -1149,7 +1148,7 @@ Reference< XInterface > SAL_CALL UIConfigurationManager::getImageManager() Sequence<Any> aPropSeq(comphelper::InitAnyPropertySequence( { {"UserConfigStorage", Any(m_xDocConfigStorage)}, - {"ModuleIdentifier", Any(m_aModuleIdentifier)}, + {"ModuleIdentifier", Any(OUString())}, })); xInit->initialize( aPropSeq ); commit e86b07e292dc91d67a72a56e7cc15b8c2a9694a9 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:15:31 2017 +0200 unused m_sUpdateTableName in OQueryController ever since commit 2106d8e648449d34b195068eef5f672a14ea64a8 Date: Sun Jun 10 21:51:19 2012 +0100 callcatcher: update unused code Change-Id: I88a13c5c414a86a82efc6883d2e1602199569dc9 diff --git a/dbaccess/source/ui/inc/querycontroller.hxx b/dbaccess/source/ui/inc/querycontroller.hxx index e0c6b452b81c..3817398ddaeb 100644 --- a/dbaccess/source/ui/inc/querycontroller.hxx +++ b/dbaccess/source/ui/inc/querycontroller.hxx @@ -72,7 +72,6 @@ namespace dbaui OUString m_sStatement; // contains the current sql statement OUString m_sUpdateCatalogName; // catalog for update data OUString m_sUpdateSchemaName; // schema for update data - OUString m_sUpdateTableName; // table for update data mutable OUString m_sName; // name of the query diff --git a/dbaccess/source/ui/querydesign/querycontroller.cxx b/dbaccess/source/ui/querydesign/querycontroller.cxx index 1ab8782d4034..331147134f1a 100644 --- a/dbaccess/source/ui/querydesign/querycontroller.cxx +++ b/dbaccess/source/ui/querydesign/querycontroller.cxx @@ -1284,7 +1284,7 @@ void OQueryController::executeQuery() { PROPERTY_ACTIVE_CONNECTION, makeAny(getConnection()) }, { PROPERTY_UPDATE_CATALOGNAME, makeAny(m_sUpdateCatalogName) }, { PROPERTY_UPDATE_SCHEMANAME, makeAny(m_sUpdateSchemaName) }, - { PROPERTY_UPDATE_TABLENAME, makeAny(m_sUpdateTableName) }, + { PROPERTY_UPDATE_TABLENAME, makeAny(OUString()) }, { PROPERTY_ESCAPE_PROCESSING, makeAny(m_bEscapeProcessing) } })); @@ -1461,7 +1461,7 @@ bool OQueryController::doSaveAsDoc(bool _bSaveAs) if ( editingQuery() ) { - xQuery->setPropertyValue( PROPERTY_UPDATE_TABLENAME, makeAny( m_sUpdateTableName ) ); + xQuery->setPropertyValue( PROPERTY_UPDATE_TABLENAME, makeAny( OUString() ) ); xQuery->setPropertyValue( PROPERTY_ESCAPE_PROCESSING, css::uno::makeAny( m_bEscapeProcessing ) ); xQuery->setPropertyValue( PROPERTY_LAYOUTINFORMATION, getViewData() ); commit a0aff40f3e0c91aaae51802bca4b6b77f18ae848 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:06:26 2017 +0200 unused sNumCharFmtName fields ever since commit 89ef4dea325a11c706ff9619ff4717124260f864 Date: Fri Jul 17 10:34:58 2015 +0200 loplugin:unusedmethods cui Change-Id: Ibfa3841168448d4f08075a684fa36262b303210f diff --git a/cui/source/inc/numpages.hxx b/cui/source/inc/numpages.hxx index 7b12d8d00879..68d63f72e8da 100644 --- a/cui/source/inc/numpages.hxx +++ b/cui/source/inc/numpages.hxx @@ -87,15 +87,14 @@ class SvxSingleNumPickTabPage : public SfxTabPage using TabPage::ActivatePage; using TabPage::DeactivatePage; - VclPtr<SvxNumValueSet> m_pExamplesVS; + VclPtr<SvxNumValueSet> m_pExamplesVS; SvxNumSettingsArr_Impl aNumSettingsArr; SvxNumRule* pActNum; SvxNumRule* pSaveNum; - sal_uInt16 nActNumLvl; + sal_uInt16 nActNumLvl; bool bModified : 1; bool bPreset : 1; - OUString sNumCharFmtName; sal_uInt16 nNumItemId; protected: @@ -209,7 +208,6 @@ class SvxBitmapPickTabPage : public SfxTabPage VclPtr<Button> m_pBtBrowseFile; std::vector<OUString> aGrfNames; - OUString sNumCharFmtName; SvxNumRule* pActNum; SvxNumRule* pSaveNum; diff --git a/cui/source/tabpages/numpages.cxx b/cui/source/tabpages/numpages.cxx index a50dba3c49e4..0a3f2857c39c 100644 --- a/cui/source/tabpages/numpages.cxx +++ b/cui/source/tabpages/numpages.cxx @@ -349,7 +349,7 @@ IMPL_LINK_NOARG(SvxSingleNumPickTabPage, NumSelectHdl_Impl, ValueSet*, void) aFmt.SetSuffix( "" ); else aFmt.SetSuffix(_pSet->sSuffix); - aFmt.SetCharFormatName(sNumCharFmtName); + aFmt.SetCharFormatName(""); aFmt.SetBulletRelSize(100); pActNum->SetLevel(i, aFmt); } @@ -957,7 +957,7 @@ IMPL_LINK_NOARG(SvxBitmapPickTabPage, NumSelectHdl_Impl, ValueSet*, void) aFmt.SetNumberingType(SVX_NUM_BITMAP); aFmt.SetPrefix( "" ); aFmt.SetSuffix( "" ); - aFmt.SetCharFormatName( sNumCharFmtName ); + aFmt.SetCharFormatName( "" ); Graphic aGraphic; if(GalleryExplorer::GetGraphicObj( GALLERY_THEME_BULLETS, nIdx, &aGraphic)) commit 6c5fdc30851f261ad8f70cb23ead82c4cbe46bb8 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 14:02:32 2017 +0200 unused m_nFetchSize in OResultSet since initial commit 77e4adbe4759f21f6cf095d954391a9ddbbec2cf Date: Mon Sep 17 22:50:49 2012 +0200 mork driver implementation Change-Id: I226c4065a5ab4404cc9e99eb3e026d153d1ccf08 diff --git a/connectivity/source/drivers/mork/MResultSet.cxx b/connectivity/source/drivers/mork/MResultSet.cxx index b3bf3b790fc4..c510fff3255f 100644 --- a/connectivity/source/drivers/mork/MResultSet.cxx +++ b/connectivity/source/drivers/mork/MResultSet.cxx @@ -71,7 +71,6 @@ OResultSet::OResultSet(OCommonStatement* pStmt, const std::shared_ptr< connectiv ,m_xMetaData(nullptr) ,m_nRowPos(0) ,m_bWasNull(false) - ,m_nFetchSize(0) ,m_nResultSetType(ResultSetType::SCROLL_INSENSITIVE) ,m_nFetchDirection(FetchDirection::FORWARD) ,m_pSQLIterator( _pSQLIterator ) @@ -607,7 +606,7 @@ void OResultSet::getFastPropertyValue( rValue <<= m_nFetchDirection; break; case PROPERTY_ID_FETCHSIZE: - rValue <<= m_nFetchSize; + rValue <<= (sal_Int32)0; break; case PROPERTY_ID_ISBOOKMARKABLE: const_cast< OResultSet* >( this )->determineReadOnly(); diff --git a/connectivity/source/drivers/mork/MResultSet.hxx b/connectivity/source/drivers/mork/MResultSet.hxx index febe6a0a4bf7..1a3c8f9d7598 100644 --- a/connectivity/source/drivers/mork/MResultSet.hxx +++ b/connectivity/source/drivers/mork/MResultSet.hxx @@ -79,7 +79,6 @@ namespace connectivity css::uno::Reference< css::sdbc::XResultSetMetaData> m_xMetaData; sal_uInt32 m_nRowPos; bool m_bWasNull; - sal_Int32 m_nFetchSize; sal_Int32 m_nResultSetType; sal_Int32 m_nFetchDirection; commit 0c9f104390032d99517e9bd1e16333fcc5fa472f Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 13:56:06 2017 +0200 unused bPageToggle in GraphicImport_Impl ever since commit 0929dfa83ca4dbc675c74854566ce4e25def0dbd Date: Sat Jan 11 11:54:14 2014 +0100 writerfilter: drop never generated rtf {XAlign,FHDR,YAlign,XRelTo,YRelTo} Change-Id: Icabe9ff848e3cc9918741e9c68d8f2312145fb74 diff --git a/writerfilter/source/dmapper/GraphicImport.cxx b/writerfilter/source/dmapper/GraphicImport.cxx index be000432868a..553a4e86a8f5 100644 --- a/writerfilter/source/dmapper/GraphicImport.cxx +++ b/writerfilter/source/dmapper/GraphicImport.cxx @@ -191,7 +191,6 @@ public: sal_Int16 nHoriOrient; sal_Int16 nHoriRelation; - bool bPageToggle; sal_Int16 nVertOrient; sal_Int16 nVertRelation; text::WrapTextMode nWrap; @@ -257,7 +256,6 @@ public: ,zOrder(-1) ,nHoriOrient( text::HoriOrientation::NONE ) ,nHoriRelation( text::RelOrientation::FRAME ) - ,bPageToggle( false ) ,nVertOrient( text::VertOrientation::NONE ) ,nVertRelation( text::RelOrientation::FRAME ) ,nWrap(text::WrapTextMode_NONE) @@ -345,7 +343,7 @@ public: xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_HORI_ORIENT_RELATION ), uno::makeAny(nHoriRelation)); xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_PAGE_TOGGLE ), - uno::makeAny(bPageToggle)); + uno::makeAny(false)); if (!bRelativeOnly) xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_VERT_ORIENT_POSITION), uno::makeAny(nTopPosition)); commit beebcb8ee0b47172978a23d8d555859daf0f4474 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 13:54:41 2017 +0200 unused bVertFlip/bHoriFlip in GraphicImport_Impl ever since commit 4a924576e415f16e0571542bb0d683529f9046ff Date: Wed Jan 15 20:24:41 2014 +0100 writerfilter: drop unused BlipDib and FSP in doctok Change-Id: I9bf644bdc4b37cb6c4a9a9ab7757c4a83a520cd7 diff --git a/writerfilter/source/dmapper/GraphicImport.cxx b/writerfilter/source/dmapper/GraphicImport.cxx index 65a20747c38f..be000432868a 100644 --- a/writerfilter/source/dmapper/GraphicImport.cxx +++ b/writerfilter/source/dmapper/GraphicImport.cxx @@ -225,9 +225,6 @@ public: bool bIsGraphic; - bool bHoriFlip; - bool bVertFlip; - bool bSizeProtected; bool bPositionProtected; @@ -284,8 +281,6 @@ public: ,eColorMode( drawing::ColorMode_STANDARD ) ,nCurrentBorderLine(BORDER_TOP) ,bIsGraphic(false) - ,bHoriFlip(false) - ,bVertFlip(false) ,bSizeProtected(false) ,bPositionProtected(false) ,nShapeOptionType(0) @@ -1293,17 +1288,7 @@ uno::Reference< text::XTextContent > GraphicImport::createGraphicObject( const b if(m_pImpl->fGamma > 0. ) xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_GAMMA ), uno::makeAny(m_pImpl->fGamma )); - if(m_pImpl->bHoriFlip) - { - xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_HORI_MIRRORED_ON_EVEN_PAGES ), - uno::makeAny( m_pImpl->bHoriFlip )); - xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_HORI_MIRRORED_ON_ODD_PAGES ), - uno::makeAny( m_pImpl->bHoriFlip )); - } - if( m_pImpl->bVertFlip ) - xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_VERT_MIRRORED ), - uno::makeAny( m_pImpl->bVertFlip )); xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_BACK_COLOR ), uno::makeAny( m_pImpl->nFillColor )); commit 1c0cbc87f23f228f66dc22428cddd7e395c1119a Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 13:50:10 2017 +0200 unused m_xStorage in RTFDocumentImpl ever since introduction in commit 328d154015f8cb733ed62d793bbbc31eb4cc971b Date: Fri Jun 10 18:56:08 2011 +0200 resolvePict: import the stream as a graphic object Change-Id: Ife03094d721621cc592be9cd94899a4e40550330 diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.cxx b/writerfilter/source/rtftok/rtfdocumentimpl.cxx index f9afa8acfe44..59ea16061a84 100644 --- a/writerfilter/source/rtftok/rtfdocumentimpl.cxx +++ b/writerfilter/source/rtftok/rtfdocumentimpl.cxx @@ -301,7 +301,7 @@ RTFDocumentImpl::RTFDocumentImpl(uno::Reference<uno::XComponentContext> const& x if (xDocumentPropertiesSupplier.is()) m_xDocumentProperties.set(xDocumentPropertiesSupplier->getDocumentProperties(), uno::UNO_QUERY); - m_pGraphicHelper.reset(new oox::GraphicHelper(m_xContext, xFrame, m_xStorage)); + m_pGraphicHelper.reset(new oox::GraphicHelper(m_xContext, xFrame, oox::StorageRef())); m_pTokenizer.reset(new RTFTokenizer(*this, m_pInStream.get(), m_xStatusIndicator)); m_pSdrImport.reset(new RTFSdrImport(*this, m_xDstDoc)); diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.hxx b/writerfilter/source/rtftok/rtfdocumentimpl.hxx index c2834268499e..8756e5fd071c 100644 --- a/writerfilter/source/rtftok/rtfdocumentimpl.hxx +++ b/writerfilter/source/rtftok/rtfdocumentimpl.hxx @@ -551,7 +551,6 @@ private: /// The settings table sprms. RTFSprms m_aSettingsTableSprms; - oox::StorageRef m_xStorage; std::shared_ptr<oox::GraphicHelper> m_pGraphicHelper; /// cell props buffer for nested tables, reset by \nestrow commit 107524b99f087af9bea02e2d4e61a04a8c3ecbe0 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 13:36:41 2017 +0200 unused aCId in ScHTMLExport ever since commit fa135fd0e05fc4ba784b4349d65f2e5ed26c0f55 Date: Tue May 31 10:30:35 2016 +0200 remove unused SID constants and associated code Change-Id: I47388fa5dcfec67bbcd9f7bf0a079d9376bc05af diff --git a/sc/source/filter/html/htmlexp.cxx b/sc/source/filter/html/htmlexp.cxx index b1ef63c92c2b..18361ebe8035 100644 --- a/sc/source/filter/html/htmlexp.cxx +++ b/sc/source/filter/html/htmlexp.cxx @@ -593,8 +593,6 @@ void ScHTMLExport::WriteBody() aGrfNm = URIHelper::SmartRel2Abs( INetURLObject(aBaseURL), aGrfNm, URIHelper::GetMaybeFileHdl()); - if ( HasCId() ) - MakeCIdURL( aGrfNm ); aLink = aGrfNm; } } @@ -602,11 +600,9 @@ void ScHTMLExport::WriteBody() else { aGrfNm = aLink; - if( bCopyLocalFileToINet || HasCId() ) + if( bCopyLocalFileToINet ) { CopyLocalFileToINet( aGrfNm, aStreamPath ); - if ( HasCId() ) - MakeCIdURL( aGrfNm ); } else aGrfNm = URIHelper::SmartRel2Abs( @@ -1319,21 +1315,6 @@ void ScHTMLExport::CopyLocalFileToINet( OUString& rFileNm, } } -void ScHTMLExport::MakeCIdURL( OUString& rURL ) -{ - if( aCId.isEmpty() ) - return; - - INetURLObject aURLObj( rURL ); - if( INetProtocol::File != aURLObj.GetProtocol() ) - return; - - OUString aLastName( aURLObj.GetLastName().toAsciiLowerCase() ); - OSL_ENSURE( !aLastName.isEmpty(), "filename without length!" ); - - rURL = "cid:" + aLastName + "." + aCId; -} - void ScHTMLExport::IncIndent( short nVal ) { sIndent[nIndent] = '\t'; diff --git a/sc/source/filter/html/htmlexp2.cxx b/sc/source/filter/html/htmlexp2.cxx index 493a4ddced41..f9698694633c 100644 --- a/sc/source/filter/html/htmlexp2.cxx +++ b/sc/source/filter/html/htmlexp2.cxx @@ -195,19 +195,15 @@ void ScHTMLExport::WriteImage( OUString& rLinkName, const Graphic& rGrf, INetURLObject(aBaseURL), aGrfNm, URIHelper::GetMaybeFileHdl()); - if ( HasCId() ) - MakeCIdURL( rLinkName ); } } } else { // Linked graphic - figure out the URL for the IMG tag - if( bCopyLocalFileToINet || HasCId() ) + if( bCopyLocalFileToINet ) { CopyLocalFileToINet( rLinkName, aStreamPath ); - if ( HasCId() ) - MakeCIdURL( rLinkName ); } else rLinkName = URIHelper::SmartRel2Abs( diff --git a/sc/source/filter/inc/htmlexp.hxx b/sc/source/filter/inc/htmlexp.hxx index e685a9f13d4d..011b0d30b650 100644 --- a/sc/source/filter/inc/htmlexp.hxx +++ b/sc/source/filter/inc/htmlexp.hxx @@ -108,7 +108,6 @@ class ScHTMLExport : public ScExportBase ScHTMLStyle aHTMLStyle; OUString aBaseURL; OUString aStreamPath; - OUString aCId; // Content-Id for Mail-Export VclPtr<OutputDevice> pAppWin; // for Pixel-work FileNameMapPtr pFileNameMap; // for CopyLocalFileToINet OUString aNonConvertibleChars; // collect nonconvertible characters @@ -144,11 +143,6 @@ class ScHTMLExport : public ScExportBase // copy a local file to internet if needed void CopyLocalFileToINet( OUString& rFileNm, const OUString& rTargetNm ); - bool HasCId() - { - return !aCId.isEmpty(); - } - void MakeCIdURL( OUString& rURL ); void PrepareGraphics( ScDrawLayer*, SCTAB nTab, SCCOL nStartCol, SCROW nStartRow, commit 4819162e452ff96f2a2e652a48a7bc3bfa3bc400 Author: Noel Grandin <noel.gran...@collabora.co.uk> Date: Tue Sep 19 11:46:59 2017 +0200 simplify calling listeners Change-Id: I4bf0d378003d1f54685f6c55c0314c117b243526 diff --git a/sc/source/core/data/table3.cxx b/sc/source/core/data/table3.cxx index 3b1a1c393a9b..64234dbd9781 100644 --- a/sc/source/core/data/table3.cxx +++ b/sc/source/core/data/table3.cxx @@ -665,30 +665,6 @@ public: } }; -class StartListeningNotifier -{ - sc::RefStartListeningHint maHint; -public: - StartListeningNotifier() {} - - void operator() ( SvtListener* p ) - { - p->Notify(maHint); - } -}; - -class StopListeningNotifier -{ - sc::RefStopListeningHint maHint; -public: - StopListeningNotifier() {} - - void operator() ( SvtListener* p ) - { - p->Notify(maHint); - } -}; - class FormulaGroupPosCollector { sc::RefQueryFormulaGroup& mrQuery; @@ -933,8 +909,9 @@ void ScTable::SortReorderByColumn( /* TODO: for performance this could be enhanced to stop and later * restart only listening to within the reordered range and keep * listening to everything outside untouched. */ - StopListeningNotifier aFunc; - std::for_each(aCellListeners.begin(), aCellListeners.end(), aFunc); + sc::RefStopListeningHint aHint; + for (auto const & l : aCellListeners) + l->Notify(aHint); } // table to keep track of column index to position in the index table. @@ -1024,8 +1001,9 @@ void ScTable::SortReorderByColumn( else // !(pArray->IsUpdateRefs()) { // Notify the cells' listeners to (re-)start listening. - StartListeningNotifier aFunc; - std::for_each(aCellListeners.begin(), aCellListeners.end(), aFunc); + sc::RefStartListeningHint aHint; + for (auto const & l : aCellListeners) + l->Notify(aHint); } // Re-join formulas at row boundaries now that all the references have @@ -1079,8 +1057,9 @@ void ScTable::SortReorderByRow( * restart only listening to within the reordered range and keep * listening to everything outside untouched. */ { - StopListeningNotifier aFunc; - std::for_each(aCellListeners.begin(), aCellListeners.end(), aFunc); + sc::RefStopListeningHint aHint; + for (auto const & l : aCellListeners) + l->Notify(aHint); } // Split formula groups at the sort range boundaries (if applicable). @@ -1172,8 +1151,9 @@ void ScTable::SortReorderByRow( // Notify the cells' listeners to (re-)start listening. { - StartListeningNotifier aFunc; - std::for_each(aCellListeners.begin(), aCellListeners.end(), aFunc); + sc::RefStartListeningHint aHint; + for (auto const & l : aCellListeners) + l->Notify(aHint); } // Re-group columns in the sorted range too. _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits