Hello Julien
On 02/01/2012 11:40 PM, julien2412 wrote:
Here are the lines :
961 // dann mal die Tabellenkoepfe raus:
962 for( SwSelBoxes::iterator it = rBoxes.begin(); it !=
rBoxes.end(); ++it )
963 {
964 pLine =
Hello Stephan,
On 24.01.2012 09:30, Stephan Bergmann wrote:
However, I have another question: I notice that, while most
occurrences are replaced with SAL_WARN, some were left as OSL_ENSURE
(which I changed to SAL_WARN; one missed a comma, anyway) and some
are changed to SAL_INFO. Was there
be skipped.
The code size shouldn't matter because after the preprocessor run the
output is exactly the same and I prefer code clarity over source code size.
regards Marcel Metz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http
Hello lo-devs,
the attached patches replace some of the ENSURE_OR_* macros, with
regular code. I can't see any advantage in these macros aside from save
some typing.
regards Marcel Metz
From 1d0aeb0f7266646da553ea7d120afcc9065118e5 Mon Sep 17 00:00:00 2001
Message-Id
ignore the memory block.
regards
Marcel Metz
---
sfx2/source/appl/appopen.cxx |6 -
sfx2/source/appl/appserv.cxx |4 -
sfx2/source/bastyp/bitset.cxx| 14 ---
sfx2/source/bastyp/minarray.cxx |9 --
sfx2/source/control/bindings.cxx | 22
sfx2/source/control
---
binfilter/bf_sfx2/source/bastyp/sfx2_bitset.cxx | 10 --
binfilter/bf_sfx2/source/bastyp/sfx2_minarray.cxx |6 --
2 files changed, 0 insertions(+), 16 deletions(-)
diff --git a/binfilter/bf_sfx2/source/bastyp/sfx2_bitset.cxx
Hello lo-devs,
this patch removes another use of the tools Table class.
regards
Marcel Metz
---
vcl/source/control/combobox.cxx | 21 +++--
1 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/vcl/source/control/combobox.cxx b/vcl/source/control/combobox.cxx
---
cui/source/dialogs/SpellDialog.cxx |1 -
cui/source/dialogs/cuitbxform.cxx |1 -
cui/source/dialogs/dlgname.cxx |1 -
cui/source/tabpages/tabarea.cxx|1 -
fileaccess/source/FileAccess.cxx |1 -
sc/source/ui/view/tabview5.cxx |1 -
---
binfilter/bf_so3/source/solink/impldde.hxx |1 +
binfilter/inc/bf_basic/sbstar.hxx |1 +
binfilter/inc/bf_sc/cellsuno.hxx |1 +
binfilter/inc/bf_sw/crsrsh.hxx |1 +
4 files changed, 4 insertions(+), 0 deletions(-)
diff --git
---
binfilter/inc/bf_sfx2/fcontnr.hxx |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/binfilter/inc/bf_sfx2/fcontnr.hxx b/binfilter/inc/bf_sfx2/fcontnr.hxx
index 07f9c75..13d62fb 100644
--- a/binfilter/inc/bf_sfx2/fcontnr.hxx
+++ b/binfilter/inc/bf_sfx2/fcontnr.hxx
@@
Hello lo-devs,
this patch removes the unused (according to OpenGrok) GetIsoFallback
function from the tools module.
regards Marcel
---
rsc/source/parser/rscdb.cxx |1 -
tools/Library_tl.mk |1 -
tools/Package_inc.mk|1 -
tools/inc/tools/isofallback.hxx |
it would be maybe a good
idea to test this patch.
regards Marcel Metz
---
basic/source/classes/sb.cxx|1 -
chart2/source/view/axes/VAxisProperties.cxx|1 -
chart2/source/view/diagram/VDiagram.cxx|1 -
chart2/source/view/main
---
cui/source/factory/cuiresmgr.cxx |1 -
cui/source/factory/dlgfact.cxx |1 -
cui/source/inc/about.hxx |1 -
dbaccess/source/core/api/viewcontainer.cxx |1 -
dbaccess/source/ui/inc/charsets.hxx
---
avmedia/source/gstreamer/gstcommon.hxx |1 -
avmedia/source/quicktime/quicktimecommon.hxx |1 -
avmedia/source/win/wincommon.hxx |1 -
basic/source/sbx/sbxres.cxx|2 ++
basic/source/sbx/sbxres.hxx
---
sd/source/filter/eppt/pptx-epptbase.cxx |1 -
sfx2/source/doc/docfile.cxx |1 -
sfx2/source/doc/objstor.cxx |1 -
vcl/unx/generic/app/salinst.cxx |1 -
4 files changed, 0 insertions(+), 4 deletions(-)
diff --git
---
binfilter/inc/bf_starmath/smmod.hxx |2 ++
binfilter/inc/bf_sw/shellres.hxx|2 +-
2 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/binfilter/inc/bf_starmath/smmod.hxx
b/binfilter/inc/bf_starmath/smmod.hxx
index f3f4ec4..1ec73db 100644
---
Hello lo-devs,
There is never an instance of mpDateTable allocated and a lot of
code is never executed because of that. This patch removes the
unused code.
regards Marcel
---
svtools/inc/svtools/calendar.hxx|3 -
svtools/source/control/calendar.cxx | 119
Hello lo-devs,
this patch reduces the copyright notice noise of the FCFGMerge tool.
regards Marcel
---
l10ntools/source/filter/merge/FCFGMerge.java |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/l10ntools/source/filter/merge/FCFGMerge.java
Hello Eike,
after testing the change tracking feature in calc I've found yet
another bug in the UI. The attached patch fixes the bug in
question.
regards Marcel
---
sc/source/ui/miscdlgs/acredlin.cxx |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
Hello lo-devs,
this patch replaces yet another Table class use.
regards Marcel Metz
---
sw/source/core/text/inftxt.hxx | 19 ---
1 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/sw/source/core/text/inftxt.hxx b/sw/source/core/text/inftxt.hxx
index 34845ec
On 21.12.2011 20:27, Michael Stahl wrote:
pushed to master, with 2 small corrections:
this here inverted the logic, Get == 0 means not found:
- if ( p != pAct !rTable.Get( p-GetActionNumber() ) )
+ if ( p != pAct rMap.find( p-GetActionNumber() ) != rMap.end() )
and in
On 21.12.2011 21:00, Eike Rathke wrote:
The assertion tells if there was a dup, in that case something was going
wrong and memory leaks.
Anyway, aparently you were confused by the two different tables/maps and
mixing iterator from different map, see
Hello lo-devs,
This patch series replaces some DECLARE_TABLE macros with
the std::map equivalent.
regards
Marcel
---
vcl/source/window/accel.cxx | 20
1 files changed, 12 insertions(+), 8 deletions(-)
diff --git a/vcl/source/window/accel.cxx
---
cui/source/dialogs/pastedlg.cxx | 20
cui/source/inc/pastedlg.hxx |4 ++--
2 files changed, 14 insertions(+), 10 deletions(-)
diff --git a/cui/source/dialogs/pastedlg.cxx b/cui/source/dialogs/pastedlg.cxx
index 47fdf2f..656a813 100644
---
---
cui/source/options/optasian.cxx | 73
+++
1 files changed, 36 insertions(+), 37 deletions(-)
diff --git a/cui/source/options/optasian.cxx b/cui/source/options/optasian.cxx
index 5cafe5d..326f4dc 100644
--- a/cui/source/options/optasian.cxx
+++
---
svl/source/misc/inettype.cxx | 38 ++
1 files changed, 22 insertions(+), 16 deletions(-)
diff --git a/svl/source/misc/inettype.cxx b/svl/source/misc/inettype.cxx
index 1cba9dc..b043f06 100644
--- a/svl/source/misc/inettype.cxx
+++
Hello LO-devs,
As the subject already mentions this patch removes unnessecary
header includes for the tools table implementation.
I'm using OpenGrok quite often for the cleanups so I would like
to know how often the indices are updated.
---
filter/source/xsltdialog/xmlfileview.hxx |
---
vcl/source/gdi/cvtsvm.cxx | 19 +++
1 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/vcl/source/gdi/cvtsvm.cxx b/vcl/source/gdi/cvtsvm.cxx
index 7d9ec4d..bb5628f 100644
--- a/vcl/source/gdi/cvtsvm.cxx
+++ b/vcl/source/gdi/cvtsvm.cxx
@@ -29,7 +29,6 @@
---
sc/source/filter/inc/filt_pch.hxx |1 -
tools/Package_inc.mk |1 -
tools/inc/tools/stack.hxx | 103
-
3 files changed, 0 insertions(+), 105 deletions(-)
delete mode 100644 tools/inc/tools/stack.hxx
diff --git
---
sc/source/filter/xml/xmlfilti.hxx | 24 ++--
1 files changed, 18 insertions(+), 6 deletions(-)
diff --git a/sc/source/filter/xml/xmlfilti.hxx b/sc/source/filter/xml/xmlfilti.hxx
index 92a094c..11a5300 100644
--- a/sc/source/filter/xml/xmlfilti.hxx
+++
---
sc/source/filter/inc/xcl97esc.hxx |4 ++--
sc/source/filter/xcl97/xcl97esc.cxx | 10 +-
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/sc/source/filter/inc/xcl97esc.hxx b/sc/source/filter/inc/xcl97esc.hxx
index df7c09d..a2af601 100644
---
---
sc/source/filter/html/htmlpars.cxx | 14 ++
sc/source/filter/inc/htmlpars.hxx |4 +---
2 files changed, 11 insertions(+), 7 deletions(-)
diff --git a/sc/source/filter/html/htmlpars.cxx b/sc/source/filter/html/htmlpars.cxx
index bb79d59f..67644c2 100644
---
---
vcl/inc/vcl/cvtsvm.hxx|5 ++---
vcl/source/gdi/cvtsvm.cxx | 16 ++--
2 files changed, 12 insertions(+), 9 deletions(-)
diff --git a/vcl/inc/vcl/cvtsvm.hxx b/vcl/inc/vcl/cvtsvm.hxx
index de4158e..370871b 100644
--- a/vcl/inc/vcl/cvtsvm.hxx
+++ b/vcl/inc/vcl/cvtsvm.hxx
---
filter/source/svg/svgwriter.hxx | 17 +
1 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/filter/source/svg/svgwriter.hxx b/filter/source/svg/svgwriter.hxx
index dd033a2..0b6048e 100644
--- a/filter/source/svg/svgwriter.hxx
+++
Hello libreoffice devs,
this is my first patch series to the lo project. It replaces the use
of the tools Stack class and corresponding macros with the std::stack
as suggested in in bug #39445 [1] (an easy hack).
I would like to continue with cleaning up the tools module. I've seen
that there
---
starmath/inc/parse.hxx |3 +-
starmath/source/math_pch.cxx |1 -
starmath/source/mathmlimport.cxx | 223
++
starmath/source/mathmlimport.hxx |8 ++-
starmath/source/parse.cxx| 222
--
5
Hello Eike,
On 01.12.2011 20:28, Eike Rathke wrote:
Hi Marcel,
On Thursday, 2011-12-01 11:08:40 +0100, Marcel Metz wrote:
this is my first patch series to the lo project.
Wow, large and great one :-)
I'll inspect details later or tomorrow, from a first glance it looks
good.
In the mean
Hello Eike,
In the mean time, as this is your first contribution to the
code base, could you please send a blanket statement to the
mailing list (preferably in this thread) that you contribute
this and further patches under LGPLv3+ and MPL 1.1 licenses?
I totally forgot that, well today
38 matches
Mail list logo