[PUSHED] Re: [PATCH} more SV_DECL_PTRARR conversions

Michael Stahl mstahl at redhat.com
Tue Jun 12 14:49:41 PDT 2012


On 12.06.2012 18:20, Noel Grandin wrote:
> Hi
> 
> These patches converts various SV_DECL_PTRARR stuff to std::set and 
> std::vector and boost::ptr_vector in various modules
> 
> Passes "make check" at global level, and compiled with debug=true

pushed, thanks.

a bunch of your patches do this:

>  typedef ::com::sun::star::uno::Reference<
> -            ::com::sun::star::util::XRefreshListener >* XRefreshListenerPtr;
> -SV_DECL_PTRARR_DEL( XRefreshListenerArr_Impl, XRefreshListenerPtr, 4 )
> +            ::com::sun::star::util::XRefreshListener > XRefreshListenerRef;
> +typedef boost::ptr_vector<XRefreshListenerRef> XRefreshListenerArr_Impl;

a ptr_vector<Reference<XFoo>> is, while not technically wrong, kind of
silly; but you've added these in Calc so i don't care that much :)

patch #16 SbCompare_UString_PropertyValue_Impl doesn't look like a
strict weak ordering to me; while looking at that noticed pre-exiting
problem that the GetIndex_Impl return value is not checked...

patch #21 SfxFilterPtrArr turns out to be unused actually



More information about the LibreOffice mailing list