[Libreoffice] [PATCH] Cleaned up the comments and code in the following files.

Neil Stalker nestalk at gmail.com
Fri Oct 1 13:39:50 PDT 2010


Here's a small patch to fix a typo that breaks the build.

diff --git a/sw/source/core/view/viewsh.cxx b/sw/source/core/view/viewsh.cxx
index 3feadff..cb351ba 100644
--- a/sw/source/core/view/viewsh.cxx
+++ b/sw/source/core/view/viewsh.cxx
@@ -251,7 +251,7 @@ void ViewShell::ImplEndAction( const BOOL bIdleEnd )

                 Imp()->pRegion = NULL;

-                / First Invert then Compress, never the other way round!
+                // First Invert then Compress, never the other way round!
                 pRegion->Invert();

                 pRegion->Compress();


2010/10/1 Cédric Bosdonnat <cedric.bosdonnat.ooo at free.fr>:
> Hi Nick,
>
> I finally found some courage to review all your patch. I removed some
> hunks that removed potentially interesting comments in German. Keep in
> mind that the Writer layout code is already complex: if we have some
> comments explaining it and its history: we'll want to keep them.
>
> Keep sending good patches, but please split them into smaller ones: it's
> quicker to review and integrate ;)
>
> Your patch is now pushed! Many thanks for it.
>
> On Tue, 2010-09-28 at 16:06 -0400, Nick Savage wrote:
>> I thought I would help out performing some of the simple tasks that the wiki
>> said needed to be done. I went through one folder of code and tried to
>> clean the comments up, as well as code that has been there commented out
>> for years.
>>
>> I wasn't sure if this is what I was supposed to be doing, so I only did
>> that one folder. I'd like to be helping out LibreOffice any way I can!
>
> --
> Cedric
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>


More information about the LibreOffice mailing list