diff options
-rw-r--r-- | cui/source/factory/dlgfact.cxx | 3 | ||||
-rw-r--r-- | cui/source/factory/dlgfact.hxx | 3 | ||||
-rw-r--r-- | include/oox/export/drawingml.hxx | 2 | ||||
-rw-r--r-- | include/oox/helper/binaryinputstream.hxx | 7 | ||||
-rw-r--r-- | include/sfx2/dinfdlg.hxx | 2 | ||||
-rw-r--r-- | include/sfx2/filedlghelper.hxx | 1 | ||||
-rw-r--r-- | include/sfx2/mgetempl.hxx | 3 | ||||
-rw-r--r-- | include/svx/svxdlg.hxx | 3 | ||||
-rw-r--r-- | linguistic/source/misc2.cxx | 10 | ||||
-rw-r--r-- | oox/source/export/drawingml.cxx | 4 | ||||
-rw-r--r-- | oox/source/helper/binaryinputstream.cxx | 6 | ||||
-rw-r--r-- | sc/source/filter/oox/biffhelper.cxx | 2 | ||||
-rw-r--r-- | sfx2/source/appl/appopen.cxx | 2 | ||||
-rw-r--r-- | sfx2/source/config/evntconf.cxx | 2 | ||||
-rw-r--r-- | sfx2/source/dialog/dinfdlg.cxx | 9 | ||||
-rw-r--r-- | sfx2/source/dialog/filedlghelper.cxx | 3 | ||||
-rw-r--r-- | sfx2/source/dialog/mgetempl.cxx | 9 | ||||
-rw-r--r-- | sfx2/source/inc/eventsupplier.hxx | 3 | ||||
-rw-r--r-- | sfx2/source/notify/eventsupplier.cxx | 7 | ||||
-rw-r--r-- | svx/source/dialog/hdft.cxx | 3 |
20 files changed, 34 insertions, 50 deletions
diff --git a/cui/source/factory/dlgfact.cxx b/cui/source/factory/dlgfact.cxx index 5f0bc3f9751b..e5306a261f1e 100644 --- a/cui/source/factory/dlgfact.cxx +++ b/cui/source/factory/dlgfact.cxx @@ -1038,13 +1038,12 @@ VclPtr<AbstractSvxSearchSimilarityDialog> AbstractDialogFactory_Impl::CreateSvxS VclPtr<SfxAbstractTabDialog> AbstractDialogFactory_Impl::CreateSvxBorderBackgroundDlg( vcl::Window* pParent, const SfxItemSet& rCoreSet, - bool bEnableSelector, bool bEnableDrawingLayerFillStyles) { VclPtrInstance<SvxBorderBackgroundDlg> pDlg( pParent, rCoreSet, - bEnableSelector, + /*bEnableSelector*/true, bEnableDrawingLayerFillStyles); return VclPtr<CuiAbstractTabDialog_Impl>::Create(pDlg); diff --git a/cui/source/factory/dlgfact.hxx b/cui/source/factory/dlgfact.hxx index 635e553c0552..d36ecabeb517 100644 --- a/cui/source/factory/dlgfact.hxx +++ b/cui/source/factory/dlgfact.hxx @@ -519,8 +519,7 @@ public: virtual VclPtr<SfxAbstractTabDialog> CreateSvxBorderBackgroundDlg( vcl::Window* pParent, const SfxItemSet& rCoreSet, - bool bEnableSelector, - bool bEnableDrawingLayerFillStyles = false) override; + bool bEnableDrawingLayerFillStyles) override; virtual VclPtr<AbstractSvxTransformTabDialog> CreateSvxTransformTabDialog( vcl::Window* pParent, const SfxItemSet* pAttr, diff --git a/include/oox/export/drawingml.hxx b/include/oox/export/drawingml.hxx index 33c301285a27..115f6d46148c 100644 --- a/include/oox/export/drawingml.hxx +++ b/include/oox/export/drawingml.hxx @@ -216,7 +216,7 @@ public: void WriteLinespacing( const css::style::LineSpacing& rLineSpacing ); OUString WriteBlip( const css::uno::Reference< css::beans::XPropertySet >& rXPropSet, - const OUString& rURL, bool bRelPathToMedia, const Graphic *pGraphic=nullptr ); + const OUString& rURL, bool bRelPathToMedia ); OUString WriteXGraphicBlip(css::uno::Reference<css::beans::XPropertySet> const & rXPropSet, css::uno::Reference<css::graphic::XGraphic> const & rxGraphic, diff --git a/include/oox/helper/binaryinputstream.hxx b/include/oox/helper/binaryinputstream.hxx index c6b59459f0d8..1b0a7e46918b 100644 --- a/include/oox/helper/binaryinputstream.hxx +++ b/include/oox/helper/binaryinputstream.hxx @@ -163,15 +163,12 @@ public: OUString readCharArrayUC( sal_Int32 nChars, rtl_TextEncoding eTextEnc ); /** Reads a Unicode character array and returns the string. + NUL characters are replaced by question marks (default). @param nChars Number of 16-bit characters to read from the stream. - - @param bAllowNulChars - True = NUL characters are inserted into the imported string. - False = NUL characters are replaced by question marks (default). */ - OUString readUnicodeArray( sal_Int32 nChars, bool bAllowNulChars = false ); + OUString readUnicodeArray( sal_Int32 nChars ); /** Reads a Unicode character array (may be compressed) and returns the string. diff --git a/include/sfx2/dinfdlg.hxx b/include/sfx2/dinfdlg.hxx index 40b8b91c4340..4fa1e321931b 100644 --- a/include/sfx2/dinfdlg.hxx +++ b/include/sfx2/dinfdlg.hxx @@ -480,7 +480,7 @@ public: virtual ~CustomPropertiesControl() override; virtual void dispose() override; - void AddLine(const OUString& sName, css::uno::Any const & rAny, bool bInteractive); + void AddLine(css::uno::Any const & rAny); bool AreAllLinesValid() const { return m_pPropertiesWin->AreAllLinesValid(); } void ClearAllLines() { m_pPropertiesWin->ClearAllLines(); } diff --git a/include/sfx2/filedlghelper.hxx b/include/sfx2/filedlghelper.hxx index 2ea5579ebe0c..5fe1cbc29787 100644 --- a/include/sfx2/filedlghelper.hxx +++ b/include/sfx2/filedlghelper.hxx @@ -246,7 +246,6 @@ public: ErrCode FileOpenDialog_Impl( const vcl::Window* pParent, sal_Int16 nDialogType, FileDialogFlags nFlags, - const OUString& rFact, std::vector<OUString>& rpURLList, OUString& rFilter, SfxItemSet *& rpSet, diff --git a/include/sfx2/mgetempl.hxx b/include/sfx2/mgetempl.hxx index 42aeef6c1135..c9c1ec6549f8 100644 --- a/include/sfx2/mgetempl.hxx +++ b/include/sfx2/mgetempl.hxx @@ -86,8 +86,7 @@ friend class SfxStyleDialog; virtual bool FillItemSet(SfxItemSet *) override; virtual void Reset(const SfxItemSet *) override; - static bool Execute_Impl( sal_uInt16 nId, const OUString& rStr, const OUString& rRefStr, - sal_uInt16 nFamily ); + static bool Execute_Impl( sal_uInt16 nId, const OUString& rStr, sal_uInt16 nFamily ); using TabPage::ActivatePage; virtual void ActivatePage(const SfxItemSet &) override; using TabPage::DeactivatePage; diff --git a/include/svx/svxdlg.hxx b/include/svx/svxdlg.hxx index 3c38f7050886..993ef8d76ee5 100644 --- a/include/svx/svxdlg.hxx +++ b/include/svx/svxdlg.hxx @@ -380,8 +380,7 @@ public: virtual VclPtr<SfxAbstractTabDialog> CreateSvxBorderBackgroundDlg( vcl::Window* pParent, const SfxItemSet& rCoreSet, - bool bEnableSelector, - bool bEnableDrawingLayerFillStyles = false) = 0; + bool bEnableDrawingLayerFillStyles) = 0; virtual VclPtr<AbstractSvxTransformTabDialog> CreateSvxTransformTabDialog( vcl::Window* pParent, const SfxItemSet* pAttr, diff --git a/linguistic/source/misc2.cxx b/linguistic/source/misc2.cxx index 2cf31ac51dff..782d190e7a1f 100644 --- a/linguistic/source/misc2.cxx +++ b/linguistic/source/misc2.cxx @@ -41,15 +41,15 @@ using namespace com::sun::star; /// @see GetDictionaryPaths enum class DictionaryPathFlags { + NONE = 0x00, INTERNAL = 0x01, USER = 0x02, - WRITABLE = 0x04 }; namespace o3tl { - template<> struct typed_flags<DictionaryPathFlags> : is_typed_flags<DictionaryPathFlags, 0x07> {}; + template<> struct typed_flags<DictionaryPathFlags> : is_typed_flags<DictionaryPathFlags, 0x03> {}; } -#define PATH_FLAG_ALL (DictionaryPathFlags::INTERNAL | DictionaryPathFlags::USER | DictionaryPathFlags::WRITABLE) +#define PATH_FLAG_ALL (DictionaryPathFlags::INTERNAL | DictionaryPathFlags::USER) namespace linguistic { @@ -112,7 +112,7 @@ static std::vector< OUString > GetMultiPaths_Impl( ++nMaxEntries; aRes.resize( nMaxEntries ); sal_Int32 nCount = 0; // number of actually added entries - if ((nPathFlags & DictionaryPathFlags::WRITABLE) && !aWritablePath.isEmpty()) + if (!aWritablePath.isEmpty()) aRes[ nCount++ ] = aWritablePath; for (int i = 0; i < 2; ++i) { @@ -134,7 +134,7 @@ static std::vector< OUString > GetMultiPaths_Impl( OUString GetDictionaryWriteablePath() { - std::vector< OUString > aPaths( GetMultiPaths_Impl( "Dictionary", DictionaryPathFlags::WRITABLE ) ); + std::vector< OUString > aPaths( GetMultiPaths_Impl( "Dictionary", DictionaryPathFlags::NONE ) ); DBG_ASSERT( aPaths.size() == 1, "Dictionary_writable path corrupted?" ); OUString aRes; if (aPaths.size() > 0) diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx index f462fa13d32a..0a41a7ddfd4a 100644 --- a/oox/source/export/drawingml.cxx +++ b/oox/source/export/drawingml.cxx @@ -1086,7 +1086,7 @@ void DrawingML::WriteImageBrightnessContrastTransparence(uno::Reference<beans::X } } -OUString DrawingML::WriteBlip( const Reference< XPropertySet >& rXPropSet, const OUString& rURL, bool bRelPathToMedia, const Graphic *pGraphic ) +OUString DrawingML::WriteBlip( const Reference< XPropertySet >& rXPropSet, const OUString& rURL, bool bRelPathToMedia ) { OUString sRelId; BitmapChecksum nChecksum = 0; @@ -1101,7 +1101,7 @@ OUString DrawingML::WriteBlip( const Reference< XPropertySet >& rXPropSet, const } if (sRelId.isEmpty()) { - sRelId = pGraphic ? WriteImage( *pGraphic, bRelPathToMedia ) : WriteImage( rURL, bRelPathToMedia ); + sRelId = WriteImage( rURL, bRelPathToMedia ); if (!rURL.isEmpty() && mpTextExport) mpTextExport->CacheRelId(nChecksum, sRelId); } diff --git a/oox/source/helper/binaryinputstream.cxx b/oox/source/helper/binaryinputstream.cxx index 6eb5a6ed75c5..bc060027e7cf 100644 --- a/oox/source/helper/binaryinputstream.cxx +++ b/oox/source/helper/binaryinputstream.cxx @@ -74,7 +74,7 @@ OUString BinaryInputStream::readCharArrayUC( sal_Int32 nChars, rtl_TextEncoding return OStringToOUString( readCharArray( nChars ), eTextEnc ); } -OUString BinaryInputStream::readUnicodeArray( sal_Int32 nChars, bool bAllowNulChars ) +OUString BinaryInputStream::readUnicodeArray( sal_Int32 nChars ) { if( nChars <= 0 ) return OUString(); @@ -85,8 +85,8 @@ OUString BinaryInputStream::readUnicodeArray( sal_Int32 nChars, bool bAllowNulCh return OUString(); aBuffer.resize( static_cast< size_t >( nCharsRead ) ); - if( !bAllowNulChars ) - ::std::replace( aBuffer.begin(), aBuffer.begin() + nCharsRead, '\0', '?' ); + // don't allow nul chars + ::std::replace( aBuffer.begin(), aBuffer.begin() + nCharsRead, '\0', '?' ); OUStringBuffer aStringBuffer; aStringBuffer.ensureCapacity( nCharsRead ); diff --git a/sc/source/filter/oox/biffhelper.cxx b/sc/source/filter/oox/biffhelper.cxx index d94f4d301954..3dbe1cda4535 100644 --- a/sc/source/filter/oox/biffhelper.cxx +++ b/sc/source/filter/oox/biffhelper.cxx @@ -102,7 +102,7 @@ union DecodedDouble { // SequenceInputStream always supports getRemaining() nCharCount = ::std::min( nCharCount, static_cast< sal_Int32 >( rStrm.getRemaining() / 2 ) ); - aString = rStrm.readUnicodeArray( nCharCount, false/*bAllowNulChars*/ ); + aString = rStrm.readUnicodeArray( nCharCount ); } } return aString; diff --git a/sfx2/source/appl/appopen.cxx b/sfx2/source/appl/appopen.cxx index ef14716ebe2e..e1cf3a6944a3 100644 --- a/sfx2/source/appl/appopen.cxx +++ b/sfx2/source/appl/appopen.cxx @@ -651,7 +651,7 @@ void SfxApplication::OpenDocExec_Impl( SfxRequest& rReq ) ErrCode nErr = sfx2::FileOpenDialog_Impl(GetTopWindow(), nDialogType, - eDialogFlags, OUString(), aURLList, + eDialogFlags, aURLList, aFilter, pSet, &aPath, nDialog, sStandardDir, aBlackList); if ( nErr == ERRCODE_ABORT ) diff --git a/sfx2/source/config/evntconf.cxx b/sfx2/source/config/evntconf.cxx index 9de6fb043487..0712e7cda534 100644 --- a/sfx2/source/config/evntconf.cxx +++ b/sfx2/source/config/evntconf.cxx @@ -244,7 +244,7 @@ void SfxEventConfiguration::ConfigureEvent( const OUString& aName, const SvxMacr SvxMacro* SfxEventConfiguration::ConvertToMacro( const css::uno::Any& rElement, SfxObjectShell* pDoc ) { - return SfxEvents_Impl::ConvertToMacro( rElement, pDoc, true/*bBlowUp*/ ); + return SfxEvents_Impl::ConvertToMacro( rElement, pDoc ); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/dialog/dinfdlg.cxx b/sfx2/source/dialog/dinfdlg.cxx index d2e4bc8bf71a..69847ed42692 100644 --- a/sfx2/source/dialog/dinfdlg.cxx +++ b/sfx2/source/dialog/dinfdlg.cxx @@ -2124,12 +2124,12 @@ IMPL_LINK_NOARG(CustomPropertiesControl, RemovedHdl, void*, void) m_pVertScroll->DoScrollAction ( ScrollType::LineUp ); } -void CustomPropertiesControl::AddLine( const OUString& sName, Any const & rAny, bool bInteractive ) +void CustomPropertiesControl::AddLine( Any const & rAny ) { - m_pPropertiesWin->AddLine( sName, rAny ); + m_pPropertiesWin->AddLine( OUString(), rAny ); long nLineCount = m_pPropertiesWin->GetTotalLineCount(); m_pVertScroll->SetRangeMax(nLineCount + 1); - if ( bInteractive && m_pPropertiesWin->GetOutputSizePixel().Height() < nLineCount * m_pPropertiesWin->GetLineHeight() ) + if ( m_pPropertiesWin->GetOutputSizePixel().Height() < nLineCount * m_pPropertiesWin->GetLineHeight() ) m_pVertScroll->DoScroll(nLineCount + 1); } @@ -2162,8 +2162,7 @@ void SfxCustomPropertiesPage::dispose() IMPL_LINK_NOARG(SfxCustomPropertiesPage, AddHdl, Button*, void) { - Any aAny; - m_pPropertiesCtrl->AddLine( OUString(), aAny, true ); + m_pPropertiesCtrl->AddLine( Any() ); } bool SfxCustomPropertiesPage::FillItemSet( SfxItemSet* rSet ) diff --git a/sfx2/source/dialog/filedlghelper.cxx b/sfx2/source/dialog/filedlghelper.cxx index 33404ae1ca2d..ea9cb326f8ee 100644 --- a/sfx2/source/dialog/filedlghelper.cxx +++ b/sfx2/source/dialog/filedlghelper.cxx @@ -2633,7 +2633,6 @@ void FileDialogHelper::DialogClosed( const DialogClosedEvent& _rEvent ) ErrCode FileOpenDialog_Impl( const vcl::Window* pParent, sal_Int16 nDialogType, FileDialogFlags nFlags, - const OUString& rFact, std::vector<OUString>& rpURLList, OUString& rFilter, SfxItemSet *& rpSet, @@ -2650,7 +2649,7 @@ ErrCode FileOpenDialog_Impl( const vcl::Window* pParent, if (nFlags & FileDialogFlags::SignPDF) pDialog.reset(new FileDialogHelper(nDialogType, nFlags, SfxResId(STR_SFX_FILTERNAME_PDF), "pdf", rStandardDir, rBlackList, pParent)); else - pDialog.reset(new FileDialogHelper(nDialogType, nFlags, rFact, nDialog, SfxFilterFlags::NONE, SfxFilterFlags::NONE, rStandardDir, rBlackList, pParent)); + pDialog.reset(new FileDialogHelper(nDialogType, nFlags, OUString(), nDialog, SfxFilterFlags::NONE, SfxFilterFlags::NONE, rStandardDir, rBlackList, pParent)); OUString aPath; if ( pPath ) diff --git a/sfx2/source/dialog/mgetempl.cxx b/sfx2/source/dialog/mgetempl.cxx index 6268c5d61ace..bc40c3e83893 100644 --- a/sfx2/source/dialog/mgetempl.cxx +++ b/sfx2/source/dialog/mgetempl.cxx @@ -348,7 +348,7 @@ IMPL_LINK_NOARG( SfxManageStyleSheetPage, EditStyleSelectHdl_Impl, ListBox&, voi IMPL_LINK_NOARG( SfxManageStyleSheetPage, EditStyleHdl_Impl, Button*, void ) { OUString aTemplName(m_pFollowLb->GetSelectedEntry()); - Execute_Impl(SID_STYLE_EDIT, aTemplName, OUString(), static_cast<sal_uInt16>(pStyle->GetFamily())); + Execute_Impl(SID_STYLE_EDIT, aTemplName, static_cast<sal_uInt16>(pStyle->GetFamily())); } IMPL_LINK_NOARG( SfxManageStyleSheetPage, EditLinkStyleSelectHdl_Impl, ListBox&, void ) @@ -364,25 +364,22 @@ IMPL_LINK_NOARG( SfxManageStyleSheetPage, EditLinkStyleHdl_Impl, Button*, void ) { OUString aTemplName(m_pBaseLb->GetSelectedEntry()); if (aTemplName != SfxResId(STR_NONE)) - Execute_Impl( SID_STYLE_EDIT, aTemplName, OUString(),static_cast<sal_uInt16>(pStyle->GetFamily()) ); + Execute_Impl( SID_STYLE_EDIT, aTemplName, static_cast<sal_uInt16>(pStyle->GetFamily()) ); } // Internal: Perform functions through the Dispatcher bool SfxManageStyleSheetPage::Execute_Impl( - sal_uInt16 nId, const OUString &rStr, const OUString& rRefStr, sal_uInt16 nFamily) + sal_uInt16 nId, const OUString &rStr, sal_uInt16 nFamily) { SfxDispatcher &rDispatcher = *SfxGetpApp()->GetDispatcher_Impl(); SfxStringItem aItem(nId, rStr); SfxUInt16Item aFamily(SID_STYLE_FAMILY, nFamily); - SfxStringItem aRefName( SID_STYLE_REFERENCE, rRefStr ); const SfxPoolItem* pItems[ 6 ]; sal_uInt16 nCount = 0; if( !rStr.isEmpty() ) pItems[ nCount++ ] = &aItem; pItems[ nCount++ ] = &aFamily; - if ( !rRefStr.isEmpty() ) - pItems[ nCount++ ] = &aRefName; pItems[ nCount++ ] = nullptr; diff --git a/sfx2/source/inc/eventsupplier.hxx b/sfx2/source/inc/eventsupplier.hxx index 7a5706de907b..3fdd009bcabd 100644 --- a/sfx2/source/inc/eventsupplier.hxx +++ b/sfx2/source/inc/eventsupplier.hxx @@ -79,7 +79,8 @@ public: // --- ::lang::XEventListener --- virtual void SAL_CALL disposing( const css::lang::EventObject& Source ) override; - static SvxMacro* ConvertToMacro( const css::uno::Any& rElement, SfxObjectShell* pDoc, bool bNormalizeMacro ); + // convert and normalize + static SvxMacro* ConvertToMacro( const css::uno::Any& rElement, SfxObjectShell* pDoc ); static void NormalizeMacro( const css::uno::Any& rIn, css::uno::Any& rOut, SfxObjectShell* pDoc ); static void NormalizeMacro( const ::comphelper::NamedValueCollection& i_eventDescriptor, diff --git a/sfx2/source/notify/eventsupplier.cxx b/sfx2/source/notify/eventsupplier.cxx index 893a2d284455..612f57b02c1b 100644 --- a/sfx2/source/notify/eventsupplier.cxx +++ b/sfx2/source/notify/eventsupplier.cxx @@ -330,15 +330,12 @@ SfxEvents_Impl::~SfxEvents_Impl() } -SvxMacro* SfxEvents_Impl::ConvertToMacro( const uno::Any& rElement, SfxObjectShell* pObjShell, bool bNormalizeMacro ) +SvxMacro* SfxEvents_Impl::ConvertToMacro( const uno::Any& rElement, SfxObjectShell* pObjShell ) { SvxMacro* pMacro = nullptr; uno::Sequence < beans::PropertyValue > aProperties; uno::Any aAny; - if ( bNormalizeMacro ) - NormalizeMacro( rElement, aAny, pObjShell ); - else - aAny = rElement; + NormalizeMacro( rElement, aAny, pObjShell ); if ( aAny >>= aProperties ) { diff --git a/svx/source/dialog/hdft.cxx b/svx/source/dialog/hdft.cxx index 342d61471afe..50bc3eeb3256 100644 --- a/svx/source/dialog/hdft.cxx +++ b/svx/source/dialog/hdft.cxx @@ -92,7 +92,7 @@ namespace svx { SvxAbstractDialogFactory* pFact = SvxAbstractDialogFactory::Create(); if(pFact) { - ScopedVclPtr<SfxAbstractTabDialog> pDlg(pFact->CreateSvxBorderBackgroundDlg( pParent, *pBBSet, true/*bEnableBackgroundSelector*/ )); + ScopedVclPtr<SfxAbstractTabDialog> pDlg(pFact->CreateSvxBorderBackgroundDlg( pParent, *pBBSet, false/*bEnableDrawingLayerFillStyles*/ )); DBG_ASSERT(pDlg, "Dialog creation failed!"); if ( pDlg->Execute() == RET_OK && pDlg->GetOutputItemSet() ) { @@ -680,7 +680,6 @@ IMPL_LINK_NOARG(SvxHFPage, BackgroundHdl, Button*, void) ScopedVclPtr<SfxAbstractTabDialog> pDlg(pFact->CreateSvxBorderBackgroundDlg( this, *pBBSet, - true/*EnableBackgroundSelector*/, mbEnableDrawingLayerFillStyles)); DBG_ASSERT(pDlg,"Dialog creation failed!"); |