desktop/source/lib/init.cxx | 3 filter/CppunitTest_filter_dxf_test.mk | 39 ---- filter/Library_gie.mk | 8 filter/Module_filter.mk | 1 filter/source/config/fragments/internalgraphicfilters/dxf_Import.xcu | 21 +- include/vcl/graphicfilter.hxx | 5 solenv/clang-format/excludelist | 32 +-- vcl/CppunitTest_vcl_filters_test.mk | 1 vcl/Library_vcl.mk | 8 vcl/inc/filter/DxfReader.hxx | 26 +++ vcl/qa/cppunit/graphicfilter/filters-dxf-test.cxx | 18 -- vcl/source/filter/FilterConfigCache.cxx | 4 vcl/source/filter/graphicfilter.cxx | 84 +--------- vcl/source/filter/idxf/idxf.cxx | 6 vcl/workben/dxffuzzer.cxx | 5 vcl/workben/fftester.cxx | 14 - 16 files changed, 94 insertions(+), 181 deletions(-)
New commits: commit d3476d27ab77160dcd737194fc13081ee297db16 Author: Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk> AuthorDate: Mon Feb 15 11:35:38 2021 +0900 Commit: Tomaž Vajngerl <qui...@gmail.com> CommitDate: Wed Feb 17 09:55:46 2021 +0100 Move DXF reader from filter module into VCL Change-Id: If1a933256401c1a971f581d1b389ca06fd77eaae Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111022 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <qui...@gmail.com> diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx index 95e76de18cdb..b400c24ab55c 100644 --- a/desktop/source/lib/init.cxx +++ b/desktop/source/lib/init.cxx @@ -6276,9 +6276,6 @@ static int lo_initialize(LibreOfficeKit* pThis, const char* pAppPath, const char InitVCL(); } - // pre-load all graphic libraries. - GraphicFilter::GetGraphicFilter().preload(); - // pre-load all component libraries. if (!xContext.is()) throw css::uno::DeploymentException("preInit: XComponentContext is not created"); diff --git a/filter/CppunitTest_filter_dxf_test.mk b/filter/CppunitTest_filter_dxf_test.mk deleted file mode 100644 index 4a10c1845cef..000000000000 --- a/filter/CppunitTest_filter_dxf_test.mk +++ /dev/null @@ -1,39 +0,0 @@ -# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*- -# -# This file is part of the LibreOffice project. -# -# This Source Code Form is subject to the terms of the Mozilla Public -# License, v. 2.0. If a copy of the MPL was not distributed with this -# file, You can obtain one at http://mozilla.org/MPL/2.0/. -# - -$(eval $(call gb_CppunitTest_CppunitTest,filter_dxf_test)) - -$(eval $(call gb_CppunitTest_use_external,filter_dxf_test,boost_headers)) - -$(eval $(call gb_CppunitTest_add_exception_objects,filter_dxf_test, \ - filter/qa/cppunit/filters-dxf-test \ -)) - -$(eval $(call gb_CppunitTest_use_libraries,filter_dxf_test, \ - gie \ - sal \ - test \ - tl \ - unotest \ - vcl \ -)) - -$(eval $(call gb_CppunitTest_use_sdk_api,filter_dxf_test)) - -$(eval $(call gb_CppunitTest_use_ure,filter_dxf_test)) -$(eval $(call gb_CppunitTest_use_vcl,filter_dxf_test)) - -$(eval $(call gb_CppunitTest_use_components,filter_dxf_test,\ - configmgr/source/configmgr \ - i18npool/util/i18npool \ -)) - -$(eval $(call gb_CppunitTest_use_configuration,filter_dxf_test)) - -# vim: set noet sw=4 ts=4: diff --git a/filter/Library_gie.mk b/filter/Library_gie.mk index e04eb08fda7f..b40da250a183 100644 --- a/filter/Library_gie.mk +++ b/filter/Library_gie.mk @@ -51,14 +51,6 @@ $(eval $(call gb_Library_use_libraries,gie,\ $(eval $(call gb_Library_add_exception_objects,gie,\ filter/source/graphicfilter/egif/egif \ filter/source/graphicfilter/egif/giflzwc \ - filter/source/graphicfilter/idxf/dxf2mtf \ - filter/source/graphicfilter/idxf/dxfblkrd \ - filter/source/graphicfilter/idxf/dxfentrd \ - filter/source/graphicfilter/idxf/dxfgrprd \ - filter/source/graphicfilter/idxf/dxfreprd \ - filter/source/graphicfilter/idxf/dxftblrd \ - filter/source/graphicfilter/idxf/dxfvec \ - filter/source/graphicfilter/idxf/idxf \ )) # vim: set noet sw=4 ts=4: diff --git a/filter/Module_filter.mk b/filter/Module_filter.mk index 607a5fc46d0c..2fb2e4c0c8e3 100644 --- a/filter/Module_filter.mk +++ b/filter/Module_filter.mk @@ -55,7 +55,6 @@ $(eval $(call gb_Module_add_check_targets,filter,\ ifneq ($(DISABLE_CVE_TESTS),TRUE) $(eval $(call gb_Module_add_check_targets,filter,\ - CppunitTest_filter_dxf_test \ CppunitTest_filter_svg \ )) endif diff --git a/filter/source/config/fragments/internalgraphicfilters/dxf_Import.xcu b/filter/source/config/fragments/internalgraphicfilters/dxf_Import.xcu index 1ab3279bd809..d4ce34e28b1e 100644 --- a/filter/source/config/fragments/internalgraphicfilters/dxf_Import.xcu +++ b/filter/source/config/fragments/internalgraphicfilters/dxf_Import.xcu @@ -15,13 +15,14 @@ * except in compliance with the License. You may obtain a copy of * the License at http://www.apache.org/licenses/LICENSE-2.0 . --> - <node oor:name="dxf_Import" oor:op="replace" > - <prop oor:name="Type"><value>dxf_AutoCAD_Interchange</value></prop> - <prop oor:name="FormatName"><value>idx</value></prop> - <prop oor:name="RealFilterName"><value>DXF - AutoCAD Interchange</value></prop> - <prop oor:name="UIComponent"/> - <prop oor:name="UIName"> - <value xml:lang="en-US">DXF - AutoCAD Interchange Format</value> - </prop> - <prop oor:name="Flags"><value>IMPORT</value></prop> - </node> + +<node oor:name="dxf_Import" oor:op="replace" > + <prop oor:name="Type"><value>dxf_AutoCAD_Interchange</value></prop> + <prop oor:name="FormatName"><value>SVDXF</value></prop> + <prop oor:name="RealFilterName"><value>DXF - AutoCAD Interchange</value></prop> + <prop oor:name="UIComponent"/> + <prop oor:name="UIName"> + <value xml:lang="en-US">DXF - AutoCAD Interchange Format</value> + </prop> + <prop oor:name="Flags"><value>IMPORT</value></prop> +</node> diff --git a/include/vcl/graphicfilter.hxx b/include/vcl/graphicfilter.hxx index f0d0511aac97..a116a5085619 100644 --- a/include/vcl/graphicfilter.hxx +++ b/include/vcl/graphicfilter.hxx @@ -88,6 +88,8 @@ namespace o3tl #define IMP_PSD "SVPSD" #define IMP_PCD "SVPCD" #define IMP_PBM "SVPBM" +#define IMP_DXF "SVDXF" + #define EXP_BMP "SVBMP" #define EXP_SVMETAFILE "SVMETAFILE" #define EXP_WMF "SVWMF" @@ -350,8 +352,6 @@ public: ErrCode compressAsPNG(const Graphic& rGraphic, SvStream& rOutputStream); - void preload(); - static ErrCode readGIF(SvStream& rStream, Graphic& rGraphic, GfxLinkType& rLinkType); static ErrCode readPNG(SvStream & rStream, Graphic & rGraphic, GfxLinkType & rLinkType, std::unique_ptr<sal_uInt8[]> & rpGraphicContent, sal_Int32& rGraphicContentSize); @@ -379,6 +379,7 @@ public: static ErrCode readPSD(SvStream & rStream, Graphic & rGraphic); static ErrCode readPCD(SvStream & rStream, Graphic & rGraphic); static ErrCode readPBM(SvStream & rStream, Graphic & rGraphic); + static ErrCode readDXF(SvStream & rStream, Graphic & rGraphic); private: OUString aFilterPath; diff --git a/solenv/clang-format/excludelist b/solenv/clang-format/excludelist index 03fe8e0d3c52..4b3e68b22373 100644 --- a/solenv/clang-format/excludelist +++ b/solenv/clang-format/excludelist @@ -3897,7 +3897,6 @@ external/unixODBC/inc/odbc/sqlext.h external/unixODBC/inc/odbc/sqltypes.h external/unixODBC/inc/odbc/sqlucode.h filter/inc/strings.hxx -filter/qa/cppunit/filters-dxf-test.cxx filter/qa/cppunit/msfilter-test.cxx filter/qa/cppunit/priority-test.cxx filter/qa/cppunit/xslt-test.cxx @@ -3948,21 +3947,6 @@ filter/source/graphicfilter/icgm/classx.cxx filter/source/graphicfilter/icgm/elements.cxx filter/source/graphicfilter/icgm/elements.hxx filter/source/graphicfilter/icgm/outact.hxx -filter/source/graphicfilter/idxf/dxf2mtf.cxx -filter/source/graphicfilter/idxf/dxf2mtf.hxx -filter/source/graphicfilter/idxf/dxfblkrd.cxx -filter/source/graphicfilter/idxf/dxfblkrd.hxx -filter/source/graphicfilter/idxf/dxfentrd.cxx -filter/source/graphicfilter/idxf/dxfentrd.hxx -filter/source/graphicfilter/idxf/dxfgrprd.cxx -filter/source/graphicfilter/idxf/dxfgrprd.hxx -filter/source/graphicfilter/idxf/dxfreprd.cxx -filter/source/graphicfilter/idxf/dxfreprd.hxx -filter/source/graphicfilter/idxf/dxftblrd.cxx -filter/source/graphicfilter/idxf/dxftblrd.hxx -filter/source/graphicfilter/idxf/dxfvec.cxx -filter/source/graphicfilter/idxf/dxfvec.hxx -filter/source/graphicfilter/idxf/idxf.cxx filter/source/msfilter/countryid.cxx filter/source/msfilter/dffpropset.cxx filter/source/msfilter/escherex.cxx @@ -14806,6 +14790,7 @@ vcl/qa/cppunit/dndtest.cxx vcl/qa/cppunit/errorhandler.cxx vcl/qa/cppunit/font.cxx vcl/qa/cppunit/fontmetric.cxx +vcl/qa/cppunit/graphicfilter/filters-dxf-test.cxx vcl/qa/cppunit/graphicfilter/filters-eps-test.cxx vcl/qa/cppunit/graphicfilter/filters-met-test.cxx vcl/qa/cppunit/graphicfilter/filters-pcd-test.cxx @@ -14922,6 +14907,21 @@ vcl/source/filter/graphicfilter.cxx vcl/source/filter/graphicfilter2.cxx vcl/source/filter/eps/eps.cxx vcl/source/filter/etiff/etiff.cxx +vcl/source/filter/idxf/dxf2mtf.cxx +vcl/source/filter/idxf/dxf2mtf.hxx +vcl/source/filter/idxf/dxfblkrd.cxx +vcl/source/filter/idxf/dxfblkrd.hxx +vcl/source/filter/idxf/dxfentrd.cxx +vcl/source/filter/idxf/dxfentrd.hxx +vcl/source/filter/idxf/dxfgrprd.cxx +vcl/source/filter/idxf/dxfgrprd.hxx +vcl/source/filter/idxf/dxfreprd.cxx +vcl/source/filter/idxf/dxfreprd.hxx +vcl/source/filter/idxf/dxftblrd.cxx +vcl/source/filter/idxf/dxftblrd.hxx +vcl/source/filter/idxf/dxfvec.cxx +vcl/source/filter/idxf/dxfvec.hxx +vcl/source/filter/idxf/idxf.cxx vcl/source/filter/ieps/ieps.cxx vcl/source/filter/igif/decode.cxx vcl/source/filter/igif/decode.hxx diff --git a/vcl/CppunitTest_vcl_filters_test.mk b/vcl/CppunitTest_vcl_filters_test.mk index ccb427dddd6c..ab5108f51b1c 100644 --- a/vcl/CppunitTest_vcl_filters_test.mk +++ b/vcl/CppunitTest_vcl_filters_test.mk @@ -10,6 +10,7 @@ $(eval $(call gb_CppunitTest_CppunitTest,vcl_filters_test)) $(eval $(call gb_CppunitTest_add_exception_objects,vcl_filters_test, \ + vcl/qa/cppunit/graphicfilter/filters-dxf-test \ vcl/qa/cppunit/graphicfilter/filters-eps-test \ vcl/qa/cppunit/graphicfilter/filters-met-test \ vcl/qa/cppunit/graphicfilter/filters-pcd-test \ diff --git a/vcl/Library_vcl.mk b/vcl/Library_vcl.mk index 89d453cd7add..696112f34b03 100644 --- a/vcl/Library_vcl.mk +++ b/vcl/Library_vcl.mk @@ -436,6 +436,14 @@ $(eval $(call gb_Library_add_exception_objects,vcl,\ vcl/source/filter/GraphicNativeTransform \ vcl/source/filter/GraphicNativeMetadata \ vcl/source/filter/GraphicFormatDetector \ + vcl/source/filter/idxf/dxf2mtf \ + vcl/source/filter/idxf/dxfblkrd \ + vcl/source/filter/idxf/dxfentrd \ + vcl/source/filter/idxf/dxfgrprd \ + vcl/source/filter/idxf/dxfreprd \ + vcl/source/filter/idxf/dxftblrd \ + vcl/source/filter/idxf/dxfvec \ + vcl/source/filter/idxf/idxf \ vcl/source/filter/ieps/ieps \ vcl/source/filter/igif/decode \ vcl/source/filter/igif/gifread \ diff --git a/vcl/inc/filter/DxfReader.hxx b/vcl/inc/filter/DxfReader.hxx new file mode 100644 index 000000000000..f1e89bf4b115 --- /dev/null +++ b/vcl/inc/filter/DxfReader.hxx @@ -0,0 +1,26 @@ +/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ +/* + * This file is part of the LibreOffice project. + * + * This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. + * + * This file incorporates work covered by the following license notice: + * + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed + * with this work for additional information regarding copyright + * ownership. The ASF licenses this file to you under the Apache + * License, Version 2.0 (the "License"); you may not use this file + * except in compliance with the License. You may obtain a copy of + * the License at http://www.apache.org/licenses/LICENSE-2.0 . + */ + +#pragma once + +#include <vcl/graph.hxx> + +VCL_DLLPUBLIC bool ImportDxfGraphic(SvStream& rStream, Graphic& rGraphic); + +/* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/filter/qa/cppunit/data/dxf/fail/.gitignore b/vcl/qa/cppunit/graphicfilter/data/dxf/fail/.gitignore similarity index 100% rename from filter/qa/cppunit/data/dxf/fail/.gitignore rename to vcl/qa/cppunit/graphicfilter/data/dxf/fail/.gitignore diff --git a/filter/qa/cppunit/data/dxf/fail/CVE-2010-1681-1.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/fail/CVE-2010-1681-1.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/fail/CVE-2010-1681-1.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/fail/CVE-2010-1681-1.dxf diff --git a/filter/qa/cppunit/data/dxf/fail/hang-1.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/fail/hang-1.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/fail/hang-1.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/fail/hang-1.dxf diff --git a/filter/qa/cppunit/data/dxf/indeterminate/.gitignore b/vcl/qa/cppunit/graphicfilter/data/dxf/indeterminate/.gitignore similarity index 100% rename from filter/qa/cppunit/data/dxf/indeterminate/.gitignore rename to vcl/qa/cppunit/graphicfilter/data/dxf/indeterminate/.gitignore diff --git a/filter/qa/cppunit/data/dxf/pass/.gitignore b/vcl/qa/cppunit/graphicfilter/data/dxf/pass/.gitignore similarity index 100% rename from filter/qa/cppunit/data/dxf/pass/.gitignore rename to vcl/qa/cppunit/graphicfilter/data/dxf/pass/.gitignore diff --git a/filter/qa/cppunit/data/dxf/pass/bigangle-1.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/pass/bigangle-1.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/pass/bigangle-1.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/pass/bigangle-1.dxf diff --git a/filter/qa/cppunit/data/dxf/pass/loop-1.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/pass/loop-1.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/pass/loop-1.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/pass/loop-1.dxf diff --git a/filter/qa/cppunit/data/dxf/pass/loop-2.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/pass/loop-2.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/pass/loop-2.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/pass/loop-2.dxf diff --git a/filter/qa/cppunit/data/dxf/pass/pyramid.dxf b/vcl/qa/cppunit/graphicfilter/data/dxf/pass/pyramid.dxf similarity index 100% rename from filter/qa/cppunit/data/dxf/pass/pyramid.dxf rename to vcl/qa/cppunit/graphicfilter/data/dxf/pass/pyramid.dxf diff --git a/filter/qa/cppunit/filters-dxf-test.cxx b/vcl/qa/cppunit/graphicfilter/filters-dxf-test.cxx similarity index 80% rename from filter/qa/cppunit/filters-dxf-test.cxx rename to vcl/qa/cppunit/graphicfilter/filters-dxf-test.cxx index 2dddac45d026..7a133500826a 100644 --- a/filter/qa/cppunit/filters-dxf-test.cxx +++ b/vcl/qa/cppunit/graphicfilter/filters-dxf-test.cxx @@ -12,15 +12,9 @@ #include <vcl/FilterConfigItem.hxx> #include <tools/stream.hxx> #include <vcl/graph.hxx> +#include <filter/DxfReader.hxx> -extern "C" -{ - SAL_DLLPUBLIC_EXPORT bool SAL_CALL - idxGraphicImport(SvStream & rStream, Graphic & rGraphic, - FilterConfigItem*); -} - -using namespace ::com::sun::star; +using namespace css; /* Implementation of Filters test */ @@ -51,17 +45,17 @@ bool DxfFilterTest::load(const OUString &, { SvFileStream aFileStream(rURL, StreamMode::READ); Graphic aGraphic; - return idxGraphicImport(aFileStream, aGraphic, nullptr); + return ImportDxfGraphic(aFileStream, aGraphic); } void DxfFilterTest::testCVEs() { +#ifndef DISABLE_CVE_TESTS testDir(OUString(), - m_directories.getURLFromSrc(u"/filter/qa/cppunit/data/dxf/")); + m_directories.getURLFromSrc(u"/vcl/qa/cppunit/graphicfilter/data/dxf/")); +#endif } CPPUNIT_TEST_SUITE_REGISTRATION(DxfFilterTest); -CPPUNIT_PLUGIN_IMPLEMENT(); - /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/vcl/source/filter/FilterConfigCache.cxx b/vcl/source/filter/FilterConfigCache.cxx index 0f19f717f2c8..d8ab4d44e0a8 100644 --- a/vcl/source/filter/FilterConfigCache.cxx +++ b/vcl/source/filter/FilterConfigCache.cxx @@ -44,7 +44,7 @@ const char* FilterConfigCache::FilterConfigCacheEntry::InternalPixelFilterNameLi IMP_BMP, IMP_GIF, IMP_PNG, IMP_JPEG, IMP_XBM, IMP_XPM, EXP_BMP, EXP_JPEG, EXP_PNG, IMP_MOV, IMP_TIFF, EXP_TIFF, IMP_TGA, IMP_PICT, IMP_MET, IMP_RAS, IMP_PCX, IMP_PSD, - IMP_PCD, IMP_PBM, nullptr + IMP_PCD, IMP_PBM, IMP_DXF, nullptr }; const char* FilterConfigCache::FilterConfigCacheEntry::InternalVectorFilterNameList[] = @@ -221,7 +221,7 @@ const char* FilterConfigCache::InternalFilterListForSvxLight[] = { "bmp","1","SVBMP", "bmp","2","SVBMP", - "dxf","1","idx", + "dxf","1","SVDXF", "eps","1","SVIEPS", "eps","2","SVEEPS", "gif","1","SVIGIF", diff --git a/vcl/source/filter/graphicfilter.cxx b/vcl/source/filter/graphicfilter.cxx index 1257827f5333..b6cf71d078f6 100644 --- a/vcl/source/filter/graphicfilter.cxx +++ b/vcl/source/filter/graphicfilter.cxx @@ -59,6 +59,7 @@ #include <filter/PsdReader.hxx> #include <filter/PcdReader.hxx> #include <filter/PbmReader.hxx> +#include <filter/DxfReader.hxx> #include <osl/module.hxx> #include <com/sun/star/uno/Reference.h> #include <com/sun/star/awt/Size.hpp> @@ -623,12 +624,9 @@ struct ImpFilterLibCacheEntry #endif OUString maFiltername; OUString maFormatName; - PFilterCall mpfnImport; ImpFilterLibCacheEntry(const OUString& rPathname, const OUString& rFiltername, const OUString& rFormatName); bool operator==( std::u16string_view rFiltername ) const { return maFiltername == rFiltername; } - - PFilterCall GetImportFunction(); }; } @@ -639,36 +637,13 @@ ImpFilterLibCacheEntry::ImpFilterLibCacheEntry( const OUString& rPathname, const maLibrary ( rPathname ), #endif maFiltername ( rFiltername ), - maFormatName ( rFormatName ), - mpfnImport ( nullptr ) + maFormatName ( rFormatName ) { #ifdef DISABLE_DYNLOADING (void) rPathname; #endif } -#ifdef DISABLE_DYNLOADING - -extern "C" bool idxGraphicImport( SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem ); - -#endif - -PFilterCall ImpFilterLibCacheEntry::GetImportFunction() -{ - if( !mpfnImport ) - { -#ifndef DISABLE_DYNLOADING - if (maFormatName == "idx") - mpfnImport = reinterpret_cast<PFilterCall>(maLibrary.getFunctionSymbol("idxGraphicImport")); - #else - if (maFormatName == "idx") - mpfnImport = idxGraphicImport; - #endif - } - - return mpfnImport; -} - namespace { class ImpFilterLibCache @@ -1325,13 +1300,6 @@ Graphic GraphicFilter::ImportUnloadedGraphic(SvStream& rIStream, sal_uInt64 size if( !pFilter ) nStatus = ERRCODE_GRFILTER_FILTERERROR; - else - { - PFilterCall pFunc = pFilter->GetImportFunction(); - - if (!pFunc) - nStatus = ERRCODE_GRFILTER_FILTERERROR; - } } if (nStatus == ERRCODE_NONE && eLinkType != GfxLinkType::NONE) @@ -1386,30 +1354,6 @@ Graphic GraphicFilter::ImportUnloadedGraphic(SvStream& rIStream, sal_uInt64 size return aGraphic; } -void GraphicFilter::preload() -{ - sal_Int32 nTokenCount = comphelper::string::getTokenCount(aFilterPath, ';'); - ImpFilterLibCache& rCache = Cache::get(); - static const std::initializer_list<std::u16string_view> aFilterNames = { - u"idx", u"ipb", u"ipd" - }; - - // Load library for each filter. - for (const auto& rFilterName : aFilterNames) - { - ImpFilterLibCacheEntry* pFilter = nullptr; - // Look at the library in each element inside the filter path. - for (sal_Int32 i = 0; i < nTokenCount; ++i) - { - pFilter = rCache.GetFilter(aFilterPath.getToken(i, ';'), SVLIBRARY("gie"), OUString(rFilterName)); - if (pFilter) - { - break; - } - } - } -} - ErrCode GraphicFilter::readGIF(SvStream & rStream, Graphic & rGraphic, GfxLinkType & rLinkType) { if (ImportGIF(rStream, rGraphic)) @@ -1760,6 +1704,14 @@ ErrCode GraphicFilter::readPBM(SvStream & rStream, Graphic & rGraphic) return ERRCODE_GRFILTER_FILTERERROR; } +ErrCode GraphicFilter::readDXF(SvStream & rStream, Graphic & rGraphic) +{ + if (ImportDxfGraphic(rStream, rGraphic)) + return ERRCODE_NONE; + else + return ERRCODE_GRFILTER_FILTERERROR; +} + ErrCode GraphicFilter::ImportGraphic( Graphic& rGraphic, const OUString& rPath, SvStream& rIStream, sal_uInt16 nFormat, sal_uInt16* pDeterminedFormat, GraphicFilterImportFlags nImportFlags, const css::uno::Sequence< css::beans::PropertyValue >* /*pFilterData*/, @@ -1909,6 +1861,10 @@ ErrCode GraphicFilter::ImportGraphic( Graphic& rGraphic, const OUString& rPath, { nStatus = readPBM(rIStream, rGraphic); } + else if (aFilterName.equalsIgnoreAsciiCase(IMP_DXF)) + { + nStatus = readDXF(rIStream, rGraphic); + } else nStatus = ERRCODE_GRFILTER_FILTERERROR; } @@ -1928,18 +1884,6 @@ ErrCode GraphicFilter::ImportGraphic( Graphic& rGraphic, const OUString& rPath, if( !pFilter ) nStatus = ERRCODE_GRFILTER_FILTERERROR; - else - { - PFilterCall pFunc = pFilter->GetImportFunction(); - - if( !pFunc ) - nStatus = ERRCODE_GRFILTER_FILTERERROR; - else - { - if( !(*pFunc)( rIStream, rGraphic, nullptr ) ) - nStatus = ERRCODE_GRFILTER_FORMATERROR; - } - } } if( nStatus == ERRCODE_NONE && ( eLinkType != GfxLinkType::NONE ) && !rGraphic.GetReaderContext() && !bLinkSet ) diff --git a/filter/source/graphicfilter/idxf/dxf2mtf.cxx b/vcl/source/filter/idxf/dxf2mtf.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxf2mtf.cxx rename to vcl/source/filter/idxf/dxf2mtf.cxx diff --git a/filter/source/graphicfilter/idxf/dxf2mtf.hxx b/vcl/source/filter/idxf/dxf2mtf.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxf2mtf.hxx rename to vcl/source/filter/idxf/dxf2mtf.hxx diff --git a/filter/source/graphicfilter/idxf/dxfblkrd.cxx b/vcl/source/filter/idxf/dxfblkrd.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfblkrd.cxx rename to vcl/source/filter/idxf/dxfblkrd.cxx diff --git a/filter/source/graphicfilter/idxf/dxfblkrd.hxx b/vcl/source/filter/idxf/dxfblkrd.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfblkrd.hxx rename to vcl/source/filter/idxf/dxfblkrd.hxx diff --git a/filter/source/graphicfilter/idxf/dxfentrd.cxx b/vcl/source/filter/idxf/dxfentrd.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfentrd.cxx rename to vcl/source/filter/idxf/dxfentrd.cxx diff --git a/filter/source/graphicfilter/idxf/dxfentrd.hxx b/vcl/source/filter/idxf/dxfentrd.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfentrd.hxx rename to vcl/source/filter/idxf/dxfentrd.hxx diff --git a/filter/source/graphicfilter/idxf/dxfgrprd.cxx b/vcl/source/filter/idxf/dxfgrprd.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfgrprd.cxx rename to vcl/source/filter/idxf/dxfgrprd.cxx diff --git a/filter/source/graphicfilter/idxf/dxfgrprd.hxx b/vcl/source/filter/idxf/dxfgrprd.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfgrprd.hxx rename to vcl/source/filter/idxf/dxfgrprd.hxx diff --git a/filter/source/graphicfilter/idxf/dxfreprd.cxx b/vcl/source/filter/idxf/dxfreprd.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfreprd.cxx rename to vcl/source/filter/idxf/dxfreprd.cxx diff --git a/filter/source/graphicfilter/idxf/dxfreprd.hxx b/vcl/source/filter/idxf/dxfreprd.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfreprd.hxx rename to vcl/source/filter/idxf/dxfreprd.hxx diff --git a/filter/source/graphicfilter/idxf/dxftblrd.cxx b/vcl/source/filter/idxf/dxftblrd.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxftblrd.cxx rename to vcl/source/filter/idxf/dxftblrd.cxx diff --git a/filter/source/graphicfilter/idxf/dxftblrd.hxx b/vcl/source/filter/idxf/dxftblrd.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxftblrd.hxx rename to vcl/source/filter/idxf/dxftblrd.hxx diff --git a/filter/source/graphicfilter/idxf/dxfvec.cxx b/vcl/source/filter/idxf/dxfvec.cxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfvec.cxx rename to vcl/source/filter/idxf/dxfvec.cxx diff --git a/filter/source/graphicfilter/idxf/dxfvec.hxx b/vcl/source/filter/idxf/dxfvec.hxx similarity index 100% rename from filter/source/graphicfilter/idxf/dxfvec.hxx rename to vcl/source/filter/idxf/dxfvec.hxx diff --git a/filter/source/graphicfilter/idxf/idxf.cxx b/vcl/source/filter/idxf/idxf.cxx similarity index 90% rename from filter/source/graphicfilter/idxf/idxf.cxx rename to vcl/source/filter/idxf/idxf.cxx index ad75d6b5e1c3..26d42b10cbe4 100644 --- a/filter/source/graphicfilter/idxf/idxf.cxx +++ b/vcl/source/filter/idxf/idxf.cxx @@ -18,16 +18,14 @@ */ +#include <filter/DxfReader.hxx> #include <vcl/gdimtf.hxx> #include <vcl/graph.hxx> #include "dxf2mtf.hxx" -class FilterConfigItem; - //================== GraphicImport - the exported function ================ -extern "C" SAL_DLLPUBLIC_EXPORT bool -idxGraphicImport( SvStream & rStream, Graphic & rGraphic, FilterConfigItem* ) +bool ImportDxfGraphic(SvStream & rStream, Graphic & rGraphic) { DXFRepresentation aDXF; DXF2GDIMetaFile aConverter; diff --git a/vcl/workben/dxffuzzer.cxx b/vcl/workben/dxffuzzer.cxx index 6ca526a1d8cf..8768072c93fb 100644 --- a/vcl/workben/dxffuzzer.cxx +++ b/vcl/workben/dxffuzzer.cxx @@ -10,6 +10,7 @@ #include <tools/stream.hxx> #include <vcl/FilterConfigItem.hxx> #include "commonfuzzer.hxx" +#include <filter/DxfReader.hxx> #include <config_features.h> #include <osl/detail/component-mapping.h> @@ -48,8 +49,6 @@ extern "C" void* lo_get_custom_widget_func(const char*) return nullptr; } -extern "C" bool idxGraphicImport(SvStream& rStream, Graphic& rGraphic, FilterConfigItem* pConfigItem); - extern "C" int LLVMFuzzerInitialize(int *argc, char ***argv) { TypicalFuzzerInitialize(argc, argv); @@ -60,7 +59,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { SvMemoryStream aStream(const_cast<uint8_t*>(data), size, StreamMode::READ); Graphic aGraphic; - (void)idxGraphicImport(aStream, aGraphic, nullptr); + (void)ImportDxfGraphic(aStream, aGraphic); return 0; } diff --git a/vcl/workben/fftester.cxx b/vcl/workben/fftester.cxx index 45deeee37169..5dafd12be683 100644 --- a/vcl/workben/fftester.cxx +++ b/vcl/workben/fftester.cxx @@ -52,6 +52,7 @@ #include <filter/PsdReader.hxx> #include <filter/PcdReader.hxx> #include <filter/PbmReader.hxx> +#include <filter/DxfReader.hxx> #include <osl/file.hxx> #include <osl/module.hxx> #include <tools/stream.hxx> @@ -165,7 +166,6 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) SvFileStream aFileStream(out, StreamMode::READ); ReadGDIMetaFile(aFileStream, aGDIMetaFile); } -#ifndef DISABLE_DYNLOADING else if (strcmp(argv[2], "pcd") == 0) { Graphic aGraphic; @@ -174,18 +174,9 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) } else if (strcmp(argv[2], "dxf") == 0) { - static PFilterCall pfnImport(nullptr); - if (!pfnImport) - { - osl::Module aLibrary; - aLibrary.loadRelative(&thisModule, "libgielo.so"); - pfnImport = reinterpret_cast<PFilterCall>( - aLibrary.getFunctionSymbol("idxGraphicImport")); - aLibrary.release(); - } Graphic aGraphic; SvFileStream aFileStream(out, StreamMode::READ); - ret = static_cast<int>((*pfnImport)(aFileStream, aGraphic, nullptr)); + ret = static_cast<int>(ImportDxfGraphic(aFileStream, aGraphic)); } else if (strcmp(argv[2], "met") == 0) { @@ -241,6 +232,7 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) SvFileStream aFileStream(out, StreamMode::READ); ret = static_cast<int>(ImportTiffGraphicImport(aFileStream, aGraphic)); } +#ifndef DISABLE_DYNLOADING else if ((strcmp(argv[2], "doc") == 0) || (strcmp(argv[2], "ww8") == 0)) { static FFilterCall pfnImport(nullptr); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits