Clean-up of split filter implementations needed

Stephan Bergmann sbergman at redhat.com
Mon Jun 3 06:48:02 PDT 2013


With 
<http://cgit.freedesktop.org/libreoffice/core/commit/?id=5658600001a41c536cc1497eb7419512cb1353d8> 
"No need for two Excel filter services" and 
<http://cgit.freedesktop.org/libreoffice/core/commit/?id=3505f0e46c2e0a743ab5de9adfbec3044a4573a6> 
"Revert 'fdo#46808, Convert oox::PowerPointExport service to new style'" 
I had noted two service implementations that were each split in two for 
no apparent good reason, and those hacks were starting to grow.

In both cases, it was not immediately obvious to me how to fold the 
import and the export implementation into a single C++ class, though. 
(In the first case, I was able to remove the additional export service 
completely by forwarding to it from the import service directly in C++. 
  In the second case, that was not trivially possible, as the export 
service is implemented in sd and the import service in oox, and the 
latter cannot include from the former.)

It would be great if anybody with deeper insight into 
oox::core::XmlFilterBase etc. could clean this up further.

Stephan


More information about the LibreOffice mailing list