commit 91570fc3f37bedd66eb11b7fb585e207e51abeaa
author Michael Meeks <michael.meeks@collabora.com> Tue Dec 31 12:04:21 2013 +0000
committer Michael Meeks <michael.meeks@collabora.com> Tue Jan 14 15:52:11 2014 +0000
tree 2a328e5b803328bfff79b6da2aee3e9954b58a89
parent 2da415ee45ff95e847275f6f39b808871f0afed9

fastparser: avoid boost::optional where it is un-necessary.

boost::optional appears to show up rather heavily on many profiles.
We already use mnElementToken == DONTKNOW to flag / use these guys.

Change-Id: Ibf2b0167f259cc601da2fb9703e880b78e60886e

sax/source/fastparser/fastparser.cxx

diff --git a/sax/source/fastparser/fastparser.cxx b/sax/source/fastparser/fastparser.cxx
index 8f02e4f..8c8ab1b 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -96,10 +96,10 @@ struct NameWithToken

struct SaxContext
{
    ::com::sun::star::uno::Reference< ::com::sun::star::xml::sax::XFastContextHandler > mxContext;
    sal_Int32                   mnElementToken;
    boost::optional< OUString > maNamespace;
    boost::optional< OUString > maElementName;
    Reference< XFastContextHandler > mxContext;
    sal_Int32 mnElementToken;
    OUString  maNamespace;
    OUString  maElementName;

    SaxContext( sal_Int32 nElementToken, const OUString& aNamespace, const OUString& aElementName ):
            mnElementToken(nElementToken)
@@ -488,7 +488,7 @@ void Entity::endElement()
        if( nElementToken != FastToken::DONTKNOW )
            xContext->endFastElement( nElementToken );
        else
            xContext->endUnknownElement( aContext.maNamespace.get(), aContext.maElementName.get() );
            xContext->endUnknownElement( aContext.maNamespace, aContext.maElementName );
    }
    catch (const Exception& e)
    {