Re: [Libreoffice] Mi first very little patch

2011-09-20 Thread CaStarCo
2011/9/20 Lionel Elie Mamane > On Mon, Sep 12, 2011 at 10:45:31PM +0200, CaStarCo wrote: > > > @Lionel , What is the QuadPart member of the structure? The high > > part? the low part? or another thing? > > The union is fully described at > http://msdn.microsoft.com/en-

[Libreoffice] Removed redundant pointer check in /sw/source/ui/uno/unomailmerge.cxx

2011-09-17 Thread CaStarCo
Hello, I've removed a redundant pointer check in the file core/sw/source/ui/uno/unomailmerge.cxx (in the line 630). The check can be removed because the pointer is checked before in the line 622 (and if the pointer is null throws a RuntimeException). I can't build libreoffice with this patch, but

[Libreoffice] About a probable bug in core/lotuswordpro/source/filter/xfilter/xfcell.cxx

2011-09-17 Thread CaStarCo
Hello :) , reading the list of http://libreoffice.boldandbusted.com/ i've found a potential bug in core/lotuswordpro/source/filter/xfilter/xfcell.cxx . The member m_pOwnerRow of the XFCell is not initialized in any constructor (and in many methodts it's setted to null), but in the line 177 the va

Re: [Libreoffice] Mi first very little patch

2011-09-12 Thread CaStarCo
@Lionel , What is the QuadPart member of the structure? The high part? the low part? or another thing? Thanks in advance :) . 2011/9/12 Lionel Elie Mamane > On Sun, Sep 11, 2011 at 10:04:52PM +0200, CaStarCo wrote: > > I've created a second (very little too) patch to reduce

Re: [Libreoffice] Mi first very little patch

2011-09-12 Thread CaStarCo
Thanks you very much :) , I'll take a look and try to make better patches. And, of course, all my patches will be licensed under MPL1.1/LGPLv3+ . ^_^ . ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/li

Re: [Libreoffice] Mi first very little patch

2011-09-11 Thread CaStarCo
Ups :( , my patch introduced a bug, it's not valid. 2011/9/11 CaStarCo > Hello, I've created a very little patch to solve the next CppCheck warning > ( http://libreoffice.boldandbusted.com/464.html ) . (This is my first > patch in LibreOffice) > > -- - Per la

Re: [Libreoffice] Mi first very little patch

2011-09-11 Thread CaStarCo
I've created a second (very little too) patch to reduce the scope of a variable. From 7c434fd7af9892fee7b120e00a135d01693265af Mon Sep 17 00:00:00 2001 From: Andreu Correa Casablanca Date: Sun, 11 Sep 2011 22:01:10 +0200 Subject: [PATCH] Reduced the scope of a variable in core/sal/osl/w32/file_dir

[Libreoffice] Mi first very little patch

2011-09-11 Thread CaStarCo
Hello, I've created a very little patch to solve the next CppCheck warning ( http://libreoffice.boldandbusted.com/464.html ) . (This is my first patch in LibreOffice) From b7646c6393ae557c9f3e11ea68651fca0589360c Mon Sep 17 00:00:00 2001 From: Andreu Correa Casablanca Date: Sun, 11 Sep 2011 21:28: