diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-02-06 11:42:25 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-02-06 11:36:17 +0000 |
| commit | fde0b8df30a65412afd5278a0dfd23ac30f62dc6 (patch) | |
| tree | 3e373d73bf50c64bc4a01af4bd254f83628ea5fc /oox | |
| parent | 3ea39bda0e948fde338bd9493639f4a6e8a8ea5b (diff) | |
loplugin:unusedenumconstants in oox
Convert FragmentHandler2::MCE_STATE to scoped enum and drop MCE_UNUSED
constant
Change-Id: Id0b3a81e61d77af5d3837527b008e196835f57cd
Reviewed-on: https://gerrit.libreoffice.org/33954
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'oox')
| -rw-r--r-- | oox/source/core/fragmenthandler2.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/oox/source/core/fragmenthandler2.cxx b/oox/source/core/fragmenthandler2.cxx index fe9b827a6aae..4bd32f4e7a77 100644 --- a/oox/source/core/fragmenthandler2.cxx +++ b/oox/source/core/fragmenthandler2.cxx @@ -58,12 +58,12 @@ bool FragmentHandler2::prepareMceContext( sal_Int32 nElement, const AttributeLis switch( nElement ) { case MCE_TOKEN( AlternateContent ): - aMceState.push_back( MCE_STARTED ); + aMceState.push_back( MCE_STATE::Started ); break; case MCE_TOKEN( Choice ): { - if (aMceState.empty() || aMceState.back() != MCE_STARTED) + if (aMceState.empty() || aMceState.back() != MCE_STATE::Started) return false; OUString aRequires = rAttribs.getString( (XML_Requires ), "none" ); @@ -80,14 +80,14 @@ bool FragmentHandler2::prepareMceContext( sal_Int32 nElement, const AttributeLis }; if (std::find(aSupportedNS.begin(), aSupportedNS.end(), aRequires) != aSupportedNS.end()) - aMceState.back() = MCE_FOUND_CHOICE; + aMceState.back() = MCE_STATE::FoundChoice; else return false; } break; case MCE_TOKEN( Fallback ): - if( !aMceState.empty() && aMceState.back() == MCE_STARTED ) + if( !aMceState.empty() && aMceState.back() == MCE_STATE::Started ) break; return false; default: |