On Tue, 2010-11-02 at 18:44 +0100, Gert Faller wrote:
> Hi,
> 
> with this one, module 'impress' should be rather clean of that.

Heh, good, liked the ? : rework to keep me pacified :-). Thanks for
this.

As an aside, stylistically I'd prefer
rtl::OUString sFoo(RTL_CONSTASCII_USTRINGPARAM("apple"));
over
rtl::OUString sFoo(rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("apple")));
or
rtl::OUString sFoo =
    rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("apple"));
even though they're all exactly equivalent, shorter text and more
direct, so I modified a few of those in passing while committing this.

C.

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to