[Libreoffice] Moving duplicate code into methods
Thorsten Behrens
thb at documentfoundation.org
Tue May 3 00:24:40 PDT 2011
Chr. Rossmanith wrote:
> I've found two duplicate code with simian and moved it into two new
> methods which saves 229 lines of code. It would be nice if someone
> could review the patch and at least suggest better names for the
> methods...
>
Nice, good work -
> @@ -104,6 +104,118 @@ using namespace ::com::sun::star::lang;
> using namespace ::com::sun::star::beans;
> using namespace ::com::sun::star::drawing;
>
> +
> +void rSegInfo_nSDat_method (EnhancedCustomShapeSegment& rSegInfo, sal_uInt16 nSDat)
>
ShapeSegmentFromBinary() maybe?
> +sal_Int32 GetNumberOfProperties ( const SvxMSDffHandle* pData )
>
and
> +void SetProperties (const SvxMSDffHandle* pData, com::sun::star::beans::PropertyValues& rPropValues)
>
looks ok to me - or maybe ShapePropertiesFromDFF or somesuch, as
set/getProps sounds somehow generic (and is quite heavily overloaded
as a name).
Additionally, please make those methods static (they're only used
inside that file) - and another loosely followed convention is to
prefix those local methods with either "lcl_" or "Impl"/"impl".
Though that's a matter of taste ... ;)
Cheers,
-- Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110503/ca5cc342/attachment.pgp>
More information about the LibreOffice
mailing list