[Libreoffice-commits] core.git: writerfilter/source
Arkadiy Illarionov
qarkai at gmail.com
Wed Jun 27 20:10:56 UTC 2018
writerfilter/source/dmapper/BorderHandler.hxx | 1 -
writerfilter/source/dmapper/CellColorHandler.hxx | 1 -
writerfilter/source/dmapper/CellMarginHandler.hxx | 2 --
writerfilter/source/dmapper/DomainMapper.cxx | 10 +++++-----
writerfilter/source/dmapper/DomainMapper_Impl.cxx | 2 +-
writerfilter/source/dmapper/DomainMapper_Impl.hxx | 2 +-
writerfilter/source/dmapper/GraphicHelpers.hxx | 3 ---
writerfilter/source/dmapper/GraphicImport.cxx | 4 ++--
writerfilter/source/dmapper/OLEHandler.hxx | 2 --
writerfilter/source/dmapper/PageBordersHandler.cxx | 2 +-
writerfilter/source/dmapper/StyleSheetTable.cxx | 6 +++---
writerfilter/source/dmapper/TablePropertiesHandler.cxx | 14 +++++++-------
writerfilter/source/dmapper/TblStylePrHandler.hxx | 2 --
writerfilter/source/dmapper/TrackChangesHandler.hxx | 2 --
14 files changed, 20 insertions(+), 33 deletions(-)
New commits:
commit f17eb4c84d0ac67fad9fc22dae19dc530185f68e
Author: Arkadiy Illarionov <qarkai at gmail.com>
Date: Wed Jun 27 21:20:43 2018 +0300
tdf#96099 Remove *HandlerPtr typedefs in writerfilter
See https://bugs.documentfoundation.org/show_bug.cgi?id=96099#c44
Change-Id: Ib2d828a66bc14faa4f341eec9210e1effdebd7c1
Reviewed-on: https://gerrit.libreoffice.org/56541
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/writerfilter/source/dmapper/BorderHandler.hxx b/writerfilter/source/dmapper/BorderHandler.hxx
index cd862d256617..a51bdff52f09 100644
--- a/writerfilter/source/dmapper/BorderHandler.hxx
+++ b/writerfilter/source/dmapper/BorderHandler.hxx
@@ -74,7 +74,6 @@ public:
void enableInteropGrabBag(const OUString& aName);
css::beans::PropertyValue getInteropGrabBag(const OUString& aName = OUString());
};
-typedef std::shared_ptr< BorderHandler > BorderHandlerPtr;
}}
#endif
diff --git a/writerfilter/source/dmapper/CellColorHandler.hxx b/writerfilter/source/dmapper/CellColorHandler.hxx
index a239c203fca3..8392a3d32d44 100644
--- a/writerfilter/source/dmapper/CellColorHandler.hxx
+++ b/writerfilter/source/dmapper/CellColorHandler.hxx
@@ -62,7 +62,6 @@ public:
void disableInteropGrabBag();
bool isInteropGrabBagEnabled();
};
-typedef std::shared_ptr< CellColorHandler > CellColorHandlerPtr;
}}
#endif
diff --git a/writerfilter/source/dmapper/CellMarginHandler.hxx b/writerfilter/source/dmapper/CellMarginHandler.hxx
index b31e9cdf47a6..349611b1012a 100644
--- a/writerfilter/source/dmapper/CellMarginHandler.hxx
+++ b/writerfilter/source/dmapper/CellMarginHandler.hxx
@@ -20,7 +20,6 @@
#define INCLUDED_WRITERFILTER_SOURCE_DMAPPER_CELLMARGINHANDLER_HXX
#include "LoggedResources.hxx"
-#include <memory>
#include <vector>
#include <com/sun/star/beans/PropertyValue.hpp>
@@ -62,7 +61,6 @@ public:
css::beans::PropertyValue getInteropGrabBag();
};
-typedef std::shared_ptr< CellMarginHandler > CellMarginHandlerPtr;
}}
#endif
diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx
index 40455d5beed5..ea40a7f80385 100644
--- a/writerfilter/source/dmapper/DomainMapper.cxx
+++ b/writerfilter/source/dmapper/DomainMapper.cxx
@@ -1326,7 +1326,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- BorderHandlerPtr pBorderHandler( new BorderHandler( true ) );
+ std::shared_ptr<BorderHandler> pBorderHandler( new BorderHandler( true ) );
pProperties->resolve(*pBorderHandler);
PropertyIds eBorderId = PropertyIds( 0 );
PropertyIds eBorderDistId = PropertyIds( 0 );
@@ -1378,7 +1378,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- CellColorHandlerPtr pCellColorHandler( new CellColorHandler );
+ std::shared_ptr<CellColorHandler> pCellColorHandler( new CellColorHandler );
pCellColorHandler->setOutputFormat( CellColorHandler::Paragraph );
bool bEnableTempGrabBag = !pCellColorHandler->isInteropGrabBagEnabled();
if( bEnableTempGrabBag )
@@ -1750,7 +1750,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- CellColorHandlerPtr pCellColorHandler( new CellColorHandler );
+ std::shared_ptr<CellColorHandler> pCellColorHandler( new CellColorHandler );
pCellColorHandler->setOutputFormat( CellColorHandler::Character );
pProperties->resolve(*pCellColorHandler);
rContext->InsertProps(pCellColorHandler->getProperties());
@@ -1854,7 +1854,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- BorderHandlerPtr pBorderHandler( new BorderHandler( true ) );
+ std::shared_ptr<BorderHandler> pBorderHandler( new BorderHandler( true ) );
pProperties->resolve(*pBorderHandler);
rContext->Insert( PROP_CHAR_TOP_BORDER, uno::makeAny( pBorderHandler->getBorderLine()));
@@ -2342,7 +2342,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const PropertyMapPtr& rContext )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get( ) )
{
- OLEHandlerPtr pOLEHandler( new OLEHandler(*this) );
+ std::shared_ptr<OLEHandler> pOLEHandler( new OLEHandler(*this) );
pProperties->resolve(*pOLEHandler);
if ( pOLEHandler->isOLEObject( ) )
{
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index 6521a08d0445..8b772f7577a9 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -1556,7 +1556,7 @@ void DomainMapper_Impl::appendTextContent(
}
}
-void DomainMapper_Impl::appendOLE( const OUString& rStreamName, const OLEHandlerPtr& pOLEHandler )
+void DomainMapper_Impl::appendOLE( const OUString& rStreamName, const std::shared_ptr<OLEHandler>& pOLEHandler )
{
try
{
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.hxx b/writerfilter/source/dmapper/DomainMapper_Impl.hxx
index dc229c28b614..e386252fd9bc 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.hxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.hxx
@@ -628,7 +628,7 @@ public:
void finishParagraph( const PropertyMapPtr& pPropertyMap );
void appendTextPortion( const OUString& rString, const PropertyMapPtr& pPropertyMap );
void appendTextContent(const css::uno::Reference<css::text::XTextContent>&, const css::uno::Sequence<css::beans::PropertyValue>&);
- void appendOLE( const OUString& rStreamName, const OLEHandlerPtr& pOleHandler );
+ void appendOLE( const OUString& rStreamName, const std::shared_ptr<OLEHandler>& pOleHandler );
void appendStarMath( const Value& v );
css::uno::Reference<css::beans::XPropertySet> appendTextSectionAfter(css::uno::Reference<css::text::XTextRange> const & xBefore);
diff --git a/writerfilter/source/dmapper/GraphicHelpers.hxx b/writerfilter/source/dmapper/GraphicHelpers.hxx
index 58a24cc5ce20..7653f54e5b79 100644
--- a/writerfilter/source/dmapper/GraphicHelpers.hxx
+++ b/writerfilter/source/dmapper/GraphicHelpers.hxx
@@ -23,8 +23,6 @@
#include "LoggedResources.hxx"
#include <com/sun/star/text/WrapTextMode.hpp>
-#include <memory>
-#include <map>
#include <utility>
namespace writerfilter {
@@ -48,7 +46,6 @@ public:
std::pair<OUString, OUString>& m_rPositionOffsets;
std::pair<OUString, OUString>& m_rAligns;
};
-typedef std::shared_ptr<PositionHandler> PositionHandlerPtr;
class WrapHandler: public LoggedProperties
{
diff --git a/writerfilter/source/dmapper/GraphicImport.cxx b/writerfilter/source/dmapper/GraphicImport.cxx
index bca23419fb28..0f828c22cd12 100644
--- a/writerfilter/source/dmapper/GraphicImport.cxx
+++ b/writerfilter/source/dmapper/GraphicImport.cxx
@@ -1024,7 +1024,7 @@ void GraphicImport::lcl_sprm(Sprm& rSprm)
case NS_ooxml::LN_CT_Anchor_positionH: // 90976;
{
// Use a special handler for the positioning
- PositionHandlerPtr pHandler( new PositionHandler( m_pImpl->m_rPositionOffsets, m_pImpl->m_rAligns ));
+ std::shared_ptr<PositionHandler> pHandler( new PositionHandler( m_pImpl->m_rPositionOffsets, m_pImpl->m_rAligns ));
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get( ) )
{
@@ -1041,7 +1041,7 @@ void GraphicImport::lcl_sprm(Sprm& rSprm)
case NS_ooxml::LN_CT_Anchor_positionV: // 90977;
{
// Use a special handler for the positioning
- PositionHandlerPtr pHandler( new PositionHandler( m_pImpl->m_rPositionOffsets, m_pImpl->m_rAligns));
+ std::shared_ptr<PositionHandler> pHandler( new PositionHandler( m_pImpl->m_rPositionOffsets, m_pImpl->m_rAligns));
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get( ) )
{
diff --git a/writerfilter/source/dmapper/OLEHandler.hxx b/writerfilter/source/dmapper/OLEHandler.hxx
index 6757d3a3d2df..6ba2fb34ae1a 100644
--- a/writerfilter/source/dmapper/OLEHandler.hxx
+++ b/writerfilter/source/dmapper/OLEHandler.hxx
@@ -20,7 +20,6 @@
#define INCLUDED_WRITERFILTER_SOURCE_DMAPPER_OLEHANDLER_HXX
#include "LoggedResources.hxx"
-#include <memory>
#include <com/sun/star/awt/Size.hpp>
#include <com/sun/star/awt/Point.hpp>
#include <com/sun/star/drawing/XShape.hpp>
@@ -98,7 +97,6 @@ public:
const css::uno::Reference<css::graphic::XGraphic>& getReplacement() const { return m_xReplacement; }
};
-typedef std::shared_ptr< OLEHandler > OLEHandlerPtr;
}}
#endif
diff --git a/writerfilter/source/dmapper/PageBordersHandler.cxx b/writerfilter/source/dmapper/PageBordersHandler.cxx
index 2bebf0d5af6b..d88f4d370f56 100644
--- a/writerfilter/source/dmapper/PageBordersHandler.cxx
+++ b/writerfilter/source/dmapper/PageBordersHandler.cxx
@@ -98,7 +98,7 @@ void PageBordersHandler::lcl_sprm( Sprm& rSprm )
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- BorderHandlerPtr pBorderHandler( new BorderHandler( true ) );
+ std::shared_ptr<BorderHandler> pBorderHandler( new BorderHandler( true ) );
pProperties->resolve(*pBorderHandler);
BorderPosition ePos = BorderPosition( 0 );
switch( rSprm.getId( ) )
diff --git a/writerfilter/source/dmapper/StyleSheetTable.cxx b/writerfilter/source/dmapper/StyleSheetTable.cxx
index 7e28500a11c0..1047d9c20904 100644
--- a/writerfilter/source/dmapper/StyleSheetTable.cxx
+++ b/writerfilter/source/dmapper/StyleSheetTable.cxx
@@ -565,7 +565,7 @@ void StyleSheetTable::lcl_sprm(Sprm & rSprm)
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get() && m_pImpl->m_pCurrentEntry->nStyleTypeCode == STYLE_TYPE_TABLE)
{
- TblStylePrHandlerPtr pTblStylePrHandler(new TblStylePrHandler(m_pImpl->m_rDMapper));
+ std::shared_ptr<TblStylePrHandler> pTblStylePrHandler(new TblStylePrHandler(m_pImpl->m_rDMapper));
pProperties->resolve(*pTblStylePrHandler);
StyleSheetEntry* pEntry = m_pImpl->m_pCurrentEntry.get();
TableStyleSheetEntry& rTableEntry = dynamic_cast<TableStyleSheetEntry&>(*pEntry);
@@ -639,7 +639,7 @@ void StyleSheetTable::lcl_sprm(Sprm & rSprm)
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- TblStylePrHandlerPtr pTblStylePrHandler( new TblStylePrHandler( m_pImpl->m_rDMapper ) );
+ std::shared_ptr<TblStylePrHandler> pTblStylePrHandler( new TblStylePrHandler( m_pImpl->m_rDMapper ) );
pProperties->resolve( *pTblStylePrHandler );
// Add the properties to the table style
@@ -697,7 +697,7 @@ void StyleSheetTable::lcl_sprm(Sprm & rSprm)
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- BorderHandlerPtr pBorderHandler( new BorderHandler(m_pImpl->m_rDMapper.IsOOXMLImport()) );
+ std::shared_ptr<BorderHandler> pBorderHandler( new BorderHandler(m_pImpl->m_rDMapper.IsOOXMLImport()) );
pProperties->resolve(*pBorderHandler);
m_pImpl->m_pCurrentEntry->pProperties->InsertProps(
pBorderHandler->getProperties());
diff --git a/writerfilter/source/dmapper/TablePropertiesHandler.cxx b/writerfilter/source/dmapper/TablePropertiesHandler.cxx
index a26f7596cd7f..bbe9067ffdec 100644
--- a/writerfilter/source/dmapper/TablePropertiesHandler.cxx
+++ b/writerfilter/source/dmapper/TablePropertiesHandler.cxx
@@ -117,7 +117,7 @@ namespace dmapper {
nToken = XML_tableRowDelete;
break;
};
- TrackChangesHandlerPtr pTrackChangesHandler( new TrackChangesHandler( nToken ) );
+ std::shared_ptr<TrackChangesHandler> pTrackChangesHandler( new TrackChangesHandler( nToken ) );
pProperties->resolve(*pTrackChangesHandler);
TablePropertyMapPtr pPropMap( new TablePropertyMap );
@@ -148,7 +148,7 @@ namespace dmapper {
throw lang::IllegalArgumentException("illegal redline token type", nullptr, 0);
break;
};
- TrackChangesHandlerPtr pTrackChangesHandler( new TrackChangesHandler( nToken ) );
+ std::shared_ptr<TrackChangesHandler> pTrackChangesHandler( new TrackChangesHandler( nToken ) );
pProperties->resolve(*pTrackChangesHandler);
TablePropertyMapPtr pPropMap( new TablePropertyMap );
@@ -206,7 +206,7 @@ namespace dmapper {
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- BorderHandlerPtr pBorderHandler(new BorderHandler(true));
+ std::shared_ptr<BorderHandler> pBorderHandler(new BorderHandler(true));
if (m_pCurrentInteropGrabBag)
pBorderHandler->enableInteropGrabBag("tblBorders");
pProperties->resolve(*pBorderHandler);
@@ -254,7 +254,7 @@ namespace dmapper {
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if (pProperties.get())
{
- CellMarginHandlerPtr pCellMarginHandler(new CellMarginHandler);
+ std::shared_ptr<CellMarginHandler> pCellMarginHandler(new CellMarginHandler);
if (m_pCurrentInteropGrabBag)
pCellMarginHandler->enableInteropGrabBag("tcMar");
pProperties->resolve(*pCellMarginHandler);
@@ -278,7 +278,7 @@ namespace dmapper {
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- CellColorHandlerPtr pCellColorHandler( new CellColorHandler);
+ std::shared_ptr<CellColorHandler> pCellColorHandler( new CellColorHandler);
pProperties->resolve( *pCellColorHandler );
TablePropertyMapPtr pTablePropMap( new TablePropertyMap );
insertTableProps( pCellColorHandler->getProperties() );
@@ -292,7 +292,7 @@ namespace dmapper {
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- CellColorHandlerPtr pCellColorHandler( new CellColorHandler );
+ std::shared_ptr<CellColorHandler> pCellColorHandler( new CellColorHandler );
pCellColorHandler->enableInteropGrabBag("shd"); //enable to store shd unsupported props in grab bag
pProperties->resolve( *pCellColorHandler );
TablePropertyMapPtr pPropertyMap = pCellColorHandler->getProperties();
@@ -312,7 +312,7 @@ namespace dmapper {
writerfilter::Reference<Properties>::Pointer_t pProperties = rSprm.getProps();
if( pProperties.get())
{
- CellMarginHandlerPtr pCellMarginHandler( new CellMarginHandler );
+ std::shared_ptr<CellMarginHandler> pCellMarginHandler( new CellMarginHandler );
if (m_pCurrentInteropGrabBag)
pCellMarginHandler->enableInteropGrabBag("tblCellMar");
pProperties->resolve( *pCellMarginHandler );
diff --git a/writerfilter/source/dmapper/TblStylePrHandler.hxx b/writerfilter/source/dmapper/TblStylePrHandler.hxx
index 6beb52ff1e27..283f94b41403 100644
--- a/writerfilter/source/dmapper/TblStylePrHandler.hxx
+++ b/writerfilter/source/dmapper/TblStylePrHandler.hxx
@@ -79,8 +79,6 @@ private:
void resolveSprmProps(Sprm & rSprm);
};
-typedef std::shared_ptr< TblStylePrHandler > TblStylePrHandlerPtr;
-
}}
#endif
diff --git a/writerfilter/source/dmapper/TrackChangesHandler.hxx b/writerfilter/source/dmapper/TrackChangesHandler.hxx
index 23c8b8e0f3b4..9143986e2ab6 100644
--- a/writerfilter/source/dmapper/TrackChangesHandler.hxx
+++ b/writerfilter/source/dmapper/TrackChangesHandler.hxx
@@ -10,7 +10,6 @@
#define INCLUDED_WRITERFILTER_SOURCE_DMAPPER_TRACKCHANGESHANDLER_HXX
#include "LoggedResources.hxx"
-#include <memory>
#include <com/sun/star/beans/PropertyValue.hpp>
#include "DomainMapper_Impl.hxx"
@@ -38,7 +37,6 @@ public:
/// Compute the UNO properties for the track changes object based on the received tokens.
css::uno::Sequence<css::beans::PropertyValue> getRedlineProperties() const;
};
-typedef std::shared_ptr<TrackChangesHandler> TrackChangesHandlerPtr;
}}
#endif
More information about the Libreoffice-commits
mailing list