Hi there,
On Sat, 2012-06-23 at 20:45 +0200, Philipp Riemer wrote:
> (1) Since it was addressed on the mailing list today (see
> <http://lists.freedesktop.org/archives/libreoffice/2012-June/034038.html>)
> I spent some time on filter/inc/filter/msfilter/msdffimp.hxx.
Thanks for this set;
> (2) In addition, I was wondering if it would be possible to shorten
> parameters (e.g. "com::sun::star::uno::Reference <
> com::sun::star::embed::XStorage >& xDestStg") by using aliases or
> other ways to point to that namespaces. But I am not skilled enough in
> C++ to do that (see e.g. patch 0003 for examples).
The great hope is LibreOffice 4 - which will discard the com::sun::star
prefix and/or do something sensible instead here, making our APIs much
easier to read.
There was also hope for a transitional set of namespace aliases, but I
forget why that was shot down, in the meantime we have that ugly,
tangled madness (sadly) :-)
> (3) Moreover, the forward declarations and "#define" blocks are widely
> spread over the whole file. Can I rearrange these so that they are all
> bundled at the top of this file? In my eyes this would make it more
> readable plus provide all needed classes and definitions on one or two
> screen pages.
Sounds good to me; as long as it's not some scripted thing across the
whole code-base (yet) ;-)
Thanks !
Michael.
--
michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
