Michael Stahl-2 wrote > > On 27/02/12 23:51, Riccardo Magliocchetti wrote: > > indeed, and that is _especially_ true if you want your fix backported to > a release branch; William, please post a no-whitespace fix for review. >
Don't pay attention to the indent /whitespace correction I did, it wasn't the main purpose of my patch, which I publish you now divided in two files: - http://nabble.documentfoundation.org/file/n3783713/0001-Fix-controls-size-and-position-on-the-document-infor.patch PATCH for the size and position (0001-Fix-controls-size-and-position-on-the-document-infor.patch) - http://nabble.documentfoundation.org/file/n3783713/0001-Fix-indent.patch PATCH indent (0001-Fix-indent.patch) I won't waste my time anymore on the whitespace cleanup, I will just keep focused on tab/indent problems I see when coding. I just took the opportunity to correct them when reading the code, in order to keep a coherence throughout the whole file which is mostly using 'foo( bar )' style. Regards, William Gathoye -- View this message in context: http://nabble.documentfoundation.org/PATCH-3-5-1-and-MASTER-Fix-dialog-controls-size-and-position-in-the-file-preperties-dialog-tp3782340p3783713.html Sent from the Dev mailing list archive at Nabble.com. _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
