dbaccess/source/core/api/RowSet.cxx                      |   49 ++++++++-------
 dbaccess/source/core/api/RowSetCache.cxx                 |   13 +--
 dbaccess/source/core/dataaccess/dataaccessdescriptor.cxx |   37 +++++------
 dbaccess/source/core/dataaccess/documentdefinition.cxx   |   16 ++--
 dbaccess/source/filter/xml/xmlHelper.cxx                 |   21 +++---
 5 files changed, 70 insertions(+), 66 deletions(-)

New commits:
commit e21ebb0a98e81a9591918f3794ece75fb1df7ab1
Author:     Julien Nabet <[email protected]>
AuthorDate: Mon Nov 8 21:59:31 2021 +0100
Commit:     Julien Nabet <[email protected]>
CommitDate: Mon Nov 8 23:35:03 2021 +0100

    Replace some macros in dbaccess part 1
    
    Remove NOTIFY_LISTENERS_CHECK + CHECK_MATRIX_POS + REGISTER_PROPERTY_BV + 
REGISTER_PROPERTY
    + MAP_END + MAP_CONST_COLUMN + MAP_CONST_CELL
    
    Change-Id: I3cd32059aef6787c0e5f8338320d7e924199830c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124892
    Tested-by: Jenkins
    Reviewed-by: Julien Nabet <[email protected]>

diff --git a/dbaccess/source/core/api/RowSet.cxx 
b/dbaccess/source/core/api/RowSet.cxx
index 4e66304cd367..4630bee25e94 100644
--- a/dbaccess/source/core/api/RowSet.cxx
+++ b/dbaccess/source/core/api/RowSet.cxx
@@ -95,24 +95,6 @@ 
com_sun_star_comp_dba_ORowSet_get_implementation(css::uno::XComponentContext* co
     return cppu::acquire(new ORowSet(context));
 }
 
-#define NOTIFY_LISTENERS_CHECK(_rListeners,T,method)                           
  \
-    std::vector< Reference< XInterface > > aListenerSeq = 
_rListeners.getElements(); \
-                                                                               
   \
-    _rGuard.clear();                                                           
   \
-    bool bCheck = std::all_of(aListenerSeq.rbegin(), aListenerSeq.rend(),      
   \
-        [&aEvt](Reference<XInterface>& rxItem) {                               
   \
-            try                                                                
   \
-            {                                                                  
   \
-                return 
static_cast<bool>(static_cast<T*>(rxItem.get())->method(aEvt)); \
-            }                                                                  
   \
-            catch( RuntimeException& )                                         
   \
-            {                                                                  
   \
-                return true;                                                   
   \
-            }                                                                  
   \
-        });                                                                    
   \
-    _rGuard.reset();
-
-
 namespace dbaccess
 {
 ORowSet::ORowSet( const Reference< css::uno::XComponentContext >& _rxContext )
@@ -1109,13 +1091,40 @@ void 
ORowSet::notifyAllListenersRowChanged(::osl::ResettableMutexGuard& _rGuard,
 bool ORowSet::notifyAllListenersCursorBeforeMove(::osl::ResettableMutexGuard& 
_rGuard)
 {
     EventObject aEvt(*m_pMySelf);
-    
NOTIFY_LISTENERS_CHECK(m_aApproveListeners,XRowSetApproveListener,approveCursorMove);
+    std::vector< Reference< XInterface > > aListenerSeq = 
m_aApproveListeners.getElements();
+    _rGuard.clear();
+    bool bCheck = std::all_of(aListenerSeq.rbegin(), aListenerSeq.rend(),
+        [&aEvt](Reference<XInterface>& rxItem) {
+            try
+            {
+                return 
static_cast<bool>(static_cast<XRowSetApproveListener*>(rxItem.get())->approveCursorMove(aEvt));
+            }
+            catch( RuntimeException& )
+            {
+                return true;
+            }
+        });
+    _rGuard.reset();
     return bCheck;
 }
 
 void ORowSet::notifyAllListenersRowBeforeChange(::osl::ResettableMutexGuard& 
_rGuard,const RowChangeEvent &aEvt)
 {
-    
NOTIFY_LISTENERS_CHECK(m_aApproveListeners,XRowSetApproveListener,approveRowChange);
+    std::vector< Reference< XInterface > > aListenerSeq = 
m_aApproveListeners.getElements();
+    _rGuard.clear();
+    bool bCheck = std::all_of(aListenerSeq.rbegin(), aListenerSeq.rend(),
+        [&aEvt](Reference<XInterface>& rxItem) {
+            try
+            {
+                return 
static_cast<bool>(static_cast<XRowSetApproveListener*>(rxItem.get())->approveRowChange(aEvt));
+            }
+            catch( RuntimeException& )
+            {
+                return true;
+            }
+        });
+    _rGuard.reset();
+
     if ( !bCheck )
         m_aErrors.raiseTypedException( 
sdb::ErrorCondition::ROW_SET_OPERATION_VETOED, *this, ::cppu::UnoType< 
RowSetVetoException >::get() );
 }
diff --git a/dbaccess/source/core/api/RowSetCache.cxx 
b/dbaccess/source/core/api/RowSetCache.cxx
index a939cc1e48a4..b588cc0320e6 100644
--- a/dbaccess/source/core/api/RowSetCache.cxx
+++ b/dbaccess/source/core/api/RowSetCache.cxx
@@ -64,8 +64,6 @@ using namespace ::com::sun::star::lang;
 using namespace ::cppu;
 using namespace ::osl;
 
-#define CHECK_MATRIX_POS(M) OSL_ENSURE(((M) >= 
static_cast<ORowSetMatrix::difference_type>(0)) && ((M) < 
static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!")
-
 // This class calls m_pCacheSet->FOO_checked(..., sal_False)
 // (where FOO is absolute, last, previous)
 // when it does not immediately care about the values in the row's columns.
@@ -406,7 +404,7 @@ void ORowSetCache::setFetchSize(sal_Int32 _nSize)
         {
             if ( rPosChange.second )
             {
-                CHECK_MATRIX_POS(*aIter);
+                OSL_ENSURE((*aIter >= 
static_cast<ORowSetMatrix::difference_type>(0)) && (*aIter < 
static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!");
                 if ( *aIter < _nSize )
                     aCacheIter->second.aIterator = m_pMatrix->begin() + 
*aIter++;
                 else
@@ -917,7 +915,8 @@ void ORowSetCache::moveWindow()
                             else
                             {
                                 // Inside overlap area: move to correct 
position
-                                CHECK_MATRIX_POS( (nDist + nStartPosOffset) );
+                                OSL_ENSURE(((nDist + nStartPosOffset) >= 
static_cast<ORowSetMatrix::difference_type>(0)) &&
+                                    ((nDist + nStartPosOffset) < 
static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!");
                                 rCacheIter.second.aIterator += nStartPosOffset;
                                 OSL_ENSURE(rCacheIter.second.aIterator >= 
m_pMatrix->begin()
                                     && rCacheIter.second.aIterator < 
m_pMatrix->end(),"Iterator out of area!");
@@ -947,7 +946,7 @@ void ORowSetCache::moveWindow()
             if ( nRowsInCache < m_nFetchSize )
             {
                 // There is some unused space in *m_pMatrix; fill it
-                CHECK_MATRIX_POS(nRowsInCache);
+                OSL_ENSURE((nRowsInCache >= 
static_cast<ORowSetMatrix::difference_type>(0)) && (nRowsInCache < 
static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!");
                 sal_Int32 nPos = m_nEndPos + 1;
                 bool bCheck = m_xCacheSet->absolute(nPos);
                 ORowSetMatrix::iterator aIter = m_pMatrix->begin() + 
nRowsInCache;
@@ -965,7 +964,7 @@ void ORowSetCache::moveWindow()
             // The rows behind this can be reused
             ORowSetMatrix::iterator aIter = m_pMatrix->begin();
             const sal_Int32 nNewStartPosInMatrix = nNewStartPos - m_nStartPos;
-            CHECK_MATRIX_POS( nNewStartPosInMatrix );
+            OSL_ENSURE((nNewStartPosInMatrix >= 
static_cast<ORowSetMatrix::difference_type>(0)) && (nNewStartPosInMatrix < 
static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!");
             // first position we reuse
             const ORowSetMatrix::const_iterator aEnd  = m_pMatrix->begin() + 
nNewStartPosInMatrix;
             // End of used portion of the matrix. Is < m_pMatrix->end() if 
less data than window size
@@ -1624,7 +1623,7 @@ void ORowSetCache::clearInsertRow()
 ORowSetMatrix::iterator ORowSetCache::calcPosition() const
 {
     sal_Int32 nValue = (m_nPosition - m_nStartPos) - 1;
-    CHECK_MATRIX_POS(nValue);
+    OSL_ENSURE((nValue >= static_cast<ORowSetMatrix::difference_type>(0)) && 
(nValue < static_cast<sal_Int32>(m_pMatrix->size())),"Position is invalid!");
     return ( nValue < 0 || nValue >= static_cast<sal_Int32>(m_pMatrix->size()) 
) ? m_pMatrix->end() : (m_pMatrix->begin() + nValue);
 }
 
diff --git a/dbaccess/source/core/dataaccess/dataaccessdescriptor.cxx 
b/dbaccess/source/core/dataaccess/dataaccessdescriptor.cxx
index 8262f089c293..97a8f5b9fb86 100644
--- a/dbaccess/source/core/dataaccess/dataaccessdescriptor.cxx
+++ b/dbaccess/source/core/dataaccess/dataaccessdescriptor.cxx
@@ -112,32 +112,29 @@ namespace
         // </properties>
     };
 
-#define REGISTER_PROPERTY( propname, member ) \
-    registerProperty( PROPERTY_##propname, PROPERTY_ID_##propname, 
PropertyAttribute::BOUND, &member, cppu::UnoType<decltype(member)>::get() )
-
     DataAccessDescriptor::DataAccessDescriptor()
         :DataAccessDescriptor_PropertyBase( m_aBHelper )
         ,m_nCommandType( CommandType::COMMAND )
         ,m_bEscapeProcessing( true )
         ,m_bBookmarkSelection( true )
     {
-        REGISTER_PROPERTY( DATASOURCENAME,      m_sDataSourceName );
-        REGISTER_PROPERTY( DATABASE_LOCATION,   m_sDatabaseLocation );
-        REGISTER_PROPERTY( CONNECTION_RESOURCE, m_sConnectionResource );
-        REGISTER_PROPERTY( CONNECTION_INFO,     m_aConnectionInfo );
-        REGISTER_PROPERTY( ACTIVE_CONNECTION,   m_xActiveConnection );
-        REGISTER_PROPERTY( COMMAND,             m_sCommand );
-        REGISTER_PROPERTY( COMMAND_TYPE,        m_nCommandType );
-        REGISTER_PROPERTY( FILTER,              m_sFilter );
-        REGISTER_PROPERTY( ORDER,               m_sOrder );
-        REGISTER_PROPERTY( HAVING_CLAUSE,       m_sHavingClause );
-        REGISTER_PROPERTY( GROUP_BY,            m_sGroupBy );
-        REGISTER_PROPERTY( ESCAPE_PROCESSING,   m_bEscapeProcessing );
-        REGISTER_PROPERTY( RESULT_SET,          m_xResultSet );
-        REGISTER_PROPERTY( SELECTION,           m_aSelection );
-        REGISTER_PROPERTY( BOOKMARK_SELECTION,  m_bBookmarkSelection );
-        REGISTER_PROPERTY( COLUMN_NAME,         m_sColumnName );
-        REGISTER_PROPERTY( COLUMN,              m_xColumn );
+        registerProperty(PROPERTY_DATASOURCENAME, PROPERTY_ID_DATASOURCENAME, 
PropertyAttribute::BOUND, &m_sDataSourceName , 
cppu::UnoType<decltype(m_sDataSourceName )>::get());
+        registerProperty(PROPERTY_DATABASE_LOCATION, 
PROPERTY_ID_DATABASE_LOCATION, PropertyAttribute::BOUND, &m_sDatabaseLocation , 
cppu::UnoType<decltype(m_sDatabaseLocation )>::get());
+        registerProperty(PROPERTY_CONNECTION_RESOURCE, 
PROPERTY_ID_CONNECTION_RESOURCE, PropertyAttribute::BOUND, 
&m_sConnectionResource , cppu::UnoType<decltype(m_sConnectionResource 
)>::get());
+        registerProperty(PROPERTY_CONNECTION_INFO, 
PROPERTY_ID_CONNECTION_INFO, PropertyAttribute::BOUND, &m_aConnectionInfo , 
cppu::UnoType<decltype(m_aConnectionInfo )>::get());
+        registerProperty(PROPERTY_ACTIVE_CONNECTION, 
PROPERTY_ID_ACTIVE_CONNECTION, PropertyAttribute::BOUND, &m_xActiveConnection , 
cppu::UnoType<decltype(m_xActiveConnection )>::get());
+        registerProperty(PROPERTY_COMMAND, PROPERTY_ID_COMMAND, 
PropertyAttribute::BOUND, &m_sCommand , cppu::UnoType<decltype(m_sCommand 
)>::get());
+        registerProperty(PROPERTY_COMMAND_TYPE, PROPERTY_ID_COMMAND_TYPE, 
PropertyAttribute::BOUND, &m_nCommandType , 
cppu::UnoType<decltype(m_nCommandType )>::get());
+        registerProperty(PROPERTY_FILTER, PROPERTY_ID_FILTER, 
PropertyAttribute::BOUND, &m_sFilter , cppu::UnoType<decltype(m_sFilter 
)>::get());
+        registerProperty(PROPERTY_ORDER, PROPERTY_ID_ORDER, 
PropertyAttribute::BOUND, &m_sOrder , cppu::UnoType<decltype(m_sOrder 
)>::get());
+        registerProperty(PROPERTY_HAVING_CLAUSE, PROPERTY_ID_HAVING_CLAUSE, 
PropertyAttribute::BOUND, &m_sHavingClause , 
cppu::UnoType<decltype(m_sHavingClause )>::get());
+        registerProperty(PROPERTY_GROUP_BY, PROPERTY_ID_GROUP_BY, 
PropertyAttribute::BOUND, &m_sGroupBy , cppu::UnoType<decltype(m_sGroupBy 
)>::get());
+        registerProperty(PROPERTY_ESCAPE_PROCESSING, 
PROPERTY_ID_ESCAPE_PROCESSING, PropertyAttribute::BOUND, &m_bEscapeProcessing , 
cppu::UnoType<decltype(m_bEscapeProcessing )>::get());
+        registerProperty(PROPERTY_RESULT_SET, PROPERTY_ID_RESULT_SET, 
PropertyAttribute::BOUND, &m_xResultSet , cppu::UnoType<decltype(m_xResultSet 
)>::get());
+        registerProperty(PROPERTY_SELECTION, PROPERTY_ID_SELECTION, 
PropertyAttribute::BOUND, &m_aSelection , cppu::UnoType<decltype(m_aSelection 
)>::get());
+        registerProperty(PROPERTY_BOOKMARK_SELECTION, 
PROPERTY_ID_BOOKMARK_SELECTION, PropertyAttribute::BOUND, &m_bBookmarkSelection 
, cppu::UnoType<decltype(m_bBookmarkSelection )>::get());
+        registerProperty(PROPERTY_COLUMN_NAME, PROPERTY_ID_COLUMN_NAME, 
PropertyAttribute::BOUND, &m_sColumnName , cppu::UnoType<decltype(m_sColumnName 
)>::get());
+        registerProperty(PROPERTY_COLUMN, PROPERTY_ID_COLUMN, 
PropertyAttribute::BOUND, &m_xColumn , cppu::UnoType<decltype(m_xColumn 
)>::get());
     }
 
     DataAccessDescriptor::~DataAccessDescriptor()
diff --git a/dbaccess/source/core/dataaccess/documentdefinition.cxx 
b/dbaccess/source/core/dataaccess/documentdefinition.cxx
index 7e6efc688a1e..4bc72018b873 100644
--- a/dbaccess/source/core/dataaccess/documentdefinition.cxx
+++ b/dbaccess/source/core/dataaccess/documentdefinition.cxx
@@ -480,16 +480,16 @@ IMPLEMENT_FORWARD_XINTERFACE3( 
ODocumentDefinition,OContentHelper,OPropertyState
 
 void ODocumentDefinition::registerProperties()
 {
-#define REGISTER_PROPERTY( name, location ) \
-    registerProperty(   PROPERTY_##name, PROPERTY_ID_##name, 
PropertyAttribute::READONLY, &location, 
cppu::UnoType<decltype(location)>::get() );
+    registerProperty(PROPERTY_NAME, PROPERTY_ID_NAME, 
PropertyAttribute::CONSTRAINED | PropertyAttribute::BOUND | 
PropertyAttribute::READONLY,
+                     &m_pImpl->m_aProps.aTitle, 
cppu::UnoType<decltype(m_pImpl->m_aProps.aTitle)>::get());
 
-#define REGISTER_PROPERTY_BV( name, location ) \
-    registerProperty(   PROPERTY_##name, PROPERTY_ID_##name, 
PropertyAttribute::CONSTRAINED | PropertyAttribute::BOUND | 
PropertyAttribute::READONLY, &location, 
cppu::UnoType<decltype(location)>::get() );
+    registerProperty(PROPERTY_AS_TEMPLATE, PROPERTY_ID_AS_TEMPLATE, 
PropertyAttribute::READONLY, &m_pImpl->m_aProps.bAsTemplate,
+                     
cppu::UnoType<decltype(m_pImpl->m_aProps.bAsTemplate)>::get());
 
-    REGISTER_PROPERTY_BV( NAME,            m_pImpl->m_aProps.aTitle            
);
-    REGISTER_PROPERTY   ( AS_TEMPLATE,     m_pImpl->m_aProps.bAsTemplate       
);
-    REGISTER_PROPERTY   ( PERSISTENT_NAME, m_pImpl->m_aProps.sPersistentName   
);
-    REGISTER_PROPERTY   ( IS_FORM,         m_bForm                             
);
+    registerProperty(PROPERTY_PERSISTENT_NAME, PROPERTY_ID_PERSISTENT_NAME, 
PropertyAttribute::READONLY, &m_pImpl->m_aProps.sPersistentName,
+                     
cppu::UnoType<decltype(m_pImpl->m_aProps.sPersistentName)>::get());
+
+    registerProperty(PROPERTY_IS_FORM, PROPERTY_ID_IS_FORM, 
PropertyAttribute::READONLY, &m_bForm, cppu::UnoType<decltype(m_bForm)>::get());
 }
 
 void SAL_CALL ODocumentDefinition::getFastPropertyValue( Any& o_rValue, 
sal_Int32 i_nHandle ) const
diff --git a/dbaccess/source/filter/xml/xmlHelper.cxx 
b/dbaccess/source/filter/xml/xmlHelper.cxx
index 38e90e4e9d3a..0501a73d160d 100644
--- a/dbaccess/source/filter/xml/xmlHelper.cxx
+++ b/dbaccess/source/filter/xml/xmlHelper.cxx
@@ -68,12 +68,11 @@ const XMLPropertyHandler* 
OPropertyHandlerFactory::GetPropertyHandler(sal_Int32
     return pHandler;
 }
 
-#define MAP_END() { nullptr, 0, 0, XML_TOKEN_INVALID, 0 , 0, 
SvtSaveOptions::ODFSVER_010, false}
 rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetTableStylesPropertySetMapper( bool bForExport )
 {
     static const XMLPropertyMapEntry s_aTableStylesProperties[] =
     {
-        MAP_END()
+        { nullptr, 0, 0, XML_TOKEN_INVALID, 0 , 0, 
SvtSaveOptions::ODFSVER_010, false}
     };
     rtl::Reference < XMLPropertyHandlerFactory> xFac = new 
::xmloff::OControlPropertyHandlerFactory();
     return new XMLPropertySetMapper(s_aTableStylesProperties, xFac, 
bForExport);
@@ -81,13 +80,14 @@ rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetTableStylesPropertySetMap
 
 rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetColumnStylesPropertySetMapper( bool bForExport )
 {
-#define MAP_CONST_COLUMN( name, prefix, token, type, context )  { name, 
sizeof(name)-1,  prefix, token, type|XML_TYPE_PROP_TABLE_COLUMN, context, 
SvtSaveOptions::ODFSVER_010, false }
     static const XMLPropertyMapEntry s_aColumnStylesProperties[] =
     {
-        MAP_CONST_COLUMN( PROPERTY_WIDTH,           XML_NAMESPACE_STYLE,    
XML_COLUMN_WIDTH,       XML_TYPE_MEASURE, 0),
-        MAP_CONST_COLUMN( PROPERTY_HIDDEN,          XML_NAMESPACE_TABLE,    
XML_DISPLAY,            XML_DB_TYPE_EQUAL|MID_FLAG_SPECIAL_ITEM, 
CTF_DB_ISVISIBLE ),
-        MAP_CONST_COLUMN( PROPERTY_NUMBERFORMAT,    XML_NAMESPACE_STYLE,    
XML_DATA_STYLE_NAME,    XML_TYPE_NUMBER|MID_FLAG_SPECIAL_ITEM, 
CTF_DB_NUMBERFORMAT),
-        MAP_END()
+        { PROPERTY_WIDTH, sizeof(PROPERTY_WIDTH)-1,  XML_NAMESPACE_STYLE, 
XML_COLUMN_WIDTH, XML_TYPE_MEASURE|XML_TYPE_PROP_TABLE_COLUMN, 0, 
SvtSaveOptions::ODFSVER_010, false },
+        { PROPERTY_HIDDEN, sizeof(PROPERTY_HIDDEN)-1,  XML_NAMESPACE_TABLE, 
XML_DISPLAY, XML_DB_TYPE_EQUAL|MID_FLAG_SPECIAL_ITEM|XML_TYPE_PROP_TABLE_COLUMN,
+          CTF_DB_ISVISIBLE, SvtSaveOptions::ODFSVER_010, false },
+        { PROPERTY_NUMBERFORMAT, sizeof(PROPERTY_NUMBERFORMAT)-1,  
XML_NAMESPACE_STYLE, XML_DATA_STYLE_NAME, 
XML_TYPE_NUMBER|MID_FLAG_SPECIAL_ITEM|XML_TYPE_PROP_TABLE_COLUMN,
+           CTF_DB_NUMBERFORMAT, SvtSaveOptions::ODFSVER_010, false },
+        { nullptr, 0, 0, XML_TOKEN_INVALID, 0 , 0, 
SvtSaveOptions::ODFSVER_010, false}
     };
     rtl::Reference < XMLPropertyHandlerFactory> xFac = new 
OPropertyHandlerFactory();
     return new XMLPropertySetMapper(s_aColumnStylesProperties, xFac, 
bForExport);
@@ -95,11 +95,10 @@ rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetColumnStylesPropertySetMa
 
 rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetCellStylesPropertySetMapper( bool bForExport )
 {
-#define MAP_CONST_CELL( name, prefix, token, type, context ) { name, 
sizeof(name)-1,  prefix, token, type|XML_TYPE_PROP_PARAGRAPH, context, 
SvtSaveOptions::ODFSVER_010, false }
 #define MAP_CONST_TEXT( name, prefix, token, type, context ) { name, 
sizeof(name)-1,  prefix, token, type|XML_TYPE_PROP_TEXT, context, 
SvtSaveOptions::ODFSVER_010, false }
     static const XMLPropertyMapEntry s_aCellStylesProperties[] =
     {
-        MAP_CONST_CELL( PROPERTY_ALIGN,             XML_NAMESPACE_FO,       
XML_TEXT_ALIGN,             XML_TYPE_TEXT_ALIGN, CTF_DB_COLUMN_TEXT_ALIGN),
+        { PROPERTY_ALIGN, sizeof(PROPERTY_ALIGN)-1,  XML_NAMESPACE_FO, 
XML_TEXT_ALIGN, XML_TYPE_TEXT_ALIGN|XML_TYPE_PROP_PARAGRAPH, 
CTF_DB_COLUMN_TEXT_ALIGN, SvtSaveOptions::ODFSVER_010, false },
         MAP_CONST_TEXT( PROPERTY_FONTNAME,          XML_NAMESPACE_STYLE,    
XML_FONT_NAME,              XML_TYPE_STRING, 0 ),
         MAP_CONST_TEXT( PROPERTY_TEXTCOLOR,         XML_NAMESPACE_FO,       
XML_COLOR,                  XML_TYPE_COLOR, 0 ),
         MAP_CONST_TEXT( PROPERTY_TEXTLINECOLOR,     XML_NAMESPACE_STYLE,    
XML_TEXT_UNDERLINE_COLOR,   
XML_TYPE_TEXT_UNDERLINE_COLOR|MID_FLAG_MULTI_PROPERTY, 0 ),
@@ -128,7 +127,7 @@ rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetCellStylesPropertySetMapp
         MAP_CONST_TEXT( PROPERTY_FONTWEIGHT,        XML_NAMESPACE_FO,       
XML_FONT_WEIGHT,                XML_TYPE_TEXT_WEIGHT, 0 ),
         MAP_CONST_TEXT( PROPERTY_FONTWIDTH,         XML_NAMESPACE_STYLE,    
XML_FONT_WIDTH,                 XML_TYPE_FONT_WIDTH, 0 ),
         MAP_CONST_TEXT( PROPERTY_FONTWORDLINEMODE,  XML_NAMESPACE_STYLE,    
XML_TEXT_UNDERLINE_MODE,        
XML_TYPE_TEXT_LINE_MODE|MID_FLAG_MERGE_PROPERTY, 0 ),
-        MAP_END()
+        { nullptr, 0, 0, XML_TOKEN_INVALID, 0 , 0, 
SvtSaveOptions::ODFSVER_010, false}
     };
     rtl::Reference < XMLPropertyHandlerFactory> xFac = new 
/*OPropertyHandlerFactory*/::xmloff::OControlPropertyHandlerFactory();
     return new XMLPropertySetMapper(s_aCellStylesProperties, xFac, bForExport);
@@ -140,7 +139,7 @@ rtl::Reference < XMLPropertySetMapper > 
OXMLHelper::GetRowStylesPropertySetMappe
     static const XMLPropertyMapEntry s_aStylesProperties[] =
     {
         MAP_CONST_ROW( PROPERTY_ROW_HEIGHT,         XML_NAMESPACE_STYLE,    
XML_ROW_HEIGHT,             XML_TYPE_MEASURE, 0),
-        MAP_END()
+        { nullptr, 0, 0, XML_TOKEN_INVALID, 0 , 0, 
SvtSaveOptions::ODFSVER_010, false}
     };
     rtl::Reference < XMLPropertyHandlerFactory> xFac = new 
OPropertyHandlerFactory();
     return new XMLPropertySetMapper(s_aStylesProperties, xFac, 
true/*bForExport*/);

Reply via email to