[Libreoffice] [PUSHING] ... pre patch question
michael.meeks at suse.com
Tue Oct 18 03:28:20 PDT 2011
First - great to see you in Paris :-) thanks for sitting down to review
this stuff together.
On Mon, 2011-10-17 at 20:53 +0200, Lior Kaplan wrote:
> I'm attaching 3 patches:
> 1. Bulk removal.
> 2. Compact the enumeration.
> 3. Just align a string (no change in content).
I'm just compile testing it now.
> From what I understand, CONTENT_TYPE_LAST gets the value of the last
> member in the enumeration (at inc/svl/inettype.hxx), so the compact
> patch actually makes sure of its new value.
> I also didn't do the compile test, help will be welcomed (I'll try to
> work on a build environment for next time).
Am testing it now; of course - if you have a build - lots of little
things become quite easy to submit patches for. In fact - many of the
RTL bugs you helpfully highlighted in your talk, are not -so- tricky to
fix with some careful application of
if( Application::GetSettings().GetLayoutRTL() ) ..'s :-)
and their ilk.
> Let me know for needed fixes/changes in the patches.
Thanks - I'll take it from here.
All the best,
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice