On Thu, 2012-08-30 at 20:52 +0000, Manal M. Alhassoun wrote:
> 
> Hello,
> 
> Could you please review the attached patch.

Looks reasonable to me. Or maybe
if (Application::GetSettings().GetLayoutRTL() != bLayoutRTL)
    pFilterBox->EnableMirroring();
is a little shorter with the same effect ?

Either way, could you add yourself to
http://wiki.documentfoundation.org/Development/Developers
and send a mail like
http://lists.freedesktop.org/archives/libreoffice/2012-July/035301.html
to this list to state that your contributions are under our preferred
MPL+/LGPLv3+ license combination, and then we can merge it in.

C.


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

Reply via email to