[Bug 54157] LibreOffice 3.7 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

--- Comment #12 from Roman Eisele b...@eikota.de ---
Adding bug 55524 - “UI: Hang when accessing ‘Insert  Manual Break...’”.

Endless hang, at least on Mac OS X. Introduced between 27th and 30th September.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 54157] LibreOffice 3.7 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Roman Eisele b...@eikota.de changed:

   What|Removed |Added

 Depends on||55524

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] gbuildification of beanshell

2012-10-02 Thread David Ostrovsky (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/745

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/745/1

gbuildification of beanshell

Change-Id: I1b71796ca95500efeffe5e01dba3e73711f920e7
---
M Makefile.top
M Module_tail_build.mk
A beanshell/ExternalProject_beanshell.mk
A beanshell/Makefile
A beanshell/Module_beanshell.mk
A beanshell/Package_beanshell.mk
A beanshell/UnpackedTarball_beanshell.mk
M beanshell/prj/build.lst
D beanshell/prj/dmake
M config_host.mk.in
M configure.in
M tail_build/prj/build.lst
12 files changed, 104 insertions(+), 5 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/745
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1b71796ca95500efeffe5e01dba3e73711f920e7
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Ostrovsky david.ostrov...@gmx.de

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] Changes to 'feature/kill-saxon'

2012-10-02 Thread Libreoffice Gerrit user
New branch 'feature/kill-saxon' available with the following commits:
commit d0931457167ae306ddfe42d79574bcd4edec05bf
Author: David Tardon dtar...@redhat.com
Date:   Thu Sep 20 10:28:56 2012 +0200

rework selection of transformer for an XSLT filter

Change-Id: I765762f11813c6b612416a8db2707bf94114c876

commit 1e0be7ca8b9e15bd6248b0f4431adcc10f6b2fdc
Author: David Tardon dtar...@redhat.com
Date:   Wed Sep 19 12:54:24 2012 +0200

convert java XSL transformer into extension

Change-Id: Ided6c480969764073056830722c8996e0df52285

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] docx import: set table above/bottom spacing to 0

2012-10-02 Thread Miklos Vajna (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/713


--
To view, visit https://gerrit.libreoffice.org/713
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e0397dd3d5d14f1e584ac7bc3d22d4638393b06
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sw/qa writerfilter/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/qa/extras/ooxmlimport/data/n779941.docx   |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |   25 +++
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |5 +++
 3 files changed, 30 insertions(+)

New commits:
commit e33a9181c76309d31f2ace01b924e404906da28c
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Wed Sep 26 15:25:39 2012 +0200

docx import: set table above/bottom spacing to 0

Conflicts:
sw/qa/extras/ooxmlimport/ooxmlimport.cxx

Change-Id: I3e0397dd3d5d14f1e584ac7bc3d22d4638393b06

diff --git a/sw/qa/extras/ooxmlimport/data/n779941.docx 
b/sw/qa/extras/ooxmlimport/data/n779941.docx
new file mode 100644
index 000..9889d81
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/n779941.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index c545a89..1267eac 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -92,6 +92,7 @@ public:
 void testInk();
 void testN779834();
 void testN779627();
+void testN779941();
 void testFdo55187();
 void testN780563();
 void testN780853();
@@ -136,6 +137,7 @@ public:
 CPPUNIT_TEST(testInk);
 CPPUNIT_TEST(testN779834);
 CPPUNIT_TEST(testN779627);
+CPPUNIT_TEST(testN779941);
 CPPUNIT_TEST(testFdo55187);
 CPPUNIT_TEST(testN780563);
 CPPUNIT_TEST(testN780853);
@@ -1006,6 +1008,29 @@ void Test::testN782345()
 CPPUNIT_ASSERT_EQUAL(style::BreakType_PAGE_BEFORE, 
getPropertystyle::BreakType(getParagraph(2), BreakType));
 }
 
+void Test::testN779941()
+{
+/*
+ * Make sure top/bottom margins of tables are set to 0 (problem was: 
bottom margin set to 0.35cm)
+ */
+load(n779941.docx);
+uno::Referencetext::XTextTablesSupplier xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Referencecontainer::XIndexAccess 
xTables(xTablesSupplier-getTextTables( ), uno::UNO_QUERY);
+uno::Referencebeans::XPropertySet 
xTableProperties(xTables-getByIndex(0), uno::UNO_QUERY);
+{
+uno::Any aValue = xTableProperties-getPropertyValue(TopMargin);
+sal_Int32 nTopMargin;
+aValue = nTopMargin;
+CPPUNIT_ASSERT_EQUAL(sal_Int32(0), nTopMargin);
+}
+{
+uno::Any aValue = xTableProperties-getPropertyValue(BottomMargin);
+sal_Int32 nBottomMargin;
+aValue = nBottomMargin;
+CPPUNIT_ASSERT_EQUAL(sal_Int32(0), nBottomMargin);
+}
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index dce337d..e710a13 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -391,6 +391,11 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 
 m_aTableProperties-Insert( PROP_TABLE_BORDER_DISTANCES, false, 
uno::makeAny( aDistances ) );
 
+// Set table above/bottom spacing to 0.
+// TODO: handle 'Around' text wrapping mode
+m_aTableProperties-Insert( PropertyIds::PROP_TOP_MARGIN, true, 
uno::makeAny( 0 ) );
+m_aTableProperties-Insert( PropertyIds::PROP_BOTTOM_MARGIN, true, 
uno::makeAny( 0 ) );
+
 //table border settings
 table::TableBorder aTableBorder;
 table::BorderLine2 aBorderLine;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: writerfilter/source

2012-10-02 Thread Libreoffice Gerrit user
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |   27 ++-
 1 file changed, 11 insertions(+), 16 deletions(-)

New commits:
commit 3cb619bd15a6017f253891f4c377fc790d8aae82
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Thu Sep 27 10:15:58 2012 +0200

docx import: properly import table position

This a follow up of commit 53b7f7df0617bcbd7bbef9a34ef53e5097eb16dc

Change-Id: Ia0f79ca24418636af14162e9f339237d847dc221
Reviewed-on: https://gerrit.libreoffice.org/714
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index e710a13..2690105 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -398,7 +398,7 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 
 //table border settings
 table::TableBorder aTableBorder;
-table::BorderLine2 aBorderLine;
+table::BorderLine2 aBorderLine, aLeftBorder;
 
 if (lcl_extractTableBorderProperty(m_aTableProperties, 
PROP_TOP_BORDER, rInfo, aBorderLine))
 {
@@ -410,10 +410,11 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 aTableBorder.BottomLine = aBorderLine;
 aTableBorder.IsBottomLineValid = sal_True;
 }
-if (lcl_extractTableBorderProperty(m_aTableProperties, 
PROP_LEFT_BORDER, rInfo, aBorderLine))
+if (lcl_extractTableBorderProperty(m_aTableProperties, 
PROP_LEFT_BORDER, rInfo, aLeftBorder))
 {
-aTableBorder.LeftLine = aBorderLine;
+aTableBorder.LeftLine = aLeftBorder;
 aTableBorder.IsLeftLineValid = sal_True;
+rInfo.nLeftBorderDistance += aLeftBorder.LineWidth * 0.5;
 }
 if (lcl_extractTableBorderProperty(m_aTableProperties, 
PROP_RIGHT_BORDER, rInfo, aBorderLine))
 {
@@ -440,23 +441,17 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 lcl_debug_TableBorder(aTableBorder);
 #endif
 
-// Mimic Office behavior : if tlbInd is defined, use it place table.
-// Otherwise, top-level table's position depends w:tblCellMar 
attribute (but not nested tables)
-if (nLeftMargin)
+// Table position in Office is computed in 2 different ways :
+// - top level tables: the goal is to have in-cell text starting at 
table indent pos (tblInd),
+//   so table's position depends on table's cells margin
+// - nested tables: the goal is to have left-most border starting at 
table_indent pos
+if (rInfo.nNestLevel  1)
 {
-m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, uno::makeAny( 
nLeftMargin - nGapHalf));
+m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, uno::makeAny( 
nLeftMargin - nGapHalf ));
 }
 else
 {
-// TODO: top-level position depends on w:tblCellMar attribute, not 
w:cellMar
-if (rInfo.nNestLevel  1)
-{
-m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, 
uno::makeAny( - nGapHalf));
-}
-else
-{
-m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, 
uno::makeAny( - nGapHalf - rInfo.nLeftBorderDistance));
-}
+m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, uno::makeAny( 
nLeftMargin - nGapHalf - rInfo.nLeftBorderDistance ));
 }
 
 m_aTableProperties-getValue( TablePropertyMap::TABLE_WIDTH, 
nTableWidth );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] docx import: properly import table position

2012-10-02 Thread Miklos Vajna (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/714

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/714
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0f79ca24418636af14162e9f339237d847dc221
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 3 commits - forms/source forms/util fpicker/source io/source

2012-10-02 Thread Libreoffice Gerrit user
 forms/source/misc/services.cxx |2 +-
 forms/util/frm.component   |2 +-
 fpicker/source/aqua/FPentry.mm |2 +-
 fpicker/source/aqua/fps_aqua.component |2 +-
 fpicker/source/generic/fpicker.component   |2 +-
 fpicker/source/generic/fpicker.cxx |2 +-
 fpicker/source/office/fps_office.component |2 +-
 fpicker/source/office/fps_office.cxx   |2 +-
 fpicker/source/win32/filepicker/FPentry.cxx|2 +-
 fpicker/source/win32/fps.component |2 +-
 io/source/TextInputStream/TextInputStream.cxx  |2 +-
 io/source/TextInputStream/textinstream.component   |2 +-
 io/source/TextOutputStream/TextOutputStream.cxx|2 +-
 io/source/TextOutputStream/textoutstream.component |2 +-
 io/source/acceptor/acceptor.component  |2 +-
 io/source/acceptor/acceptor.cxx|2 +-
 io/source/connector/connector.component|2 +-
 io/source/connector/connector.cxx  |2 +-
 io/source/stm/factreg.cxx  |2 +-
 io/source/stm/streams.component|2 +-
 20 files changed, 20 insertions(+), 20 deletions(-)

New commits:
commit 3cba672351b2736e35fcaba1b4b465da5b4ecc9a
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 11:19:05 2012 +0300

Use prefix

Change-Id: I84ec9d6eed209eb429f5b5d4a667d3450a1435e1

diff --git a/io/source/TextInputStream/TextInputStream.cxx 
b/io/source/TextInputStream/TextInputStream.cxx
index 8015c3b..5e87cf6 100644
--- a/io/source/TextInputStream/TextInputStream.cxx
+++ b/io/source/TextInputStream/TextInputStream.cxx
@@ -493,7 +493,7 @@ SAL_DLLPUBLIC_EXPORT sal_Bool SAL_CALL component_canUnload( 
TimeValue *pTime )
 }
 
 
//==
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL textinstream_component_getFactory(
 const sal_Char * pImplName, void * pServiceManager, void * pRegistryKey )
 {
 return component_getFactoryHelper( pImplName, pServiceManager, 
pRegistryKey , g_entries );
diff --git a/io/source/TextInputStream/textinstream.component 
b/io/source/TextInputStream/textinstream.component
index 8c02149..5d543a1 100644
--- a/io/source/TextInputStream/textinstream.component
+++ b/io/source/TextInputStream/textinstream.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=textinstream
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.io.TextInputStream
 service name=com.sun.star.io.TextInputStream/
diff --git a/io/source/TextOutputStream/TextOutputStream.cxx 
b/io/source/TextOutputStream/TextOutputStream.cxx
index da32f34..beabc01 100644
--- a/io/source/TextOutputStream/TextOutputStream.cxx
+++ b/io/source/TextOutputStream/TextOutputStream.cxx
@@ -292,7 +292,7 @@ SAL_DLLPUBLIC_EXPORT sal_Bool SAL_CALL component_canUnload( 
TimeValue *pTime )
 }
 
 
//==
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL textoutstream_component_getFactory(
 const sal_Char * pImplName, void * pServiceManager, void * pRegistryKey )
 {
 return component_getFactoryHelper( pImplName, pServiceManager, 
pRegistryKey , g_entries );
diff --git a/io/source/TextOutputStream/textoutstream.component 
b/io/source/TextOutputStream/textoutstream.component
index f498f5a..807489c 100644
--- a/io/source/TextOutputStream/textoutstream.component
+++ b/io/source/TextOutputStream/textoutstream.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=textoutstream
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.io.TextOutputStream
 service name=com.sun.star.io.TextOutputStream/
diff --git a/io/source/acceptor/acceptor.component 
b/io/source/acceptor/acceptor.component
index 1fcb6ab..0b4877b 100644
--- a/io/source/acceptor/acceptor.component
+++ b/io/source/acceptor/acceptor.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=acceptor
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.io.Acceptor
 service name=com.sun.star.connection.Acceptor/
diff --git a/io/source/acceptor/acceptor.cxx b/io/source/acceptor/acceptor.cxx
index 

[Libreoffice-commits] .: sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/filter/ww8/docxattributeoutput.cxx |  145 ---
 sw/source/filter/ww8/docxattributeoutput.hxx |4 
 2 files changed, 115 insertions(+), 34 deletions(-)

New commits:
commit 9751056ba806ba9614392f3c70ada3cbd1251814
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Thu Sep 27 17:00:08 2012 +0200

docx export: export default table cell margins, based on 1st cell

Change-Id: I1a697c2a60c7979774242fb6c9b0f66baa3bb72e
Reviewed-on: https://gerrit.libreoffice.org/716
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 29aafd2..cf6f43e 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -1460,7 +1460,7 @@ static void impl_borderLine( FSHelperPtr pSerializer, 
sal_Int32 elementToken, co
 pSerializer-singleElementNS( XML_w, elementToken, xAttrs );
 }
 
-static void impl_pageBorders( FSHelperPtr pSerializer, const SvxBoxItem rBox, 
bool bUseStartEnd = false )
+static void impl_pageBorders( FSHelperPtr pSerializer, const SvxBoxItem rBox, 
sal_Int32 tag, bool bUseStartEnd = false, bool bWriteTag = true, const 
SvxBoxItem* pDefaultBorders = 0)
 {
 static const sal_uInt16 aBorders[] =
 {
@@ -1474,12 +1474,91 @@ static void impl_pageBorders( FSHelperPtr pSerializer, 
const SvxBoxItem rBox, b
 XML_bottom,
 bUseStartEnd ? XML_end : XML_right
 };
+bool tagWritten = false;
 const sal_uInt16* pBrd = aBorders;
 for( int i = 0; i  4; ++i, ++pBrd )
 {
 const SvxBorderLine* pLn = rBox.GetLine( *pBrd );
 if ( pLn )
+{
+if ( pDefaultBorders )
+{
+const SvxBorderLine* pRefLn = pDefaultBorders-GetLine( *pBrd 
);
+
+// If border is equal to default border: do not output
+if ( pRefLn  *pLn == *pRefLn) {
+continue;
+}
+}
+
+if (!tagWritten  bWriteTag) {
+pSerializer-startElementNS( XML_w, tag, FSEND );
+tagWritten = true;
+}
+
 impl_borderLine( pSerializer, aXmlElements[i], pLn, 0 );
+
+// When exporting default borders, we need to export these 2 attr
+if ( pDefaultBorders == 0 ) {
+if ( i == 2 )
+impl_borderLine( pSerializer, XML_insideH, pLn, 0 );
+else if ( i == 3 )
+impl_borderLine( pSerializer, XML_insideV, pLn, 0 );
+}
+}
+}
+if (tagWritten  bWriteTag) {
+pSerializer-endElementNS( XML_w, tag );
+}
+}
+
+static void impl_cellMargins( FSHelperPtr pSerializer, const SvxBoxItem rBox, 
sal_Int32 tag, bool bUseStartEnd = false, const SvxBoxItem* pDefaultMargins = 0)
+{
+static const sal_uInt16 aBorders[] =
+{
+BOX_LINE_TOP, BOX_LINE_LEFT, BOX_LINE_BOTTOM, BOX_LINE_RIGHT
+};
+
+const sal_Int32 aXmlElements[] =
+{
+XML_top,
+bUseStartEnd ? XML_start : XML_left,
+XML_bottom,
+bUseStartEnd ? XML_end : XML_right
+};
+bool tagWritten = false;
+const sal_uInt16* pBrd = aBorders;
+for( int i = 0; i  4; ++i, ++pBrd )
+{
+sal_Int32 nDist = sal_Int32( rBox.GetDistance( *pBrd ) );
+
+if ( aBorders[i] == BOX_LINE_LEFT ) {
+// Office's cell margin is measured from the right of the border.
+// While LO's cell spacing is measured from the center of the 
border.
+// So we add half left-border width to tblIndent value
+const SvxBorderLine* pLn = rBox.GetLine( *pBrd );
+if (pLn)
+nDist -= pLn-GetWidth() * 0.5;
+}
+
+if (pDefaultMargins)
+{
+// Skip output if cell margin == table default margin
+if (sal_Int32( pDefaultMargins-GetDistance( *pBrd ) ) == nDist)
+continue;
+}
+
+if (!tagWritten) {
+pSerializer-startElementNS( XML_w, tag, FSEND );
+tagWritten = true;
+}
+pSerializer-singleElementNS( XML_w, aXmlElements[i],
+   FSNS( XML_w, XML_w ), OString::valueOf( nDist ).getStr( ),
+   FSNS( XML_w, XML_type ), dxa,
+   FSEND );
+}
+if (tagWritten) {
+pSerializer-endElementNS( XML_w, tag );
 }
 }
 
@@ -1526,41 +1605,20 @@ void DocxAttributeOutput::TableCellProperties( 
ww8::WW8TableNodeInfoInner::Point
 FSEND );
 }
 
-// The cell borders
-m_pSerializer-startElementNS( XML_w, XML_tcBorders, FSEND );
-SwFrmFmt *pFmt = pTblBox-GetFrmFmt( );
-impl_pageBorders( m_pSerializer, pFmt-GetBox( ), !bEcma );
-m_pSerializer-endElementNS( XML_w, XML_tcBorders );
+const SvxBoxItem rBox = pTblBox-GetFrmFmt( )-GetBox( );
+

[PUSHED] docx export: export default table cell margins, based on 1st...

2012-10-02 Thread Miklos Vajna (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/716

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/716
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a697c2a60c7979774242fb6c9b0f66baa3bb72e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sw/qa

2012-10-02 Thread Libreoffice Gerrit user
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx |   49 +++
 1 file changed, 49 insertions(+)

New commits:
commit 9a5eb5f51d29b3af4e2d23653508196cc68c0b87
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Thu Sep 27 17:00:37 2012 +0200

docx export: add unit test for 9751056

Change-Id: I25aa07d3e1811fdf4ab0124e14794e70d7de82ec

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index c3d7fc4..6f8689a 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -30,6 +30,13 @@
 #include com/sun/star/frame/XStorable.hpp
 #include com/sun/star/style/TabStop.hpp
 #include com/sun/star/view/XViewSettingsSupplier.hpp
+#include com/sun/star/text/XTextFrame.hpp
+#include com/sun/star/text/XTextTable.hpp
+#include com/sun/star/text/XTextFramesSupplier.hpp
+#include com/sun/star/text/XTextViewCursorSupplier.hpp
+#include com/sun/star/style/ParagraphAdjust.hpp
+#include com/sun/star/view/XSelectionSupplier.hpp
+#include com/sun/star/table/BorderLine2.hpp
 
 #include unotools/tempfile.hxx
 
@@ -55,6 +62,7 @@ public:
 void testMathRad();
 void testMathSubscripts();
 void testMathVerticalStacks();
+void testTablePosition();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX)  !defined(WNT)
@@ -77,6 +85,7 @@ public:
 CPPUNIT_TEST(testMathRad);
 CPPUNIT_TEST(testMathSubscripts);
 CPPUNIT_TEST(testMathVerticalStacks);
+CPPUNIT_TEST(testTablePosition);
 #endif
 CPPUNIT_TEST_SUITE_END();
 
@@ -348,6 +357,46 @@ void Test::testMathVerticalStacks()
 //CHECK_FORMULA( binom {a} {binom {b} {c}}, getFormula( getRun( 
getParagraph( 4 ), 1 )));
 }
 
+void Test::testTablePosition()
+{
+// This is the reverse test of n779957 from ooxmlimport
+// We want to verify that a full round-trip does not break formatting
+// Of course if import code is wrong, this tests will fail regardless of 
export code.
+roundtrip( ../../ooxmlimport/data/n779957.docx );
+
+sal_Int32 xCoordsFromOffice[] = { 2500, -1000, 0, 0 };
+sal_Int32 cellLeftMarginFromOffice[] = { 250, 100, 0, 0 };
+
+uno::Referencetext::XTextTablesSupplier xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Referenceframe::XModel xModel(mxComponent, uno::UNO_QUERY);
+uno::Referencecontainer::XIndexAccess 
xTables(xTablesSupplier-getTextTables( ), uno::UNO_QUERY);
+
+for (int i=0; i4; i++) {
+uno::Referencetext::XTextTable xTable1 (xTables-getByIndex(i), 
uno::UNO_QUERY);
+// Verify X coord
+uno::Referenceview::XSelectionSupplier 
xCtrl(xModel-getCurrentController(), uno::UNO_QUERY);
+xCtrl-select(uno::makeAny(xTable1));
+uno::Referencetext::XTextViewCursorSupplier 
xTextViewCursorSupplier(xCtrl, uno::UNO_QUERY);
+uno::Referencetext::XTextViewCursor 
xCursor(xTextViewCursorSupplier-getViewCursor(), uno::UNO_QUERY);
+awt::Point pos = xCursor-getPosition();
+CPPUNIT_ASSERT_DOUBLES_EQUAL_MESSAGE(Incorrect X coord computed from 
docx,
+xCoordsFromOffice[i], pos.X, 1);
+
+// Verify left margin of 1st cell :
+//  * Office left margins are measured relative to the right of the 
border
+//  * LO left spacing is measured from the center of the border
+uno::Referencetable::XCell xCell = xTable1-getCellByName(A1);
+uno::Reference beans::XPropertySet  xPropSet(xCell, 
uno::UNO_QUERY_THROW);
+sal_Int32 aLeftMargin = -1;
+xPropSet-getPropertyValue(LeftBorderDistance) = aLeftMargin;
+uno::Any aLeftBorder = xPropSet-getPropertyValue(LeftBorder);
+table::BorderLine2 aLeftBorderLine;
+aLeftBorder = aLeftBorderLine;
+CPPUNIT_ASSERT_DOUBLES_EQUAL_MESSAGE(Incorrect left spacing computed 
from docx cell margin,
+cellLeftMarginFromOffice[i], aLeftMargin - 0.5 * 
aLeftBorderLine.LineWidth, 1);
+}
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/filter/ww8/docxattributeoutput.cxx |   11 +--
 1 file changed, 5 insertions(+), 6 deletions(-)

New commits:
commit 559a1a5d28d94e915150f94d5267a4f720d175b0
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Fri Sep 28 09:58:44 2012 +0200

docx export: always write tblInd attribute (even when equal to 0)

Otherwise when opening the docx in Office table may get misplaced.

Change-Id: Ic1df8d941c72a224d2db189ededfc5e134e5683a
Reviewed-on: https://gerrit.libreoffice.org/718
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index cf6f43e..bce14c8 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -1813,12 +1813,11 @@ void DocxAttributeOutput::TableDefinition( 
ww8::WW8TableNodeInfoInner::Pointer_t
 
 TableBidi( pTableTextNodeInfoInner );
 
-// Table indent
-if ( nIndent != 0 )
-m_pSerializer-singleElementNS( XML_w, XML_tblInd,
-FSNS( XML_w, XML_w ), OString::valueOf( nIndent ).getStr( ),
-FSNS( XML_w, XML_type ), dxa,
-FSEND );
+// Table indent (need to get written even if == 0)
+m_pSerializer-singleElementNS( XML_w, XML_tblInd,
+FSNS( XML_w, XML_w ), OString::valueOf( nIndent ).getStr( ),
+FSNS( XML_w, XML_type ), dxa,
+FSEND );
 
 // Merge the marks for the ordered elements
 m_pSerializer-mergeTopMarks( );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] docx export: add unit test for 9751056

2012-10-02 Thread Miklos Vajna (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/717


--
To view, visit https://gerrit.libreoffice.org/717
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I25aa07d3e1811fdf4ab0124e14794e70d7de82ec
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PUSHED] docx export: always write tblInd attribute (even when equal ...

2012-10-02 Thread Miklos Vajna (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/718

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/718
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1df8d941c72a224d2db189ededfc5e134e5683a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 5 commits - lingucomponent/source oox/source oox/util pyuno/source remotebridges/source scripting/source scripting/util

2012-10-02 Thread Libreoffice Gerrit user
 lingucomponent/source/spellcheck/macosxspell/MacOSXSpell.component |2 +-
 lingucomponent/source/spellcheck/macosxspell/macreg.mm |2 +-
 oox/source/core/services.cxx   |2 +-
 oox/util/oox.component |2 +-
 pyuno/source/loader/pythonloader.component |2 +-
 pyuno/source/loader/pyuno_loader.cxx   |2 +-
 remotebridges/source/unourl_resolver/unourl_resolver.cxx   |4 ++--
 remotebridges/source/unourl_resolver/uuresolver.component  |2 +-
 scripting/source/basprov/basprov.component |2 +-
 scripting/source/basprov/basprov.cxx   |2 +-
 scripting/source/dlgprov/dlgprov.component |2 +-
 scripting/source/dlgprov/dlgprov.cxx   |2 +-
 scripting/source/protocolhandler/protocolhandler.component |2 +-
 scripting/source/protocolhandler/scripthandler.cxx |2 +-
 scripting/source/provider/MasterScriptProvider.cxx |2 +-
 scripting/source/stringresource/stringresource.component   |2 +-
 scripting/source/stringresource/stringresource.cxx |2 +-
 scripting/source/vbaevents/service.cxx |2 +-
 scripting/source/vbaevents/vbaevents.component |2 +-
 scripting/util/scriptframe.component   |2 +-
 20 files changed, 21 insertions(+), 21 deletions(-)

New commits:
commit 671746f57cd3337ba49a0be512080dd5b69e3d40
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 11:45:48 2012 +0300

Use prefix

Change-Id: Id1ac9a800b58595771be22097d77d3a7c5a8862c

diff --git a/oox/source/core/services.cxx b/oox/source/core/services.cxx
index 16d1e5f..aec6559 100644
--- a/oox/source/core/services.cxx
+++ b/oox/source/core/services.cxx
@@ -65,7 +65,7 @@ static ::cppu::ImplementationEntry const spServices[] =
 
 } // namespace
 
-extern C SAL_DLLPUBLIC_EXPORT void* SAL_CALL component_getFactory( const 
char* pImplName, void* pServiceManager, void* pRegistryKey )
+extern C SAL_DLLPUBLIC_EXPORT void* SAL_CALL oox_component_getFactory( const 
char* pImplName, void* pServiceManager, void* pRegistryKey )
 {
 return ::cppu::component_getFactoryHelper( pImplName, pServiceManager, 
pRegistryKey, spServices );
 }
diff --git a/oox/util/oox.component b/oox/util/oox.component
index 0c3157f..54f4655 100644
--- a/oox/util/oox.component
+++ b/oox/util/oox.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=oox
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.oox.core.FastTokenHandler
 service name=com.sun.star.xml.sax.FastTokenHandler/
commit 1b075ea2cbeef67bf556cade80f2fcf8130be571
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 11:45:11 2012 +0300

Use prefix

Change-Id: I0323e3eb9446c4f2b18f84f5ed33909a877163c8

diff --git a/scripting/source/basprov/basprov.component 
b/scripting/source/basprov/basprov.component
index 5268cfe..09ed542 100644
--- a/scripting/source/basprov/basprov.component
+++ b/scripting/source/basprov/basprov.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=basprov
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.scripting.ScriptProviderForBasic
 service name=com.sun.star.script.browse.BrowseNode/
diff --git a/scripting/source/basprov/basprov.cxx 
b/scripting/source/basprov/basprov.cxx
index 06675e8..bf4a449 100644
--- a/scripting/source/basprov/basprov.cxx
+++ b/scripting/source/basprov/basprov.cxx
@@ -586,7 +586,7 @@ namespace basprov
 
 extern C
 {
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL basprov_component_getFactory(
 const sal_Char * pImplName, lang::XMultiServiceFactory * 
pServiceManager,
 registry::XRegistryKey * pRegistryKey )
 {
diff --git a/scripting/source/dlgprov/dlgprov.component 
b/scripting/source/dlgprov/dlgprov.component
index dffc48e..66dd111 100644
--- a/scripting/source/dlgprov/dlgprov.component
+++ b/scripting/source/dlgprov/dlgprov.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=dlgprov
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.scripting.DialogProvider
 service name=com.sun.star.awt.ContainerWindowProvider/
diff 

Re: [PUSHED] [PATCH 2/3] fdo#51304: Remove @author annotation and commented code

2012-10-02 Thread Stephan Bergmann

On 10/01/2012 08:03 PM, Thorsten Behrens wrote:

I also did not apply your printStackTrace() logging calls, I think
at least some of them report on exceptions that were seemingly
previously ignored on purpose?

Stephan, or anyone else more up-to-date with contemporary java
idiom, any opinions here?


I have not much of an opinion there.  The wizards code being what it is, 
it is probably tedious and pointless to try and determine whether 
individual exception-swallowing catch-blocks were really meant that way 
or were just incompetent coding.  So I'd personally be fine both with 
keeping the blocks empty and with adding printStackTrace calls wholesale.


(Just one nit, System.err is the default target for printStackTrace, so 
it can be omitted from calls.)


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: filter/Library_PptImporter.mk filter/source

2012-10-02 Thread Libreoffice Gerrit user
 filter/Library_PptImporter.mk|1 
 filter/source/msfilter/powerpoint/pptimporteruno.cxx |   63 +++
 2 files changed, 64 insertions(+)

New commits:
commit 1a28e705322463f2ed25acace19c8edc33466eeb
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Tue Oct 2 12:04:00 2012 +0300

Revert As far as I can see the PptImporter library is not a UNO component

I don't trust myself.

This reverts commit 2726f7f4ca5cde91565203ef055561558ea66e7d.

diff --git a/filter/Library_PptImporter.mk b/filter/Library_PptImporter.mk
index 8fee1bd..f1d172a 100644
--- a/filter/Library_PptImporter.mk
+++ b/filter/Library_PptImporter.mk
@@ -35,6 +35,7 @@ $(eval $(call gb_Library_use_libraries,PptImporter,\
 
 $(eval $(call gb_Library_add_exception_objects,PptImporter,\
filter/source/msfilter/powerpoint/pptimporter \
+   filter/source/msfilter/powerpoint/pptimporteruno \
filter/source/msfilter/powerpoint/ppttoxml \
 ))
 
diff --git a/filter/source/msfilter/powerpoint/pptimporteruno.cxx 
b/filter/source/msfilter/powerpoint/pptimporteruno.cxx
new file mode 100644
index 000..be0b49e
--- /dev/null
+++ b/filter/source/msfilter/powerpoint/pptimporteruno.cxx
@@ -0,0 +1,63 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ *
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * Copyright 2000, 2010 Oracle and/or its affiliates.
+ *
+ * OpenOffice.org - a multi-platform office productivity suite
+ *
+ * This file is part of OpenOffice.org.
+ *
+ * OpenOffice.org is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License version 3
+ * only, as published by the Free Software Foundation.
+ *
+ * OpenOffice.org is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License version 3 for more details
+ * (a copy is included in the LICENSE file that accompanied this code).
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * version 3 along with OpenOffice.org.  If not, see
+ * http://www.openoffice.org/license.html
+ * for a copy of the LGPLv3 License.
+ *
+ /
+
+
+#include pptimporter.hxx
+
+#include cppuhelper/factory.hxx
+#include uno/mapping.hxx
+
+static REF( NMSP_UNO::XInterface ) SAL_CALL create_PptImporter( const REF( 
NMSP_LANG::XMultiServiceFactory ) rxFact )
+{
+return REF( NMSP_UNO::XInterface )( *new PptImporter( rxFact ) );
+}
+
+extern C SAL_DLLPUBLIC_EXPORT void* SAL_CALL component_getFactory( const 
sal_Char* pImplName, void* pServiceManager, void* /* pRegistryKey */ )
+{
+REF( NMSP_LANG::XSingleServiceFactory ) xFactory;
+void*   pRet = 0;
+
+if( rtl_str_compare( pImplName, com.sun.star.presentation.PptImporter ) 
== 0 )
+{
+const NMSP_RTL::OUString aServiceName( B2UCONST( 
com.sun.star.presentation.PptImporter ) );
+
+xFactory = REF( NMSP_LANG::XSingleServiceFactory )( 
NMSP_CPPU::createSingleFactory(
+reinterpret_cast NMSP_LANG::XMultiServiceFactory* ( 
pServiceManager ),
+B2UCONST( com.sun.star.presentation.PptImporter ),
+create_PptImporter, SEQ( NMSP_RTL::OUString )( 
aServiceName, 1 ) ) );
+}
+if( xFactory.is() )
+{
+xFactory-acquire();
+pRet = xFactory.get();
+}
+
+return pRet;
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - sdext/source shell/source

2012-10-02 Thread Libreoffice Gerrit user
 sdext/source/minimizer/minimizer.component   |3 ++-
 sdext/source/minimizer/pppoptimizeruno.cxx   |2 +-
 sdext/source/pdfimport/pdfimport.component   |3 ++-
 sdext/source/pdfimport/services.cxx  |2 +-
 sdext/source/presenter/PresenterComponent.cxx|4 ++--
 sdext/source/presenter/presenter.component   |3 ++-
 shell/source/backends/gconfbe/gconfbackend.cxx   |2 +-
 shell/source/backends/gconfbe/gconfbe1.component |2 +-
 shell/source/backends/kde4be/kde4backend.cxx |2 +-
 shell/source/backends/kde4be/kde4be1.component   |2 +-
 shell/source/backends/kdebe/kdebackend.cxx   |2 +-
 shell/source/backends/kdebe/kdebe1.component |2 +-
 shell/source/backends/macbe/macbe1.component |2 +-
 shell/source/backends/macbe/macbecdef.cxx|2 +-
 shell/source/backends/wininetbe/wininetbe1.component |2 +-
 shell/source/backends/wininetbe/wininetbecdef.cxx|2 +-
 shell/source/cmdmail/cmdmail.component   |2 +-
 shell/source/cmdmail/cmdmailentry.cxx|2 +-
 shell/source/unix/exec/shellexecentry.cxx|2 +-
 shell/source/unix/exec/syssh.component   |2 +-
 shell/source/win32/SysShentry.cxx|2 +-
 shell/source/win32/simplemail/smplmail.component |2 +-
 shell/source/win32/simplemail/smplmailentry.cxx  |2 +-
 shell/source/win32/syssh.component   |2 +-
 24 files changed, 28 insertions(+), 25 deletions(-)

New commits:
commit 565acfef761e37a0172154aae0bdace1ce3c
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 12:04:10 2012 +0300

Use prefix

Change-Id: Ife93e37f8b24f3ce96b55c5a221f0f929b789f60

diff --git a/shell/source/backends/gconfbe/gconfbackend.cxx 
b/shell/source/backends/gconfbe/gconfbackend.cxx
index 82ea9fe..263fa7d 100644
--- a/shell/source/backends/gconfbe/gconfbackend.cxx
+++ b/shell/source/backends/gconfbe/gconfbackend.cxx
@@ -217,7 +217,7 @@ static cppu::ImplementationEntry const services[] = {
 
 }
 
-extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL gconfbe1_component_getFactory(
 char const * pImplName, void * pServiceManager, void * pRegistryKey)
 {
 return cppu::component_getFactoryHelper(
diff --git a/shell/source/backends/gconfbe/gconfbe1.component 
b/shell/source/backends/gconfbe/gconfbe1.component
index 7638e05..b48806c 100644
--- a/shell/source/backends/gconfbe/gconfbe1.component
+++ b/shell/source/backends/gconfbe/gconfbe1.component
@@ -26,7 +26,7 @@
 *
 **--
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=gconfbe1
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.configuration.backend.GconfBackend
 service name=com.sun.star.configuration.backend.GconfBackend/
diff --git a/shell/source/backends/kde4be/kde4backend.cxx 
b/shell/source/backends/kde4be/kde4backend.cxx
index 1c87c48..314cd5a 100644
--- a/shell/source/backends/kde4be/kde4backend.cxx
+++ b/shell/source/backends/kde4be/kde4backend.cxx
@@ -215,7 +215,7 @@ static cppu::ImplementationEntry const services[] = {
 
 }
 
-extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL kde4be1_component_getFactory(
 char const * pImplName, void * pServiceManager, void * pRegistryKey)
 {
 return cppu::component_getFactoryHelper(
diff --git a/shell/source/backends/kde4be/kde4be1.component 
b/shell/source/backends/kde4be/kde4be1.component
index 8ea7ae1..9bceccc 100644
--- a/shell/source/backends/kde4be/kde4be1.component
+++ b/shell/source/backends/kde4be/kde4be1.component
@@ -26,7 +26,7 @@
 *
 **--
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=kde4be1
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.configuration.backend.KDE4Backend
 service name=com.sun.star.configuration.backend.KDE4Backend/
diff --git a/shell/source/backends/kdebe/kdebackend.cxx 
b/shell/source/backends/kdebe/kdebackend.cxx
index 4faf911..ad82efc 100644
--- a/shell/source/backends/kdebe/kdebackend.cxx
+++ b/shell/source/backends/kdebe/kdebackend.cxx
@@ -215,7 +215,7 @@ static cppu::ImplementationEntry const services[] = {
 
 }
 
-extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL kdebe1_component_getFactory(
 char const * pImplName, void * pServiceManager, void * pRegistryKey)
 {
 return cppu::component_getFactoryHelper(
diff --git a/shell/source/backends/kdebe/kdebe1.component 

[Libreoffice-commits] .: 2 commits - cpputools/source stoc/source

2012-10-02 Thread Libreoffice Gerrit user
 cpputools/source/registercomponent/registercomponent.cxx |6 ++-
 cpputools/source/unoexe/unoexe.cxx   |   28 ++-
 stoc/source/javaloader/javaloader.component  |2 -
 stoc/source/javaloader/javaloader.cxx|2 -
 stoc/source/javavm/javavm.component  |2 -
 stoc/source/javavm/javavm.cxx|2 -
 6 files changed, 20 insertions(+), 22 deletions(-)

New commits:
commit d50f3c5922411304b4291953a4cc9f07f4730892
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Oct 2 11:01:23 2012 +0200

stoc: prefix more libs; only bootstrap is remaining

Change-Id: Ie9032c6faa493da47e76bc9ff467c4de4b318ca9

diff --git a/cpputools/source/registercomponent/registercomponent.cxx 
b/cpputools/source/registercomponent/registercomponent.cxx
index 06c38f5..950952b 100644
--- a/cpputools/source/registercomponent/registercomponent.cxx
+++ b/cpputools/source/registercomponent/registercomponent.cxx
@@ -722,13 +722,15 @@ static void bootstrap(
 OUString(),
 
OUString(RTL_CONSTASCII_USTRINGPARAM(com.sun.star.comp.stoc.JavaVirtualMachine)),
 xSMgr,
-Reference XRegistryKey  () );
+Reference XRegistryKey  (),
+javavm );
 Reference XInterface  r2 = loadSharedLibComponentFactory(
 OUString(RTL_CONSTASCII_USTRINGPARAM(javaloader.uno 
SAL_DLLEXTENSION)),
 OUString(),
 
OUString(RTL_CONSTASCII_USTRINGPARAM(com.sun.star.comp.stoc.JavaComponentLoader)),
 xSMgr,
-Reference XRegistryKey  () );
+Reference XRegistryKey  (),
+javaloader );
 Reference XSet xSet( xSMgr, UNO_QUERY );
 if( r.is()  r2.is()  xSet.is() )
 {
diff --git a/stoc/source/javaloader/javaloader.component 
b/stoc/source/javaloader/javaloader.component
index bb2ea0f..5fd7d86 100644
--- a/stoc/source/javaloader/javaloader.component
+++ b/stoc/source/javaloader/javaloader.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=javaloader
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.stoc.JavaComponentLoader
 service name=com.sun.star.loader.Java/
diff --git a/stoc/source/javaloader/javaloader.cxx 
b/stoc/source/javaloader/javaloader.cxx
index 7bfe440..cb48b6c 100644
--- a/stoc/source/javaloader/javaloader.cxx
+++ b/stoc/source/javaloader/javaloader.cxx
@@ -435,7 +435,7 @@ static struct ImplementationEntry g_entries[] =
 
 extern C
 {
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL javaloader_component_getFactory(
 const sal_Char * pImplName, void * pServiceManager, void * pRegistryKey )
 {
 return component_getFactoryHelper( pImplName, pServiceManager, 
pRegistryKey , g_entries );
diff --git a/stoc/source/javavm/javavm.component 
b/stoc/source/javavm/javavm.component
index 1394b5b..570ce4c 100644
--- a/stoc/source/javavm/javavm.component
+++ b/stoc/source/javavm/javavm.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=javavm
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.stoc.JavaVirtualMachine
 service name=com.sun.star.java.JavaVirtualMachine/
diff --git a/stoc/source/javavm/javavm.cxx b/stoc/source/javavm/javavm.cxx
index fbb1c89..4667fee 100644
--- a/stoc/source/javavm/javavm.cxx
+++ b/stoc/source/javavm/javavm.cxx
@@ -598,7 +598,7 @@ private:
 
 }
 
-extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(sal_Char 
const * pImplName,
+extern C SAL_DLLPUBLIC_EXPORT void * SAL_CALL 
javavm_component_getFactory(sal_Char const * pImplName,
 void * pServiceManager,
 void * pRegistryKey)
 {
commit 95630ef04b9df092653fb2f68d91a7e6100f7a4e
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Oct 2 10:57:38 2012 +0200

these libs are now prefixed

Change-Id: Ida3cc87ef2ef6fdb40b03d0401fbefa5e7cd7c92

diff --git a/cpputools/source/unoexe/unoexe.cxx 
b/cpputools/source/unoexe/unoexe.cxx
index d9a92e3..db3e63b 100644
--- a/cpputools/source/unoexe/unoexe.cxx
+++ b/cpputools/source/unoexe/unoexe.cxx
@@ -257,29 +257,25 @@ void createInstance(
 Reference XMultiServiceFactory  xSF( xMgr, UNO_QUERY );
 // acceptor
 xSet-insert( makeAny( loadSharedLibComponentFactory(
-OUString( RTL_CONSTASCII_USTRINGPARAM(
-  acceptor.uno SAL_DLLEXTENSION) ),
+

Error while building LO on Windows

2012-10-02 Thread Ulrich Kitzinger

Hello,

I am trying to build Libre Office on Windows, as described in 
http://wiki.documentfoundation.org/Development/BuildingOnWindowsWithCygwinAndMSVCExpress. 
During instsetoo_native I get the error:


: ##
: Packaging process
: ##
: Performance Info: Execute packaging startMon Oct  1 15:28:24 2012 
(02:21 min.)
: chdir: 
E:/LibreOfficeBuild/libreoffice-lhm/core/instsetoo_native/wntmsci12.pro/LibreOffice/native/logging/en-US_de/

: chdir: /tmp/ooopackaging/i_2001349097963/wntmsci12.pro
: Systemcall: makecab.exe /V3 /F 
E:/LibreOfficeBuild/libreoffice-lhm/core/instsetoo_native/wntmsci12.pro/LibreOffice/native/ddf/en-US_de/libreoffice1.ddf 
21 |
: makecab_error (Try 1): makecab.exe /V3 /F 
E:/LibreOfficeBuild/libreoffice-lhm/core/instsetoo_native/wntmsci12.pro/LibreOffice/native/ddf/en-US_de/libreoffice1.ddf 
21 |

:  !makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp


I suppose this is because the ddf-file libreoffice1.ddf contains the lines:

LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\caption\com.sun.PresenterScreen-windows_x86%2Fpresenter.xhp 
com2.sun.PresenterScreen_windows_x86%2Fpresenter.xhp
LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\content\com.sun.PresenterScreen-windows_x86%2Fpresenter.xhp 
com3.sun.PresenterScreen_windows_x86%2Fpresenter.xhp


But only for that file, presenter.xhp. All other entries are without 
%2F, for example:



LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\help.db_ 
help1.db_
LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\help.ht_ 
help1.ht_


Any hints, what's wrong?

Many thanks,
Ulrich
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Bug Confirmation Triage Event (Doodle)

2012-10-02 Thread Joel Madero
Hi All,

Trying to find a time that works for as many people as possible, 1 hour
event, doodle can be found here:

http://doodle.com/3tfr888ma5cgkww4

If another time works better, propose it and we'll see what we can get to.
I can't go any earlier than the time listed on Weekdays (that's 5am here).

Regards,
Joel
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


RE: OUString is mutable?

2012-10-02 Thread Dennis E. Hamilton
In the WinRT APIs (not to be confused with the Windows RT on Atom) introduced 
with Windows 8, strings are immutable.  I suspect that hardware features may be 
exploited to ensure that they stay that way.  

I haven't got my head around how that works with the still-supported BSTR at 
the native level.  There can be some simple testing to see if access violations 
are triggered even when the data structure is accessed internally via native 
code.

In addition to the considerations already listed, there are also security 
considerations applicable to strings, having to do with their use in native 
code exploits to access other heap or code storage by addressing beyond or 
before the string location.  I suppose that genuinely-immutable strings might 
provide some safeguard against exploits of that nature.

Regarding the mention that the latest Java VM is using UTF8 internally instead 
of unsigned short arrays is rather daunting.  There is an easy way to test it 
-- see if char values that are not admissible UTF16 codes can be used in 
construction of a string and then extracted correctly.  If they can, there is 
no way that transformation to and from UTF8 occurred.  If they can't, it is an 
interesting breaking change in Java.  With regard to string literals, it would 
be interesting to see what can be introduced into those via escape codes too.
 
 - Dennis

-Original Message-
From: libreoffice-bounces+dennis.hamilton=acm@lists.freedesktop.org 
[mailto:libreoffice-bounces+dennis.hamilton=acm@lists.freedesktop.org] On 
Behalf Of Michael Stahl
Sent: Monday, October 01, 2012 06:51
To: libreoffice@lists.freedesktop.org
Subject: Re: OUString is mutable?

[ ... ]

it appears that there are people who do see good reasons for immutable
strings :)

[ ... ]

from what i can see the advantages of immutable strings include:

- somebody reading the code can rely on the fact that the buffer in the
  string is not mutated
- immutable data types are much less error prone as hashtable/map keys
  (you don't want to modify a key after it has been inserted, because
  that is practically guaranteed to violate container's invariants)
- the buffer can be shared across threads (but not the string wrapper
  itself, which makes this .. less of an advantage)
- there are potential space and allocation savings with sharing
  sub-string representations (though OUString doesn't do that)
- it allows for caching the hash value, though OUString doesn't do that
  (and i don't know if space overhead is worth it...)

[ ... ]

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: writerfilter/source

2012-10-02 Thread Libreoffice Gerrit user
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 12c220ae8e55a033ab6aeca27c0e26a643d2001a
Author: Miklos Vajna vmik...@suse.cz
Date:   Tue Oct 2 12:25:41 2012 +0300

Make Clang happy

Change-Id: I69c886d0a1875ec674f40c2422dc1490418c1da3

diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index 2690105..f8786c9 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -393,8 +393,8 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 
 // Set table above/bottom spacing to 0.
 // TODO: handle 'Around' text wrapping mode
-m_aTableProperties-Insert( PropertyIds::PROP_TOP_MARGIN, true, 
uno::makeAny( 0 ) );
-m_aTableProperties-Insert( PropertyIds::PROP_BOTTOM_MARGIN, true, 
uno::makeAny( 0 ) );
+m_aTableProperties-Insert( PROP_TOP_MARGIN, true, uno::makeAny( 
sal_Int32( 0 ) ) );
+m_aTableProperties-Insert( PROP_BOTTOM_MARGIN, true, uno::makeAny( 
sal_Int32( 0 ) ) );
 
 //table border settings
 table::TableBorder aTableBorder;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: filter/Library_PptImporter.mk filter/Module_filter.mk filter/source Repository.mk

2012-10-02 Thread Libreoffice Gerrit user
 Repository.mk|1 
 filter/Library_PptImporter.mk|   42 -
 filter/Module_filter.mk  |1 
 filter/source/msfilter/powerpoint/pptcom.hxx |   62 ---
 filter/source/msfilter/powerpoint/pptimporter.cxx|  148 ---
 filter/source/msfilter/powerpoint/pptimporter.hxx|   85 --
 filter/source/msfilter/powerpoint/pptimporteruno.cxx |   63 
 filter/source/msfilter/powerpoint/ppttoxml.cxx   |   83 --
 filter/source/msfilter/powerpoint/ppttoxml.hxx   |   41 -
 9 files changed, 526 deletions(-)

New commits:
commit 4ea8ddb05b3e5f65ee8cf73ae87500d066c13ea5
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Oct 2 11:29:48 2012 +0200

PptImporter is unused; it was also missing in scp2

Change-Id: I87c8d46d791525e3e5b6866edc6423e92cbcca81

diff --git a/Repository.mk b/Repository.mk
index fdc4ce6..a4daa93 100644
--- a/Repository.mk
+++ b/Repository.mk
@@ -217,7 +217,6 @@ $(eval $(call gb_Helper_register_libraries,PLAINLIBS_NONE, \
 endif
 
 $(eval $(call gb_Helper_register_libraries,OOOLIBS, \
-PptImporter \
 abp \
 acc \
adabas \
diff --git a/filter/Library_PptImporter.mk b/filter/Library_PptImporter.mk
deleted file mode 100644
index f1d172a..000
--- a/filter/Library_PptImporter.mk
+++ /dev/null
@@ -1,42 +0,0 @@
-#
-# This file is part of the LibreOffice project.
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-#
-# This file incorporates work covered by the following license notice:
-#
-#   Licensed to the Apache Software Foundation (ASF) under one or more
-#   contributor license agreements. See the NOTICE file distributed
-#   with this work for additional information regarding copyright
-#   ownership. The ASF licenses this file to you under the Apache
-#   License, Version 2.0 (the License); you may not use this file
-#   except in compliance with the License. You may obtain a copy of
-#   the License at http://www.apache.org/licenses/LICENSE-2.0 .
-#
-
-$(eval $(call gb_Library_Library,PptImporter))
-
-$(eval $(call gb_Library_use_sdk_api,PptImporter))
-
-$(eval $(call gb_Library_set_include,PptImporter,\
-   $$(INCLUDE) \
-))
-
-$(eval $(call gb_Library_use_libraries,PptImporter,\
-   sot \
-   tl \
-   cppuhelper \
-   cppu \
-   sal \
-   $(gb_UWINAPI) \
-))
-
-$(eval $(call gb_Library_add_exception_objects,PptImporter,\
-   filter/source/msfilter/powerpoint/pptimporter \
-   filter/source/msfilter/powerpoint/pptimporteruno \
-   filter/source/msfilter/powerpoint/ppttoxml \
-))
-
-# vim: set noet sw=4 ts=4:
diff --git a/filter/Module_filter.mk b/filter/Module_filter.mk
index b3ec2a3..99bebb5 100644
--- a/filter/Module_filter.mk
+++ b/filter/Module_filter.mk
@@ -26,7 +26,6 @@ $(eval $(call gb_Module_add_targets,filter,\
AllLangResTarget_xsltdlg \
Configuration_filter \
CustomTarget_svg \
-   Library_PptImporter \
Library_egi \
Library_eme \
Library_epb \
diff --git a/filter/source/msfilter/powerpoint/pptcom.hxx 
b/filter/source/msfilter/powerpoint/pptcom.hxx
deleted file mode 100644
index f25e6ad..000
--- a/filter/source/msfilter/powerpoint/pptcom.hxx
+++ /dev/null
@@ -1,62 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- *
- * This file incorporates work covered by the following license notice:
- *
- *   Licensed to the Apache Software Foundation (ASF) under one or more
- *   contributor license agreements. See the NOTICE file distributed
- *   with this work for additional information regarding copyright
- *   ownership. The ASF licenses this file to you under the Apache
- *   License, Version 2.0 (the License); you may not use this file
- *   except in compliance with the License. You may obtain a copy of
- *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-#ifndef _PPTCOM_HXX
-#define _PPTCOM_HXX
-
-#include rtl/ustring.hxx
-
-#include com/sun/star/uno/Reference.h
-#include com/sun/star/uno/RuntimeException.hpp
-#include com/sun/star/lang/XMultiServiceFactory.hpp
-#include com/sun/star/lang/XComponent.hpp
-#include com/sun/star/registry/XRegistryKey.hpp
-#include com/sun/star/io/XInputStream.hpp
-#include com/sun/star/xml/sax/XDocumentHandler.hpp
-#include xmloff/xmlexp.hxx
-#include xmloff/nmspmap.hxx
-#include cppuhelper/implbase1.hxx
-#include cppuhelper/implbase4.hxx
-#include com/sun/star/document/XFilter.hpp
-#include 

[Libreoffice-commits] .: sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/ui/app/applab.cxx |   77 +---
 1 file changed, 23 insertions(+), 54 deletions(-)

New commits:
commit 2f3b4edc414b30a149db336ef1d3d8bf716698f0
Author: Winfried Donkers o...@dci-electronics.nl
Date:   Fri Sep 28 12:15:11 2012 +0200

fdo#52673 fix for layout problems with version 3.6 and up

Change-Id: I626ec5bce931005dccbc8f4d235e8208ae98f9a5

diff --git a/sw/source/ui/app/applab.cxx b/sw/source/ui/app/applab.cxx
index 55989b8..1ed815e 100644
--- a/sw/source/ui/app/applab.cxx
+++ b/sw/source/ui/app/applab.cxx
@@ -91,21 +91,19 @@ const char MASTER_LABEL[] = MasterLabel;
 
 const SwFrmFmt *lcl_InsertBCText( SwWrtShell rSh, const SwLabItem rItem,
 SwFrmFmt rFmt,
-sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bPage)
+sal_uInt16 nCol, sal_uInt16 nRow )
 {
 SfxItemSet aSet(rSh.GetAttrPool(), RES_ANCHOR, RES_ANCHOR,
 RES_VERT_ORIENT, RES_VERT_ORIENT, RES_HORI_ORIENT, 
RES_HORI_ORIENT, 0 );
 sal_uInt16 nPhyPageNum, nVirtPageNum;
 rSh.GetPageNum( nPhyPageNum, nVirtPageNum );
 
-aSet.Put(SwFmtAnchor(bPage ? FLY_AS_CHAR : FLY_AT_PAGE, nPhyPageNum));
-if (!bPage)
-{
-aSet.Put(SwFmtHoriOrient(rItem.lLeft + nCol * rItem.lHDist,
-
text::HoriOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-aSet.Put(SwFmtVertOrient(rItem.lUpper + nRow * rItem.lVDist,
-
text::VertOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-}
+//anchor frame to page
+aSet.Put( SwFmtAnchor( FLY_AT_PAGE, nPhyPageNum ) );
+aSet.Put( SwFmtHoriOrient( rItem.lLeft + nCol * rItem.lHDist,
+   text::HoriOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
+aSet.Put( SwFmtVertOrient( rItem.lUpper + nRow * rItem.lVDist,
+   text::VertOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
 const SwFrmFmt *pFmt = rSh.NewFlyFrm(aSet, sal_True,  rFmt );  // Insert 
Fly
 OSL_ENSURE( pFmt, Fly not inserted );
 
@@ -130,21 +128,19 @@ const SwFrmFmt *lcl_InsertBCText( SwWrtShell rSh, const 
SwLabItem rItem,
 
 const SwFrmFmt *lcl_InsertLabText( SwWrtShell rSh, const SwLabItem rItem,
 SwFrmFmt rFmt, SwFldMgr rFldMgr,
-sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bLast, 
sal_Bool bPage)
+sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bLast )
 {
 SfxItemSet aSet(rSh.GetAttrPool(), RES_ANCHOR, RES_ANCHOR,
 RES_VERT_ORIENT, RES_VERT_ORIENT, RES_HORI_ORIENT, 
RES_HORI_ORIENT, 0 );
 sal_uInt16 nPhyPageNum, nVirtPageNum;
 rSh.GetPageNum( nPhyPageNum, nVirtPageNum );
 
-aSet.Put(SwFmtAnchor(bPage ? FLY_AS_CHAR : FLY_AT_PAGE, nPhyPageNum));
-if (!bPage)
-{
-aSet.Put(SwFmtHoriOrient(rItem.lLeft + nCol * rItem.lHDist,
-
text::HoriOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-aSet.Put(SwFmtVertOrient(rItem.lUpper + nRow * rItem.lVDist,
-
text::VertOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-}
+//anchor frame to page
+aSet.Put( SwFmtAnchor( FLY_AT_PAGE, nPhyPageNum ) );
+aSet.Put( SwFmtHoriOrient( rItem.lLeft + nCol * rItem.lHDist,
+   text::HoriOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
+aSet.Put( SwFmtVertOrient( rItem.lUpper + nRow * rItem.lVDist,
+   text::VertOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
 const SwFrmFmt *pFmt = rSh.NewFlyFrm(aSet, sal_True,  rFmt );  // Insert 
Fly
 OSL_ENSURE( pFmt, Fly not inserted );
 
@@ -300,26 +296,11 @@ void SwModule::InsertLab(SfxRequest rReq, sal_Bool 
bLabel)
 SwFmtFrmSize aFrmSize(  ATT_FIX_SIZE, iWidth, iHeight );
 pFmt-SetFmtAttr( aFrmSize );
 
-SwFrmFmt* pFmtEORow = new SwFrmFmt (*pFmt);
-SwFrmFmt* pFmtEOCol = new SwFrmFmt (*pFmt);
-SwFrmFmt* pFmtEOColEORow = new SwFrmFmt (*pFmt);
-
-SvxULSpaceItem aFrmULSpace( 0, (sal_uInt16)(rItem.lVDist - 
rItem.lHeight),
-RES_UL_SPACE );
+//frame represents label itself, no border space
 SvxULSpaceItem aFrmNoULSpace( 0, 0, RES_UL_SPACE );
-
-SvxLRSpaceItem aFrmLRSpace( 0, (sal_uInt16)(rItem.lHDist - 
rItem.lWidth),
-0, 0, RES_LR_SPACE );
 SvxLRSpaceItem aFrmNoLRSpace( 0, 0, 0, 0, RES_LR_SPACE );
-
-pFmt-SetFmtAttr(aFrmULSpace);
-pFmt-SetFmtAttr(aFrmLRSpace);
-pFmtEORow-SetFmtAttr(aFrmULSpace);
-pFmtEORow-SetFmtAttr(aFrmNoLRSpace);
-pFmtEOCol-SetFmtAttr(aFrmNoULSpace);
-   

[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/ui/app/applab.cxx |   77 +---
 1 file changed, 23 insertions(+), 54 deletions(-)

New commits:
commit 4bbcf581ad7a478387491058b38e6dd8a344af44
Author: Winfried Donkers o...@dci-electronics.nl
Date:   Fri Sep 28 12:15:11 2012 +0200

fdo#53673 fix for layout problems with version 3.6 and up

Change-Id: I626ec5bce931005dccbc8f4d235e8208ae98f9a5

diff --git a/sw/source/ui/app/applab.cxx b/sw/source/ui/app/applab.cxx
index 4159f18..fe2dd11 100644
--- a/sw/source/ui/app/applab.cxx
+++ b/sw/source/ui/app/applab.cxx
@@ -91,21 +91,19 @@ const char MASTER_LABEL[] = MasterLabel;
 
 const SwFrmFmt *lcl_InsertBCText( SwWrtShell rSh, const SwLabItem rItem,
 SwFrmFmt rFmt,
-sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bPage)
+sal_uInt16 nCol, sal_uInt16 nRow )
 {
 SfxItemSet aSet(rSh.GetAttrPool(), RES_ANCHOR, RES_ANCHOR,
 RES_VERT_ORIENT, RES_VERT_ORIENT, RES_HORI_ORIENT, 
RES_HORI_ORIENT, 0 );
 sal_uInt16 nPhyPageNum, nVirtPageNum;
 rSh.GetPageNum( nPhyPageNum, nVirtPageNum );
 
-aSet.Put(SwFmtAnchor(bPage ? FLY_AS_CHAR : FLY_AT_PAGE, nPhyPageNum));
-if (!bPage)
-{
-aSet.Put(SwFmtHoriOrient(rItem.lLeft + nCol * rItem.lHDist,
-
text::HoriOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-aSet.Put(SwFmtVertOrient(rItem.lUpper + nRow * rItem.lVDist,
-
text::VertOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-}
+//anchor frame to page
+aSet.Put( SwFmtAnchor( FLY_AT_PAGE, nPhyPageNum ) );
+aSet.Put( SwFmtHoriOrient( rItem.lLeft + nCol * rItem.lHDist,
+   text::HoriOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
+aSet.Put( SwFmtVertOrient( rItem.lUpper + nRow * rItem.lVDist,
+   text::VertOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
 const SwFrmFmt *pFmt = rSh.NewFlyFrm(aSet, sal_True,  rFmt );  // Insert 
Fly
 OSL_ENSURE( pFmt, Fly not inserted );
 
@@ -130,21 +128,19 @@ const SwFrmFmt *lcl_InsertBCText( SwWrtShell rSh, const 
SwLabItem rItem,
 
 const SwFrmFmt *lcl_InsertLabText( SwWrtShell rSh, const SwLabItem rItem,
 SwFrmFmt rFmt, SwFldMgr rFldMgr,
-sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bLast, 
sal_Bool bPage)
+sal_uInt16 nCol, sal_uInt16 nRow, sal_Bool bLast )
 {
 SfxItemSet aSet(rSh.GetAttrPool(), RES_ANCHOR, RES_ANCHOR,
 RES_VERT_ORIENT, RES_VERT_ORIENT, RES_HORI_ORIENT, 
RES_HORI_ORIENT, 0 );
 sal_uInt16 nPhyPageNum, nVirtPageNum;
 rSh.GetPageNum( nPhyPageNum, nVirtPageNum );
 
-aSet.Put(SwFmtAnchor(bPage ? FLY_AS_CHAR : FLY_AT_PAGE, nPhyPageNum));
-if (!bPage)
-{
-aSet.Put(SwFmtHoriOrient(rItem.lLeft + nCol * rItem.lHDist,
-
text::HoriOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-aSet.Put(SwFmtVertOrient(rItem.lUpper + nRow * rItem.lVDist,
-
text::VertOrientation::NONE, text::RelOrientation::PAGE_FRAME ));
-}
+//anchor frame to page
+aSet.Put( SwFmtAnchor( FLY_AT_PAGE, nPhyPageNum ) );
+aSet.Put( SwFmtHoriOrient( rItem.lLeft + nCol * rItem.lHDist,
+   text::HoriOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
+aSet.Put( SwFmtVertOrient( rItem.lUpper + nRow * rItem.lVDist,
+   text::VertOrientation::NONE, 
text::RelOrientation::PAGE_FRAME ) );
 const SwFrmFmt *pFmt = rSh.NewFlyFrm(aSet, sal_True,  rFmt );  // Insert 
Fly
 OSL_ENSURE( pFmt, Fly not inserted );
 
@@ -300,26 +296,11 @@ void SwModule::InsertLab(SfxRequest rReq, sal_Bool 
bLabel)
 SwFmtFrmSize aFrmSize(  ATT_FIX_SIZE, iWidth, iHeight );
 pFmt-SetFmtAttr( aFrmSize );
 
-SwFrmFmt* pFmtEORow = new SwFrmFmt (*pFmt);
-SwFrmFmt* pFmtEOCol = new SwFrmFmt (*pFmt);
-SwFrmFmt* pFmtEOColEORow = new SwFrmFmt (*pFmt);
-
-SvxULSpaceItem aFrmULSpace( 0, (sal_uInt16)(rItem.lVDist - 
rItem.lHeight),
-RES_UL_SPACE );
+//frame represents label itself, no border space
 SvxULSpaceItem aFrmNoULSpace( 0, 0, RES_UL_SPACE );
-
-SvxLRSpaceItem aFrmLRSpace( 0, (sal_uInt16)(rItem.lHDist - 
rItem.lWidth),
-0, 0, RES_LR_SPACE );
 SvxLRSpaceItem aFrmNoLRSpace( 0, 0, 0, 0, RES_LR_SPACE );
-
-pFmt-SetFmtAttr(aFrmULSpace);
-pFmt-SetFmtAttr(aFrmLRSpace);
-pFmtEORow-SetFmtAttr(aFrmULSpace);
-pFmtEORow-SetFmtAttr(aFrmNoLRSpace);
-pFmtEOCol-SetFmtAttr(aFrmNoULSpace);
-   

[Libreoffice-commits] .: Branch 'libreoffice-3-6' - i18npool/source

2012-10-02 Thread Libreoffice Gerrit user
 i18npool/source/localedata/data/hu_HU.xml |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 29bd9606cf7c915a61f31418cf187de8f5aa84e9
Author: László Németh nem...@numbertext.org
Date:   Sat Sep 29 00:05:34 2012 +0200

Add back compatible Hungarian date format acceptance

Change-Id: I6b6d62cda241250db89cdcabaa04952d36e1acfd
Reviewed-on: https://gerrit.libreoffice.org/720
Reviewed-by: Thorsten Behrens tbehr...@suse.com
Tested-by: Thorsten Behrens tbehr...@suse.com
Signed-off-by: Andras Timar ati...@suse.com

diff --git a/i18npool/source/localedata/data/hu_HU.xml 
b/i18npool/source/localedata/data/hu_HU.xml
index dc3be12..8ad831e 100644
--- a/i18npool/source/localedata/data/hu_HU.xml
+++ b/i18npool/source/localedata/data/hu_HU.xml
@@ -35,6 +35,8 @@
 MeasurementSystemmetric/MeasurementSystem
   /LC_CTYPE
   LC_FORMAT
+DateAcceptancePatternM-D/DateAcceptancePattern
+DateAcceptancePatternM.D/DateAcceptancePattern
 FormatElement msgid=FixedFormatskey1 default=true type=medium 
usage=FIXED_NUMBER formatindex=0
   FormatCodeStandard/FormatCode
 /FormatElement
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Error while building LO on Windows

2012-10-02 Thread Stephan Bergmann

On 10/01/2012 04:02 PM, Ulrich Kitzinger wrote:

:  !makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp
: makecab_error: Missing % after variable name: %2Fpresenter.xhp


I suppose this is because the ddf-file libreoffice1.ddf contains the lines:

LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\caption\com.sun.PresenterScreen-windows_x86%2Fpresenter.xhp
com2.sun.PresenterScreen_windows_x86%2Fpresenter.xhp
LibreOffice\native\zip\en-US_de\00\presenter-screen_oxt\help\en-US\content\com.sun.PresenterScreen-windows_x86%2Fpresenter.xhp
com3.sun.PresenterScreen_windows_x86%2Fpresenter.xhp


That should long since have been addressed with 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=a297372210396260da57f34da3790f76682603cc 
Quote .ddf content (potentially containing stuff like %2F).


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 2 commits - vbahelper/source vbahelper/util writerfilter/source writerfilter/util

2012-10-02 Thread Libreoffice Gerrit user
 vbahelper/source/msforms/service.cxx|2 +-
 vbahelper/util/msforms.component|2 +-
 writerfilter/source/filter/WriterFilter.cxx |2 +-
 writerfilter/util/writerfilter.component|2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 9da34c09bfd27ba0b1c423ebb558d997b5213018
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 12:59:32 2012 +0300

Use prefix

Change-Id: Ic63d4b6403438fee2d6c70e329ee33d2cde7426b

diff --git a/vbahelper/source/msforms/service.cxx 
b/vbahelper/source/msforms/service.cxx
index 7a32b49..9bc7953 100644
--- a/vbahelper/source/msforms/service.cxx
+++ b/vbahelper/source/msforms/service.cxx
@@ -42,7 +42,7 @@ extern sdecl::ServiceDecl const serviceDecl;
 
 extern C
 {
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL msforms_component_getFactory(
 const sal_Char * pImplName, lang::XMultiServiceFactory * 
pServiceManager,
 registry::XRegistryKey * pRegistryKey )
 {
diff --git a/vbahelper/util/msforms.component b/vbahelper/util/msforms.component
index 5e4c28b..f4fc168 100644
--- a/vbahelper/util/msforms.component
+++ b/vbahelper/util/msforms.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=msforms
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=ControlProviderImpl
 service name=ooo.vba.ControlProvider/
commit 13ec17ce63dcba161169e8f6d351e92be1e5647e
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 12:54:32 2012 +0300

Use prefix

Change-Id: I128623b27bd88795e21de13d7a6fc960de3b6ea8

diff --git a/writerfilter/source/filter/WriterFilter.cxx 
b/writerfilter/source/filter/WriterFilter.cxx
index 68550cf..7e139c5 100644
--- a/writerfilter/source/filter/WriterFilter.cxx
+++ b/writerfilter/source/filter/WriterFilter.cxx
@@ -51,7 +51,7 @@ static struct ::cppu::ImplementationEntry s_component_entries 
[] =
 { 0, 0, 0, 0, 0, 0 } // terminate with NULL
 };
 
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(sal_Char const * 
implName, ::com::sun::star::lang::XMultiServiceFactory * xMgr, 
::com::sun::star::registry::XRegistryKey * xRegistry )
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL 
writerfilter_component_getFactory(sal_Char const * implName, 
::com::sun::star::lang::XMultiServiceFactory * xMgr, 
::com::sun::star::registry::XRegistryKey * xRegistry )
 {
 return ::cppu::component_getFactoryHelper(implName, xMgr, xRegistry, 
s_component_entries );
 }
diff --git a/writerfilter/util/writerfilter.component 
b/writerfilter/util/writerfilter.component
index 0101ea0..b5deea5 100644
--- a/writerfilter/util/writerfilter.component
+++ b/writerfilter/util/writerfilter.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=writerfilter
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=com.sun.star.comp.Writer.WriterFilter
 service name=com.sun.star.document.ExportFilter/
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] fdo#51304: Remove @author annotation

2012-10-02 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/746

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/46/746/1

fdo#51304: Remove @author annotation

This patch remove @author annotation in java source files.It's cleanup

Change-Id: I934db56ea4d1afdc533deb8a7003a58b29b51777
Signed-off-by: José Guilherme Vanz guilherme@gmail.com
---
M reportbuilder/java/com/sun/star/report/DataRow.java
M reportbuilder/java/com/sun/star/report/DataSource.java
M reportbuilder/java/com/sun/star/report/DataSourceException.java
M reportbuilder/java/com/sun/star/report/DataSourceFactory.java
M reportbuilder/java/com/sun/star/report/ImageService.java
M reportbuilder/java/com/sun/star/report/InputRepository.java
M reportbuilder/java/com/sun/star/report/JobDefinitionException.java
M reportbuilder/java/com/sun/star/report/JobProgressIndicator.java
M reportbuilder/java/com/sun/star/report/OfficeToken.java
M reportbuilder/java/com/sun/star/report/OutputRepository.java
M reportbuilder/java/com/sun/star/report/ReportEngine.java
M reportbuilder/java/com/sun/star/report/ReportJob.java
M reportbuilder/java/com/sun/star/report/ReportJobDefinition.java
M reportbuilder/java/com/sun/star/report/ReportJobFactory.java
M reportbuilder/java/com/sun/star/report/SOImageService.java
M reportbuilder/java/com/sun/star/report/StorageRepository.java
M reportbuilder/java/com/sun/star/report/function/metadata/AuthorFunction.java
M 
reportbuilder/java/com/sun/star/report/function/metadata/AuthorFunctionDescription.java
M 
reportbuilder/java/com/sun/star/report/function/metadata/MetaDataFunctionCategory.java
M reportbuilder/java/com/sun/star/report/function/metadata/TitleFunction.java
M 
reportbuilder/java/com/sun/star/report/function/metadata/TitleFunctionDescription.java
M reportbuilder/java/com/sun/star/report/pentaho/OfficeNamespaces.java
M reportbuilder/java/com/sun/star/report/pentaho/PentahoFormulaContext.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/AbstractReportElementLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/FixedTextLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/FormatValueUtility.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/FormattedTextLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/ImageElementContext.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/ImageElementLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/ObjectOleLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeDetailLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeGroupInstanceSectionLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeGroupLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeGroupSectionLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficePageSectionLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeRepeatingStructureLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeReportLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeTableLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/OfficeTableTemplateLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/TableCellLayoutController.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/VariablesCollection.java
M 
reportbuilder/java/com/sun/star/report/pentaho/layoutprocessor/VariablesDeclarationLayoutController.java
M reportbuilder/java/com/sun/star/report/pentaho/loader/InputResourceKey.java
M reportbuilder/java/com/sun/star/report/pentaho/model/DataStyle.java
M reportbuilder/java/com/sun/star/report/pentaho/model/FixedTextElement.java
M reportbuilder/java/com/sun/star/report/pentaho/model/FontFaceDeclsSection.java
M reportbuilder/java/com/sun/star/report/pentaho/model/FontFaceElement.java
M reportbuilder/java/com/sun/star/report/pentaho/model/FormatCondition.java
M reportbuilder/java/com/sun/star/report/pentaho/model/FormattedTextElement.java
M reportbuilder/java/com/sun/star/report/pentaho/model/ImageElement.java
M reportbuilder/java/com/sun/star/report/pentaho/model/ObjectOleElement.java
M reportbuilder/java/com/sun/star/report/pentaho/model/OfficeDetailSection.java
M reportbuilder/java/com/sun/star/report/pentaho/model/OfficeDocument.java
M reportbuilder/java/com/sun/star/report/pentaho/model/OfficeGroup.java
M 
reportbuilder/java/com/sun/star/report/pentaho/model/OfficeGroupInstanceSection.java
M reportbuilder/java/com/sun/star/report/pentaho/model/OfficeGroupSection.java
M 

[Libreoffice-commits] .: Branch 'feature/android-single-dso' - 9 commits - android/experimental cpputools/source oox/source oox/util sdext/source shell/source starmath/source stoc/source vbahelper/sou

2012-10-02 Thread Libreoffice Gerrit user
 android/experimental/DocumentLoader/Makefile |2 +
 android/experimental/DocumentLoader/native-code.cxx  |8 
 cpputools/source/registercomponent/registercomponent.cxx |6 ++-
 cpputools/source/unoexe/unoexe.cxx   |   28 ++-
 oox/source/core/services.cxx |2 -
 oox/util/oox.component   |2 -
 sdext/source/minimizer/minimizer.component   |3 +
 sdext/source/minimizer/pppoptimizeruno.cxx   |2 -
 sdext/source/pdfimport/pdfimport.component   |3 +
 sdext/source/pdfimport/services.cxx  |2 -
 sdext/source/presenter/PresenterComponent.cxx|4 +-
 sdext/source/presenter/presenter.component   |3 +
 shell/source/backends/gconfbe/gconfbackend.cxx   |2 -
 shell/source/backends/gconfbe/gconfbe1.component |2 -
 shell/source/backends/kde4be/kde4backend.cxx |2 -
 shell/source/backends/kde4be/kde4be1.component   |2 -
 shell/source/backends/kdebe/kdebackend.cxx   |2 -
 shell/source/backends/kdebe/kdebe1.component |2 -
 shell/source/backends/macbe/macbe1.component |2 -
 shell/source/backends/macbe/macbecdef.cxx|2 -
 shell/source/backends/wininetbe/wininetbe1.component |2 -
 shell/source/backends/wininetbe/wininetbecdef.cxx|2 -
 shell/source/cmdmail/cmdmail.component   |2 -
 shell/source/cmdmail/cmdmailentry.cxx|2 -
 shell/source/unix/exec/shellexecentry.cxx|2 -
 shell/source/unix/exec/syssh.component   |2 -
 shell/source/win32/SysShentry.cxx|2 -
 shell/source/win32/simplemail/smplmail.component |2 -
 shell/source/win32/simplemail/smplmailentry.cxx  |2 -
 shell/source/win32/syssh.component   |2 -
 starmath/source/accessibility.cxx|2 -
 stoc/source/javaloader/javaloader.component  |2 -
 stoc/source/javaloader/javaloader.cxx|2 -
 stoc/source/javavm/javavm.component  |2 -
 stoc/source/javavm/javavm.cxx|2 -
 vbahelper/source/msforms/service.cxx |2 -
 vbahelper/util/msforms.component |2 -
 writerfilter/source/filter/WriterFilter.cxx  |2 -
 writerfilter/util/writerfilter.component |2 -
 39 files changed, 65 insertions(+), 54 deletions(-)

New commits:
commit 115c492e85206fce18b39405f1f82a5e4e39cd38
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 12:59:32 2012 +0300

Use prefix

Change-Id: Ic63d4b6403438fee2d6c70e329ee33d2cde7426b

diff --git a/vbahelper/source/msforms/service.cxx 
b/vbahelper/source/msforms/service.cxx
index 7a32b49..9bc7953 100644
--- a/vbahelper/source/msforms/service.cxx
+++ b/vbahelper/source/msforms/service.cxx
@@ -42,7 +42,7 @@ extern sdecl::ServiceDecl const serviceDecl;
 
 extern C
 {
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL msforms_component_getFactory(
 const sal_Char * pImplName, lang::XMultiServiceFactory * 
pServiceManager,
 registry::XRegistryKey * pRegistryKey )
 {
diff --git a/vbahelper/util/msforms.component b/vbahelper/util/msforms.component
index 5e4c28b..f4fc168 100644
--- a/vbahelper/util/msforms.component
+++ b/vbahelper/util/msforms.component
@@ -17,7 +17,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  --
 
-component loader=com.sun.star.loader.SharedLibrary
+component loader=com.sun.star.loader.SharedLibrary prefix=msforms
 xmlns=http://openoffice.org/2010/uno-components;
   implementation name=ControlProviderImpl
 service name=ooo.vba.ControlProvider/
commit c14ba0883e0caa23570f25f6aa57a27acbf273cb
Author: Tor Lillqvist t...@iki.fi
Date:   Tue Oct 2 12:54:32 2012 +0300

Use prefix

Change-Id: I128623b27bd88795e21de13d7a6fc960de3b6ea8

diff --git a/writerfilter/source/filter/WriterFilter.cxx 
b/writerfilter/source/filter/WriterFilter.cxx
index 68550cf..7e139c5 100644
--- a/writerfilter/source/filter/WriterFilter.cxx
+++ b/writerfilter/source/filter/WriterFilter.cxx
@@ -51,7 +51,7 @@ static struct ::cppu::ImplementationEntry s_component_entries 
[] =
 { 0, 0, 0, 0, 0, 0 } // terminate with NULL
 };
 
-SAL_DLLPUBLIC_EXPORT void * SAL_CALL component_getFactory(sal_Char const * 
implName, ::com::sun::star::lang::XMultiServiceFactory * xMgr, 
::com::sun::star::registry::XRegistryKey * xRegistry )
+SAL_DLLPUBLIC_EXPORT void * SAL_CALL 
writerfilter_component_getFactory(sal_Char const * implName, 
::com::sun::star::lang::XMultiServiceFactory * xMgr, 
::com::sun::star::registry::XRegistryKey * xRegistry )
 {
 return 

[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/filter/ww8/ww8par2.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 55fe92fb396ebcfffe30831ebb460bfb3e6274aa
Author: Caolán McNamara caol...@redhat.com
Date:   Mon Oct 1 23:57:06 2012 +0100

Resolves: fdo#54862 extra ++n causing merged cells to be skipped

commit 567c1db25bd705faac44203e4a3d01d0f5e1385c reverted a pile
of other commits, including 858b5b4f36a357fe7192e7c2ed9cc3cdfc81fd8f
but didn't revert the ++n of that commit, leading to merge groups
getting skipped

Change-Id: Ie2b46ebc433df636ebe0092f4a992631b7f06a3f
(cherry picked from commit b966a09c2da9441961c93c44be556399575db849)

Signed-off-by: Michael Stahl mst...@redhat.com

diff --git a/sw/source/filter/ww8/ww8par2.cxx b/sw/source/filter/ww8/ww8par2.cxx
index 1b5bd75..679f116 100644
--- a/sw/source/filter/ww8/ww8par2.cxx
+++ b/sw/source/filter/ww8/ww8par2.cxx
@@ -2751,7 +2751,6 @@ void WW8TabDesc::FinishSwTable()
nRowSpan :
  ((-1) * (nRowSpan - n));
 pCurrentBox-setRowSpan( nRowSpanSet );
-++n;
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED][3-6] Re: [REVIEW 3-6] fdo#54862 .doc merged cells import regression fix

2012-10-02 Thread Michael Stahl
On 02/10/12 01:14, Caolán McNamara wrote:
 As https://bugs.freedesktop.org/show_bug.cgi?id=54862
 fix
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=b966a09c2da9441961c93c44be556399575db849
 
 commit 567c1db25bd705faac44203e4a3d01d0f5e1385c reverted a pile of other
 commits, including 858b5b4f36a357fe7192e7c2ed9cc3cdfc81fd8f but didn't
 revert the ++n of that commit, leading to merge groups getting skipped.
 Quite likely a bunch of .doc table complaints lead back to this.

pushed to libreoffice-3-6


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Easy Hack -- fdo #44718 Add Unicode 2034 (Triple Prime) to OpenSymbol

2012-10-02 Thread Caolán McNamara
On Mon, 2012-10-01 at 11:03 +0100, Michael Meeks wrote:
 On Sun, 2012-09-30 at 20:04 -0700, Joel Madero wrote:
  Can someone take a look at it real quick and see if you can
  point to the right files and throw a comment in there?
 
   I added some pointers.
 
  I'll keep looking for more easy hacks.
  https://bugs.freedesktop.org/show_bug.cgi?id=44718
 
   Be nice if someone can work out what Topic / Skills are right
 (basically font editing) - and annotate it to turn it into an EasyHack.

Olivier and Julien were our last two people to add symbols to that font
and keywords/aliases for the symbol to starmath. So they may be able to
help or to add some practical advice to the bug.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: OUString is mutable?

2012-10-02 Thread Stephan Bergmann

On 10/01/2012 06:05 PM, Dennis E. Hamilton wrote:

Regarding the mention that the latest Java VM is using UTF8 internally instead 
of unsigned short arrays is rather daunting.  There is an easy way to test it 
-- see if char values that are not admissible UTF16 codes can be used in 
construction of a string and then extracted correctly.  If they can, there is 
no way that transformation to and from UTF8 occurred.  If they can't, it is an 
interesting breaking change in Java.  With regard to string literals, it would 
be interesting to see what can be introduced into those via escape codes too.


Note that the JVM traditionally also makes use of a modified form of 
UTF-8 (encoding surrogate code points individually, and encoding \u 
as 0xC0 0x80), see the JNI spec.


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED] [PATCH 2/3] fdo#51304: Remove @author annotation and commented code

2012-10-02 Thread José Guilherme Vanz
Hi!

I unterstood... Thanks for comment! I'll do more and better next time
:-)


2012/10/1 Thorsten Behrens t...@documentfoundation.org

 guilherme@gmail.com wrote:
  This patch removes '@author' annotation, code in comments
  and adjust some empty catches. It's cleanup patch
 
 Hi José Guilherme,

 partially pushed - I left the hunks commenting empty methods as
 FIXME instead of removing them straight away unapplied. If the
 method is pointless, why not remove them in one go?

 I also did not apply your printStackTrace() logging calls, I think
 at least some of them report on exceptions that were seemingly
 previously ignored on purpose?

 Stephan, or anyone else more up-to-date with contemporary java
 idiom, any opinions here?

 Thanks for the nice cleanup anyways! :)

 Cheers,

 -- Thorsten




-- 
Att. José Guilherme Vanz
Se você é capaz de tremer de indignação cada vez que se comete uma
injustiça no mundo, então somos companheiros - Che Guevara
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: beanshell/ExternalProject_beanshell.mk beanshell/Makefile beanshell/Module_beanshell.mk beanshell/Package_beanshell.mk beanshell/prj beanshell/UnpackedTarball_beanshell.mk con

2012-10-02 Thread Libreoffice Gerrit user
 Makefile.top   |2 +-
 Module_tail_build.mk   |1 +
 beanshell/ExternalProject_beanshell.mk |   33 +
 beanshell/Makefile |7 +++
 beanshell/Module_beanshell.mk  |   22 ++
 beanshell/Package_beanshell.mk |   16 
 beanshell/UnpackedTarball_beanshell.mk |   18 ++
 beanshell/prj/build.lst|5 ++---
 config_host.mk.in  |1 +
 configure.in   |2 ++
 tail_build/prj/build.lst   |2 +-
 11 files changed, 104 insertions(+), 5 deletions(-)

New commits:
commit 02e400865cb9a51b623a3d866d3236f0d6c899ca
Author: David Ostrovsky da...@ostrovsky.org
Date:   Tue Oct 2 09:31:35 2012 +0200

gbuildification of beanshell

Change-Id: I1b71796ca95500efeffe5e01dba3e73711f920e7
Reviewed-on: https://gerrit.libreoffice.org/745
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/Makefile.top b/Makefile.top
index e188cdc..de91f56 100644
--- a/Makefile.top
+++ b/Makefile.top
@@ -44,6 +44,7 @@ basebmp\
 basegfx\
 basic\
 bean\
+beanshell\
 binaryurp\
 bluez_bluetooth\
 boost\
@@ -196,7 +197,6 @@ xmlsecurity\
 xsltml\
 
 dmake_modules:=\
-beanshell\
 berkeleydb\
 binfilter \
 cairo\
diff --git a/Module_tail_build.mk b/Module_tail_build.mk
index a0c958b..86a9c11 100644
--- a/Module_tail_build.mk
+++ b/Module_tail_build.mk
@@ -37,6 +37,7 @@ $(eval $(call gb_Module_add_moduledirs,tail_end,\
basegfx \
basic \
bean \
+   $(call gb_Helper_optional,BSH,beanshell) \
binaryurp \
$(call gb_Helper_optional,BLUEZ,bluez_bluetooth) \
bridges \
diff --git a/beanshell/ExternalProject_beanshell.mk 
b/beanshell/ExternalProject_beanshell.mk
new file mode 100644
index 000..b1bfa9f
--- /dev/null
+++ b/beanshell/ExternalProject_beanshell.mk
@@ -0,0 +1,33 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call gb_ExternalProject_ExternalProject,beanshell))
+
+$(eval $(call gb_ExternalProject_use_unpacked,beanshell,beanshell))
+
+$(eval $(call gb_ExternalProject_register_targets,beanshell,\
+   build \
+))
+
+$(call gb_ExternalProject_get_state_target,beanshell,build) :
+   cd $(call gb_UnpackedTarball_get_dir,beanshell)  \
+   ANT_OPTS=$$ANT_OPTS -Dfile.encoding=ISO-8859-1 \
+   $(ANT) \
+   -q \
+   -f build.xml \
+   -Dbuild.label=build-libreoffice \
+   $(if $(filter yes,$(JAVACISGCJ))\
+   ,-Dbuild.compiler=gcj \
+   ,-Dant.build.javac.source=$(JAVA_SOURCE_VER) \
+   -Dant.build.javac.target=$(JAVA_TARGET_VER) \
+   ) \
+   $(if $(debug),-Dbuild.debug=on)  \
+   touch $@
+
+# vim: set noet sw=4 ts=4:
diff --git a/beanshell/Makefile b/beanshell/Makefile
new file mode 100644
index 000..ccb1c85
--- /dev/null
+++ b/beanshell/Makefile
@@ -0,0 +1,7 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+
+module_directory:=$(dir $(realpath $(firstword $(MAKEFILE_LIST
+
+include $(module_directory)/../solenv/gbuild/partial_build.mk
+
+# vim: set noet sw=4 ts=4:
diff --git a/beanshell/Module_beanshell.mk b/beanshell/Module_beanshell.mk
new file mode 100644
index 000..e739b36
--- /dev/null
+++ b/beanshell/Module_beanshell.mk
@@ -0,0 +1,22 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call gb_Module_Module,beanshell))
+
+ifneq ($(SOLAR_JAVA),)
+ifneq ($(filter BSH,$(BUILD_TYPE)),)
+$(eval $(call gb_Module_add_targets,beanshell,\
+   ExternalProject_beanshell \
+   Package_beanshell \
+   UnpackedTarball_beanshell \
+))
+endif
+endif
+
+# vim: set noet sw=4 ts=4:
diff --git a/beanshell/Package_beanshell.mk b/beanshell/Package_beanshell.mk
new file mode 100644
index 000..983c1fc
--- /dev/null
+++ b/beanshell/Package_beanshell.mk
@@ -0,0 +1,16 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call 

Re: [PUSHED][PATCH] Update French spelling dictionary

2012-10-02 Thread Caolán McNamara
On Mon, 2012-10-01 at 02:28 -0700, Olivier R. wrote:
 Hello *,
 
 Here is a patch to update the French spelling dictionary.

Pushed now to master, thanks for this.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH] soltools conversion to gbuild

2012-10-02 Thread Andras Timar
2012/9/27 David Ostrovsky david.ostrov...@gmx.de:
 i submitted gerrit patch that convert soltools to gbuild:
 https://gerrit.libreoffice.org/#/c/698

I don't know why tinderboxes do not complain, but for me unistd.h is
not delivered to solver/wntmsci12.pro/inc, and build fails early.

Andras
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: distro-configs/LibreOfficeWin32.conf distro-configs/LibreOfficeWin64.conf

2012-10-02 Thread Libreoffice Gerrit user
 distro-configs/LibreOfficeWin32.conf |1 -
 distro-configs/LibreOfficeWin64.conf |1 -
 2 files changed, 2 deletions(-)

New commits:
commit 59e6d18bea73417b2b5cd044d0489f882e847df6
Author: Andras Timar ati...@suse.com
Date:   Tue Oct 2 14:39:54 2012 +0200

remove deprecated configure switch --disable-activex-component

Change-Id: I3a5b19ac9f7c09a7bb5c919b3a9bc18ab646fd76

diff --git a/distro-configs/LibreOfficeWin32.conf 
b/distro-configs/LibreOfficeWin32.conf
index 1a34632..7273ed4 100644
--- a/distro-configs/LibreOfficeWin32.conf
+++ b/distro-configs/LibreOfficeWin32.conf
@@ -6,7 +6,6 @@
 --without-agfa-monotype-fonts
 --enable-extension-integration
 --with-java-target-version=1.5
---disable-activex-component
 --disable-gtk
 --enable-binfilter
 --enable-ext-scripting-beanshell
diff --git a/distro-configs/LibreOfficeWin64.conf 
b/distro-configs/LibreOfficeWin64.conf
index 2c0b022..344551e 100644
--- a/distro-configs/LibreOfficeWin64.conf
+++ b/distro-configs/LibreOfficeWin64.conf
@@ -2,7 +2,6 @@
 --without-agfa-monotype-fonts
 --with-java-target-version=1.5
 --enable-cl-x64
---disable-activex-component
 --disable-gtk
 --enable-ext-scripting-beanshell
 --enable-ext-scripting-javascript
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Drop support for Java 5?

2012-10-02 Thread Michael Stahl
On 02/10/12 14:37, d.ostrov...@idaia.de wrote:
 Hi Christian,
 
 Zitat von Christian Lohmaier lohmaier+libreoff...@googlemail.com:
 
 Hi David, *,

 On Tue, Oct 2, 2012 at 1:27 PM,  d.ostrov...@idaia.de wrote:

 Java 7 is today the current version, Java 6 is the old one and Java 5 is the
 ancient one. Does it still make sense to support it in LO 3.7 or even in LO
 4.0?

 Mac OSX 10.4 only comes with 1.5.
 
 what about to drop support for Mac OS X 10.4 then?

that is on the LO4 todo list :)

it's a bit annoying for users to bump platform requirements in a minor
release, so LO4 sounds like the better target, and isn't that far off.

doesn't make sense to support an OS release that the OS vendor hasn't
been supporting for years.  same goes for Windows 2000.   iirc JRE5 is
also no longer supported by Oracle.

a more difficult question is, should we require OS X 10.6 and thereby
drop support for OS X on PPC altogether?

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 8 commits - chart2/source drawinglayer/source svl/inc svl/source xmloff/source

2012-10-02 Thread Libreoffice Gerrit user
 chart2/source/controller/dialogs/res_DataLabel.cxx  |4 
 drawinglayer/source/primitive2d/metafileprimitive2d.cxx |   10 +-
 svl/inc/svl/ctloptions.hxx  |   20 ++--
 svl/inc/svl/imageitm.hxx|4 
 svl/inc/svl/zformat.hxx |   10 +-
 svl/source/config/ctloptions.cxx|   74 
 svl/source/fsstor/oinputstreamcontainer.cxx |6 -
 svl/source/fsstor/oinputstreamcontainer.hxx |5 -
 svl/source/fsstor/ostreamcontainer.cxx  |   16 +--
 svl/source/fsstor/ostreamcontainer.hxx  |6 -
 svl/source/inc/passwordcontainer.hxx|   58 ++--
 svl/source/items/imageitm.cxx   |8 -
 svl/source/numbers/zforfind.cxx |2 
 svl/source/numbers/zformat.cxx  |   66 +++---
 svl/source/passwordcontainer/passwordcontainer.cxx  |   24 ++---
 xmloff/source/style/xmlnumfe.cxx|   28 +++---
 16 files changed, 170 insertions(+), 171 deletions(-)

New commits:
commit 5d8c2eddead8c9144e337d64a559e1621d36da43
Author: Noel Grandin n...@peralex.com
Date:   Mon Oct 1 12:54:52 2012 +0200

String-OUString in svl::ImpSvNumberformatInfo

Change-Id: Iff00b06a1e4e7c485e70ef0537fdea07ce3d42e5

diff --git a/svl/inc/svl/zformat.hxx b/svl/inc/svl/zformat.hxx
index 0418fca..1a35282 100644
--- a/svl/inc/svl/zformat.hxx
+++ b/svl/inc/svl/zformat.hxx
@@ -65,7 +65,7 @@ enum NfHackConversion
 
 struct ImpSvNumberformatInfo// Struct for FormatInfo
 {
-String* sStrArray;  // Array of symbols
+OUString* sStrArray;// Array of symbols
 short* nTypeArray;  // Array of infos
 sal_uInt16 nThousand;   // Count of group separator sequences
 sal_uInt16 nCntPre; // Count of digits before decimal point
@@ -281,7 +281,7 @@ public:
 // Substring of a subformat code nNumFor (0..3)
 // nPos == 0x = last substring
 // bString==true: first/last SYMBOLTYPE_STRING or SYMBOLTYPE_CURRENCY
-const String* GetNumForString( sal_uInt16 nNumFor, sal_uInt16 nPos,
+const OUString* GetNumForString( sal_uInt16 nNumFor, sal_uInt16 nPos,
 bool bString = false ) const;
 
 // Subtype of a subformat code nNumFor (0..3)
diff --git a/svl/source/numbers/zforfind.cxx b/svl/source/numbers/zforfind.cxx
index fb5bbe3..69d6761 100644
--- a/svl/source/numbers/zforfind.cxx
+++ b/svl/source/numbers/zforfind.cxx
@@ -2466,7 +2466,7 @@ bool ImpSvNumberInputScan::ScanStringNumFor(
 if ( !pFormat )
 return false;
 const ::utl::TransliterationWrapper* pTransliteration = 
pFormatter-GetTransliteration();
-const String* pStr;
+const OUString* pStr;
 rtl::OUString aString( rString );
 bool bFound = false;
 bool bFirst = true;
diff --git a/svl/source/numbers/zformat.cxx b/svl/source/numbers/zformat.cxx
index 5cf224b..5c4c31c 100644
--- a/svl/source/numbers/zformat.cxx
+++ b/svl/source/numbers/zformat.cxx
@@ -353,7 +353,7 @@ void ImpSvNumFor::Enlarge(sal_uInt16 nAnz)
 if ( nAnz )
 {
 aI.nTypeArray = new short[nAnz];
-aI.sStrArray  = new String[nAnz];
+aI.sStrArray  = new OUString[nAnz];
 }
 else
 {
@@ -2051,13 +2051,13 @@ bool SvNumberformat::GetOutputString(String sString,
 if( bStarFlag )
 {
 OutString += (sal_Unicode) 0x1B;
-OutString += rInfo.sStrArray[i].GetChar(1);
+OutString += rInfo.sStrArray[i][1];
 bRes = true;
 }
 break;
 case NF_SYMBOLTYPE_BLANK:
 InsertBlanks( OutString, OutString.Len(),
-rInfo.sStrArray[i].GetChar(1) );
+rInfo.sStrArray[i][1] );
 break;
 case NF_KEY_GENERAL :   // #77026# General is the same as @
 case NF_SYMBOLTYPE_DEL :
@@ -2319,13 +2319,13 @@ bool SvNumberformat::GetOutputString(double fNumber,
 if( bStarFlag )
 {
 OutString += (sal_Unicode) 0x1B;
-OutString += rInfo.sStrArray[i].GetChar(1);
+OutString += rInfo.sStrArray[i][1];
 bRes = true;
 }
 break;
 case NF_SYMBOLTYPE_BLANK:
 InsertBlanks( OutString, OutString.Len(),
-rInfo.sStrArray[i].GetChar(1) );
+rInfo.sStrArray[i][1] );
 break;
 

Re: [PATCH] sal_bool and String conversions

2012-10-02 Thread Michael Stahl
On 02/10/12 08:33, Noel Grandin wrote:
 Hi
 
 The first 4 patches perform sal_Bool-bool conversions in the SVL module.
 
 The next 8 patches perform UniString-OUString conversions.

pushed about 8 of those.

 Patch 7 and 11 need careful checking because they add methods to 
 OUStringBuffer and OUString.

 
 diff --git a/sal/inc/rtl/ustring.h b/sal/inc/rtl/ustring.h
 index 5b4982e..1595c0f 100644
 --- a/sal/inc/rtl/ustring.h
 +++ b/sal/inc/rtl/ustring.h
 @@ -1500,7 +1500,7 @@ SAL_DLLPUBLIC void SAL_CALL 
 rtl_uString_newReplaceFirstAsciiLAsciiL(
  */
  SAL_DLLPUBLIC void SAL_CALL rtl_uString_newReplaceAll(
  rtl_uString ** newStr, rtl_uString * str, rtl_uString const * from,
 -rtl_uString const * to) SAL_THROW_EXTERN_C();
 +rtl_uString const * to, sal_Int32 fromIndex) SAL_THROW_EXTERN_C();
  
  /** Create a new string by replacing all occurrences of a given substring 
 with
  another substring.

that is not ABI compatible.

 -OUString replaceAll(OUString const  from, OUString const  to) const {
 +OUString replaceAll(OUString const  from, OUString const  to, int 
 fromIndex = 0) const {

this should be source and binary compatible i think, since it's inline
function?

am unsure about this:

 +OUStringBuffer  operator+=( const OUString str )
 +{
 +return append( str.getStr(), str.getLength() );
 +}
 +
 +/**
 +Appends the string representation of the codechar/code array
 +argument to this string buffer.
 +
 +The characters of the array argument are appended, in order, to
 +the contents of this string buffer. The length of this string
 +buffer increases by the length of the argument.
 +
 +@param   str   the characters to be appended.
 +@return  this string buffer.
 +@since LibreOffice 3.7
 + */
 +OUStringBuffer  operator+=( const sal_Unicode * str )
 +{
 +return append( str, rtl_ustr_getLength( str ) );
 +}

  /**
 +Appends the string representation of the ASCII codechar/code
 +argument to this string buffer.
 +
 +The argument is appended to the contents of this string buffer.
 +The length of this string buffer increases by code1/code.
 +
 +@param   c   an ASCII codechar/code.
 +@return  this string buffer.
 +
 +@since LibreOffice 3.7
 + */
 +OUStringBuffer  operator+=(char c)
 +{
 +return append(c);
 +}

hmmm... do we really want to have 2 sets of methods append() and
operator+= that do the same thing, possibly with different overloads?

perhaps we should deprecate the append methods... but those often have
addition radix etc, parameters... and is it possible to chain += like
append() in a single statement?

didn't push the 2 patches that depend on the sal changes.


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread Michael Meeks

On Tue, 2012-10-02 at 14:53 +0200, Michael Stahl wrote:
  Mac OSX 10.4 only comes with 1.5.
  
  what about to drop support for Mac OS X 10.4 then?
 
 that is on the LO4 todo list :)

Worth adding to the wiki page so we don't forget I imagine:

http://wiki.documentfoundation.org/Development/LibreOffice4

ATB,

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: config_host.mk.in configure.in desktop/source scp2/source solenv/bin

2012-10-02 Thread Libreoffice Gerrit user
 config_host.mk.in   |2 +
 configure.in|   22 +++
 desktop/source/splash/splash.cxx|   46 ++--
 scp2/source/ooo/common_brand.scp|   18 
 solenv/bin/modules/installer/ziplist.pm |2 +
 5 files changed, 88 insertions(+), 2 deletions(-)

New commits:
commit b14073218ea28a5bf05cd639e30215787d67114c
Author: Ivan Timofeev timofeev@gmail.com
Date:   Sat Sep 29 20:03:12 2012 +0400

fdo#52268: splash: change color and position of messages

Change-Id: If6c607fc9f0be1f7a71eda8d555399451938c5fc

diff --git a/config_host.mk.in b/config_host.mk.in
index d6f5f16..48b470e 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -465,6 +465,8 @@ export PRODUCTNAME=@PRODUCTNAME@
 export PRODUCTVERSION=@PRODUCTVERSION@
 export PROGRESSBARCOLOR=@PROGRESSBARCOLOR@
 export PROGRESSFRAMECOLOR=@PROGRESSFRAMECOLOR@
+export PROGRESSTEXTCOLOR=@PROGRESSTEXTCOLOR@
+export PROGRESSTEXTBASELINE=@PROGRESSTEXTBASELINE@
 export PROGRESSPOSITION=@PROGRESSPOSITION@
 export PROGRESSSIZE=@PROGRESSSIZE@
 export PROEXT=@PROEXT@
diff --git a/configure.in b/configure.in
index 93b828d..34dfce7 100644
--- a/configure.in
+++ b/configure.in
@@ -12053,6 +12053,28 @@ else
 fi
 AC_SUBST(PROGRESSFRAMECOLOR)
 
+AC_MSG_CHECKING([for custom 'intro' progress text color])
+PROGRESSTEXTCOLOR=
+if test -z $with_intro_progressbar_text_color; then
+   PROGRESSTEXTCOLOR=255,255,255
+   AC_MSG_RESULT([none])
+else
+   PROGRESSTEXTCOLOR=$with_intro_progressbar_text_color
+   AC_MSG_RESULT([$PROGRESSTEXTCOLOR])
+fi
+AC_SUBST(PROGRESSTEXTCOLOR)
+
+AC_MSG_CHECKING([for custom 'intro' progress text baseline])
+PROGRESSTEXTBASELINE=
+if test -z $with_intro_progressbar_text_baseline; then
+   PROGRESSTEXTBASELINE=287
+   AC_MSG_RESULT([none])
+else
+   PROGRESSTEXTBASELINE=$with_intro_progressbar_text_baseline
+   AC_MSG_RESULT([$PROGRESSTEXTBASELINE])
+fi
+AC_SUBST(PROGRESSTEXTBASELINE)
+
 AC_MSG_CHECKING([for alternative branding images directory])
 INTRO_BITMAP=
 ABOUT_BACKGROUND_SVG=
diff --git a/desktop/source/splash/splash.cxx b/desktop/source/splash/splash.cxx
index 38de2eb..e55b5b2 100644
--- a/desktop/source/splash/splash.cxx
+++ b/desktop/source/splash/splash.cxx
@@ -86,6 +86,7 @@ private:
 BitmapEx_aIntroBmp;
 Color   _cProgressFrameColor;
 Color   _cProgressBarColor;
+Color   _cProgressTextColor;
 bool_bNativeProgress;
 OUString_sAppName;
 OUString_sProgressText;
@@ -101,7 +102,7 @@ private:
 sal_Bool_bFullScreenSplash;
 sal_Bool_bProgressEnd;
 long _height, _width, _tlx, _tly, _barwidth;
-long _barheight, _barspace;
+long _barheight, _barspace, _textBaseline;
 double _fXPos, _fYPos;
 double _fWidth, _fHeight;
 const long _xoffset, _yoffset;
@@ -130,6 +131,7 @@ SplashScreen::SplashScreen()
 , _vdev(*((IntroWindow*)this))
 , _cProgressFrameColor(sal::static_int_cast ColorData (NOT_LOADED))
 , _cProgressBarColor(sal::static_int_cast ColorData (NOT_LOADED))
+, _cProgressTextColor(sal::static_int_cast ColorData (NOT_LOADED))
 , _bNativeProgress(true)
 , _iMax(100)
 , _iProgress(0)
@@ -144,6 +146,7 @@ SplashScreen::SplashScreen()
 , _barwidth(NOT_LOADED)
 , _barheight(NOT_LOADED)
 , _barspace(2)
+, _textBaseline(NOT_LOADED)
 , _fXPos(-1.0)
 , _fYPos(-1.0)
 , _fWidth(-1.0)
@@ -297,6 +300,9 @@ SplashScreen::initialize( const 
::com::sun::star::uno::Sequence ::com::sun::sta
 }
 }
 
+if ( NOT_LOADED == _textBaseline )
+_textBaseline = _height;
+
 if ( sal::static_int_cast ColorData (NOT_LOADED) ==
  _cProgressFrameColor.GetColor() )
 _cProgressFrameColor = Color( COL_LIGHTGRAY );
@@ -311,6 +317,10 @@ SplashScreen::initialize( const 
::com::sun::star::uno::Sequence ::com::sun::sta
 _cProgressBarColor = Color( COL_BLUE );
 }
 
+if ( sal::static_int_cast ColorData (NOT_LOADED) ==
+ _cProgressTextColor.GetColor() )
+_cProgressTextColor = Color( COL_BLACK );
+
 Application::AddEventListener(
 LINK( this, SplashScreen, AppEventListenerHdl ) );
 
@@ -363,6 +373,10 @@ void SplashScreen::loadConfig()
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressFrameColor ) ) );
 OUString sProgressBarColor = implReadBootstrapKey(
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressBarColor ) ) );
+OUString sProgressTextColor = implReadBootstrapKey(
+OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressTextColor ) ) );
+OUString sProgressTextBaseline = implReadBootstrapKey(
+OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressTextBaseline ) ) );
 OUString sSize = implReadBootstrapKey(
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressSize ) ) );
 OUString sPosition = 

[Libreoffice-commits] .: Branch 'libreoffice-3-6' - config_host.mk.in configure.in desktop/source scp2/source solenv/bin

2012-10-02 Thread Libreoffice Gerrit user
 config_host.mk.in   |2 +
 configure.in|   22 +++
 desktop/source/splash/splash.cxx|   46 ++--
 scp2/source/ooo/common_brand.scp|   18 
 solenv/bin/modules/installer/ziplist.pm |2 +
 5 files changed, 88 insertions(+), 2 deletions(-)

New commits:
commit 71b584680e4527283ccc4fa14169bc7d28d32344
Author: Ivan Timofeev timofeev@gmail.com
Date:   Sat Sep 29 20:03:12 2012 +0400

fdo#52268: splash: change color and position of messages

Change-Id: If6c607fc9f0be1f7a71eda8d555399451938c5fc
Signed-off-by: Jan Holesovsky ke...@suse.cz

diff --git a/config_host.mk.in b/config_host.mk.in
index 96635db..4fb80d1 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -413,6 +413,8 @@ export PRODUCTNAME=@PRODUCTNAME@
 export PRODUCTVERSION=@PRODUCTVERSION@
 export PROGRESSBARCOLOR=@PROGRESSBARCOLOR@
 export PROGRESSFRAMECOLOR=@PROGRESSFRAMECOLOR@
+export PROGRESSTEXTCOLOR=@PROGRESSTEXTCOLOR@
+export PROGRESSTEXTBASELINE=@PROGRESSTEXTBASELINE@
 export PROGRESSPOSITION=@PROGRESSPOSITION@
 export PROGRESSSIZE=@PROGRESSSIZE@
 export PROEXT=@PROEXT@
diff --git a/configure.in b/configure.in
index 11e20a0..651963a 100644
--- a/configure.in
+++ b/configure.in
@@ -11310,6 +11310,28 @@ else
 fi
 AC_SUBST(PROGRESSFRAMECOLOR)
 
+AC_MSG_CHECKING([for custom 'intro' progress text color])
+PROGRESSTEXTCOLOR=
+if test -z $with_intro_progressbar_text_color; then
+   PROGRESSTEXTCOLOR=255,255,255
+   AC_MSG_RESULT([none])
+else
+   PROGRESSTEXTCOLOR=$with_intro_progressbar_text_color
+   AC_MSG_RESULT([$PROGRESSTEXTCOLOR])
+fi
+AC_SUBST(PROGRESSTEXTCOLOR)
+
+AC_MSG_CHECKING([for custom 'intro' progress text baseline])
+PROGRESSTEXTBASELINE=
+if test -z $with_intro_progressbar_text_baseline; then
+   PROGRESSTEXTBASELINE=287
+   AC_MSG_RESULT([none])
+else
+   PROGRESSTEXTBASELINE=$with_intro_progressbar_text_baseline
+   AC_MSG_RESULT([$PROGRESSTEXTBASELINE])
+fi
+AC_SUBST(PROGRESSTEXTBASELINE)
+
 AC_MSG_CHECKING([for alternative branding images directory])
 INTRO_BITMAP=
 ABOUT_BACKGROUND_SVG=
diff --git a/desktop/source/splash/splash.cxx b/desktop/source/splash/splash.cxx
index 0ef211a..6678aa8 100644
--- a/desktop/source/splash/splash.cxx
+++ b/desktop/source/splash/splash.cxx
@@ -88,6 +88,7 @@ private:
 BitmapEx_aIntroBmp;
 Color   _cProgressFrameColor;
 Color   _cProgressBarColor;
+Color   _cProgressTextColor;
 bool_bNativeProgress;
 OUString_sAppName;
 OUString_sProgressText;
@@ -103,7 +104,7 @@ private:
 sal_Bool_bFullScreenSplash;
 sal_Bool_bProgressEnd;
 long _height, _width, _tlx, _tly, _barwidth;
-long _barheight, _barspace;
+long _barheight, _barspace, _textBaseline;
 double _fXPos, _fYPos;
 double _fWidth, _fHeight;
 const long _xoffset, _yoffset;
@@ -132,6 +133,7 @@ SplashScreen::SplashScreen()
 , _vdev(*((IntroWindow*)this))
 , _cProgressFrameColor(sal::static_int_cast ColorData (NOT_LOADED))
 , _cProgressBarColor(sal::static_int_cast ColorData (NOT_LOADED))
+, _cProgressTextColor(sal::static_int_cast ColorData (NOT_LOADED))
 , _bNativeProgress(true)
 , _iMax(100)
 , _iProgress(0)
@@ -146,6 +148,7 @@ SplashScreen::SplashScreen()
 , _barwidth(NOT_LOADED)
 , _barheight(NOT_LOADED)
 , _barspace(2)
+, _textBaseline(NOT_LOADED)
 , _fXPos(-1.0)
 , _fYPos(-1.0)
 , _fWidth(-1.0)
@@ -299,6 +302,9 @@ SplashScreen::initialize( const 
::com::sun::star::uno::Sequence ::com::sun::sta
 }
 }
 
+if ( NOT_LOADED == _textBaseline )
+_textBaseline = _height;
+
 if ( sal::static_int_cast ColorData (NOT_LOADED) ==
  _cProgressFrameColor.GetColor() )
 _cProgressFrameColor = Color( COL_LIGHTGRAY );
@@ -313,6 +319,10 @@ SplashScreen::initialize( const 
::com::sun::star::uno::Sequence ::com::sun::sta
 _cProgressBarColor = Color( COL_BLUE );
 }
 
+if ( sal::static_int_cast ColorData (NOT_LOADED) ==
+ _cProgressTextColor.GetColor() )
+_cProgressTextColor = Color( COL_BLACK );
+
 Application::AddEventListener(
 LINK( this, SplashScreen, AppEventListenerHdl ) );
 
@@ -365,6 +375,10 @@ void SplashScreen::loadConfig()
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressFrameColor ) ) );
 OUString sProgressBarColor = implReadBootstrapKey(
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressBarColor ) ) );
+OUString sProgressTextColor = implReadBootstrapKey(
+OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressTextColor ) ) );
+OUString sProgressTextBaseline = implReadBootstrapKey(
+OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressTextBaseline ) ) );
 OUString sSize = implReadBootstrapKey(
 OUString( RTL_CONSTASCII_USTRINGPARAM( ProgressSize ) ) 

[PATCH] Remove dead code

2012-10-02 Thread Takeshi Abe (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/747

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/47/747/1

Remove dead code

SW_CRSR_TIMER was undefined

Change-Id: Ib851170ea61547e0ceaa6721c0b2fb90fecaecd3
---
M sw/inc/crsrsh.hxx
M sw/inc/viscrs.hxx
M sw/source/core/crsr/viscrs.cxx
M sw/source/core/layout/layact.cxx
M sw/source/ui/docvw/edtwin.cxx
5 files changed, 0 insertions(+), 100 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/747
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib851170ea61547e0ceaa6721c0b2fb90fecaecd3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Takeshi Abe t...@fixedpoint.jp

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH][REVIEW 3-6] fdo#52268: splash: change color and position of messages

2012-10-02 Thread Jan Holesovsky
Hi Ivan,

Ivan Timofeev píše v So 29. 09. 2012 v 21:52 +0400:

 this patch fixes bug 52268:
 https://bugs.freedesktop.org/show_bug.cgi?id=52268
 Splash Screen: Messages on first start displayed in wrong colour in
 the wrong place

Thanks f or that! :-)  Pushed that both to master and libreoffice-3-6.
I'll notify the package maintainers in a separate mail - if they use
different graphics, they might need to change the defaults.

All the best,
Kendy

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread d . ostrovsky

Hi Michael,

Zitat von Michael Meeks michael.me...@suse.com:



On Tue, 2012-10-02 at 14:53 +0200, Michael Stahl wrote:

 Mac OSX 10.4 only comes with 1.5.

 what about to drop support for Mac OS X 10.4 then?

that is on the LO4 todo list :)


Worth adding to the wiki page so we don't forget I imagine:

http://wiki.documentfoundation.org/Development/LibreOffice4


done:

https://wiki.documentfoundation.org/Development/LibreOffice4#Discontinued_support_for_obsolete_platforms_and_libraries

Ciao
David
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread Christian Lohmaier
Hi David, *,

On Tue, Oct 2, 2012 at 2:37 PM,  d.ostrov...@idaia.de wrote:
 Zitat von Christian Lohmaier lohmaier+libreoff...@googlemail.com:
 On Tue, Oct 2, 2012 at 1:27 PM,  d.ostrov...@idaia.de wrote:

 Mac OSX 10.4 only comes with 1.5.

 what about to drop support for Mac OS X 10.4 then?

Needless to say that I won't be happy abount it.

As dropping 10.4 won't bring any benefits in terms of Mac OSX features
to be used, this almost automatically means also dropping 10.5, and
that in turn means dropping PPC support.

ciao
Christian
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread Rene Engelhard
Hi,

On Tue, Oct 02, 2012 at 01:27:21PM +0200, d.ostrov...@idaia.de wrote:
 Hi,
[...]
 Java 7 is today the current version, Java 6 is the old one and Java
 5 is the ancient one. Does it still make sense to support it in LO
 3.7 or even in LO 4.0?
 
 Would it be an option to drop Java 1.5 support (upgrading configure.in and
 all other places) and as the consequence drop stax module?

There's platforms who don't have a JDK (yet) available and *still* need
to use gcj-jdk. Which only can do 1.5.: 
http://www.debian.org/ports/kfreebsd-gnu/

(But that one will get openjdk when wheezy is installed on the Debian
builds - so I am told...)

Regards,

Rene
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread Noel Grandin


On 2012-10-02 15:53, Christian Lohmaier wrote:

Hi David, *,

On Tue, Oct 2, 2012 at 2:37 PM,  d.ostrov...@idaia.de wrote:

Zitat von Christian Lohmaier lohmaier+libreoff...@googlemail.com:

On Tue, Oct 2, 2012 at 1:27 PM,  d.ostrov...@idaia.de wrote:

Mac OSX 10.4 only comes with 1.5.

what about to drop support for Mac OS X 10.4 then?

Needless to say that I won't be happy abount it.

As dropping 10.4 won't bring any benefits in terms of Mac OSX features
to be used, this almost automatically means also dropping 10.5, and
that in turn means dropping PPC support.




At some point in time, we have to balance the amount of development 
resources we have against the number of people running a particular 
platform.


People on that platform, are, of course, welcome to maintain a fork that 
supports that platform.

Git makes this particularly easy.



Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 2 commits - desktop/scripts sw/source

2012-10-02 Thread Libreoffice Gerrit user
 desktop/scripts/gdbtrace  |1 +
 sw/source/filter/html/wrthtml.cxx |   24 
 2 files changed, 25 insertions(+)

New commits:
commit 3751efff81a9bc1e53c8cbee74770bbed6d4f8de
Author: Michael Meeks michael.me...@suse.com
Date:   Mon Oct 1 16:26:54 2012 +0100

initialize various fields in constructor.

Change-Id: Ib2165121786353062a52e6dd65cca6dab8584e60

diff --git a/sw/source/filter/html/wrthtml.cxx 
b/sw/source/filter/html/wrthtml.cxx
index 131c04e..0ebd737 100644
--- a/sw/source/filter/html/wrthtml.cxx
+++ b/sw/source/filter/html/wrthtml.cxx
@@ -99,6 +99,30 @@ static sal_Char sIndentTabs[MAX_INDENT_LEVEL+2] =
 \t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t;
 
 SwHTMLWriter::SwHTMLWriter( const String rBaseURL )
+, bCfgOutStyles( sal_False )
+, bCfgPreferStyles( sal_False )
+, bCfgFormFeed( sal_False )
+, bCfgStarBasic( sal_False )
+, bCfgCpyLinkedGrfs( sal_False )
+, bFirstLine( sal_False )
+, bTagOn( sal_False )
+, bTxtAttr( sal_False )
+, bOutOpts( sal_False )
+, bOutTable( sal_False )
+, bOutHeader( sal_False )
+, bOutFooter( sal_False )
+, bOutFlyFrame( sal_False )
+, bFirstCSS1Rule( sal_False )
+, bFirstCSS1Property( sal_False )
+, bPoolCollTextModified( sal_False )
+, bCSS1IgnoreFirstPageDesc( sal_False )
+, bNoAlign( sal_False )
+, bClearLeft( sal_False )
+, bClearRight( sal_False )
+, bLFPossible( sal_False )
+, bPreserveForm( sal_False )
+, bCfgNetscape4( sal_False )
+
 {
 SetBaseURL( rBaseURL );
 bFirstLine = sal_True;
commit 0ee9572db3934aa4e739c424765d4fd36a59a291
Author: Michael Meeks michael.me...@suse.com
Date:   Mon Oct 1 16:26:35 2012 +0100

ignore various unhelpful signals in our --backtrace gdb setup.

Change-Id: I8ede40ebf5954e0492e790ce148c9b7f1f1e62df

diff --git a/desktop/scripts/gdbtrace b/desktop/scripts/gdbtrace
index 548ffe6..f5fbf63 100644
--- a/desktop/scripts/gdbtrace
+++ b/desktop/scripts/gdbtrace
@@ -1,5 +1,6 @@
 set pagination off
 echo log will be saved as gdbtrace.log, this will take some time, patience...\n
+handle SIGPIPE SIGXCPU SIG33 SIG35 SIGPWR nostop noprint
 set logging redirect on
 set logging file gdbtrace.log
 set logging on
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PATCH] sal_bool and String conversions

2012-10-02 Thread Stephan Bergmann

On 10/02/2012 03:01 PM, Michael Stahl wrote:

On 02/10/12 08:33, Noel Grandin wrote:

Patch 7 and 11 need careful checking because they add methods to
OUStringBuffer and OUString.




diff --git a/sal/inc/rtl/ustring.h b/sal/inc/rtl/ustring.h
index 5b4982e..1595c0f 100644
--- a/sal/inc/rtl/ustring.h
+++ b/sal/inc/rtl/ustring.h
@@ -1500,7 +1500,7 @@ SAL_DLLPUBLIC void SAL_CALL 
rtl_uString_newReplaceFirstAsciiLAsciiL(
  */
  SAL_DLLPUBLIC void SAL_CALL rtl_uString_newReplaceAll(
  rtl_uString ** newStr, rtl_uString * str, rtl_uString const * from,
-rtl_uString const * to) SAL_THROW_EXTERN_C();
+rtl_uString const * to, sal_Int32 fromIndex) SAL_THROW_EXTERN_C();

  /** Create a new string by replacing all occurrences of a given substring with
  another substring.


that is not ABI compatible.


-OUString replaceAll(OUString const  from, OUString const  to) const {
+OUString replaceAll(OUString const  from, OUString const  to, int 
fromIndex = 0) const {


this should be source and binary compatible i think, since it's inline
function?


Yes, changing the parameter list should be safe.  (Though the changes to 
the function's body are not, see above.)



am unsure about this:


+OUStringBuffer  operator+=( const OUString str )
+{
+return append( str.getStr(), str.getLength() );
+}
+
+/**
+Appends the string representation of the codechar/code array
+argument to this string buffer.
+
+The characters of the array argument are appended, in order, to
+the contents of this string buffer. The length of this string
+buffer increases by the length of the argument.
+
+@param   str   the characters to be appended.
+@return  this string buffer.
+@since LibreOffice 3.7
+ */
+OUStringBuffer  operator+=( const sal_Unicode * str )
+{
+return append( str, rtl_ustr_getLength( str ) );
+}



  /**
+Appends the string representation of the ASCII codechar/code
+argument to this string buffer.
+
+The argument is appended to the contents of this string buffer.
+The length of this string buffer increases by code1/code.
+
+@param   c   an ASCII codechar/code.
+@return  this string buffer.
+
+@since LibreOffice 3.7
+ */
+OUStringBuffer  operator+=(char c)
+{
+return append(c);
+}


hmmm... do we really want to have 2 sets of methods append() and
operator+= that do the same thing, possibly with different overloads?

perhaps we should deprecate the append methods... but those often have
addition radix etc, parameters... and is it possible to chain += like
append() in a single statement?


I would just stick to append-only for OUStringBuffer.

(Also, in general, it makes sense to keep changes to oustring/oustrbuf 
in sync with changes to ostring/ostrbuf.)


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Warning for package maintainers: Splash screen

2012-10-02 Thread Jan Holesovsky
Hi,

I have just pushed Ivan's

http://cgit.freedesktop.org/libreoffice/core/commit/?id=b14073218ea28a5bf05cd639e30215787d67114c

to libreoffice-3-6:

http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-3-6id=71b584680e4527283ccc4fa14169bc7d28d32344

If you are using the default graphics, no action is needed.

If you are using your own splash screen, though, you might want to
adjust the baseline of the text that appears during startup using
--with-intro-progressbar-text-baseline and
--with-intro-progressbar-text-color.

To see it on Linux, you need to run ./soffice.bin, instead of ./soffice,
in order to bypass oosplash.

All the best,
Kendy

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: configure.in

2012-10-02 Thread Libreoffice Gerrit user
 configure.in |   14 ++
 1 file changed, 14 insertions(+)

New commits:
commit a9097843e6b2935dcab9f038b80e5a51bc5cfc94
Author: Jan Holesovsky ke...@suse.cz
Date:   Tue Oct 2 16:00:09 2012 +0200

Help text for newly introduced switches.

Change-Id: I3f148d0dd61adc7008fdf64d077818c22a0e7d09

diff --git a/configure.in b/configure.in
index 34dfce7..fb54ec3 100644
--- a/configure.in
+++ b/configure.in
@@ -1980,6 +1980,20 @@ AC_ARG_WITH(intro-progressbar-frame-color,
   Usage: 
--with-intro-progressbar-frame-color=207,208,211
 ],,)
 
+AC_ARG_WITH(intro-progressbar-text-color,
+AS_HELP_STRING([--with-intro-progressbar-text-color],
+[Set color of progress bar text on intro screen. Comma separated RGB 
values in decimal format.])
+[
+  Usage: 
--with-intro-progressbar-text-color=207,208,211
+],,)
+
+AC_ARG_WITH(intro-progressbar-text-baseline,
+AS_HELP_STRING([--with-intro-progressbar-text-baseline],
+[Set horizontal position of progress bar text on intro screen. Value 
in decimal format.])
+[
+  Usage: --with-intro-progressbar-text-baseline=250
+],,)
+
 AC_ARG_WITH(flat-logo-svg,
 [  --with-flat-logo-svgAllows specification of the flat Logo SVG.
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Re: [PATCH][REVIEW 3-6] fdo#52268: splash: change color and position of messages

2012-10-02 Thread Jan Holesovsky
Hi Ivan,

Marking it pushed; and want to ask you to cherry-pick the following
addition to the libreoffice-3-6 branch :-)

http://cgit.freedesktop.org/libreoffice/core/commit/?id=a9097843e6b2935dcab9f038b80e5a51bc5cfc94

Thank you,
Kendy

Jan Holesovsky píše v Út 02. 10. 2012 v 15:46 +0200:
 Hi Ivan,
 
 Ivan Timofeev píše v So 29. 09. 2012 v 21:52 +0400:
 
  this patch fixes bug 52268:
  https://bugs.freedesktop.org/show_bug.cgi?id=52268
  Splash Screen: Messages on first start displayed in wrong colour in
  the wrong place
 
 Thanks f or that! :-)  Pushed that both to master and libreoffice-3-6.
 I'll notify the package maintainers in a separate mail - if they use
 different graphics, they might need to change the defaults.
 
 All the best,
 Kendy
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: xmloff/source

2012-10-02 Thread Libreoffice Gerrit user
 xmloff/source/text/XMLTextMasterPageExport.cxx |   14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

New commits:
commit d92345561c998f7382cf9ef0fdcd29096f978435
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Oct 2 14:41:59 2012 +0100

don't blow up on .odt export with !hasValue footers/headers

Change-Id: I2507c23b7b17b9172f720548d32cdfe299d4fa20

diff --git a/xmloff/source/text/XMLTextMasterPageExport.cxx 
b/xmloff/source/text/XMLTextMasterPageExport.cxx
index a449f92..ab7f854 100644
--- a/xmloff/source/text/XMLTextMasterPageExport.cxx
+++ b/xmloff/source/text/XMLTextMasterPageExport.cxx
@@ -127,20 +127,21 @@ void XMLTextMasterPageExport::exportMasterPageContent(
 else
 {
 aAny = rPropSet-getPropertyValue( sHeaderOn );
-sal_Bool bHeader = *(sal_Bool *)aAny.getValue();
+sal_Bool bHeader = sal_False;
+aAny = bHeader;
 
 sal_Bool bHeaderFirst = sal_False;
 if( bHeader )
 {
 aAny = rPropSet-getPropertyValue( sFirstShareContent );
-bHeaderFirst = !*(sal_Bool *)aAny.getValue();
+aAny = bHeaderFirst;
 }
 
 sal_Bool bHeaderLeft = sal_False;
 if( bHeader )
 {
 aAny = rPropSet-getPropertyValue( sHeaderShareContent );
-bHeaderLeft = !*(sal_Bool *)aAny.getValue();
+aAny = bHeaderLeft;
 }
 
 if( xHeaderText.is() )
@@ -174,20 +175,21 @@ void XMLTextMasterPageExport::exportMasterPageContent(
 }
 
 aAny = rPropSet-getPropertyValue( sFooterOn );
-sal_Bool bFooter = *(sal_Bool *)aAny.getValue();
+sal_Bool bFooter = sal_False;
+aAny = bFooter;
 
 sal_Bool bFooterFirst = sal_False;
 if( bFooter )
 {
 aAny = rPropSet-getPropertyValue( sFirstShareContent );
-bFooterFirst = !*(sal_Bool *)aAny.getValue();
+aAny = bFooterFirst;
 }
 
 sal_Bool bFooterLeft = sal_False;
 if( bFooter )
 {
 aAny = rPropSet-getPropertyValue( sFooterShareContent );
-bFooterLeft = !*(sal_Bool *)aAny.getValue();
+aAny = bFooterLeft;
 }
 
 if( xFooterText.is() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/source/filter/html/wrthtml.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 4afd166c229e27d741b48a6f099d2f00d01b05de
Author: Jan Holesovsky ke...@suse.cz
Date:   Tue Oct 2 16:10:46 2012 +0200

Fix build.

Change-Id: I291cb5422f5338d64e6d8973c8dc863f4fb982f5

diff --git a/sw/source/filter/html/wrthtml.cxx 
b/sw/source/filter/html/wrthtml.cxx
index 0ebd737..baba51c 100644
--- a/sw/source/filter/html/wrthtml.cxx
+++ b/sw/source/filter/html/wrthtml.cxx
@@ -99,7 +99,7 @@ static sal_Char sIndentTabs[MAX_INDENT_LEVEL+2] =
 \t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t;
 
 SwHTMLWriter::SwHTMLWriter( const String rBaseURL )
-, bCfgOutStyles( sal_False )
+: bCfgOutStyles( sal_False )
 , bCfgPreferStyles( sal_False )
 , bCfgFormFeed( sal_False )
 , bCfgStarBasic( sal_False )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: ww8export tinderbox failure ?

2012-10-02 Thread Jan Holesovsky
Hi Caolan,

Caolán McNamara píše v St 26. 09. 2012 v 16:55 +0100:

   Getting a backtrace with symbols would indeed help, I guess.
  
   Didn't help me. IIRC the crash is because a component somewhere is not 
  loaded, leading to a NULL pointer dereference later. And the component is 
  not 
  loaded, because the GUID read for it from some OLE stream is rubbish. I got 
  rather lost at that point and gave up in the hope somebody else would fix 
  it.
 
 And I presume 
 VALGRIND=memcheck
 make 
 /local/home/tinderbox/master/workdir/unxlngx6.pro/CppunitTest/sw_subsequent_ww8export.test
 didn't give any hints either ?

Got the attached once, but then nothing when I tried with a sw rebuilt
with debug=true to get symbols; hopefully it will be of some use at
least for a wild guess...

All the best,
Kendy
==14856== Memcheck, a memory error detector
==14856== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==14856== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==14856== Command: /local/libreoffice/master-linux/solver/unxlngx6.pro/bin/cppunit/cppunittester /local/libreoffice/master-linux/workdir/unxlngx6.pro/LinkTarget/CppunitTest/libtest_sw_subsequent_ww8export.so --headless -env:LIBLANGTAG_SHARE=file:///local/libreoffice/master-linux/solver/unxlngx6.pro -env:CONFIGURATION_LAYERS=xcsxcu:file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/registry\ module:file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/registry/spool\ xcsxcu:file:///local/libreoffice/master-linux/solver/unxlngx6.pro/unittest/registry -env:UNO_TYPES=file:///local/libreoffice/master-linux/solver/unxlngx6.pro/bin/offapi.rdb\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/bin/udkapi.rdb -env:UNO_SERVICES=file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/ure/services.rdb\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/basic/util/sb.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/comphelper/util/comphelp.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/configmgr/source/configmgr.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/dbaccess/util/dba.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/fileaccess/source/fileacc.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/filter/source/config/cache/filterconfig1.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/forms/util/frm.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/framework/util/fwk.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/i18npool/util/i18npool.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/package/source/xstor/xstor.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/package/util/package2.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/sax/source/expatwrap/expwrap.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/sw/util/msword.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/sw/util/sw.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/sw/util/swd.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/sfx2/util/sfx.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/svl/source/fsstor/fsstorage.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/svtools/util/svt.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/toolkit/util/tk.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/ucb/source/core/ucb1.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/ucb/source/ucp/file/ucpfile1.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/unotools/util/utl.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/unoxml/source/service/unoxml.component\ file:///local/libreoffice/master-linux/solver/unxlngx6.pro/xml/component/xmlhelp/util/ucpchelp1.component -env:URE_INTERNAL_LIB_DIR=file:///local/libreoffice/master-linux/solver/unxlngx6.pro/lib -env:LO_LIB_DIR=file:///local/libreoffice/master-linux/solver/unxlngx6.pro/lib --protector unoexceptionprotector.so unoexceptionprotector --protector unobootstrapprotector.so unobootstrapprotector
==14856== 
(process:14856): LangTag-DEBUG: Loading the internal extension handler: default
(process:14856): LangTag-DEBUG: Loading the internal extension handler: empty
(process:14856): LangTag-DEBUG: Loading the internal extension handler: wildcard
==14856== Conditional jump or move depends on uninitialised value(s)
==14856==at 

Re: [PATCH] sal_bool and String conversions

2012-10-02 Thread Noel Grandin


On 2012-10-02 16:00, Stephan Bergmann wrote:



am unsure about this:


+OUStringBuffer  operator+=( const OUString str )
+{
+return append( str.getStr(), str.getLength() );
+}
+
+/**
+Appends the string representation of the codechar/code 
array

+argument to this string buffer.
+
+The characters of the array argument are appended, in 
order, to

+the contents of this string buffer. The length of this string
+buffer increases by the length of the argument.
+
+@param   str   the characters to be appended.
+@return  this string buffer.
+@since LibreOffice 3.7
+ */
+OUStringBuffer  operator+=( const sal_Unicode * str )
+{
+return append( str, rtl_ustr_getLength( str ) );
+}



  /**
+Appends the string representation of the ASCII 
codechar/code

+argument to this string buffer.
+
+The argument is appended to the contents of this string 
buffer.

+The length of this string buffer increases by code1/code.
+
+@param   c   an ASCII codechar/code.
+@return  this string buffer.
+
+@since LibreOffice 3.7
+ */
+OUStringBuffer  operator+=(char c)
+{
+return append(c);
+}


hmmm... do we really want to have 2 sets of methods append() and
operator+= that do the same thing, possibly with different overloads?

perhaps we should deprecate the append methods... but those often have
addition radix etc, parameters... and is it possible to chain += like
append() in a single statement?


I would just stick to append-only for OUStringBuffer.

(Also, in general, it makes sense to keep changes to oustring/oustrbuf 
in sync with changes to ostring/ostrbuf.)




Part of my motivation for adding this, besides making String-OUString 
conversion easier, was that OUString already has the operator+= methods.




Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Drop support for Java 5?

2012-10-02 Thread Michael Stahl
On 02/10/12 15:48, d.ostrov...@idaia.de wrote:
 Hi Michael,
 
 Zitat von Michael Meeks michael.me...@suse.com:
 

 On Tue, 2012-10-02 at 14:53 +0200, Michael Stahl wrote:
 Mac OSX 10.4 only comes with 1.5.

 what about to drop support for Mac OS X 10.4 then?

 that is on the LO4 todo list :)

what i meant to say is that i had added this to the below wiki page some
months ago...

  Worth adding to the wiki page so we don't forget I imagine:

  http://wiki.documentfoundation.org/Development/LibreOffice4

 done:
 
 https://wiki.documentfoundation.org/Development/LibreOffice4#Discontinued_support_for_obsolete_platforms_and_libraries

...but apparently both of you couldn't find it on there so i guess it's
and improvement that the topic now has its own header :)

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'feature/submodules' - configure.in

2012-10-02 Thread Libreoffice Gerrit user
 configure.in |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 20116cc0b1ff8a6dfe848676bc77866566e78f38
Author: Norbert Thiebaud nthieb...@gmail.com
Date:   Tue Oct 2 10:09:30 2012 -0500

typo in configure.in wrt the name of the helpcontent2 module

Change-Id: Id0d7335872c2e2c074769bd7edc1b7301b1ebdbe

diff --git a/configure.in b/configure.in
index e2f1744..10e1b1d 100644
--- a/configure.in
+++ b/configure.in
@@ -4292,7 +4292,7 @@ if test $with_help != no -a $_os != iOS -a $_os != 
Android; then
 BUILD_TYPE=$BUILD_TYPE HELP
 SCPDEFS=$SCPDEFS -DWITH_HELP
 GIT_REPO_NAMES=$GIT_REPO_NAMES help
-GIT_NEEDED_SUBMODULES=helcontent2 $GIT_NEEDED_SUBMODULES
+GIT_NEEDED_SUBMODULES=helpcontent2 $GIT_NEEDED_SUBMODULES
 else
 AC_MSG_RESULT([no])
 fi
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: FDO Font Size (Email)

2012-10-02 Thread Korrawit Pruegsanusak
Hello Joel, all,

On Tue, Oct 2, 2012 at 10:05 PM, Joel Madero jmadero@gmail.com wrote:
 Anyone else seeing that the updated FDO font size for comments (for bugs you
 are CC'ed on) are tiny?

Me too! :-)

I just found https://bugs.freedesktop.org/userprefs.cgi, that you may
set Preferred email format to Text Only. This should help, but I
didn't try yet. So, could you please report back if it really helps?
;) Thanks.

Best Regards,
-- 
Korrawit Pruegsanusak
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'feature/submodules' - g

2012-10-02 Thread Libreoffice Gerrit user
 g |   11 +++
 1 file changed, 11 insertions(+)

New commits:
commit 1b33fd35e6ca50dd229fa3e9af084a82b7cd8cd1
Author: Norbert Thiebaud nthieb...@gmail.com
Date:   Tue Oct 2 10:32:45 2012 -0500

when switching to a pre-submodule point, clean-up the soft-link point

Change-Id: I5a8f144bb4ad167cfa0e6763e3cf273b9127d65b

diff --git a/g b/g
index d73b8a6..7c6551c 100755
--- a/g
+++ b/g
@@ -137,6 +137,7 @@ do_checkout()
 local cmd
 local create_branch=0
 local branch
+local module
 
 git checkout $@ || return $?
 for cmd in $@ ; do
@@ -154,6 +155,11 @@ local branch
fi
 else
# now that is the nasty case we moved prior to submodules
+   # delete the submodules left over if any
+   for module in $SUBMODULES_ALL ; do
+   echo clean-up submodule $module
+   rm -fr ${module}
+   done
# make sure we have the needed repo in clone
./g clone  ./g -f checkout $@ || return $?
 fi
@@ -167,6 +173,11 @@ do_reset()
git submodule update || return $?
 else
# now that is the nasty case we moved prior to submodules
+   # delete the submodules left over if any
+   for module in $SUBMODULES_ALL ; do
+   echo clean-up submodule $module
+   rm -fr ${module}
+   done
# make sure we have the needed repo in clone
./g clone  ./g -f reset $@
 fi
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - vcl/aqua vcl/inc

2012-10-02 Thread Libreoffice Gerrit user
 vcl/aqua/source/gdi/atsui/salatslayout.cxx |2 +-
 vcl/aqua/source/gdi/atsui/salgdi.cxx   |   29 ++---
 vcl/inc/aqua/atsui/salgdi.h|1 -
 3 files changed, 3 insertions(+), 29 deletions(-)

New commits:
commit da9a26f5f49ea56422bb791d4f96725283d3e8c6
Author: Sergey Farbotka z8serge...@gmail.com
Date:   Tue Oct 2 17:36:09 2012 +0200

Fix fdo#55142 - use proper output offsets.

Change-Id: I76e6158f1ac64b9dd555cf8aee3ce8bdd5d035c5

diff --git a/vcl/aqua/source/gdi/atsui/salatslayout.cxx 
b/vcl/aqua/source/gdi/atsui/salatslayout.cxx
index 4145dbf..5d9ad09 100644
--- a/vcl/aqua/source/gdi/atsui/salatslayout.cxx
+++ b/vcl/aqua/source/gdi/atsui/salatslayout.cxx
@@ -437,7 +437,7 @@ void ATSLayout::DrawText( SalGraphics rGraphics ) const
 {
 const SubPortion rSubPortion = *it;
 // calculate sub-portion offset for rotated text
-Fixed nXOfsFixed = 0, nYOfsFixed = 0;
+Fixed nXOfsFixed = rSubPortion.mnXOffset, nYOfsFixed = 0;
 if( rAquaGraphics.mnATSUIRotation != 0 )
 {
 const double fRadians = rAquaGraphics.mnATSUIRotation * 
(M_PI/0xB4);
commit d6418f680b8861dfea1ce1be6e9358b23cfb3064
Author: Thorsten Behrens tbehr...@suse.com
Date:   Tue Oct 2 17:31:56 2012 +0200

Revert Fix fdo#55142 - wrong character pos calculation on mac.

Caused fdo#55350, better fix in the next commit.

This reverts commit f1651ee579bf38a49b839dbcb28ac38c0163a3d2.

diff --git a/vcl/aqua/source/gdi/atsui/salgdi.cxx 
b/vcl/aqua/source/gdi/atsui/salgdi.cxx
index a57068d..5c0bff7 100644
--- a/vcl/aqua/source/gdi/atsui/salgdi.cxx
+++ b/vcl/aqua/source/gdi/atsui/salgdi.cxx
@@ -354,8 +354,6 @@ AquaSalGraphics::AquaSalGraphics()
 {
 // create the style object for font attributes
 ATSUCreateStyle( maATSUStyle );
-
-ResetFontStyle();
 }
 
 // ---
@@ -581,29 +579,6 @@ bool AquaSalGraphics::AddTempDevFont( ImplDevFontList*,
 
 // ---
 
-void AquaSalGraphics::ResetFontStyle()
-{
-ATSUClearStyle(maATSUStyle);
-
-// Set justification attributes
-ATSJustPriorityWidthDeltaOverrides nPriorityJustOverrides;
-memset(nPriorityJustOverrides, 0, sizeof(nPriorityJustOverrides));
-
-nPriorityJustOverrides[kJUSTLetterPriority].growFlags = 
kJUSTOverrideLimits;
-nPriorityJustOverrides[kJUSTLetterPriority].shrinkFlags = 
kJUSTOverrideLimits;
-
-ATSUAttributeTagtheTag = kATSUPriorityJustOverrideTag;
-ByteCount   theSize = 
sizeof(ATSJustPriorityWidthDeltaOverrides);
-ATSUAttributeValuePtr   thePtr = nPriorityJustOverrides;
-OSStatus eStatus = ATSUSetAttributes(maATSUStyle, 1, theTag, theSize, 
thePtr);
-if (eStatus != noErr)
-{
-DBG_WARNING(AquaSalGraphics::ResetFontStyle() : Could not override 
justification attributes!\n);
-}
-}
-
-// ---
-
 // callbacks from ATSUGlyphGetCubicPaths() fore GetGlyphOutline()
 struct GgoData { basegfx::B2DPolygon maPolygon; basegfx::B2DPolyPolygon* 
mpPolyPoly; };
 
@@ -723,7 +698,7 @@ sal_uInt16 AquaSalGraphics::SetFont( FontSelectPattern* 
pReqFont, int /*nFallbac
 {
 if( !pReqFont )
 {
-ResetFontStyle();
+ATSUClearStyle( maATSUStyle );
 mpMacFontData = NULL;
 return 0;
 }
@@ -809,7 +784,7 @@ sal_uInt16 AquaSalGraphics::SetFont( FontSelectPattern* 
pReqFont, int /*nFallbac
 if( eStatus != noErr )
 {
 DBG_WARNING( AquaSalGraphics::SetFont() : Could not set font 
attributes!\n);
-ResetFontStyle();
+ATSUClearStyle( maATSUStyle );
 mpMacFontData = NULL;
 return 0;
 }
diff --git a/vcl/inc/aqua/atsui/salgdi.h b/vcl/inc/aqua/atsui/salgdi.h
index eeb7aa4..19fe1f4 100644
--- a/vcl/inc/aqua/atsui/salgdi.h
+++ b/vcl/inc/aqua/atsui/salgdi.h
@@ -353,7 +353,6 @@ private:
 bool GetRawFontData( const PhysicalFontFace* pFontData,
  std::vectorunsigned char rBuffer,
  bool* pJustCFF );
-void ResetFontStyle();
 };
 
 // --- some trivial inlines
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Drop support for Java 5?

2012-10-02 Thread Michael Meeks

On Tue, 2012-10-02 at 15:57 +0200, Noel Grandin wrote:
 At some point in time, we have to balance the amount of development 
 resources we have against the number of people running a particular 
 platform.

True - but is there any real, concrete win to dropping it - beyond not
having to build a single external module, that we already build ? ;-)
Clearly we need a cost/benefit on both sides.

MacOS PPC is around 0.7% of total downloads, small but not
insignificant - clearly many tens-of-thousands of people are using it.

ATB,

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 54157] LibreOffice 3.7 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Bug 54157 depends on bug 55350, which changed state.

Bug 55350 Summary: Letter spacing and expand single word do not work anymore 
on MacOSX
https://bugs.freedesktop.org/show_bug.cgi?id=55350

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 2 commits - nss/prj soltools/Executable_makedepend.mk

2012-10-02 Thread Libreoffice Gerrit user
 nss/prj/build.lst |2 +-
 soltools/Executable_makedepend.mk |4 
 2 files changed, 5 insertions(+), 1 deletion(-)

New commits:
commit 9b95cd5ada9aeb92eaf29bd8a03dfa186bc79ac2
Author: Andras Timar ati...@suse.com
Date:   Tue Oct 2 17:47:11 2012 +0200

nss depends on soltools

Change-Id: Ic532b35cf282186843d886fda578bb20845db415

diff --git a/nss/prj/build.lst b/nss/prj/build.lst
index 14963ff..2377ac2 100644
--- a/nss/prj/build.lst
+++ b/nss/prj/build.lst
@@ -1,3 +1,3 @@
-ns nss : solenv NULL
+ns nss : solenv soltools NULL
 ns nss nmake   -   usr1
ns_mkout NULL
 ns nss nmake   -   all 
ns_nss NULL
commit 960e57178a81688fef65d3fe339afaa5e3bf2afb
Author: Andras Timar ati...@suse.com
Date:   Tue Oct 2 17:45:35 2012 +0200

makedepend depends on unistd.h

Change-Id: I02df1b67dd9470a4a7225fd8986a52bd88a86e93

diff --git a/soltools/Executable_makedepend.mk 
b/soltools/Executable_makedepend.mk
index e301a09..96a3682 100644
--- a/soltools/Executable_makedepend.mk
+++ b/soltools/Executable_makedepend.mk
@@ -13,6 +13,10 @@ $(eval $(call 
gb_Executable_add_exception_objects,makedepend,\
 soltools/mkdepend/collectdircontent \
 ))
 
+$(eval $(call gb_Executable_use_package,makedepend,\
+soltools_inc \
+)) 
+
 $(eval $(call gb_Executable_add_defs,makedepend,\
 -DNO_X11 \
 -DXP_PC \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 44446] LibreOffice 3.6 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Michael Meeks michael.me...@novell.com changed:

   What|Removed |Added

 Depends on||46278

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: FDO Font Size (Email)

2012-10-02 Thread Lionel Elie Mamane
Ah, the bugzilla mails are multipart/alternative plain+html now!
*That's* why they suddenly started to go to spam for me!

On Tue, Oct 02, 2012 at 08:40:19AM -0700, Joel Madero wrote:
 I like the HTML style :( But I'll try the text one, as soon as someone
 comments on one of the hundred(s) of bugs that I'm a CC on I'll report back
 with a report.
 
 
 Regards,
 Joel
 
 On Tue, Oct 2, 2012 at 8:16 AM, Korrawit Pruegsanusak 
 detective.conan.1...@gmail.com wrote:
 
  Hello Joel, all,
 
  On Tue, Oct 2, 2012 at 10:05 PM, Joel Madero jmadero@gmail.com
  wrote:
   Anyone else seeing that the updated FDO font size for comments (for bugs
  you
   are CC'ed on) are tiny?
 
  Me too! :-)
 
  I just found https://bugs.freedesktop.org/userprefs.cgi, that you may
  set Preferred email format to Text Only. This should help, but I
  didn't try yet. So, could you please report back if it really helps?
  ;) Thanks.
 
  Best Regards,
  --
  Korrawit Pruegsanusak
 
 
 
 

 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: configure.in

2012-10-02 Thread Libreoffice Gerrit user
 configure.in |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit b31f9715ff0682b6ed4dcf230bf263da4468b00b
Author: Luboš Luňák l.lu...@suse.cz
Date:   Tue Oct 2 18:21:46 2012 +0200

poppler 0.20.0 makes sdext test fail, fail already in configure

But I expect this was too much work to add when the first person
ran into this.

Change-Id: I8cc84ddec2ffaa76b408f9fb7102ab694347099d

diff --git a/configure.in b/configure.in
index fb54ec3..fba3bc2 100644
--- a/configure.in
+++ b/configure.in
@@ -9703,6 +9703,9 @@ if test x$enable_ext_pdfimport != xno -a 
x$enable_extension_integration !=
   AC_MSG_RESULT([external])
   SYSTEM_POPPLER=YES
   PKG_CHECK_MODULES( POPPLER, poppler = 0.8.0 )
+  # sdext test fails with 0.20.0
+  PKG_CHECK_MODULES( POPPLER_NOT_0_20_0, poppler != 0.20.0,
+, [AC_MSG_ERROR([Poppler 0.20.0 is broken, do not use 
--with-system-poppler or use a different version])])
   AC_LANG_PUSH([C++])
   save_CXXFLAGS=$CXXFLAGS
   save_CPPFLAGS=$CPPFLAGS
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Remove not used macros from lotusworkpro

2012-10-02 Thread Marcos Souza (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/749

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/49/749/1

Remove not used macros from lotusworkpro

Change-Id: I0c5b52030e9837f5ee9aa6258b6ccb16dd3f03c0
Signed-off-by: Marcos Paulo de Souza marcos.souza@gmail.com
---
M lotuswordpro/source/filter/bentoid.hxx
M lotuswordpro/source/filter/lwpdoc.hxx
M lotuswordpro/source/filter/lwpfribheader.hxx
M lotuswordpro/source/filter/lwpgrfobj.cxx
M lotuswordpro/source/filter/lwplayout.hxx
M lotuswordpro/source/filter/lwpoleobject.hxx
M lotuswordpro/source/filter/xfilter/xfframe.cxx
7 files changed, 2 insertions(+), 63 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/749
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0c5b52030e9837f5ee9aa6258b6ccb16dd3f03c0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza marcos.souza@gmail.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[WIKI] new RSS extension

2012-10-02 Thread Florian Effenberger

Hello,

in course of our wiki migration, the current SimpleFeed extension will 
not be available anymore. Therefore, several pages in the wiki, 
especially the EasyHacks page, need to be adapted.


Pages can be identified as they use the feed tag.

I have installed the RSS extension on the soon-to-be-new wiki, and also 
on the old wiki, so appropriate measures can be taken in time. As soon 
as the migration is done (probably this month), the old extension will 
not be available anymore.


Please have a look at your pages, and adjust them according to the new 
extension: http://www.mediawiki.org/wiki/Extension:RSS#Example


Thanks,
Florian

--
Florian Effenberger, Chairman of the Board (Vorstandsvorsitzender)
Tel: +49 8341 99660880 | Mobile: +49 151 14424108
The Document Foundation, Zimmerstr. 69, 10117 Berlin, Germany
Gemeinnützige rechtsfähige Stiftung des bürgerlichen Rechts
Legal details: http://www.documentfoundation.org/imprint
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [REVIEW-3-6] 3x calc cond-format patches

2012-10-02 Thread Kohei Yoshida

On 10/01/2012 01:07 PM, Markus Mohrhard wrote:

Hey,

2012/10/1 Michael Meeks michael.me...@suse.com:

Hi guys,

 These three together (worked on by Markus  myself) improve some of the
ergonomics around the conditional formatting dialog, and fix a minor
sizing issue.

 I'd love to get them into -3-6 - review/sign-off appreciated; the
slightly larger fix is a squash of three from master.




Patch 1 and 2 already have a sign-off from me I just forgot to push my
3-6. I'll do that in some minutes. I can't sign-off the third patch
because it contains some parts of my work.


You can use my sign-off for the 3rd patch if you need to.

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH] sal_bool and String conversions

2012-10-02 Thread Stephan Bergmann

On 10/02/2012 04:22 PM, Noel Grandin wrote:

Part of my motivation for adding this, besides making String-OUString
conversion easier, was that OUString already has the operator+= methods.


Yes, but OUStringBuffer only has append() until now, and---as Michael 
already pointed out---some append() overloads take multiple parameters, 
so cannot map naturally to operator+=.  So I see more value in keeping 
the append() vs. operator+= difference between OUStringBuffer and 
OUString, than in removing that difference but instead introducing 
differences between what OUStringBuffer::append() supports vs. what any 
new OUStringBuffer::operator+= would support.


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 7 commits - configure.in desktop/source starmath/inc starmath/source

2012-10-02 Thread Libreoffice Gerrit user
 configure.in |2 -
 desktop/source/splash/splash.cxx |1 
 starmath/inc/edit.hxx|4 +-
 starmath/inc/utility.hxx |   10 +-
 starmath/source/cursor.cxx   |2 -
 starmath/source/edit.cxx |   64 ++-
 starmath/source/mathmlexport.cxx |2 -
 starmath/source/mathmlimport.cxx |3 -
 starmath/source/unomodel.cxx |   18 --
 starmath/source/utility.cxx  |   29 +
 starmath/source/view.cxx |   12 +--
 11 files changed, 58 insertions(+), 89 deletions(-)

New commits:
commit 2d25eea6949196d2275d8b2bd484e6dab0163a33
Author: Ivan Timofeev timofeev@gmail.com
Date:   Tue Oct 2 20:53:27 2012 +0400

splash screen: fix RTL mode

Change-Id: I6a00cba8fda11351ed5340f21678c330ce028a2d

diff --git a/desktop/source/splash/splash.cxx b/desktop/source/splash/splash.cxx
index e55b5b2..95cdc67 100644
--- a/desktop/source/splash/splash.cxx
+++ b/desktop/source/splash/splash.cxx
@@ -155,6 +155,7 @@ SplashScreen::SplashScreen()
 , _yoffset(18)
 {
 loadConfig();
+_vdev.EnableRTL(IsRTLEnabled());
 }
 
 SplashScreen::~SplashScreen()
commit 509d964e985b58ddc4c20ccc163809a6f3623d8a
Author: Ivan Timofeev timofeev@gmail.com
Date:   Tue Oct 2 20:43:41 2012 +0400

correct help string: horizontal - vertical

Change-Id: Id67da9d90a8fa21b99733c448fc32e81c6f1ab46

diff --git a/configure.in b/configure.in
index fba3bc2..2974db0 100644
--- a/configure.in
+++ b/configure.in
@@ -1989,7 +1989,7 @@ AC_ARG_WITH(intro-progressbar-text-color,
 
 AC_ARG_WITH(intro-progressbar-text-baseline,
 AS_HELP_STRING([--with-intro-progressbar-text-baseline],
-[Set horizontal position of progress bar text on intro screen. Value 
in decimal format.])
+[Set vertical position of progress bar text on intro screen. Value in 
decimal format.])
 [
   Usage: --with-intro-progressbar-text-baseline=250
 ],,)
commit 86b9e14bc8fed823ae39ebc5fee62e98c524d3fe
Author: Ivan Timofeev timofeev@gmail.com
Date:   Tue Oct 2 16:59:26 2012 +0400

String - OUString

Change-Id: I88873eec738a068152e7d7be0dc4ab86fcfe345a

diff --git a/starmath/inc/edit.hxx b/starmath/inc/edit.hxx
index 67e6574..be7b52d 100644
--- a/starmath/inc/edit.hxx
+++ b/starmath/inc/edit.hxx
@@ -113,7 +113,7 @@ public:
 voidPaste();
 voidDelete();
 voidSelectAll();
-voidInsertText(const String rText);
+voidInsertText(const OUString rText);
 voidInsertCommand(sal_uInt16 nCommand);
 voidMarkError(const Point rPos);
 voidSelNextMark();
diff --git a/starmath/source/cursor.cxx b/starmath/source/cursor.cxx
index c3514ad..8ba0529 100644
--- a/starmath/source/cursor.cxx
+++ b/starmath/source/cursor.cxx
@@ -484,7 +484,7 @@ SmNodeList::iterator 
SmCursor::TakeSelectedNodesFromList(SmNodeList *pLineList,
 }
 //If we need to save the selected text
 if(pSelectedNodes  len2  0) {
-String str = aText.copy(start2, len2);
+OUString str = aText.copy(start2, len2);
 SmTextNode* pSeg2 = new SmTextNode(aToken, eFontDesc);
 pSeg2-ChangeText(str);
 pSelectedNodes-push_back(pSeg2);
diff --git a/starmath/source/edit.cxx b/starmath/source/edit.cxx
index 1c0510a..b6d83e0 100644
--- a/starmath/source/edit.cxx
+++ b/starmath/source/edit.cxx
@@ -912,12 +912,12 @@ void SmEditWindow::Delete()
 }
 }
 
-void SmEditWindow::InsertText(const String Text)
+void SmEditWindow::InsertText(const OUString rText)
 {
 OSL_ENSURE( pEditView, EditView missing );
 if (pEditView)
 {
-pEditView-InsertText(Text);
+pEditView-InsertText(rText);
 aModifyTimer.Start();
 StartCursorMove();
 }
diff --git a/starmath/source/view.cxx b/starmath/source/view.cxx
index 5707439..2f7e0a7 100644
--- a/starmath/source/view.cxx
+++ b/starmath/source/view.cxx
@@ -1466,8 +1466,6 @@ bool SmViewShell::Insert( SfxMedium rMedium )
 RTL_LOGFILE_CONTEXT( aLog, starmath: SmViewShell::Insert );
 
 SmDocShell *pDoc = GetDoc();
-String aText( pDoc-GetText() );
-String aTemp = aText;
 bool bRet = false;
 
 uno::Reference  embed::XStorage  xStorage = rMedium.GetStorage();
@@ -1485,15 +1483,13 @@ bool SmViewShell::Insert( SfxMedium rMedium )
 
 if( bRet )
 {
-aText = pDoc-GetText();
+OUString aText = pDoc-GetText();
 SmEditWindow *pEditWin = GetEditWindow();
 if (pEditWin)
 pEditWin-InsertText( aText );
 else
 {
 OSL_FAIL( EditWindow missing );
-aTemp += aText;
-aText  = aTemp;
 }
 
 pDoc-Parse();
@@ -1514,8 +1510,6 @@ bool 

Re: [PUSHED] Re: [PATCH][REVIEW 3-6] fdo#52268: splash: change color and position of messages

2012-10-02 Thread Ivan Timofeev
Hi Kendy,

On 02.10.2012 18:03, Jan Holesovsky wrote:
 Marking it pushed; and want to ask you to cherry-pick the following
 addition to the libreoffice-3-6 branch :-)
 
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=a9097843e6b2935dcab9f038b80e5a51bc5cfc94

oh, right, I forgot about that, thanks! You have my sign-off.

 +AC_ARG_WITH(intro-progressbar-text-baseline,
 +AS_HELP_STRING([--with-intro-progressbar-text-baseline],
 +[Set horizontal position of progress bar text on intro screen. Value 
 in decimal format.])
 +[
 +  Usage: --with-intro-progressbar-text-baseline=250
 +],,)

It should be vertical position I believe. :)
http://cgit.freedesktop.org/libreoffice/core/commit/?id=509d964e985b58ddc4c20ccc163809a6f3623d8a

Horizontally the text is left-aligned to the progress bar, it is not
configurable. Wait... I said left-aligned? How about RTL? Yeah, it is
broken! :)
http://cgit.freedesktop.org/libreoffice/core/commit/?id=2d25eea6949196d2275d8b2bd484e6dab0163a33

Could you please cherry-pick these two commits to 3-6 as well?

Thanks!

Ivan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - oox/source sc/source

2012-10-02 Thread Libreoffice Gerrit user
 oox/source/export/vmlexport.cxx |3 ++-
 sc/source/filter/oox/commentsbuffer.cxx |3 +--
 sc/source/filter/xcl97/xcl97rec.cxx |2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit ebc8ebaff65ac02d410c8e9e39c2776131955cd7
Author: Noel Power noel.po...@novell.com
Date:   Tue Oct 2 18:04:55 2012 +0100

fdo#46278 - misc comment import/export fixes

detect whether note/comment is shown on import
export state of note ( shown/hidden )

parts of Change-Id: I59b446175217479ce7960287aa540df8c6b2b1e
Signed-off-by: Michael Meeks michael.me...@suse.com

diff --git a/oox/source/export/vmlexport.cxx b/oox/source/export/vmlexport.cxx
index 62933fe..8c758b4 100644
--- a/oox/source/export/vmlexport.cxx
+++ b/oox/source/export/vmlexport.cxx
@@ -666,7 +666,8 @@ void VMLExport::Commit( EscherPropertyContainer rProps, 
const Rectangle rRect
 break;
 
 case ESCHER_Prop_fHidden:
-m_pShapeStyle-append( ;visibility:hidden );
+if ( !it-nPropValue )
+m_pShapeStyle-append( ;visibility:hidden );
 break;
 default:
 #if OSL_DEBUG_LEVEL  0
diff --git a/sc/source/filter/oox/commentsbuffer.cxx 
b/sc/source/filter/oox/commentsbuffer.cxx
index 3bbf751..39d57a0 100644
--- a/sc/source/filter/oox/commentsbuffer.cxx
+++ b/sc/source/filter/oox/commentsbuffer.cxx
@@ -195,8 +195,7 @@ void Comment::finalizeImport()
 // position and formatting
 pNoteShape-convertFormatting( xAnnoShape );
 // visibility
-const ::oox::vml::ClientData* pClientData = 
pNoteShape-getClientData();
-xAnno-setIsVisible( pClientData  
pClientData-mbVisible );
+bVisible = pNoteShape-getTypeModel().mbVisible;
 }
 }
 break;
diff --git a/sc/source/filter/xcl97/xcl97rec.cxx 
b/sc/source/filter/xcl97/xcl97rec.cxx
index f1f8a40..212bb6b 100644
--- a/sc/source/filter/xcl97/xcl97rec.cxx
+++ b/sc/source/filter/xcl97/xcl97rec.cxx
@@ -566,7 +566,7 @@ VmlCommentExporter::VmlCommentExporter( 
sax_fastparser::FSHelperPtr p, ScAddress
 void VmlCommentExporter::Commit( EscherPropertyContainer rProps, const 
Rectangle rRect )
 {
 lcl_FillProps( rProps, mpCaption, mbVisible );
-rProps.AddOpt( ESCHER_Prop_fHidden, 1 );// bool field
+rProps.AddOpt( ESCHER_Prop_fHidden, mbVisible ); // bool field
 
 VMLExport::Commit( rProps, rRect );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 44446] LibreOffice 3.6 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Bug 6 depends on bug 46278, which changed state.

Bug 46278 Summary: FILESAVE: visibility settings of comments for cells are not 
saved in .xlsx format
https://bugs.freedesktop.org/show_bug.cgi?id=46278

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sal/inc sal/rtl sal/util

2012-10-02 Thread Libreoffice Gerrit user
 sal/inc/rtl/string.h   |   17 +
 sal/inc/rtl/string.hxx |   36 
 sal/inc/rtl/ustring.h  |   17 +
 sal/inc/rtl/ustring.hxx|   36 
 sal/rtl/source/strtmpl.cxx |   23 +++
 sal/util/sal.map   |6 ++
 6 files changed, 87 insertions(+), 48 deletions(-)

New commits:
commit 595ba03fd4769c069bbe0431ab878956ef1c37ad
Author: Michael Meeks michael.me...@suse.com
Date:   Tue Oct 2 18:39:30 2012 +0100

update string copy semantics to be undefined in a non-crashing way.

Change-Id: I03bb4db5931932280e368012cbaee6bef2854dd6

diff --git a/sal/inc/rtl/string.h b/sal/inc/rtl/string.h
index c38e106..6dd4236 100644
--- a/sal/inc/rtl/string.h
+++ b/sal/inc/rtl/string.h
@@ -889,6 +889,23 @@ SAL_DLLPUBLIC void SAL_CALL rtl_string_newFromStr( 
rtl_String ** newStr, const s
  */
 SAL_DLLPUBLIC void SAL_CALL rtl_string_newFromStr_WithLength( rtl_String ** 
newStr, const sal_Char * value, sal_Int32 len ) SAL_THROW_EXTERN_C();
 
+/** Allocate a new string that is a substring of this string.
+
+The substring begins at the specified beginIndex and contains count
+characters. Meaningless combinations such as negative beginIndex,
+or beginIndex + count greater than the length of the string have
+undefined behaviour.
+
+@param beginIndex   the beginning index, inclusive.
+@param countthe number of characters.
+@returnthe specified substring.
+
+@since LibreOffice 3.7
+ */
+SAL_DLLPUBLIC void SAL_CALL rtl_string_newFromSubString(
+rtl_String ** newStr, const rtl_String * from,
+sal_Int32 beginIndex, sal_Int32 count ) SAL_THROW_EXTERN_C();
+
 /**
  @internal
  @since LibreOffice 3.6
diff --git a/sal/inc/rtl/string.hxx b/sal/inc/rtl/string.hxx
index 84ab48e..d58999b 100644
--- a/sal/inc/rtl/string.hxx
+++ b/sal/inc/rtl/string.hxx
@@ -1024,31 +1024,27 @@ public:
 /**
   Returns a new string that is a substring of this string.
 
-  The substring begins at the specified beginIndex.  It is an error for
-  beginIndex to be negative or to be greater than the length of this 
string.
+  The substring begins at the specified beginIndex. If
+  beginIndex is negative or be greater than the length of
+  this string, behaviour is undefined.
 
   @param beginIndex   the beginning index, inclusive.
   @returnthe specified substring.
 */
 OString copy( sal_Int32 beginIndex ) const SAL_THROW(())
 {
-assert(beginIndex = 0  beginIndex = getLength());
-if ( beginIndex == 0 )
-return *this;
-else
-{
-rtl_String* pNew = 0;
-rtl_string_newFromStr_WithLength( pNew, pData-buffer+beginIndex, 
getLength()-beginIndex );
-return OString( pNew, (DO_NOT_ACQUIRE*)0 );
-}
+rtl_String *pNew = 0;
+rtl_string_newFromSubString( pNew, pData, beginIndex, getLength() - 
beginIndex );
+return OString( pNew, (DO_NOT_ACQUIRE*)0 );
 }
 
 /**
   Returns a new string that is a substring of this string.
 
   The substring begins at the specified beginIndex and contains count
-  characters.  It is an error for either beginIndex or count to be 
negative,
-  or for beginIndex + count to be greater than the length of this string.
+  characters.  If either beginIndex or count are negative,
+  or beginIndex + count are greater than the length of this string
+  then behaviour is undefined.
 
   @param beginIndex   the beginning index, inclusive.
   @param countthe number of characters.
@@ -1056,17 +1052,9 @@ public:
 */
 OString copy( sal_Int32 beginIndex, sal_Int32 count ) const SAL_THROW(())
 {
-assert(beginIndex = 0  beginIndex = getLength()  count = 0
- sal::static_int_castsal_uInt32(count) =
-   sal::static_int_castsal_uInt32(getLength() - beginIndex));
-if ( (beginIndex == 0)  (count == getLength()) )
-return *this;
-else
-{
-rtl_String* pNew = 0;
-rtl_string_newFromStr_WithLength( pNew, pData-buffer+beginIndex, 
count );
-return OString( pNew, (DO_NOT_ACQUIRE*)0 );
-}
+rtl_String *pNew = 0;
+rtl_string_newFromSubString( pNew, pData, beginIndex, count );
+return OString( pNew, (DO_NOT_ACQUIRE*)0 );
 }
 
 /**
diff --git a/sal/inc/rtl/ustring.h b/sal/inc/rtl/ustring.h
index 5b4982e..231dcdc 100644
--- a/sal/inc/rtl/ustring.h
+++ b/sal/inc/rtl/ustring.h
@@ -1226,6 +1226,23 @@ SAL_DLLPUBLIC void SAL_CALL rtl_uString_newFromStr(
 SAL_DLLPUBLIC void SAL_CALL rtl_uString_newFromStr_WithLength(
 rtl_uString ** newStr, const sal_Unicode * value, sal_Int32 len ) 
SAL_THROW_EXTERN_C();
 
+/** Allocate a new string that is a substring of this string.
+
+The substring 

[Libreoffice-commits] .: Branch 'feature/submodules' - g

2012-10-02 Thread Libreoffice Gerrit user
 g |   36 ++--
 1 file changed, 34 insertions(+), 2 deletions(-)

New commits:
commit ba9e29b24d2e83b522ccee8fd3dfb11bcf0d
Author: Norbert Thiebaud nthieb...@gmail.com
Date:   Tue Oct 2 12:59:44 2012 -0500

add tag support to ./g, try to optimize pre-post submerge situation

Change-Id: I052817865da6df193c0451eccc3b8b6f507845a3

diff --git a/g b/g
index 7c6551c..f70cac1 100755
--- a/g
+++ b/g
@@ -125,6 +125,31 @@ get_configured_submodules()
 fi
 }
 
+do_shortcut_update()
+{
+local module
+local repo
+
+for module in $SUBMODULES_CONFIGURED ; do
+   if [ ! -d ${module?}/.git ] ; then
+   case ${module?} in
+   helcontent2)
+   if [ -d clone/help/.git ] ; then
+   repo=clone/help/.git
+   fi
+   ;;
+   *)
+   if [ -d clone/${module?}/.git ] ; then
+   repo=clone/${module?}/.git
+   fi
+   ;;
+   esac
+   if [ -n $repo ] ; then
+   cp -r ${repo?} ${module?}/.
+   fi
+done
+}
+
 do_git_cmd()
 {
 echo cmd:$@
@@ -147,11 +172,13 @@ local module
create_branch=1
elif [ $create_branch = 1 ] ; then
branch=$arg
+   create_branch=0
fi
 done
 if [ -f .gitmodules ] ; then
+   git submodule update
if [ -n $branch ] ; then
-   git submodules foreach git branch ${branch} HEAD || return $?
+   git submodules foreach git checkout -b ${branch} HEAD || return $?
fi
 else
# now that is the nasty case we moved prior to submodules
@@ -189,6 +216,8 @@ do_init_modules()
 local module
 local configured
 
+do_shortcut_update
+
 for module in $SUBMODULES_CONFIGURED ; do
configured=$(git config --local --get submodule.${module}.url)
if [ -z $configured ] ; then
@@ -265,7 +294,7 @@ case $COMMAND in
do_git_cmd ${COMMAND} $@
;;
 checkout)
-   do_checkout $@  git submodule foreach git checkout $@
+   do_checkout $@
;;
 clone)
do_init_modules  git submodule update  refresh_all_hooks
@@ -290,6 +319,9 @@ case $COMMAND in
 reset)
do_reset
;;
+tag)
+   do_git_cmd ${COMMAND} $@
+   ;;
 *)
echo ./g does not support command:$COMMAND 12
exit 1;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sc/inc sc/source

2012-10-02 Thread Libreoffice Gerrit user
 sc/inc/colorscale.hxx  |   16 ++
 sc/source/core/data/colorscale.cxx |  206 +
 2 files changed, 88 insertions(+), 134 deletions(-)

New commits:
commit dde7ff4da28e754c2542191f363d8de78388e75a
Author: Markus Mohrhard markus.mohrh...@googlemail.com
Date:   Tue Oct 2 18:15:38 2012 +0200

simplify and speed up color format rendering

Change-Id: I7f9abc1e20ffb7d49ebae0c0f9d5e50f91b074c0

diff --git a/sc/inc/colorscale.hxx b/sc/inc/colorscale.hxx
index df3e17a..146c10b 100644
--- a/sc/inc/colorscale.hxx
+++ b/sc/inc/colorscale.hxx
@@ -189,10 +189,24 @@ public:
 virtual void DataChanged(const ScRange rRange) = 0;
 virtual void SetParent(ScConditionalFormat* pParent);
 
+virtual void startRendering();
+virtual void endRendering();
+
 protected:
-void getValues( std::vectordouble rValues ) const;
+std::vectordouble getValues() const;
+
+double getMinValue() const;
+double getMaxValue() const;
 
 ScConditionalFormat* mpParent;
+
+private:
+
+struct ScColorFormatCache
+{
+std::vectordouble maValues;
+};
+mutable boost::scoped_ptrScColorFormatCache mpCache;
 };
 
 class SC_DLLPUBLIC ScColorScaleFormat : public ScColorFormat
diff --git a/sc/source/core/data/colorscale.cxx 
b/sc/source/core/data/colorscale.cxx
index b224b51..6951e8b 100644
--- a/sc/source/core/data/colorscale.cxx
+++ b/sc/source/core/data/colorscale.cxx
@@ -34,6 +34,8 @@
 #include iostream
 #endif
 
+#include algorithm
+
 ScColorScaleEntry::ScColorScaleEntry():
 mnVal(0),
 mpCell(NULL),
@@ -239,103 +241,6 @@ ScColorScaleEntryType ScColorScaleEntry::GetType() const
 return meType;
 }
 
-namespace {
-
-double getMinValue(const ScRange rRange, ScDocument* pDoc)
-{
-double aMinValue = std::numeric_limitsdouble::max();
-//iterate through columns
-SCTAB nTab = rRange.aStart.Tab();
-for(SCCOL nCol = rRange.aStart.Col(); nCol = rRange.aEnd.Col(); ++nCol)
-{
-for(SCROW nRow = rRange.aStart.Row(); nRow = rRange.aEnd.Row(); 
++nRow)
-{
-ScAddress aAddr(nCol, nRow, rRange.aStart.Tab());
-CellType eType = pDoc-GetCellType(aAddr);
-if(eType == CELLTYPE_VALUE)
-{
-double aVal = pDoc-GetValue(nCol, nRow, nTab);
-if( aVal  aMinValue )
-aMinValue = aVal;
-}
-else if(eType == CELLTYPE_FORMULA)
-{
-
if(static_castScFormulaCell*(pDoc-GetCell(aAddr))-IsValue())
-{
-double aVal = pDoc-GetValue(nCol, nRow, nTab);
-if( aVal  aMinValue )
-aMinValue = aVal;
-}
-}
-}
-}
-return aMinValue;
-}
-
-double getMaxValue(const ScRange rRange, ScDocument* pDoc)
-{
-double aMaxValue = std::numeric_limitsdouble::min();
-//iterate through columns
-SCTAB nTab = rRange.aStart.Tab();
-for(SCCOL nCol = rRange.aStart.Col(); nCol = rRange.aEnd.Col(); ++nCol)
-{
-for(SCROW nRow = rRange.aStart.Row(); nRow = rRange.aEnd.Row(); 
++nRow)
-{
-ScAddress aAddr(nCol, nRow, rRange.aStart.Tab());
-CellType eType = pDoc-GetCellType(aAddr);
-if(eType == CELLTYPE_VALUE)
-{
-double aVal = pDoc-GetValue(nCol, nRow, nTab);
-if( aVal  aMaxValue )
-aMaxValue = aVal;
-}
-else if(eType == CELLTYPE_FORMULA)
-{
-
if(static_castScFormulaCell*(pDoc-GetCell(aAddr))-IsValue())
-{
-double aVal = pDoc-GetValue(nCol, nRow, nTab);
-if( aVal  aMaxValue )
-aMaxValue = aVal;
-}
-}
-}
-}
-return aMaxValue;
-}
-
-double getMinValue(const ScRangeList rList, ScDocument* pDoc)
-{
-double aMinValue = std::numeric_limitsdouble::max();
-
-size_t n = rList.size();
-for(size_t i = 0; i  n; ++i)
-{
-const ScRange* pRange = rList[i];
-double aVal = getMinValue(*pRange, pDoc);
-if( aVal  aMinValue )
-aMinValue = aVal;
-}
-return aMinValue;
-}
-
-double getMaxValue(const ScRangeList rList, ScDocument* pDoc)
-{
-double aMaxVal = std::numeric_limitsdouble::min();
-
-size_t n = rList.size();
-for(size_t i = 0; i  n; ++i)
-{
-const ScRange* pRange = rList[i];
-double aVal = getMaxValue(*pRange, pDoc);
-if( aVal  aMaxVal )
-aMaxVal = aVal;
-}
-
-return aMaxVal;
-}
-
-}
-
 double ScColorScaleFormat::GetMinValue() const
 {
 const_iterator itr = maColorScales.begin();
@@ -344,7 +249,7 @@ double ScColorScaleFormat::GetMinValue() const
 return itr-GetValue();
 else
 {
-return getMinValue(GetRange(), mpDoc);
+return getMinValue();
 }
 }
 
@@ -356,7 

[Libreoffice-commits] .: Branch 'feature/submodules' - translations

2012-10-02 Thread Libreoffice Gerrit user
 translations |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b54ebb07e9610dfc3f295d000bf0ccea22c00864
Author: Norbert Thiebaud nthieb...@gmail.com
Date:   Tue Oct 2 13:24:09 2012 -0500

Updated core
Project: translations  f83717cf73f4549d9ae09bc557d59316d4165394
null
add a README to the translations module

Change-Id: Iccfc83d5e9ec87802e88f83da1181493630805ea
Reviewed-on: https://gerrit.libreoffice.org/751
Reviewed-by: Norbert Thiebaud nthieb...@gmail.com
Tested-by: Norbert Thiebaud nthieb...@gmail.com

diff --git a/translations b/translations
index 751ae9d..f83717c 16
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit 751ae9de9563da8b26aef8e5c0f80ad502fbfe81
+Subproject commit f83717cf73f4549d9ae09bc557d59316d4165394
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 37361] LibreOffice 3.5 most annoying bugs

2012-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37361

Bug 37361 depends on bug 46805, which changed state.

Bug 46805 Summary: large autocorrect database make LibO freeze when start typing
https://bugs.freedesktop.org/show_bug.cgi?id=46805

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: lotuswordpro/source

2012-10-02 Thread Libreoffice Gerrit user
 lotuswordpro/source/filter/bentoid.hxx |   17 -
 lotuswordpro/source/filter/lwpdoc.hxx  |9 -
 lotuswordpro/source/filter/lwpfribheader.hxx   |3 ---
 lotuswordpro/source/filter/lwpgrfobj.cxx   |   19 ++-
 lotuswordpro/source/filter/lwplayout.hxx   |   14 --
 lotuswordpro/source/filter/lwpoleobject.hxx|2 --
 lotuswordpro/source/filter/xfilter/xfframe.cxx |1 -
 7 files changed, 2 insertions(+), 63 deletions(-)

New commits:
commit 02a7bf55603d8b1e74b83bb2b02fa86e1e31807b
Author: Marcos Paulo de Souza marcos.souza@gmail.com
Date:   Tue Oct 2 13:29:18 2012 -0300

Remove not used macros from lotusworkpro

Change-Id: I0c5b52030e9837f5ee9aa6258b6ccb16dd3f03c0
Signed-off-by: Marcos Paulo de Souza marcos.souza@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/749
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/lotuswordpro/source/filter/bentoid.hxx 
b/lotuswordpro/source/filter/bentoid.hxx
index 135b190..d34cc45 100644
--- a/lotuswordpro/source/filter/bentoid.hxx
+++ b/lotuswordpro/source/filter/bentoid.hxx
@@ -61,20 +61,11 @@ namespace OpenStormBento
 
 #define BEN_OBJID_TOC 1 // TOC object ID
 #define BEN_PROPID_TOC_SEED 2 // TOC object starting seed
-#define BEN_PROPID_TOC_MIN_SEED 3 // TOC object minimum seed
-#define BEN_PROPID_TOC_OBJECT 4 // TOC object
-#define BEN_PROPID_TOC_CONTAINER 5 // TOC object entire container
-#define BEN_PROPID_TOC_DELETED 6 // TOC object amount deleted
 #define BEN_TYPEID_TOC_TYPE 19 // TOC object property value type
-#define BEN_OBJID_WRITEABLE 20 // Predefined ID's below this value will be
-   // written to container TOC.  Predefined IDs
-   // above this are not written
 #define BEN_TYPEID_7_BIT_ASCII 21 // 7-bit ASCII
 #define BEN_PROPID_GLOBAL_TYPE_NAME 23 // Global type name
 #define BEN_PROPID_GLOBAL_PROPERTY_NAME 24 // Global property name
 #define BEN_PROPID_OBJ_REFERENCES 31 // Referenced objects from an object
-#define BEN_TYPEID_OBJ_REFERENCES_DATA 32 // Reference data
-#define BEN_OBJID_MIN_GENERAL_ID 100 // Min object ID for general std objects
 
 #define BEN_NEW_OBJECT 1 // object + property + type
 #define BEN_NEW_PROPERTY 2 // property + type
@@ -95,14 +86,6 @@ namespace OpenStormBento
 #define BEN_SEGMENT_CODE_END 14
 
 #define BEN_REFERENCE_LIST_ID 15 // references recording object ID
-#define BEN_RESERVED4B 16
-#define BEN_RESERVED4C 17
-#define BEN_RESERVED8A 18 // reserved for future use of 8 byte entry
-#define BEN_RESERVED8B 19 // reserved for future use of 8 byte entry
-#define BEN_RESERVED8C 20 // reserved for future use of 8 byte entry
-#define BEN_RESERVED12A 21 // reserved for future use of 12 byte entry
-#define BEN_RESERVED12B 22 // reserved for future use of 12 byte entry
-#define BEN_RESERVED12C 23 // reserved for future use of 12 byte entry
 #define BEN_END_OF_BUFFER 24 // end of current buffer, go to next
 #define BEN_READ_PAST_END_OF_TOC 50 // Our code
 #define BEN_NOOP 0xFF
diff --git a/lotuswordpro/source/filter/lwpdoc.hxx 
b/lotuswordpro/source/filter/lwpdoc.hxx
index a920e64..0e4a737 100644
--- a/lotuswordpro/source/filter/lwpdoc.hxx
+++ b/lotuswordpro/source/filter/lwpdoc.hxx
@@ -98,16 +98,7 @@ private:
 enum
 {
 DOC_PROTECTED = 0x0004UL,
-/*#define DOC_USECONTENTS   0x0008UL
-#define DOC_LOCKREVISIONS   0x0200UL
-#define DOC_LOCAL   0x0400UL
-*/
 DOC_CHILDDOC =  0x0800UL
-/*#define DOC_DOPREVIEW 0x0200UL
-#define DOC_REBUILDENDNOTES 0x0400UL
-#define DOC_FILEPROTECTION  0x1000UL
-#define DOC_PERSISTENTBITS  (DOC_PROTECTED | DOC_USECONTENTS | 
DOC_LOCKREVISIONS | DOC_LOCAL | DOC_CHILDDOC | DOC_DOPREVIEW | 
DOC_REBUILDENDNOTES | DOC_FILEPROTECTION)
-*/
 };
 
 //Code cleaning by change some members to local variables in Read()
diff --git a/lotuswordpro/source/filter/lwpfribheader.hxx 
b/lotuswordpro/source/filter/lwpfribheader.hxx
index 03757ce..143799e 100644
--- a/lotuswordpro/source/filter/lwpfribheader.hxx
+++ b/lotuswordpro/source/filter/lwpfribheader.hxx
@@ -68,10 +68,7 @@
 #define FRIB_MTAG_CHARSTYLE 3
 #define FRIB_MTAG_ATTRIBUTE 4
 #define FRIB_MTAG_LANGUAGE  5
-#define FRIB_MTAG_CHARBORDER6
-#define FRIB_MTAG_AMIKAKE   7
 #define FRIB_MTAG_CODEPAGE  8
-#define FRIB_MTAG_STXLINK   9   // SmarText; do not #ifdef out
 
 #define FT_MAXIMUM  30
 
diff --git a/lotuswordpro/source/filter/lwpgrfobj.cxx 
b/lotuswordpro/source/filter/lwpgrfobj.cxx
index 718d5f5..aa6fa88 100644
--- a/lotuswordpro/source/filter/lwpgrfobj.cxx
+++ b/lotuswordpro/source/filter/lwpgrfobj.cxx
@@ -85,9 +85,7 @@
 #include osl/thread.h
 
 #define EF_NONE 0x
-#define EF_FTP 0x0001
 #define EF_ODMA 0x0002
-#define EF_NOS 

[PUSHED] Remove not used macros from lotusworkpro

2012-10-02 Thread Michael Stahl (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/749

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/749
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0c5b52030e9837f5ee9aa6258b6ccb16dd3f03c0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza marcos.souza@gmail.com
Gerrit-Reviewer: Michael Stahl mst...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sw/inc sw/source

2012-10-02 Thread Libreoffice Gerrit user
 sw/inc/crsrsh.hxx|6 ---
 sw/inc/viscrs.hxx|   14 
 sw/source/core/crsr/viscrs.cxx   |   64 ---
 sw/source/core/layout/layact.cxx |3 -
 sw/source/ui/docvw/edtwin.cxx|   13 ---
 5 files changed, 100 deletions(-)

New commits:
commit c2eb9698eff7412e6973e659f26e7e39eeaf2887
Author: Takeshi Abe t...@fixedpoint.jp
Date:   Tue Oct 2 22:36:28 2012 +0900

Remove dead code

SW_CRSR_TIMER was undefined

Change-Id: Ib851170ea61547e0ceaa6721c0b2fb90fecaecd3
Reviewed-on: https://gerrit.libreoffice.org/747
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/sw/inc/crsrsh.hxx b/sw/inc/crsrsh.hxx
index 4c7e48c..e8678e9 100644
--- a/sw/inc/crsrsh.hxx
+++ b/sw/inc/crsrsh.hxx
@@ -750,12 +750,6 @@ public:
 // is the whole document protected/hidden (for UI...)
 sal_Bool IsAllProtect() const { return bAllProtect; }
 
-#ifdef SW_CRSR_TIMER
-// Set flag at VisCrsr. Is it triggered by Timer (TRUE) or directly 
(FALSE).
-// Default is triggert by Timer.
-sal_Bool ChgCrsrTimerFlag( sal_Bool bTimerOn = sal_True );
-#endif
-
 sal_Bool BasicActionPend() const{ return nBasicActionCnt != 
nStartAction; }
 
 sal_Bool GotoRegion( const String rName );
diff --git a/sw/inc/viscrs.hxx b/sw/inc/viscrs.hxx
index 5b0b925..f8b274b 100644
--- a/sw/inc/viscrs.hxx
+++ b/sw/inc/viscrs.hxx
@@ -39,9 +39,6 @@ class SwShellCrsr;
 // From here classes/methods for non-text cursor.
 
 class SwVisCrsr
-#ifdef SW_CRSR_TIMER
-: private Timer
-#endif
 {
 friend void _InitCore();
 friend void _FinitCore();
@@ -49,16 +46,9 @@ class SwVisCrsr
 sal_Bool bIsVisible : 1;
 sal_Bool bIsDragCrsr : 1;
 
-#ifdef SW_CRSR_TIMER
-sal_Bool bTimerOn : 1;
-#endif
-
 Cursor aTxtCrsr;
 const SwCrsrShell* pCrsrShell;
 
-#ifdef SW_CRSR_TIMER
-virtual void Timeout();
-#endif
 void _SetPosAndShow();
 
 public:
@@ -70,10 +60,6 @@ public:
 
 sal_Bool IsVisible() const { return bIsVisible; }
 void SetDragCrsr( sal_Bool bFlag = sal_True ) { bIsDragCrsr = bFlag; }
-
-#ifdef SW_CRSR_TIMER
-sal_Bool ChgTimerFlag( sal_Bool bTimerOn = sal_True );
-#endif
 };
 
 
diff --git a/sw/source/core/crsr/viscrs.cxx b/sw/source/core/crsr/viscrs.cxx
index 09eebb7..88d8404 100644
--- a/sw/source/core/crsr/viscrs.cxx
+++ b/sw/source/core/crsr/viscrs.cxx
@@ -70,22 +70,12 @@ SwVisCrsr::SwVisCrsr( const SwCrsrShell * pCShell )
 bIsVisible = aTxtCrsr.IsVisible();
 bIsDragCrsr = sal_False;
 aTxtCrsr.SetWidth( 0 );
-
-#ifdef SW_CRSR_TIMER
-bTimerOn = sal_True;
-SetTimeout( 50 ); // 50 millisecond delay
-#endif
 }
 
 
 
 SwVisCrsr::~SwVisCrsr()
 {
-#ifdef SW_CRSR_TIMER
-if( bTimerOn )
-Stop(); // stop timer
-#endif
-
 if( bIsVisible  aTxtCrsr.IsVisible() )
 aTxtCrsr.Hide();
 
@@ -103,21 +93,7 @@ void SwVisCrsr::Show()
 
 // display at all?
 if( pCrsrShell-VisArea().IsOver( pCrsrShell-aCharRect ) )
-#ifdef SW_CRSR_TIMER
-{
-if( bTimerOn )
-Start();// start timer
-else
-{
-if( IsActive() )
-Stop(); // stop timer
-
-_SetPosAndShow();
-}
-}
-#else
 _SetPosAndShow();
-#endif
 }
 }
 
@@ -129,51 +105,11 @@ void SwVisCrsr::Hide()
 {
 bIsVisible = sal_False;
 
-#ifdef SW_CRSR_TIMER
-if( IsActive() )
-Stop(); // stop timer
-#endif
-
 if( aTxtCrsr.IsVisible() )  // Shouldn't the flags be in effect?
 aTxtCrsr.Hide();
 }
 }
 
-#ifdef SW_CRSR_TIMER
-
-void SwVisCrsr::Timeout()
-{
-OSL_ENSURE( !bIsDragCrsr, stop timer before );
-if( bIsVisible )
-{
-if ( !pCrsrShell-GetWin() ) // SwFrmFmt::GetGraphic suspends Win 
temporarily!
-Start();
-else
-_SetPosAndShow();
-}
-}
-
-sal_Bool SwCrsrShell::ChgCrsrTimerFlag( sal_Bool bTimerOn )
-{
-return pVisCrsr-ChgTimerFlag( bTimerOn );
-}
-
-
-sal_Bool SwVisCrsr::ChgTimerFlag( sal_Bool bFlag )
-{
-bOld = bTimerOn;
-if( !bFlag  bIsVisible  IsActive() )
-{
-Stop(); // stop timer
-_SetPosAndShow();
-}
-bTimerOn = bFlag;
-return bOld;
-}
-
-#endif
-
-
 void SwVisCrsr::_SetPosAndShow()
 {
 SwRect aRect;
diff --git a/sw/source/core/layout/layact.cxx b/sw/source/core/layout/layact.cxx
index d542238..26de885 100644
--- a/sw/source/core/layout/layact.cxx
+++ b/sw/source/core/layout/layact.cxx
@@ -2330,9 +2330,6 @@ SwLayIdle::SwLayIdle( SwRootFrm *pRt, SwViewImp *pI ) :
 sal_Bool bVis = sal_False;
 if ( pSh-ISA(SwCrsrShell) )
 {
-#ifdef SW_CRSR_TIMER
-((SwCrsrShell*)pSh)-ChgCrsrTimerFlag( sal_False );
-#endif
 bVis = 
((SwCrsrShell*)pSh)-GetCharRect().IsOver(pSh-VisArea());
 }
 

[PUSHED] Remove dead code

2012-10-02 Thread Michael Stahl (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/747

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/747
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib851170ea61547e0ceaa6721c0b2fb90fecaecd3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Takeshi Abe t...@fixedpoint.jp
Gerrit-Reviewer: Michael Stahl mst...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: libvisio/libvisio-0.0.19.patch libvisio/UnpackedTarball_visio.mk

2012-10-02 Thread Libreoffice Gerrit user
 libvisio/UnpackedTarball_visio.mk |9 ++
 libvisio/libvisio-0.0.19.patch|  116 ++
 2 files changed, 125 insertions(+)

New commits:
commit 7192f0ddaa937f6c63bfffed0b35146a0f97943a
Author: Fridrich Å trba fridrich.st...@bluewin.ch
Date:   Tue Oct 2 23:12:49 2012 +0200

Clean the path pushed to the API as much as possible

Change-Id: I5e68bc5aedc4c6e350e62b30472a1b3f0803c520

diff --git a/libvisio/UnpackedTarball_visio.mk 
b/libvisio/UnpackedTarball_visio.mk
index aa9113b..15429c5 100644
--- a/libvisio/UnpackedTarball_visio.mk
+++ b/libvisio/UnpackedTarball_visio.mk
@@ -11,4 +11,13 @@ $(eval $(call gb_UnpackedTarball_UnpackedTarball,visio))
 
 $(eval $(call gb_UnpackedTarball_set_tarball,visio,$(VISIO_TARBALL)))
 
+$(eval $(call gb_UnpackedTarball_set_patchlevel,visio,0))
+
+visio_patches :=
+# Somehow draw is very picky about what is a closed path
+visio_patches += libvisio-0.0.19.patch
+
+$(eval $(call gb_UnpackedTarball_add_patches,visio,\
+   $(foreach patch,$(visio_patches),libvisio/$(patch)) \
+))
 # vim: set noet sw=4 ts=4:
diff --git a/libvisio/libvisio-0.0.19.patch b/libvisio/libvisio-0.0.19.patch
new file mode 100644
index 000..ee3c17f
--- /dev/null
+++ b/libvisio/libvisio-0.0.19.patch
@@ -0,0 +1,116 @@
+--- src/lib/VSDXContentCollector.cpp
 src/lib/VSDXContentCollector.cpp
+@@ -676,7 +676,6 @@ double 
libvisio::VSDXContentCollector::_linePropertiesMarkerScale(unsigned marke
+ 
+ void libvisio::VSDXContentCollector::_flushCurrentPath()
+ {
+-  WPXPropertyListVector path;
+   WPXPropertyList fillPathProps(m_styleProps);
+   fillPathProps.insert(draw:stroke, none);
+   WPXPropertyList linePathProps(m_styleProps);
+@@ -695,6 +694,7 @@ void libvisio::VSDXContentCollector::_flushCurrentPath()
+   if (needsGroup)
+ m_shapeOutputDrawing-addStartLayer(WPXPropertyList());
+ 
++  std::vectorWPXPropertyList tmpPath;
+   if (m_styleProps[draw:fill]  m_styleProps[draw:fill]-getStr() != 
none)
+   {
+ bool firstPoint = true;
+@@ -708,39 +708,84 @@ void libvisio::VSDXContentCollector::_flushCurrentPath()
+   }
+   else if (m_currentFillGeometry[i][libwpg:path-action]-getStr() == 
M)
+   {
+-if (path.count()  !wasMove)
++if (!tmpPath.empty())
+ {
+-  WPXPropertyList closedPath;
+-  closedPath.insert(libwpg:path-action, Z);
+-  path.append(closedPath);
++  if (!wasMove)
++  {
++WPXPropertyList closedPath;
++closedPath.insert(libwpg:path-action, Z);
++tmpPath.push_back(closedPath);
++  }
++  else
++  {
++tmpPath.pop_back();
++  }
+ }
+ wasMove = true;
+   }
+   else
+ wasMove = false;
+-  path.append(m_currentFillGeometry[i]);
++  tmpPath.push_back(m_currentFillGeometry[i]);
+ }
+-if (path.count()  !wasMove)
++if (!tmpPath.empty())
+ {
+-  WPXPropertyList closedPath;
+-  closedPath.insert(libwpg:path-action, Z);
+-  path.append(closedPath);
++  if (!wasMove)
++  {
++WPXPropertyList closedPath;
++closedPath.insert(libwpg:path-action, Z);
++tmpPath.push_back(closedPath);
++  }
++  else
++tmpPath.pop_back();
+ }
+-if (path.count())
++if (!tmpPath.empty())
+ {
++  WPXPropertyListVector path;
++  for (unsigned i = 0; i  tmpPath.size(); ++i)
++path.append(tmpPath[i]);
+   m_shapeOutputDrawing-addStyle(fillPathProps, WPXPropertyListVector());
+   m_shapeOutputDrawing-addPath(path);
+ }
+   }
+   m_currentFillGeometry.clear();
+-  path = WPXPropertyListVector();
++  tmpPath.clear();
+ 
+   if (m_styleProps[draw:stroke]  m_styleProps[draw:stroke]-getStr() != 
none)
+   {
++bool firstPoint = true;
++bool wasMove = false;
+ for (unsigned i = 0; i  m_currentLineGeometry.size(); i++)
+-  path.append(m_currentLineGeometry[i]);
+-if (path.count())
+ {
++  if (firstPoint)
++  {
++firstPoint = false;
++wasMove = true;
++  }
++  else if (m_currentLineGeometry[i][libwpg:path-action]-getStr() == 
M)
++  {
++if (!tmpPath.empty())
++{
++  if (wasMove)
++  {
++tmpPath.pop_back();
++  }
++}
++wasMove = true;
++  }
++  else
++wasMove = false;
++  tmpPath.push_back(m_currentLineGeometry[i]);
++}
++if (!tmpPath.empty())
++{
++  if (wasMove)
++tmpPath.pop_back();
++}
++if (!tmpPath.empty())
++{
++  WPXPropertyListVector path;
++  for (unsigned i = 0; i  tmpPath.size(); ++i)
++path.append(tmpPath[i]);
+   m_shapeOutputDrawing-addStyle(linePathProps, WPXPropertyListVector());
+   m_shapeOutputDrawing-addPath(path);
+ }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org

[Libreoffice-commits] .: wizards/Pyuno_commonwizards.mk

2012-10-02 Thread Libreoffice Gerrit user
 wizards/Pyuno_commonwizards.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 58deab04a99ab0ad49d4f33348dc801ca932c0ca
Author: Xisco Fauli aniste...@gmail.com
Date:   Tue Oct 2 23:27:33 2012 +0200

pyfax: Add TextElement to makefile

Change-Id: Iddc7ae6ce49d815e05f3f537569434489e3ccf53

diff --git a/wizards/Pyuno_commonwizards.mk b/wizards/Pyuno_commonwizards.mk
index fb7ac10..5653dac 100644
--- a/wizards/Pyuno_commonwizards.mk
+++ b/wizards/Pyuno_commonwizards.mk
@@ -46,6 +46,7 @@ $(eval $(call gb_Pyuno_add_files,wizards,\
 common/HelpIds.py \
 common/NumberFormatter.py \
 common/PropertySetHelper.py \
+common/TextElement.py \
 document/__init__.py \
 document/OfficeDocument.py \
 text/__init__.py \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: config_host.mk.in configure.in Makefile.top Module_tail_build.mk RepositoryModule_ooo.mk rhino/ExternalProject_rhino.mk rhino/Makefile rhino/makefile.mk rhino/Module_rhino.mk

2012-10-02 Thread Libreoffice Gerrit user
 Makefile.top   |4 -
 Module_tail_build.mk   |1 
 RepositoryModule_ooo.mk|1 
 config_host.mk.in  |1 
 configure.in   |2 
 rhino/ExternalProject_rhino.mk |   37 +++
 rhino/Makefile |7 ++
 rhino/Module_rhino.mk  |   22 ++
 rhino/OfficeScriptInfo.java|  128 +++
 rhino/Package_rhino.mk |   16 
 rhino/UnpackedTarball_rhino.mk |   24 +++
 rhino/makefile.mk  |   75 --
 rhino/prj/build.lst|5 -
 rhino/prj/d.lst|1 
 rhino/rhino1_5R5.patch |  134 -
 tail_build/prj/build.lst   |2 
 16 files changed, 244 insertions(+), 216 deletions(-)

New commits:
commit ae1f2f6b932d43e6770022039d9cbe3f3960ab27
Author: David Ostrovsky da...@ostrovsky.org
Date:   Tue Oct 2 23:22:37 2012 +0200

gbuildification of rhino

Default patch level 3 was not working here and must
be overriden with gb_UnpackedTarball_set_patchlevel
function with 2.

New java file OfficeScriptInfo.java was extracted from
rhino1_5R5.patch and added with gb_UnpackedTarball_add_file.

Change-Id: Iba25b2350caffd943af93f59fa2139332b009ef2
Reviewed-on: https://gerrit.libreoffice.org/752
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/Makefile.top b/Makefile.top
index de91f56..bb4f73e 100644
--- a/Makefile.top
+++ b/Makefile.top
@@ -121,13 +121,14 @@ padmin\
 psprint_config\
 pyuno\
 qadevOOo\
+rdbmaker\
 readlicense_oo\
 regexp\
 registry\
 remotebridges\
 reportbuilder\
 reportdesign\
-rdbmaker\
+rhino\
 ridljar\
 rsc\
 sal\
@@ -245,7 +246,6 @@ postgresql\
 postprocess\
 python\
 redland\
-rhino\
 setup_native\
 stax\
 stlport\
diff --git a/Module_tail_build.mk b/Module_tail_build.mk
index 86a9c11..fa63c17 100644
--- a/Module_tail_build.mk
+++ b/Module_tail_build.mk
@@ -120,6 +120,7 @@ $(eval $(call gb_Module_add_moduledirs,tail_end,\
remotebridges \
reportbuilder \
reportdesign \
+   $(call gb_Helper_optional,RHINO,rhino) \
ridljar \
rsc \
salhelper \
diff --git a/RepositoryModule_ooo.mk b/RepositoryModule_ooo.mk
index b654664..a8153e1 100644
--- a/RepositoryModule_ooo.mk
+++ b/RepositoryModule_ooo.mk
@@ -129,6 +129,7 @@ $(eval $(call gb_Module_add_moduledirs,ooo,\
 remotebridges \
 reportbuilder \
 reportdesign \
+   $(call gb_Helper_optional,RHINO,rhino) \
 ridljar \
 rsc \
 sal \
diff --git a/config_host.mk.in b/config_host.mk.in
index 48b470e..9052d70 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -481,6 +481,7 @@ export RANLIB=@RANLIB@
 export REDLAND_CFLAGS=@REDLAND_CFLAGS@
 export REDLAND_LIBS=@REDLAND_LIBS@
 export RHINO_JAR=@RHINO_JAR@
+export RHINO_TARBALL=@RHINO_TARBALL@
 export RPM=@RPM@
 export RTL_OS=@RTL_OS@
 export RTL_ARCH=@RTL_ARCH@
diff --git a/configure.in b/configure.in
index 2974db0..45c3d85 100644
--- a/configure.in
+++ b/configure.in
@@ -10107,6 +10107,7 @@ if test x$enable_ext_scripting_javascript = xyes -a 
x$enable_extension_inte
else
AC_MSG_RESULT([internal])
SYSTEM_RHINO=NO
+   RHINO_TARBALL=798b2ffdc8bcfe7bca2cf92b62caf685-rhino1_5R5.zip
BUILD_TYPE=$BUILD_TYPE RHINO
fi
 else
@@ -10117,6 +10118,7 @@ fi
 AC_SUBST(ENABLE_SCRIPTING_JAVASCRIPT)
 AC_SUBST(SYSTEM_RHINO)
 AC_SUBST(RHINO_JAR)
+AC_SUBST(RHINO_TARBALL)
 
 dnl Scripting provider for Python extension?
 dnl We always provide this unless we have disabled Python completely
diff --git a/rhino/ExternalProject_rhino.mk b/rhino/ExternalProject_rhino.mk
new file mode 100644
index 000..0ca364e
--- /dev/null
+++ b/rhino/ExternalProject_rhino.mk
@@ -0,0 +1,37 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+# for RSCREVISION
+include $(SOLARENV)/inc/minor.mk
+
+$(eval $(call gb_ExternalProject_ExternalProject,rhino))
+
+$(eval $(call gb_ExternalProject_use_unpacked,rhino,rhino))
+
+$(eval $(call gb_ExternalProject_register_targets,rhino,\
+   build \
+))
+
+$(call gb_ExternalProject_get_state_target,rhino,build) :
+   cd $(call gb_UnpackedTarball_get_dir,rhino)  \
+   $(ANT) \
+   -q \
+   -f build.xml \
+   -Dbuild.label=build-$(RSCREVISION) \
+   -DTARFILE_LOCATION=$(TARFILE_LOCATION) \
+   $(if $(filter yes,$(JAVACISGCJ))\
+   ,-Dbuild.compiler=gcj \
+   ,-Dant.build.javac.source=$(JAVA_SOURCE_VER) \
+   -Dant.build.javac.target=$(JAVA_TARGET_VER) \
+  

[PUSHED] gbuildification of rhino

2012-10-02 Thread Michael Stahl (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/752

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/752
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iba25b2350caffd943af93f59fa2139332b009ef2
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Ostrovsky david.ostrov...@gmx.de
Gerrit-Reviewer: Michael Stahl mst...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] gbuildification of languagetool

2012-10-02 Thread David Ostrovsky (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/753

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/53/753/1

gbuildification of languagetool

It is not clear how to remove a file:
REMOVE_FILES=openoffice/images/dialogh.png.
Let it for now in place.

Change-Id: Ie8739fd207ca7dc9343c92c93cf092b3af674bf0
---
M Makefile.top
M RepositoryModule_ooo.mk
M config_host.mk.in
M configure.in
A languagetool/ExternalProject_languagetool.mk
A languagetool/Makefile
A languagetool/Module_languagetool.mk
A languagetool/Package_languagetool.mk
A languagetool/UnpackedTarball_languagetool.mk
M languagetool/prj/build.lst
M languagetool/prj/d.lst
D languagetool/prj/dmake
12 files changed, 103 insertions(+), 4 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/753
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie8739fd207ca7dc9343c92c93cf092b3af674bf0
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Ostrovsky david.ostrov...@gmx.de

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED] [PATCH 2/3] fdo#51304: Remove @author annotation and commented code

2012-10-02 Thread Thorsten Behrens
José Guilherme Vanz wrote:
 2012/10/1 Thorsten Behrens t...@documentfoundation.org
  partially pushed - I left the hunks commenting empty methods as
  FIXME instead of removing them straight away unapplied. If the
  method is pointless, why not remove them in one go?
 

Hi José Guilherme,

want to go ahead  kill those methods you had annotated? I'll gladly
merge that.

Cheers,

-- Thorsten


pgpgcDPTPu7ql.pgp
Description: PGP signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH] gbuildification of rhino

2012-10-02 Thread Norbert Thiebaud
David,

to answer your question on IRC.

yes the default patchlevel for gbuild is 3, because most fo the patch
we had are like that...
BUT,
I pushed a patch some time ago so that if a patch is named *.n where
n is from 0-9, then gbuild will apply that patch with the patchlevel n
(regardless of the default patchlevel or even of the level set by
gb_UnpackedTarball_set_patchlevel()

see: http://wiki.documentfoundation.org/Development/Patching_External_Libraries

I strongly recommend that we use that naming convention rather than
use the api call to override the default. so the cleaner solution is
just the rename the patch

Norbert
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'feature/calc-xml-source' - sc/source

2012-10-02 Thread Libreoffice Gerrit user
 sc/source/filter/orcus/orcusfiltersimpl.cxx |9 +
 1 file changed, 9 insertions(+)

New commits:
commit f7fed308a69eeba8fae7b389f1c26a1a6f326b12
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Oct 2 19:46:47 2012 -0400

orcus from master now has a new interface.

Change-Id: I1d03d5dd2b12cdc1eab2d599199c333d49d306b8

diff --git a/sc/source/filter/orcus/orcusfiltersimpl.cxx 
b/sc/source/filter/orcus/orcusfiltersimpl.cxx
index 7152b68..e5c5507 100644
--- a/sc/source/filter/orcus/orcusfiltersimpl.cxx
+++ b/sc/source/filter/orcus/orcusfiltersimpl.cxx
@@ -61,6 +61,9 @@ public:
 virtual void set_formula_result(row_t row, col_t col, const char* p, 
size_t n);
 virtual void set_shared_formula(
 row_t row, col_t col, formula_grammar_t grammar, size_t sindex,
+const char* p_formula, size_t n_formula);
+virtual void set_shared_formula(
+row_t row, col_t col, formula_grammar_t grammar, size_t sindex,
 const char* p_formula, size_t n_formula, const char* p_range, size_t 
n_range);
 virtual void set_shared_formula(row_t row, col_t col, size_t sindex);
 virtual void set_string(row_t row, col_t col, size_t sindex);
@@ -122,6 +125,12 @@ void ScOrcusSheet::set_formula_result(row_t /*row*/, col_t 
/*col*/, const char*
 
 void ScOrcusSheet::set_shared_formula(
 row_t /*row*/, col_t /*col*/, formula_grammar_t /*grammar*/, size_t 
/*sindex*/,
+const char* /*p_formula*/, size_t /*n_formula*/)
+{
+}
+
+void ScOrcusSheet::set_shared_formula(
+row_t /*row*/, col_t /*col*/, formula_grammar_t /*grammar*/, size_t 
/*sindex*/,
 const char* /*p_formula*/, size_t /*n_formula*/, const char* /*p_range*/, 
size_t /*n_range*/)
 {
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/vs2012' - libwpd/ExternalProject_libwpd.mk

2012-10-02 Thread Libreoffice Gerrit user
 libwpd/ExternalProject_libwpd.mk |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 26545fdea00953a48f1818b8ff50339503e296a8
Author: Peter Foley pefol...@verizon.net
Date:   Tue Oct 2 21:48:12 2012 -0400

fix libwpd for vs90

Change-Id: Ia8a21cd512ac425207eb31a194db9f4c9b10fee4

diff --git a/libwpd/ExternalProject_libwpd.mk b/libwpd/ExternalProject_libwpd.mk
index f91314a..e8547ea 100644
--- a/libwpd/ExternalProject_libwpd.mk
+++ b/libwpd/ExternalProject_libwpd.mk
@@ -17,12 +17,12 @@ $(eval $(call gb_ExternalProject_register_targets,libwpd,\
 
 ifeq ($(OS)$(COM),WNTMSC)
 
-ifeq ($(CCNUMVER),90)
+ifeq ($(VCVER),90)
 $(call gb_ExternalProject_get_state_target,libwpd,build) :
cd $(EXTERNAL_WORKDIR)/build/win32 \
 $(COMPATH)/vcpackages/vcbuild.exe libwpd.vcproj Release|Win32 \
 touch $@
-else ifeq ($(CCNUMVER),100)
+else ifeq ($(VCVER),100)
 $(call gb_ExternalProject_get_state_target,libwpd,build) :
cd $(EXTERNAL_WORKDIR)/build/win32 \
 msbuild.exe libwpd.vcxproj \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'feature/unitymenus-quantal'

2012-10-02 Thread Libreoffice Gerrit user
New branch 'feature/unitymenus-quantal' available with the following commits:
commit 767bae2cefd4cfcaa852c2c39e651bab81e4e134
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Wed Oct 3 00:30:40 2012 +0200

use sal/log.hxx style logging

commit 6f08ada4936c4da0e551bc1219d84fa6111ceb56
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 14:01:25 2012 +0200

revoke the watch when dying

commit 714d9e3b7dfb40228ed3f742f4735e5ca419fefa
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 12:35:21 2012 +0200

some moving around of casts

commit 755e4d5a05f98b55cb0e52b79b66598c663a315c
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 09:43:41 2012 +0200

remove submenus in the menu model, if they are gone in the vcl menu and do 
the same with superfluous additional entries/sections

commit d5e3b1258b23c8399035cbbe4760a5e461909031
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 08:18:11 2012 +0200

remove UpdateNativeMenu2 and clear remove submenus on updates

commit b55ca3ed8b75e566972232fe70fbe8a4917a5deb
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 08:00:36 2012 +0200

move UpdateNativeMenu2 back into UpdateNativeMenu

commit b3e7fc8d3b3fdf211b005d759952fa1f13765622
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 06:22:04 2012 +0200

tweak menu destruction

commit 8a07525a2d732fb516b92fd0ec23c999599b56fe
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 04:55:03 2012 +0200

make UpdateNativeMenu2 and CanUpdate members

commit 7b1679567ac93e4f53f314942f9282d5fba155bc
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 04:54:09 2012 +0200

refactor out dbus setup

commit 3e5272191e4562a5aea9b665a270fe3c05836e77
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 03:37:52 2012 +0200

tweak on_registrar_available to just touch the window props, the menu has 
to take it from there

commit 76a30691dbe04616df0f9b47e723a7adb34daa8c
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 03:10:51 2012 +0200

activate/deactivate do not need the solarmutex, better NULL the menu when 
freeing it, SetFrame should only be used on a menubar

commit 08f19c7fe2a2eb17bd5e1d192e840f667962e460
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 03:01:28 2012 +0200

update when registrar is available, dont set menumodel on SetFrame, we do 
that on update

commit cdeac1b88d4a4ede9ba6affa15dc1642a8f1aabe
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 02:24:41 2012 +0200

make UpdateNativeMenu a member

commit 735cf920be6a6c4373724f5f7c85e5e66d178243
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 02:13:47 2012 +0200

remove obsolete GenerateMenu wrapper

commit 73c96795c3b03f332ef0a8e7159c49b9e5835c77
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Mon Oct 1 02:04:26 2012 +0200

grab model from frame on update

commit 6a50cb83e18e319e2a0e3e68dfb44a88a089e2ef
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Fri Sep 28 19:16:18 2012 +0200

moved dbus and menumodel ownership to frame, where it belongs

commit 224642f1e46562bdf934e20d0b22315ee0e542cd
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Wed Sep 19 20:05:56 2012 +0100

Fixed a crash updating the menu after opening an application from main 
screen.

Change-Id: If7b603dca5aa33e50a63e04fda3bce594840cc57

commit 1657cf233b8519cde3a063c157232c7a7a68b88b
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Mon Sep 17 23:37:25 2012 +0100

Removed some debug messages.

Change-Id: I27d9d82d6fcd27323c86d24a675198186c185281

commit 8b9424f44827190be9ba6a32df3aff179042cbbf
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Mon Sep 17 23:35:42 2012 +0100

Memory management improvements on GLOMenu and GLOActionGroup classes.

Change-Id: I1c4fb268a08c2b343ac99ee9294f536c1444f707

commit bdcf5af1b1a3767e4a411d350c44c04ec963f7d3
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Mon Sep 17 14:43:03 2012 +0100

Improved memory management of GLOActionGroup.

Change-Id: Iaeee32c2990cb6514605d3f7bc959682d87d5d38

commit 56cdbe4f8c04501e5f3204f2352513650ffd5290
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Thu Sep 13 20:08:07 2012 +0100

Fixed a crash when opening an application from LO main screen.

Change-Id: I942bcf761c7b9ba11c47bc618d55bdc85a472705

commit 2b4086d4ba993a5a0fb17e2632f0f7b3b353fcea
Author: Antonio Fernandez antonio.fernan...@aentos.es
Date:   Wed Sep 12 20:45:31 2012 +0100

Menus are generated on demand now, but with some issues.

Change-Id: Icec9b685e720a369cff4d2bbfc5bf4ba6614e390

commit 

  1   2   3   >