This patch is bringing up a small question: I thought we were trying to standardize the code to use sal_* types everywhere...
Can anyone confirm/deny that? Regards, Marc-André Laverdière Software Security Scientist Innovation Labs, Tata Consultancy Services Hyderabad, India On 07/27/2011 06:54 AM, Kohei Yoshida wrote: > sw/source/filter/html/parcss1.cxx | 4 ++-- > sw/source/filter/html/parcss1.hxx | 3 +-- > sw/source/filter/html/svxcss1.cxx | 29 +++++++++-------------------- > sw/source/filter/html/svxcss1.hxx | 13 ++++--------- > 4 files changed, 16 insertions(+), 33 deletions(-) > > New commits: > commit 04113a04a20d758d914833fc22d646831478bf1b > Author: Kohei Yoshida <kyosh...@novell.com> > Date: Tue Jul 26 21:18:06 2011 -0400 > > Replace SV_*_PTRARR with boost::ptr_vector. > > diff --git a/sw/source/filter/html/parcss1.cxx > b/sw/source/filter/html/parcss1.cxx > index 5c0955f..3f3ccde 100644 > --- a/sw/source/filter/html/parcss1.cxx > +++ b/sw/source/filter/html/parcss1.cxx > @@ -1230,10 +1230,10 @@ sal_Bool CSS1Parser::ParseStyleOption( const String& > rIn ) > return sal_True; > } > > -sal_Bool CSS1Parser::SelectorParsed( const CSS1Selector * /* pSelector */, > sal_Bool /*bFirst*/ ) > +bool CSS1Parser::SelectorParsed( CSS1Selector* /* pSelector */, bool > /*bFirst*/ ) > { > // Selektor loeschen > - return sal_True; > + return true; > } > > sal_Bool CSS1Parser::DeclarationParsed( const String& /*rProperty*/, > diff --git a/sw/source/filter/html/parcss1.hxx > b/sw/source/filter/html/parcss1.hxx > index 3a6fa46..5df550b 100644 > --- a/sw/source/filter/html/parcss1.hxx > +++ b/sw/source/filter/html/parcss1.hxx > @@ -270,8 +270,7 @@ protected: > // Deklaration. Wird sal_True zurueckgegeben, wird der Selektor > // geloscht, sonst nicht. > // Die Implementierung dieser Methode gibt nur sal_True zuruck. > - virtual sal_Bool SelectorParsed( const CSS1Selector *pSelector, > - sal_Bool bFirst ); > + virtual bool SelectorParsed( CSS1Selector* pSelector, bool bFirst ); > > // Diese Methode wird fuer jede geparsete Property aufgerufen. Wird > // sal_True zurueckgegeben wird der Selektor geloscht, sonst nicht. > diff --git a/sw/source/filter/html/svxcss1.cxx > b/sw/source/filter/html/svxcss1.cxx > index 2fcf673..93b5fad 100644 > --- a/sw/source/filter/html/svxcss1.cxx > +++ b/sw/source/filter/html/svxcss1.cxx > @@ -71,11 +71,6 @@ typedef void (*FnParseCSS1Prop)( const CSS1Expression > *pExpr, > SvxCSS1PropertyInfo& rPropInfo, > const SvxCSS1Parser& rParser ); > > -SV_IMPL_PTRARR( CSS1Selectors, CSS1Selector* ) > - > - > -/* */ > - > static CSS1PropertyEnum const aFontSizeTable[] = > { > { sCSS1_PV_xx_small, 0 }, > @@ -720,31 +715,26 @@ sal_Bool SvxCSS1Parser::StyleParsed( const CSS1Selector > * /*pSelector*/, > return sal_True; > } > > -sal_Bool SvxCSS1Parser::SelectorParsed( const CSS1Selector *pSelector, > - sal_Bool bFirst ) > +bool SvxCSS1Parser::SelectorParsed( CSS1Selector *pSelector, bool bFirst ) > { > if( bFirst ) > { > OSL_ENSURE( pSheetItemSet, "Where is the Item-Set for Style-Sheets?" > ); > > - // Dieses ist der erste Selektor einer Rule, also muessen > - // die bisher geparsten Items auf die Styles verteilt werden > -// pSheetPropInfo->CreateBoxItem( *pSheetItemSet, > GetDfltBorderDist() ); > - for( sal_uInt16 i=0; i<aSelectors.Count(); i++ ) > + for (size_t i = 0; i < aSelectors.size(); ++i) > { > - StyleParsed( aSelectors[i], *pSheetItemSet, *pSheetPropInfo ); > + StyleParsed( &aSelectors[i], *pSheetItemSet, *pSheetPropInfo ); > } > pSheetItemSet->ClearItem(); > pSheetPropInfo->Clear(); > > // und die naechste Rule vorbereiten > - if( aSelectors.Count() ) > - aSelectors.DeleteAndDestroy( 0, aSelectors.Count() ); > + aSelectors.clear(); > } > > - aSelectors.C40_INSERT( CSS1Selector, pSelector, aSelectors.Count() ); > + aSelectors.push_back(pSelector); > > - return sal_False; // den Selektor haben wir gespeichert. Loeschen > toedlich! > + return false; // den Selektor haben wir gespeichert. Loeschen toedlich! > } > > > @@ -840,14 +830,13 @@ sal_Bool SvxCSS1Parser::ParseStyleSheet( const String& > rIn ) > > sal_Bool bSuccess = CSS1Parser::ParseStyleSheet( rIn ); > > - for( sal_uInt16 i=0; i<aSelectors.Count(); i++ ) > + for (size_t i = 0; i < aSelectors.size(); ++i) > { > - StyleParsed( aSelectors[i], *pSheetItemSet, *pSheetPropInfo ); > + StyleParsed( &aSelectors[i], *pSheetItemSet, *pSheetPropInfo ); > } > > // und etwas aufrauemen > - if( aSelectors.Count() ) > - aSelectors.DeleteAndDestroy( 0, aSelectors.Count() ); > + aSelectors.clear(); > pSheetItemSet->ClearItem(); > pSheetPropInfo->Clear(); > > diff --git a/sw/source/filter/html/svxcss1.hxx > b/sw/source/filter/html/svxcss1.hxx > index 95d682d..b34c0ba 100644 > --- a/sw/source/filter/html/svxcss1.hxx > +++ b/sw/source/filter/html/svxcss1.hxx > @@ -39,6 +39,8 @@ > #include <rtl/textenc.h> > #include "parcss1.hxx" > > +#include <boost/ptr_container/ptr_vector.hpp> > + > class SfxItemPool; > class SvxBoxItem; > class FontList; > @@ -93,21 +95,14 @@ enum SvxCSS1PageBreak > #define CSS1_SCRIPT_CTL 0x04 > #define CSS1_SCRIPT_ALL 0x07 > > -/* */ > - > struct CSS1PropertyEnum > { > const sal_Char *pName; // Wert einer Property > sal_uInt16 nEnum; // und der dazugehoerige Wert > eines Enums > }; > > - > -/* */ > - > namespace editeng { class SvxBorderLine; } > > -SV_DECL_PTRARR_DEL( CSS1Selectors, CSS1Selector*, 1, 1 ) > - > #define SVX_CSS1_BORDERINFO_WIDTH 1 > #define SVX_CSS1_BORDERINFO_COLOR 2 > #define SVX_CSS1_BORDERINFO_STYLE 4 > @@ -227,6 +222,7 @@ inline sal_Bool operator<( const SvxCSS1MapEntry& rE1, > const SvxCSS1MapEntry& rE > > class SvxCSS1Parser : public CSS1Parser > { > + typedef ::boost::ptr_vector<CSS1Selector> CSS1Selectors; > CSS1Selectors aSelectors; // Liste der "offenen" Selectoren > > SvxCSS1Map aIds; > @@ -277,8 +273,7 @@ protected: > // zuletzt angelegten Styles kopiert. > // Diese Methode sollte in abgleiteten Parsern nicht mehr > // ueberladen werden! > - virtual sal_Bool SelectorParsed( const CSS1Selector *pSelector, > - sal_Bool bFirst ); > + virtual bool SelectorParsed( CSS1Selector *pSelector, bool bFirst ); > > // Diese Methode wird fuer jede geparste Property aufgerufen > // sie fuegt das Item in den ItemSet 'pItemSet' ein > _______________________________________________ > Libreoffice-commits mailing list > libreoffice-comm...@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits > _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice