sax/source/fastparser/fastparser.cxx |  135 -----------------------------------
 1 file changed, 135 deletions(-)

New commits:
commit 840a97f98f03491256a93d95e6614d41cdcddbcf
Author: Thorsten Behrens <thorsten.behr...@cib.de>
Date:   Fri Aug 11 00:10:30 2017 +0200

    sax parser: remove long-disabled code
    
    Entity ref and decl was disabled since 2014, let's remove it now.
    
    Change-Id: I35dac78b8530bab99e955bf68dfe3f8951a63f99
    Reviewed-on: https://gerrit.libreoffice.org/40998
    Reviewed-by: Thorsten Behrens <thorsten.behr...@cib.de>
    Tested-by: Thorsten Behrens <thorsten.behr...@cib.de>

diff --git a/sax/source/fastparser/fastparser.cxx 
b/sax/source/fastparser/fastparser.cxx
index 4cdd3b034962..8188724a91d9 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -238,13 +238,6 @@ public:
     void callbackEndElement();
     void callbackCharacters( const xmlChar* s, int nLen );
     void callbackProcessingInstruction( const xmlChar *target, const xmlChar 
*data );
-#if 0
-    bool callbackExternalEntityRef( XML_Parser parser, const xmlChar 
*openEntityNames, const xmlChar *base, const xmlChar *systemId, const xmlChar 
*publicId);
-    void callbackEntityDecl(const xmlChar *entityName, int is_parameter_entity,
-            const xmlChar *value, int value_length, const xmlChar *base,
-            const xmlChar *systemId, const xmlChar *publicId,
-            const xmlChar *notationName);
-#endif
 
     void pushEntity( const Entity& rEntity );
     void popEntity();
@@ -268,9 +261,6 @@ private:
     void DefineNamespace( const OString& rPrefix, const OUString& namespaceURL 
);
 
 private:
-#if 0
-    FastSaxParser* mpFront;
-#endif
     osl::Mutex maMutex; ///< Protecting whole parseStream() execution
     ::rtl::Reference< FastLocatorImpl >     mxDocumentLocator;
     NamespaceMap                            maNamespaceMap;
@@ -334,24 +324,6 @@ static void call_callbackProcessingInstruction( void 
*userData, const xmlChar *t
     pFastParser->callbackProcessingInstruction( target, data );
 }
 
-#if 0
-static void call_callbackEntityDecl(void *userData, const xmlChar *entityName,
-        int is_parameter_entity, const xmlChar *value, int value_length,
-        const xmlChar *base, const xmlChar *systemId,
-        const xmlChar *publicId, const xmlChar *notationName)
-{
-    FastSaxParserImpl* pFastParser = 
reinterpret_cast<FastSaxParserImpl*>(userData);
-    pFastParser->callbackEntityDecl(entityName, is_parameter_entity, value,
-            value_length, base, systemId, publicId, notationName);
-}
-
-static int call_callbackExternalEntityRef( XML_Parser parser,
-        const xmlChar *openEntityNames, const xmlChar *base, const xmlChar 
*systemId, const xmlChar *publicId )
-{
-    FastSaxParserImpl* pFastParser = reinterpret_cast<FastSaxParserImpl*>( 
XML_GetUserData( parser ) );
-    return pFastParser->callbackExternalEntityRef( parser, openEntityNames, 
base, systemId, publicId );
-}
-#endif
 }
 
 class FastLocatorImpl : public WeakImplHelper< XLocator >
@@ -651,9 +623,6 @@ void Entity::saveException( const Any & e )
 namespace sax_fastparser {
 
 FastSaxParserImpl::FastSaxParserImpl() :
-#if 0
-    mpFront(pFront),
-#endif
     m_bIgnoreMissingNSDecl(false),
     mpTop(nullptr)
 {
@@ -1040,10 +1009,6 @@ void FastSaxParserImpl::parse()
     callbacks.characters = call_callbackCharacters;
     callbacks.processingInstruction = call_callbackProcessingInstruction;
     callbacks.initialized = XML_SAX2_MAGIC;
-#if 0
-    XML_SetEntityDeclHandler(entity.mpParser, call_callbackEntityDecl);
-    XML_SetExternalEntityRefHandler( entity.mpParser, 
call_callbackExternalEntityRef );
-#endif
     int nRead = 0;
     do
     {
@@ -1305,106 +1270,6 @@ void FastSaxParserImpl::callbackProcessingInstruction( 
const xmlChar *target, co
         rEntity.processingInstruction( rEvent.msNamespace, 
rEvent.msElementName );
 }
 
-#if 0
-void FastSaxParserImpl::callbackEntityDecl(
-    SAL_UNUSED_PARAMETER const xmlChar * /*entityName*/,
-    SAL_UNUSED_PARAMETER int /*is_parameter_entity*/,
-    const xmlChar *value, SAL_UNUSED_PARAMETER int /*value_length*/,
-    SAL_UNUSED_PARAMETER const xmlChar * /*base*/,
-    SAL_UNUSED_PARAMETER const xmlChar * /*systemId*/,
-    SAL_UNUSED_PARAMETER const xmlChar * /*publicId*/,
-    SAL_UNUSED_PARAMETER const xmlChar * /*notationName*/)
-{
-    if (value) { // value != 0 means internal entity
-        SAL_INFO("sax", "FastSaxParser: internal entity declaration, 
stopping");
-        XML_StopParser(getEntity().mpParser, XML_FALSE);
-        getEntity().saveException( SAXParseException(
-            "FastSaxParser: internal entity declaration, stopping",
-            static_cast<OWeakObject*>(mpFront), Any(),
-            mxDocumentLocator->getPublicId(),
-            mxDocumentLocator->getSystemId(),
-            mxDocumentLocator->getLineNumber(),
-            mxDocumentLocator->getColumnNumber() ) );
-    } else {
-        SAL_INFO("sax", "FastSaxParser: ignoring external entity declaration");
-    }
-}
-
-bool FastSaxParserImpl::callbackExternalEntityRef(
-    XML_Parser parser, const xmlChar *context,
-    SAL_UNUSED_PARAMETER const xmlChar * /*base*/, const xmlChar *systemId,
-    const xmlChar *publicId )
-{
-    bool bOK = true;
-    InputSource source;
-
-    Entity& rCurrEntity = getEntity();
-    Entity aNewEntity( rCurrEntity );
-
-    if( rCurrEntity.mxEntityResolver.is() ) try
-    {
-        aNewEntity.maStructSource = 
rCurrEntity.mxEntityResolver->resolveEntity(
-            OUString( publicId, strlen( publicId ), RTL_TEXTENCODING_UTF8 ) ,
-            OUString( systemId, strlen( systemId ), RTL_TEXTENCODING_UTF8 ) );
-    }
-    catch (const SAXParseException & e)
-    {
-        rCurrEntity.saveException( e );
-        bOK = false;
-    }
-    catch (const SAXException& e)
-    {
-        rCurrEntity.saveException( SAXParseException(
-            e.Message, e.Context, e.WrappedException,
-            mxDocumentLocator->getPublicId(),
-            mxDocumentLocator->getSystemId(),
-            mxDocumentLocator->getLineNumber(),
-            mxDocumentLocator->getColumnNumber() ) );
-        bOK = false;
-    }
-
-    if( aNewEntity.maStructSource.aInputStream.is() )
-    {
-        aNewEntity.mpParser = XML_ExternalEntityParserCreate( parser, context, 
0 );
-        if( !aNewEntity.mpParser )
-        {
-            return false;
-        }
-
-        aNewEntity.maConverter.setInputStream( 
aNewEntity.maStructSource.aInputStream );
-        pushEntity( aNewEntity );
-        try
-        {
-            parse();
-        }
-        catch (const SAXParseException& e)
-        {
-            rCurrEntity.saveException( e );
-            bOK = false;
-        }
-        catch (const IOException& e)
-        {
-            SAXException aEx;
-            aEx.WrappedException <<= e;
-            rCurrEntity.saveException( aEx );
-            bOK = false;
-        }
-        catch (const RuntimeException& e)
-        {
-            SAXException aEx;
-            aEx.WrappedException <<= e;
-            rCurrEntity.saveException( aEx );
-            bOK = false;
-        }
-
-        popEntity();
-        XML_ParserFree( aNewEntity.mpParser );
-    }
-
-    return bOK;
-}
-#endif
-
 FastSaxParser::FastSaxParser() : mpImpl(new FastSaxParserImpl) {}
 
 FastSaxParser::~FastSaxParser()
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to