dbaccess/source/ui/dlg/UserAdmin.cxx | 2 - filter/source/pdf/impdialog.cxx | 2 - include/sfx2/objsh.hxx | 42 ++++++++++++++++++++------------- include/sfx2/passwd.hxx | 24 ++++++++++++------- sc/source/ui/docshell/docsh.cxx | 20 +++++++-------- sc/source/ui/docshell/docsh4.cxx | 2 - sc/source/ui/inc/docsh.hxx | 2 - sc/source/ui/view/tabvwsh3.cxx | 2 - sd/source/ui/docshell/docshel4.cxx | 4 +-- sfx2/source/dialog/passwd.cxx | 18 +++++++------- sfx2/source/dialog/securitypage.cxx | 2 - sfx2/source/doc/docinsert.cxx | 2 - sfx2/source/doc/objcont.cxx | 4 +-- sfx2/source/doc/objmisc.cxx | 30 +++++++++++------------ sfx2/source/doc/objserv.cxx | 14 ++++------- sfx2/source/doc/objstor.cxx | 44 +++++++++++++++++------------------ sfx2/source/doc/objxtor.cxx | 4 +-- sfx2/source/inc/objshimp.hxx | 16 ++++++------ sfx2/source/view/frame.cxx | 2 - sfx2/source/view/viewfrm.cxx | 4 +-- sfx2/source/view/viewprn.cxx | 2 - sfx2/source/view/viewsh.cxx | 4 +-- starmath/source/document.cxx | 4 +-- sw/inc/docsh.hxx | 2 - sw/source/ui/dialog/uiregionsw.cxx | 4 +-- sw/source/uibase/app/docsh.cxx | 16 ++++++------ sw/source/uibase/uiview/view2.cxx | 2 - 27 files changed, 145 insertions(+), 129 deletions(-)
New commits: commit b5f5a386504e320b022f8609c9c36652ae2d3d18 Author: Noel Grandin <n...@peralex.com> Date: Fri Apr 10 15:52:37 2015 +0200 convert SHOWEXTRAS constants to scoped enum Change-Id: I84bff53e55fc93ad423835ebc3bf129546dfb7c3 diff --git a/dbaccess/source/ui/dlg/UserAdmin.cxx b/dbaccess/source/ui/dlg/UserAdmin.cxx index 6afe533..66556c6 100644 --- a/dbaccess/source/ui/dlg/UserAdmin.cxx +++ b/dbaccess/source/ui/dlg/UserAdmin.cxx @@ -193,7 +193,7 @@ IMPL_LINK( OUserAdmin, UserHdl, PushButton *, pButton ) if(pButton == m_pNEWUSER) { SfxPasswordDialog aPwdDlg(this); - aPwdDlg.ShowExtras(SHOWEXTRAS_ALL); + aPwdDlg.ShowExtras(SfxShowExtras::ALL); if(aPwdDlg.Execute()) { Reference<XDataDescriptorFactory> xUserFactory(m_xUsers,UNO_QUERY); diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx index e8d1e1e..64224bd 100644 --- a/filter/source/pdf/impdialog.cxx +++ b/filter/source/pdf/impdialog.cxx @@ -1194,7 +1194,7 @@ IMPL_LINK_NOARG(ImpPDFTabSecurityPage, ClickmaPbSetPwdHdl) SfxPasswordDialog aPwdDialog( this, &msUserPwdTitle ); aPwdDialog.SetMinLen( 0 ); aPwdDialog.ShowMinLengthText(false); - aPwdDialog.ShowExtras( SHOWEXTRAS_CONFIRM | SHOWEXTRAS_PASSWORD2 | SHOWEXTRAS_CONFIRM2 ); + aPwdDialog.ShowExtras( SfxShowExtras::CONFIRM | SfxShowExtras::PASSWORD2 | SfxShowExtras::CONFIRM2 ); aPwdDialog.SetText(msStrSetPwd); aPwdDialog.SetGroup2Text(msOwnerPwdTitle); aPwdDialog.AllowAsciiOnly(); diff --git a/include/sfx2/passwd.hxx b/include/sfx2/passwd.hxx index 363e9b9..4b241d0 100644 --- a/include/sfx2/passwd.hxx +++ b/include/sfx2/passwd.hxx @@ -27,15 +27,23 @@ #include <vcl/fixed.hxx> #include <vcl/layout.hxx> #include <sfx2/app.hxx> +#include <o3tl/typed_flags_set.hxx> // defines --------------------------------------------------------------- -#define SHOWEXTRAS_NONE ((sal_uInt16)0x0000) -#define SHOWEXTRAS_USER ((sal_uInt16)0x0001) -#define SHOWEXTRAS_CONFIRM ((sal_uInt16)0x0002) -#define SHOWEXTRAS_PASSWORD2 ((sal_uInt16)0x0004) -#define SHOWEXTRAS_CONFIRM2 ((sal_uInt16)0x0008) -#define SHOWEXTRAS_ALL ((sal_uInt16)(SHOWEXTRAS_USER | SHOWEXTRAS_CONFIRM)) +enum class SfxShowExtras +{ + NONE = 0x0000, + USER = 0x0001, + CONFIRM = 0x0002, + PASSWORD2 = 0x0004, + CONFIRM2 = 0x0008, + ALL = USER | CONFIRM +}; +namespace o3tl +{ + template<> struct typed_flags<SfxShowExtras> : is_typed_flags<SfxShowExtras, 0x0f> {}; +} // class SfxPasswordDialog ----------------------------------------------- @@ -65,7 +73,7 @@ private: OUString maEmptyPwdStr; OUString maMainPwdStr; sal_uInt16 mnMinLen; - sal_uInt16 mnExtras; + SfxShowExtras mnExtras; bool mbAsciiOnly; DECL_DLLPRIVATE_LINK(EditModifyHdl, Edit*); @@ -105,7 +113,7 @@ public: { mpPassword1ED->SetHelpId( rId ); } - void ShowExtras(sal_uInt16 nExtras) + void ShowExtras(SfxShowExtras nExtras) { mnExtras = nExtras; } diff --git a/sc/source/ui/docshell/docsh4.cxx b/sc/source/ui/docshell/docsh4.cxx index d9b7355..a9dd15c 100644 --- a/sc/source/ui/docshell/docsh4.cxx +++ b/sc/source/ui/docshell/docsh4.cxx @@ -1137,7 +1137,7 @@ bool ScDocShell::ExecuteChangeProtectionDialog( vcl::Window* _pParent, bool bJus pDlg->SetHelpId( GetStaticInterface()->GetSlot(SID_CHG_PROTECT)->GetCommand() ); pDlg->SetEditHelpId( HID_CHG_PROTECT ); if ( !bProtected ) - pDlg->ShowExtras( SHOWEXTRAS_CONFIRM ); + pDlg->ShowExtras( SfxShowExtras::CONFIRM ); if ( pDlg->Execute() == RET_OK ) aPassword = pDlg->GetPassword(); pDlg.reset(); diff --git a/sc/source/ui/view/tabvwsh3.cxx b/sc/source/ui/view/tabvwsh3.cxx index f69cabf..2cfa826 100644 --- a/sc/source/ui/view/tabvwsh3.cxx +++ b/sc/source/ui/view/tabvwsh3.cxx @@ -1002,7 +1002,7 @@ void ScTabViewShell::Execute( SfxRequest& rReq ) pDlg->SetMinLen( 0 ); pDlg->SetHelpId( GetStaticInterface()->GetSlot(FID_PROTECT_DOC)->GetCommand() ); pDlg->SetEditHelpId( HID_PASSWD_DOC ); - pDlg->ShowExtras( SHOWEXTRAS_CONFIRM ); + pDlg->ShowExtras( SfxShowExtras::CONFIRM ); if (pDlg->Execute() == RET_OK) { diff --git a/sfx2/source/dialog/passwd.cxx b/sfx2/source/dialog/passwd.cxx index ca67649..237f8eb 100644 --- a/sfx2/source/dialog/passwd.cxx +++ b/sfx2/source/dialog/passwd.cxx @@ -65,9 +65,9 @@ IMPL_LINK_INLINE_END(SfxPasswordDialog, EditModifyHdl, Edit *, pEdit) IMPL_LINK_NOARG(SfxPasswordDialog, OKHdl) { - bool bConfirmFailed = ( ( mnExtras & SHOWEXTRAS_CONFIRM ) == SHOWEXTRAS_CONFIRM ) && - ( GetConfirm() != GetPassword() ); - if( ( mnExtras & SHOWEXTRAS_CONFIRM2 ) == SHOWEXTRAS_CONFIRM2 && ( GetConfirm2() != GetPassword2() ) ) + bool bConfirmFailed = bool( mnExtras & SfxShowExtras::CONFIRM ) && + ( GetConfirm() != GetPassword() ); + if( ( mnExtras & SfxShowExtras::CONFIRM2 ) && ( GetConfirm2() != GetPassword2() ) ) bConfirmFailed = true; if ( bConfirmFailed ) { @@ -89,7 +89,7 @@ SfxPasswordDialog::SfxPasswordDialog(vcl::Window* pParent, const OUString* pGrou , maMinLenPwdStr1(SFX2_RESSTR(STR_PASSWD_MIN_LEN1)) , maEmptyPwdStr(SFX2_RESSTR(STR_PASSWD_EMPTY)) , mnMinLen(5) - , mnExtras(0) + , mnExtras(SfxShowExtras::NONE) , mbAsciiOnly(false) { get(mpPassword1Box, "password1frame"); @@ -175,25 +175,25 @@ short SfxPasswordDialog::Execute() mpConfirm2FT->Hide(); mpConfirm2ED->Hide(); - if (mnExtras != SHOWEXTRAS_NONE) + if (mnExtras != SfxShowExtras::NONE) mpPassword1FT->Show(); - if (mnExtras & SHOWEXTRAS_USER) + if (mnExtras & SfxShowExtras::USER) { mpUserFT->Show(); mpUserED->Show(); } - if (mnExtras & SHOWEXTRAS_CONFIRM) + if (mnExtras & SfxShowExtras::CONFIRM) { mpConfirm1FT->Show(); mpConfirm1ED->Show(); } - if (mnExtras & SHOWEXTRAS_PASSWORD2) + if (mnExtras & SfxShowExtras::PASSWORD2) { mpPassword2Box->Show(); mpPassword2FT->Show(); mpPassword2ED->Show(); } - if (mnExtras & SHOWEXTRAS_CONFIRM2) + if (mnExtras & SfxShowExtras::CONFIRM2) { mpConfirm2FT->Show(); mpConfirm2ED->Show(); diff --git a/sfx2/source/dialog/securitypage.cxx b/sfx2/source/dialog/securitypage.cxx index b3fcb3d..7fb1d0c 100644 --- a/sfx2/source/dialog/securitypage.cxx +++ b/sfx2/source/dialog/securitypage.cxx @@ -102,7 +102,7 @@ static bool lcl_GetPassword( SfxPasswordDialog aPasswdDlg( pParent ); aPasswdDlg.SetMinLen( 1 ); if (bProtect) - aPasswdDlg.ShowExtras( SHOWEXTRAS_CONFIRM ); + aPasswdDlg.ShowExtras( SfxShowExtras::CONFIRM ); if (RET_OK == aPasswdDlg.Execute() && !aPasswdDlg.GetPassword().isEmpty()) { rPassword = aPasswdDlg.GetPassword(); diff --git a/sfx2/source/doc/docinsert.cxx b/sfx2/source/doc/docinsert.cxx index e51854e..d114453 100644 --- a/sfx2/source/doc/docinsert.cxx +++ b/sfx2/source/doc/docinsert.cxx @@ -191,7 +191,7 @@ IMPL_LINK_NOARG(DocumentInserter, DialogClosedHdl) { // ask for the password SfxPasswordDialog aPasswordDlg( NULL ); - aPasswordDlg.ShowExtras( SHOWEXTRAS_CONFIRM ); + aPasswordDlg.ShowExtras( SfxShowExtras::CONFIRM ); short nRet = aPasswordDlg.Execute(); if ( RET_OK == nRet ) { diff --git a/sw/source/ui/dialog/uiregionsw.cxx b/sw/source/ui/dialog/uiregionsw.cxx index fbe660f..305dc1b 100644 --- a/sw/source/ui/dialog/uiregionsw.cxx +++ b/sw/source/ui/dialog/uiregionsw.cxx @@ -1233,7 +1233,7 @@ IMPL_LINK( SwEditRegionDlg, ChangePasswdHdl, Button *, pBox ) if(!pRepr->GetTempPasswd().getLength() || bChange) { SfxPasswordDialog aPasswdDlg(this); - aPasswdDlg.ShowExtras(SHOWEXTRAS_CONFIRM); + aPasswdDlg.ShowExtras(SfxShowExtras::CONFIRM); if(RET_OK == aPasswdDlg.Execute()) { const OUString sNewPasswd( aPasswdDlg.GetPassword() ); @@ -1671,7 +1671,7 @@ IMPL_LINK( SwInsertSectionTabPage, ChangePasswdHdl, Button *, pButton ) if(!m_aNewPasswd.getLength() || bChange) { SfxPasswordDialog aPasswdDlg(this); - aPasswdDlg.ShowExtras(SHOWEXTRAS_CONFIRM); + aPasswdDlg.ShowExtras(SfxShowExtras::CONFIRM); if(RET_OK == aPasswdDlg.Execute()) { const OUString sNewPasswd( aPasswdDlg.GetPassword() ); diff --git a/sw/source/uibase/uiview/view2.cxx b/sw/source/uibase/uiview/view2.cxx index ed126fe..2dba59d 100644 --- a/sw/source/uibase/uiview/view2.cxx +++ b/sw/source/uibase/uiview/view2.cxx @@ -626,7 +626,7 @@ void SwView::Execute(SfxRequest &rReq) SfxPasswordDialog aPasswdDlg( pParent ); aPasswdDlg.SetMinLen( 1 ); if(!aPasswd.getLength()) - aPasswdDlg.ShowExtras(SHOWEXTRAS_CONFIRM); + aPasswdDlg.ShowExtras(SfxShowExtras::CONFIRM); if (aPasswdDlg.Execute()) { sal_uInt16 nOn = nsRedlineMode_t::REDLINE_ON; commit f2109254942074d07ad52bf9fe6571405fd79618 Author: Noel Grandin <n...@peralex.com> Date: Fri Apr 10 15:27:43 2015 +0200 remove unused constants Change-Id: Ib467f1326b224dbf87282174559c7076d834b7b7 diff --git a/include/sfx2/objsh.hxx b/include/sfx2/objsh.hxx index 49fea82..8b82d76 100644 --- a/include/sfx2/objsh.hxx +++ b/include/sfx2/objsh.hxx @@ -504,13 +504,7 @@ public: GetDialogContainer(); StarBASIC* GetBasic() const; - // Interface Document content, Organizer -#define CONTENT_STYLE 0 -#define CONTENT_CONFIG 1 -#define CONTENT_MACRO 2 - - - virtual std::set<Color> GetDocColors(); + virtual std::set<Color> GetDocColors(); virtual void LoadStyles( SfxObjectShell &rSource ); void ReadNote( INote * ); commit 43fbf02984f405864f89fb3e36eff32f9d757b57 Author: Noel Grandin <n...@peralex.com> Date: Fri Apr 10 15:27:33 2015 +0200 convert HiddenWarningFact to scoped enum Change-Id: I23579d746a3c612de68345794731f1ff2cbdf605 diff --git a/include/sfx2/objsh.hxx b/include/sfx2/objsh.hxx index 3da1ad3..49fea82 100644 --- a/include/sfx2/objsh.hxx +++ b/include/sfx2/objsh.hxx @@ -159,7 +159,7 @@ namespace o3tl template<> struct typed_flags<HiddenInformation> : is_typed_flags<HiddenInformation, 0x07> {}; } -enum HiddenWarningFact +enum class HiddenWarningFact { WhenSaving = 0, WhenPrinting, diff --git a/sfx2/source/doc/objserv.cxx b/sfx2/source/doc/objserv.cxx index 8c4db55..119e52f 100644 --- a/sfx2/source/doc/objserv.cxx +++ b/sfx2/source/doc/objserv.cxx @@ -405,7 +405,7 @@ void SfxObjectShell::ExecFile_Impl(SfxRequest &rReq) if( SID_SIGNATURE == nId || SID_MACRO_SIGNATURE == nId ) { - if ( QueryHiddenInformation( WhenSigning, NULL ) == RET_YES ) + if ( QueryHiddenInformation( HiddenWarningFact::WhenSigning, NULL ) == RET_YES ) ( SID_SIGNATURE == nId ) ? SignDocumentContent() : SignScriptingContent(); return; } @@ -653,7 +653,7 @@ void SfxObjectShell::ExecFile_Impl(SfxRequest &rReq) SfxStoringHelper aHelper; - if ( QueryHiddenInformation( bIsPDFExport ? WhenCreatingPDF : WhenSaving, NULL ) == RET_YES ) + if ( QueryHiddenInformation( bIsPDFExport ? HiddenWarningFact::WhenCreatingPDF : HiddenWarningFact::WhenSaving, NULL ) == RET_YES ) { aHelper.GUIStoreModel( GetModel(), OUString::createFromAscii( pSlot->GetUnoName() ), diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx index 062ac57..b1ed9b3 100644 --- a/sfx2/source/doc/objstor.cxx +++ b/sfx2/source/doc/objstor.cxx @@ -2959,25 +2959,25 @@ sal_Int16 SfxObjectShell::QueryHiddenInformation( HiddenWarningFact eFact, vcl:: switch ( eFact ) { - case WhenSaving : + case HiddenWarningFact::WhenSaving : { nResId = STR_HIDDENINFO_CONTINUE_SAVING; eOption = SvtSecurityOptions::E_DOCWARN_SAVEORSEND; break; } - case WhenPrinting : + case HiddenWarningFact::WhenPrinting : { nResId = STR_HIDDENINFO_CONTINUE_PRINTING; eOption = SvtSecurityOptions::E_DOCWARN_PRINT; break; } - case WhenSigning : + case HiddenWarningFact::WhenSigning : { nResId = STR_HIDDENINFO_CONTINUE_SIGNING; eOption = SvtSecurityOptions::E_DOCWARN_SIGNING; break; } - case WhenCreatingPDF : + case HiddenWarningFact::WhenCreatingPDF : { nResId = STR_HIDDENINFO_CONTINUE_CREATEPDF; eOption = SvtSecurityOptions::E_DOCWARN_CREATEPDF; @@ -2991,7 +2991,7 @@ sal_Int16 SfxObjectShell::QueryHiddenInformation( HiddenWarningFact eFact, vcl:: { OUString sMessage( SfxResId(STR_HIDDENINFO_CONTAINS).toString() ); HiddenInformation nWantedStates = HiddenInformation::RECORDEDCHANGES | HiddenInformation::NOTES; - if ( eFact != WhenPrinting ) + if ( eFact != HiddenWarningFact::WhenPrinting ) nWantedStates |= HiddenInformation::DOCUMENTVERSIONS; HiddenInformation nStates = GetHiddenInformationState( nWantedStates ); bool bWarning = false; diff --git a/sfx2/source/view/viewprn.cxx b/sfx2/source/view/viewprn.cxx index 34716d6..1644459 100644 --- a/sfx2/source/view/viewprn.cxx +++ b/sfx2/source/view/viewprn.cxx @@ -675,7 +675,7 @@ void SfxViewShell::ExecPrint_Impl( SfxRequest &rReq ) return; } - if ( !bSilent && pDoc->QueryHiddenInformation( WhenPrinting, NULL ) != RET_YES ) + if ( !bSilent && pDoc->QueryHiddenInformation( HiddenWarningFact::WhenPrinting, NULL ) != RET_YES ) return; // should we print only the selection or the whole document diff --git a/sfx2/source/view/viewsh.cxx b/sfx2/source/view/viewsh.cxx index abfbbbc..af454dd 100644 --- a/sfx2/source/view/viewsh.cxx +++ b/sfx2/source/view/viewsh.cxx @@ -548,7 +548,7 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq ) { SfxObjectShell* pDoc = GetObjectShell(); if ( pDoc && pDoc->QueryHiddenInformation( - WhenSaving, &GetViewFrame()->GetWindow() ) != RET_YES ) + HiddenWarningFact::WhenSaving, &GetViewFrame()->GetWindow() ) != RET_YES ) break; @@ -609,7 +609,7 @@ void SfxViewShell::ExecMisc_Impl( SfxRequest &rReq ) SfxBluetoothModel aModel; SfxObjectShell* pDoc = GetObjectShell(); if ( pDoc && pDoc->QueryHiddenInformation( - WhenSaving, &GetViewFrame()->GetWindow() ) != RET_YES ) + HiddenWarningFact::WhenSaving, &GetViewFrame()->GetWindow() ) != RET_YES ) break; uno::Reference < frame::XFrame > xFrame( pFrame->GetFrame().GetFrameInterface() ); SfxMailModel::SendMailResult eResult = aModel.SaveAndSend( xFrame, OUString() ); commit 5651f2d9f64c213b4106e2d5a433f4c525d41bbf Author: Noel Grandin <n...@peralex.com> Date: Fri Apr 10 14:28:43 2015 +0200 convert HIDDENINFORMATION_ constants into scoped enum Change-Id: I94136c926fc8fa69ce2ce2b97fa34dc984dbe095 diff --git a/include/sfx2/objsh.hxx b/include/sfx2/objsh.hxx index 40a18f4..3da1ad3 100644 --- a/include/sfx2/objsh.hxx +++ b/include/sfx2/objsh.hxx @@ -147,9 +147,17 @@ namespace o3tl template<> struct typed_flags<SfxLoadedFlags> : is_typed_flags<SfxLoadedFlags, 0x03> {}; } -#define HIDDENINFORMATION_RECORDEDCHANGES 0x0001 -#define HIDDENINFORMATION_NOTES 0x0002 -#define HIDDENINFORMATION_DOCUMENTVERSIONS 0x0004 +enum class HiddenInformation +{ + NONE = 0x0000, + RECORDEDCHANGES = 0x0001, + NOTES = 0x0002, + DOCUMENTVERSIONS = 0x0004 +}; +namespace o3tl +{ + template<> struct typed_flags<HiddenInformation> : is_typed_flags<HiddenInformation, 0x07> {}; +} enum HiddenWarningFact { @@ -419,7 +427,7 @@ public: virtual bool PrepareClose(bool bUI = true); virtual bool IsInformationLost(); - virtual sal_uInt16 GetHiddenInformationState( sal_uInt16 nStates ); + virtual HiddenInformation GetHiddenInformationState( HiddenInformation nStates ); sal_Int16 QueryHiddenInformation( HiddenWarningFact eFact, vcl::Window* pParent ); bool IsSecurityOptOpenReadOnly() const; void SetSecurityOptOpenReadOnly( bool bOpenReadOnly = true ); diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx index 0ab97e1..1261dab 100644 --- a/sc/source/ui/docshell/docsh.cxx +++ b/sc/source/ui/docshell/docsh.cxx @@ -230,17 +230,17 @@ SCTAB ScDocShell::GetSaveTab() return nTab; } -sal_uInt16 ScDocShell::GetHiddenInformationState( sal_uInt16 nStates ) +HiddenInformation ScDocShell::GetHiddenInformationState( HiddenInformation nStates ) { - // get global state like HIDDENINFORMATION_DOCUMENTVERSIONS - sal_uInt16 nState = SfxObjectShell::GetHiddenInformationState( nStates ); + // get global state like HiddenInformation::DOCUMENTVERSIONS + HiddenInformation nState = SfxObjectShell::GetHiddenInformationState( nStates ); - if ( nStates & HIDDENINFORMATION_RECORDEDCHANGES ) + if ( nStates & HiddenInformation::RECORDEDCHANGES ) { if ( aDocument.GetChangeTrack() && aDocument.GetChangeTrack()->GetFirst() ) - nState |= HIDDENINFORMATION_RECORDEDCHANGES; + nState |= HiddenInformation::RECORDEDCHANGES; } - if ( nStates & HIDDENINFORMATION_NOTES ) + if ( nStates & HiddenInformation::NOTES ) { SCTAB nTableCount = aDocument.GetTableCount(); bool bFound = false; @@ -251,7 +251,7 @@ sal_uInt16 ScDocShell::GetHiddenInformationState( sal_uInt16 nStates ) } if (bFound) - nState |= HIDDENINFORMATION_NOTES; + nState |= HiddenInformation::NOTES; } return nState; diff --git a/sc/source/ui/inc/docsh.hxx b/sc/source/ui/inc/docsh.hxx index e0576a5..340508c 100644 --- a/sc/source/ui/inc/docsh.hxx +++ b/sc/source/ui/inc/docsh.hxx @@ -413,7 +413,7 @@ public: void BeforeXMLLoading(); void AfterXMLLoading(bool bRet); - virtual sal_uInt16 GetHiddenInformationState( sal_uInt16 nStates ) SAL_OVERRIDE; + virtual HiddenInformation GetHiddenInformationState( HiddenInformation nStates ) SAL_OVERRIDE; const ScOptSolverSave* GetSolverSaveData() const { return pSolverSaveData; } // may be null void SetSolverSaveData( const ScOptSolverSave& rData ); diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx index cfd7bfc..062ac57 100644 --- a/sfx2/source/doc/objstor.cxx +++ b/sfx2/source/doc/objstor.cxx @@ -2939,13 +2939,13 @@ bool SfxObjectShell::CanReload_Impl() -sal_uInt16 SfxObjectShell::GetHiddenInformationState( sal_uInt16 nStates ) +HiddenInformation SfxObjectShell::GetHiddenInformationState( HiddenInformation nStates ) { - sal_uInt16 nState = 0; - if ( nStates & HIDDENINFORMATION_DOCUMENTVERSIONS ) + HiddenInformation nState = HiddenInformation::NONE; + if ( nStates & HiddenInformation::DOCUMENTVERSIONS ) { if ( GetMedium()->GetVersionList().getLength() ) - nState |= HIDDENINFORMATION_DOCUMENTVERSIONS; + nState |= HiddenInformation::DOCUMENTVERSIONS; } return nState; @@ -2990,25 +2990,25 @@ sal_Int16 SfxObjectShell::QueryHiddenInformation( HiddenWarningFact eFact, vcl:: if ( SvtSecurityOptions().IsOptionSet( eOption ) ) { OUString sMessage( SfxResId(STR_HIDDENINFO_CONTAINS).toString() ); - sal_uInt16 nWantedStates = HIDDENINFORMATION_RECORDEDCHANGES | HIDDENINFORMATION_NOTES; + HiddenInformation nWantedStates = HiddenInformation::RECORDEDCHANGES | HiddenInformation::NOTES; if ( eFact != WhenPrinting ) - nWantedStates |= HIDDENINFORMATION_DOCUMENTVERSIONS; - sal_uInt16 nStates = GetHiddenInformationState( nWantedStates ); + nWantedStates |= HiddenInformation::DOCUMENTVERSIONS; + HiddenInformation nStates = GetHiddenInformationState( nWantedStates ); bool bWarning = false; - if ( ( nStates & HIDDENINFORMATION_RECORDEDCHANGES ) == HIDDENINFORMATION_RECORDEDCHANGES ) + if ( nStates & HiddenInformation::RECORDEDCHANGES ) { sMessage += SfxResId(STR_HIDDENINFO_RECORDCHANGES).toString(); sMessage += "\n"; bWarning = true; } - if ( ( nStates & HIDDENINFORMATION_NOTES ) == HIDDENINFORMATION_NOTES ) + if ( nStates & HiddenInformation::NOTES ) { sMessage += SfxResId(STR_HIDDENINFO_NOTES).toString(); sMessage += "\n"; bWarning = true; } - if ( ( nStates & HIDDENINFORMATION_DOCUMENTVERSIONS ) == HIDDENINFORMATION_DOCUMENTVERSIONS ) + if ( nStates & HiddenInformation::DOCUMENTVERSIONS ) { sMessage += SfxResId(STR_HIDDENINFO_DOCVERSIONS).toString(); sMessage += "\n"; diff --git a/sw/inc/docsh.hxx b/sw/inc/docsh.hxx index 9fbb874..fa2f547 100644 --- a/sw/inc/docsh.hxx +++ b/sw/inc/docsh.hxx @@ -288,7 +288,7 @@ public: virtual ::com::sun::star::uno::Sequence< OUString > GetEventNames() SAL_OVERRIDE; /// #i20883# Digital Signatures and Encryption - virtual sal_uInt16 GetHiddenInformationState( sal_uInt16 nStates ) SAL_OVERRIDE; + virtual HiddenInformation GetHiddenInformationState( HiddenInformation nStates ) SAL_OVERRIDE; /** #i42634# Overwrites SfxObjectShell::UpdateLinks This new function is necessary to trigger update of links in docs diff --git a/sw/source/uibase/app/docsh.cxx b/sw/source/uibase/app/docsh.cxx index 4f75597..71e8251 100644 --- a/sw/source/uibase/app/docsh.cxx +++ b/sw/source/uibase/app/docsh.cxx @@ -878,17 +878,17 @@ sal_uLong SwDocShell::GetMiscStatus() const } // #i20883# Digital Signatures and Encryption -sal_uInt16 SwDocShell::GetHiddenInformationState( sal_uInt16 nStates ) +HiddenInformation SwDocShell::GetHiddenInformationState( HiddenInformation nStates ) { - // get global state like HIDDENINFORMATION_DOCUMENTVERSIONS - sal_uInt16 nState = SfxObjectShell::GetHiddenInformationState( nStates ); + // get global state like HiddenInformation::DOCUMENTVERSIONS + HiddenInformation nState = SfxObjectShell::GetHiddenInformationState( nStates ); - if ( nStates & HIDDENINFORMATION_RECORDEDCHANGES ) + if ( nStates & HiddenInformation::RECORDEDCHANGES ) { if ( !GetDoc()->getIDocumentRedlineAccess().GetRedlineTbl().empty() ) - nState |= HIDDENINFORMATION_RECORDEDCHANGES; + nState |= HiddenInformation::RECORDEDCHANGES; } - if ( nStates & HIDDENINFORMATION_NOTES ) + if ( nStates & HiddenInformation::NOTES ) { OSL_ENSURE( GetWrtShell(), "No SwWrtShell, no information" ); if ( GetWrtShell() ) @@ -900,7 +900,7 @@ sal_uInt16 SwDocShell::GetHiddenInformationState( sal_uInt16 nStates ) { if( pFirst->GetTxtFld() && pFirst->IsFldInDoc() ) { - nState |= HIDDENINFORMATION_NOTES; + nState |= HiddenInformation::NOTES; break; } pFirst = aIter.Next(); commit ccfa9fa13db230cc66207bc42d528b773c4cf4f6 Author: Noel Grandin <n...@peralex.com> Date: Fri Apr 10 12:17:41 2015 +0200 convert SFX_LOADED_ to scoped enum Change-Id: I065a07e1bb5f5a7f85429c919830b6b9605206ea diff --git a/include/sfx2/objsh.hxx b/include/sfx2/objsh.hxx index b92656e..40a18f4 100644 --- a/include/sfx2/objsh.hxx +++ b/include/sfx2/objsh.hxx @@ -135,9 +135,17 @@ namespace o3tl #define SFX_TITLE_HISTORY 7 #define SFX_TITLE_MAXLEN 10 // this gives the limits on length -#define SFX_LOADED_MAINDOCUMENT 1 -#define SFX_LOADED_IMAGES 2 -#define SFX_LOADED_ALL 3 +enum class SfxLoadedFlags +{ + NONE = 0, + MAINDOCUMENT = 1, + IMAGES = 2, + ALL = MAINDOCUMENT | IMAGES +}; +namespace o3tl +{ + template<> struct typed_flags<SfxLoadedFlags> : is_typed_flags<SfxLoadedFlags, 0x03> {}; +} #define HIDDENINFORMATION_RECORDEDCHANGES 0x0001 #define HIDDENINFORMATION_NOTES 0x0002 @@ -438,7 +446,7 @@ public: void AbortImport(); bool IsAbortingImport() const; bool IsReloading() const; - void FinishedLoading( sal_uInt16 nWhich = SFX_LOADED_ALL ); + void FinishedLoading( SfxLoadedFlags nWhich = SfxLoadedFlags::ALL ); void TemplateDisconnectionAfterLoad(); bool IsLoading() const; bool IsLoadingFinished() const; diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx index 2ecbb0c..0ab97e1 100644 --- a/sc/source/ui/docshell/docsh.cxx +++ b/sc/source/ui/docshell/docsh.cxx @@ -595,7 +595,7 @@ bool ScDocShell::Load( SfxMedium& rMedium ) aDocument.InvalidateTableArea(); bIsEmpty = false; - FinishedLoading( SFX_LOADED_MAINDOCUMENT | SFX_LOADED_IMAGES ); + FinishedLoading( SfxLoadedFlags::ALL ); return bRet; } @@ -1503,7 +1503,7 @@ bool ScDocShell::ConvertFrom( SfxMedium& rMedium ) aUpdater.update(); } } - FinishedLoading( SFX_LOADED_MAINDOCUMENT | SFX_LOADED_IMAGES ); + FinishedLoading( SfxLoadedFlags::ALL ); // invalidate eventually temporary table areas if ( bRet ) @@ -1548,7 +1548,7 @@ bool ScDocShell::LoadExternal( SfxMedium& rMed ) return false; } - FinishedLoading(SFX_LOADED_MAINDOCUMENT | SFX_LOADED_IMAGES); + FinishedLoading(SfxLoadedFlags::ALL); return true; } diff --git a/sd/source/ui/docshell/docshel4.cxx b/sd/source/ui/docshell/docshel4.cxx index f653189..4232b3f 100644 --- a/sd/source/ui/docshell/docshel4.cxx +++ b/sd/source/ui/docshell/docshel4.cxx @@ -309,7 +309,7 @@ bool DrawDocShell::Load( SfxMedium& rMedium ) SetVisArea( Rectangle( pPage->GetAllObjBoundRect() ) ); } - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); const INetURLObject aUrl; SfxObjectShell::SetAutoLoad( aUrl, 0, false ); @@ -486,7 +486,7 @@ bool DrawDocShell::ConvertFrom( SfxMedium& rMedium ) bRet = SdGRFFilter( rMedium, *this ).Import(); } - FinishedLoading( SFX_LOADED_MAINDOCUMENT | SFX_LOADED_IMAGES ); + FinishedLoading( SfxLoadedFlags::ALL ); // tell SFX to change viewshell when in preview mode if( IsPreview() ) diff --git a/sfx2/source/doc/objcont.cxx b/sfx2/source/doc/objcont.cxx index f9a25a3..95fd6b8 100644 --- a/sfx2/source/doc/objcont.cxx +++ b/sfx2/source/doc/objcont.cxx @@ -599,7 +599,7 @@ sal_uInt32 SfxObjectShell::GetModifyPasswordHash() const bool SfxObjectShell::SetModifyPasswordHash( sal_uInt32 nHash ) { if ( ( !IsReadOnly() && !IsReadOnlyUI() ) - || !(pImp->nFlagsInProgress & SFX_LOADED_MAINDOCUMENT ) ) + || !(pImp->nFlagsInProgress & SfxLoadedFlags::MAINDOCUMENT ) ) { // the hash can be changed only in editable documents, // or during loading of document @@ -618,7 +618,7 @@ uno::Sequence< beans::PropertyValue > SfxObjectShell::GetModifyPasswordInfo() co bool SfxObjectShell::SetModifyPasswordInfo( const uno::Sequence< beans::PropertyValue >& aInfo ) { if ( ( !IsReadOnly() && !IsReadOnlyUI() ) - || !(pImp->nFlagsInProgress & SFX_LOADED_MAINDOCUMENT ) ) + || !(pImp->nFlagsInProgress & SfxLoadedFlags::MAINDOCUMENT ) ) { // the hash can be changed only in editable documents, // or during loading of document diff --git a/sfx2/source/doc/objmisc.cxx b/sfx2/source/doc/objmisc.cxx index 7d75118..f3ac27c 100644 --- a/sfx2/source/doc/objmisc.cxx +++ b/sfx2/source/doc/objmisc.cxx @@ -1155,7 +1155,7 @@ void SfxObjectShell::SetAutoLoad( bool SfxObjectShell::IsLoadingFinished() const { - return ( pImp->nLoadedFlags == SFX_LOADED_ALL ); + return ( pImp->nLoadedFlags == SfxLoadedFlags::ALL ); } void SfxObjectShell::InitOwnModel_Impl() @@ -1194,14 +1194,14 @@ void SfxObjectShell::InitOwnModel_Impl() } } -void SfxObjectShell::FinishedLoading( sal_uInt16 nFlags ) +void SfxObjectShell::FinishedLoading( SfxLoadedFlags nFlags ) { bool bSetModifiedTRUE = false; SFX_ITEMSET_ARG( pMedium->GetItemSet(), pSalvageItem, SfxStringItem, SID_DOC_SALVAGE, false ); - if( ( nFlags & SFX_LOADED_MAINDOCUMENT ) && !(pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) - && !(pImp->nFlagsInProgress & SFX_LOADED_MAINDOCUMENT )) + if( ( nFlags & SfxLoadedFlags::MAINDOCUMENT ) && !(pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) + && !(pImp->nFlagsInProgress & SfxLoadedFlags::MAINDOCUMENT )) { - pImp->nFlagsInProgress |= SFX_LOADED_MAINDOCUMENT; + pImp->nFlagsInProgress |= SfxLoadedFlags::MAINDOCUMENT; static_cast<SfxHeaderAttributes_Impl*>(GetHeaderAttributes())->SetAttributes(); pImp->bImportDone = true; if( !IsAbortingImport() ) @@ -1225,13 +1225,13 @@ void SfxObjectShell::FinishedLoading( sal_uInt16 nFlags ) bHasName = true; // the document is loaded, so the name should already available GetTitle( SFX_TITLE_DETECT ); InitOwnModel_Impl(); - pImp->nFlagsInProgress &= ~SFX_LOADED_MAINDOCUMENT; + pImp->nFlagsInProgress &= ~SfxLoadedFlags::MAINDOCUMENT; } - if( ( nFlags & SFX_LOADED_IMAGES ) && !(pImp->nLoadedFlags & SFX_LOADED_IMAGES ) - && !(pImp->nFlagsInProgress & SFX_LOADED_IMAGES )) + if( ( nFlags & SfxLoadedFlags::IMAGES ) && !(pImp->nLoadedFlags & SfxLoadedFlags::IMAGES ) + && !(pImp->nFlagsInProgress & SfxLoadedFlags::IMAGES )) { - pImp->nFlagsInProgress |= SFX_LOADED_IMAGES; + pImp->nFlagsInProgress |= SfxLoadedFlags::IMAGES; uno::Reference<document::XDocumentProperties> xDocProps( getDocProperties()); OUString url(xDocProps->getAutoloadURL()); @@ -1241,12 +1241,12 @@ void SfxObjectShell::FinishedLoading( sal_uInt16 nFlags ) if( !bSetModifiedTRUE && IsEnableSetModified() ) SetModified( false ); Invalidate( SID_SAVEASDOC ); - pImp->nFlagsInProgress &= ~SFX_LOADED_IMAGES; + pImp->nFlagsInProgress &= ~SfxLoadedFlags::IMAGES; } pImp->nLoadedFlags |= nFlags; - if ( !pImp->nFlagsInProgress ) + if ( pImp->nFlagsInProgress == SfxLoadedFlags::NONE ) { // in case of reentrance calls the first called FinishedLoading() call on the stack // should do the notification, in result the notification is done when all the FinishedLoading() calls are finished @@ -1256,7 +1256,7 @@ void SfxObjectShell::FinishedLoading( sal_uInt16 nFlags ) else SetModified( false ); - if ( (pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) && (pImp->nLoadedFlags & SFX_LOADED_IMAGES ) ) + if ( (pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) && (pImp->nLoadedFlags & SfxLoadedFlags::IMAGES ) ) { SFX_ITEMSET_ARG( pMedium->GetItemSet(), pTemplateItem, SfxBoolItem, SID_TEMPLATE, false); bool bTemplate = pTemplateItem && pTemplateItem->GetValue(); @@ -1402,7 +1402,7 @@ bool SfxObjectShell::IsLoading() const Has FinishedLoading been called? */ { - return !( pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ); + return !( pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ); } @@ -1414,11 +1414,11 @@ void SfxObjectShell::CancelTransfers() by RegisterTransfer. */ { - if( ( pImp->nLoadedFlags & SFX_LOADED_ALL ) != SFX_LOADED_ALL ) + if( ( pImp->nLoadedFlags & SfxLoadedFlags::ALL ) != SfxLoadedFlags::ALL ) { AbortImport(); if( IsLoading() ) - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); } } diff --git a/sfx2/source/doc/objserv.cxx b/sfx2/source/doc/objserv.cxx index 1a10c39..8c4db55 100644 --- a/sfx2/source/doc/objserv.cxx +++ b/sfx2/source/doc/objserv.cxx @@ -1050,7 +1050,7 @@ void SfxObjectShell::GetState_Impl(SfxItemSet &rSet) case SID_SAVEASDOC: { - if( ( pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) != SFX_LOADED_MAINDOCUMENT ) + if( !( pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) ) { rSet.DisableItem( nWhich ); break; @@ -1064,7 +1064,7 @@ void SfxObjectShell::GetState_Impl(SfxItemSet &rSet) case SID_SAVEACOPY: { - if( ( pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) != SFX_LOADED_MAINDOCUMENT ) + if( !( pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) ) { rSet.DisableItem( nWhich ); break; @@ -1234,13 +1234,11 @@ void SfxObjectShell::StateProps_Impl(SfxItemSet &rSet) } case SID_DOC_LOADING: - rSet.Put( SfxBoolItem( nSID, SFX_LOADED_MAINDOCUMENT != - ( pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) ) ); + rSet.Put( SfxBoolItem( nSID, ! ( pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) ) ); break; case SID_IMG_LOADING: - rSet.Put( SfxBoolItem( nSID, SFX_LOADED_IMAGES != - ( pImp->nLoadedFlags & SFX_LOADED_IMAGES ) ) ); + rSet.Put( SfxBoolItem( nSID, ! ( pImp->nLoadedFlags & SfxLoadedFlags::IMAGES ) ) ); break; } } diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx index 32dcfef..cfd7bfc 100644 --- a/sfx2/source/doc/objstor.cxx +++ b/sfx2/source/doc/objstor.cxx @@ -657,7 +657,7 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed ) } pMed->GetItemSet()->Put( SfxStringItem( SID_DOC_BASEURL, aBaseURL ) ); - pImp->nLoadedFlags = 0; + pImp->nLoadedFlags = SfxLoadedFlags::NONE; pImp->bModelInitialized = false; //TODO/LATER: make a clear strategy how to handle "UsesStorage" etc. @@ -718,7 +718,7 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed ) // Load if ( !GetError() ) { - pImp->nLoadedFlags = 0; + pImp->nLoadedFlags = SfxLoadedFlags::NONE; pImp->bModelInitialized = false; bOk = xStorage.is() && LoadOwnFormat( *pMed ); if ( bOk ) @@ -748,7 +748,7 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed ) if ( GetError() == ERRCODE_NONE ) { - pImp->nLoadedFlags = 0; + pImp->nLoadedFlags = SfxLoadedFlags::NONE; pImp->bModelInitialized = false; if ( pMedium->GetFilter() && ( pMedium->GetFilter()->GetFilterFlags() & SfxFilterFlags::STARONEFILTER ) ) { @@ -774,7 +774,7 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed ) {} } UpdateLinks(); - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); } else { @@ -832,10 +832,10 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed ) } // If not loaded asynchronously call FinishedLoading - if ( !( pImp->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) && - ( !pMedium->GetFilter() || pMedium->GetFilter()->UsesStorage() ) + if ( !( pImp->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) && + ( !pMedium->GetFilter() || pMedium->GetFilter()->UsesStorage() ) ) - FinishedLoading( SFX_LOADED_MAINDOCUMENT ); + FinishedLoading( SfxLoadedFlags::MAINDOCUMENT ); if( IsOwnStorageFormat_Impl(*pMed) && pMed->GetFilter() ) { diff --git a/sfx2/source/doc/objxtor.cxx b/sfx2/source/doc/objxtor.cxx index 67504bc..6114411 100644 --- a/sfx2/source/doc/objxtor.cxx +++ b/sfx2/source/doc/objxtor.cxx @@ -246,8 +246,8 @@ SfxObjectShell_Impl::SfxObjectShell_Impl( SfxObjectShell& _rDocShell ) ,nEventId ( 0) ,pReloadTimer ( 0) ,pMarkData( 0 ) - ,nLoadedFlags ( SFX_LOADED_ALL ) - ,nFlagsInProgress( 0 ) + ,nLoadedFlags ( SfxLoadedFlags::ALL ) + ,nFlagsInProgress( SfxLoadedFlags::NONE ) ,bModalMode( false ) ,bRunningMacro( false ) ,bReloadAvailable( false ) diff --git a/sfx2/source/inc/objshimp.hxx b/sfx2/source/inc/objshimp.hxx index ff3f2b3..8867860 100644 --- a/sfx2/source/inc/objshimp.hxx +++ b/sfx2/source/inc/objshimp.hxx @@ -95,31 +95,31 @@ struct SfxObjectShell_Impl : public ::sfx2::IMacroDocumentAccess m_bConfigOptionsChecked:1; // whether or not the user options are checked after the Options dialog is closed. IndexBitSet aBitSet; - sal_uInt32 lErr; + sal_uInt32 lErr; sal_uInt16 nEventId; // If Open/Create as to be sent // before Activate AutoReloadTimer_Impl *pReloadTimer; MarkData_Impl* pMarkData; - sal_uInt16 nLoadedFlags; - sal_uInt16 nFlagsInProgress; + SfxLoadedFlags nLoadedFlags; + SfxLoadedFlags nFlagsInProgress; bool bModalMode; bool bRunningMacro; bool bReloadAvailable; - sal_uInt16 nAutoLoadLocks; - SfxModule* pModule; + sal_uInt16 nAutoLoadLocks; + SfxModule* pModule; SfxObjectShellFlags eFlags; bool bReadOnlyUI; tools::SvRef<SvRefBase> xHeaderAttributes; ::rtl::Reference< SfxBaseModel > pBaseModel; - sal_uInt16 nStyleFilter; + sal_uInt16 nStyleFilter; bool bDisposing; bool m_bEnableSetModified; bool m_bIsModified; - Rectangle m_aVisArea; - MapUnit m_nMapUnit; + Rectangle m_aVisArea; + MapUnit m_nMapUnit; bool m_bCreateTempStor; ::com::sun::star::uno::Reference< ::com::sun::star::embed::XStorage > m_xDocStorage; diff --git a/sfx2/source/view/frame.cxx b/sfx2/source/view/frame.cxx index 4b36968..82c0381 100644 --- a/sfx2/source/view/frame.cxx +++ b/sfx2/source/view/frame.cxx @@ -284,7 +284,7 @@ void SfxFrame::CancelTransfers( bool /*bCancelLoadEnv*/ ) { pImp->bInCancelTransfers = true; SfxObjectShell* pObj = GetCurrentDocument(); - if( pObj ) //&& !( pObj->Get_Impl()->nLoadedFlags & SFX_LOADED_ALL )) + if( pObj ) //&& !( pObj->Get_Impl()->nLoadedFlags & SfxLoadedFlags::ALL )) { SfxViewFrame* pFrm; for( pFrm = SfxViewFrame::GetFirst( pObj ); diff --git a/sfx2/source/view/viewfrm.cxx b/sfx2/source/view/viewfrm.cxx index c2d63d9..509fa7d 100644 --- a/sfx2/source/view/viewfrm.cxx +++ b/sfx2/source/view/viewfrm.cxx @@ -331,7 +331,7 @@ void SfxViewFrame::ExecReload_Impl( SfxRequest& rReq ) // Due to Double occupancy in toolboxes (with or without Ctrl), // it is also possible that the slot is enabled, but Ctrl-click // despite this is not! - if( !pSh || !pSh->HasName() || !(pSh->Get_Impl()->nLoadedFlags & SFX_LOADED_MAINDOCUMENT )) + if( !pSh || !pSh->HasName() || !(pSh->Get_Impl()->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT )) break; SfxMedium* pMed = pSh->GetMedium(); @@ -862,7 +862,7 @@ void SfxViewFrame::StateReload_Impl( SfxItemSet& rSet ) const SfxShell *pFSh; if ( !pSh || !pSh->HasName() || - !( pSh->Get_Impl()->nLoadedFlags & SFX_LOADED_MAINDOCUMENT ) || + !( pSh->Get_Impl()->nLoadedFlags & SfxLoadedFlags::MAINDOCUMENT ) || ( pSh->GetCreateMode() == SfxObjectCreateMode::EMBEDDED && ( !(pVSh = pSh->GetViewShell()) || !(pFSh = pVSh->GetFormShell()) || diff --git a/starmath/source/document.cxx b/starmath/source/document.cxx index c111245..6252dcd 100644 --- a/starmath/source/document.cxx +++ b/starmath/source/document.cxx @@ -740,7 +740,7 @@ bool SmDocShell::ConvertFrom(SfxMedium &rMedium) Repaint(); } - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); return bSuccess; } @@ -790,7 +790,7 @@ bool SmDocShell::Load( SfxMedium& rMedium ) Repaint(); } - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); return bRet; } diff --git a/sw/source/uibase/app/docsh.cxx b/sw/source/uibase/app/docsh.cxx index a6d1739..4f75597 100644 --- a/sw/source/uibase/app/docsh.cxx +++ b/sw/source/uibase/app/docsh.cxx @@ -1088,7 +1088,7 @@ void SwDocShell::LoadingFinished() // before <FinishedLoading(..)> is called. const bool bHasDocToStayModified( m_pDoc->getIDocumentState().IsModified() && m_pDoc->getIDocumentLinksAdministration().LinksUpdated() ); - FinishedLoading( SFX_LOADED_ALL ); + FinishedLoading( SfxLoadedFlags::ALL ); SfxViewFrame* pVFrame = SfxViewFrame::GetFirst(this); if(pVFrame) { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits