Kaydet (Commit) 20475c78 authored tarafından Tor Lillqvist's avatar Tor Lillqvist

Retire the --disable-export option

It was never fully implemented, and its usefulness is unclear.

Change-Id: I86dee8ee6326bdfaa178260c76188261aef2fb4c
üst dc199952
...@@ -97,7 +97,6 @@ export DISABLE_ACTIVEX=@DISABLE_ACTIVEX@ ...@@ -97,7 +97,6 @@ export DISABLE_ACTIVEX=@DISABLE_ACTIVEX@
export DISABLE_ATL=@DISABLE_ATL@ export DISABLE_ATL=@DISABLE_ATL@
export DISABLE_CVE_TESTS=@DISABLE_CVE_TESTS@ export DISABLE_CVE_TESTS=@DISABLE_CVE_TESTS@
export DISABLE_DYNLOADING=@DISABLE_DYNLOADING@ export DISABLE_DYNLOADING=@DISABLE_DYNLOADING@
export DISABLE_EXPORT=@DISABLE_EXPORT@
export DISABLE_OPENSSL=@DISABLE_OPENSSL@ export DISABLE_OPENSSL=@DISABLE_OPENSSL@
export DISABLE_PYTHON=@DISABLE_PYTHON@ export DISABLE_PYTHON=@DISABLE_PYTHON@
export DLOPEN_LIBS=@DLOPEN_LIBS@ export DLOPEN_LIBS=@DLOPEN_LIBS@
......
...@@ -967,13 +967,6 @@ libo_FUZZ_ARG_ENABLE(extension-integration, ...@@ -967,13 +967,6 @@ libo_FUZZ_ARG_ENABLE(extension-integration,
product. Use this switch to disable the integration.]) product. Use this switch to disable the integration.])
) )
libo_FUZZ_ARG_ENABLE(export,
AS_HELP_STRING([--disable-export],
[Disable (some) code for document export. Useful when building viewer-only apps that lack
save/export functionality, to avoid having an excessive amount of code and data used
only for exporrt linked in. Work in progress, use only if you are hacking on it.])
)
AC_ARG_ENABLE(avmedia, AC_ARG_ENABLE(avmedia,
AS_HELP_STRING([--disable-avmedia], AS_HELP_STRING([--disable-avmedia],
[Disable displaying and inserting AV media in documents. Work in progress, use only if you are hacking on it.]) [Disable displaying and inserting AV media in documents. Work in progress, use only if you are hacking on it.])
...@@ -2464,15 +2457,6 @@ if test $_os != iOS -a $_os != Android; then ...@@ -2464,15 +2457,6 @@ if test $_os != iOS -a $_os != Android; then
AC_DEFINE(HAVE_FEATURE_MULTIUSER_ENVIRONMENT) AC_DEFINE(HAVE_FEATURE_MULTIUSER_ENVIRONMENT)
fi fi
DISABLE_EXPORT=''
if test "$enable_export" != no; then
BUILD_TYPE="$BUILD_TYPE EXPORT"
else
DISABLE_EXPORT='TRUE'
SCPDEFS="$SCPDES -DDISABLE_EXPORT"
fi
AC_SUBST(DISABLE_EXPORT)
# Whether to build "avmedia" functionality or not. # Whether to build "avmedia" functionality or not.
if test -z "$enable_avmedia"; then if test -z "$enable_avmedia"; then
......
...@@ -21,7 +21,6 @@ ...@@ -21,7 +21,6 @@
--disable-systray --disable-systray
--disable-gltf --disable-gltf
--disable-collada --disable-collada
--disable-export
--disable-report-builder --disable-report-builder
--disable-lpsolve --disable-lpsolve
--disable-coinmp --disable-coinmp
......
...@@ -70,10 +70,9 @@ $(eval $(call gb_Library_add_exception_objects,svgfilter,\ ...@@ -70,10 +70,9 @@ $(eval $(call gb_Library_add_exception_objects,svgfilter,\
filter/source/svg/svgreader \ filter/source/svg/svgreader \
filter/source/svg/tokenmap \ filter/source/svg/tokenmap \
filter/source/svg/units \ filter/source/svg/units \
$(if $(filter EXPORT,$(BUILD_TYPE)), \ filter/source/svg/svgexport \
filter/source/svg/svgexport \ filter/source/svg/svgfontexport \
filter/source/svg/svgfontexport \ filter/source/svg/svgwriter \
filter/source/svg/svgwriter) \
)) ))
# vim: set noet sw=4 ts=4: # vim: set noet sw=4 ts=4:
...@@ -103,7 +103,6 @@ sal_Bool SAL_CALL SVGFilter::filter( const Sequence< PropertyValue >& rDescripto ...@@ -103,7 +103,6 @@ sal_Bool SAL_CALL SVGFilter::filter( const Sequence< PropertyValue >& rDescripto
if( mxDstDoc.is() ) if( mxDstDoc.is() )
bRet = implImport( rDescriptor ); bRet = implImport( rDescriptor );
#ifndef DISABLE_EXPORT
else if( mxSrcDoc.is() ) else if( mxSrcDoc.is() )
{ {
// #i124608# detect selection // #i124608# detect selection
...@@ -274,7 +273,6 @@ sal_Bool SAL_CALL SVGFilter::filter( const Sequence< PropertyValue >& rDescripto ...@@ -274,7 +273,6 @@ sal_Bool SAL_CALL SVGFilter::filter( const Sequence< PropertyValue >& rDescripto
bRet = implExport( rDescriptor ); bRet = implExport( rDescriptor );
} }
} }
#endif
else else
bRet = false; bRet = false;
......
...@@ -218,10 +218,8 @@ private: ...@@ -218,10 +218,8 @@ private:
UCharSetMapMap mTextFieldCharSets; UCharSetMapMap mTextFieldCharSets;
Reference< XInterface > mCreateOjectsCurrentMasterPage; Reference< XInterface > mCreateOjectsCurrentMasterPage;
UOStringMap mTextShapeIdListMap; UOStringMap mTextShapeIdListMap;
#ifndef DISABLE_EXPORT
MetaBitmapActionSet mEmbeddedBitmapActionSet; MetaBitmapActionSet mEmbeddedBitmapActionSet;
ObjectMap mEmbeddedBitmapActionMap; ObjectMap mEmbeddedBitmapActionMap;
#endif
ObjectMap* mpObjects; ObjectMap* mpObjects;
Reference< XComponent > mxSrcDoc; Reference< XComponent > mxSrcDoc;
Reference< XComponent > mxDstDoc; Reference< XComponent > mxDstDoc;
......
...@@ -236,7 +236,6 @@ endif ...@@ -236,7 +236,6 @@ endif
gb_GLOBALDEFS += \ gb_GLOBALDEFS += \
$(call gb_Helper_define_if_set,\ $(call gb_Helper_define_if_set,\
DISABLE_DYNLOADING \ DISABLE_DYNLOADING \
DISABLE_EXPORT \
ENABLE_LTO \ ENABLE_LTO \
) )
......
...@@ -103,8 +103,6 @@ public: ...@@ -103,8 +103,6 @@ public:
explicit ImpFilterOutputStream( SvStream& rStm ) : mrStm( rStm ) {} explicit ImpFilterOutputStream( SvStream& rStm ) : mrStm( rStm ) {}
}; };
#ifndef DISABLE_EXPORT
static bool DirEntryExists( const INetURLObject& rObj ) static bool DirEntryExists( const INetURLObject& rObj )
{ {
bool bExists = false; bool bExists = false;
...@@ -153,8 +151,6 @@ static void KillDirEntry( const OUString& rMainUrl ) ...@@ -153,8 +151,6 @@ static void KillDirEntry( const OUString& rMainUrl )
} }
} }
#endif // !DISABLE_EXPORT
// Helper functions // Helper functions
sal_uInt8* ImplSearchEntry( sal_uInt8* pSource, sal_uInt8 const * pDest, sal_uLong nComp, sal_uLong nSize ) sal_uInt8* ImplSearchEntry( sal_uInt8* pSource, sal_uInt8 const * pDest, sal_uLong nComp, sal_uLong nSize )
...@@ -821,8 +817,6 @@ sal_uInt16 GraphicFilter::ImpTestOrFindFormat( const OUString& rPath, SvStream& ...@@ -821,8 +817,6 @@ sal_uInt16 GraphicFilter::ImpTestOrFindFormat( const OUString& rPath, SvStream&
return GRFILTER_OK; return GRFILTER_OK;
} }
#ifndef DISABLE_EXPORT
static Graphic ImpGetScaledGraphic( const Graphic& rGraphic, FilterConfigItem& rConfigItem ) static Graphic ImpGetScaledGraphic( const Graphic& rGraphic, FilterConfigItem& rConfigItem )
{ {
Graphic aGraphic; Graphic aGraphic;
...@@ -919,8 +913,6 @@ static Graphic ImpGetScaledGraphic( const Graphic& rGraphic, FilterConfigItem& r ...@@ -919,8 +913,6 @@ static Graphic ImpGetScaledGraphic( const Graphic& rGraphic, FilterConfigItem& r
return aGraphic; return aGraphic;
} }
#endif
static OUString ImpCreateFullFilterPath( const OUString& rPath, const OUString& rFilterName ) static OUString ImpCreateFullFilterPath( const OUString& rPath, const OUString& rFilterName )
{ {
OUString aPathURL; OUString aPathURL;
...@@ -1797,14 +1789,6 @@ sal_uInt16 GraphicFilter::ImportGraphic( Graphic& rGraphic, const OUString& rPat ...@@ -1797,14 +1789,6 @@ sal_uInt16 GraphicFilter::ImportGraphic( Graphic& rGraphic, const OUString& rPat
sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const INetURLObject& rPath, sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const INetURLObject& rPath,
sal_uInt16 nFormat, const css::uno::Sequence< css::beans::PropertyValue >* pFilterData ) sal_uInt16 nFormat, const css::uno::Sequence< css::beans::PropertyValue >* pFilterData )
{ {
#ifdef DISABLE_EXPORT
(void) rGraphic;
(void) rPath;
(void) nFormat;
(void) pFilterData;
return GRFILTER_FORMATERROR;
#else
SAL_INFO( "vcl.filter", "GraphicFilter::ExportGraphic() (thb)" ); SAL_INFO( "vcl.filter", "GraphicFilter::ExportGraphic() (thb)" );
sal_uInt16 nRetValue = GRFILTER_FORMATERROR; sal_uInt16 nRetValue = GRFILTER_FORMATERROR;
SAL_WARN_IF( rPath.GetProtocol() == INetProtocol::NotValid, "vcl", "GraphicFilter::ExportGraphic() : ProtType == INetProtocol::NotValid" ); SAL_WARN_IF( rPath.GetProtocol() == INetProtocol::NotValid, "vcl", "GraphicFilter::ExportGraphic() : ProtType == INetProtocol::NotValid" );
...@@ -1821,35 +1805,19 @@ sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const INetURLO ...@@ -1821,35 +1805,19 @@ sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const INetURLO
KillDirEntry( aMainUrl ); KillDirEntry( aMainUrl );
} }
return nRetValue; return nRetValue;
#endif
} }
#ifdef DISABLE_DYNLOADING #ifdef DISABLE_DYNLOADING
#ifndef DISABLE_EXPORT
extern "C" bool egiGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem ); extern "C" bool egiGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem );
extern "C" bool epsGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem ); extern "C" bool epsGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem );
extern "C" bool etiGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem ); extern "C" bool etiGraphicExport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem );
#endif #endif
#endif
sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const OUString& rPath, sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const OUString& rPath,
SvStream& rOStm, sal_uInt16 nFormat, const css::uno::Sequence< css::beans::PropertyValue >* pFilterData ) SvStream& rOStm, sal_uInt16 nFormat, const css::uno::Sequence< css::beans::PropertyValue >* pFilterData )
{ {
#ifdef DISABLE_EXPORT
(void) rGraphic;
(void) rPath;
(void) rOStm;
(void) nFormat;
(void) pFilterData;
(void) nExpGraphHint;
return GRFILTER_FORMATERROR;
#else
SAL_INFO( "vcl.filter", "GraphicFilter::ExportGraphic() (thb)" ); SAL_INFO( "vcl.filter", "GraphicFilter::ExportGraphic() (thb)" );
sal_uInt16 nFormatCount = GetExportFormatCount(); sal_uInt16 nFormatCount = GetExportFormatCount();
...@@ -2150,7 +2118,6 @@ sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const OUString ...@@ -2150,7 +2118,6 @@ sal_uInt16 GraphicFilter::ExportGraphic( const Graphic& rGraphic, const OUString
ImplSetError( nStatus, &rOStm ); ImplSetError( nStatus, &rOStm );
} }
return nStatus; return nStatus;
#endif
} }
...@@ -2194,14 +2161,13 @@ IMPL_LINK( GraphicFilter, FilterCallback, ConvertData&, rData, bool ) ...@@ -2194,14 +2161,13 @@ IMPL_LINK( GraphicFilter, FilterCallback, ConvertData&, rData, bool )
nFormat = GetImportFormatNumberForShortName( OStringToOUString( aShortName, RTL_TEXTENCODING_UTF8) ); nFormat = GetImportFormatNumberForShortName( OStringToOUString( aShortName, RTL_TEXTENCODING_UTF8) );
bRet = ImportGraphic( rData.maGraphic, OUString(), rData.mrStm, nFormat ) == 0; bRet = ImportGraphic( rData.maGraphic, OUString(), rData.mrStm, nFormat ) == 0;
} }
#ifndef DISABLE_EXPORT
else if( !aShortName.isEmpty() ) else if( !aShortName.isEmpty() )
{ {
// Export // Export
nFormat = GetExportFormatNumberForShortName( OStringToOUString(aShortName, RTL_TEXTENCODING_UTF8) ); nFormat = GetExportFormatNumberForShortName( OStringToOUString(aShortName, RTL_TEXTENCODING_UTF8) );
bRet = ExportGraphic( rData.maGraphic, OUString(), rData.mrStm, nFormat ) == 0; bRet = ExportGraphic( rData.maGraphic, OUString(), rData.mrStm, nFormat ) == 0;
} }
#endif
return bRet; return bRet;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment