[Libreoffice] [PATCH] cppcheck scope

This patch should fix the cppcheck style warnings for grfmgr.cxx: core/svtools/source/graphic/grfmgr.cxx 278 variableScope style The scope of the variable 'nNewLeft' can be reduced 278 variableScope style The scope of the variable 'nNewTop' can be reduced 278 variableScope

[Libreoffice] [PATCH]: fixes compiler warnings for vcl/workben/svpclient.cxx

Hi, this patch fixes two unchecked return value-warnings in svpclient // Kenneth diff --git a/vcl/workben/svpclient.cxx b/vcl/workben/svpclient.cxx index e7e32d7..af56dc3 100644 --- a/vcl/workben/svpclient.cxx +++ b/vcl/workben/svpclient.cxx @@ -248,10 +248,15 @@ rtl::OString

[Libreoffice] Compiler warnings ICC

Hi, there are a lot of compiler warnings when building ICC. It seems that building ICC consists of extracting src/fdb27bfe2dbe2e7b57ae194d9bf36bab-SampleICC-1.3.2.tar.gz and applying a patch, then compiling it. Most, if not all, of the warnings are in the SampleICC code. IccTagLut.cpp:4336:

Re: [Libreoffice] [PUSHED] cpp cleanliness: fixed some memleaks binfilter

2011/1/28 Michael Meeks michael.me...@novell.com Hi there, On Thu, 2011-01-27 at 20:55 +0100, Kenneth Venken wrote: these patches remove some comments from ./filters/binfilter/bf_svx/source/editeng/svx_editobj.cxx and I've pushed these two. fixes two memleaks

[Libreoffice] [PATCH] Cpp Cleanliness: unread variable

hi, this removes unread variable pTargetPage from ./impress/sd/source/core/drawdoc2.cxxhttp://libreoffice.boldandbusted.com/368.html -- Kenneth From 790ae5ce71d14ce6100fc97d81b24f960e8323d5 Mon Sep 17 00:00:00 2001 From: Kenneth Venken kenneth.ven...@gmail.com Date: Sun, 30 Jan 2011 01:05:55

[Libreoffice] [PATCH: 2] Cpp Cleanliness: unread variable

Hi, this removes the unreadVariable lame_frame_size in ./filters/filter/source/flash/swfwriter.cxxhttp://libreoffice.boldandbusted.com/269.html -- Kenneth From 10e64f12f50fb2186676b053d9d130fdbccaf91b Mon Sep 17 00:00:00 2001 From: Kenneth Venken kenneth.ven...@gmail.com Date: Sun, 30 Jan 2011

[Libreoffice] [PATCH] Cpp Cleanliness: memleaks

Fixes two memleaks. -- Kenneth From 4cd88904ea50619dbc0965ea2683ea270a74013b Mon Sep 17 00:00:00 2001 From: Kenneth Venken kenneth.ven...@gmail.com Date: Wed, 26 Jan 2011 19:39:13 +0100 Subject: [PATCH] cpp cleanliness: fixed some memleaks --- fpicker/source/unx/kde/kdecommandthread.cxx |1

[Libreoffice] [PATCH] Cpp Cleanliness: redundant assignment to self

Hi, these patches solve some of the 'redundant assignment to self' warnings. Released under LGPLv3 -- Kenneth From 29ee756d84415238dbbd4785dc6d7f4d2b0be296 Mon Sep 17 00:00:00 2001 From: Kenneth Venken kenneth.ven...@gmail.com Date: Tue, 25 Jan 2011 18:39:18 +0100 Subject: [PATCH] Cpp

[Libreoffice] more SAL_N_ELEMENTS

hey, more use of the SAL_N_ELEMENTS macro like replacing the STATIC_TABLE_SIZE macro. -- Kenneth From 62a94ea8d83b129ef479304865d3f2085b191c32 Mon Sep 17 00:00:00 2001 From: Kenneth Venken kenneth.ven...@gmail.com Date: Fri, 3 Dec 2010 01:41:20 +0100 Subject: [PATCH] more Sal n elements

Re: [Libreoffice] [PATCH] SAL_N_ELEMENTS for ure

Michael Meeks schreef op ma 18-10-2010 om 19:49 [+0100]: Hi there, On Mon, 2010-10-18 at 13:50 +0200, Kenneth Venken wrote: this patch replaces all occurrences of sizeof.*/.*sizeof.*\[ with SAL_N_ELEMENTS in ure Thanks - there were a few sillies in there: #include sal

[Libreoffice] [LibreOffice][PATCH]SAL_N_ELEMENTS

some patches that replace sizeof(a)/sizeof(a[0]) with SAL_N_ELEMENTS(a) diff --git a/sc/source/ui/vba/vbapagesetup.cxx b/sc/source/ui/vba/vbapagesetup.cxx index 3b1a8ae..c80e407 100644 --- a/sc/source/ui/vba/vbapagesetup.cxx +++ b/sc/source/ui/vba/vbapagesetup.cxx @@ -41,6 +41,7 @@ #include

Re: [Libreoffice] [PATCH] SAL_N_ELEMENTS changes for components

Michael Meeks schreef op vr 15-10-2010 om 18:23 [+0100]: Hi Kenneth, Hi Michael, On Fri, 2010-10-15 at 01:29 +0200, Kenneth Venken wrote: this is my first patch for Libreoffice. contributed under the LGPLv3+ Welcome ! :-) many thanks for your work, it built nicely. What piece