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

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@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@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice