[Libreoffice-commits] .: solenv/gbuild

2012-03-22 Thread Norbert Thiebaud
 solenv/gbuild/gbuild.mk |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 9d2968d6dd12cda797aa31bc0f0798f835cd073a
Author: Norbert Thiebaud 
Date:   Thu Mar 22 23:57:09 2012 -0500

disable gb_FULLDEPS when TINDERBUILD is set _and_ we are really building

diff --git a/solenv/gbuild/gbuild.mk b/solenv/gbuild/gbuild.mk
index a26f451..d16d33a 100644
--- a/solenv/gbuild/gbuild.mk
+++ b/solenv/gbuild/gbuild.mk
@@ -113,14 +113,14 @@ endif
 
 # for clean, setuplocal and removelocal goals we switch off dependencies
 ifneq ($(filter cleanpackmodule clean setuplocal removelocal showdeliverables 
help debugrun,$(MAKECMDGOALS)),)
-ifdef TINDERBUILD
 gb_FULLDEPS := $(false)
 else
+ifdef TINDERBUILD
 gb_FULLDEPS := $(false)
-endif
 else
 gb_FULLDEPS := $(true)
 endif
+endif
 
 # save user-supplied flags for latter use
 ifneq ($(strip $(CFLAGS)),)
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 4 commits - desktop/Module_desktop.mk desktop/Pagein_calc.mk desktop/Pagein_common.mk desktop/Pagein_draw.mk desktop/Pagein_impress.mk desktop/Pagein.mk desktop/Pagein_writer.

2012-03-22 Thread David Tardon
 Repository.mk|4 -
 desktop/Module_desktop.mk|   10 ---
 desktop/Pagein.mk|  104 ---
 desktop/Pagein_calc.mk   |4 -
 desktop/Pagein_common.mk |   26 -
 desktop/Pagein_draw.mk   |4 -
 desktop/Pagein_impress.mk|4 -
 desktop/Pagein_writer.mk |4 -
 solenv/gbuild/Pagein.mk  |   77 
 solenv/gbuild/TargetLocations.mk |3 +
 solenv/gbuild/gbuild.mk  |1 
 11 files changed, 93 insertions(+), 148 deletions(-)

New commits:
commit 01eb464eb418159f1f11ce0dce5e1a8a10e01408
Author: David Tardon 
Date:   Sun Mar 18 15:32:28 2012 +0100

remove duplicate entries

diff --git a/desktop/Pagein_common.mk b/desktop/Pagein_common.mk
index 4d66d3f..258ba19 100644
--- a/desktop/Pagein_common.mk
+++ b/desktop/Pagein_common.mk
@@ -28,9 +28,6 @@
 $(eval $(call gb_Pagein_Pagein,common))
 
 # sorted in approx. reverse load order (ld.so.1)
-# TODO: Hmm, so it looks like there are duplicates in the list... Moreover,
-# some that are conditional above are not conditional here (e.g., icule).
-# I have doubts about gconfbe, desktopbe and localebe too.
 $(eval $(call gb_Pagein_add_objects,common,\
 i18npool \
 $(if $(findstring YES,$(SYSTEM_ICU)),,\
@@ -75,19 +72,13 @@ $(eval $(call gb_Pagein_add_objects,common,\
 oovbaapi.rdb \
 deployment \
 deploymentmisc \
-ucb1 \
 xstor \
-package2 \
 filterconfig \
 uui \
-lng \
 svt \
 spl \
-basegfx \
 avmedia \
 helplinker \
-vclplug_gen \
-icule \
 sax \
 gconfbe \
 fsstorage \
commit 424a03e72f10d123bf67a2b36c306b8c09e34669
Author: David Tardon 
Date:   Sun Mar 18 15:24:42 2012 +0100

simplify pagein creation

diff --git a/desktop/Pagein_calc.mk b/desktop/Pagein_calc.mk
index afbbb15..0423b25 100644
--- a/desktop/Pagein_calc.mk
+++ b/desktop/Pagein_calc.mk
@@ -27,7 +27,7 @@
 
 $(eval $(call gb_Pagein_Pagein,calc))
 
-$(eval $(call gb_Pagein_add_libs,calc,\
+$(eval $(call gb_Pagein_add_objects,calc,\
 sc \
 scui \
 svx \
diff --git a/desktop/Pagein_common.mk b/desktop/Pagein_common.mk
index 06cb024..4d66d3f 100644
--- a/desktop/Pagein_common.mk
+++ b/desktop/Pagein_common.mk
@@ -28,7 +28,10 @@
 $(eval $(call gb_Pagein_Pagein,common))
 
 # sorted in approx. reverse load order (ld.so.1)
-$(eval $(call gb_Pagein_add_libs,common,\
+# TODO: Hmm, so it looks like there are duplicates in the list... Moreover,
+# some that are conditional above are not conditional here (e.g., icule).
+# I have doubts about gconfbe, desktopbe and localebe too.
+$(eval $(call gb_Pagein_add_objects,common,\
 i18npool \
 $(if $(findstring YES,$(SYSTEM_ICU)),,\
 icui18n \
@@ -52,9 +55,6 @@ $(eval $(call gb_Pagein_add_libs,common,\
 sot \
 xcr \
 sb \
-))
-
-$(eval $(call gb_Pagein_add_libs_with_dir,common,\
 stocservices \
 bootstrap \
 reg \
@@ -63,10 +63,6 @@ $(eval $(call gb_Pagein_add_libs_with_dir,common,\
 cppuhelper \
 cppu \
 sal \
-,../ure-link/lib \
-))
-
-$(eval $(call gb_Pagein_add_libs,common,\
 ucbhelper \
 comphelper \
 tl \
@@ -74,18 +70,9 @@ $(eval $(call gb_Pagein_add_libs,common,\
 svl \
 vcl \
 tk \
-))
-
-$(eval $(call gb_Pagein_add_objects,common,\
 ../ure-link/share/misc/types.rdb \
 services.rdb \
 oovbaapi.rdb \
-))
-
-# TODO: Hmm, so it looks like there are duplicates in the list... Moreover,
-# some that are conditional above are not conditional here (e.g., icule).
-# I have doubts about gconfbe, desktopbe and localebe too.
-$(eval $(call gb_Pagein_add_libs,common,\
 deployment \
 deploymentmisc \
 ucb1 \
diff --git a/desktop/Pagein_draw.mk b/desktop/Pagein_draw.mk
index 5795b05..be3b8e0 100644
--- a/desktop/Pagein_draw.mk
+++ b/desktop/Pagein_draw.mk
@@ -27,7 +27,7 @@
 
 $(eval $(call gb_Pagein_Pagein,draw))
 
-$(eval $(call gb_Pagein_add_libs,draw,\
+$(eval $(call gb_Pagein_add_objects,draw,\
 sd \
 sdui \
 svx \
diff --git a/desktop/Pagein_impress.mk b/desktop/Pagein_impress.mk
index ae331f8..5104722 100644
--- a/desktop/Pagein_impress.mk
+++ b/desktop/Pagein_impress.mk
@@ -27,7 +27,7 @@
 
 $(eval $(call gb_Pagein_Pagein,impress))
 
-$(eval $(call gb_Pagein_add_libs,impress,\
+$(eval $(call gb_Pagein_add_objects,impress,\
 sd \
 sdui \
 svx \
diff --git a/desktop/Pagein_writer.mk b/desktop/Pagein_writer.mk
index 206e18a..c9694a5 100644
--- a/desktop/Pagein_writer.mk
+++ b/desktop/Pagein_writer.mk
@@ -27,7 +27,7 @@
 
 $(eval $(call gb_Pagein_Pagein,writer))
 
-$(eval $(call gb_Pagein_add_libs,writer,\
+$(eval $(call gb_Pagein_add_objects,writer,\
 sw \
 swui \
 svx \
diff --git a/solenv/gbuild/Pagein.mk b/solenv/gbuild/Pagein.mk
index ead8f8a..60d98ac 100644
--- a/solenv/gbuild/Pagein.mk
+++ b/solenv/gbuild/Pagein.mk
@@ -25,20 +25,16 @@
 # in which case the provisio

[Libreoffice-commits] .: sc/source

2012-03-22 Thread Markus Mohrhard
 sc/source/filter/html/htmlexp.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit c4744121f12fde51de2d92d2442d590d93725ffc
Author: Markus Mohrhard 
Date:   Fri Mar 23 04:27:45 2012 +0100

increase nCol before next iteratation to prevent loop, fdo#46220

diff --git a/sc/source/filter/html/htmlexp.cxx 
b/sc/source/filter/html/htmlexp.cxx
index 538eb09..af0689a 100644
--- a/sc/source/filter/html/htmlexp.cxx
+++ b/sc/source/filter/html/htmlexp.cxx
@@ -769,7 +769,11 @@ void ScHTMLExport::WriteTables()
 while( nCol <= nEndCol )
 {
 if( pDoc->ColHidden(nCol, nTab) )
+{
+++nCol;
 continue;
+}
+
 if( nWidth != ToPixel( pDoc->GetColWidth( nCol, nTab ) ) )
 {
 if( nSpan != 0 )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sc/qa

2012-03-22 Thread Kohei Yoshida
 sc/qa/unit/ucalc.cxx |  102 ---
 1 file changed, 49 insertions(+), 53 deletions(-)

New commits:
commit e4380ae1e7d5330e26f4a382f3a86a6059b9b53f
Author: Kohei Yoshida 
Date:   Thu Mar 22 23:15:07 2012 -0400

Test for changing dimension member visibility & a little cleanup.

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index cb6ab44..965424f 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -1297,6 +1297,17 @@ ScDPObject* createDPFromRange(
 return createDPFromSourceDesc(pDoc, aSheetDesc, aFields, nFieldCount, 
bFilterButton);
 }
 
+ScRange refresh(ScDPObject* pDPObj)
+{
+bool bOverFlow = false;
+ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
+CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
+
+pDPObj->Output(aOutRange.aStart);
+aOutRange = pDPObj->GetOutRange();
+return aOutRange;
+}
+
 ScRange refreshGroups(ScDPCollection* pDPs, ScDPObject* pDPObj)
 {
 // We need to first create group data in the cache, then the group data in
@@ -1307,13 +1318,7 @@ ScRange refreshGroups(ScDPCollection* pDPs, ScDPObject* 
pDPObj)
 CPPUNIT_ASSERT_MESSAGE("There should be only one table linked to this 
cache.", aRefs.size() == 1);
 pDPObj->ReloadGroupTableData();
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
-return aOutRange;
+return refresh(pDPObj);
 }
 
 class AutoCalcSwitch
@@ -1554,13 +1559,7 @@ void Test::testPivotTableLabels()
pDPs->GetCount() == 1);
 pDPObj->SetName(pDPs->CreateNewName());
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
-
+ScRange aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][5] = {
@@ -1616,13 +1615,7 @@ void Test::testPivotTableDateLabels()
pDPs->GetCount() == 1);
 pDPObj->SetName(pDPs->CreateNewName());
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
-
+ScRange aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][5] = {
@@ -1705,12 +1698,7 @@ void Test::testPivotTableFilters()
pDPs->GetCount() == 1);
 pDPObj->SetName(pDPs->CreateNewName());
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
+ScRange aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][2] = {
@@ -1743,8 +1731,7 @@ void Test::testPivotTableFilters()
 OUString aPage(RTL_CONSTASCII_USTRINGPARAM("A"));
 pDim->SetCurrentPage(&aPage);
 pDPObj->SetSaveData(aSaveData);
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
+aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][2] = {
@@ -1773,8 +1760,7 @@ void Test::testPivotTableFilters()
 rEntry.GetQueryItem().mfVal = 1;
 aDesc.SetQueryParam(aQueryParam);
 pDPObj->SetSheetDesc(aDesc);
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
+aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][2] = {
@@ -1852,12 +1838,7 @@ void Test::testPivotTableNamedSource()
pDPs->GetCount() == 1);
 pDPObj->SetName(pDPs->CreateNewName());
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
+ScRange aOutRange = refresh(pDPObj);
 {
 // Expected output table content.  0 = empty cell
 const char* aOutputCheck[][5] = {
@@ -2111,12 +2092,7 @@ void Test::testPivotTableDuplicateDataFields()
pDPs->GetCount() == 1);
 pDPObj->SetName(pDPs->CreateNewName());
 
-bool bOverFlow = false;
-ScRange aOutRange = pDPObj->GetNewOutputRange(bOverFlow);
-CPPUNIT_ASSERT_MESSAGE("Table overflow!?", !bOverFlow);
-
-pDPObj->Output(aOutRange.aStart);
-aOutRange = pDPObj->GetOutRange();
+Sc

[Libreoffice-commits] .: sc/source

2012-03-22 Thread Kohei Yoshida
 sc/source/core/data/dpobject.cxx   |3 +++
 sc/source/ui/docshell/dbdocfun.cxx |2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 76e533a5c6a54ffbbb98880d966e5b060e9626b6
Author: Kohei Yoshida 
Date:   Thu Mar 22 22:38:56 2012 -0400

More bug fixes related to grouped dimension handling.

diff --git a/sc/source/core/data/dpobject.cxx b/sc/source/core/data/dpobject.cxx
index 2f0e510..9e7af6d 100644
--- a/sc/source/core/data/dpobject.cxx
+++ b/sc/source/core/data/dpobject.cxx
@@ -564,6 +564,9 @@ void ScDPObject::ReloadGroupTableData()
 // No dimension data. Most likey it doesn't have any group dimensions.
 return;
 
+if (!mpTableData)
+return;
+
 ScDPGroupTableData* pData = 
dynamic_cast(mpTableData.get());
 if (pData)
 {
diff --git a/sc/source/ui/docshell/dbdocfun.cxx 
b/sc/source/ui/docshell/dbdocfun.cxx
index afb3513..6339e5f 100644
--- a/sc/source/ui/docshell/dbdocfun.cxx
+++ b/sc/source/ui/docshell/dbdocfun.cxx
@@ -1328,7 +1328,7 @@ bool ScDBDocFunc::DataPilotUpdate( ScDPObject* pOldObj, 
const ScDPObject* pNewOb
 pDestObj->ClearTableData();
 
 if (pDestObj->HasGroups())
-pDestObj->ClearTableData();
+pDestObj->ReloadGroupTableData();
 
 pDestObj->InvalidateData(); // before getting the 
new output area
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-03-22 Thread Kohei Yoshida
 sc/inc/dpobject.hxx|1 +
 sc/source/core/data/dpobject.cxx   |   12 
 sc/source/core/data/dpsave.cxx |2 +-
 sc/source/ui/docshell/dbdocfun.cxx |3 +++
 sc/source/ui/view/gridwin2.cxx |7 +++
 5 files changed, 20 insertions(+), 5 deletions(-)

New commits:
commit 1d8b583b0676f9c9eed8efd092445908ebea26d9
Author: Kohei Yoshida 
Date:   Thu Mar 22 22:20:04 2012 -0400

Get correct string value for group item value.

Unlike the old version, the new version no longer stores string value
for non-string items.  We need to call GetFormattedString() for that.

diff --git a/sc/source/core/data/dpsave.cxx b/sc/source/core/data/dpsave.cxx
index e112e6c..5255b72 100644
--- a/sc/source/core/data/dpsave.cxx
+++ b/sc/source/core/data/dpsave.cxx
@@ -1246,7 +1246,7 @@ void 
ScDPSaveData::BuildAllDimensionMembers(ScDPTableData* pData)
 for (size_t j = 0; j < mMemberCount; ++j)
 {
 const ScDPItemData* pMemberData = pData->GetMemberById( nDimIndex, 
rMembers[j] );
-::rtl::OUString aMemName = pMemberData->GetString();
+rtl::OUString aMemName = pData->GetFormattedString(nDimIndex, 
*pMemberData);
 if (iter->GetExistingMemberByName(aMemName))
 // this member instance already exists. nothing to do.
 continue;
commit 7f68157264ec5c474b2a0827d2ee35af20ff0c3d
Author: Kohei Yoshida 
Date:   Thu Mar 22 22:03:55 2012 -0400

When updating pivot table with groups, we need to clear the table data.

diff --git a/sc/inc/dpobject.hxx b/sc/inc/dpobject.hxx
index 917759d..8c1ed99 100644
--- a/sc/inc/dpobject.hxx
+++ b/sc/inc/dpobject.hxx
@@ -138,6 +138,7 @@ public:
 voidInvalidateData();
 void ClearTableData();
 void ReloadGroupTableData();
+bool HasGroups() const;
 
 voidOutput( const ScAddress& rPos );
 ScRange GetNewOutputRange( bool& rOverflow );
diff --git a/sc/source/core/data/dpobject.cxx b/sc/source/core/data/dpobject.cxx
index 9da61d1..2f0e510 100644
--- a/sc/source/core/data/dpobject.cxx
+++ b/sc/source/core/data/dpobject.cxx
@@ -584,6 +584,18 @@ void ScDPObject::ReloadGroupTableData()
 bSettingsChanged = true;
 }
 
+bool ScDPObject::HasGroups() const
+{
+if (!pSaveData)
+return false;
+
+const ScDPDimensionSaveData* pDimData = 
pSaveData->GetExistingDimensionData();
+if (!pDimData)
+return false;
+
+return pDimData->HasGroupDimensions();
+}
+
 void ScDPObject::ClearSource()
 {
 Reference< XComponent > xObjectComp( xSource, UNO_QUERY );
diff --git a/sc/source/ui/docshell/dbdocfun.cxx 
b/sc/source/ui/docshell/dbdocfun.cxx
index c1464b1..afb3513 100644
--- a/sc/source/ui/docshell/dbdocfun.cxx
+++ b/sc/source/ui/docshell/dbdocfun.cxx
@@ -1327,6 +1327,9 @@ bool ScDBDocFunc::DataPilotUpdate( ScDPObject* pOldObj, 
const ScDPObject* pNewOb
 if ( pNewObj == pOldObj && pDestObj->IsImportData() )
 pDestObj->ClearTableData();
 
+if (pDestObj->HasGroups())
+pDestObj->ClearTableData();
+
 pDestObj->InvalidateData(); // before getting the 
new output area
 
 //  make sure the table has a name (not set by dialog)
diff --git a/sc/source/ui/view/gridwin2.cxx b/sc/source/ui/view/gridwin2.cxx
index 0f37c83..bf3402c 100644
--- a/sc/source/ui/view/gridwin2.cxx
+++ b/sc/source/ui/view/gridwin2.cxx
@@ -552,9 +552,8 @@ void ScGridWindow::UpdateDPFromFieldPopupMenu()
 return;
 
 ScDPObject* pDPObj = pDPData->mpDPObj;
-ScDPObject aNewDPObj(*pDPObj);
-aNewDPObj.BuildAllDimensionMembers();
-ScDPSaveData* pSaveData = aNewDPObj.GetSaveData();
+pDPObj->BuildAllDimensionMembers();
+ScDPSaveData* pSaveData = pDPObj->GetSaveData();
 
 bool bIsDataLayout;
 OUString aDimName = pDPObj->GetDimName(pDPData->mnDim, bIsDataLayout);
@@ -601,7 +600,7 @@ void ScGridWindow::UpdateDPFromFieldPopupMenu()
 pDim->UpdateMemberVisibility(aResult);
 
 ScDBDocFunc aFunc(*pViewData->GetDocShell());
-aFunc.DataPilotUpdate(pDPObj, &aNewDPObj, true, false);
+aFunc.DataPilotUpdate(pDPObj, pDPObj, true, false);
 }
 
 void ScGridWindow::UpdateVisibleRange()
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: svtools/source

2012-03-22 Thread Markus Mohrhard
 svtools/source/control/valueacc.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit ffc7968ef27d662bdb2b7418eb0344039312f0bb
Author: Markus Mohrhard 
Date:   Fri Mar 23 02:56:41 2012 +0100

Memcheck: Jump depends on unitialised variable

diff --git a/svtools/source/control/valueacc.cxx 
b/svtools/source/control/valueacc.cxx
index 22680c4..37c48c2 100644
--- a/svtools/source/control/valueacc.cxx
+++ b/svtools/source/control/valueacc.cxx
@@ -48,6 +48,7 @@ ValueSetItem::ValueSetItem( ValueSet& rParent )
 : mrParent(rParent)
 , mnId(0)
 , meType(VALUESETITEM_NONE)
+, mbVisible(true)
 , mpData(NULL)
 , mpxAcc(NULL)
 {
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - sc/qa

2012-03-22 Thread Kohei Yoshida
 sc/qa/unit/ucalc.cxx |  226 ++-
 1 file changed, 225 insertions(+), 1 deletion(-)

New commits:
commit a3d71e97eaf1958a4d5ab41a6b8788c95d85d2bb
Author: Kohei Yoshida 
Date:   Thu Mar 22 21:26:26 2012 -0400

New unit test for pivot table with date grouping.

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 89315af..cb6ab44 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -75,6 +75,7 @@
 #include 
 
 #include 
+#include 
 #include 
 
 #include 
@@ -161,8 +162,8 @@ public:
 void testPivotTableDuplicateDataFields();
 
 void testPivotTableNormalGrouping();
-
 void testPivotTableNumberGrouping();
+void testPivotTableDateGrouping();
 
 void testSheetCopy();
 void testSheetMove();
@@ -222,6 +223,7 @@ public:
 CPPUNIT_TEST(testPivotTableDuplicateDataFields);
 CPPUNIT_TEST(testPivotTableNormalGrouping);
 CPPUNIT_TEST(testPivotTableNumberGrouping);
+CPPUNIT_TEST(testPivotTableDateGrouping);
 CPPUNIT_TEST(testSheetCopy);
 CPPUNIT_TEST(testSheetMove);
 CPPUNIT_TEST(testExternalRef);
@@ -1245,7 +1247,6 @@ ScDPObject* createDPFromSourceDesc(
 ScDPSaveDimension* pDim = aSaveData.GetNewDimensionByName(aDimName);
 pDim->SetOrientation(static_cast(aFields[i].eOrient));
 pDim->SetUsedHierarchy(0);
-pDim->SetShowEmpty(true);
 
 if (aFields[i].eOrient == sheet::DataPilotFieldOrientation_DATA)
 {
@@ -2424,6 +2425,123 @@ void Test::testPivotTableNumberGrouping()
 m_pDoc->DeleteTab(0);
 }
 
+void Test::testPivotTableDateGrouping()
+{
+m_pDoc->InsertTab(0, OUString(RTL_CONSTASCII_USTRINGPARAM("Data")));
+m_pDoc->InsertTab(1, OUString(RTL_CONSTASCII_USTRINGPARAM("Table")));
+
+// Raw data
+const char* aData[][2] = {
+{ "Date", "Value" },
+{ "2011-01-01", "1" },
+{ "2011-03-02", "2" },
+{ "2012-01-04", "3" },
+{ "2012-02-23", "4" },
+{ "2012-02-24", "5" },
+{ "2012-03-15", "6" },
+{ "2011-09-03", "7" },
+{ "2012-12-25", "8" }
+};
+
+// Dimension definition
+DPFieldDef aFields[] = {
+{ "Date", sheet::DataPilotFieldOrientation_ROW, 0 },
+{ "Value", sheet::DataPilotFieldOrientation_DATA, 
sheet::GeneralFunction_SUM },
+};
+
+ScAddress aPos(1,1,0);
+ScRange aDataRange = insertRangeData(m_pDoc, aPos, aData, 
SAL_N_ELEMENTS(aData));
+CPPUNIT_ASSERT_MESSAGE("failed to insert range data at correct position", 
aDataRange.aStart == aPos);
+
+ScDPObject* pDPObj = createDPFromRange(
+m_pDoc, aDataRange, aFields, SAL_N_ELEMENTS(aFields), false);
+
+ScDPCollection* pDPs = m_pDoc->GetDPCollection();
+bool bSuccess = pDPs->InsertNewTable(pDPObj);
+
+CPPUNIT_ASSERT_MESSAGE("failed to insert a new pivot table object into 
document.", bSuccess);
+CPPUNIT_ASSERT_MESSAGE("there should be only one data pilot table.",
+   pDPs->GetCount() == 1);
+pDPObj->SetName(pDPs->CreateNewName());
+
+ScDPSaveData* pSaveData = pDPObj->GetSaveData();
+CPPUNIT_ASSERT_MESSAGE("No save data !?", pSaveData);
+ScDPDimensionSaveData* pDimData = pSaveData->GetDimensionData();
+CPPUNIT_ASSERT_MESSAGE("No dimension data !?", pDimData);
+
+rtl::OUString aBaseDimName(RTL_CONSTASCII_USTRINGPARAM("Date"));
+
+ScDPNumGroupInfo aInfo;
+aInfo.mbEnable = true;
+aInfo.mbAutoStart = true;
+aInfo.mbAutoEnd = true;
+{
+// Turn the Date dimension into months.  The first of the date
+// dimensions is always a number-group dimension which replaces the
+// original dimension.
+ScDPSaveNumGroupDimension aGroup(aBaseDimName, aInfo, 
sheet::DataPilotFieldGroupBy::MONTHS);
+pDimData->AddNumGroupDimension(aGroup);
+}
+
+{
+// Add quarter dimension.  This will be an additional dimension.
+rtl::OUString aGroupDimName =
+pDimData->CreateDateGroupDimName(
+sheet::DataPilotFieldGroupBy::QUARTERS, *pDPObj, true, NULL);
+ScDPSaveGroupDimension aGroupDim(aBaseDimName, aGroupDimName);
+aGroupDim.SetDateInfo(aInfo, sheet::DataPilotFieldGroupBy::QUARTERS);
+pDimData->AddGroupDimension(aGroupDim);
+
+// Set orientation.
+ScDPSaveDimension* pDim = pSaveData->GetDimensionByName(aGroupDimName);
+pDim->SetOrientation(sheet::DataPilotFieldOrientation_ROW);
+pSaveData->SetPosition(pDim, 0); // set it to the left end.
+}
+
+{
+// Add year dimension.  This is a new dimension also.
+rtl::OUString aGroupDimName =
+pDimData->CreateDateGroupDimName(
+sheet::DataPilotFieldGroupBy::YEARS, *pDPObj, true, NULL);
+ScDPSaveGroupDimension aGroupDim(aBaseDimName, aGroupDimName);
+aGroupDim.SetDateInfo(aInfo, sheet::DataPilotFieldGroupBy::YEARS);
+pDimData->AddGroupDimension(aGroupDim);
+
+// Set o

[Libreoffice-commits] .: sc/source

2012-03-22 Thread Markus Mohrhard
 sc/source/core/tool/scmatrix.cxx |   33 -
 1 file changed, 16 insertions(+), 17 deletions(-)

New commits:
commit bbe088bc757fa120f50e4e104bdcd46c90f1feb1
Author: Markus Mohrhard 
Date:   Fri Mar 23 02:05:36 2012 +0100

cache calls to mdds:mixed_type_matrix::size, related fdo#47299

diff --git a/sc/source/core/tool/scmatrix.cxx b/sc/source/core/tool/scmatrix.cxx
index 5256e9a..47ed6e9 100644
--- a/sc/source/core/tool/scmatrix.cxx
+++ b/sc/source/core/tool/scmatrix.cxx
@@ -170,6 +170,7 @@ class ScMatrixImpl
 ScMatrix::DensityType meType;
 ScInterpreter* pErrorInterpreter;
 boolmbCloneIfConst; // Whether the matrix is cloned with a 
CloneIfConst() call.
+MatrixImplType::size_pair_type  maCachedSize;
 
 ScMatrixImpl();
 ScMatrixImpl(const ScMatrixImpl&);
@@ -243,6 +244,7 @@ ScMatrixImpl::ScMatrixImpl(SCSIZE nC, SCSIZE nR, 
ScMatrix::DensityType eType) :
 pErrorInterpreter(NULL),
 mbCloneIfConst(true)
 {
+maCachedSize = maMat.size();
 }
 
 ScMatrixImpl::~ScMatrixImpl()
@@ -253,6 +255,7 @@ ScMatrixImpl::~ScMatrixImpl()
 void ScMatrixImpl::Clear()
 {
 maMat.clear();
+maCachedSize = maMat.size();
 }
 
 void ScMatrixImpl::SetImmutable(bool bVal)
@@ -268,6 +271,7 @@ bool ScMatrixImpl::IsImmutable() const
 void ScMatrixImpl::Resize(SCSIZE nC, SCSIZE nR)
 {
 maMat.resize(nR, nC);
+maCachedSize = maMat.size();
 }
 
 ScMatrix::DensityType ScMatrixImpl::GetDensityType() const
@@ -282,40 +286,35 @@ void ScMatrixImpl::SetErrorInterpreter( ScInterpreter* p)
 
 void ScMatrixImpl::GetDimensions( SCSIZE& rC, SCSIZE& rR) const
 {
-MatrixImplType::size_pair_type aDims = maMat.size();
-rR = aDims.first;
-rC = aDims.second;
+rR = maCachedSize.first;
+rC = maCachedSize.second;
 }
 
 SCSIZE ScMatrixImpl::GetElementCount() const
 {
-MatrixImplType::size_pair_type aDims = maMat.size();
-return aDims.first * aDims.second;
+return maCachedSize.first * maCachedSize.second;
 }
 
 bool ScMatrixImpl::ValidColRow( SCSIZE nC, SCSIZE nR) const
 {
-MatrixImplType::size_pair_type aDims = maMat.size();
-return nR < aDims.first && nC < aDims.second;
+return nR < maCachedSize.first && nC < maCachedSize.second;
 }
 
 bool ScMatrixImpl::ValidColRowReplicated( SCSIZE & rC, SCSIZE & rR ) const
 {
-pair aDims = maMat.size();
-
-if (aDims.second == 1 && aDims.first == 1)
+if (maCachedSize.second == 1 && maCachedSize.first == 1)
 {
 rC = 0;
 rR = 0;
 return true;
 }
-else if (aDims.second == 1 && rR < aDims.first)
+else if (maCachedSize.second == 1 && rR < maCachedSize.first)
 {
 // single column matrix.
 rC = 0;
 return true;
 }
-else if (aDims.first == 1 && rC < aDims.second)
+else if (maCachedSize.first == 1 && rC < maCachedSize.second)
 {
 // single row matrix.
 rR = 0;
@@ -639,8 +638,7 @@ bool ScMatrixImpl::IsNumeric() const
 
 void ScMatrixImpl::MatCopy(ScMatrixImpl& mRes) const
 {
-MatrixImplType::size_pair_type s1 = maMat.size(), s2 = mRes.maMat.size();
-if (s1.first > s2.first || s1.second > s2.second)
+if (maCachedSize.first > mRes.maCachedSize.first || maCachedSize.second > 
mRes.maCachedSize.second)
 {
 // destination matrix is not large enough.
 OSL_FAIL("ScMatrixImpl::MatCopy: dimension error");
@@ -648,12 +646,14 @@ void ScMatrixImpl::MatCopy(ScMatrixImpl& mRes) const
 }
 
 mRes.maMat.assign(maMat);
+mRes.maCachedSize = mRes.maMat.size();
 }
 
 void ScMatrixImpl::MatTrans(ScMatrixImpl& mRes) const
 {
 mRes.maMat = maMat;
 mRes.maMat.transpose();
+mRes.maCachedSize = mRes.maMat.size();
 }
 
 void ScMatrixImpl::FillDouble( double fVal, SCSIZE nC1, SCSIZE nR1, SCSIZE 
nC2, SCSIZE nR2 )
@@ -720,8 +720,7 @@ template 
 bool EvalMatrix(const MatrixImplType& rMat)
 {
 _Evaluator aEval;
-pair aDim = rMat.size();
-size_t nRows = aDim.first, nCols = aDim.second;
+size_t nRows = rMat.size().first, nCols = rMat.size().second;
 for (size_t i = 0; i < nRows; ++i)
 {
 for (size_t j = 0; j < nCols; ++j)
@@ -911,7 +910,7 @@ size_t ScMatrixImpl::Count(bool bCountStrings) const
 
 void ScMatrixImpl::CalcPosition(SCSIZE nIndex, SCSIZE& rC, SCSIZE& rR) const
 {
-SCSIZE nRowSize = maMat.size().first;
+SCSIZE nRowSize = maCachedSize.first;
 rC = nIndex / nRowSize;
 rR = nIndex - rC*nRowSize;
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: solenv/gbuild

2012-03-22 Thread Norbert Thiebaud
 solenv/gbuild/gbuild.mk |4 
 1 file changed, 4 insertions(+)

New commits:
commit 7be7f9a7863cbd862dd5a5462c508db03c56c8d1
Author: Norbert Thiebaud 
Date:   Thu Mar 22 17:26:11 2012 -0500

gbuild: allow the tinderboxes to bypass the dependencies generations

diff --git a/solenv/gbuild/gbuild.mk b/solenv/gbuild/gbuild.mk
index 119f6e5..5ad2378 100644
--- a/solenv/gbuild/gbuild.mk
+++ b/solenv/gbuild/gbuild.mk
@@ -113,8 +113,12 @@ endif
 
 # for clean, setuplocal and removelocal goals we switch off dependencies
 ifneq ($(filter cleanpackmodule clean setuplocal removelocal showdeliverables 
help debugrun,$(MAKECMDGOALS)),)
+ifdef TINDERBUILD
 gb_FULLDEPS := $(false)
 else
+gb_FULLDEPS := $(false)
+endif
+else
 gb_FULLDEPS := $(true)
 endif
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 3 commits - javaunohelper/Jar_juh.mk jurt/Jar_jurt.mk scripting/Jar_ScriptFramework.mk scripting/Jar_ScriptProviderForBeanShell.mk scripting/Jar_ScriptProviderForJava.mk scrip

2012-03-22 Thread Stephan Bergmann
 javaunohelper/Jar_juh.mk   |2 
++
 jurt/Jar_jurt.mk   |3 
+++
 scripting/Jar_ScriptFramework.mk   |2 
++
 scripting/Jar_ScriptProviderForBeanShell.mk|4 
++--
 scripting/Jar_ScriptProviderForJava.mk |5 
+
 scripting/Jar_ScriptProviderForJavaScript.mk   |2 
+-
 scripting/java/Framework/MANIFEST.MF   |2 
++
 scripting/java/com/sun/star/script/framework/provider/java/MANIFEST.MF |2 
++
 solenv/gbuild/Jar.mk   |2 
+-
 9 files changed, 20 insertions(+), 4 deletions(-)

New commits:
commit 2413fd4f9f784fbde821b087b78ecf3a6afb4f31
Author: Stephan Bergmann 
Date:   Thu Mar 22 23:28:06 2012 +0100

Fix URE jar manifests after gbuild'ification

diff --git a/javaunohelper/Jar_juh.mk b/javaunohelper/Jar_juh.mk
index 142805d..ef319ba 100644
--- a/javaunohelper/Jar_juh.mk
+++ b/javaunohelper/Jar_juh.mk
@@ -35,6 +35,8 @@ $(eval $(call gb_Jar_add_jars,juh,\
 
 $(eval $(call gb_Jar_set_packageroot,juh,com))
 
+$(eval $(call gb_Jar_set_jarclasspath,juh,ridl.jar jurt.jar ../../lib/ 
../bin/))
+
 $(eval $(call gb_Jar_set_manifest,juh,$(SRCDIR)/javaunohelper/util/manifest))
 
 $(eval $(call gb_Jar_set_componentfile,juh,javaunohelper/util/juh,URE))
diff --git a/jurt/Jar_jurt.mk b/jurt/Jar_jurt.mk
index 8a6a87d..4cd6bb8 100644
--- a/jurt/Jar_jurt.mk
+++ b/jurt/Jar_jurt.mk
@@ -36,6 +36,9 @@ $(eval $(call gb_Jar_set_packageroot,jurt,com))
 
 $(eval $(call gb_Jar_set_manifest,jurt,$(SRCDIR)/jurt/util/manifest))
 
+$(eval $(call gb_Jar_set_jarclasspath,jurt, \
+ridl.jar unoloader.jar ../../lib/ ../bin/))
+
 $(eval $(call gb_Jar_add_sourcefiles,jurt,\
 jurt/com/sun/star/comp/bridgefactory/BridgeFactory \
 jurt/com/sun/star/comp/connections/Acceptor \
commit 91d60be7eeafa1f7872bda452bfb38936003f517
Author: Stephan Bergmann 
Date:   Thu Mar 22 23:27:32 2012 +0100

fdo#46102: Fix scripting jar manifests after gbuild'ification

diff --git a/scripting/Jar_ScriptFramework.mk b/scripting/Jar_ScriptFramework.mk
index 05774b9..1e6a4be 100644
--- a/scripting/Jar_ScriptFramework.mk
+++ b/scripting/Jar_ScriptFramework.mk
@@ -37,6 +37,8 @@ $(eval $(call gb_Jar_add_jars,ScriptFramework,\
 
 $(eval $(call 
gb_Jar_set_componentfile,ScriptFramework,scripting/java/ScriptFramework,OOO))
 
+$(eval $(call 
gb_Jar_set_manifest,ScriptFramework,$(SRCDIR)/scripting/java/Framework/MANIFEST.MF))
+
 $(eval $(call gb_Jar_set_packageroot,ScriptFramework,com))
 
 $(eval $(call gb_Jar_add_sourcefiles,ScriptFramework,\
diff --git a/scripting/Jar_ScriptProviderForBeanShell.mk 
b/scripting/Jar_ScriptProviderForBeanShell.mk
index 753ea10..aee4ad5 100644
--- a/scripting/Jar_ScriptProviderForBeanShell.mk
+++ b/scripting/Jar_ScriptProviderForBeanShell.mk
@@ -45,10 +45,10 @@ $(eval $(call 
gb_Jar_set_manifest,ScriptProviderForBeanShell,$(SRCDIR)/scripting
 #TODO: Ensure "file://$(BSH_JAR)" is a proper file URL:
 ifeq ($(SYSTEM_BSH),YES)
 $(eval $(call gb_Jar_set_jarclasspath,ScriptProviderForBeanShell, \
-unoil.jar ScriptFramework.jar file://$(BSH_JAR)))
+ScriptFramework.jar file://$(BSH_JAR)))
 else
 $(eval $(call gb_Jar_set_jarclasspath,ScriptProviderForBeanShell, \
-unoil.jar ScriptFramework.jar bsh.jar))
+ScriptFramework.jar bsh.jar))
 endif
 
 $(eval $(call 
gb_Jar_set_componentfile,ScriptProviderForBeanShell,scripting/java/ScriptProviderForBeanShell,OOO))
diff --git a/scripting/Jar_ScriptProviderForJava.mk 
b/scripting/Jar_ScriptProviderForJava.mk
index 39b747b..19e1995 100644
--- a/scripting/Jar_ScriptProviderForJava.mk
+++ b/scripting/Jar_ScriptProviderForJava.mk
@@ -38,6 +38,11 @@ $(eval $(call gb_Jar_add_jars,ScriptProviderForJava,\
 
 $(eval $(call 
gb_Jar_set_componentfile,ScriptProviderForJava,scripting/java/ScriptProviderForJava,OOO))
 
+$(eval $(call gb_Jar_set_jarclasspath,ScriptProviderForJava, \
+ScriptFramework.jar))
+
+$(eval $(call 
gb_Jar_set_manifest,ScriptProviderForJava,$(SRCDIR)/scripting/java/com/sun/star/script/framework/provider/java/MANIFEST.MF))
+
 $(eval $(call gb_Jar_set_packageroot,ScriptProviderForJava,com))
 
 $(eval $(call gb_Jar_add_sourcefiles,ScriptProviderForJava,\
diff --git a/scripting/Jar_ScriptProviderForJavaScript.mk 
b/scripting/Jar_ScriptProviderForJavaScript.mk
index 5f69085..bd405cc 100644
--- a/scripting/Jar_ScriptProviderForJavaScript.mk
+++ b/scripting/Jar_ScriptProviderForJavaScript.mk
@@ -40,7 +40,7 @@ $(eval $(call gb_Jar_add_jars,ScriptProviderForJavaScript,\
 $(eval $(call 
gb_Jar_set_manifest,ScriptProviderForJavaScript,$(SRCDIR)/scripting/java/com/sun/star/script/framework/provider/javascript/MANIFEST.MF))
 
 $(eval $(call gb_Jar_set_jarclasspath,ScriptProviderForJavaScript, \
-unoil.jar ScriptFramework.jar js.jar))
+ScriptFramework.jar j

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

2012-03-22 Thread Michael Stahl
 sw/source/core/edit/edundo.cxx |4 
 sw/source/core/frmedt/feshview.cxx |8 
 2 files changed, 8 insertions(+), 4 deletions(-)

New commits:
commit f24153cded54954da7f0d80941707715c78e4627
Author: Michael Stahl 
Date:   Thu Mar 22 23:03:05 2012 +0100

fdo#39003: fix autoformat Undo cursors:

SwEditShell::Undo: Because the Undo actions nowadays create new shell
cursors directly (while previously they worked on a SwUndoIter), it is
necessary to remove the shell cursor ring before Pop() because Pop(false)
only restores the current cursor from the stack, not the rest of the ring.
(regression from CWS undoapi, 1ba0c1878116ada5dc4ebdecdb1e0a3da9758547)

diff --git a/sw/source/core/edit/edundo.cxx b/sw/source/core/edit/edundo.cxx
index 95c9252..26b64b0 100644
--- a/sw/source/core/edit/edundo.cxx
+++ b/sw/source/core/edit/edundo.cxx
@@ -142,6 +142,10 @@ bool SwEditShell::Undo(sal_uInt16 const nCount)
 .getStr());
 }
 
+if (bRestoreCrsr)
+{   // fdo#39003 Pop does not touch the rest of the cursor ring
+KillPams(); // so call this first to get rid of unwanted cursors
+}
 Pop( !bRestoreCrsr );
 
 GetDoc()->SetRedlineMode( eOld );
commit 06d1d69fc8c214251cf9dbc7f87756651657d3bf
Author: Michael Stahl 
Date:   Thu Mar 22 14:52:41 2012 +0100

SwFEShell::ShouldObjectBeSelected leaks pPos

diff --git a/sw/source/core/frmedt/feshview.cxx 
b/sw/source/core/frmedt/feshview.cxx
index 7c6dd99..be852a9 100644
--- a/sw/source/core/frmedt/feshview.cxx
+++ b/sw/source/core/frmedt/feshview.cxx
@@ -1163,13 +1163,13 @@ sal_Bool SwFEShell::ShouldObjectBeSelected(const Point& 
rPt)
 dynamic_cast(pCntntFrm);
 if ( pTxtFrm )
 {
-SwPosition* pPos =
-new SwPosition( *(pTxtFrm->GetTxtNode()) );
+SwPosition aPos( *(pTxtFrm->GetTxtNode()) );
 Point aTmpPt( rPt );
-if ( pTxtFrm->GetKeyCrsrOfst( pPos, aTmpPt ) )
+if (pTxtFrm->GetKeyCrsrOfst(&aPos, aTmpPt))
 {
 SwRect aCursorCharRect;
-if ( pTxtFrm->GetCharRect( 
aCursorCharRect, *pPos ) )
+if (pTxtFrm->GetCharRect(aCursorCharRect,
+aPos))
 {
 if ( aCursorCharRect.IsOver( SwRect( 
pObj->GetLastBoundRect() ) ) )
 {
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 6 commits - android/Bootstrap android/experiments configure.in sal/android sal/osl vcl/source

2012-03-22 Thread Tor Lillqvist
 android/Bootstrap/src/org/libreoffice/android/Bootstrap.java   
 |   20 -
 android/experiments/DocumentLoader/AndroidManifest.xml 
 |2 
 android/experiments/DocumentLoader/Makefile
 |  180 +++---
 android/experiments/DocumentLoader/jni/Android.mk  
 |8 
 android/experiments/DocumentLoader/jni/Application.mk  
 |3 
 android/experiments/DocumentLoader/local.properties.in 
 |2 
 android/experiments/DocumentLoader/project.properties  
 |3 
 
android/experiments/DocumentLoader/src/org/libreoffice/android/examples/DocumentLoader.java
 |   45 +-
 configure.in   
 |7 
 sal/android/lo-bootstrap.c 
 |  130 +--
 sal/osl/unx/process_impl.cxx   
 |3 
 vcl/source/app/svmain.cxx  
 |   22 +
 12 files changed, 331 insertions(+), 94 deletions(-)

New commits:
commit 3e425dfa4b9c2a0198a13b2fb18364a32603080f
Author: Tor Lillqvist 
Date:   Thu Mar 22 22:49:16 2012 +0200

More Android hacking

diff --git a/android/experiments/DocumentLoader/Makefile 
b/android/experiments/DocumentLoader/Makefile
index 5612fbf..0c31954 100644
--- a/android/experiments/DocumentLoader/Makefile
+++ b/android/experiments/DocumentLoader/Makefile
@@ -3,6 +3,19 @@ include ../../../config_host.mk
 # The package of this app
 APP_PACKAGE=org.libreoffice.android.examples
 
+# We can't keep assuming APP_DATA_PATH like this, surely this can vary with
+# Android versions and whatnot, this is temporary and works at least with the
+# SDK 16 emulator...
+
+# Probably would be best to just stop fooling around with the possibilities to
+# set various stuff with the -env command line parameters (and environment
+# variables?) and in a plethora of rc files, and hardcode construction of
+# *all* required pathnames based on the app installation location for Android
+# (and iOS), etc. We don't really win anything by having so many layers of
+# configurability on platforms like Android and iOS where apps based on LO
+# code are very much self-contained pre-packaged thingies.
+APP_DATA_PATH=/data/data/$(APP_PACKAGE)
+
 SODEST=libs/armeabi-v7a
 OBJLOCAL=obj/local/armeabi-v7a
 
@@ -31,32 +44,65 @@ copy-stuff:
 # them.
 #
for F in $(strip \
+  analysislo \
+  basebmplo \
+  basegfxlo \
+  bootstrap.uno \
   comphelpgcc3 \
+  datelo \
+  expwrap.uno \
+  fastsax.uno \
+  fileacc \
+  fontconfig \
+  forlo \
+  foruilo \
+  freetype \
+  frmlo \
   gcc3_uno \
   i18nisolang1gcc3 \
+  i18npaperlo \
+  i18npool.uno \
   i18nutilgcc3 \
   icudatalo \
   icui18nlo \
+  iculelo \
   icuuclo \
+  introspection.uno \
   java_uno \
   juh \
   juhx \
   jvmaccessgcc3 \
+  jvmfwk \
   lo-bootstrap \
   localedata_en \
   localedata_others \
   mergedlo \
+  msfilterlo \
+  ooxlo \
+  reflection.uno \
   reg \
+  saxlo \
+  sclo \
+  scfiltlo \
+  sotlo \
+  stocservices.uno \
   store \
+  svllo \
+  textinstream.uno \
+  tllo \
   ucbhelper4gcc3 \
   uno_cppu \
+  uno_cppuhelpergcc3 \
   uno_sal \
   uno_salhelpergcc3 \
   uno_cppuhelpergcc3 \
+  unoxmllo \
+  utllo \
+  vbahelperlo \
+  vcllo \
   xml2 \
   xmlreader \
-  bootstrap.uno \
-  i18npool.uno \
+  xstor \
  ); do \
$(call COPYSO,$(OUTDIR)/lib/lib$${F}.so); \
done
@@ -66,14 +112,58 @@ copy-stuff:
 #
 # Then other "assets". Let the directory structure under assets mimic
 # that under solver for now.
-   mkdir -p assets/bin assets/lib assets/xml/ure 
assets/ComponentTarget/i18npool/util
+   mkdir -p assets/bin/ure assets/lib assets/program assets/xml/ure 
assets

[Libreoffice-commits] .: Branch 'feature/tubes2' - sc/Library_sc.mk tubes/Module_tubes.mk tubes/Package_inc.mk

2012-03-22 Thread Eike Rathke
 sc/Library_sc.mk  |   23 +++
 tubes/Module_tubes.mk |1 +
 tubes/Package_inc.mk  |   39 +++
 3 files changed, 63 insertions(+)

New commits:
commit 276c2a616000acc8890740ab9503da3b73e872e6
Author: Eike Rathke 
Date:   Thu Mar 22 21:31:31 2012 +0100

tubes: pull in tubes headers and linkage

diff --git a/sc/Library_sc.mk b/sc/Library_sc.mk
index a3554ce..42fd0fd 100644
--- a/sc/Library_sc.mk
+++ b/sc/Library_sc.mk
@@ -45,6 +45,28 @@ $(eval $(call gb_Library_add_api,sc,\
 udkapi \
 ))
 
+
+ifeq ($(ENABLE_TELEPATHY),TRUE)
+
+$(eval $(call gb_Library_add_linked_libs,sc,\
+   tubes \
+))
+
+$(eval $(call gb_Library_add_cxxflags,sc,\
+   $$(TELEPATHY_CFLAGS) \
+))
+
+$(eval $(call gb_Library_add_libs,sc,\
+   $$(TELEPATHY_LIBS) \
+))
+
+$(eval $(call gb_Library_use_externals,sc,\
+   telepathy \
+))
+
+endif
+
+
 $(eval $(call gb_Library_add_linked_libs,sc,\
avmedia \
basegfx \
@@ -313,6 +335,7 @@ $(eval $(call gb_Library_add_exception_objects,sc,\
sc/source/ui/cctrl/popmenu \
sc/source/ui/cctrl/tbinsert \
sc/source/ui/cctrl/tbzoomsliderctrl \
+   sc/source/ui/collab/collab \
sc/source/ui/collab/sendfunc \
sc/source/ui/dbgui/asciiopt \
sc/source/ui/dbgui/consdlg \
diff --git a/tubes/Module_tubes.mk b/tubes/Module_tubes.mk
index fb1c9d7..5746ce5 100644
--- a/tubes/Module_tubes.mk
+++ b/tubes/Module_tubes.mk
@@ -31,6 +31,7 @@ ifeq ($(ENABLE_TELEPATHY),TRUE)
 
 $(eval $(call gb_Module_add_targets,tubes,\
Library_tubes \
+   Package_inc \
Executable_liboapprover \
 ))
 
diff --git a/tubes/Package_inc.mk b/tubes/Package_inc.mk
new file mode 100644
index 000..3ac8876
--- /dev/null
+++ b/tubes/Package_inc.mk
@@ -0,0 +1,39 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+# Version: MPL 1.1 / GPLv3+ / LGPLv3+
+#
+# The contents of this file are subject to the Mozilla Public License Version
+# 1.1 (the "License"); you may not use this file except in compliance with
+# the License or as specified alternatively below. You may obtain a copy of
+# the License at http://www.mozilla.org/MPL/
+#
+# Software distributed under the License is distributed on an "AS IS" basis,
+# WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
+# for the specific language governing rights and limitations under the
+# License.
+#
+# Major Contributor(s):
+# Copyright (C) 2012 Red Hat, Inc., Eike Rathke 
+#
+# All Rights Reserved.
+#
+# For minor contributions see the git repository.
+#
+# Alternatively, the contents of this file may be used under the terms of
+# either the GNU General Public License Version 3 or later (the "GPLv3+"), or
+# the GNU Lesser General Public License Version 3 or later (the "LGPLv3+"),
+# in which case the provisions of the GPLv3+ or the LGPLv3+ are applicable
+# instead of those above.
+
+$(eval $(call gb_Package_Package,tubes_inc,$(SRCDIR)/tubes/inc))
+
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/conference.hxx,tubes/conference.hxx))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/constants.h,tubes/constants.h))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/contact-list.hxx,tubes/contact-list.hxx))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/file-transfer-helper.h,tubes/file-transfer-helper.h))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/manager.hxx,tubes/manager.hxx))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/packet.hxx,tubes/packet.hxx))
+$(eval $(call 
gb_Package_add_file,tubes_inc,inc/tubes/tubesdllapi.h,tubes/tubesdllapi.h))
+
+#$(eval $(call gb_Package_add_file,tubes_inc,inc/tubes/,tubes/))
+
+# vim: set noet sw=4 ts=4:
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-03-22 Thread Michael Meeks
 sc/source/ui/collab/sendfunc.cxx |   38 ++
 sc/source/ui/inc/docfunc.hxx |3 +--
 2 files changed, 39 insertions(+), 2 deletions(-)

New commits:
commit 96fc7b84dd6820acd6fb59116ee69bf383bbdb80
Author: Michael Meeks 
Date:   Thu Mar 22 19:44:44 2012 +

tubes: re-use autorecovery to serialize to a file for exchange

diff --git a/sc/source/ui/collab/sendfunc.cxx b/sc/source/ui/collab/sendfunc.cxx
index f1aec1b..9d9a10c 100644
--- a/sc/source/ui/collab/sendfunc.cxx
+++ b/sc/source/ui/collab/sendfunc.cxx
@@ -34,6 +34,14 @@
 #include "docsh.hxx"
 #include "docfunc.hxx"
 
+// new file send/recv fun ...
+#include 
+#include 
+#include 
+#include 
+
+namespace css = ::com::sun::star;
+
 namespace {
 
 rtl::OUString cellToString( ScBaseCell *pCell )
@@ -282,6 +290,32 @@ class ScDocFuncSend : public ScDocFunc
 mpChain->RecvMessage( rOp.toString() );
 }
 
+void SendFile( const rtl::OUString &rURL )
+{
+(void)rURL;
+
+String aTmpPath = utl::TempFile::CreateTempName();
+aTmpPath.Append( rtl::OUString( ".ods" ) );
+
+rtl::OUString aFileURL;
+::utl::LocalFileHelper::ConvertPhysicalNameToURL( aTmpPath, aFileURL );
+
+::comphelper::MediaDescriptor aDescriptor;
+// some issue with hyperlinks:
+aDescriptor[::comphelper::MediaDescriptor::PROP_DOCUMENTBASEURL()] <<= 
::rtl::OUString();
+try {
+css::uno::Reference< css::document::XDocumentRecovery > 
xDocRecovery(
+rDocShell.GetBaseModel(), css::uno::UNO_QUERY_THROW);
+
+xDocRecovery->storeToRecoveryFile( aFileURL, 
aDescriptor.getAsConstPropertyValueList() );
+} catch (const css::uno::Exception &ex) {
+fprintf( stderr, "exception foo !\n" );
+}
+
+fprintf( stderr, "Temp file is '%s'\n",
+ rtl::OUStringToOString( aFileURL, RTL_TEXTENCODING_UTF8 
).getStr() );
+}
+
 public:
 // FIXME: really ScDocFunc should be an abstract base, so
 // we don't need the rDocSh hack/pointer
@@ -314,6 +348,10 @@ public:
 aOp.appendString( rText );
 aOp.appendBool( bApi );
 SendMessage( aOp );
+
+if ( rtl::OUString( rText ) == "saveme" )
+SendFile( rText );
+
 return true; // needs some code auditing action
 }
 
diff --git a/sc/source/ui/inc/docfunc.hxx b/sc/source/ui/inc/docfunc.hxx
index e47442e..1303f5e 100644
--- a/sc/source/ui/inc/docfunc.hxx
+++ b/sc/source/ui/inc/docfunc.hxx
@@ -57,7 +57,7 @@ struct ScCellMergeOption;
 
 class ScDocFunc
 {
-private:
+protected:
 ScDocShell&rDocShell;
 
 sal_BoolAdjustRowHeight( const ScRange& rRange, sal_Bool bPaint = 
true );
@@ -67,7 +67,6 @@ private:
 sal_Bool& rCancel, sal_Bool bApi );
 voidNotifyInputHandler( const ScAddress& rPos );
 
-protected:
 ScDocFunc( ScDocShell& rDocSh ): rDocShell(rDocSh) {}
 public:
 virtual ~ScDocFunc() {}
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes2' - 2 commits - configure.in tubes/source

2012-03-22 Thread Will Thompson
 configure.in |2 +-
 tubes/source/manager.cxx |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 5cbdd431f1ea30904050eb1f2339c78209c4d401
Author: Will Thompson 
Date:   Thu Mar 22 19:13:09 2012 +

configure: Depend on new enough tp-glib for FT api

diff --git a/configure.in b/configure.in
index 7fdabdb..34a8b8b 100644
--- a/configure.in
+++ b/configure.in
@@ -8369,7 +8369,7 @@ AC_MSG_CHECKING([whether to enable Telepathy support])
 if test "$_os" != "WINNT" -a "$_os" != "Darwin" -a "$enable_telepathy" = 
"yes"; then
 ENABLE_TELEPATHY="TRUE"
 AC_MSG_RESULT([yes])
-PKG_CHECK_MODULES( TELEPATHY, telepathy-glib >= 0.16.4 glib-2.0 
gobject-2.0 gthread-2.0 gio-2.0 dbus-1 dbus-glib-1 )
+PKG_CHECK_MODULES( TELEPATHY, telepathy-glib >= 0.17.1 glib-2.0 
gobject-2.0 gthread-2.0 gio-2.0 dbus-1 dbus-glib-1 )
 else
 AC_MSG_RESULT([no])
 fi
commit 362ddfd74b9eb0bc16bd78998ca62edc7aa327f7
Author: Will Thompson 
Date:   Thu Mar 22 17:35:00 2012 +

tubes: don't bypass approval for tubes

Setting BypassApproval makes incoming channels skip the approver
step—the step which triggers the dialog box asking whether the user
wants to accept the offer.

diff --git a/tubes/source/manager.cxx b/tubes/source/manager.cxx
index 1f6e05c..8403270 100644
--- a/tubes/source/manager.cxx
+++ b/tubes/source/manager.cxx
@@ -303,7 +303,7 @@ bool TeleManager::connect()
 
 pImpl->mpClient = tp_simple_handler_new_with_factory(
 pFactory,   // factory
-TRUE,   // bypass_approval
+FALSE,  // bypass_approval
 FALSE,  // requests
 getFullClientName().getStr(),   // name
 FALSE,  // uniquify
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes2' - 2 commits - sc/inc sc/source

2012-03-22 Thread Michael Meeks
 sc/inc/postit.hxx|2 
 sc/source/ui/collab/sendfunc.cxx |  204 ++-
 sc/source/ui/inc/docfunc.hxx |2 
 3 files changed, 181 insertions(+), 27 deletions(-)

New commits:
commit bc05d8964c9cfeba00b21b4ba19411da2226d07c
Author: Michael Meeks 
Date:   Thu Mar 22 17:39:25 2012 +

tubes: escape and quote strings if necessary.

diff --git a/sc/source/ui/collab/sendfunc.cxx b/sc/source/ui/collab/sendfunc.cxx
index d7a30a3..f1aec1b 100644
--- a/sc/source/ui/collab/sendfunc.cxx
+++ b/sc/source/ui/collab/sendfunc.cxx
@@ -38,13 +38,13 @@ namespace {
 
 rtl::OUString cellToString( ScBaseCell *pCell )
 {
-(void)pCell;
+(void)pCell; // FIXME: implement me
 return rtl::OUString();
 }
 
 ScBaseCell *stringToCell( const rtl::OUString &rString )
 {
-(void)rString;
+(void)rString; // FIXME: implement me
 return NULL;
 }
 
@@ -66,20 +66,20 @@ public:
   appendSeparator();
   }
 
-  void appendString( const String &rStr )
+  void appendString( const rtl::OUString &rStr )
   {
-  String aQuoted( rStr );
-  if ( ScGlobal::FindUnquoted( aQuoted, sal_Unicode( '"' ) ) != 
STRING_NOTFOUND )
+  if ( rStr.indexOf( sal_Unicode( '"' ) ) >= 0 ||
+   rStr.indexOf( sal_Unicode( ';' ) ) >= 0 )
+  {
+  String aQuoted( rStr );
   ScGlobal::AddQuotes( aQuoted, sal_Unicode( '"' ) );
-  aMessage.append( aQuoted );
+  aMessage.append( aQuoted );
+  }
+  else
+  aMessage.append( rStr );
   appendSeparator();
   }
 
-  void appendString( const rtl::OUString &rStr )
-  {
-  aMessage.append( String( rStr ) );
-  }
-
   void appendAddress( const ScAddress &rPos )
   {
   rtl::OUString aStr;
@@ -124,38 +124,49 @@ public:
 {
 // will need to handle escaping etc.
 // Surely someone else wrote this before ! [!?]
-sal_Int32 n = 0, nStart = 0;
 enum {
-IN_TEXT, CHECK_QUOTE
-} eState = IN_TEXT;
+IN_TEXT, CHECK_QUOTE, FIND_LAST_QUOTE, SKIP_SEMI
+} eState = CHECK_QUOTE;
 
-while (n < rString.getLength())
+sal_Int32 nStart = 0;
+for (sal_Int32 n = 0; n < rString.getLength(); n++)
 {
+if (rString[n] == '\\')
+{
+n++; // skip next char
+continue;
+}
 switch (eState) {
 case CHECK_QUOTE:
 if (rString[n] == '"')
 {
-xub_StrLen nLen = ScGlobal::FindUnquoted( rString, '"', n 
+ 1 );
-if (nLen == STRING_NOTFOUND)
-{
-fprintf( stderr, "Error: no closing '\"' \n" );
-nLen = rString.getLength();
-}
-maArgs.push_back( rString.copy( n + 1, nLen - n - 1 ) );
-n = nLen;
-if ( nLen < rString.getLength() && rString[ nLen + 1 ] )
-;
-eState = IN_TEXT;
+nStart = n + 1;
+eState = FIND_LAST_QUOTE;
 break;
-} // drop through
+}
+// else drop through
 case IN_TEXT:
 if (rString[n] == ';')
 {
-maArgs.push_back( rString.copy( nStart, n ) );
-n = nStart = n + 1;
+maArgs.push_back( rString.copy( nStart, n - nStart ) );
+nStart = n + 1;
+eState = CHECK_QUOTE;
+}
+break;
+case FIND_LAST_QUOTE:
+if (rString[n] == '"')
+{
+maArgs.push_back( rString.copy( nStart, n - nStart ) );
+eState = SKIP_SEMI;
+break;
+}
+break;
+case SKIP_SEMI:
+if (rString[n] == ';')
+{
+nStart = n + 1;
 eState = CHECK_QUOTE;
-} else
-n++;
+}
 break;
 }
 }
commit 7cb8ce011729046c38837798afe0b6d96c0ff2e6
Author: Michael Meeks 
Date:   Thu Mar 22 17:06:27 2012 +

tubes: proxy more docfunc functions, and improve escaping / parsing

diff --git a/sc/inc/postit.hxx b/sc/inc/postit.hxx
index e553c92..0237020 100644
--- a/sc/inc/postit.hxx
+++ b/sc/inc/postit.hxx
@@ -310,8 +310,6 @@ public:
 void CopyFromClip(const ScNotes& maNotes, ScDocument* pDoc, SCCOL nCol1, 
SCROW nRow1, SCCOL nCol2, SCROW nRow2, SCsCOL nDx, SCsROW nDy, SCTAB nTab, bool 
bCloneCaption);
 
 void erase(SCCOL nCol1, SCROW nRow1, SCCOL nCol2, SCROW nRow2, bool 
bForgetCaption = false);
-
-
 };
 
 // 
diff --git a/sc/source/ui/collab/sendfunc.cxx b/sc/sourc

[Libreoffice-commits] .: starmath/source

2012-03-22 Thread Ivan Timofeev
 starmath/source/parse.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit f52138deee9e129d0566f06d1d2138b6b159d16a
Author: Ivan Timofeev 
Date:   Thu Mar 22 10:03:03 2012 +0400

fdo#41739: Math: do not skip "newline" on error

diff --git a/starmath/source/parse.cxx b/starmath/source/parse.cxx
index b386cf3..e5c1ff4 100644
--- a/starmath/source/parse.cxx
+++ b/starmath/source/parse.cxx
@@ -2424,7 +2424,9 @@ void SmParser::Error(SmParseError eError)
 
 AddError(eError, pSNode);
 
-NextToken();
+// Even if the newline token is unexpected, do not skip it. (fdo#41739)
+if (m_aCurToken.eType != TNEWLINE)
+NextToken();
 }
 
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - sal/inc

2012-03-22 Thread Will Thompson
 sal/inc/rtl/bootstrap.h   |8 
 sal/inc/rtl/bootstrap.hxx |4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

New commits:
commit cb9e45382f2c36ac9e746eab3f03b37515935530
Author: Will Thompson 
Date:   Thu Mar 22 16:53:43 2012 +

Clarify rtl_bootstrap_get documentation.

diff --git a/sal/inc/rtl/bootstrap.h b/sal/inc/rtl/bootstrap.h
index 46e7323..9070569 100644
--- a/sal/inc/rtl/bootstrap.h
+++ b/sal/inc/rtl/bootstrap.h
@@ -118,8 +118,8 @@ extern "C" {
 
 
 Must be called before rtl_bootstrap_get(). May not be called twice.
-If it is never called, a the filename executable.ini (win)
-or execuablerc (unx) is assumed.
+If it is never called, the filename is based on the name of the executable,
+with the suffix ".ini" on Windows or "rc" on Unix.
 
 @param pFileUri URL of the ini file; must not be null, must not be the 
empty
 string
commit 8077dbeeb2965f1a9e7477906287240f9fcd4c35
Author: Will Thompson 
Date:   Thu Mar 22 09:57:36 2012 +

rtl::bootstap: rename filename parameter to contain "uri"

diff --git a/sal/inc/rtl/bootstrap.h b/sal/inc/rtl/bootstrap.h
index 3f89bde..46e7323 100644
--- a/sal/inc/rtl/bootstrap.h
+++ b/sal/inc/rtl/bootstrap.h
@@ -121,10 +121,10 @@ extern "C" {
 If it is never called, a the filename executable.ini (win)
 or execuablerc (unx) is assumed.
 
-@param pName URL of the ini file; must not be null, must not be the empty
+@param pFileUri URL of the ini file; must not be null, must not be the 
empty
 string
 */
-SAL_DLLPUBLIC void SAL_CALL rtl_bootstrap_setIniFileName( rtl_uString *pName )
+SAL_DLLPUBLIC void SAL_CALL rtl_bootstrap_setIniFileName( rtl_uString 
*pFileUri )
 SAL_THROW_EXTERN_C();
 
 /**
diff --git a/sal/inc/rtl/bootstrap.hxx b/sal/inc/rtl/bootstrap.hxx
index 60ad350..789c17c 100644
--- a/sal/inc/rtl/bootstrap.hxx
+++ b/sal/inc/rtl/bootstrap.hxx
@@ -41,9 +41,9 @@ namespace rtl
 
 public:
 /**
-   @see rtl_bootstrap_setIniFileName()
+ * @see rtl_bootstrap_setIniFileName()
  */
-static inline void SAL_CALL setIniFilename( const ::rtl::OUString 
&sFile );
+static inline void SAL_CALL setIniFilename( const ::rtl::OUString 
&sFileUri );
 
 /** Retrieves a bootstrap parameter
@param sName name of the bootstrap value. case insensitive.
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-03-22 Thread Miklos Vajna
 sw/source/core/doc/doc.cxx  |1 
 sw/source/core/inc/layouter.hxx |9 
 sw/source/core/layout/layouter.cxx  |   48 
 sw/source/core/layout/objectformattertxtfrm.cxx |6 ---
 sw/source/core/text/txtfly.cxx  |3 -
 5 files changed, 1 insertion(+), 66 deletions(-)

New commits:
commit b4b8f6966d5c2d308874f6213e68ecc620e92777
Author: Cédric Bosdonnat 
Date:   Thu Mar 22 14:52:37 2012 +0100

n#750258: removed strange non-wrapping condition

In the bug file, some text wrapping was badly computed when showing the
document (but not at the first rendering) due to that weird NotToWrap
mecahnism in SwLayouter. Checked the original issue i#40155 that
removing this does hurt.

(cherry picked from commit 8a233f17ae589b33e3b54ef9ebb1fcff41ef6cd7)

Signed-off-by: Miklos Vajna 

diff --git a/sw/source/core/doc/doc.cxx b/sw/source/core/doc/doc.cxx
index d07ef90..5a2f44c 100644
--- a/sw/source/core/doc/doc.cxx
+++ b/sw/source/core/doc/doc.cxx
@@ -1822,7 +1822,6 @@ void SwDoc::ClearSwLayouterEntries()
 {
 SwLayouter::ClearMovedFwdFrms( *this );
 SwLayouter::ClearObjsTmpConsiderWrapInfluence( *this );
-SwLayouter::ClearFrmsNotToWrap( *this );
 // #i65250#
 SwLayouter::ClearMoveBwdLayoutInfo( *this );
 }
diff --git a/sw/source/core/inc/layouter.hxx b/sw/source/core/inc/layouter.hxx
index 135263e..4350b95 100644
--- a/sw/source/core/inc/layouter.hxx
+++ b/sw/source/core/inc/layouter.hxx
@@ -68,9 +68,6 @@ class SwLayouter
 SwMovedFwdFrmsByObjPos* mpMovedFwdFrms;
 // --> #i35911#
 SwObjsMarkedAsTmpConsiderWrapInfluence* mpObjsTmpConsiderWrapInfl;
-// --> #i40155# - data structure to collect frames, which are
-// marked not to wrap around objects.
-std::vector< const SwFrm* > maFrmsNotToWrap;
 
 public:
 // --> #i65250#
@@ -148,12 +145,6 @@ public:
 static void InsertObjForTmpConsiderWrapInfluence(
 const SwDoc& _rDoc,
 SwAnchoredObject& _rAnchoredObj );
-// --> #i40155#
-static void ClearFrmsNotToWrap( const SwDoc& _rDoc );
-static void InsertFrmNotToWrap( const SwDoc& _rDoc,
-const SwFrm& _rFrm );
-static bool FrmNotToWrap( const IDocumentLayoutAccess& _rIDLA,
-  const SwFrm& _rFrm );
 // --> #i65250#
 static bool MoveBwdSuppressed( const SwDoc& p_rDoc,
const SwFlowFrm& p_rFlowFrm,
diff --git a/sw/source/core/layout/layouter.cxx 
b/sw/source/core/layout/layouter.cxx
index 3105bce..5ed1712 100644
--- a/sw/source/core/layout/layouter.cxx
+++ b/sw/source/core/layout/layouter.cxx
@@ -440,54 +440,6 @@ void SwLayouter::InsertObjForTmpConsiderWrapInfluence(
 
 _rDoc.GetLayouter()->mpObjsTmpConsiderWrapInfl->Insert( _rAnchoredObj );
 }
-// #i40155#
-void SwLayouter::ClearFrmsNotToWrap( const SwDoc& _rDoc )
-{
-if ( _rDoc.GetLayouter() )
-{
-const_cast(_rDoc).GetLayouter()->maFrmsNotToWrap.clear();
-}
-}
-
-void SwLayouter::InsertFrmNotToWrap( const SwDoc& _rDoc,
- const SwFrm& _rFrm )
-{
-if ( !_rDoc.GetLayouter() )
-{
-const_cast(_rDoc).SetLayouter( new SwLayouter() );
-}
-
-if ( !SwLayouter::FrmNotToWrap( _rDoc, _rFrm ) )
-{
-const_cast(_rDoc).GetLayouter()->maFrmsNotToWrap.push_back( 
&_rFrm );
-}
-}
-
-bool SwLayouter::FrmNotToWrap( const IDocumentLayoutAccess& _rDLA,
-   const SwFrm& _rFrm )
-{
-const SwLayouter* pLayouter = _rDLA.GetLayouter();
-if ( !pLayouter )
-{
-return false;
-}
-else
-{
-bool bFrmNotToWrap( false );
-std::vector< const SwFrm* >::const_iterator aIter =
-pLayouter->maFrmsNotToWrap.begin();
-for ( ; aIter != pLayouter->maFrmsNotToWrap.end(); ++aIter )
-{
-const SwFrm* pFrm = *(aIter);
-if ( pFrm == &_rFrm )
-{
-bFrmNotToWrap = true;
-break;
-}
-}
-return bFrmNotToWrap;
-}
-}
 
 void LOOPING_LOUIE_LIGHT( bool bCondition, const SwTxtFrm& rTxtFrm )
 {
diff --git a/sw/source/core/layout/objectformattertxtfrm.cxx 
b/sw/source/core/layout/objectformattertxtfrm.cxx
index 3ceb54a..c1d8eb8 100644
--- a/sw/source/core/layout/objectformattertxtfrm.cxx
+++ b/sw/source/core/layout/objectformattertxtfrm.cxx
@@ -296,9 +296,6 @@ bool SwObjectFormatterTxtFrm::DoFormatObj( 
SwAnchoredObject& _rAnchoredObj,
   mrAnchorTxtFrm.GetFollow() &&
   mrAnchorTxtFrm.GetFollow()->GetOfst() == 0 )
 {
-SwLayouter::InsertFrmNotToWrap(
-
*(mrAnchorTxtFrm.FindPageFrm()->GetFmt()->GetDoc()),
-mrAnchor

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

2012-03-22 Thread Caolán McNamara
 framework/inc/services/pathsettings.hxx|4 --
 framework/source/services/pathsettings.cxx |   54 +
 2 files changed, 26 insertions(+), 32 deletions(-)

New commits:
commit 82af1483c8a259a9f111866f427d04452b27cb47
Author: Caolán McNamara 
Date:   Thu Mar 22 13:37:43 2012 +

annoying PATHES->PATHS typos

diff --git a/framework/inc/services/pathsettings.hxx 
b/framework/inc/services/pathsettings.hxx
index e587cc5..660a24b 100644
--- a/framework/inc/services/pathsettings.hxx
+++ b/framework/inc/services/pathsettings.hxx
@@ -63,7 +63,6 @@
 /* enable it if you whish to migrate old user settings (using the old cfg 
schema) on demand 
disable it in case only the new schema must be used.
  */
-#define MIGRATE_OLD_USER_PATHES
 
 namespace framework
 {
@@ -213,10 +212,9 @@ class PathSettings : public  css::lang::XTypeProvider  
   ,
 /** filter "real user defined paths" from the old configuration schema
 and set it as UserPaths on the new schema.
 Can be removed with new major release ... */
-#ifdef MIGRATE_OLD_USER_PATHES
+
 void impl_mergeOldUserPaths(  PathSettings::PathInfo& rPath,
  const OUStringList&   lOld );
-#endif
 
 /** reload one path directly from the new configuration schema (because
 it was updated by any external code) */
diff --git a/framework/source/services/pathsettings.cxx 
b/framework/source/services/pathsettings.cxx
index 4a5dd53..d70d2b3 100644
--- a/framework/source/services/pathsettings.cxx
+++ b/framework/source/services/pathsettings.cxx
@@ -63,7 +63,7 @@
 // __
 //  non exported const
 
-const ::rtl::OUString 
CFGPROP_USERPATHES(RTL_CONSTASCII_USTRINGPARAM("UserPaths"));
+const ::rtl::OUString 
CFGPROP_USERPATHS(RTL_CONSTASCII_USTRINGPARAM("UserPaths"));
 const ::rtl::OUString 
CFGPROP_WRITEPATH(RTL_CONSTASCII_USTRINGPARAM("WritePath"));
 
 /*
@@ -73,13 +73,13 @@ const ::rtl::OUString 
CFGPROP_WRITEPATH(RTL_CONSTASCII_USTRINGPARAM("WritePath")
 3 : write path "Template_write"string
  */
 
-const ::rtl::OUString 
POSTFIX_INTERNAL_PATHES(RTL_CONSTASCII_USTRINGPARAM("_internal"));
-const ::rtl::OUString 
POSTFIX_USER_PATHES(RTL_CONSTASCII_USTRINGPARAM("_user"));
+const ::rtl::OUString 
POSTFIX_INTERNAL_PATHS(RTL_CONSTASCII_USTRINGPARAM("_internal"));
+const ::rtl::OUString POSTFIX_USER_PATHS(RTL_CONSTASCII_USTRINGPARAM("_user"));
 const ::rtl::OUString 
POSTFIX_WRITE_PATH(RTL_CONSTASCII_USTRINGPARAM("_writable"));
 
 const sal_Int32 IDGROUP_OLDSTYLE= 0;
-const sal_Int32 IDGROUP_INTERNAL_PATHES = 1;
-const sal_Int32 IDGROUP_USER_PATHES = 2;
+const sal_Int32 IDGROUP_INTERNAL_PATHS = 1;
+const sal_Int32 IDGROUP_USER_PATHS = 2;
 const sal_Int32 IDGROUP_WRITE_PATH  = 3;
 
 const sal_Int32 IDGROUP_COUNT   = 4;
@@ -270,7 +270,7 @@ OUStringList PathSettings::impl_readOldFormat(const 
::rtl::OUString& sPath)
 // NO substitution here ! It's done outside ...
 PathSettings::PathInfo PathSettings::impl_readNewFormat(const ::rtl::OUString& 
sPath)
 {
-const static ::rtl::OUString 
CFGPROP_INTERNALPATHES(RTL_CONSTASCII_USTRINGPARAM("InternalPaths"));
+const static ::rtl::OUString 
CFGPROP_INTERNALPATHS(RTL_CONSTASCII_USTRINGPARAM("InternalPaths"));
 const static ::rtl::OUString 
CFGPROP_ISSINGLEPATH(RTL_CONSTASCII_USTRINGPARAM("IsSinglePath"));
 
 css::uno::Reference< css::container::XNameAccess > xCfg = fa_getCfgNew();
@@ -283,11 +283,11 @@ PathSettings::PathInfo 
PathSettings::impl_readNewFormat(const ::rtl::OUString& s
 
 // read internal path list
 css::uno::Reference< css::container::XNameAccess > xIPath;
-xPath->getByName(CFGPROP_INTERNALPATHES) >>= xIPath;
+xPath->getByName(CFGPROP_INTERNALPATHS) >>= xIPath;
 aPathVal.lInternalPaths << xIPath->getElementNames();
 
 // read user defined path list
-aPathVal.lUserPaths << xPath->getByName(CFGPROP_USERPATHES);
+aPathVal.lUserPaths << xPath->getByName(CFGPROP_USERPATHS);
 
 // read the writeable path
 xPath->getByName(CFGPROP_WRITEPATH) >>= aPathVal.sWritePath;
@@ -332,7 +332,7 @@ void PathSettings::impl_storePath(const 
PathSettings::PathInfo& aPath)
 {
 ::comphelper::ConfigurationHelper::writeRelativeKey(xCfgNew,
 
aResubstPath.sPathName,
-CFGPROP_USERPATHES,
+CFGPROP_USERPATHS,
 
css::uno::makeAny(aResubstPath.lUserPaths.getAsConstList()));
 }
 
@@ -360,7 +360,6 @@ void PathSettings::impl_storePath(const 
PathSettings::PathInfo& aPath)
 }
 
 //-
-#ifdef MIGRATE_OLD_USER_PATHES
 void PathSettings:

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

2012-03-22 Thread Caolán McNamara
 sw/source/core/text/txtfly.cxx |   11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

New commits:
commit 347bb1634b10eba577742fe8a7edb4b2dd69265d
Author: Cédric Bosdonnat 
Date:   Thu Mar 22 14:27:43 2012 +0100

Revert "make text wrapping work in headers/footers too (fdo#39155)"

This reverts commit e2a8fb0427e6b33a0fd3873bb7e077a6e5da8ebb. This is a
feature, not a bug and would cause loads of documents to be badly
rendered.

Link to the spec that brough that feature:

http://www.openoffice.org/specs/writer/compatibility/adjust-text-wrapping.sxw

Conflicts:

sw/source/core/text/txtfly.cxx
(cherry picked from commit 2a4e77a6f94d35919237b9db9de184e70943fc13)

Conflicts:

sw/source/core/text/txtfly.cxx

diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 7a7ddf5..8eb06ca 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -1378,7 +1378,14 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 
 const SwSortedObjs *pSorted = pPage->GetSortedObjs();
 const sal_uInt32 nCount = pSorted ? pSorted->Count() : 0;
-const sal_Bool bWrapAllowed = !SwLayouter::FrmNotToWrap( 
*pCurrFrm->GetTxtNode()->getIDocumentLayoutAccess(), *pCurrFrm );
+// --> #108724# Page header/footer content doesn't have to wrap around
+//  floating screen objects
+const bool bFooterHeader = 0 != pCurrFrm->FindFooterOrHeader();
+const IDocumentSettingAccess* pIDSA = 
pCurrFrm->GetTxtNode()->getIDocumentSettingAccess();
+// #i40155# - check, if frame is marked not to wrap
+const sal_Bool bWrapAllowed = ( 
pIDSA->get(IDocumentSettingAccess::USE_FORMER_TEXT_WRAPPING) ||
+( !pCurrFrm->IsInFtn() && !bFooterHeader ) 
) &&
+  !SwLayouter::FrmNotToWrap( 
*pCurrFrm->GetTxtNode()->getIDocumentLayoutAccess(), *pCurrFrm );
 
 bOn = sal_False;
 
@@ -1390,7 +1397,6 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 // #i28701# - consider complete frame area for new
 // text wrapping
 SwRect aRect;
-const IDocumentSettingAccess* pIDSA = 
pCurrFrm->GetTxtNode()->getIDocumentSettingAccess();
 if ( pIDSA->get(IDocumentSettingAccess::USE_FORMER_TEXT_WRAPPING) )
 {
 aRect = pCurrFrm->Prt();
@@ -1406,7 +1412,6 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 const long nRight = (aRect.*fnRect->fnGetRight)() - 1;
 const long nLeft = (aRect.*fnRect->fnGetLeft)() + 1;
 const sal_Bool bR2L = pCurrFrm->IsRightToLeft();
-const bool bFooterHeader = ( pCurrFrm->FindFooterOrHeader() != NULL );
 
 const IDocumentDrawModelAccess* pIDDMA = 
pCurrFrm->GetTxtNode()->getIDocumentDrawModelAccess();
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes2' - tubes/source

2012-03-22 Thread Will Thompson
 tubes/source/manager.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 52c85e7bfb00696b4b68331ab6139c98d93329b9
Author: Will Thompson 
Date:   Thu Mar 22 15:40:10 2012 +

tubes manager: fix a compiler warning

diff --git a/tubes/source/manager.cxx b/tubes/source/manager.cxx
index ed53a10..8dd0fe2 100644
--- a/tubes/source/manager.cxx
+++ b/tubes/source/manager.cxx
@@ -164,10 +164,11 @@ static void TeleManager_DBusChannelHandler(
 tp_handle_channels_context_accept( pContext);
 else
 {
-GError aError = { TP_ERRORS, TP_ERROR_CONFUSED,
+GError *pError = g_error_new_literal( TP_ERRORS, TP_ERROR_CONFUSED,
 "None of these channels were LibreOffice D-Bus tubes; "
-"why did the Channel Dispatcher give them to us?" };
-tp_handle_channels_context_fail( pContext, &aError);
+"why did the Channel Dispatcher give them to us?");
+tp_handle_channels_context_fail( pContext, pError);
+g_clear_error (&pError);
 }
 }
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes2' - 2 commits - tubes/inc tubes/source

2012-03-22 Thread Will Thompson
 tubes/inc/tubes/conference.hxx |5 ++--
 tubes/inc/tubes/manager.hxx|2 -
 tubes/source/conference.cxx|   21 ++--
 tubes/source/manager.cxx   |   43 +++--
 4 files changed, 56 insertions(+), 15 deletions(-)

New commits:
commit ed93ef3dadd72b79cf88e695cecd0da2600f6e11
Author: Will Thompson 
Date:   Thu Mar 22 13:30:11 2012 +

Throw an error if we get channels we don't understand.

diff --git a/tubes/source/manager.cxx b/tubes/source/manager.cxx
index e7060cc..ed53a10 100644
--- a/tubes/source/manager.cxx
+++ b/tubes/source/manager.cxx
@@ -127,6 +127,7 @@ static void TeleManager_DBusChannelHandler(
 TpHandleChannelsContext*pContext,
 gpointerpUserData)
 {
+bool aAccepted = false;
 INFO_LOGGER_F( "TeleManager_DBusChannelHandler");
 
 TeleManager* pManager = reinterpret_cast(pUserData);
@@ -146,6 +147,7 @@ static void TeleManager_DBusChannelHandler(
 if (tp_channel_get_channel_type_id( pChannel) == 
TP_IFACE_QUARK_CHANNEL_TYPE_DBUS_TUBE)
 {
 SAL_INFO( "tubes", "accepting");
+aAccepted = true;
 g_object_ref( pAccount);
 tp_cli_channel_type_dbus_tube_call_accept( pChannel, -1,
 TP_SOCKET_ACCESS_CONTROL_CREDENTIALS,
@@ -158,7 +160,15 @@ static void TeleManager_DBusChannelHandler(
 }
 }
 
-tp_handle_channels_context_accept( pContext);
+if (aAccepted)
+tp_handle_channels_context_accept( pContext);
+else
+{
+GError aError = { TP_ERRORS, TP_ERROR_CONFUSED,
+"None of these channels were LibreOffice D-Bus tubes; "
+"why did the Channel Dispatcher give them to us?" };
+tp_handle_channels_context_fail( pContext, &aError);
+}
 }
 
 
commit 7e2d156a570646fd508b92968a338ed9c3f28ce7
Author: Will Thompson 
Date:   Thu Mar 22 13:24:00 2012 +

tubes: make Conference hold TpAccount

diff --git a/tubes/inc/tubes/conference.hxx b/tubes/inc/tubes/conference.hxx
index b9fd366..a5ce7fb 100644
--- a/tubes/inc/tubes/conference.hxx
+++ b/tubes/inc/tubes/conference.hxx
@@ -46,7 +46,7 @@ class TeleConference : public 
boost::enable_shared_from_this
 {
 public:
 
-TeleConference( TeleManager* pManager, TpChannel* pChannel, const 
rtl::OString& rSessionId );
+TeleConference( TeleManager* pManager, TpAccount *pAccount, TpChannel* 
pChannel, const rtl::OString& rSessionId );
 ~TeleConference();
 
 /// Close channel and call finalize()
@@ -71,7 +71,7 @@ public:
 // --- following only to be called only by manager's callbacks ---
 // TODO: make friends instead
 
-voidsetChannel( TpChannel* pChannel );
+voidsetChannel( TpAccount* pAccount, TpChannel* 
pChannel );
 TpChannel*  getChannel() const  { return mpChannel; }
 boolofferTube();
 boolsetTube( DBusConnection* pTube );
@@ -92,6 +92,7 @@ private:
 
 rtl::OStringmaSessionId;
 TeleManager*mpManager;
+TpAccount*  mpAccount;
 TpChannel*  mpChannel;
 DBusConnection* mpTube;
 TelePacketQueue maPacketQueue;
diff --git a/tubes/inc/tubes/manager.hxx b/tubes/inc/tubes/manager.hxx
index 10f20ef..954e412 100644
--- a/tubes/inc/tubes/manager.hxx
+++ b/tubes/inc/tubes/manager.hxx
@@ -120,7 +120,7 @@ public:
 
 voiddisconnect();
 
-voidacceptTube( TpChannel* pChannel, const char* 
pAddress );
+voidacceptTube( TpAccount* pAccount, TpChannel* 
pChannel, const char* pAddress );
 
 /** Send data to all registered conferences.
 
diff --git a/tubes/source/conference.cxx b/tubes/source/conference.cxx
index 830db5f..823a9a9 100644
--- a/tubes/source/conference.cxx
+++ b/tubes/source/conference.cxx
@@ -210,16 +210,20 @@ static void TeleConference_TubeChannelStateChangedHandler(
 }
 
 
-TeleConference::TeleConference( TeleManager* pManager, TpChannel* pChannel, 
const rtl::OString& rSessionId )
+TeleConference::TeleConference( TeleManager* pManager, TpAccount* pAccount, 
TpChannel* pChannel, const rtl::OString& rSessionId )
 :
 maSessionId( rSessionId ),
 mpManager( pManager),
+mpAccount( pAccount),
 mpChannel( pChannel),
 mpTube( NULL),
 meTubeChannelState( TP_TUBE_CHANNEL_STATE_NOT_OFFERED),
 mbTubeOfferedHandlerInvoked( false),
 mbTubeChannelStateChangedHandlerInvoked( false)
 {
+if (mpAccount)
+g_object_ref( mpAccount);
+
 if (mpChannel)
 g_object_ref( mpChannel);
 }
@@ -231,14 +235,21 @@ TeleConference::~TeleConference()
 }
 
 
-void TeleConference::setChannel( TpChannel* pChannel )
+void TeleConference::setChannel( TpAccount *pAccount, TpChannel* pChannel )
 {
 OSL_ENSURE( !mpChannel, "TeleConference::setChannel: already have

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

2012-03-22 Thread Cédric Bosdonnat
 sw/source/core/doc/doc.cxx  |1 
 sw/source/core/inc/layouter.hxx |9 
 sw/source/core/layout/layouter.cxx  |   48 
 sw/source/core/layout/objectformattertxtfrm.cxx |6 ---
 sw/source/core/text/txtfly.cxx  |   10 +++--
 5 files changed, 7 insertions(+), 67 deletions(-)

New commits:
commit 8a233f17ae589b33e3b54ef9ebb1fcff41ef6cd7
Author: Cédric Bosdonnat 
Date:   Thu Mar 22 14:52:37 2012 +0100

n#750258: removed strange non-wrapping condition

In the bug file, some text wrapping was badly computed when showing the
document (but not at the first rendering) due to that weird NotToWrap
mecahnism in SwLayouter. Checked the original issue i#40155 that
removing this does hurt.

diff --git a/sw/source/core/doc/doc.cxx b/sw/source/core/doc/doc.cxx
index 64f14f4..4761473 100644
--- a/sw/source/core/doc/doc.cxx
+++ b/sw/source/core/doc/doc.cxx
@@ -1898,7 +1898,6 @@ void SwDoc::ClearSwLayouterEntries()
 {
 SwLayouter::ClearMovedFwdFrms( *this );
 SwLayouter::ClearObjsTmpConsiderWrapInfluence( *this );
-SwLayouter::ClearFrmsNotToWrap( *this );
 // #i65250#
 SwLayouter::ClearMoveBwdLayoutInfo( *this );
 }
diff --git a/sw/source/core/inc/layouter.hxx b/sw/source/core/inc/layouter.hxx
index 135263e..4350b95 100644
--- a/sw/source/core/inc/layouter.hxx
+++ b/sw/source/core/inc/layouter.hxx
@@ -68,9 +68,6 @@ class SwLayouter
 SwMovedFwdFrmsByObjPos* mpMovedFwdFrms;
 // --> #i35911#
 SwObjsMarkedAsTmpConsiderWrapInfluence* mpObjsTmpConsiderWrapInfl;
-// --> #i40155# - data structure to collect frames, which are
-// marked not to wrap around objects.
-std::vector< const SwFrm* > maFrmsNotToWrap;
 
 public:
 // --> #i65250#
@@ -148,12 +145,6 @@ public:
 static void InsertObjForTmpConsiderWrapInfluence(
 const SwDoc& _rDoc,
 SwAnchoredObject& _rAnchoredObj );
-// --> #i40155#
-static void ClearFrmsNotToWrap( const SwDoc& _rDoc );
-static void InsertFrmNotToWrap( const SwDoc& _rDoc,
-const SwFrm& _rFrm );
-static bool FrmNotToWrap( const IDocumentLayoutAccess& _rIDLA,
-  const SwFrm& _rFrm );
 // --> #i65250#
 static bool MoveBwdSuppressed( const SwDoc& p_rDoc,
const SwFlowFrm& p_rFlowFrm,
diff --git a/sw/source/core/layout/layouter.cxx 
b/sw/source/core/layout/layouter.cxx
index 344b388..aa2ac46 100644
--- a/sw/source/core/layout/layouter.cxx
+++ b/sw/source/core/layout/layouter.cxx
@@ -440,54 +440,6 @@ void SwLayouter::InsertObjForTmpConsiderWrapInfluence(
 
 _rDoc.GetLayouter()->mpObjsTmpConsiderWrapInfl->Insert( _rAnchoredObj );
 }
-// #i40155#
-void SwLayouter::ClearFrmsNotToWrap( const SwDoc& _rDoc )
-{
-if ( _rDoc.GetLayouter() )
-{
-const_cast(_rDoc).GetLayouter()->maFrmsNotToWrap.clear();
-}
-}
-
-void SwLayouter::InsertFrmNotToWrap( const SwDoc& _rDoc,
- const SwFrm& _rFrm )
-{
-if ( !_rDoc.GetLayouter() )
-{
-const_cast(_rDoc).SetLayouter( new SwLayouter() );
-}
-
-if ( !SwLayouter::FrmNotToWrap( _rDoc, _rFrm ) )
-{
-const_cast(_rDoc).GetLayouter()->maFrmsNotToWrap.push_back( 
&_rFrm );
-}
-}
-
-bool SwLayouter::FrmNotToWrap( const IDocumentLayoutAccess& _rDLA,
-   const SwFrm& _rFrm )
-{
-const SwLayouter* pLayouter = _rDLA.GetLayouter();
-if ( !pLayouter )
-{
-return false;
-}
-else
-{
-bool bFrmNotToWrap( false );
-std::vector< const SwFrm* >::const_iterator aIter =
-pLayouter->maFrmsNotToWrap.begin();
-for ( ; aIter != pLayouter->maFrmsNotToWrap.end(); ++aIter )
-{
-const SwFrm* pFrm = *(aIter);
-if ( pFrm == &_rFrm )
-{
-bFrmNotToWrap = true;
-break;
-}
-}
-return bFrmNotToWrap;
-}
-}
 
 void LOOPING_LOUIE_LIGHT( bool bCondition, const SwTxtFrm& rTxtFrm )
 {
diff --git a/sw/source/core/layout/objectformattertxtfrm.cxx 
b/sw/source/core/layout/objectformattertxtfrm.cxx
index 3ceb54a..c1d8eb8 100644
--- a/sw/source/core/layout/objectformattertxtfrm.cxx
+++ b/sw/source/core/layout/objectformattertxtfrm.cxx
@@ -296,9 +296,6 @@ bool SwObjectFormatterTxtFrm::DoFormatObj( 
SwAnchoredObject& _rAnchoredObj,
   mrAnchorTxtFrm.GetFollow() &&
   mrAnchorTxtFrm.GetFollow()->GetOfst() == 0 )
 {
-SwLayouter::InsertFrmNotToWrap(
-
*(mrAnchorTxtFrm.FindPageFrm()->GetFmt()->GetDoc()),
-mrAnchorTxtFrm );
 SwLayouter::RemoveMovedFwdFrm(
 
*(mrAnchorTxtFrm.Fi

[Libreoffice-commits] .: solenv/gbuild

2012-03-22 Thread Stephan Bergmann
 solenv/gbuild/platform/WNT_INTEL_GCC.mk |   13 -
 1 file changed, 12 insertions(+), 1 deletion(-)

New commits:
commit 1e18fdb2fa0812f91685d9580d48e4d2f8452a26
Author: Stephan Bergmann 
Date:   Thu Mar 22 15:01:35 2012 +0100

MinGW: work around newly ambiguous boost::bind calls with GCC 4.6

diff --git a/solenv/gbuild/platform/WNT_INTEL_GCC.mk 
b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
index 363f1f3..1ba676f 100644
--- a/solenv/gbuild/platform/WNT_INTEL_GCC.mk
+++ b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
@@ -34,6 +34,10 @@ gb_TMPDIR:=$(if $(TMPDIR),$(TMPDIR),/tmp)
 include $(GBUILDDIR)/platform/com_GCC_defs.mk
 include $(GBUILDDIR)/platform/windows.mk
 
+gb_CCVER := $(shell $(gb_CC) -dumpversion | $(gb_AWK) -F. -- \
+'{ print $$1*1+$$2*100+$$3 }')
+gb_GccLess470 := $(shell expr $(gb_CCVER) \< 40700)
+
 gb_RC := $(WINDRES)
 
 ifeq ($(GXX_INCLUDE_PATH),)
@@ -45,7 +49,14 @@ gb_COMPILERDEFS += \
-D_NATIVE_WCHAR_T_DEFINED \
-D_MSC_EXTENSIONS \
-D_FORCENAMELESSUNION \
-   -DBOOST_MEM_FN_ENABLE_CDECL \
+
+# Until GCC 4.6, MinGW used __cdecl by default, and BOOST_MEM_FN_ENABLE_CDECL
+# would result in ambiguous calls to overloaded boost::bind; since GCC 4.7,
+# MinGW uses __thiscall by default, so now needs BOOST_MEM_FN_ENABLE_CDECL for
+# uses of boost::bind with functions annotated with SAL_CALL:
+ifeq ($(gb_GccLess470),0)
+gb_COMPILERDEFS += -DBOOST_MEM_FN_ENABLE_CDECL
+endif
 
 gb_RCDEFS := \
 -DWINVER=0x0400 \
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: basctl/source

2012-03-22 Thread Fridrich Strba
 basctl/source/basicide/basides1.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 0a8596dd8ebbbc80e87d4bdfafe3cf53355b7d43
Author: Fridrich Å trba 
Date:   Thu Mar 22 14:46:02 2012 +0100

Revert "fix debug build"

This reverts commit 0589dd3fb5c5bf2fdcdbccfab31fbe26c0a9727d.

Given the find above, this assert will never be triggered anyway

diff --git a/basctl/source/basicide/basides1.cxx 
b/basctl/source/basicide/basides1.cxx
index d9f5b63..6702f93 100644
--- a/basctl/source/basicide/basides1.cxx
+++ b/basctl/source/basicide/basides1.cxx
@@ -468,7 +468,6 @@ void BasicIDEShell::ExecuteGlobal( SfxRequest& rReq )
 if ( aIDEWindowTable.find( rTabId.GetValue() ) !=  
aIDEWindowTable.end() )
 {
 IDEBaseWindow* pWin = aIDEWindowTable[ rTabId.GetValue() ];
-DBG_ASSERT( pWin, "Window nicht im Liste, aber in TabBar ?" );
 ::rtl::OUString aNewName( rModName.GetValue() );
 ::rtl::OUString aOldName( pWin->GetName() );
 if ( aNewName != aOldName )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: configure.in

2012-03-22 Thread Christian Lohmaier
 configure.in |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 7c4da6d5191bbd7ceb299dc422f62987683d743c
Author: Christian Lohmaier 
Date:   Thu Mar 22 14:11:01 2012 +0100

Revert "AC_PROG_GREP (I'm pedantic, am I not?)" - XCode 2.5.x doesn't know 
it

This reverts commit 4762788eb4ab7f2278861aaf9b9d6d665db61310.

diff --git a/configure.in b/configure.in
index c6df5f1..cdd4b42 100644
--- a/configure.in
+++ b/configure.in
@@ -63,7 +63,7 @@ AC_SUBST(CROSS_COMPILING)
 
 AC_PROG_EGREP
 # AC_PROG_EGREP doesn't set GREP on all systems as well
-AC_PROG_GREP
+AC_PATH_PROG(GREP, grep)
 
 if test "$build_os" = "cygwin"; then
 EXEEXT_FOR_BUILD=.exe
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes2' - 3 commits - tubes/inc tubes/qa tubes/source

2012-03-22 Thread Will Thompson
 tubes/inc/tubes/conference.hxx |3 -
 tubes/inc/tubes/manager.hxx|   10 ++
 tubes/qa/test_manager.cxx  |   63 +
 tubes/source/conference.cxx|6 ---
 tubes/source/manager.cxx   |   12 +++
 5 files changed, 60 insertions(+), 34 deletions(-)

New commits:
commit b881624cc1225c2c851bfc391ac4647aa2539b8d
Author: Will Thompson 
Date:   Thu Mar 22 12:59:15 2012 +

tubes: make startBuddySession take a TpContact *.

I removed Conference::setTarget() because it's redundant: you can get
the target from the channel. And maTarget was unused anyway.

diff --git a/tubes/inc/tubes/conference.hxx b/tubes/inc/tubes/conference.hxx
index 8f68333..b9fd366 100644
--- a/tubes/inc/tubes/conference.hxx
+++ b/tubes/inc/tubes/conference.hxx
@@ -71,8 +71,6 @@ public:
 // --- following only to be called only by manager's callbacks ---
 // TODO: make friends instead
 
-/// "bu...@jabber.example.org" or "r...@conference.example.org" or "UUID"
-voidsetTarget( const rtl::OString& rTarget );
 voidsetChannel( TpChannel* pChannel );
 TpChannel*  getChannel() const  { return mpChannel; }
 boolofferTube();
@@ -92,7 +90,6 @@ public:
 
 private:
 
-rtl::OStringmaTarget;
 rtl::OStringmaSessionId;
 TeleManager*mpManager;
 TpChannel*  mpChannel;
diff --git a/tubes/inc/tubes/manager.hxx b/tubes/inc/tubes/manager.hxx
index c704780..10f20ef 100644
--- a/tubes/inc/tubes/manager.hxx
+++ b/tubes/inc/tubes/manager.hxx
@@ -105,15 +105,13 @@ public:
 
 /** Start a session with a buddy.
 
-@param rAccount
-The account (JID) to use. This must be a valid JID that has been
-setup with Empathy or another Telepathy client providing
-Jabber/XMPP.
+@param pAccount
+The account to use. This must be a valid Jabber account.
 
-@param rBuddy
+@param pBuddy
 The buddy to be connected. Must be a contact of rAccount.
  */
-boolstartBuddySession( TpAccount *pAccount, const 
rtl::OString& rBuddy );
+boolstartBuddySession( TpAccount *pAccount, TpContact 
*pBuddy);
 
 voidunregisterConference( TeleConferencePtr 
pConference );
 
diff --git a/tubes/qa/test_manager.cxx b/tubes/qa/test_manager.cxx
index debfef5..b053652 100644
--- a/tubes/qa/test_manager.cxx
+++ b/tubes/qa/test_manager.cxx
@@ -224,8 +224,10 @@ void TestTeleTubes::testPrepareAccountManager2()
 void TestTeleTubes::testStartBuddySession1()
 {
 TpAccount *pAcc1 = mpManager1->getAccount(maOffererIdentifier);
-CPPUNIT_ASSERT ( pAcc1 != 0);
-bool bStarted = mpManager1->startBuddySession( pAcc1, 
maAccepterIdentifier);
+CPPUNIT_ASSERT( pAcc1 != 0);
+/* This has to run after testContactList has run successfully. */
+CPPUNIT_ASSERT( mpAccepterContact != 0);
+bool bStarted = mpManager1->startBuddySession( pAcc1, mpAccepterContact);
 CPPUNIT_ASSERT( bStarted == true);
 }
 
diff --git a/tubes/source/conference.cxx b/tubes/source/conference.cxx
index f5875e0..830db5f 100644
--- a/tubes/source/conference.cxx
+++ b/tubes/source/conference.cxx
@@ -231,12 +231,6 @@ TeleConference::~TeleConference()
 }
 
 
-void TeleConference::setTarget( const rtl::OString& rTarget )
-{
-maTarget = rTarget;
-}
-
-
 void TeleConference::setChannel( TpChannel* pChannel )
 {
 OSL_ENSURE( !mpChannel, "TeleConference::setChannel: already have 
channel");
diff --git a/tubes/source/manager.cxx b/tubes/source/manager.cxx
index 96c4b3e..5497435 100644
--- a/tubes/source/manager.cxx
+++ b/tubes/source/manager.cxx
@@ -392,7 +392,7 @@ bool TeleManager::startGroupSession( const rtl::OUString& 
rUConferenceRoom, cons
 
 
 /* TODO: factor out common code with startGroupSession() */
-bool TeleManager::startBuddySession( TpAccount *pAccount, const rtl::OString& 
rBuddy )
+bool TeleManager::startBuddySession( TpAccount *pAccount, TpContact *pBuddy )
 {
 INFO_LOGGER( "TeleManager::startBuddySession");
 
@@ -404,16 +404,15 @@ bool TeleManager::startBuddySession( TpAccount *pAccount, 
const rtl::OString& rB
 maConferences.push_back( pConference);
 
 /* TODO: associate the document with this session and conference */
-
-pConference->setTarget( rBuddy);
-
+const char *pIdentifier = tp_contact_get_identifier( pBuddy);
 SAL_INFO( "tubes", "TeleManager::startBuddySession: creating channel 
request from "
-<< tp_account_get_path_suffix( pAccount) << " to " << 
rBuddy.getStr());
+<< tp_account_get_path_suffix( pAccount)
+<< " to " << pIdentifier);
 
 GHashTable* pRequest = tp_asv_new(
 TP_PROP_CHANNEL_CHANNEL_TYPE, G_TYPE_STRING, 
TP_IFACE_CHANNEL_TYPE_DBUS_TUBE,
 TP_PROP_CHANNEL_TARGET_HANDLE_TYPE, TP_TYPE_HANDLE, 
TP_HANDLE_TYP

[Libreoffice-commits] .: cppuhelper/inc

2012-03-22 Thread Stephan Bergmann
 cppuhelper/inc/cppuhelper/implementationentry.hxx |   16 
 1 file changed, 12 insertions(+), 4 deletions(-)

New commits:
commit aacea7da465b6667e80d87a6c074e092a8908cf6
Author: Stephan Bergmann 
Date:   Thu Mar 22 13:48:45 2012 +0100

Make SAL_CALL placement work with both MinGW and MSVC

diff --git a/cppuhelper/inc/cppuhelper/implementationentry.hxx 
b/cppuhelper/inc/cppuhelper/implementationentry.hxx
index 655ac2b..747ec7f 100644
--- a/cppuhelper/inc/cppuhelper/implementationentry.hxx
+++ b/cppuhelper/inc/cppuhelper/implementationentry.hxx
@@ -31,6 +31,14 @@
 #include 
 #include "cppuhelperdllapi.h"
 
+// MinGW wants it the one way around while MSVC wants it the other (and
+// everywhere else, SAL_CALL is empty, so doesn't matter):
+#if defined __GNUC__
+#define MY_FN_PTR(name) SAL_CALL (* name)
+#else
+#define MY_FN_PTR(name) (SAL_CALL * name)
+#endif
+
 namespace cppu
 {
 /** One struct instance represents all data necessary for registering one 
service implementation.
@@ -45,17 +53,17 @@ struct ImplementationEntry
 /** Function that returns the implementation-name of the implementation
(same as XServiceInfo.getImplementationName() ).
  */
- rtl::OUString SAL_CALL ( * getImplementationName )();
+ rtl::OUString MY_FN_PTR( getImplementationName )();
 
 /** Function that returns all supported servicenames of the implementation
( same as XServiceInfo.getSupportedServiceNames() ).
 */
- com::sun::star::uno::Sequence< rtl::OUString > SAL_CALL ( * 
getSupportedServiceNames ) ();
+ com::sun::star::uno::Sequence< rtl::OUString > MY_FN_PTR( 
getSupportedServiceNames ) ();
 
 /** Function that creates a SingleComponentFactory.
 */
- ::com::sun::star::uno::Reference< 
::com::sun::star::lang::XSingleComponentFactory > SAL_CALL
- ( * createFactory )(
+ ::com::sun::star::uno::Reference< 
::com::sun::star::lang::XSingleComponentFactory >
+ MY_FN_PTR( createFactory )(
  ComponentFactoryFunc fptr,
  ::rtl::OUString const & rImplementationName,
  ::com::sun::star::uno::Sequence< ::rtl::OUString > const & 
rServiceNames,
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-03-22 Thread Michael Meeks
 sc/source/ui/collab/sendfunc.cxx |  189 ---
 sc/source/ui/view/viewfunc.cxx   |3 
 2 files changed, 156 insertions(+), 36 deletions(-)

New commits:
commit 503f0d340997c95d9077be0fec21ee07aa3d4a02
Author: Michael Meeks 
Date:   Thu Mar 22 12:31:15 2012 +

tubes: chain noddy serialize & de-serialize to partial docfunc re-impl.

diff --git a/sc/source/ui/collab/sendfunc.cxx b/sc/source/ui/collab/sendfunc.cxx
index ccc785d..0de59bf 100644
--- a/sc/source/ui/collab/sendfunc.cxx
+++ b/sc/source/ui/collab/sendfunc.cxx
@@ -27,45 +27,172 @@
  */
 
 #include "sal/config.h"
+
+#include 
+
 #include "cell.hxx"
 #include "docsh.hxx"
 #include "docfunc.hxx"
 
 namespace {
 
-// Ye noddy protocol !
-// method name ',' then arguments comma separated
-class ScChangeOp
+// Ye noddy mangling - needs improvement ...
+// method name ';' then arguments ; separated
+class ScChangeOpWriter
 {
   rtl::OUStringBuffer aMessage;
+  void appendSeparator()
+  {
+  aMessage.append( sal_Unicode( ';' ) );
+  }
 public:
-  ScChangeOp( const char *pName )
+
+  ScChangeOpWriter( const char *pName )
+  {
+  aMessage.appendAscii( pName );
+  appendSeparator();
+  }
+
+  void appendString( const rtl::OUString &rStr )
   {
-aMessage.appendAscii( pName );
-aMessage.append( sal_Unicode( ',' ) );
+  aMessage.append( rStr );
+  appendSeparator();
   }
 
-  void appendString( const ScAddress &rPos )
+  void appendString( const String &rStr )
   {
-rtl::OUString aStr;
-rPos.Format( aStr );
-aMessage.append( aStr );
+  aMessage.append( rStr );
+  appendSeparator();
   }
 
   void appendAddress( const ScAddress &rPos )
   {
-(void)rPos;
+  rtl::OUString aStr;
+  rPos.Format( aStr, SCA_VALID );
+  aMessage.append( aStr );
+  appendSeparator();
+  }
+
+  void appendInt( sal_Int32 i )
+  {
+  aMessage.append( i );
+  appendSeparator();
+  }
+
+  void appendBool( sal_Bool b )
+  {
+  aMessage.appendAscii( b ? "true" : "false" );
+  appendSeparator();
+  }
+
+  rtl::OString toString()
+  {
+  return rtl::OUStringToOString( aMessage.toString(), 
RTL_TEXTENCODING_UTF8 );
   }
 };
 
-class ScDocFuncIntercept : public ScDocFunc
+struct ProtocolError {
+const char *message;
+};
+
+class ScChangeOpReader {
+std::vector< rtl::OString > maArgs;
+
+public:
+
+ScChangeOpReader( const rtl::OString &rString)
+{
+// will need to handle escaping etc.
+for (sal_Int32 n = 0; n >= 0 && n < rString.getLength();)
+maArgs.push_back( rString.getToken( 0, ';', n ) );
+}
+~ScChangeOpReader() {}
+
+rtl::OString getMethod()
+{
+return maArgs[0];
+}
+
+size_t getArgCount() { return maArgs.size(); }
+
+rtl::OUString getString( sal_Int32 n )
+{
+if (n > 0 && (size_t)n < getArgCount() )
+return rtl::OUString( maArgs[n].getStr(), maArgs[n].getLength(),
+  RTL_TEXTENCODING_UTF8 );
+else
+return rtl::OUString();
+}
+
+ScAddress getAddress( sal_Int32 n )
+{
+ScAddress aAddr;
+rtl::OUString aToken( getString( n ) );
+aAddr.Parse( aToken );
+return aAddr;
+}
+
+bool getBool( sal_Int32 n )
+{
+if (n > 0 && (size_t)n < getArgCount() )
+return maArgs[n].equalsIgnoreAsciiCase( "true" );
+else
+return false;
+}
+};
+
+
+class ScDocFuncRecv : public ScDocFunc
 {
+ScDocFunc *mpChain;
 public:
-ScDocFuncIntercept( ScDocShell& rDocSh ) : ScDocFunc( rDocSh )
+// FIXME: really ScDocFunc should be an abstract base
+ScDocFuncRecv( ScDocShell& rDocSh, ScDocFunc *pChain )
+: ScDocFunc( rDocSh ),
+  mpChain( pChain )
 {
-fprintf( stderr, "Interceptor created !\n" );
+fprintf( stderr, "Receiver created !\n" );
 }
-virtual ~ScDocFuncIntercept() {}
+virtual ~ScDocFuncRecv() {}
+
+void RecvMessage( const rtl::OString &rString )
+{
+try {
+ScChangeOpReader aReader( rString );
+// FIXME: have some hash to enumeration mapping here
+if ( aReader.getMethod() == "setNormalString" )
+mpChain->SetNormalString( aReader.getAddress( 1 ), 
aReader.getString( 2 ),
+  aReader.getBool( 3 ) );
+else
+fprintf( stderr, "Error: unknown message '%s' (%d)\n",
+ rString.getStr(), (int)aReader.getArgCount() );
+} catch (const ProtocolError &e) {
+fprintf( stderr, "Error: protocol twisting '%s'\n", e.message );
+}
+}
+};
+
+class ScDocFuncSend : public ScDocFunc
+{
+ScDocFuncRecv *mpChain;
+
+void SendMessage( ScChangeOpWriter &rOp )
+{
+fprintf( stderr, "Op: '%s'\n", rOp.toString().getStr() );
+mpChain->RecvMessage( rOp.toString() );
+}
+
+public:
+// FIXME: rea

[Libreoffice-commits] .: binfilter/bf_sw binfilter/inc

2012-03-22 Thread Caolán McNamara
 binfilter/bf_sw/source/core/bastyp/sw_index.cxx |8 ++--
 binfilter/inc/bf_sw/index.hxx   |2 --
 2 files changed, 2 insertions(+), 8 deletions(-)

New commits:
commit 1ac6932d891754769359cd53041cf46a32723602
Author: Caolán McNamara 
Date:   Thu Mar 22 12:28:09 2012 +

WaE: need a virtual dtor

diff --git a/binfilter/bf_sw/source/core/bastyp/sw_index.cxx 
b/binfilter/bf_sw/source/core/bastyp/sw_index.cxx
index 1356f25..26cee14 100644
--- a/binfilter/bf_sw/source/core/bastyp/sw_index.cxx
+++ b/binfilter/bf_sw/source/core/bastyp/sw_index.cxx
@@ -309,16 +309,12 @@ int SwIndex::nSerial = 0;
 /*N*/ {
 /*N*/ }
 
-#ifdef DBG_UTIL
-
-
 /*N*/ SwIndexReg::~SwIndexReg()
 /*N*/ {
+#ifdef DBG_UTIL
 /*N*/   OSL_ENSURE( !pFirst || !pLast, "Es sind noch Indizies angemeldet" );
-/*N*/ }
-
 #endif
-
+/*N*/ }
 
 /*N*/ void SwIndexReg::Update( const SwIndex& rIdx, xub_StrLen nDiff, BOOL 
bNeg )
 /*N*/ {
diff --git a/binfilter/inc/bf_sw/index.hxx b/binfilter/inc/bf_sw/index.hxx
index b985e72..4d14ea2 100644
--- a/binfilter/inc/bf_sw/index.hxx
+++ b/binfilter/inc/bf_sw/index.hxx
@@ -126,9 +126,7 @@ protected:
 
 public:
 SwIndexReg();
-#ifdef DBG_UTIL
 virtual ~SwIndexReg();
-#endif
 
 // rtti, abgeleitete moegens gleichtun oder nicht. Wenn sie es gleichtun
 // kann ueber das SwIndexReg typsicher gecastet werden.
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 6 commits - basic/source cosv/source filter/Module_filter.mk shell/source solenv/bin soltools/mkdepend

2012-03-22 Thread Caolán McNamara
 basic/source/uno/dlgcont.cxx  |   11 +--
 cosv/source/storage/file.cxx  |5 ++---
 filter/Module_filter.mk   |3 ---
 shell/source/unix/sysshell/recently_used_file.cxx |3 ++-
 solenv/bin/linkoo |1 -
 soltools/mkdepend/pr.c|   10 ++
 6 files changed, 11 insertions(+), 22 deletions(-)

New commits:
commit ed0ffbba06f754eea1d37452dbdc684e8a4b72ad
Author: Caolán McNamara 
Date:   Thu Mar 22 12:21:20 2012 +

over-engineered thread-safe static OUString

This is a ridiculous over-engineered thread-safe static string for
something which is called so few times, and costs 1k of long-term
memory as the OUString dtor needs to be put into a callback
queue to get called at module unload time.

Just return a new string every time, *rolls eyes*

diff --git a/basic/source/uno/dlgcont.cxx b/basic/source/uno/dlgcont.cxx
index 3843039..fa99fe1 100644
--- a/basic/source/uno/dlgcont.cxx
+++ b/basic/source/uno/dlgcont.cxx
@@ -567,16 +567,7 @@ Sequence< OUString > 
SfxDialogLibraryContainer::getSupportedServiceNames_static(
 
 OUString SfxDialogLibraryContainer::getImplementationName_static()
 {
-static OUString aImplName;
-static sal_Bool bNeedsInit = sal_True;
-
-MutexGuard aGuard( Mutex::getGlobalMutex() );
-if( bNeedsInit )
-{
-aImplName = 
OUString(RTL_CONSTASCII_USTRINGPARAM("com.sun.star.comp.sfx2.DialogLibraryContainer"));
-bNeedsInit = sal_False;
-}
-return aImplName;
+return OUString("com.sun.star.comp.sfx2.DialogLibraryContainer");
 }
 
 Reference< XInterface > SAL_CALL SfxDialogLibraryContainer::Create( const 
Reference< XComponentContext >& ) throw( Exception )
commit c8587d05bd50c710fdbfae475aa495a8d46e6094
Author: Caolán McNamara 
Date:   Thu Mar 22 09:20:22 2012 +

confirmed that tests pass on windows

diff --git a/filter/Module_filter.mk b/filter/Module_filter.mk
index 4f9773d..24fe97c 100644
--- a/filter/Module_filter.mk
+++ b/filter/Module_filter.mk
@@ -75,13 +75,10 @@ $(eval $(call gb_Module_add_targets,filter,\
 ))
 endif
 
-ifneq ($(OS),WNT)
-# TODO, see if it links and runs under windows
 $(eval $(call gb_Module_add_check_targets,filter,\
 CppunitTest_filter_tga_test \
 CppunitTest_filter_tiff_test \
 ))
-endif
 
 # TODO
 #$(eval $(call gb_Module_add_subsequentcheck_targets,filter,\
commit 52a6e0a6fa86f80ee91d52abc779b6db667c367a
Author: Caolán McNamara 
Date:   Thu Mar 22 09:04:18 2012 +

testtool is gone these days

diff --git a/solenv/bin/linkoo b/solenv/bin/linkoo
index daef867..86ac247 100755
--- a/solenv/bin/linkoo
+++ b/solenv/bin/linkoo
@@ -414,7 +414,6 @@ if (!-f "$OOO_INSTALL/" . $brand_program_dir . "/ooenv") {
 open ($ooenv, ">$OOO_INSTALL/" . $brand_program_dir . "/ooenv") || die 
"Can't open $OOO_INSTALL/" . $brand_program_dir . "/ooenv: $!";
 print $ooenv "thisdir=$OOO_INSTALL/" . $brand_program_dir . "/\n";
 print $ooenv $env_script;
-print $ooenv "# testtool\nexport SRC_ROOT=$OOO_BUILD\n";
 close ($ooenv);
 }
 
commit 1a4276c511240d51e1956d157655d219dc8254fc
Author: Caolán McNamara 
Date:   Wed Mar 21 21:33:04 2012 +

WaE: ignoring lockf return value

diff --git a/shell/source/unix/sysshell/recently_used_file.cxx 
b/shell/source/unix/sysshell/recently_used_file.cxx
index 0336a7f..cf27852 100644
--- a/shell/source/unix/sysshell/recently_used_file.cxx
+++ b/shell/source/unix/sysshell/recently_used_file.cxx
@@ -100,7 +100,8 @@ recently_used_file::recently_used_file() :
 //
 recently_used_file::~recently_used_file()
 {
-lockf(fileno(file_), F_ULOCK, 0);
+int ret = lockf(fileno(file_), F_ULOCK, 0);
+SAL_WARN_IF(ret != 0, "shell", "cannot unlock recently unused file");
 fclose(file_);
 }
 
commit 7dd0898578f9825fe470d58ed2d0e5bc155b66c7
Author: Caolán McNamara 
Date:   Wed Mar 21 21:13:17 2012 +

WaE: ignoring return value of 'fwrite'

diff --git a/cosv/source/storage/file.cxx b/cosv/source/storage/file.cxx
index 16665d7..0618fae 100644
--- a/cosv/source/storage/file.cxx
+++ b/cosv/source/storage/file.cxx
@@ -104,9 +104,8 @@ File::do_write( const void *i_pSrc,
 
 if ( eLastIO == io_write )
 ::fseek( pStream, 0, SEEK_CUR );
-uintt ret = position();
-::fwrite( i_pSrc, 1, i_nNrofBytes, pStream );
-ret = position() - ret;
+
+uintt ret = ::fwrite( i_pSrc, 1, i_nNrofBytes, pStream );
 
 eLastIO = io_write;
 return ret;
commit d7eb0ef560fb6b549f0054140e3b42d0a991bffe
Author: Caolán McNamara 
Date:   Wed Mar 21 20:50:07 2012 +

WaE: ignoring return value of 'fwrite'

diff --git a/soltools/mkdepend/pr.c b/soltools/mkdepend/pr.c
index 082e312..f472f28 100644
--- a/soltools/mkdepend/pr.c
+++ b/soltools/mkdepend/pr.c
@@ -29,7 +29,7 @@ in this Software without prior written authorization from the 
X 

[Libreoffice-commits] .: Branch 'feature/tubes2' - 2 commits - .gitignore tubes/inc tubes/qa tubes/README tubes/source tubes/test-config.ini

2012-03-22 Thread Will Thompson
 .gitignore   |3 +++
 tubes/README |   10 --
 tubes/inc/tubes/manager.hxx  |6 +++---
 tubes/qa/test-config.ini.example |5 +
 tubes/qa/test_manager.cxx|   14 ++
 tubes/source/manager.cxx |   11 ++-
 tubes/test-config.ini|3 ---
 7 files changed, 27 insertions(+), 25 deletions(-)

New commits:
commit 9fc45729868d76f450cdbf3bc427a25f55de61aa
Author: Will Thompson 
Date:   Thu Mar 22 11:22:44 2012 +

tubes: Give a useful error when test-config.ini is missing.

diff --git a/.gitignore b/.gitignore
index c1aef58..ce66673 100644
--- a/.gitignore
+++ b/.gitignore
@@ -130,3 +130,6 @@ test/user-template/user/psprint/pspfontcache
 # Somepeople love debugging
 gdbtrace.log
 valgrind.log
+
+# Tubes test stuff
+/tubes/qa/test-config.ini
diff --git a/tubes/README b/tubes/README
index 84d4892..3f2f637 100644
--- a/tubes/README
+++ b/tubes/README
@@ -37,12 +37,10 @@ Status 2012-03-20:
 is silenced down ... :-(
 
 * for the cppunittest needed:
-  * a jabber daemon running on localhost.localdomain
-  * two accounts configured in Empathy
-* libo1@localhost.localdomain
-* libo2@localhost.localdomain
-  * libo1 and libo2 must be contacts/buddies of each other
-  * both accounts need to be online in Empathy
+  * Configure two Jabber accounts in Empathy
+  * Both must be online and on each other's contact list
+  * Copy qa/test-config.ini.example to qa/test-config.ini, and specify those
+two accounts' JIDs in it.
 
 * very nasty GMainLoop handling for cppunittest, HOPEFULLY we will get rid of
   that in a real LibO
diff --git a/tubes/qa/test-config.ini.example b/tubes/qa/test-config.ini.example
new file mode 100644
index 000..2d03ca8
--- /dev/null
+++ b/tubes/qa/test-config.ini.example
@@ -0,0 +1,5 @@
+# Locally-configured Jabber accounts used by Tubes test suite.  These accounts
+# must both be signed in and on each others' contact lists for the tests to
+# work.
+offerer=libo1@localhost.localdomain
+accepter=libo2@localhost.localdomain
diff --git a/tubes/qa/test_manager.cxx b/tubes/qa/test_manager.cxx
index f7461ba..ea5806d 100644
--- a/tubes/qa/test_manager.cxx
+++ b/tubes/qa/test_manager.cxx
@@ -88,6 +88,7 @@ public:
 private:
 // XXX The Jabber accounts specified in test-config.ini need to be setup in
 // Empathy, enabled, connected, and on each other's rosters.
+rtl::OUString maTestConfigIniURL;
 rtl::BootstrapmaTestConfig;
 
 rtl::OUString maAcc1;
@@ -101,12 +102,15 @@ static TeleManager* mpManager2 = NULL;
 static sal_uInt32 nSentPackets = 0;
 
 TestTeleTubes::TestTeleTubes()
-: maTestConfig(getURLFromSrc("/tubes/test-config.ini"))
+: maTestConfigIniURL(getURLFromSrc("/tubes/qa/test-config.ini")),
+  maTestConfig(maTestConfigIniURL)
 {
 TeleManager::addSuffixToNames( "TeleTest");
 
-CPPUNIT_ASSERT( maTestConfig.getFrom("offerer", maAcc1));
-CPPUNIT_ASSERT( maTestConfig.getFrom("accepter", maAcc2));
+CPPUNIT_ASSERT_MESSAGE( "See README for how to set up test-config.ini",
+maTestConfig.getFrom("offerer", maAcc1));
+CPPUNIT_ASSERT_MESSAGE( "See README for how to set up test-config.ini",
+maTestConfig.getFrom("accepter", maAcc2));
 }
 
 TestTeleTubes::~TestTeleTubes()
diff --git a/tubes/test-config.ini b/tubes/test-config.ini
deleted file mode 100644
index dba2be1..000
--- a/tubes/test-config.ini
+++ /dev/null
@@ -1,3 +0,0 @@
-# Locally-configured Jabber accounts used by Tubes test suite.
-offerer=libo1@localhost.localdomain
-accepter=libo2@localhost.localdomain
commit 8f0ec4cef3f3dc78f27a3e08b68eaa9d0f45608e
Author: Will Thompson 
Date:   Thu Mar 22 11:06:42 2012 +

Make startBuddySession() take a TpAccount *.

diff --git a/tubes/inc/tubes/manager.hxx b/tubes/inc/tubes/manager.hxx
index 7f325c3..122b959 100644
--- a/tubes/inc/tubes/manager.hxx
+++ b/tubes/inc/tubes/manager.hxx
@@ -113,7 +113,7 @@ public:
 @param rBuddy
 The buddy to be connected. Must be a contact of rAccount.
  */
-boolstartBuddySession( const rtl::OUString& rAccount, 
const rtl::OUString& rBuddy );
+boolstartBuddySession( TpAccount *pAccount, const 
rtl::OUString& rBuddy );
 
 voidunregisterConference( TeleConferencePtr 
pConference );
 
@@ -195,14 +195,14 @@ public:
  */
 static void addSuffixToNames( const char* pName );
 
+TpAccount*  getAccount( const rtl::OString& rAccountID );
+
 private:
 
 TeleConferenceVectormaConferences;
 
 boolmbChannelReadyHandlerInvoked : 1;
 
-TpAccount*  getAccount( const rtl::OString& rAccountID );
-
 static TeleManagerImpl* pImpl;
 static sal_uInt32   nRefCount;
 static rtl::OString aNameSuffix;
diff --git a/tubes/qa/test_manager.cxx b/tubes/qa/test_manager.cxx
index 63077dc.

[Libreoffice-commits] .: sc/source

2012-03-22 Thread Takeshi Abe
 sc/source/core/data/dptabsrc.cxx  |1 -
 sc/source/core/data/pivot2.cxx|1 -
 sc/source/core/data/table5.cxx|1 -
 sc/source/core/tool/interpr4.cxx  |1 -
 sc/source/core/tool/interpr5.cxx  |5 -
 sc/source/filter/excel/excimp8.cxx|1 -
 sc/source/filter/excel/excrecds.cxx   |1 -
 sc/source/filter/excel/impop.cxx  |1 -
 sc/source/filter/excel/xechart.cxx|1 -
 sc/source/filter/excel/xestring.cxx   |2 --
 sc/source/filter/excel/xltoolbar.cxx  |1 -
 sc/source/filter/html/htmlexp.cxx |1 -
 sc/source/filter/lotus/op.cxx |8 
 sc/source/filter/xml/xmlexprt.cxx |3 ---
 sc/source/filter/xml/xmlfonte.cxx |1 -
 sc/source/filter/xml/xmlimprt.cxx |1 -
 sc/source/ui/cctrl/tbinsert.cxx   |1 -
 sc/source/ui/docshell/docsh.cxx   |1 -
 sc/source/ui/drawfunc/drawsh.cxx  |3 ---
 sc/source/ui/drawfunc/fuins2.cxx  |1 -
 sc/source/ui/miscdlgs/autofmt.cxx |1 -
 sc/source/ui/miscdlgs/scuiautofmt.cxx |1 -
 sc/source/ui/miscdlgs/textdlgs.cxx|1 -
 sc/source/ui/unoobj/chartuno.cxx  |1 -
 sc/source/ui/unoobj/docuno.cxx|1 -
 sc/source/ui/unoobj/scdetect.cxx  |4 
 sc/source/ui/unoobj/servuno.cxx   |6 --
 sc/source/ui/vba/vbaapplication.cxx   |4 
 sc/source/ui/vba/vbarange.cxx |1 -
 sc/source/ui/vba/vbaworksheet.cxx |1 -
 sc/source/ui/view/cellsh1.cxx |2 --
 sc/source/ui/view/gridwin.cxx |2 --
 sc/source/ui/view/preview.cxx |1 -
 sc/source/ui/view/prevwsh.cxx |2 --
 sc/source/ui/view/tabvwsh4.cxx|2 --
 sc/source/ui/view/viewfun2.cxx|4 
 sc/source/ui/view/viewfun3.cxx|2 --
 sc/source/ui/view/viewfun4.cxx|1 -
 sc/source/ui/view/viewfun5.cxx|1 -
 39 files changed, 74 deletions(-)

New commits:
commit eedbe7163bf3979ae392b3b88227ee68e1df12fe
Author: Takeshi Abe 
Date:   Thu Mar 22 20:52:52 2012 +0900

removed duplicate includes in sc

diff --git a/sc/source/core/data/dptabsrc.cxx b/sc/source/core/data/dptabsrc.cxx
index b69b102..4d680ae 100644
--- a/sc/source/core/data/dptabsrc.cxx
+++ b/sc/source/core/data/dptabsrc.cxx
@@ -63,7 +63,6 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
diff --git a/sc/source/core/data/pivot2.cxx b/sc/source/core/data/pivot2.cxx
index a4c823a..2992023 100644
--- a/sc/source/core/data/pivot2.cxx
+++ b/sc/source/core/data/pivot2.cxx
@@ -53,7 +53,6 @@
 #include "refupdat.hxx"
 #include "stlpool.hxx"
 #include "stlsheet.hxx"
-#include "pivot.hxx"
 
 using ::com::sun::star::sheet::DataPilotFieldReference;
 using ::rtl::OUString;
diff --git a/sc/source/core/data/table5.cxx b/sc/source/core/data/table5.cxx
index 020c721..0a7de50 100644
--- a/sc/source/core/data/table5.cxx
+++ b/sc/source/core/data/table5.cxx
@@ -44,7 +44,6 @@
 #include "global.hxx"
 #include "rechead.hxx"
 #include "stlpool.hxx"
-#include "stlsheet.hxx"
 #include "brdcst.hxx"
 #include "tabprotection.hxx"
 #include "globstr.hrc"
diff --git a/sc/source/core/tool/interpr4.cxx b/sc/source/core/tool/interpr4.cxx
index 9c6794b..8764b45 100644
--- a/sc/source/core/tool/interpr4.cxx
+++ b/sc/source/core/tool/interpr4.cxx
@@ -40,7 +40,6 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
diff --git a/sc/source/core/tool/interpr5.cxx b/sc/source/core/tool/interpr5.cxx
index 07dcaf5..6fd612a 100644
--- a/sc/source/core/tool/interpr5.cxx
+++ b/sc/source/core/tool/interpr5.cxx
@@ -35,9 +35,6 @@
 #include 
 #include 
 
-#if OSL_DEBUG_LEVEL > 1
-#include 
-#endif
 #include 
 #include 
 
@@ -51,8 +48,6 @@
 #include "globstr.hrc"
 #include "cellkeytranslator.hxx"
 
-#include 
-#include 
 #include 
 
 using ::std::vector;
diff --git a/sc/source/filter/excel/excimp8.cxx 
b/sc/source/filter/excel/excimp8.cxx
index 41d8c38..91d5a01 100644
--- a/sc/source/filter/excel/excimp8.cxx
+++ b/sc/source/filter/excel/excimp8.cxx
@@ -109,7 +109,6 @@
 #include 
 #include 
 #include 
-#include 
 #include "xltoolbar.hxx"
 #include 
 #include 
diff --git a/sc/source/filter/excel/excrecds.cxx 
b/sc/source/filter/excel/excrecds.cxx
index 828b2d6..c8e0856 100644
--- a/sc/source/filter/excel/excrecds.cxx
+++ b/sc/source/filter/excel/excrecds.cxx
@@ -69,7 +69,6 @@
 #include "cell.hxx"
 #include "document.hxx"
 #include "scextopt.hxx"
-#include "patattr.hxx"
 #include "attrib.hxx"
 #include "progress.hxx"
 #include "dociter.hxx"
diff --git a/sc/source/filter/excel/impop.cxx b/sc/source/filter/excel/impop.cxx
index c7504ce..8c7a48b 100644
--- a/sc/source/filter/excel/impop.cxx
+++ b/sc/source/filter/excel/impop.cxx
@@ -61,7 +61,6 @@
 #include "olinetab.hxx"
 #include "stlsheet.hxx"
 #include "stlpool.hxx"
-#include "compiler.hxx"
 #include "viewopti.hxx"
 #include "docoptio.hxx"
 #include "scextopt.hxx"
diff --git a/sc/source/filter/excel/xechart.cxx 
b/sc/source/filte

[Libreoffice-commits] .: sw/source

2012-03-22 Thread Fridrich Strba
 sw/source/core/doc/docdraw.cxx|3 --
 sw/source/core/doc/docedt.cxx |9 ++
 sw/source/core/docnode/ndcopy.cxx |   56 +++---
 sw/source/core/inc/mvsave.hxx |5 +--
 4 files changed, 34 insertions(+), 39 deletions(-)

New commits:
commit db51589575d37ca608067ad9904dcc6a09651e8c
Author: Bartosz Kosiorek 
Date:   Thu Mar 15 07:26:54 2012 +0100

Convert SVARRAY SORT into STL set

diff --git a/sw/source/core/doc/docdraw.cxx b/sw/source/core/doc/docdraw.cxx
index 2d5671c..cbff064 100644
--- a/sw/source/core/doc/docdraw.cxx
+++ b/sw/source/core/doc/docdraw.cxx
@@ -76,9 +76,6 @@
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::linguistic2;
 
-
-SV_IMPL_VARARR_SORT( _ZSortFlys, _ZSortFly )
-
 /*
 |*
 |*  SwDoc::GroupSelection / SwDoc::UnGroupSelection
diff --git a/sw/source/core/doc/docedt.cxx b/sw/source/core/doc/docedt.cxx
index 26a2ad8..6c73ba1 100644
--- a/sw/source/core/doc/docedt.cxx
+++ b/sw/source/core/doc/docedt.cxx
@@ -163,7 +163,6 @@ struct _SaveRedline
 
 SV_DECL_PTRARR_DEL( _SaveRedlines, _SaveRedline*, 0 )
 
-SV_IMPL_VARARR( _SaveFlyArr, _SaveFly )
 SV_IMPL_PTRARR( _SaveRedlines, _SaveRedline* )
 
 bool lcl_MayOverwrite( const SwTxtNode *pNode, const xub_StrLen nPos )
@@ -191,7 +190,7 @@ void _RestFlyInRange( _SaveFlyArr & rArr, const 
SwNodeIndex& rSttIdx,
   const SwNodeIndex* pInsertPos )
 {
 SwPosition aPos( rSttIdx );
-for( sal_uInt16 n = 0; n < rArr.Count(); ++n )
+for( size_t n = 0; n < rArr.size(); ++n )
 {
 // create new anchor
 _SaveFly& rSave = rArr[n];
@@ -234,7 +233,7 @@ void _SaveFlyInRange( const SwNodeRange& rRg, _SaveFlyArr& 
rArr )
 {
 _SaveFly aSave( pAPos->nNode.GetIndex() - rRg.aStart.GetIndex(),
 pFmt, sal_False );
-rArr.Insert( aSave, rArr.Count());
+rArr.push_back( aSave );
 pFmt->DelFrms();
 rFmts.Remove( n--, 1 );
 }
@@ -295,7 +294,7 @@ void _SaveFlyInRange( const SwPaM& rPam, const SwNodeIndex& 
rInsPos,
 {
 _SaveFly aSave( pAPos->nNode.GetIndex() - rSttNdIdx.GetIndex(),
 pFmt, bInsPos );
-rArr.Insert( aSave, rArr.Count());
+rArr.push_back( aSave );
 pFmt->DelFrms();
 rFmts.Remove( n--, 1 );
 }
@@ -1188,7 +1187,7 @@ bool SwDoc::MoveNodeRange( SwNodeRange& rRange, 
SwNodeIndex& rPos,
 }
 
 // move the Flys to the new position
-if( aSaveFlyArr.Count() )
+if( !aSaveFlyArr.empty() )
 _RestFlyInRange( aSaveFlyArr, aIdx, NULL );
 
 // Add the Bookmarks back to the Document
diff --git a/sw/source/core/docnode/ndcopy.cxx 
b/sw/source/core/docnode/ndcopy.cxx
index 4aabfba..97557f2 100644
--- a/sw/source/core/docnode/ndcopy.cxx
+++ b/sw/source/core/docnode/ndcopy.cxx
@@ -53,6 +53,8 @@
 #include 
 #include 
 #include 
+#include 
+#include 
 
 #ifdef DBG_UTIL
 #define CHECK_TABLE(t) (t).CheckConsistency();
@@ -328,7 +330,7 @@ sal_Bool lcl_CopyTblLine( const SwTableLine*& rpLine, void* 
pPara );
 
 sal_Bool lcl_CopyTblBox( const SwTableBox*& rpBox, void* pPara )
 {
-_CopyTable* pCT = (_CopyTable*)pPara;
+_CopyTable* pCT = reinterpret_cast< _CopyTable* >(pPara);
 
 SwTableBoxFmt* pBoxFmt = (SwTableBoxFmt*)rpBox->GetFrmFmt();
 pCT->rMapArr.ForEach( lcl_SrchNew, &pBoxFmt );
@@ -392,7 +394,7 @@ sal_Bool lcl_CopyTblBox( const SwTableBox*& rpBox, void* 
pPara )
 
 sal_Bool lcl_CopyTblLine( const SwTableLine*& rpLine, void* pPara )
 {
-_CopyTable* pCT = (_CopyTable*)pPara;
+_CopyTable* pCT = reinterpret_cast< _CopyTable* >(pPara);
 SwTableLineFmt* pLineFmt = (SwTableLineFmt*)rpLine->GetFrmFmt();
 pCT->rMapArr.ForEach( lcl_SrchNew, &pLineFmt );
 if( pLineFmt == rpLine->GetFrmFmt() )   // ein neues anlegen ??
@@ -1403,7 +1405,7 @@ void SwDoc::CopyFlyInFlyImpl( const SwNodeRange& rRg,
 //die Ordnungsnummer (wird nur im DrawModel verwaltet)
 //beibehalten.
 SwDoc *const pDest = rStartIdx.GetNode().GetDoc();
-_ZSortFlys aArr;
+::std::set< _ZSortFly > aSet;
 sal_uInt16 nArrLen = GetSpzFrmFmts()->Count();
 
 for ( sal_uInt16 n = 0; n < nArrLen; ++n )
@@ -1467,22 +1469,20 @@ void SwDoc::CopyFlyInFlyImpl( const SwNodeRange& rRg,
 }
 }
 if( bAdd )
-aArr.Insert( _ZSortFly( pFmt, pAnchor, nArrLen + aArr.Count() 
));
+aSet.insert( _ZSortFly( pFmt, pAnchor, nArrLen + aSet.size() 
));
 }
 }
 
 //Alle kopierten (also die neu erzeugten) Rahmen in ein weiteres Array
 //stopfen. Dort sizten sie passend zu den Originalen, damit hinterher
 //die Chains entsprechend aufgebaut werden koennen.
-SvPtrarr aNewArr( 10 );
+::std::vector< SwFrmFmt* > aVecSwFrmFmt;
 
-  

[Libreoffice-commits] .: Branch 'feature/tubes2' - tubes/CppunitTest_tubes_test.mk tubes/qa tubes/test-config.ini

2012-03-22 Thread Will Thompson
 tubes/CppunitTest_tubes_test.mk |2 ++
 tubes/qa/test_manager.cxx   |   22 +++---
 tubes/test-config.ini   |3 +++
 3 files changed, 20 insertions(+), 7 deletions(-)

New commits:
commit 16e48ea17024c7b97b3048bf4463ae443f930455
Author: Will Thompson 
Date:   Thu Mar 22 10:43:57 2012 +

tubes: move hardcoded test accounts to an .ini file

diff --git a/tubes/CppunitTest_tubes_test.mk b/tubes/CppunitTest_tubes_test.mk
index 7b558d7..57a53a5 100644
--- a/tubes/CppunitTest_tubes_test.mk
+++ b/tubes/CppunitTest_tubes_test.mk
@@ -33,6 +33,7 @@ $(eval $(call 
gb_CppunitTest_add_exception_objects,tubes_test, \
 $(eval $(call gb_CppunitTest_add_linked_libs,tubes_test, \
 tubes \
 sal \
+unotest \
 $(gb_STDLIBS) \
 ))
 
@@ -42,6 +43,7 @@ $(eval $(call gb_CppunitTest_add_libs,tubes_test, \
 
 $(eval $(call gb_CppunitTest_set_include,tubes_test,\
 -I$(realpath $(SRCDIR)/tubes/inc) \
+-I$(realpath $(OUTDIR)/inc/udkapi) \
 $$(TELEPATHY_CFLAGS) \
 $$(INCLUDE) \
 ))
diff --git a/tubes/qa/test_manager.cxx b/tubes/qa/test_manager.cxx
index 69266d0..63077dc 100644
--- a/tubes/qa/test_manager.cxx
+++ b/tubes/qa/test_manager.cxx
@@ -37,10 +37,12 @@
 #include 
 #include 
 #include 
+#include 
+#include 
 
 namespace {
 
-class TestTeleTubes: public CppUnit::TestFixture
+class TestTeleTubes: public test::BootstrapFixtureBase
 {
 public:
 
@@ -83,22 +85,28 @@ public:
 CPPUNIT_TEST( testFailAlways ); // need failure to display SAL_LOG, 
comment out for real builds
 CPPUNIT_TEST_SUITE_END();
 
+private:
+// XXX The Jabber accounts specified in test-config.ini need to be setup in
+// Empathy, enabled, connected, and on each other's rosters.
+rtl::BootstrapmaTestConfig;
+
+rtl::OUString maAcc1;
+rtl::OUString maAcc2;
 };
 
 // static, not members, so they actually survive cppunit test iteration
 static TeleManager* mpManager1 = NULL;
 static TeleManager* mpManager2 = NULL;
 
-// XXX The accounts need to be setup in Empathy and a jabber daemon needs
-// to be running on localhost.localdomain and both accounts need to be
-// enabled and connected.
-static rtl::OUString sAcc1( RTL_CONSTASCII_USTRINGPARAM( 
"libo1@localhost.localdomain"));
-static rtl::OUString sAcc2( RTL_CONSTASCII_USTRINGPARAM( 
"libo2@localhost.localdomain"));
 static sal_uInt32 nSentPackets = 0;
 
 TestTeleTubes::TestTeleTubes()
+: maTestConfig(getURLFromSrc("/tubes/test-config.ini"))
 {
 TeleManager::addSuffixToNames( "TeleTest");
+
+CPPUNIT_ASSERT( maTestConfig.getFrom("offerer", maAcc1));
+CPPUNIT_ASSERT( maTestConfig.getFrom("accepter", maAcc2));
 }
 
 TestTeleTubes::~TestTeleTubes()
@@ -182,7 +190,7 @@ void TestTeleTubes::testPrepareAccountManager2()
 
 void TestTeleTubes::testStartBuddySession1()
 {
-bool bStarted = mpManager1->startBuddySession( sAcc1, sAcc2);
+bool bStarted = mpManager1->startBuddySession( maAcc1, maAcc2);
 CPPUNIT_ASSERT( bStarted == true);
 }
 
diff --git a/tubes/test-config.ini b/tubes/test-config.ini
new file mode 100644
index 000..dba2be1
--- /dev/null
+++ b/tubes/test-config.ini
@@ -0,0 +1,3 @@
+# Locally-configured Jabber accounts used by Tubes test suite.
+offerer=libo1@localhost.localdomain
+accepter=libo2@localhost.localdomain
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/tubes' - 3 commits - Repository.mk tubes/Executable_liboapprover.mk tubes/LibreOfficeApprover.client tubes/LibreOffice.client tubes/Module_tubes.mk tubes/org.f

2012-03-22 Thread Will Thompson
 Repository.mk  |1 
 tubes/Executable_liboapprover.mk   |   47 +
 tubes/LibreOffice.client   |   11 
 tubes/LibreOfficeApprover.client   |8 
 tubes/Module_tubes.mk  |1 
 tubes/README   |   17 
 tubes/org.freedesktop.Telepathy.Client.LibreOffice.service |3 
 tubes/org.freedesktop.Telepathy.Client.LibreOfficeApprover.service |3 
 tubes/source/approver.c|  246 
++
 9 files changed, 337 insertions(+)

New commits:
commit 4b088d4ee611f05a003acc6146afba7a574c5e66
Author: Will Thompson 
Date:   Wed Mar 21 18:51:24 2012 +

tubes approver: add error handling in case spawning LibO fails

diff --git a/tubes/source/approver.c b/tubes/source/approver.c
index fdf0799..5f4bbde 100644
--- a/tubes/source/approver.c
+++ b/tubes/source/approver.c
@@ -45,16 +45,21 @@ handle_with_cb (GObject *source,
 gpointer user_data)
 {
   TpChannelDispatchOperation *cdo = TP_CHANNEL_DISPATCH_OPERATION (source);
-  GError *error;
+  GtkDialog *dialog = GTK_DIALOG (user_data);
+  GError *error = NULL;
 
   if (!tp_channel_dispatch_operation_handle_with_finish (cdo, result, &error))
 {
   g_print ("HandleWith() failed: %s\n", error->message);
+  gtk_message_dialog_format_secondary_markup (dialog,
+  "Error\n\nAsking LibreOffice to accept the session failed: 
%s",
+  error->message);
   g_error_free (error);
   return;
 }
 
   g_print ("HandleWith() succeeded\n");
+  gtk_widget_destroy (dialog);
 }
 
 static void
@@ -64,7 +69,7 @@ close_cb (GObject *source,
 
 {
   TpChannelDispatchOperation *cdo = TP_CHANNEL_DISPATCH_OPERATION (source);
-  GError *error;
+  GError *error = NULL;
 
   if (!tp_channel_dispatch_operation_close_channels_finish (cdo, result, 
&error))
 {
@@ -90,16 +95,19 @@ dialog_response_cb (
 g_print ("Approve channels\n");
 
 tp_channel_dispatch_operation_handle_with_async (cdo, NULL,
-handle_with_cb, NULL);
+handle_with_cb, dialog);
+
+gtk_dialog_set_response_sensitive (dialog, GTK_RESPONSE_ACCEPT, FALSE);
+gtk_dialog_set_response_sensitive (dialog, GTK_RESPONSE_REJECT, FALSE);
 }
 else
 {
 g_print ("Reject channels\n");
 
-tp_channel_dispatch_operation_close_channels_async (cdo, close_cb, 
NULL);
+tp_channel_dispatch_operation_close_channels_async (cdo, close_cb, 
dialog);
+gtk_widget_destroy (dialog);
 }
 
-gtk_widget_destroy (dialog);
 g_object_unref (cdo);
 }
 
commit 4da2b141304a47a300c97406b44d6aec05ce1bf3
Author: Will Thompson 
Date:   Wed Mar 21 18:38:27 2012 +

Add WIP .client and .service files.

These are roughly what need to get installed to make the approver (the
little dialog box that pops up for incoming offers) and handler
(LibreOffice itself) service-activatable.

Without these, your contacts will only see that you're capable of
LibreOffice tubes while you're running LibreOffice, and you won't get
a prompt unless liboapprover is running.

diff --git a/tubes/LibreOffice.client b/tubes/LibreOffice.client
new file mode 100644
index 000..e454611
--- /dev/null
+++ b/tubes/LibreOffice.client
@@ -0,0 +1,11 @@
+[org.freedesktop.Telepathy.Client]
+Interfaces = org.freedesktop.Telepathy.Client.Handler;
+
+[org.freedesktop.Telepathy.Client.Handler]
+BypassApproval = false
+
+[org.freedesktop.Telepathy.Client.Handler.HandlerChannelFilter 0]
+org.freedesktop.Telepathy.Channel.TargetHandleType u = 1
+org.freedesktop.Telepathy.Channel.ChannelType s = 
org.freedesktop.Telepathy.Channel.Type.DBusTube
+org.freedesktop.Telepathy.Channel.Type.DBusTube.ServiceName s = 
org.freedesktop.Telepathy.Client.LibreOfficeTeleTest
+
diff --git a/tubes/LibreOfficeApprover.client b/tubes/LibreOfficeApprover.client
new file mode 100644
index 000..da6d511
--- /dev/null
+++ b/tubes/LibreOfficeApprover.client
@@ -0,0 +1,8 @@
+[org.freedesktop.Telepathy.Client]
+Interfaces = org.freedesktop.Telepathy.Client.Approver;
+
+[org.freedesktop.Telepathy.Client.Approver.ApproverChannelFilter 0]
+org.freedesktop.Telepathy.Channel.TargetHandleType u = 1
+org.freedesktop.Telepathy.Channel.ChannelType s = 
org.freedesktop.Telepathy.Channel.Type.DBusTube
+org.freedesktop.Telepathy.Channel.Type.DBusTube.ServiceName s = 
org.freedesktop.Telepathy.Client.LibreOfficeTeleTest
+
diff --git a/tubes/README b/tubes/README
index 3527a72..84d4892 100644
--- a/tubes/README
+++ b/tubes/README
@@ -8,6 +8,23 @@ the same TeleConference abstraction.
 
 To enable configure LibO with --enable-telepathy
 
+If you have a Telepathy-enabled LibreOffice installed to /usr (including
+liboapprover) you may also want to install the .service and .c

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

2012-03-22 Thread Miklos Vajna
 oox/inc/oox/drawingml/drawingmltypes.hxx  |2 +-
 oox/inc/oox/vml/vmlshape.hxx  |1 +
 oox/source/token/properties.txt   |1 +
 oox/source/vml/vmlshape.cxx   |   14 +-
 oox/source/vml/vmlshapecontext.cxx|1 +
 sw/source/core/layout/anchoreddrawobject.cxx  |7 +--
 writerfilter/source/dmapper/DomainMapper_Impl.cxx |7 +++
 7 files changed, 29 insertions(+), 4 deletions(-)

New commits:
commit 51cfbf0cfaec395a99a00f2c20fcba96de9a4427
Author: Miklos Vajna 
Date:   Thu Mar 22 10:34:24 2012 +0100

n#751054 fix VML import of absolutely positioned pictures

There were multiple issues here:

- convertEmuToHmm() not handling negative values
- position:absolute style property being ignored
- mso-position-vertical-relative is not converted to
  text::RelOrientation
- SwAnchoredDrawObject::_SetPositioningAttr() re-positioning already
  positioned objects
- DomainMapper_Impl::PushShapeContext() inserting positioned objects as
  character

diff --git a/oox/inc/oox/drawingml/drawingmltypes.hxx 
b/oox/inc/oox/drawingml/drawingmltypes.hxx
index 600b1bc..71174b3 100644
--- a/oox/inc/oox/drawingml/drawingmltypes.hxx
+++ b/oox/inc/oox/drawingml/drawingmltypes.hxx
@@ -156,7 +156,7 @@ inline sal_Int64 convertHmmToEmu( sal_Int32 nValue )
 /** Converts the passed 64-bit integer value from EMUs to 1/100 mm. */
 inline sal_Int32 convertEmuToHmm( sal_Int64 nValue )
 {
-return getLimitedValue< sal_Int32, sal_Int64 >( (nValue + EMU_PER_HMM / 2) 
/ EMU_PER_HMM, 0, SAL_MAX_INT32 );
+return getLimitedValue< sal_Int32, sal_Int64 >( (nValue + EMU_PER_HMM / 2) 
/ EMU_PER_HMM, SAL_MIN_INT32, SAL_MAX_INT32 );
 }
 
 // 
diff --git a/oox/inc/oox/vml/vmlshape.hxx b/oox/inc/oox/vml/vmlshape.hxx
index 1998a02..37e9c09 100644
--- a/oox/inc/oox/vml/vmlshape.hxx
+++ b/oox/inc/oox/vml/vmlshape.hxx
@@ -79,6 +79,7 @@ struct ShapeTypeModel
 ::rtl::OUString maHeight;   /// Height of the shape 
bounding box (number with unit).
 ::rtl::OUString maMarginLeft;   /// X position of the shape 
bounding box to shape anchor (number with unit).
 ::rtl::OUString maMarginTop;/// Y position of the shape 
bounding box to shape anchor (number with unit).
+::rtl::OUString maPositionVerticalRelative; /// The Y position is 
relative to this.
 
 StrokeModel maStrokeModel;  /// Border line formatting.
 FillModel   maFillModel;/// Shape fill formatting.
diff --git a/oox/source/token/properties.txt b/oox/source/token/properties.txt
index 176ae08..560f879 100644
--- a/oox/source/token/properties.txt
+++ b/oox/source/token/properties.txt
@@ -498,6 +498,7 @@ VerticalAlign
 VerticalSplitMode
 VerticalSplitPositionTwips
 VertOrientPosition
+VertOrientRelation
 ViewBox
 Visible
 VisibleFlag
diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index c2fbd65..bdecb8b 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -42,6 +42,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include "oox/drawingml/shapepropertymap.hxx"
@@ -59,6 +60,7 @@
 using ::com::sun::star::beans::XPropertySet;
 using ::com::sun::star::uno::Any;
 
+using namespace ::com::sun::star;
 using namespace ::com::sun::star::text;
 
 namespace oox {
@@ -524,11 +526,21 @@ Reference< XShape > ComplexShape::implConvertAndInsert( 
const Reference< XShapes
 if( xShape.is() )
 {
 OUString aGraphicUrl = 
rFilter.getGraphicHelper().importEmbeddedGraphicObject( aGraphicPath );
+PropertySet aPropSet( xShape );
 if( !aGraphicUrl.isEmpty() )
 {
-PropertySet aPropSet( xShape );
 aPropSet.setProperty( PROP_GraphicURL, aGraphicUrl );
 }
+// If the shape has an absolute position, set the properties 
accordingly.
+if 
(maTypeModel.maPosition.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("absolute")))
+{
+aPropSet.setProperty(PROP_HoriOrientPosition, rShapeRect.X);
+aPropSet.setProperty(PROP_VertOrientPosition, rShapeRect.Y);
+}
+if 
(maTypeModel.maPositionVerticalRelative.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("page")))
+{
+aPropSet.setProperty(PROP_VertOrientRelation, 
text::RelOrientation::PAGE_FRAME);
+}
 }
 return xShape;
 }
diff --git a/oox/source/vml/vmlshapecontext.cxx 
b/oox/source/vml/vmlshapecontext.cxx
index e0e38ce..91703fc 100644
--- a/oox/source/vml/vmlshapecontext.cxx
+++ b/oox/source/vml/vmlshapecontext.cxx
@@ -346,6 +346,7 @@ void ShapeTypeContext::setStyle( const OUString& rStyle )
 else if( aName.equalsAsciiL( RTL_CONSTASCII_STRINGPARA

[Libreoffice-commits] .: Branch 'feature/tubes2' - sal/inc

2012-03-22 Thread Will Thompson
 sal/inc/rtl/bootstrap.h   |2 +-
 sal/inc/rtl/bootstrap.hxx |4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 180fc63ab8fe7d37ed3da364784a7f0946b918cb
Author: Will Thompson 
Date:   Thu Mar 22 09:57:36 2012 +

rtl::bootstap: rename filename parameter to contain "uri"

diff --git a/sal/inc/rtl/bootstrap.h b/sal/inc/rtl/bootstrap.h
index 3f89bde..1df1461 100644
--- a/sal/inc/rtl/bootstrap.h
+++ b/sal/inc/rtl/bootstrap.h
@@ -124,7 +124,7 @@ extern "C" {
 @param pName URL of the ini file; must not be null, must not be the empty
 string
 */
-SAL_DLLPUBLIC void SAL_CALL rtl_bootstrap_setIniFileName( rtl_uString *pName )
+SAL_DLLPUBLIC void SAL_CALL rtl_bootstrap_setIniFileName( rtl_uString 
*pFileUri )
 SAL_THROW_EXTERN_C();
 
 /**
diff --git a/sal/inc/rtl/bootstrap.hxx b/sal/inc/rtl/bootstrap.hxx
index 60ad350..789c17c 100644
--- a/sal/inc/rtl/bootstrap.hxx
+++ b/sal/inc/rtl/bootstrap.hxx
@@ -41,9 +41,9 @@ namespace rtl
 
 public:
 /**
-   @see rtl_bootstrap_setIniFileName()
+ * @see rtl_bootstrap_setIniFileName()
  */
-static inline void SAL_CALL setIniFilename( const ::rtl::OUString 
&sFile );
+static inline void SAL_CALL setIniFilename( const ::rtl::OUString 
&sFileUri );
 
 /** Retrieves a bootstrap parameter
@param sName name of the bootstrap value. case insensitive.
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: oowintool

2012-03-22 Thread Fridrich Strba
 oowintool |   10 +-
 1 file changed, 1 insertion(+), 9 deletions(-)

New commits:
commit a75fd2d50e8c0ad82bcd0bbfeafd6a7f0153b7ba
Author: Fridrich Å trba 
Date:   Thu Mar 22 10:41:44 2012 +0100

Look for properly registered windows sdks instead for random version numbers

diff --git a/oowintool b/oowintool
index ce55e88..06e6607 100755
--- a/oowintool
+++ b/oowintool
@@ -96,15 +96,7 @@ sub print_windows_sdk_home()
 {
 my ($value, $key);
 
-$value = reg_get_value ('HKEY_LOCAL_MACHINE/SOFTWARE/Microsoft/Microsoft 
SDKs/Windows/v6.1/InstallationFolder');
-
-if (!defined $value) {
-$value = reg_get_value 
('HKEY_LOCAL_MACHINE/SOFTWARE/Microsoft/Microsoft 
SDKs/Windows/v7.1/InstallationFolder');
-}
-
-if (!defined $value) {
-$value = reg_get_value 
('HKEY_LOCAL_MACHINE/SOFTWARE/Microsoft/Microsoft 
SDKs/Windows/CurrentInstallFolder');
-}
+$value = reg_get_value ('HKEY_LOCAL_MACHINE/SOFTWARE/Microsoft/Microsoft 
SDKs/Windows/CurrentInstallFolder');
 
 if (!defined $value) {
 $value = reg_get_value 
('HKEY_LOCAL_MACHINE/SOFTWARE/Microsoft/MicrosoftSDK/Directories/Install Dir');
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 4 commits - cppuhelper/inc sal/inc solenv/gbuild unoxml/source

2012-03-22 Thread Stephan Bergmann
 cppuhelper/inc/cppuhelper/implementationentry.hxx |8 
 sal/inc/sal/types.h   |5 -
 solenv/gbuild/platform/WNT_INTEL_GCC.mk   |4 
 unoxml/source/rdf/librdf_repository.cxx   |   12 ++--
 4 files changed, 14 insertions(+), 15 deletions(-)

New commits:
commit 7ec7f0df2960b74552224f0a9f7489f152b5faa1
Author: Stephan Bergmann 
Date:   Thu Mar 22 10:36:13 2012 +0100

MinGW apparently needs to be forced into __cdecl

...otherwise bridges/source/cpp_uno/mingw_intel C++ ABI assumptions would 
not
meet what i686-w64-mingw32-g++ emits (though it is unclear to me how this 
shall
ever have worked before, as kendy claims it once did; also, at least for me,
"wine soffice.exe" still does not work, as there is apparently confusion 
about
C++ exception stack unwinding between GCC and cpp_uno/mingw_intel code).

diff --git a/sal/inc/sal/types.h b/sal/inc/sal/types.h
index 7575be4..52b008f 100644
--- a/sal/inc/sal/types.h
+++ b/sal/inc/sal/types.h
@@ -256,13 +256,8 @@ typedef void *   sal_Handle;
 #endif // defined(_MSC_VER)
 #   define SAL_DLLPRIVATE
 #   define SAL_DLLPUBLIC_TEMPLATE
-#if defined(_MSC_VER)
 #   define SAL_CALL __cdecl
 #   define SAL_CALL_ELLIPSE __cdecl
-#else
-#   define SAL_CALL
-#   define SAL_CALL_ELLIPSE
-#endif
 #elif defined SAL_UNX
 #   if   defined(__SUNPRO_CC) && (__SUNPRO_CC >= 0x550)
 # define SAL_DLLPUBLIC_EXPORT  __global
diff --git a/solenv/gbuild/platform/WNT_INTEL_GCC.mk 
b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
index 161ad2c..363f1f3 100644
--- a/solenv/gbuild/platform/WNT_INTEL_GCC.mk
+++ b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
@@ -45,6 +45,7 @@ gb_COMPILERDEFS += \
-D_NATIVE_WCHAR_T_DEFINED \
-D_MSC_EXTENSIONS \
-D_FORCENAMELESSUNION \
+   -DBOOST_MEM_FN_ENABLE_CDECL \
 
 gb_RCDEFS := \
 -DWINVER=0x0400 \
commit ab2186d2c699f1ca041f7e3bde9620139a6f671c
Author: Stephan Bergmann 
Date:   Thu Mar 22 10:30:53 2012 +0100

MinGW: other DLLs must not pick sal's special __main

diff --git a/solenv/gbuild/platform/WNT_INTEL_GCC.mk 
b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
index 7d947ed..161ad2c 100644
--- a/solenv/gbuild/platform/WNT_INTEL_GCC.mk
+++ b/solenv/gbuild/platform/WNT_INTEL_GCC.mk
@@ -84,8 +84,11 @@ endif
 gb_LinkTarget_EXCEPTIONFLAGS += \
-fno-enforce-eh-specs \
 
+# At least sal defines its own __main, which would cause DLLs linking against
+# sal to pick up sal's __main instead of the one from MinGW's dllcrt2.o:
 gb_LinkTarget_LDFLAGS := \
-Wl,--export-all-symbols \
+   -Wl,--exclude-symbols,__main \
-Wl,--enable-stdcall-fixup \
-Wl,--enable-runtime-pseudo-reloc-v2 \
$(subst -L../lib , ,$(SOLARLIB)) \
commit 4fc8059d4452e3aca7b1edaf5fba873dca29987b
Author: Stephan Bergmann 
Date:   Thu Mar 22 10:27:48 2012 +0100

Fix SAL_CALL placement (so MinGW won't complain)

diff --git a/cppuhelper/inc/cppuhelper/implementationentry.hxx 
b/cppuhelper/inc/cppuhelper/implementationentry.hxx
index 173501d..655ac2b 100644
--- a/cppuhelper/inc/cppuhelper/implementationentry.hxx
+++ b/cppuhelper/inc/cppuhelper/implementationentry.hxx
@@ -45,17 +45,17 @@ struct ImplementationEntry
 /** Function that returns the implementation-name of the implementation
(same as XServiceInfo.getImplementationName() ).
  */
- rtl::OUString ( SAL_CALL * getImplementationName )();
+ rtl::OUString SAL_CALL ( * getImplementationName )();
 
 /** Function that returns all supported servicenames of the implementation
( same as XServiceInfo.getSupportedServiceNames() ).
 */
- com::sun::star::uno::Sequence< rtl::OUString > ( SAL_CALL * 
getSupportedServiceNames ) ();
+ com::sun::star::uno::Sequence< rtl::OUString > SAL_CALL ( * 
getSupportedServiceNames ) ();
 
 /** Function that creates a SingleComponentFactory.
 */
- ::com::sun::star::uno::Reference< 
::com::sun::star::lang::XSingleComponentFactory >
- ( SAL_CALL * createFactory )(
+ ::com::sun::star::uno::Reference< 
::com::sun::star::lang::XSingleComponentFactory > SAL_CALL
+ ( * createFactory )(
  ComponentFactoryFunc fptr,
  ::rtl::OUString const & rImplementationName,
  ::com::sun::star::uno::Sequence< ::rtl::OUString > const & 
rServiceNames,
commit 17c4bb3be34c234dab7d4986ddae50e6a7bfa065
Author: Stephan Bergmann 
Date:   Thu Mar 22 10:26:38 2012 +0100

Needless SAL_CALL

diff --git a/unoxml/source/rdf/librdf_repository.cxx 
b/unoxml/source/rdf/librdf_repository.cxx
index 85b40e9..f97d3be 100644
--- a/unoxml/source/rdf/librdf_repository.cxx
+++ b/unoxml/source/rdf/librdf_repository.cxx
@@ -335,7 +335,7 @@ public:
 const NamedGraphMap_t::iterator SAL_CALL clearGraph(
 const uno::Reference< rdf::XURI > & i_xName,
 bool i_Internal = false );
-void SAL_CALL addStatementGraph(
+void addStatementGraph(
 co

[Libreoffice-commits] .: sc/source

2012-03-22 Thread Stephan Bergmann
 sc/source/ui/unoobj/chart2uno.cxx |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit e1b4216816ba927f171030101f272f7ec0eb8314
Author: Stephan Bergmann 
Date:   Thu Mar 22 10:06:16 2012 +0100

ChartTokenMap can contain null pointers

db3786ee3aa0d8911042c9bbcea02cf36d96dfa1 caused chart2_unoapi test to crash

diff --git a/sc/source/ui/unoobj/chart2uno.cxx 
b/sc/source/ui/unoobj/chart2uno.cxx
index 469eb1d..9f5e3cd 100644
--- a/sc/source/ui/unoobj/chart2uno.cxx
+++ b/sc/source/ui/unoobj/chart2uno.cxx
@@ -309,9 +309,10 @@ Chart2PositionMap::Chart2PositionMap(SCCOL nAllColCount,  
SCROW nAllRowCount,
 ScTokenPtrMap::iterator tokenIter = pCol->begin();
 for (SCROW nRow = 0; !bFoundValues && nRow < 
nSmallestValueRowIndex; ++nRow)
 {
-if (tokenIter != pCol->end() && nRow>=nHeaderRowCount)
+ScToken* pToken =
+tokenIter == pCol->end() ? 0 : tokenIter->second;
+if (pToken && nRow>=nHeaderRowCount)
 {
-ScToken* pToken = tokenIter->second;
 ScRange aRange;
 bool bExternal = false;
 StackVar eType = pToken->GetType();
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: libcdr/libcdr-0.0.5-zip.patch libcdr/makefile.mk

2012-03-22 Thread Fridrich Strba
 libcdr/libcdr-0.0.5-zip.patch |  214 ++
 libcdr/makefile.mk|3 
 2 files changed, 216 insertions(+), 1 deletion(-)

New commits:
commit be79f65e92626a4c6dcda9b31922c53bc7e03349
Author: Fridrich Å trba 
Date:   Thu Mar 22 09:54:00 2012 +0100

Some changes in libcdr to appease valgrind

diff --git a/libcdr/libcdr-0.0.5-zip.patch b/libcdr/libcdr-0.0.5-zip.patch
new file mode 100644
index 000..0354cf9
--- /dev/null
+++ b/libcdr/libcdr-0.0.5-zip.patch
@@ -0,0 +1,214 @@
+--- misc/libcdr-0.0.5/src/lib/CDRInternalStream.cpp2012-03-15 
11:30:05.0 +0100
 misc/build/libcdr-0.0.5/src/lib/CDRInternalStream.cpp  2012-03-22 
09:51:18.381882859 +0100
+@@ -49,19 +49,19 @@
+   m_offset(0),
+   m_buffer()
+ {
+-  unsigned long tmpNumBytesRead = 0;
+-
+-  const unsigned char *tmpBuffer = 0;
++  if (!size)
++return;
+ 
+   if (!compressed)
+   {
+-tmpBuffer = input->read(size, tmpNumBytesRead);
++unsigned long tmpNumBytesRead = 0;
++const unsigned char *tmpBuffer = input->read(size, tmpNumBytesRead);
+ 
+ if (size != tmpNumBytesRead)
+   return;
+ 
+ m_buffer = std::vector(size);
+-memcpy(&m_buffer[0], &tmpBuffer[0], size);
++memcpy(&m_buffer[0], tmpBuffer, size);
+   }
+   else
+   {
+@@ -80,7 +80,8 @@
+ if (ret != Z_OK)
+   return;
+ 
+-tmpBuffer = input->read(size, tmpNumBytesRead);
++unsigned long tmpNumBytesRead = 0;
++const unsigned char *tmpBuffer = input->read(size, tmpNumBytesRead);
+ 
+ if (size != tmpNumBytesRead)
+   return;
+@@ -99,6 +100,7 @@
+   case Z_DATA_ERROR:
+   case Z_MEM_ERROR:
+ (void)inflateEnd(&strm);
++m_buffer.clear();
+ return;
+   }
+ 
+@@ -109,7 +111,7 @@
+ 
+ }
+ while (strm.avail_out == 0);
+-
++(void)inflateEnd(&strm);
+   }
+ }
+ 
+--- misc/libcdr-0.0.5/src/lib/CDRZipStream.cpp 2012-03-16 11:20:15.0 
+0100
 misc/build/libcdr-0.0.5/src/lib/CDRZipStream.cpp   2012-03-22 
09:51:00.332335588 +0100
+@@ -30,6 +30,7 @@
+ 
+ 
+ #include 
++#include 
+ #include "CDRZipStream.h"
+ #include "CDRInternalStream.h"
+ #include "libcdr_utils.h"
+@@ -231,9 +232,9 @@
+   return true;
+ }
+ 
+-static bool findCentralDirectoryEnd(WPXInputStream *input, long &startOffset)
++static bool findCentralDirectoryEnd(WPXInputStream *input)
+ {
+-  input->seek(startOffset, WPX_SEEK_SET);
++  input->seek(0, WPX_SEEK_SET);
+   try
+   {
+ while (!input->atEOS())
+@@ -242,7 +243,6 @@
+   if (signature == CDIR_END_SIG)
+   {
+ input->seek(-4, WPX_SEEK_CUR);
+-startOffset = input->tell();
+ return true;
+   }
+   else
+@@ -256,9 +256,9 @@
+   return false;
+ }
+ 
+-static bool isZipStream(WPXInputStream *input, long &startOffset)
++static bool isZipStream(WPXInputStream *input)
+ {
+-  if (!findCentralDirectoryEnd(input, startOffset))
++  if (!findCentralDirectoryEnd(input))
+ return false;
+   CentralDirectoryEnd end;
+   if (!readCentralDirectoryEnd(input, end))
+@@ -276,17 +276,16 @@
+   return true;
+ }
+ 
+-static bool findDataStream(WPXInputStream *input, unsigned &size, bool 
&compressed, long &startOffset, const char *name)
++static bool findDataStream(WPXInputStream *input, CentralDirectoryEntry 
&entry, const char *name)
+ {
+   unsigned short name_size = strlen(name);
+-  if (!findCentralDirectoryEnd(input, startOffset))
++  if (!findCentralDirectoryEnd(input))
+ return false;
+   CentralDirectoryEnd end;
+   if (!readCentralDirectoryEnd(input, end))
+ return false;
+   input->seek(end.cdir_offset, WPX_SEEK_SET);
+-  CentralDirectoryEntry entry;
+-  while (!input->atEOS() && input->tell() < startOffset && input->tell() < 
end.cdir_offset + end.cdir_size)
++  while (!input->atEOS() && (unsigned)input->tell() < end.cdir_offset + 
end.cdir_size)
+   {
+ if (!readCentralDirectoryEntry(input, entry))
+   return false;
+@@ -303,17 +302,63 @@
+ return false;
+   if (!areHeadersConsistent(header, entry))
+ return false;
+-  size = entry.uncompressed_size;
+-  compressed = (entry.compression != 0);
+   return true;
+ }
+ 
++WPXInputStream *getSubstream(WPXInputStream *input, const char *name)
++{
++  CentralDirectoryEntry entry;
++  if (!findDataStream(input, entry, name))
++return 0;
++  if (!entry.compression)
++return new CDRInternalStream(input, entry.compressed_size);
++  else
++  {
++int ret;
++z_stream strm;
++
++/* allocate inflate state */
++strm.zalloc = Z_NULL;
++strm.zfree = Z_NULL;
++strm.opaque = Z_NULL;
++strm.avail_in = 0;
++strm.next_in = Z_NULL;
++ret = inflateInit2(&strm,-MAX_WBITS);
++if (ret != Z_OK)
++  return 0;
++
++unsigned long numBytesRead = 0;
++const unsigned char *compressedData = input->read(entry.compressed_size, 
numBytesRead);
++if (numBytesRead != entry.compressed_size)
++  return 0;
++
++strm.avail_in = numBytesRead;
++strm.next_in = (Bytef

[Libreoffice-commits] .: basctl/source

2012-03-22 Thread Fridrich Strba
 basctl/source/basicide/basides1.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 0589dd3fb5c5bf2fdcdbccfab31fbe26c0a9727d
Author: Fridrich Å trba 
Date:   Thu Mar 22 09:22:42 2012 +0100

fix debug build

diff --git a/basctl/source/basicide/basides1.cxx 
b/basctl/source/basicide/basides1.cxx
index 6702f93..d9f5b63 100644
--- a/basctl/source/basicide/basides1.cxx
+++ b/basctl/source/basicide/basides1.cxx
@@ -468,6 +468,7 @@ void BasicIDEShell::ExecuteGlobal( SfxRequest& rReq )
 if ( aIDEWindowTable.find( rTabId.GetValue() ) !=  
aIDEWindowTable.end() )
 {
 IDEBaseWindow* pWin = aIDEWindowTable[ rTabId.GetValue() ];
+DBG_ASSERT( pWin, "Window nicht im Liste, aber in TabBar ?" );
 ::rtl::OUString aNewName( rModName.GetValue() );
 ::rtl::OUString aOldName( pWin->GetName() );
 if ( aNewName != aOldName )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: basctl/source

2012-03-22 Thread Stephan Bergmann
 basctl/source/basicide/basides1.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit a603634f3539d472b66f71419f02a14260dcaf5b
Author: Stephan Bergmann 
Date:   Thu Mar 22 08:12:08 2012 +0100

Fixed previous commit for --enable-dbgutil mode

diff --git a/basctl/source/basicide/basides1.cxx 
b/basctl/source/basicide/basides1.cxx
index d0c9fba..6702f93 100644
--- a/basctl/source/basicide/basides1.cxx
+++ b/basctl/source/basicide/basides1.cxx
@@ -465,7 +465,6 @@ void BasicIDEShell::ExecuteGlobal( SfxRequest& rReq )
 DBG_ASSERT( rReq.GetArgs(), "arguments expected" );
 const SfxUInt16Item &rTabId = (const 
SfxUInt16Item&)rReq.GetArgs()->Get(SID_BASICIDE_ARG_TABID );
 const SfxStringItem &rModName = (const 
SfxStringItem&)rReq.GetArgs()->Get(SID_BASICIDE_ARG_MODULENAME );
-DBG_ASSERT( pWin, "Window nicht im Liste, aber in TabBar ?" );
 if ( aIDEWindowTable.find( rTabId.GetValue() ) !=  
aIDEWindowTable.end() )
 {
 IDEBaseWindow* pWin = aIDEWindowTable[ rTabId.GetValue() ];
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits