summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarkus Mohrhard <markus.mohrhard@googlemail.com>2016-11-28 01:51:45 +0100
committerMarkus Mohrhard <markus.mohrhard@googlemail.com>2016-11-28 15:52:20 +0000
commit039261b3f63a8949fd0f20e7cb80d017f26c9f08 (patch)
tree069cd3af4db1862d51c277045126737f72bd805b
parent764358a43b83cdae9269af47070e3cb4739c96c0 (diff)
orcus is mandatory now
We use it as a json parser, for calc cell styles and tests. Making it optional is just making the code more complex and introduces more sources for errors. Change-Id: I8769628a4ab9519cafc3d43db7c0007e0aa265dc Reviewed-on: https://gerrit.libreoffice.org/31307 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
-rw-r--r--RepositoryExternal.mk9
-rw-r--r--config_host.mk.in1
-rw-r--r--config_host/config_orcus.h.in6
-rw-r--r--configure.ac74
-rw-r--r--framework/source/loadenv/loadenv.cxx13
-rw-r--r--sc/CppunitTest_sc_subsequent_filters_test.mk3
-rw-r--r--sc/Library_scfilt.mk3
-rw-r--r--sc/qa/unit/helper/qahelper.cxx13
-rw-r--r--sc/qa/unit/subsequent_filters-test.cxx10
-rw-r--r--sc/source/filter/ftools/ftools.cxx8
-rw-r--r--sc/source/filter/html/htmlpars.cxx11
-rw-r--r--sc/source/ui/docshell/datastream.cxx21
-rw-r--r--sc/source/ui/docshell/docsh.cxx3
-rw-r--r--sc/source/ui/docshell/docsh2.cxx3
-rw-r--r--sc/source/ui/view/cellsh2.cxx4
-rw-r--r--sc/source/ui/view/tabvwshc.cxx4
16 files changed, 28 insertions, 158 deletions
diff --git a/RepositoryExternal.mk b/RepositoryExternal.mk
index 9650bf98253e..f024f0525b49 100644
--- a/RepositoryExternal.mk
+++ b/RepositoryExternal.mk
@@ -3128,8 +3128,6 @@ endef
endif # SYSTEM_PYTHON
# ORCUS
-ifeq ($(ENABLE_ORCUS),TRUE)
-
ifneq ($(SYSTEM_LIBORCUS),)
define gb_LinkTarget__use_orcus
@@ -3220,13 +3218,6 @@ endif # MSC
endif # SYSTEM_LIBORCUS
-else # ENABLE_ORCUS != TRUE
-
-gb_LinkTarget__use_orcus :=
-gb_LinkTarget__use_orcus-parser :=
-
-endif
-
ifeq ($(ENABLE_EOT),TRUE)
ifneq ($(SYSTEM_LIBEOT),)
diff --git a/config_host.mk.in b/config_host.mk.in
index 13c0b25a6938..9bf9d69f3a4f 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -132,7 +132,6 @@ export ENABLE_EVOAB2=@ENABLE_EVOAB2@
export ENABLE_FIREBIRD_SDBC=@ENABLE_FIREBIRD_SDBC@
export ENABLE_FORMULA_LOGGER=@ENABLE_FORMULA_LOGGER@
export ENABLE_GIO=@ENABLE_GIO@
-export ENABLE_ORCUS=@ENABLE_ORCUS@
export ENABLE_GLTF=@ENABLE_GLTF@
export SYSTEM_LIBGLTF=@SYSTEM_LIBGLTF@
export LIBGLTF_CFLAGS=@LIBGLTF_CFLAGS@
diff --git a/config_host/config_orcus.h.in b/config_host/config_orcus.h.in
deleted file mode 100644
index 68e3757d5821..000000000000
--- a/config_host/config_orcus.h.in
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef CONFIG_ORCUS_H
-#define CONFIG_ORCUS_H
-
-#define ENABLE_ORCUS 0
-
-#endif
diff --git a/configure.ac b/configure.ac
index 4480ed04527b..77fefaa1dc88 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1044,11 +1044,6 @@ AC_ARG_ENABLE(breakpad,
[Enables breakpad for crash reporting.])
)
-AC_ARG_ENABLE(orcus,
- AS_HELP_STRING([--enable-orcus],
- [Enables orcus for extra file import filters for Calc.])
-)
-
AC_ARG_ENABLE(fetch-external,
AS_HELP_STRING([--disable-fetch-external],
[Disables fetching external tarballs from web sources.])
@@ -9206,48 +9201,36 @@ AC_SUBST(ENABLE_BREAKPAD)
dnl ===================================================================
dnl Orcus
dnl ===================================================================
-
-AC_MSG_CHECKING([whether to enable orcus])
-if test -z "$enable_orcus" -o "$enable_orcus" != no; then
- AC_MSG_RESULT([yes])
- ENABLE_ORCUS="TRUE"
- AC_DEFINE(ENABLE_ORCUS)
-
- libo_CHECK_SYSTEM_MODULE([orcus],[ORCUS],[liborcus-0.12 >= 0.12.0])
- if test "$with_system_orcus" != "yes"; then
- if test "$SYSTEM_BOOST" = "TRUE"; then
- # ===========================================================
- # Determine if we are going to need to link with Boost.System
- # ===========================================================
- dnl This seems to be necessary since boost 1.50 (1.48 does not need it,
- dnl 1.49 is untested). The macro BOOST_THREAD_DONT_USE_SYSTEM mentioned
- dnl in documentation has no effect.
- AC_MSG_CHECKING([if we need to link with Boost.System])
- AC_LANG_PUSH([C++])
- AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
- @%:@include <boost/version.hpp>
- ]],[[
- #if BOOST_VERSION >= 105000
- # error yes, we need to link with Boost.System
- #endif
- ]])],[
- AC_MSG_RESULT([no])
- ],[
- AC_MSG_RESULT([yes])
- AX_BOOST_SYSTEM
- ])
- AC_LANG_POP([C++])
- fi
+libo_CHECK_SYSTEM_MODULE([orcus],[ORCUS],[liborcus-0.12 >= 0.12.0])
+if test "$with_system_orcus" != "yes"; then
+ if test "$SYSTEM_BOOST" = "TRUE"; then
+ # ===========================================================
+ # Determine if we are going to need to link with Boost.System
+ # ===========================================================
+ dnl This seems to be necessary since boost 1.50 (1.48 does not need it,
+ dnl 1.49 is untested). The macro BOOST_THREAD_DONT_USE_SYSTEM mentioned
+ dnl in documentation has no effect.
+ AC_MSG_CHECKING([if we need to link with Boost.System])
+ AC_LANG_PUSH([C++])
+ AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
+ @%:@include <boost/version.hpp>
+ ]],[[
+ #if BOOST_VERSION >= 105000
+ # error yes, we need to link with Boost.System
+ #endif
+ ]])],[
+ AC_MSG_RESULT([no])
+ ],[
+ AC_MSG_RESULT([yes])
+ AX_BOOST_SYSTEM
+ ])
+ AC_LANG_POP([C++])
fi
- dnl FIXME by renaming SYSTEM_LIBORCUS to SYSTEM_ORCUS in the build system world
- SYSTEM_LIBORCUS=$SYSTEM_ORCUS
- AC_SUBST([BOOST_SYSTEM_LIB])
- AC_SUBST(SYSTEM_LIBORCUS)
-
-else
- AC_MSG_RESULT([no])
fi
-AC_SUBST(ENABLE_ORCUS)
+dnl FIXME by renaming SYSTEM_LIBORCUS to SYSTEM_ORCUS in the build system world
+SYSTEM_LIBORCUS=$SYSTEM_ORCUS
+AC_SUBST([BOOST_SYSTEM_LIB])
+AC_SUBST(SYSTEM_LIBORCUS)
dnl ===================================================================
dnl HarfBuzz
@@ -12862,7 +12845,6 @@ AC_CONFIG_HEADERS([config_host/config_lgpl.h])
AC_CONFIG_HEADERS([config_host/config_liblangtag.h])
AC_CONFIG_HEADERS([config_host/config_locales.h])
AC_CONFIG_HEADERS([config_host/config_mpl.h])
-AC_CONFIG_HEADERS([config_host/config_orcus.h])
AC_CONFIG_HEADERS([config_host/config_kde4.h])
AC_CONFIG_HEADERS([config_host/config_oox.h])
AC_CONFIG_HEADERS([config_host/config_opengl.h])
diff --git a/framework/source/loadenv/loadenv.cxx b/framework/source/loadenv/loadenv.cxx
index 27285125f464..8e1288269ede 100644
--- a/framework/source/loadenv/loadenv.cxx
+++ b/framework/source/loadenv/loadenv.cxx
@@ -84,8 +84,6 @@
#include <vcl/svapp.hxx>
#include <cppuhelper/implbase.hxx>
-#include <config_orcus.h>
-
const char PROP_TYPES[] = "Types";
const char PROP_NAME[] = "Name";
@@ -653,8 +651,6 @@ LoadEnv::EContentType LoadEnv::classifyContent(const OUString&
namespace {
-#if ENABLE_ORCUS
-
bool queryOrcusTypeAndFilter(const uno::Sequence<beans::PropertyValue>& rDescriptor, OUString& rType, OUString& rFilter)
{
OUString aURL;
@@ -712,15 +708,6 @@ bool queryOrcusTypeAndFilter(const uno::Sequence<beans::PropertyValue>& rDescrip
return false;
}
-#else
-
-bool queryOrcusTypeAndFilter(const uno::Sequence<beans::PropertyValue>&, OUString&, OUString&)
-{
- return false;
-}
-
-#endif
-
}
void LoadEnv::impl_detectTypeAndFilter()
diff --git a/sc/CppunitTest_sc_subsequent_filters_test.mk b/sc/CppunitTest_sc_subsequent_filters_test.mk
index ef3e197cf7b7..b2e10fd9f439 100644
--- a/sc/CppunitTest_sc_subsequent_filters_test.mk
+++ b/sc/CppunitTest_sc_subsequent_filters_test.mk
@@ -119,7 +119,6 @@ $(eval $(call gb_CppunitTest_use_components,sc_subsequent_filters_test,\
))
endif
-ifeq ($(ENABLE_ORCUS),TRUE)
$(eval $(call gb_CppunitTest_use_externals,sc_subsequent_filters_test,\
orcus \
orcus-parser \
@@ -129,8 +128,6 @@ $(eval $(call gb_CppunitTest_use_externals,sc_subsequent_filters_test,\
zlib \
))
-endif
-
$(eval $(call gb_CppunitTest_use_configuration,sc_subsequent_filters_test))
# vim: set noet sw=4 ts=4:
diff --git a/sc/Library_scfilt.mk b/sc/Library_scfilt.mk
index 467a2d69634f..8abfd870c4de 100644
--- a/sc/Library_scfilt.mk
+++ b/sc/Library_scfilt.mk
@@ -221,7 +221,6 @@ $(eval $(call gb_Library_add_exception_objects,scfilt,\
sc/source/filter/oox/worksheetsettings \
))
-ifeq ($(ENABLE_ORCUS),TRUE)
$(eval $(call gb_Library_use_externals,scfilt,\
orcus \
orcus-parser \
@@ -238,6 +237,4 @@ $(eval $(call gb_Library_add_exception_objects,scfilt,\
sc/source/filter/orcus/filterdetect \
))
-endif
-
# vim: set noet sw=4 ts=4:
diff --git a/sc/qa/unit/helper/qahelper.cxx b/sc/qa/unit/helper/qahelper.cxx
index 3e1b4e5703f9..6599fe18d31a 100644
--- a/sc/qa/unit/helper/qahelper.cxx
+++ b/sc/qa/unit/helper/qahelper.cxx
@@ -22,14 +22,10 @@
#include <cppunit/Asserter.h>
#include <cppunit/AdditionalMessage.h>
-#include <config_orcus.h>
-
-#if ENABLE_ORCUS
#if defined(_WIN32)
#define __ORCUS_STATIC_LIB
#endif
#include <orcus/csv_parser.hpp>
-#endif
#include <fstream>
@@ -120,8 +116,6 @@ void loadFile(const OUString& aFileName, std::string& aContent)
aContent = aOStream.str();
}
-#if ENABLE_ORCUS
-
void testFile(OUString& aFileName, ScDocument& rDoc, SCTAB nTab, StringType aStringFormat)
{
csv_handler aHandler(&rDoc, nTab, aStringFormat);
@@ -170,13 +164,6 @@ void testCondFile(OUString& aFileName, ScDocument* pDoc, SCTAB nTab)
}
}
-#else
-
-void testFile(OUString&, ScDocument&, SCTAB, StringType) {}
-void testCondFile(OUString&, ScDocument*, SCTAB) {}
-
-#endif
-
void testFormats(ScBootstrapFixture* pTest, ScDocument* pDoc, sal_Int32 nFormat)
{
//test Sheet1 with csv file
diff --git a/sc/qa/unit/subsequent_filters-test.cxx b/sc/qa/unit/subsequent_filters-test.cxx
index 30b003b401d5..de3c6d1342ee 100644
--- a/sc/qa/unit/subsequent_filters-test.cxx
+++ b/sc/qa/unit/subsequent_filters-test.cxx
@@ -65,16 +65,12 @@
#include <formula/errorcodes.hxx>
#include "externalrefmgr.hxx"
#include <stlpool.hxx>
-#include <config_orcus.h>
#include <attrib.hxx>
-#if ENABLE_ORCUS
#include <orcusfiltersimpl.hxx>
#include "orcusfilters.hxx"
#include "filter.hxx"
#include "orcusinterface.hxx"
-#endif
-
#include <com/sun/star/drawing/XDrawPageSupplier.hpp>
#include <com/sun/star/drawing/XControlShape.hpp>
@@ -163,9 +159,7 @@ public:
void testCondFormatParentXLSX();
void testColorScaleNumWithRefXLSX();
- #if ENABLE_ORCUS
void testOrcusODSStyleInterface();
- #endif
void testLiteralInFormulaXLS();
@@ -295,9 +289,7 @@ public:
CPPUNIT_TEST(testCondFormatParentXLSX);
CPPUNIT_TEST(testColorScaleNumWithRefXLSX);
- #if ENABLE_ORCUS
CPPUNIT_TEST(testOrcusODSStyleInterface);
- #endif
CPPUNIT_TEST(testLiteralInFormulaXLS);
@@ -2854,7 +2846,6 @@ void ScFiltersTest::testColorScaleNumWithRefXLSX()
xDocSh->DoClose();
}
-#if ENABLE_ORCUS
void ScFiltersTest::testOrcusODSStyleInterface()
{
ScDocument aDoc;
@@ -3069,7 +3060,6 @@ void ScFiltersTest::testOrcusODSStyleInterface()
CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name10 :Error with ver justify", static_cast<sal_uInt16>(SVX_VER_JUSTIFY_CENTER), pVerJustify->GetValue());
}
-#endif
void ScFiltersTest::testLiteralInFormulaXLS()
{
diff --git a/sc/source/filter/ftools/ftools.cxx b/sc/source/filter/ftools/ftools.cxx
index 5b889cc44ce0..ea78de7c3df8 100644
--- a/sc/source/filter/ftools/ftools.cxx
+++ b/sc/source/filter/ftools/ftools.cxx
@@ -35,11 +35,7 @@
#include "stlsheet.hxx"
#include "compiler.hxx"
-#include <config_orcus.h>
-
-#if ENABLE_ORCUS
#include "orcusfiltersimpl.hxx"
-#endif
// ScFilterTools::ReadLongDouble()
@@ -357,12 +353,8 @@ ScFormatFilterPluginImpl::~ScFormatFilterPluginImpl() {}
ScOrcusFilters* ScFormatFilterPluginImpl::GetOrcusFilters()
{
-#if ENABLE_ORCUS
static ScOrcusFiltersImpl aImpl;
return &aImpl;
-#else
- return NULL;
-#endif
}
ScFormatFilterPlugin * SAL_CALL ScFilterCreate()
diff --git a/sc/source/filter/html/htmlpars.cxx b/sc/source/filter/html/htmlpars.cxx
index df3abb0346e0..ec6ef25fbce4 100644
--- a/sc/source/filter/html/htmlpars.cxx
+++ b/sc/source/filter/html/htmlpars.cxx
@@ -56,11 +56,8 @@
#include "document.hxx"
#include "rangelst.hxx"
-#include <config_orcus.h>
#include <o3tl/make_unique.hxx>
-#if ENABLE_ORCUS
#include <orcus/css_parser.hpp>
-#endif
#include <com/sun/star/document/XDocumentProperties.hpp>
#include <com/sun/star/document/XDocumentPropertiesSupplier.hpp>
@@ -3079,8 +3076,6 @@ void ScHTMLQueryParser::CloseTable( const ImportInfo& rInfo )
mpCurrTable = mpCurrTable->CloseTable( rInfo );
}
-#if ENABLE_ORCUS
-
namespace {
/**
@@ -3204,12 +3199,6 @@ void ScHTMLQueryParser::ParseStyle(const OUString& rStrm)
}
}
-#else
-
-void ScHTMLQueryParser::ParseStyle(const OUString&) {}
-
-#endif
-
IMPL_LINK( ScHTMLQueryParser, HTMLImportHdl, ImportInfo&, rInfo, void )
{
switch( rInfo.eState )
diff --git a/sc/source/ui/docshell/datastream.cxx b/sc/source/ui/docshell/datastream.cxx
index 9c6c869a8e28..367b73c2a842 100644
--- a/sc/source/ui/docshell/datastream.cxx
+++ b/sc/source/ui/docshell/datastream.cxx
@@ -26,16 +26,13 @@
#include <stringutil.hxx>
#include <documentlinkmgr.hxx>
-#include <config_orcus.h>
#include "officecfg/Office/Calc.hxx"
-#if ENABLE_ORCUS
#if defined(_WIN32)
#define __ORCUS_STATIC_LIB
#endif
#include <orcus/csv_parser.hpp>
-#endif
#include <queue>
@@ -66,8 +63,6 @@ inline double getNow()
return static_cast<double>(now.Seconds) + static_cast<double>(now.Nanosec) / 1000000000.0;
}
-#if ENABLE_ORCUS
-
class CSVHandler
{
DataStream::Line& mrLine;
@@ -106,8 +101,6 @@ public:
}
};
-#endif
-
}
namespace datastreams {
@@ -135,9 +128,7 @@ class ReaderThread : public salhelper::Thread
osl::Condition maCondReadStream;
osl::Condition maCondConsume;
-#if ENABLE_ORCUS
orcus::csv::parser_config maConfig;
-#endif
public:
@@ -147,10 +138,8 @@ public:
mnColCount(nColCount),
mbTerminate(false)
{
-#if ENABLE_ORCUS
maConfig.delimiters.push_back(',');
maConfig.text_qualifier = '"';
-#endif
}
virtual ~ReaderThread() override
@@ -238,11 +227,9 @@ private:
{
rLine.maCells.clear();
mpStream->ReadLine(rLine.maLine);
-#if ENABLE_ORCUS
CSVHandler aHdl(rLine, mnColCount);
orcus::csv_parser<CSVHandler> parser(rLine.maLine.getStr(), rLine.maLine.getLength(), aHdl, maConfig);
parser.parse();
-#endif
}
aGuard.reset(); // lock
@@ -510,8 +497,6 @@ void DataStream::MoveData()
}
}
-#if ENABLE_ORCUS
-
void DataStream::Text2Doc()
{
Line aLine = ConsumeLine();
@@ -566,12 +551,6 @@ void DataStream::Text2Doc()
++mnLinesSinceRefresh;
}
-#else
-
-void DataStream::Text2Doc() {}
-
-#endif
-
bool DataStream::ImportData()
{
if (!mbValuesInLine)
diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx
index 9d1e8fe8d1c2..cf4590182705 100644
--- a/sc/source/ui/docshell/docsh.cxx
+++ b/sc/source/ui/docshell/docsh.cxx
@@ -116,7 +116,6 @@
#include "refreshtimerprotector.hxx"
#include <orcus/orcus_import_ods.hpp>
#include <orcusfiltersimpl.hxx>
-#include <config_orcus.h>
#include <officecfg/Office/Calc.hxx>
#include <comphelper/processfactory.hxx>
@@ -594,7 +593,6 @@ bool ScDocShell::Load( SfxMedium& rMedium )
aDocument.GetStyleSheetPool()->CreateStandardStyles();
aDocument.UpdStlShtPtrsFrmNms();
-#if ENABLE_ORCUS
if (!mbUcalcTest)
{
/* Create styles that are imported through Orcus */
@@ -610,7 +608,6 @@ bool ScDocShell::Load( SfxMedium& rMedium )
if (pOrcus)
pOrcus->importODS_Styles(aDocument, aPath);
}
-#endif
bRet = LoadXML( &rMedium, nullptr );
}
diff --git a/sc/source/ui/docshell/docsh2.cxx b/sc/source/ui/docshell/docsh2.cxx
index e211ee371c69..539d8f3bd0ca 100644
--- a/sc/source/ui/docshell/docsh2.cxx
+++ b/sc/source/ui/docshell/docsh2.cxx
@@ -26,7 +26,6 @@
#include <editeng/unolingu.hxx>
#include <orcus/orcus_import_ods.hpp>
#include <orcusfiltersimpl.hxx>
-#include <config_orcus.h>
#include <config_folders.h>
#include "drwlayer.hxx"
@@ -60,7 +59,6 @@ bool ScDocShell::InitNew( const uno::Reference < embed::XStorage >& xStor )
aDocument.GetStyleSheetPool()->CreateStandardStyles();
aDocument.UpdStlShtPtrsFrmNms();
-#if ENABLE_ORCUS
if (!mbUcalcTest)
{
/* Create styles that are imported through Orcus */
@@ -75,7 +73,6 @@ bool ScDocShell::InitNew( const uno::Reference < embed::XStorage >& xStor )
if (pOrcus)
pOrcus->importODS_Styles(aDocument, aPath);
}
-#endif
// SetDocumentModified is not allowed anymore in Load/InitNew!
InitItems();
diff --git a/sc/source/ui/view/cellsh2.cxx b/sc/source/ui/view/cellsh2.cxx
index c3073f36c173..3d4277e984a3 100644
--- a/sc/source/ui/view/cellsh2.cxx
+++ b/sc/source/ui/view/cellsh2.cxx
@@ -66,8 +66,6 @@
#include "markdata.hxx"
#include <documentlinkmgr.hxx>
-#include <config_orcus.h>
-
#include <memory>
using namespace com::sun::star;
@@ -1208,11 +1206,9 @@ void ScCellShell::GetDBState( SfxItemSet& rSet )
}
}
break;
-#if !ENABLE_ORCUS
case SID_MANAGE_XML_SOURCE:
rSet.DisableItem(nWhich);
break;
-#endif
}
nWhich = aIter.NextWhich();
}
diff --git a/sc/source/ui/view/tabvwshc.cxx b/sc/source/ui/view/tabvwshc.cxx
index cdc2ee864676..0c478f45ebaa 100644
--- a/sc/source/ui/view/tabvwshc.cxx
+++ b/sc/source/ui/view/tabvwshc.cxx
@@ -77,8 +77,6 @@
#include "PivotLayoutDialog.hxx"
-#include <config_orcus.h>
-
void ScTabViewShell::SetCurRefDlgId( sal_uInt16 nNew )
{
// CurRefDlgId is stored in ScModule to find if a ref dialog is open,
@@ -444,9 +442,7 @@ VclPtr<SfxModelessDialog> ScTabViewShell::CreateRefDialog(
case SID_MANAGE_XML_SOURCE:
{
-#if ENABLE_ORCUS
pResult = VclPtr<ScXMLSourceDlg>::Create(pB, pCW, pParent, pDoc);
-#endif
}
break;