[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