[Libreoffice-commits] core.git: sc/source

2013-03-26 Thread Tor Lillqvist
 sc/source/filter/inc/XclImpChangeTrack.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit fd652cf5cbd4480ca8754f8faaf96f0f4455ddc0
Author: Tor Lillqvist 
Date:   Tue Mar 26 09:05:18 2013 +0200

WaE: 'ScCellValue' defined as a struct here but previously declared as a 
class

Change-Id: Ia848c70ce38884a6bf3f3bac81ddf8e9c479f218

diff --git a/sc/source/filter/inc/XclImpChangeTrack.hxx 
b/sc/source/filter/inc/XclImpChangeTrack.hxx
index f1e18fb..2fceacd 100644
--- a/sc/source/filter/inc/XclImpChangeTrack.hxx
+++ b/sc/source/filter/inc/XclImpChangeTrack.hxx
@@ -29,7 +29,7 @@
 
 //___
 
-class ScCellValue;
+struct ScCellValue;
 class ScChangeAction;
 class ScChangeTrack;
 class XclImpChTrFmlConverter;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [Libreoffice-commits] core.git: make bridges work with clang on x64 linux

2013-03-26 Thread Stephan Bergmann

On 03/25/2013 07:56 PM, Peter Foley wrote:

When I tried to build libreoffice with clang 3.2 for x86-64 linux I
got a segfault in CustomTarget_uno_test.


What Linux version would that be?


After I made the above change the segfault went away.
I don't really understand the bridges module so there might be a
better way to fix this, but disabling optimization on that file worked
for me.


Yeah, disabling optimization sometimes helps to work around symptoms. 
The bad thing with it though is that as long as it is unclear why 
exactly it helped work around a problem, nobody ever dares undo that 
noopt change again.


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


[Libreoffice-commits] core.git: 2 commits - filter/Configuration_filter.mk filter/source writerperfect/source

2013-03-26 Thread Laurent Alonso
 filter/Configuration_filter.mk|5 +
 filter/source/config/fragments/filters/DocMaker.xcu   |2 
 filter/source/config/fragments/filters/Mariner_Write.xcu  |2 
 filter/source/config/fragments/filters/Nisus_Writer.xcu   |2 
 filter/source/config/fragments/filters/TexEdit.xcu|   30 ++
 filter/source/config/fragments/filters/WriterPlus.xcu |   30 ++
 filter/source/config/fragments/filters/eDoc_Document.xcu  |2 
 filter/source/config/fragments/types/writer_DocMaker.xcu  |4 -
 filter/source/config/fragments/types/writer_LightWayText.xcu  |2 
 filter/source/config/fragments/types/writer_Mariner_Write.xcu |2 
 filter/source/config/fragments/types/writer_Nisus_Writer.xcu  |4 -
 filter/source/config/fragments/types/writer_TeachText.xcu |2 
 filter/source/config/fragments/types/writer_TexEdit.xcu   |4 -
 filter/source/config/fragments/types/writer_WriterPlus.xcu|2 
 filter/source/config/fragments/types/writer_ZWrite.xcu|2 
 filter/source/config/fragments/types/writer_eDoc_Document.xcu |4 -
 filter/source/t602/t602filter.cxx |   27 ++---
 writerperfect/source/writer/MWAWImportFilter.cxx  |4 -
 18 files changed, 91 insertions(+), 39 deletions(-)

New commits:
commit 2b28b922e11e2d45eeeba8765096d974f4908157
Author: Laurent Alonso 
Date:   Tue Mar 26 08:43:51 2013 +0100

Some fixes for mwaw-based filters

Change-Id: Ibb517b9a4690f20baa423472c001314a4d4c43f7

diff --git a/filter/Configuration_filter.mk b/filter/Configuration_filter.mk
index 7ca392c..192a68f 100644
--- a/filter/Configuration_filter.mk
+++ b/filter/Configuration_filter.mk
@@ -338,7 +338,9 @@ $(call 
filter_Configuration_add_types,fcfg_langpack,fcfg_writer_types.xcu,filter
writer_MindWrite \
writer_Nisus_Writer \
writer_TeachText \
+   writer_TexEdit \
writer_WriteNow \
+   writer_WriterPlus \
writer_ZWrite \
writer_T602_Document \
writer_LotusWordPro_Document \
@@ -372,7 +374,6 @@ $(call 
filter_Configuration_add_filters,fcfg_langpack,fcfg_writer_filters.xcu,fi
DocMaker \
eDoc_Document \
FullWrite_Professional \
-   HanMac_Word_J \
HanMac_Word_K \
LightWayText \
Mac_Word \
@@ -383,7 +384,9 @@ $(call 
filter_Configuration_add_filters,fcfg_langpack,fcfg_writer_filters.xcu,fi
MindWrite \
Nisus_Writer \
TeachText \
+   TexEdit \
WriteNow \
+   WriterPlus \
ZWrite \
T602Document \
LotusWordPro \
diff --git a/filter/source/config/fragments/filters/DocMaker.xcu 
b/filter/source/config/fragments/filters/DocMaker.xcu
index 968abdc..9170686 100644
--- a/filter/source/config/fragments/filters/DocMaker.xcu
+++ b/filter/source/config/fragments/filters/DocMaker.xcu
@@ -21,7 +21,7 @@
 com.sun.star.comp.Writer.MWAWImportFilter
 
 
-DocMaker (v4) Document
+DOCMaker (v4) Document
 
 0
 writer_DocMaker
diff --git a/filter/source/config/fragments/filters/Mariner_Write.xcu 
b/filter/source/config/fragments/filters/Mariner_Write.xcu
index 42f140e..e5a342f 100644
--- a/filter/source/config/fragments/filters/Mariner_Write.xcu
+++ b/filter/source/config/fragments/filters/Mariner_Write.xcu
@@ -21,7 +21,7 @@
 com.sun.star.comp.Writer.MWAWImportFilter
 
 
-Mariner Write v1.6 - v3.5
+Mariner Write Mac Classic v1.6 - 
v3.5
 
 0
 writer_Mariner_Write
diff --git a/filter/source/config/fragments/filters/Nisus_Writer.xcu 
b/filter/source/config/fragments/filters/Nisus_Writer.xcu
index 02df227..66403e5 100644
--- a/filter/source/config/fragments/filters/Nisus_Writer.xcu
+++ b/filter/source/config/fragments/filters/Nisus_Writer.xcu
@@ -21,7 +21,7 @@
 com.sun.star.comp.Writer.MWAWImportFilter
 
 
-Nisus Writer Document v3.4 - 
6.5
+Nisus Writer Mac Classic v3.4 - 
6.5
 
 0
 writer_Nisus_Writer
diff --git a/filter/source/config/fragments/filters/TexEdit.xcu 
b/filter/source/config/fragments/filters/TexEdit.xcu
index 36b353d..d505f85 100644
--- a/filter/source/config/fragments/filters/TexEdit.xcu
+++ b/filter/source/config/fragments/filters/TexEdit.xcu
@@ -21,6 +21,36 @@
 com.sun.star.comp.Writer.MWAWImportFilter
 
 
+TexEdit v2 Document
+
+0
+writer_TexEdit
+
+com.sun.star.text.TextDocument
+
+
+
+IMPORT ALIEN USESOPTIONS 3RDPARTYFILTER 
PREFERRED
+
+com.sun.star.comp.Writer.MWAWImportFilter
+
+
 Tex-Edit v2 Document
 
 0
diff --git a/filter/source/config/fragments/filters/WriterPlus.xcu 
b/filter/source/config/fragments/filters/Write

[Libreoffice-commits] core.git: Branch 'libreoffice-4-0' - sfx2/inc sfx2/source

2013-03-26 Thread Cédric Bosdonnat
 sfx2/inc/sfx2/thumbnailview.hxx   |1 +
 sfx2/source/control/thumbnailview.cxx |   31 +--
 sfx2/source/doc/templatedlg.cxx   |2 +-
 3 files changed, 31 insertions(+), 3 deletions(-)

New commits:
commit 369cb8e216f9287234d601f4b28e0332a2ec29a5
Author: Cédric Bosdonnat 
Date:   Mon Mar 25 16:40:37 2013 +0100

fdo#61390: allow arrow keys to show hidden thumbnail items

Change-Id: I70924b4c9578122a1c5a5b6c8a67889d6d003a2e
(cherry picked from commit cc599e914e7ade9f10da460fc1c77ea07454d7ff)
Reviewed-on: https://gerrit.libreoffice.org/3033
Reviewed-by: Miklos Vajna 
Tested-by: Miklos Vajna 

diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 95e6b53..9e29c9a 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -277,6 +277,7 @@ protected:
 using Window::ImplInit;
 
 void CalculateItemPositions ();
+void MakeItemVisible( sal_uInt16 nId );
 
 SFX2_DLLPRIVATE void ImplInit();
 SFX2_DLLPRIVATE void ImplInitSettings( bool bFont, bool 
bForeground, bool bBackground );
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index f78a308..08be7cb 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -553,13 +553,40 @@ void ThumbnailView::KeyInput( const KeyEvent& rKEvt )
 Control::KeyInput( rKEvt );
 }
 
-if ( pNext && pNext->isVisible() )
+if ( pNext )
 {
 deselectItems();
 SelectItem(pNext->mnId);
+MakeItemVisible(pNext->mnId);
 }
 }
 
+void ThumbnailView::MakeItemVisible( sal_uInt16 nItemId )
+{
+// Get the item row
+size_t nPos = 0;
+bool bFound = false;
+for ( size_t i = 0; !bFound && i < mFilteredItemList.size(); ++i )
+{
+ThumbnailViewItem* pItem = mFilteredItemList[i];
+if ( pItem->mnId == nItemId )
+{
+nPos = i;
+bFound = true;
+}
+}
+sal_uInt16 nRow = nPos / mnCols;
+
+// Move the visible rows as little as possible to include that one
+if ( nRow < mnFirstLine )
+mnFirstLine = nRow;
+else if ( nRow > mnFirstLine + mnVisLines )
+mnFirstLine = nRow - mnVisLines;
+
+CalculateItemPositions();
+Invalidate();
+}
+
 void ThumbnailView::MouseButtonDown( const MouseEvent& rMEvt )
 {
 if ( rMEvt.IsLeft() )
@@ -703,11 +730,11 @@ void ThumbnailView::LoseFocus()
 
 void ThumbnailView::Resize()
 {
+Control::Resize();
 CalculateItemPositions();
 
 if ( IsReallyVisible() && IsUpdateMode() )
 Invalidate();
-Control::Resize();
 }
 
 void ThumbnailView::StateChanged( StateChangedType nType )
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index b9d3945..e5e4014 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -331,7 +331,7 @@ void SfxTemplateManagerDlg::Resize()
 Point aViewPos = maView->GetPosPixel();
 aViewPos.setY(nToolbarsHeight);
 aViewPos.setX(0);
-Size aThumbSize(aWinSize.getWidth(), 
maTabControl.GetTabPageSizePixel().getWidth() - aViewPos.getY());
+Size aThumbSize(aWinSize.getWidth(), 
maTabControl.GetTabPageSizePixel().getHeight() - aViewPos.getY());
 maView->SetPosSizePixel(aViewPos, aThumbSize);
 
 if (aWinSize.getHeight() < aViewPos.getY() + aThumbSize.getHeight() + 
PADDING_DLG_BORDER)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-4-0] fdo#61390: allow arrow keys to show hidden thumbnail items

2013-03-26 Thread Miklos Vajna (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3033

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I70924b4c9578122a1c5a5b6c8a67889d6d003a2e
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric 
Gerrit-Reviewer: Miklos Vajna 

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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-0' - sfx2/source

2013-03-26 Thread Cédric Bosdonnat
 sfx2/source/control/thumbnailview.cxx |   22 +-
 1 file changed, 13 insertions(+), 9 deletions(-)

New commits:
commit edc8b07e5f69413b3710ddf192a84a862b0e17af
Author: Cédric Bosdonnat 
Date:   Mon Mar 25 17:58:49 2013 +0100

fdo#62659: only select items in ThumbnailView on TabStop

Change-Id: I024df8520f0daf2cc2e9e5f041e1ff18cb3689ec
(cherry picked from commit f8a3582d4cbcf6a705f724b921e370c739ecee35)
Reviewed-on: https://gerrit.libreoffice.org/3034
Reviewed-by: Miklos Vajna 
Tested-by: Miklos Vajna 

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 08be7cb..0da633f 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -126,6 +126,7 @@ void ThumbnailView::ImplDeleteItems()
 }
 
 mItemList.clear();
+mFilteredItemList.clear();
 }
 
 void ThumbnailView::ImplInitSettings( bool bFont, bool bForeground, bool 
bBackground )
@@ -680,17 +681,20 @@ void ThumbnailView::Paint( const Rectangle &aRect)
 
 void ThumbnailView::GetFocus()
 {
-// Select the first item if nothing selected
-int nSelected = -1;
-for (size_t i = 0, n = mItemList.size(); i < n && nSelected == -1; ++i)
+if(GETFOCUS_TAB & GetGetFocusFlags())
 {
-if (mItemList[i]->isSelected())
-nSelected = i;
-}
+// Select the first item if nothing selected
+int nSelected = -1;
+for (size_t i = 0, n = mItemList.size(); i < n && nSelected == -1; ++i)
+{
+if (mItemList[i]->isSelected())
+nSelected = i;
+}
 
-if ( nSelected == -1 && mItemList.size( ) > 0 )
-{
-SelectItem( 1 );
+if ( nSelected == -1 && mItemList.size( ) > 0 )
+{
+SelectItem( 1 );
+}
 }
 
 // Tell the accessible object that we got the focus.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-4-0] fdo#62659: only select items in ThumbnailView on TabStop

2013-03-26 Thread Miklos Vajna (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3034

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I024df8520f0daf2cc2e9e5f041e1ff18cb3689ec
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric 
Gerrit-Reviewer: Miklos Vajna 

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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-0' - i18npool/source

2013-03-26 Thread Eike Rathke
 i18npool/source/calendar/calendar_gregorian.cxx |   19 ++-
 1 file changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 6f9b1eb8611ad706725388b15038a6673143fe34
Author: Eike Rathke 
Date:   Mon Mar 25 21:21:41 2013 +0100

resolved fdo#44286 some historical dates were decremented

Depending on historical timezone data with odd offsets some historical
dates were decremented by one day due to a rounding error.

Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
(cherry picked from commit 6013fe19a40dd16ce435a2428f7405b51930689e)
Reviewed-on: https://gerrit.libreoffice.org/3036
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/i18npool/source/calendar/calendar_gregorian.cxx 
b/i18npool/source/calendar/calendar_gregorian.cxx
index a2a5324..e01abaf 100644
--- a/i18npool/source/calendar/calendar_gregorian.cxx
+++ b/i18npool/source/calendar/calendar_gregorian.cxx
@@ -26,6 +26,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -283,8 +284,19 @@ Calendar_gregorian::getUniqueID() throw(RuntimeException)
 void SAL_CALL
 Calendar_gregorian::setDateTime( double timeInDays ) throw(RuntimeException)
 {
+// ICU handles dates in milliseconds as double values and uses floor()
+// to obtain integer values, which may yield a date decremented by one
+// for odd (historical) timezone values where the computed value due to
+// rounding errors has a fractional part in milliseconds. Ensure we
+// pass a value without fraction here. If not, that may lead to
+// fdo#44286 or fdo#52619 and the like, e.g. when passing
+// -2136315212000.000244 instead of -2136315212000.00
+double fM = timeInDays * U_MILLIS_PER_DAY;
+double fR = rtl::math::round( fM );
+SAL_INFO_IF( fM != fR, "i18npool",
+"Calendar_gregorian::setDateTime: " << std::fixed << fM << " 
rounded to " << fR);
 UErrorCode status;
-body->setTime(timeInDays * U_MILLIS_PER_DAY, status = U_ZERO_ERROR);
+body->setTime( fR, status = U_ZERO_ERROR);
 if ( !U_SUCCESS(status) ) throw ERROR;
 getValue();
 }
@@ -679,6 +691,11 @@ void Calendar_gregorian::setValue() throw(RuntimeException)
 DUMP_ICU_CAL_MSG(("%s\n","setValue() after Zone/DST glitch 
2nd resubmit"));
 DUMP_I18N_CAL_MSG(("%s\n","setValue() after Zone/DST 
glitch 2nd resubmit"));
 }
+SAL_INFO( "i18npool", "Calendar_gregorian::setValue:" <<
+"  nZone0 " << nZone0 << ", nDST0 " << nDST0 <<
+", nZone1 " << nZone1 << ", nDST1 " << nDST1 <<
+", nZone2 " << nZone2 << ", nDST2 " << nDST2 <<
+", nZone3 " << nZone3 << ", nDST3 " << nDST3);
 }
 }
 #if erDUMP_ICU_CALENDAR || erDUMP_I18N_CALENDAR
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-4-0] resolved fdo#44286 some historical dates were decremented

2013-03-26 Thread Thorsten Behrens (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3036

Approvals:
  Thorsten Behrens: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Eike Rathke 
Gerrit-Reviewer: Thorsten Behrens 

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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-0' - sw/source

2013-03-26 Thread Joren De Cuyper
 sw/source/ui/table/splittbl.cxx |6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit 64a3b58a9ace11c6a01708e8de14712e27e02043
Author: Joren De Cuyper 
Date:   Mon Mar 25 22:11:59 2013 +0100

fix fdo#62670 - TABLE: Splitting a table options are mixed

Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Reviewed-on: https://gerrit.libreoffice.org/3042
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 

diff --git a/sw/source/ui/table/splittbl.cxx b/sw/source/ui/table/splittbl.cxx
index f82d642..d85bb95 100644
--- a/sw/source/ui/table/splittbl.cxx
+++ b/sw/source/ui/table/splittbl.cxx
@@ -30,7 +30,7 @@ SwSplitTblDlg::SwSplitTblDlg( Window *pParent, SwWrtShell 
&rSh )
 get(mpCntntCopyRB, "copyheading");
 get(mpBoxAttrCopyWithParaRB, "customheadingapplystyle");
 get(mpBoxAttrCopyNoParaRB, "customheading");
-get(mpBorderCopyRB, "customheading");
+get(mpBorderCopyRB, "noheading");
 }
 
 void SwSplitTblDlg::Apply()
@@ -38,7 +38,7 @@ void SwSplitTblDlg::Apply()
 m_nSplit = HEADLINE_CNTNTCOPY;
 if(mpBoxAttrCopyWithParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATRCOLLCOPY;
-if(mpBoxAttrCopyNoParaRB->IsChecked())
+else if(mpBoxAttrCopyNoParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATTRCOPY;
 else if(mpBorderCopyRB->IsChecked())
 m_nSplit = HEADLINE_BORDERCOPY;
@@ -46,6 +46,4 @@ void SwSplitTblDlg::Apply()
 rShell.SplitTable(m_nSplit);
 }
 
-
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-0-2' - i18npool/source

2013-03-26 Thread Eike Rathke
 i18npool/source/calendar/calendar_gregorian.cxx |   19 ++-
 1 file changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 7d7c4aa9f07e309e9185bab77a197b3dbe1f2dd1
Author: Eike Rathke 
Date:   Mon Mar 25 21:21:41 2013 +0100

resolved fdo#44286 some historical dates were decremented

Depending on historical timezone data with odd offsets some historical
dates were decremented by one day due to a rounding error.

Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
(cherry picked from commit 6013fe19a40dd16ce435a2428f7405b51930689e)
Reviewed-on: https://gerrit.libreoffice.org/3037
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 
Reviewed-by: Markus Mohrhard 
Tested-by: Markus Mohrhard 

diff --git a/i18npool/source/calendar/calendar_gregorian.cxx 
b/i18npool/source/calendar/calendar_gregorian.cxx
index a2a5324..e01abaf 100644
--- a/i18npool/source/calendar/calendar_gregorian.cxx
+++ b/i18npool/source/calendar/calendar_gregorian.cxx
@@ -26,6 +26,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -283,8 +284,19 @@ Calendar_gregorian::getUniqueID() throw(RuntimeException)
 void SAL_CALL
 Calendar_gregorian::setDateTime( double timeInDays ) throw(RuntimeException)
 {
+// ICU handles dates in milliseconds as double values and uses floor()
+// to obtain integer values, which may yield a date decremented by one
+// for odd (historical) timezone values where the computed value due to
+// rounding errors has a fractional part in milliseconds. Ensure we
+// pass a value without fraction here. If not, that may lead to
+// fdo#44286 or fdo#52619 and the like, e.g. when passing
+// -2136315212000.000244 instead of -2136315212000.00
+double fM = timeInDays * U_MILLIS_PER_DAY;
+double fR = rtl::math::round( fM );
+SAL_INFO_IF( fM != fR, "i18npool",
+"Calendar_gregorian::setDateTime: " << std::fixed << fM << " 
rounded to " << fR);
 UErrorCode status;
-body->setTime(timeInDays * U_MILLIS_PER_DAY, status = U_ZERO_ERROR);
+body->setTime( fR, status = U_ZERO_ERROR);
 if ( !U_SUCCESS(status) ) throw ERROR;
 getValue();
 }
@@ -679,6 +691,11 @@ void Calendar_gregorian::setValue() throw(RuntimeException)
 DUMP_ICU_CAL_MSG(("%s\n","setValue() after Zone/DST glitch 
2nd resubmit"));
 DUMP_I18N_CAL_MSG(("%s\n","setValue() after Zone/DST 
glitch 2nd resubmit"));
 }
+SAL_INFO( "i18npool", "Calendar_gregorian::setValue:" <<
+"  nZone0 " << nZone0 << ", nDST0 " << nDST0 <<
+", nZone1 " << nZone1 << ", nDST1 " << nDST1 <<
+", nZone2 " << nZone2 << ", nDST2 " << nDST2 <<
+", nZone3 " << nZone3 << ", nDST3 " << nDST3);
 }
 }
 #if erDUMP_ICU_CALENDAR || erDUMP_I18N_CALENDAR
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/source

2013-03-26 Thread Joren De Cuyper
 sw/source/ui/table/splittbl.cxx |6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit b23214f768e94f3fbdb0317b5fd458c55a9118db
Author: Joren De Cuyper 
Date:   Mon Mar 25 22:11:59 2013 +0100

fix fdo#62670 - TABLE: Splitting a table options are mixed

Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Reviewed-on: https://gerrit.libreoffice.org/3041
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 

diff --git a/sw/source/ui/table/splittbl.cxx b/sw/source/ui/table/splittbl.cxx
index f82d642..d85bb95 100644
--- a/sw/source/ui/table/splittbl.cxx
+++ b/sw/source/ui/table/splittbl.cxx
@@ -30,7 +30,7 @@ SwSplitTblDlg::SwSplitTblDlg( Window *pParent, SwWrtShell 
&rSh )
 get(mpCntntCopyRB, "copyheading");
 get(mpBoxAttrCopyWithParaRB, "customheadingapplystyle");
 get(mpBoxAttrCopyNoParaRB, "customheading");
-get(mpBorderCopyRB, "customheading");
+get(mpBorderCopyRB, "noheading");
 }
 
 void SwSplitTblDlg::Apply()
@@ -38,7 +38,7 @@ void SwSplitTblDlg::Apply()
 m_nSplit = HEADLINE_CNTNTCOPY;
 if(mpBoxAttrCopyWithParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATRCOLLCOPY;
-if(mpBoxAttrCopyNoParaRB->IsChecked())
+else if(mpBoxAttrCopyNoParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATTRCOPY;
 else if(mpBorderCopyRB->IsChecked())
 m_nSplit = HEADLINE_BORDERCOPY;
@@ -46,6 +46,4 @@ void SwSplitTblDlg::Apply()
 rShell.SplitTable(m_nSplit);
 }
 
-
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-4-0] fix fdo#62670 - TABLE: Splitting a table options are mixed

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3042

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Joren De Cuyper 
Gerrit-Reviewer: Petr Mladek 

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


[PUSHED libreoffice-4-0-2] resolved fdo#44286 some historical dates were decremented

2013-03-26 Thread Markus Mohrhard (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3037

Approvals:
  Markus Mohrhard: Verified; Looks good to me, approved
  Petr Mladek: Verified; Looks good to me, but someone else must approve
  Thorsten Behrens: Verified; Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-2
Gerrit-Owner: Eike Rathke 
Gerrit-Reviewer: Markus Mohrhard 
Gerrit-Reviewer: Petr Mladek 
Gerrit-Reviewer: Thorsten Behrens 

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


[PUSHED] fix fdo#62670 - TABLE: Splitting a table options are mixed

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3041

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Joren De Cuyper 
Gerrit-Reviewer: Petr Mladek 

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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-0-2' - sw/source

2013-03-26 Thread Joren De Cuyper
 sw/source/ui/table/splittbl.cxx |6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit aa85ff19b4acaf98adfaa8787afce5698e7b085e
Author: Joren De Cuyper 
Date:   Mon Mar 25 22:11:59 2013 +0100

fix fdo#62670 - TABLE: Splitting a table options are mixed

Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Reviewed-on: https://gerrit.libreoffice.org/3043
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 
Reviewed-by: Fridrich Strba 
Reviewed-by: Michael Meeks 
Tested-by: Michael Meeks 

diff --git a/sw/source/ui/table/splittbl.cxx b/sw/source/ui/table/splittbl.cxx
index f82d642..d85bb95 100644
--- a/sw/source/ui/table/splittbl.cxx
+++ b/sw/source/ui/table/splittbl.cxx
@@ -30,7 +30,7 @@ SwSplitTblDlg::SwSplitTblDlg( Window *pParent, SwWrtShell 
&rSh )
 get(mpCntntCopyRB, "copyheading");
 get(mpBoxAttrCopyWithParaRB, "customheadingapplystyle");
 get(mpBoxAttrCopyNoParaRB, "customheading");
-get(mpBorderCopyRB, "customheading");
+get(mpBorderCopyRB, "noheading");
 }
 
 void SwSplitTblDlg::Apply()
@@ -38,7 +38,7 @@ void SwSplitTblDlg::Apply()
 m_nSplit = HEADLINE_CNTNTCOPY;
 if(mpBoxAttrCopyWithParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATRCOLLCOPY;
-if(mpBoxAttrCopyNoParaRB->IsChecked())
+else if(mpBoxAttrCopyNoParaRB->IsChecked())
 m_nSplit = HEADLINE_BOXATTRCOPY;
 else if(mpBorderCopyRB->IsChecked())
 m_nSplit = HEADLINE_BORDERCOPY;
@@ -46,6 +46,4 @@ void SwSplitTblDlg::Apply()
 rShell.SplitTable(m_nSplit);
 }
 
-
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-4-0-2] fix fdo#62670 - TABLE: Splitting a table options are mixed

2013-03-26 Thread Michael Meeks (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3043

Approvals:
  Petr Mladek: Verified; Looks good to me, but someone else must approve
  Michael Meeks: Verified; Looks good to me, approved
  Fridrich Strba: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ba647844aaae3a57ee91d5785c82ca082d3e10b
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0-2
Gerrit-Owner: Joren De Cuyper 
Gerrit-Reviewer: Fridrich Strba 
Gerrit-Reviewer: Michael Meeks 
Gerrit-Reviewer: Petr Mladek 

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


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

2013-03-26 Thread Eike Rathke
 i18npool/source/calendar/calendar_gregorian.cxx |   19 ++-
 1 file changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 9dfdb7955c1f5781340dc688cc0229dea9cb4c2d
Author: Eike Rathke 
Date:   Mon Mar 25 21:21:41 2013 +0100

resolved fdo#44286 some historical dates were decremented

Depending on historical timezone data with odd offsets some historical
dates were decremented by one day due to a rounding error.

Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
(cherry picked from commit 6013fe19a40dd16ce435a2428f7405b51930689e)
Reviewed-on: https://gerrit.libreoffice.org/3039
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 

diff --git a/i18npool/source/calendar/calendar_gregorian.cxx 
b/i18npool/source/calendar/calendar_gregorian.cxx
index e76c431..fc7dcf6 100644
--- a/i18npool/source/calendar/calendar_gregorian.cxx
+++ b/i18npool/source/calendar/calendar_gregorian.cxx
@@ -35,6 +35,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -292,8 +293,19 @@ Calendar_gregorian::getUniqueID() throw(RuntimeException)
 void SAL_CALL
 Calendar_gregorian::setDateTime( double timeInDays ) throw(RuntimeException)
 {
+// ICU handles dates in milliseconds as double values and uses floor()
+// to obtain integer values, which may yield a date decremented by one
+// for odd (historical) timezone values where the computed value due to
+// rounding errors has a fractional part in milliseconds. Ensure we
+// pass a value without fraction here. If not, that may lead to
+// fdo#44286 or fdo#52619 and the like, e.g. when passing
+// -2136315212000.000244 instead of -2136315212000.00
+double fM = timeInDays * U_MILLIS_PER_DAY;
+double fR = rtl::math::round( fM );
+SAL_INFO_IF( fM != fR, "i18npool",
+"Calendar_gregorian::setDateTime: " << std::fixed << fM << " 
rounded to " << fR);
 UErrorCode status;
-body->setTime(timeInDays * U_MILLIS_PER_DAY, status = U_ZERO_ERROR);
+body->setTime( fR, status = U_ZERO_ERROR);
 if ( !U_SUCCESS(status) ) throw ERROR;
 getValue();
 }
@@ -688,6 +700,11 @@ void Calendar_gregorian::setValue() throw(RuntimeException)
 DUMP_ICU_CAL_MSG(("%s\n","setValue() after Zone/DST glitch 
2nd resubmit"));
 DUMP_I18N_CAL_MSG(("%s\n","setValue() after Zone/DST 
glitch 2nd resubmit"));
 }
+SAL_INFO( "i18npool", "Calendar_gregorian::setValue:" <<
+"  nZone0 " << nZone0 << ", nDST0 " << nDST0 <<
+", nZone1 " << nZone1 << ", nDST1 " << nDST1 <<
+", nZone2 " << nZone2 << ", nDST2 " << nDST2 <<
+", nZone3 " << nZone3 << ", nDST3 " << nDST3);
 }
 }
 #if erDUMP_ICU_CALENDAR || erDUMP_I18N_CALENDAR
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED libreoffice-3-6] resolved fdo#44286 some historical dates were decremented

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3039

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3216b1c043fe5ab9035a558031f1a58efa0c3b3d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Eike Rathke 
Gerrit-Reviewer: Petr Mladek 

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


[Libreoffice-commits] core.git: sc/qa

2013-03-26 Thread Markus Mohrhard
 sc/qa/unit/ucalc.cxx |   29 +
 1 file changed, 29 insertions(+)

New commits:
commit 80996c374b4ff2514b2e952cfd572ae499b481a1
Author: Markus Mohrhard 
Date:   Mon Mar 25 14:51:27 2013 +0100

add inital test case for fdo#62206

Change-Id: I0a88f4cbd70f5acf84b9c1c6704ac0be92b79c5b

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index f72bdd0..ec07695 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -68,6 +68,8 @@
 #include "interpre.hxx"
 #include "columniterator.hxx"
 #include "types.hxx"
+#include "conditio.hxx"
+#include "globstr.hrc"
 
 #include "formula/IFunctionDescription.hxx"
 
@@ -272,6 +274,7 @@ public:
  * Test formula & formula grouping
  */
 void testFormulaGrouping();
+void testCondFormatINSDEL();
 
 CPPUNIT_TEST_SUITE(Test);
 CPPUNIT_TEST(testCollator);
@@ -336,6 +339,7 @@ public:
 CPPUNIT_TEST(testAnchoredRotatedShape);
 CPPUNIT_TEST(testCellTextWidth);
 CPPUNIT_TEST(testFormulaGrouping);
+CPPUNIT_TEST(testCondFormatINSDEL);
 CPPUNIT_TEST_SUITE_END();
 
 private:
@@ -6274,6 +6278,31 @@ void Test::testFormulaGrouping()
 }
 }
 
+void Test::testCondFormatINSDEL()
+{
+// fdo#62206
+m_pDoc->InsertTab(0, "Test");
+ScConditionalFormatList* pList = m_pDoc->GetCondFormList(0);
+
+ScConditionalFormat* pFormat = new ScConditionalFormat(1, m_pDoc);
+ScRangeList aRangeList(ScRange(0,0,0,0,3,0));
+pFormat->AddRange(aRangeList);
+ScCondFormatEntry* pEntry = new 
ScCondFormatEntry(SC_COND_DIRECT,"=B2","",m_pDoc,ScAddress(0,0,0),ScGlobal::GetRscString(STR_STYLENAME_RESULT));
+pFormat->AddEntry(pEntry);
+
+m_pDoc->AddCondFormatData(pFormat->GetRange(), 0, 1);
+pList->InsertNew(pFormat);
+
+m_pDoc->InsertCol(0,0,MAXROW,0,0,2);
+const ScRangeList& rRange = pFormat->GetRange();
+CPPUNIT_ASSERT(rRange == ScRange(2,0,0,2,3,0));
+
+OUString aExpr = pEntry->GetExpression(ScAddress(2,0,0), 0);
+CPPUNIT_ASSERT_EQUAL(aExpr, OUString("D2"));
+
+m_pDoc->DeleteTab(0);
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] libcdr.git: src/lib

2013-03-26 Thread Fridrich Štrba
 src/lib/CDRContentCollector.h |8 
 src/lib/CDRParser.cpp |5 +++--
 2 files changed, 7 insertions(+), 6 deletions(-)

New commits:
commit 4144e6e95a6ff2a73b5c94af87eb51a25c56e103
Author: Fridrich Å trba 
Date:   Tue Mar 26 09:02:55 2013 +0100

Some cosmetical fixes

diff --git a/src/lib/CDRContentCollector.h b/src/lib/CDRContentCollector.h
index ba0cd28..0f8019c 100644
--- a/src/lib/CDRContentCollector.h
+++ b/src/lib/CDRContentCollector.h
@@ -73,8 +73,8 @@ public:
   void collectPageSize(double, double, double, double) {}
   void collectPolygonTransform(unsigned numAngles, unsigned nextPoint, double 
rx, double ry, double cx, double cy);
   void collectBitmap(unsigned imageId, double x1, double x2, double y1, double 
y2);
-  void collectBmp(unsigned, unsigned, unsigned, unsigned, unsigned, const 
std::vector&, const std::vector&) {}
-  void collectBmp(unsigned, const std::vector&) {}
+  void collectBmp(unsigned, unsigned, unsigned, unsigned, unsigned, const 
std::vector &, const std::vector &) {}
+  void collectBmp(unsigned, const std::vector &) {}
   void collectBmpf(unsigned, unsigned, unsigned, const std::vector &) {}
   void collectPpdt(const std::vector > &points, 
const std::vector &knotVector);
   void collectFillTransform(const CDRTransforms &fillTrafo);
@@ -87,8 +87,8 @@ public:
   void collectVectorPattern(unsigned id, const WPXBinaryData &data);
   void collectPaletteEntry(unsigned, unsigned, const CDRColor &) {}
   void collectFont(unsigned, unsigned short, const WPXString &) {}
-  void collectText(unsigned, unsigned, const std::vector&,
-   const std::vector&, const std::map&) {}
+  void collectText(unsigned, unsigned, const std::vector &,
+   const std::vector &, const std::map &) {}
   void collectArtisticText(double x, double y);
   void collectParagraphText(double x, double y, double width, double height);
   void collectStlt(const std::map &) {}
diff --git a/src/lib/CDRParser.cpp b/src/lib/CDRParser.cpp
index 52cea0b..6cc31de 100644
--- a/src/lib/CDRParser.cpp
+++ b/src/lib/CDRParser.cpp
@@ -115,7 +115,7 @@ static void processNameForEncoding(WPXString &name, 
unsigned short &encoding)
   else if (length >= 4 && (found=fontName.find("GOST", 0, 4)) != 
std::string::npos)
   {
 encoding = 0xcc;
-   found = std::string::npos;
+found = std::string::npos;
   }
 
   if (found != std::string::npos)
@@ -2386,7 +2386,8 @@ void libcdr::CDRParser::readFont(WPXInputStream *input, 
unsigned length)
 break;
 }
   }
-  processNameForEncoding(name, fontEncoding);
+  if (!fontEncoding)
+processNameForEncoding(name, fontEncoding);
   m_collector->collectFont(fontId, fontEncoding, name);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sfx2/source svl/inc

2013-03-26 Thread Michael Stahl
 sfx2/source/dialog/templdlg.cxx |   54 
 sfx2/source/inc/templdgi.hxx|2 +
 svl/inc/svl/style.hrc   |1 
 3 files changed, 42 insertions(+), 15 deletions(-)

New commits:
commit 44dd8b838b8dfa74b895a870a68200cf4f3574e0
Author: Michael Stahl 
Date:   Tue Mar 26 11:33:09 2013 +0100

fdo#61429: Stylist: save selection of "Hierarchical" entry

This is unfortunately rather ugly because that one is "very special".

Change-Id: Ie2bc9ce2690fbc742b4510e453ca1ef310614256

diff --git a/sfx2/source/dialog/templdlg.cxx b/sfx2/source/dialog/templdlg.cxx
index d6d1af3..1b98850 100644
--- a/sfx2/source/dialog/templdlg.cxx
+++ b/sfx2/source/dialog/templdlg.cxx
@@ -744,6 +744,7 @@ SfxCommonTemplateDialog_Impl::SfxCommonTemplateDialog_Impl( 
SfxBindings* pB, Sfx
 bUpdateByExampleDisabled( sal_False ),
 bTreeDrag   ( sal_True ),
 bHierarchical   ( sal_False ),
+m_bWantHierarchical ( sal_False ),
 bBindingUpdate  ( sal_True )
 {
 aFmtLb.SetAccessibleName(SfxResId(STR_STYLE_ELEMTLIST).toString());
@@ -1232,8 +1233,10 @@ void 
SfxCommonTemplateDialog_Impl::UpdateStyles_Impl(sal_uInt16 nFlags)
 }
 
 // if the tree view again, select family hierarchy
-if(pTreeBox)
+if (pTreeBox || m_bWantHierarchical)
+{
 
aFilterLb.SelectEntry(SfxResId(STR_STYLE_FILTER_HIERARCHICAL).toString());
+}
 
 // show maximum 14 entries
 aFilterLb.SetDropDownLineCount( MAX_FILTER_ENTRIES );
@@ -1241,12 +1244,21 @@ void 
SfxCommonTemplateDialog_Impl::UpdateStyles_Impl(sal_uInt16 nFlags)
 }
 else
 {
-if( nActFilter < aFilterLb.GetEntryCount() - 1)
-aFilterLb.SelectEntryPos(nActFilter + 1);
-else
+if (m_bWantHierarchical)
 {
 nActFilter = 0;
-aFilterLb.SelectEntryPos(1);
+aFilterLb.SelectEntry(SfxResId(
+STR_STYLE_FILTER_HIERARCHICAL).toString());
+}
+else
+{
+if (nActFilter < aFilterLb.GetEntryCount() - 1)
+aFilterLb.SelectEntryPos(nActFilter + 1);
+else
+{
+nActFilter = 0;
+aFilterLb.SelectEntryPos(1);
+}
 }
 }
 
@@ -1590,7 +1602,6 @@ void SfxCommonTemplateDialog_Impl::Notify(SfxBroadcaster& 
/*rBC*/, const SfxHint
 
 
 //-
-
 // Other filters; can be switched by the users or as a result of new or
 // editing, if the current document has been assigned a different filter.
 void SfxCommonTemplateDialog_Impl::FilterSelect(
@@ -1601,14 +1612,7 @@ void SfxCommonTemplateDialog_Impl::FilterSelect(
 if( nEntry != nActFilter || bForce )
 {
 nActFilter = nEntry;
-SfxViewFrame *pViewFrame = pBindings->GetDispatcher_Impl()->GetFrame();
-SfxObjectShell *pDocShell = pViewFrame->GetObjectShell();
-if (pDocShell)
-{
-pDocShell->SetAutoStyleFilterIndex(nActFilter);
-SaveFactoryStyleFilter( pDocShell, nActFilter );
-}
-
+SfxObjectShell *const pDocShell = SaveSelection();
 SfxStyleSheetBasePool *pOldStyleSheetPool = pStyleSheetPool;
 pStyleSheetPool = pDocShell? pDocShell->GetStyleSheetPool(): 0;
 if ( pOldStyleSheetPool != pStyleSheetPool )
@@ -1700,6 +1704,8 @@ IMPL_LINK( SfxCommonTemplateDialog_Impl, FilterSelectHdl, 
ListBox *, pBox )
 {
 // Turn on treeView
 bHierarchical=sal_True;
+m_bWantHierarchical = sal_True;
+SaveSelection(); // fdo#61429 store "hierarchical"
 const String aSelectEntry( GetSelectedEntry());
 aFmtLb.Hide();
 
@@ -1732,6 +1738,7 @@ IMPL_LINK( SfxCommonTemplateDialog_Impl, FilterSelectHdl, 
ListBox *, pBox )
 aFmtLb.Show();
 // If bHierarchical, then the family can have changed
 // minus one since hierarchical is inserted at the start
+m_bWantHierarchical = sal_False; // before FilterSelect
 FilterSelect(pBox->GetSelectEntryPos() - 1, bHierarchical );
 bHierarchical=sal_False;
 }
@@ -1874,6 +1881,10 @@ sal_Int32 
SfxCommonTemplateDialog_Impl::LoadFactoryStyleFilter( SfxObjectShell*
 sal_Int32 nDefault = -1;
 nFilter = aFactoryProps.getUnpackedValueOrDefault( 
"ooSetupFactoryStyleFilter", nDefault );
 
+m_bWantHierarchical =
+(nFilter & SFXSTYLEBIT_HIERARCHY) ? sal_True : sal_False;
+nFilter &= ~SFXSTYLEBIT_HIERARCHY; // clear it
+
 return nFilter;
 }
 
@@ -1884,10 +1895,23 @@ void 
SfxCommonTemplateDialog_Impl::SaveFactoryStyleFilter( SfxObjectShell* i_pOb
 OSL_ENSURE( i_pObjSh, 
"SfxCommonTemplateDialog_Impl::LoadFactoryStyleFil

Re: [PATCH] fdo#51296:UI: Calc doesn't take care of security setting "Ctrl-click required to follow hyperlinks

2013-03-26 Thread Akash Shetye
Ooops sorry for that Kohei. I'll keep that in mind always.

On Mon, Mar 25, 2013 at 5:56 PM, Akash Shetye  wrote:
> > Hey,
> >
> > Petr thank you for your suggestions, I happened to overlook the
> "Ctrl-click
> > to open hyperlink" prefix string to the tooltips in excitement oh having
> a
> > patch :P. Now that I have taken a look at sc/ code, I find that the
> prefixed
> > string is not hardcoded into the sw/ code. Also how sc/ and sw/ handle
> > hyperlinks is very differently. I tried probing into the vcl part about
> > fixedhyperlink.cxx to trace things back up from there but things aren't
> > working well. Just knowing which part of code is doing the tooltip
> content
> > setting in sc/ will be cool enough. Thanks for cc'ing the experts.
> >
> > And kohei thank you as well, I will take care of the issues you have
> pointed
> > about in the other thread. Till then I am combing the code.
> >
> > Akash.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[License] License statement for Libreoffice

2013-03-26 Thread vincent
All of my past & future contributions to LibreOffice may be
licensed under the MPL/LGPLv3+ dual license.___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] dictionaries.git: Changes to 'refs/tags/libreoffice-4.0.2.2'

2013-03-26 Thread Petr Mladek
Tag 'libreoffice-4.0.2.2' created by Petr Mladek  at 
2013-03-26 12:02 -0700

Tag libreoffice-4.0.2.2
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.9 (GNU/Linux)

iQIcBAABAgAGBQJRUYBTAAoJEPQ0oe+v7q6j8/AP/jfTExh//xtJofV7TQquHP68
c5/gqXkxetN27lp+jAhsxrcO42h/vY1PneNfEhglkB9NMgKGva+ZSughRvXEGG1L
K3hLj409yp+cVE3PDD4ZSk7WDmmCWvCruwlQSWxBxb/wZ0UwSIoseoLVCf5BFNRa
SZCeTs2XD4Z19Qe1bT2XuuQw3KhFHV817eoDpmY9jxLD4eaP8zrlO42aRF+TQIfI
SzdAtt1jgC/fVLWipyQox3u8+2lfe3f7dE7Y/7934eTztX8qV1LYelJEmLwZGCJq
XHU/bRUjB3g0bLYgGlKZv5uKPWEv3llN96Zl3fNq64K0uTXWmCPFFCOKBXPfjtBD
h/HXHNNUDS5+SZ+Db/atihOZq8nJJChwuYcBgjggE2iCOh57jq+dWezgBxTaCTMm
EwwOoOD/XNEAwD7Q2yON/rNK6rzwryEcXVxgNYrOXPy4bA6papAIy3NreXL5kfxF
A8Hva10gTMB2yNaCkHuE6lQpzrZU4y0CudXTUQCu6qoXTvSH/cm/+2GKkivsQiIw
aMEGe6L4aHQbFaczq9DNkU+DacGqesQM3OsJFL8cMgXCQSqH13IzZ8K0ZRFtSFUA
XOmzU19L/5ZNYTRiDclc+N3lJ0+yjTKfS55+w6JrDbjoVje/YlRnOWNOKSrPaPOo
rs0bof9H7JfHkapDUNat
=SPiY
-END PGP SIGNATURE-

Changes since sdremote-1.0.2-3:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] help.git: Changes to 'refs/tags/libreoffice-4.0.2.2'

2013-03-26 Thread Petr Mladek
Tag 'libreoffice-4.0.2.2' created by Petr Mladek  at 
2013-03-26 12:02 -0700

Tag libreoffice-4.0.2.2
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.9 (GNU/Linux)

iQIcBAABAgAGBQJRUYBTAAoJEPQ0oe+v7q6jJYIP/jyu5cNAr2Qhi36fnMsG8jET
Tqdxltz7QyIdQ5ypWPs52Q0OVYTuyI0WDuDYKX37+1/csWrHWVESosjkrp2+ywgx
MhTGxW0DLp4A+ZDsUZfr9dXP8fG9FhNIV5BLwLFGq685I1YSSTKoa0PMpXBLKDL5
kCIE8mRBqK7kyqkEMjJ1LZ0NqwR9dMahf9ul37M7nC2cTV4ClDlICpcSHQSgdeQi
mrQrWMfir8PVVfQIiWO03CIhcHsiW2mBT6IAQAzrhrfcgJbZGA2NChR47MOzDR8K
/bPBQifYy7E404yowZ08suX1nMvJtlO7Rx92EH/6kFaP7NdEtWKYoIeVJtLflLyQ
2stnC64gNBVtchjVXykZrf5pL1OIsSMrSTp1Tce/X5Fu52UUqvHTJYWxOVkbfAfT
FCcn90Vq2DIDws6nU1usf4QTrH3X6I6SRpG7B6qta94x5u/iGyBBdjONJZAAI6as
/AgHArLXSFi+sWdtQ9sRK2J44VJB/zZbO9h+dSGQt8RvR1gqRTfTx7iKikvq564e
l74pAHTV2lYt8I+k1YSIys2cIHlvnMisl0wo8gai8JOTVDvaUmKafZsgf2OGBgpy
mCgGXQ7MLZGZQDIxglwadAtctBxqvM2avugg+Aj+uDFYYYHyJ1MoJnBaK/3FGmlU
pPA7KZXHZK1h7HU0ngPu
=rmbx
-END PGP SIGNATURE-

Changes since sdremote-1.0.2-1:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] translations.git: Changes to 'refs/tags/libreoffice-4.0.2.2'

2013-03-26 Thread Petr Mladek
Tag 'libreoffice-4.0.2.2' created by Petr Mladek  at 
2013-03-26 12:02 -0700

Tag libreoffice-4.0.2.2
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.9 (GNU/Linux)

iQIcBAABAgAGBQJRUYBTAAoJEPQ0oe+v7q6jntoQALFMHcFKFMD/Ya7bHLG3frcm
am4gQBz+Wve4RB4ocs40rri1axIwcijcXWh6BbBvKw3qFYRgngrNAs/q5MjIRQlD
+FPDW4l5Kqy8aOHphEwn9qkYURi4PW6Sv+IHntxeM2E27PRk2QCyWMHlv+4FEizI
u8pyeUGmq/nGRZ05NG+4gmdRVQnw+bvJ6QlXwK/tLsTl12COVuwKxXYX6q3TKLdZ
y4CqQzTrTxH2Sgu67wXnh68MkYA8QsjU26KPvn7653JEosZGXIvJ7LpqI6OqQjkH
tmTyY3D7B6C7fvQ4Flm0zsAaYLcGSBEuozU6m7llnYTh7gdoao6HMgNBbX9bduIi
uEdn64bF+fsjxafs6/qYLx7xovVd0ADuxkhUcQ584TRkIYBOkTB3DcEOlLk0o5mq
Yg3ILBV9P4vNI4DbCHhbSNkOHJlTQauMubzDJhxRm7wxkDsL8SmMGqshOyx9ECjk
f50uOCYXrWWg2PSnMf/SzZHRh7pFMBbwXLS1jkcv3Ls7MLBMzMFyz49H6VKBKvMP
jFPXjlx9FYeplkJHKsf/idFNaX7qNaz2GYTmydEGCGKg51rBedN3sottoan0jsqU
iooDoF/RM9osPhztvd3Z6x+vjEkBCrw9iwjvxxH4DWGyqvG1UuxdcE4BiZtMUXWN
HoYrcrEdciS2A8LIBWhp
=X3+A
-END PGP SIGNATURE-

Changes since sdremote-1.0.2-6:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/tags/libreoffice-4.0.2.2'

2013-03-26 Thread Petr Mladek
Tag 'libreoffice-4.0.2.2' created by Petr Mladek  at 
2013-03-26 12:02 -0700

Tag libreoffice-4.0.2.2
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.9 (GNU/Linux)

iQIcBAABAgAGBQJRUYBTAAoJEPQ0oe+v7q6j5Q0QALT6U49eSZXVhhMu6JSF1RPx
G8hj2GY5cb5TNZcVnFinoYxboX+Tu21iCdlQXeXvt01cUirTeKYFQ/LQO8nby6+P
wjvVkbnk3MvtyIB0c400h2XGwVGJZGIaZlAYjJVD56b6K44CDjyGU802hnfpkWR4
ivLxt3AUETbcBHXVu/9j0jRH9grATDJsxyOlC2nHGtDn7FHuKPbjCsbBXNEN0XyL
hPc76G9mdxuJNW/U6vR/yZqpyNUhujJK26EFyGRJVMUvfTYmM4W/qGlvxpESvXJi
QCRWU1cG30XlK8rR1BD2E9PXKxXDA/dEIOxh2O8grkIIG2RpZwTorYHqTDRm1WUY
uSqMxRxO99qaPCmH8N0xhgWQK2dd7cC0iTwYMFvxCNtTvKAPKnRWTCTqdrCYeTPE
u6K2BMzMcD4K9gYV76mOLlwi8ld0nlDEnf4SaE69xzZRIr4dYITn81vKYI0lPDmN
MY4oa2C0YXnQfVt0X7lkTacrwavxVuitDBr+0FsacM6c6v6ugam1+aReFykFBv6s
SIGFVMF9zvyfiNMRzNOlh/aWAHGTOEp33FCFdXysAexgYNacxi7CjXJLtT31jetj
zonkhfwWXTt8g6qRh12azQDeZ0MCv4W9SC0CSZCNur6Pv4sKoNQAYUDcqqr7LWvk
nT5/ZNKo/x0DVE1EZhII
=FPqm
-END PGP SIGNATURE-

Changes since sdremote-1.0.0-270:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Deselect thumbnailview items when clearing the view.

2013-03-26 Thread Rafael Dominguez (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3052

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/52/3052/1

Deselect thumbnailview items when clearing the view.

Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 10 insertions(+), 0 deletions(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 43bab23..2ba5ea6 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -114,6 +114,16 @@
 for ( size_t i = 0; i < n; ++i )
 {
 ThumbnailViewItem *const pItem = mItemList[i];
+
+// deselect all current selected items and fire events
+if (pItem->isSelected())
+{
+pItem->setSelection(false);
+maItemStateHdl.Call(pItem);
+
+// fire accessible event???
+}
+
 if ( pItem->isVisible() && ImplHasAccessibleListeners() )
 {
 ::com::sun::star::uno::Any aOldAny, aNewAny;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 

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


[PATCH] Fix template manager item selection behaviour with mouse.

2013-03-26 Thread Rafael Dominguez (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3053

To pull it, you can do:

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

Fix template manager item selection behaviour with mouse.

Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
---
M sfx2/source/control/thumbnailview.cxx
1 file changed, 14 insertions(+), 9 deletions(-)



diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 2ba5ea6..62d4670 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -605,15 +605,22 @@
 {
 if ( rMEvt.GetClicks() == 1 )
 {
-if (pItem->isSelected() && rMEvt.IsMod1())
-pItem->setSelection(false);
+if (rMEvt.IsMod1())
+{
+//Keep selected item group state and just invert current 
desired one state
+pItem->setSelection(!pItem->isSelected());
+}
 else
 {
-if (!pItem->isSelected() && !rMEvt.IsMod1())
-deselectItems( );
-
+//If we got a group of selected items deselect the rest 
and only keep the desired one
+//mark items as not selected to not fire unnecessary 
change state events.
+pItem->setSelection(false);
+deselectItems();
 pItem->setSelection(true);
+}
 
+if (pItem->isSelected())
+{
 bool bClickOnTitle = 
pItem->getTextArea().IsInside(rMEvt.GetPosPixel());
 pItem->setEditTitle(bClickOnTitle);
 }
@@ -622,13 +629,11 @@
 DrawItem(pItem);
 
 maItemStateHdl.Call(pItem);
+
+//fire accessible event??
 }
 else if ( rMEvt.GetClicks() == 2 )
 {
-// The mouse button down event 1 click right before is 
pointless
-pItem->setSelection(false);
-maItemStateHdl.Call(pItem);
-
 Rectangle aRect(pItem->getDrawArea());
 
 if (aRect.IsInside(rMEvt.GetPosPixel()))

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 

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


[PATCH] Syncronize thumbnails ids with SfxDocumentTemplates when mov...

2013-03-26 Thread Rafael Dominguez (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3054

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/54/3054/1

Syncronize thumbnails ids with SfxDocumentTemplates when moving.

Change-Id: I9ee17e00f769ca16f566b1397d09b76a09273912
---
M sfx2/inc/sfx2/templatelocalview.hxx
M sfx2/source/control/templatelocalview.cxx
M sfx2/source/doc/templatedlg.cxx
3 files changed, 62 insertions(+), 47 deletions(-)



diff --git a/sfx2/inc/sfx2/templatelocalview.hxx 
b/sfx2/inc/sfx2/templatelocalview.hxx
index cf12ddc..7ea226c 100644
--- a/sfx2/inc/sfx2/templatelocalview.hxx
+++ b/sfx2/inc/sfx2/templatelocalview.hxx
@@ -56,9 +56,9 @@
 bool removeTemplate (const sal_uInt16 nItemId, const sal_uInt16 
nSrcItemId);
 
 bool moveTemplate (const ThumbnailViewItem* pItem, const sal_uInt16 
nSrcItem,
-   const sal_uInt16 nTargetItem, bool bCopy);
+   const sal_uInt16 nTargetItem);
 
-bool moveTemplates (const std::set &rItems, const sal_uInt16 nTargetItem, 
bool bCopy);
+bool moveTemplates (const std::set &rItems, const sal_uInt16 nTargetItem);
 
 bool copyFrom (const sal_uInt16 nRegionItemId, const BitmapEx &rThumbnail, 
const OUString &rPath);
 
diff --git a/sfx2/source/control/templatelocalview.cxx 
b/sfx2/source/control/templatelocalview.cxx
index 9bb94c3..ddab413 100644
--- a/sfx2/source/control/templatelocalview.cxx
+++ b/sfx2/source/control/templatelocalview.cxx
@@ -289,11 +289,8 @@
 }
 
 bool TemplateLocalView::moveTemplate (const ThumbnailViewItem *pItem, const 
sal_uInt16 nSrcItem,
-   const sal_uInt16 nTargetItem, bool 
bCopy)
+   const sal_uInt16 nTargetItem)
 {
-bool bRet = true;
-bool bRefresh = false;
-
 TemplateContainerItem *pTarget = NULL;
 TemplateContainerItem *pSrc = NULL;
 
@@ -313,16 +310,13 @@
 
 const TemplateViewItem *pViewItem = static_cast(pItem);
 
-bool bOK;
+bool bCopy = 
!mpDocTemplates->Move(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnDocId);
 
 if (bCopy)
-bOK = 
mpDocTemplates->Copy(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnId-1);
-else
-bOK = 
mpDocTemplates->Move(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnId-1);
-
-if (!bOK)
-return false;
-
+{
+if 
(!mpDocTemplates->Copy(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnDocId))
+return false;
+}
 // move template to destination
 
 TemplateItemProperties aTemplateItem;
@@ -340,37 +334,45 @@
 // remove template from region cached data
 
 std::vector::iterator aIter;
-for (aIter = pSrc->maTemplates.begin(); aIter != 
pSrc->maTemplates.end(); ++aIter)
+for (aIter = pSrc->maTemplates.begin(); aIter != 
pSrc->maTemplates.end();)
 {
-if (aIter->nId == pViewItem->mnId)
+if (aIter->nDocId == pViewItem->mnDocId)
 {
-pSrc->maTemplates.erase(aIter);
-
-RemoveItem(pViewItem->mnId);
-break;
+aIter = pSrc->maTemplates.erase(aIter);
 }
+else
+{
+// Keep region document id syncronized with 
SfxDocumentTemplates
+if (aIter->nDocId > pViewItem->mnDocId)
+--aIter->nDocId;
+
+++aIter;
+}
+}
+
+// Keep view document id syncronized with SfxDocumentTemplates
+std::vector::iterator pItemIter = 
mItemList.begin();
+for (; pItemIter != mItemList.end(); ++pItemIter)
+{
+if (static_cast(*pItemIter)->mnDocId > 
pViewItem->mnDocId)
+--static_cast(*pItemIter)->mnDocId;
 }
 }
 
-bRefresh = true;
-}
-else
-bRet = false;
-
-if (bRefresh)
-{
 lcl_updateThumbnails(pSrc);
 lcl_updateThumbnails(pTarget);
 
 CalculateItemPositions();
 Invalidate();
+
+return true;
 }
 
-return bRet;
+return false;
 }
 
 bool TemplateLocalView::moveTemplates(const std::set &rItems,
-  const sal_uInt16 nTargetItem, bool bCopy)
+  const sal_uInt16 nTargetItem)
 {
 bool ret = true;
 bool refresh = false;
@@ -400,17 +402,15 @@
 {
 const TemplateViewItem *pViewItem = static_cast(*aSelIter);
 
-bool bOK;
+bool bCopy = 
!mpDocTemplates->Move(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnDocId);
 
 if (bCopy)
-bOK = 
mpDocTemplates->Copy(nTargetRegion,nTargetIdx,nSrcRegionId,pViewItem->mnId-1);
-else
-bOK = 
mpDocTempl

[Libreoffice-commits] core.git: sc/source

2013-03-26 Thread Stephan Bergmann
 sc/source/core/tool/chgtrack.cxx |6 --
 1 file changed, 6 deletions(-)

New commits:
commit 6e33a2bdadefb8b4d3f3d49ad0d843bde60a4623
Author: Stephan Bergmann 
Date:   Tue Mar 26 12:30:53 2013 +0100

warning C4702: unreachable code

Change-Id: I711c7b1968a3ef8ff88ab71b4de837b599519b8a

diff --git a/sc/source/core/tool/chgtrack.cxx b/sc/source/core/tool/chgtrack.cxx
index 8912ffa..b1b683c 100644
--- a/sc/source/core/tool/chgtrack.cxx
+++ b/sc/source/core/tool/chgtrack.cxx
@@ -1733,27 +1733,21 @@ ScChangeActionContentCellType 
ScChangeActionContent::GetContentCellType( const S
 case CELLTYPE_STRING :
 case CELLTYPE_EDIT :
 return SC_CACCT_NORMAL;
-//break;
 case CELLTYPE_FORMULA :
 switch (rCell.mpFormula->GetMatrixFlag())
 {
 case MM_NONE :
 return SC_CACCT_NORMAL;
-//break;
 case MM_FORMULA :
 case MM_FAKE :
 return SC_CACCT_MATORG;
-//break;
 case MM_REFERENCE :
 return SC_CACCT_MATREF;
-//break;
 }
 return SC_CACCT_NORMAL;
-//break;
 default:
 return SC_CACCT_NONE;
 }
-return SC_CACCT_NONE;
 }
 
 ScChangeActionContentCellType ScChangeActionContent::GetContentCellType( const 
ScCellIterator& rIter )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: starmath/source

2013-03-26 Thread Luboš Luňák
 starmath/source/ooxmlimport.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 269b72069746578dd0d89de6b44adb715c39173b
Author: Luboš Luňák 
Date:   Tue Mar 26 12:41:16 2013 +0100

fix iint/iiint import from docx math

Use the correct LO names for the symbols.

Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29

diff --git a/starmath/source/ooxmlimport.cxx b/starmath/source/ooxmlimport.cxx
index 0b1beda..e3a48ca 100644
--- a/starmath/source/ooxmlimport.cxx
+++ b/starmath/source/ooxmlimport.cxx
@@ -530,10 +530,10 @@ OUString SmOoxmlImport::handleNary()
 ret = "int";
 break;
 case MS_IINT:
-ret = "liint";
+ret = "iint";
 break;
 case MS_IIINT:
-ret = "liiint";
+ret = "iiint";
 break;
 case MS_LINT:
 ret = "lint";
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[ABANDONED] fix iint/iiint import from docx math

2013-03-26 Thread via Code Review
Luboš Luňák has abandoned this change.

Change subject: fix iint/iiint import from docx math
..


Patch Set 1: Abandoned

Wrong branch.

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Luboš Luňák 

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


[PATCH] fix iint/iiint import from docx math

2013-03-26 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3055

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/55/3055/1

fix iint/iiint import from docx math

Use the correct LO names for the symbols.

Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29
---
M starmath/source/ooxmlimport.cxx
1 file changed, 2 insertions(+), 2 deletions(-)



diff --git a/starmath/source/ooxmlimport.cxx b/starmath/source/ooxmlimport.cxx
index 0b1beda..e3a48ca 100644
--- a/starmath/source/ooxmlimport.cxx
+++ b/starmath/source/ooxmlimport.cxx
@@ -530,10 +530,10 @@
 ret = "int";
 break;
 case MS_IINT:
-ret = "liint";
+ret = "iint";
 break;
 case MS_IIINT:
-ret = "liiint";
+ret = "iiint";
 break;
 case MS_LINT:
 ret = "lint";

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Luboš Luňák 

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


[ANNOUNCE] libreoffice-4.0.2.2 tag created

2013-03-26 Thread Petr Mladek
Hi,

there have been created the tag libreoffice-4.0.2.2, aka rc2. The
corresponding official builds will be available within next few days.
It will be used as final if no blocker is found.

See the attached list of changes against 4.0.2.1.

Now, you might switch your current 4-0 source tree to it using:

./g fetch --tags
./g checkout -b tag-libreoffice-4.0.2.2 libreoffice-4.0.2.2

Linux distro packages might find source tarballs at
http://dev-builds.libreoffice.org/pre-releases/src/
They will be available from the official page together with the builds.


See also the schedule at 
http://wiki.documentfoundation.org/ReleasePlan#4.0_release
and release criteria at http://wiki.documentfoundation.org/Release_Criteria


Best Regards,
Petr
deb#703486 keep migrating more user extensions after one failed [Stephan Bergmann]
fdo#44286 [EDITING] Calc decreases one day of a date typed (Brazilian-portuguese locale and timezone) [Eike Rathke]
fdo#52159 EDITING: Error bars values become Y-values when copying charts from calc to writer [Markus Mohrhard]
fdo#54275 Linux: Unable to "Save" (or "Save As") New Document on Samba (gvfs) Mounted Share [Stephan Bergmann]
fdo#57905 UI: Autocorrect Options can not be closed using 'Close-X' at the top right corner of dialog [Joren De Cuyper]
fdo#60462 Small errors in the layout margins of the text boxes [Joren De Cuyper]
fdo#60752 Wrong result because 'Property Let' fails to execute [Noel Power]
fdo#60769 FILEOPEN, FILESAVE, "Could not save document, error in writing sub document context.xml" then corrupt file. [Miklos Vajna]
fdo#61256 Saving particular odg file resets font formatting [Michael Meeks]
fdo#61952 FORMATTING: crash when switching between left and right page [Michael Stahl]
fdo#62111 FILESAVE Libreoffice hangs when exporting a particular .ods to .xlsx [Noel Power]
fdo#62299 Libvisio: Visio document cannot be opend by Draw [Fridrich Štrba]
fdo#62384 Turkish Lira sign should precede numbers, not follow [Andras Timar]
fdo#62617 displaying of object groups having subobjects on different layers is broken [David Tardon]
fdo#62670 TABLE: Splitting a table options are mixed [Joren De Cuyper]
rhbz#895690 Libreoffice causing errors when SAVING files by gvfs-mount (Samba) [Stephan Bergmann]
rhbz#906137 Slide show mode inverts outputs [Caolán McNamara]
rhbz#918938 Crash in Impress when right-clicking selected word [Caolán McNamara]
rhbz#919020 Converting String to Double [Eike Rathke]
* {{deb|703486}} keep migrating more user extensions after one failed [Stephan 
Bergmann]
* {{fdo|44286}} [EDITING] Calc decreases one day of a date typed 
(Brazilian-portuguese locale and timezone) [Eike Rathke]
* {{fdo|52159}} EDITING: Error bars values become Y-values when copying charts 
from calc to writer [Markus Mohrhard]
* {{fdo|54275}} Linux: Unable to "Save" (or "Save As") New Document on Samba 
(gvfs) Mounted Share [Stephan Bergmann]
* {{fdo|57905}} UI: Autocorrect Options can not be closed using 'Close-X' at 
the top right corner of dialog [Joren De Cuyper]
* {{fdo|60462}} Small errors in the layout margins of the text boxes [Joren De 
Cuyper]
* {{fdo|60752}} Wrong result because 'Property Let' fails to execute [Noel 
Power]
* {{fdo|60769}} FILEOPEN, FILESAVE, "Could not save document, error in writing 
sub document context.xml" then corrupt file. [Miklos Vajna]
* {{fdo|61256}} Saving particular odg file resets font formatting [Michael 
Meeks]
* {{fdo|61952}} FORMATTING: crash when switching between left and right page 
[Michael Stahl]
* {{fdo|62111}} FILESAVE Libreoffice hangs when exporting a particular .ods to 
.xlsx [Noel Power]
* {{fdo|62299}} Libvisio: Visio document cannot be opend by Draw [Fridrich 
Štrba]
* {{fdo|62384}} Turkish Lira sign should precede numbers, not follow [Andras 
Timar]
* {{fdo|62617}} displaying of object groups having subobjects on different 
layers is broken [David Tardon]
* {{fdo|62670}} TABLE: Splitting a table options are mixed [Joren De Cuyper]
* {{rhbz|895690}} Libreoffice causing errors when SAVING files by gvfs-mount 
(Samba) [Stephan Bergmann]
* {{rhbz|906137}} Slide show mode inverts outputs [Caolán McNamara]
* {{rhbz|918938}} Crash in Impress when right-clicking selected word [Caolán 
McNamara]
* {{rhbz|919020}} Converting String to Double [Eike Rathke]
+ common
+ version 4.0.2.2, tag libreoffice-4.0.2.2 [Petr Mladek]
+ core
+ -Werror=shadow/-Werror=unused-variable [Stephan Bergmann]
+ basic CDbl() and CSng() scan localized number (rhbz#919020) [Eike Rathke]
+ crash in danish thesaurus/spell interaction (rhbz#918938) [Caolán McNamara]
+ display groups on multiple layers correctly (fdo#62617) [David Tardon]
+ don't fail if the end of characters is reached after a field (fdo#62299) [Fridrich Štrba]
+ error bar range is also part of chart range, related (fdo#52159) [Markus Mohrhard]
+ fix for don't count non-default empty rows as rows to repeat (fdo#62111) [Noel Power]
+ fix GVFS UCP, so saving docs works again (fdo#5

[Libreoffice-commits] core.git: Branch 'libreoffice-4-0-2' - instsetoo_native/util solenv/inc

2013-03-26 Thread Petr Mladek
 instsetoo_native/util/openoffice.lst |   40 +--
 solenv/inc/minor.mk  |4 +--
 2 files changed, 22 insertions(+), 22 deletions(-)

New commits:
commit ea8a0f9631f632a5f886be15ddee982269acea96
Author: Petr Mladek 
Date:   Tue Mar 26 12:54:08 2013 +0100

bump product version to 4.0.2.2+, number to 2

Change-Id: Ia55ce588963e082dbcbde3fb5652242332f3299a

diff --git a/instsetoo_native/util/openoffice.lst 
b/instsetoo_native/util/openoffice.lst
index a7ffcf6..f37fcd1 100644
--- a/instsetoo_native/util/openoffice.lst
+++ b/instsetoo_native/util/openoffice.lst
@@ -4,7 +4,7 @@ Globals
 {
 variables
 {
-UREPACKAGEVERSION 4.0.2.1
+UREPACKAGEVERSION 4.0.2.2
 URELAYERVERSION 1
 REFERENCEOOOMAJORMINOR 3.4
 UNIXBASISROOTNAME libreoffice4.0
@@ -50,12 +50,12 @@ LibreOffice
 {
 PRODUCTNAME LibreOffice
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 POSTVERSIONEXTENSION
 POSTVERSIONEXTENSIONUNIX
 BRANDPACKAGEVERSION 4.0
 USERDIRPRODUCTVERSION 4
-ABOUTBOXPRODUCTVERSION 4.0.2.1
+ABOUTBOXPRODUCTVERSION 4.0.2.2
 ABOUTBOXPRODUCTVERSIONSUFFIX +
 BASEPRODUCTVERSION 4.0
 PCPFILENAME libreoffice.pcp
@@ -65,7 +65,7 @@ LibreOffice
 FILEFORMATNAME OpenOffice.org
 FILEFORMATVERSION 1.0
 WRITERCOMPATIBILITYVERSIONOOO11 OpenOffice.org 1.1
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION {buildid}
 LICENSENAME LGPL
 GLOBALFILEGID gid_File_Lib_Vcl
@@ -100,13 +100,13 @@ LibreOffice_Dev
 {
 PRODUCTNAME LOdev
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 UNIXBASISROOTNAME lodev4.0
 POSTVERSIONEXTENSION
 POSTVERSIONEXTENSIONUNIX
 BRANDPACKAGEVERSION 4.0
 USERDIRPRODUCTVERSION 4
-ABOUTBOXPRODUCTVERSION 4.0.2.1
+ABOUTBOXPRODUCTVERSION 4.0.2.2
 ABOUTBOXPRODUCTVERSIONSUFFIX +
 BASEPRODUCTVERSION 4.0
 DEVELOPMENTPRODUCT 1
@@ -121,7 +121,7 @@ LibreOffice_Dev
 FILEFORMATNAME OpenOffice.org
 FILEFORMATVERSION 1.0
 WRITERCOMPATIBILITYVERSIONOOO11 OpenOffice.org 1.1
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION {buildid}
 LICENSENAME LGPL
 GLOBALFILEGID gid_File_Lib_Vcl
@@ -159,9 +159,9 @@ URE
 {
 PRODUCTNAME URE
 PRODUCTVERSION 4.0
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION 1
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 BRANDPACKAGEVERSION 4.0
 LICENSENAME LGPL
 NOVERSIONINDIRNAME 1
@@ -192,11 +192,11 @@ LibreOffice_SDK
 {
 PRODUCTNAME LibreOffice
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 POSTVERSIONEXTENSION SDK
 POSTVERSIONEXTENSIONUNIX sdk
 BRANDPACKAGEVERSION 4.0
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION {buildid}
 PACK_INSTALLED 1
 DMG_VOLUMEEXTENSION SDK
@@ -231,12 +231,12 @@ LibreOffice_Dev_SDK
 {
 PRODUCTNAME LOdev
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 UNIXBASISROOTNAME lodev4.0
 POSTVERSIONEXTENSION SDK
 POSTVERSIONEXTENSIONUNIX sdk
 BRANDPACKAGEVERSION 4.0
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION {buildid}
 BASISPACKAGEPREFIX lodevbasis
 UREPACKAGEPREFIX lodev
@@ -276,11 +276,11 @@ LibreOffice_Test
 {
 PRODUCTNAME LibreOffice
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 POSTVERSIONEXTENSION TEST
 POSTVERSIONEXTENSIONUNIX test
 BRANDPACKAGEVERSION 4.0
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 PACKAGEREVISION {buildid}
 PACK_INSTALLED 1
 DMG_VOLUMEEXTENSION TEST
@@ -315,12 +315,12 @@ LibreOffice_Dev_Test
 {
 PRODUCTNAME LOdev
 PRODUCTVERSION 4.0
-PRODUCTEXTENSION .2.1
+PRODUCTEXTENSION .2.2
 UNIXBASISROOTNAME lodev4.0
 POSTVERSIONEXTENSION TEST
 POSTVERSIONEXTENSIONUNIX test
 BRANDPACKAGEVERSION 4.0
-PACKAGEVERSION 4.0.2.1
+PACKAGEVERSION 4.0.2.2
 

[PATCH libreoffice-4-0] fdo#61429: Stylist: save selection of "Hierarchical" entry

2013-03-26 Thread Michael Stahl (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3056

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/56/3056/1

fdo#61429: Stylist: save selection of "Hierarchical" entry

This is unfortunately rather ugly because that one is "very special".

(cherry picked from commit 44dd8b838b8dfa74b895a870a68200cf4f3574e0)

Conflicts:
sfx2/source/dialog/templdlg.cxx

Change-Id: Ie2bc9ce2690fbc742b4510e453ca1ef310614256
---
M sfx2/source/dialog/templdlg.cxx
M sfx2/source/inc/templdgi.hxx
M svl/inc/svl/style.hrc
3 files changed, 42 insertions(+), 15 deletions(-)



diff --git a/sfx2/source/dialog/templdlg.cxx b/sfx2/source/dialog/templdlg.cxx
index a81c27d..94e8082 100644
--- a/sfx2/source/dialog/templdlg.cxx
+++ b/sfx2/source/dialog/templdlg.cxx
@@ -753,6 +753,7 @@
 bUpdateByExampleDisabled( sal_False ),
 bTreeDrag   ( sal_True ),
 bHierarchical   ( sal_False ),
+m_bWantHierarchical ( sal_False ),
 bBindingUpdate  ( sal_True )
 {
 aFmtLb.SetAccessibleName(SfxResId(STR_STYLE_ELEMTLIST).toString());
@@ -1278,8 +1279,10 @@
 }
 
 // if the tree view again, select family hierarchy
-if(pTreeBox)
+if (pTreeBox || m_bWantHierarchical)
+{
 
aFilterLb.SelectEntry(SfxResId(STR_STYLE_FILTER_HIERARCHICAL).toString());
+}
 
 // show maximum 14 entries
 aFilterLb.SetDropDownLineCount( MAX_FILTER_ENTRIES );
@@ -1287,12 +1290,21 @@
 }
 else
 {
-if( nActFilter < aFilterLb.GetEntryCount() - 1)
-aFilterLb.SelectEntryPos(nActFilter + 1);
-else
+if (m_bWantHierarchical)
 {
 nActFilter = 0;
-aFilterLb.SelectEntryPos(1);
+aFilterLb.SelectEntry(SfxResId(
+STR_STYLE_FILTER_HIERARCHICAL).toString());
+}
+else
+{
+if (nActFilter < aFilterLb.GetEntryCount() - 1)
+aFilterLb.SelectEntryPos(nActFilter + 1);
+else
+{
+nActFilter = 0;
+aFilterLb.SelectEntryPos(1);
+}
 }
 }
 
@@ -1635,7 +1647,6 @@
 
 
 //-
-
 // Other filters; can be switched by the users or as a result of new or
 // editing, if the current document has been assigned a different filter.
 void SfxCommonTemplateDialog_Impl::FilterSelect(
@@ -1646,14 +1657,7 @@
 if( nEntry != nActFilter || bForce )
 {
 nActFilter = nEntry;
-SfxViewFrame *pViewFrame = pBindings->GetDispatcher_Impl()->GetFrame();
-SfxObjectShell *pDocShell = pViewFrame->GetObjectShell();
-if (pDocShell)
-{
-pDocShell->SetAutoStyleFilterIndex(nActFilter);
-SaveFactoryStyleFilter( pDocShell, nActFilter );
-}
-
+SfxObjectShell *const pDocShell = SaveSelection();
 SfxStyleSheetBasePool *pOldStyleSheetPool = pStyleSheetPool;
 pStyleSheetPool = pDocShell? pDocShell->GetStyleSheetPool(): 0;
 if ( pOldStyleSheetPool != pStyleSheetPool )
@@ -1744,6 +1748,8 @@
 {
 // Turn on treeView
 bHierarchical=sal_True;
+m_bWantHierarchical = sal_True;
+SaveSelection(); // fdo#61429 store "hierarchical"
 const String aSelectEntry( GetSelectedEntry());
 aFmtLb.Hide();
 
@@ -1776,6 +1782,7 @@
 aFmtLb.Show();
 // If bHierarchical, then the family can have changed
 // minus one since hierarchical is inserted at the start
+m_bWantHierarchical = sal_False; // before FilterSelect
 FilterSelect(pBox->GetSelectEntryPos() - 1, bHierarchical );
 bHierarchical=sal_False;
 }
@@ -1918,6 +1925,10 @@
 sal_Int32 nDefault = -1;
 nFilter = aFactoryProps.getUnpackedValueOrDefault( 
DEFINE_CONST_UNICODE("ooSetupFactoryStyleFilter"), nDefault );
 
+m_bWantHierarchical =
+(nFilter & SFXSTYLEBIT_HIERARCHY) ? sal_True : sal_False;
+nFilter &= ~SFXSTYLEBIT_HIERARCHY; // clear it
+
 return nFilter;
 }
 
@@ -1928,10 +1939,23 @@
 OSL_ENSURE( i_pObjSh, 
"SfxCommonTemplateDialog_Impl::LoadFactoryStyleFilter(): no ObjectShell" );
 Sequence< PropertyValue > lProps(1);
 lProps[0].Name = DEFINE_CONST_UNICODE("ooSetupFactoryStyleFilter");
-lProps[0].Value = makeAny( i_nFilter );;
+lProps[0].Value = makeAny(
+i_nFilter | (m_bWantHierarchical ? SFXSTYLEBIT_HIERARCHY : 0));
 xModuleManager->replaceByName( getModuleIdentifier( xModuleManager, 
i_pObjSh ), makeAny( lProps ) );
 }
 
+SfxObjectShell* SfxCommonTemplateDialog_Impl::SaveSelection()
+{
+SfxViewFrame *const 
pViewFrame(pBindings->GetDispatcher_Impl()->GetFrame());
+   

Re: Trouble regarding fdo#42893

2013-03-26 Thread Michael Stahl
On 26/03/13 12:50, Janit Anjaria wrote:
> I have coded the concept patch but i have been getting an error where
> the writer "hangs" and doesnt give an appropriate output.

possibly there is an infinite loop, attach gdb to soffice.bin and then
look at the backtrace, and try to step through the methods until you
locate loop that does not terminate.

https://wiki.documentfoundation.org/Development/How_to_debug
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: unotools/inc unotools/source vcl/source

2013-03-26 Thread Chr . Rossmanith
 unotools/inc/unotools/fontdefs.hxx |4 +++-
 unotools/source/misc/fontcvt.cxx   |   15 +--
 vcl/source/gdi/outdev3.cxx |7 +--
 3 files changed, 17 insertions(+), 9 deletions(-)

New commits:
commit 4e907bc366d8691a33ec0be389f211e22c6e87d3
Author: Chr. Rossmanith 
Date:   Fri Mar 22 18:06:02 2013 +0100

Replace String with OUString in RecodeString

Change-Id: I987f738d1cd7d640a253f3cd31864290b6d763bf
Reviewed-on: https://gerrit.libreoffice.org/2919
Reviewed-by: Eike Rathke 
Reviewed-by: Fridrich Strba 
Tested-by: Fridrich Strba 

diff --git a/unotools/inc/unotools/fontdefs.hxx 
b/unotools/inc/unotools/fontdefs.hxx
index f923464..60c37d4 100644
--- a/unotools/inc/unotools/fontdefs.hxx
+++ b/unotools/inc/unotools/fontdefs.hxx
@@ -22,6 +22,8 @@
 #include 
 #include 
 #include 
+#include 
+#include 
 
 namespace utl {
   class FontSubstConfiguration;
@@ -54,7 +56,7 @@ public:
 const char* mpSubsFontName;
 sal_Unicode (*mpCvtFunc)( sal_Unicode );
 sal_Unicode RecodeChar( sal_Unicode c ) const;
-voidRecodeString( String& rStra, xub_StrLen nIndex, 
xub_StrLen nLen ) const;
+voidRecodeString( OUString& rStra, sal_Int32 nIndex, 
sal_Int32 nLen ) const;
 static const ConvertChar* GetRecodeData( const String& rOrgFontName, const 
String& rMapFontName );
 };
 
diff --git a/unotools/source/misc/fontcvt.cxx b/unotools/source/misc/fontcvt.cxx
index fe870ed..99e07ee 100644
--- a/unotools/source/misc/fontcvt.cxx
+++ b/unotools/source/misc/fontcvt.cxx
@@ -1372,15 +1372,17 @@ sal_Unicode ConvertChar::RecodeChar( sal_Unicode cChar 
) const
 
 // recode the string assuming the character codes are symbol codes
 // from an traditional symbol font (i.e. U+F020..U+F0FF)
-void ConvertChar::RecodeString( String& rStr, xub_StrLen nIndex, xub_StrLen 
nLen ) const
+void ConvertChar::RecodeString( OUString& rStr, sal_Int32 nIndex, sal_Int32 
nLen ) const
 {
-sal_uLong nLastIndex = (sal_uLong)nIndex + nLen;
-if( nLastIndex > rStr.Len() )
-nLastIndex = rStr.Len();
+sal_Int32 nLastIndex = nIndex + nLen;
+OUStringBufferaTmpStr(rStr);
+
+if( nLastIndex > aTmpStr.getLength() )
+nLastIndex = aTmpStr.getLength();
 
 for(; nIndex < nLastIndex; ++nIndex )
 {
-sal_Unicode cOrig = rStr.GetChar( nIndex );
+sal_Unicode cOrig = rStr[ nIndex ];
 // only recode symbols and their U+00xx aliases
 if( ((cOrig < 0x0020) || (cOrig > 0x00FF))
 &&  ((cOrig < 0xF020) || (cOrig > 0xF0FF)) )
@@ -1389,8 +1391,9 @@ void ConvertChar::RecodeString( String& rStr, xub_StrLen 
nIndex, xub_StrLen nLen
 // recode a symbol
 sal_Unicode cNew = RecodeChar( cOrig );
 if( cOrig != cNew )
-rStr.SetChar( nIndex, cNew );
+aTmpStr[ nIndex ] = cNew;
 }
+rStr = aTmpStr.makeStringAndClear();
 }
 
 //===
diff --git a/vcl/source/gdi/outdev3.cxx b/vcl/source/gdi/outdev3.cxx
index df1016c..565fcae 100644
--- a/vcl/source/gdi/outdev3.cxx
+++ b/vcl/source/gdi/outdev3.cxx
@@ -5836,8 +5836,11 @@ SalLayout* OutputDevice::ImplLayout( const String& 
rOrigStr,
 
 // convert from logical units to physical units
 // recode string if needed
-if( mpFontEntry->mpConversion )
-mpFontEntry->mpConversion->RecodeString( aStr, 0, aStr.Len() );
+if( mpFontEntry->mpConversion ) {
+OUString aTmpStr(aStr); // only needed until aStr is OUString as well
+mpFontEntry->mpConversion->RecodeString( aTmpStr, 0, 
aTmpStr.getLength() );
+aStr = String(aTmpStr);
+}
 
 long nPixelWidth = nLogicalWidth;
 if( nLogicalWidth && mbMap )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/generic vcl/inc vcl/source

2013-03-26 Thread Chr . Rossmanith
 vcl/generic/print/genpspgraphics.cxx |   14 --
 vcl/inc/generic/genpspgraphics.h |2 +-
 vcl/inc/salgdi.hxx   |8 ++--
 vcl/source/gdi/outdev3.cxx   |   11 ---
 vcl/source/gdi/salgdilayout.cxx  |2 +-
 5 files changed, 20 insertions(+), 17 deletions(-)

New commits:
commit bc1af14a145f62afb58d6b63e10605d6230073cd
Author: Chr. Rossmanith 
Date:   Sun Mar 24 09:53:00 2013 +0100

Use OUString and sal_Int32 in filterText()

Change-Id: I31437125b51b07be490de3d979e193fad9750f51
Reviewed-on: https://gerrit.libreoffice.org/2956
Reviewed-by: Chris Sherlock 
Reviewed-by: Eike Rathke 
Reviewed-by: Fridrich Strba 
Tested-by: Fridrich Strba 

diff --git a/vcl/generic/print/genpspgraphics.cxx 
b/vcl/generic/print/genpspgraphics.cxx
index ec48150..eb6a529 100644
--- a/vcl/generic/print/genpspgraphics.cxx
+++ b/vcl/generic/print/genpspgraphics.cxx
@@ -1235,12 +1235,13 @@ void GenPspGraphics::AnnounceFonts( ImplDevFontList* 
pFontList, const psp::FastP
 pFontList->Add( pFD );
 }
 
-bool GenPspGraphics::filterText( const rtl::OUString& rOrig, rtl::OUString& 
rNewText, xub_StrLen nIndex, xub_StrLen& rLen, xub_StrLen& rCutStart, 
xub_StrLen& rCutStop )
+bool GenPspGraphics::filterText( const rtl::OUString& rOrig, rtl::OUString& 
rNewText, sal_Int32 nIndex, sal_Int32& rLen, sal_Int32& rCutStart, sal_Int32& 
rCutStop )
 {
 if( ! m_pPhoneNr )
 return false;
 
-rCutStop = rCutStart = STRING_NOTFOUND;
+rNewText = rOrig;
+rCutStop = rCutStart = -1;
 
 #define FAX_PHONE_TOKEN  "@@#"
 #define FAX_PHONE_TOKEN_LENGTH   3
@@ -1260,7 +1261,7 @@ bool GenPspGraphics::filterText( const rtl::OUString& 
rOrig, rtl::OUString& rNew
 {
 nStart = nPos;
 m_bPhoneCollectionActive = true;
-m_aPhoneCollection = rtl::OUString();
+m_aPhoneCollection = "";
 bRet = true;
 bStarted = true;
 }
@@ -1286,13 +1287,13 @@ bool GenPspGraphics::filterText( const rtl::OUString& 
rOrig, rtl::OUString& rNew
 aPhoneNr.append( m_aPhoneCollection );
 aPhoneNr.append( "" );
 *m_pPhoneNr = aPhoneNr.makeStringAndClear();
-m_aPhoneCollection = rtl::OUString();
+m_aPhoneCollection = "";
 }
 }
 if( m_aPhoneCollection.getLength() > 1024 )
 {
 m_bPhoneCollectionActive = false;
-m_aPhoneCollection = rtl::OUString();
+m_aPhoneCollection = "";
 bRet = false;
 }
 
@@ -1301,7 +1302,8 @@ bool GenPspGraphics::filterText( const rtl::OUString& 
rOrig, rtl::OUString& rNew
 rLen -= nStop - nStart;
 rCutStart = nStart+nIndex;
 rCutStop = nStop+nIndex;
-rNewText = ( rCutStart ? rOrig.copy( 0, rCutStart ) : rtl::OUString() 
) + rOrig.copy( rCutStop );
+if (rCutStart != rCutStop)
+rNewText = ( rCutStart ? rOrig.copy( 0, rCutStart ) : 
rtl::OUString() ) + rOrig.copy( rCutStop );
 }
 
 return bRet && m_bSwallowFaxNo;
diff --git a/vcl/inc/generic/genpspgraphics.h b/vcl/inc/generic/genpspgraphics.h
index 99a2f35..36c0e60 100644
--- a/vcl/inc/generic/genpspgraphics.h
+++ b/vcl/inc/generic/genpspgraphics.h
@@ -175,7 +175,7 @@ public:
 virtual voidinvert( sal_uIntPtr nPoints, const SalPoint* 
pPtAry, SalInvert nFlags );
 
 virtual sal_BooldrawEPS( long nX, long nY, long nWidth, long 
nHeight, void* pPtr, sal_uIntPtr nSize );
-virtual boolfilterText( const rtl::OUString& rOrigText, 
rtl::OUString& rNewText, xub_StrLen nIndex, xub_StrLen& rLen, xub_StrLen& 
rCutStart, xub_StrLen& rCutStop );
+virtual boolfilterText( const rtl::OUString& rOrigText, 
rtl::OUString& rNewText, sal_Int32 nIndex, sal_Int32& rLen, sal_Int32& 
rCutStart, sal_Int32& rCutStop );
 
 virtual booldrawAlphaBitmap( const SalTwoRect&,
  const SalBitmap& rSourceBitmap,
diff --git a/vcl/inc/salgdi.hxx b/vcl/inc/salgdi.hxx
index ac2d036..21d7380 100644
--- a/vcl/inc/salgdi.hxx
+++ b/vcl/inc/salgdi.hxx
@@ -346,12 +346,8 @@ public:
 true: a substitution has taken place and rNewText rLen, rCutStart and 
rCutStop have been filled accordingly
 false: no substitution has taken place, rNewText, rLen, rCutStart, 
rCutStop remain unchanged
  */
-virtual boolfilterText( const rtl::OUString& rOrigText,
-rtl::OUString& rNewText,
-xub_StrLen nIndex,
-xub_StrLen& rLen,
-xub_StrLen& rCutStart,
-xub_StrLen& rCutStop );
+virtual boolfilterText( const rtl::OUString& rOrigText, 
rtl::OUString& rNewText,
+sal_Int32 nIndex, sal_Int32& rLen, 
sal_Int32& rCutS

[PUSHED] Replace String with OUString in RecodeString

2013-03-26 Thread Fridrich Strba (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2919

Approvals:
  Eike Rathke: Looks good to me, approved
  Fridrich Strba: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I987f738d1cd7d640a253f3cd31864290b6d763bf
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Christina Roßmanith 
Gerrit-Reviewer: Eike Rathke 
Gerrit-Reviewer: Fridrich Strba 

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


[PUSHED] Use OUString and sal_Int32 in filterText()

2013-03-26 Thread Fridrich Strba (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2956

Approvals:
  Eike Rathke: Looks good to me, approved
  Chris Sherlock: Looks good to me, but someone else must approve
  Fridrich Strba: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I31437125b51b07be490de3d979e193fad9750f51
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Christina Roßmanith 
Gerrit-Reviewer: Chris Sherlock 
Gerrit-Reviewer: Eike Rathke 
Gerrit-Reviewer: Fridrich Strba 

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


[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-03-26 Thread Chr . Rossmanith
 vcl/inc/vcl/outdev.hxx |9 +++--
 vcl/source/gdi/outdev3.cxx |   45 ++---
 2 files changed, 29 insertions(+), 25 deletions(-)

New commits:
commit a89fda4b7f09856ba136f0a4df7929f7ee3f796b
Author: Chr. Rossmanith 
Date:   Sun Mar 24 10:15:45 2013 +0100

Use OUString and sal_Int32 in ImplPrepareLayoutArgs() and GetTextIsRTL()

Change-Id: Ib1b659aa56a38efdfc2dddb5e362ac1eb54972f1
Reviewed-on: https://gerrit.libreoffice.org/2959
Reviewed-by: Eike Rathke 
Reviewed-by: Fridrich Strba 
Tested-by: Fridrich Strba 

diff --git a/vcl/inc/vcl/outdev.hxx b/vcl/inc/vcl/outdev.hxx
index e51ec2b..5f00757 100644
--- a/vcl/inc/vcl/outdev.hxx
+++ b/vcl/inc/vcl/outdev.hxx
@@ -382,10 +382,8 @@ public:
 xub_StrLen nLen, const Point& 
rLogicPos = Point(0,0),
 long nLogicWidth=0, const 
sal_Int32* pLogicDXArray=NULL,
 bool bFilter = false ) const;
-SAL_DLLPRIVATE ImplLayoutArgs ImplPrepareLayoutArgs( String&,
-xub_StrLen nIndex, xub_StrLen nLen,
-long nPixelWidth,
-const sal_Int32* pPixelDXArray ) 
const;
+SAL_DLLPRIVATE ImplLayoutArgs ImplPrepareLayoutArgs( OUString&, const 
sal_Int32 nIndex, const sal_Int32 nLen,
+long nPixelWidth, const sal_Int32* 
pPixelDXArray ) const;
 SAL_DLLPRIVATE SalLayout*   ImplGlyphFallbackLayout( SalLayout*, 
ImplLayoutArgs& ) const;
 
 static
@@ -1060,8 +1058,7 @@ public:
 
 sal_uInt16  GetBitCount() const;
 
-sal_BoolGetTextIsRTL( const String&, xub_StrLen nIndex,
-xub_StrLen nLen ) const;
+sal_BoolGetTextIsRTL( const OUString&, sal_Int32 nIndex, 
sal_Int32 nLen ) const;
 
 /** Query the existence and depth of the alpha channel
 
diff --git a/vcl/source/gdi/outdev3.cxx b/vcl/source/gdi/outdev3.cxx
index a2e2e1b..1fedcbd 100644
--- a/vcl/source/gdi/outdev3.cxx
+++ b/vcl/source/gdi/outdev3.cxx
@@ -5665,13 +5665,13 @@ void OutputDevice::DrawStretchText( const Point& 
rStartPt, sal_uLong nWidth,
 mpAlphaVDev->DrawStretchText( rStartPt, nWidth, rStr, nIndex, nLen );
 }
 
-ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( String& rStr,
-   xub_StrLen nMinIndex, xub_StrLen nLen,
+ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( OUString& rStr,
+   const sal_Int32 nMinIndex, const 
sal_Int32 nLen,
long nPixelWidth, const sal_Int32* 
pDXArray ) const
 {
 // get string length for calculating extents
-xub_StrLen nEndIndex = rStr.Len();
-if( (sal_uLong)nMinIndex + nLen < nEndIndex )
+sal_Int32 nEndIndex = rStr.getLength();
+if( nMinIndex + nLen < nEndIndex )
 nEndIndex = nMinIndex + nLen;
 
 // don't bother if there is nothing to do
@@ -5686,8 +5686,8 @@ ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( 
String& rStr,
 else if( 0 == (mnTextLayoutMode & TEXT_LAYOUT_BIDI_RTL) )
 {
 // disable Bidi if no RTL hint and no RTL codes used
-const sal_Unicode* pStr = rStr.GetBuffer() + nMinIndex;
-const sal_Unicode* pEnd = rStr.GetBuffer() + nEndIndex;
+const sal_Unicode* pStr = rStr.getStr() + nMinIndex;
+const sal_Unicode* pEnd = rStr.getStr() + nEndIndex;
 for( ; pStr < pEnd; ++pStr )
 if( ((*pStr >= 0x0580) && (*pStr < 0x0800))   // middle eastern 
scripts
 ||  ((*pStr >= 0xFB18) && (*pStr < 0xFE00))   // hebrew + arabic A 
presentation forms
@@ -5711,8 +5711,8 @@ ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( 
String& rStr,
 else
 {
 // disable CTL for non-CTL text
-const sal_Unicode* pStr = rStr.GetBuffer() + nMinIndex;
-const sal_Unicode* pEnd = rStr.GetBuffer() + nEndIndex;
+const sal_Unicode* pStr = rStr.getStr() + nMinIndex;
+const sal_Unicode* pEnd = rStr.getStr() + nEndIndex;
 for( ; pStr < pEnd; ++pStr )
 if( ((*pStr >= 0x0300) && (*pStr < 0x0370))   // diacritical marks
 ||  ((*pStr >= 0x0590) && (*pStr < 0x10A0))   // many CTL scripts
@@ -5731,9 +5731,10 @@ ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( 
String& rStr,
 if( meTextLanguage ) //TODO: (mnTextLayoutMode & 
TEXT_LAYOUT_SUBSTITUTE_DIGITS)
 {
 // disable character localization when no digits used
-const sal_Unicode* pBase = rStr.GetBuffer();
+const sal_Unicode* pBase = rStr.getStr();
 const sal_Unicode* pStr = pBase + nMinIndex;
 const sal_Unicode* pEnd = pBase + nEndIndex;
+OUStringBuffer sTmpStr(rStr);
 for( ; pStr < pEnd; ++pStr )
 {
 // TODO: are there 

[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-03-26 Thread Chr . Rossmanith
 vcl/inc/vcl/outdev.hxx |   12 -
 vcl/source/gdi/outdev3.cxx |   55 +
 2 files changed, 26 insertions(+), 41 deletions(-)

New commits:
commit ffb837eed2c6f843f12f9368ed2183660a0036a1
Author: Chr. Rossmanith 
Date:   Sun Mar 24 16:30:20 2013 +0100

Use OUString and sal_Int32 in GetTextArray() and ImplLayout()

Change-Id: I2c1e5b7d53c0d78f2ccf9ac317a7ff40298cd68d
Reviewed-on: https://gerrit.libreoffice.org/2967
Reviewed-by: Eike Rathke 
Reviewed-by: Fridrich Strba 
Tested-by: Fridrich Strba 

diff --git a/vcl/inc/vcl/outdev.hxx b/vcl/inc/vcl/outdev.hxx
index 5f00757..44a5979 100644
--- a/vcl/inc/vcl/outdev.hxx
+++ b/vcl/inc/vcl/outdev.hxx
@@ -378,10 +378,9 @@ public:
 SAL_DLLPRIVATE bool ImplSelectClipRegion( const Region&, 
SalGraphics* pGraphics = NULL );
 SAL_DLLPRIVATE void ImplSetClipRegion( const Region* pRegion );
 
-SAL_DLLPRIVATE SalLayout*   ImplLayout( const String&, xub_StrLen nIndex,
-xub_StrLen nLen, const Point& 
rLogicPos = Point(0,0),
-long nLogicWidth=0, const 
sal_Int32* pLogicDXArray=NULL,
-bool bFilter = false ) const;
+SAL_DLLPRIVATE SalLayout*   ImplLayout( const OUString&, sal_Int32 nIndex, 
sal_Int32 nLen,
+const Point& rLogicPos = 
Point(0,0), long nLogicWidth=0,
+const sal_Int32* 
pLogicDXArray=NULL, bool bFilter = false ) const;
 SAL_DLLPRIVATE ImplLayoutArgs ImplPrepareLayoutArgs( OUString&, const 
sal_Int32 nIndex, const sal_Int32 nLen,
 long nPixelWidth, const sal_Int32* 
pPixelDXArray ) const;
 SAL_DLLPRIVATE SalLayout*   ImplGlyphFallbackLayout( SalLayout*, 
ImplLayoutArgs& ) const;
@@ -583,9 +582,8 @@ public:
const sal_Int32* pDXAry = NULL,
xub_StrLen nIndex = 0,
xub_StrLen nLen = STRING_LEN );
-longGetTextArray( const XubString& rStr, sal_Int32* pDXAry 
= NULL,
-  xub_StrLen nIndex = 0,
-  xub_StrLen nLen = STRING_LEN ) const;
+longGetTextArray( const OUString& rStr, sal_Int32* pDXAry 
= NULL,
+  sal_Int32 nIndex = 0, sal_Int32 nLen = 
-1 ) const;
 boolGetCaretPositions( const XubString&, sal_Int32* 
pCaretXArray,
   xub_StrLen nIndex, xub_StrLen nLen,
   sal_Int32* pDXAry = NULL, long nWidth = 
0,
diff --git a/vcl/source/gdi/outdev3.cxx b/vcl/source/gdi/outdev3.cxx
index 1fedcbd..56d1d4a 100644
--- a/vcl/source/gdi/outdev3.cxx
+++ b/vcl/source/gdi/outdev3.cxx
@@ -5475,7 +5475,11 @@ long OutputDevice::GetTextWidth( const String& rStr,
 {
 DBG_CHKTHIS( OutputDevice, ImplDbgCheckOutputDevice );
 
-long nWidth = GetTextArray( rStr, NULL, nIndex, nLen );
+sal_Int32 nLen2 = (nLen == STRING_LEN) ? -1 : nLen; // only needed until 
nLen is sal_Int32
+sal_Int32 nIndex2 = nIndex;// ditto
+OUString aTmpStr(rStr);
+long nWidth = GetTextArray( aTmpStr, NULL, nIndex2, nLen2 );
+
 return nWidth;
 }
 
@@ -5532,15 +5536,17 @@ void OutputDevice::DrawTextArray( const Point& 
rStartPt, const String& rStr,
 mpAlphaVDev->DrawTextArray( rStartPt, rStr, pDXAry, nIndex, nLen );
 }
 
-long OutputDevice::GetTextArray( const String& rStr, sal_Int32* pDXAry,
- xub_StrLen nIndex, xub_StrLen nLen ) const
+long OutputDevice::GetTextArray( const OUString& rStr, sal_Int32* pDXAry,
+ sal_Int32 nIndex, sal_Int32 nLen ) const
 {
+// MEM: default nLen = STRING_LENGTH
 DBG_CHKTHIS( OutputDevice, ImplDbgCheckOutputDevice );
 
-if( nIndex >= rStr.Len() )
+if( nIndex >= rStr.getLength() )
 return 0;
-if( (sal_uLong)nIndex+nLen >= rStr.Len() )
-nLen = rStr.Len() - nIndex;
+
+if( nLen < 0 || nIndex+nLen >= rStr.getLength() )
+nLen = rStr.getLength() - nIndex;
 
 // do layout
 SalLayout* pSalLayout = ImplLayout( rStr, nIndex, nLen );
@@ -5776,12 +5782,8 @@ ImplLayoutArgs OutputDevice::ImplPrepareLayoutArgs( 
OUString& rStr,
 return aLayoutArgs;
 }
 
-SalLayout* OutputDevice::ImplLayout( const String& rOrigStr,
- xub_StrLen nMinIndex,
- xub_StrLen nLen,
- const Point& rLogicalPos,
- long nLogicalWidth,
- const sal_Int32* pDXArray,
+SalLayout* OutputDevice::ImplLayout( const OUString& rOrigStr, sal_Int32 
nMinIndex, sal_Int3

[PUSHED] Use OUString and sal_Int32 in GetTextArray() and ImplLayout(...

2013-03-26 Thread Fridrich Strba (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2967

Approvals:
  Eike Rathke: Looks good to me, approved
  Fridrich Strba: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2c1e5b7d53c0d78f2ccf9ac317a7ff40298cd68d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Christina Roßmanith 
Gerrit-Reviewer: Eike Rathke 
Gerrit-Reviewer: Fridrich Strba 

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


[PUSHED] Use OUString and sal_Int32 in ImplPrepareLayoutArgs() and Ge...

2013-03-26 Thread Fridrich Strba (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2959

Approvals:
  Eike Rathke: Looks good to me, approved
  Fridrich Strba: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1b659aa56a38efdfc2dddb5e362ac1eb54972f1
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Christina Roßmanith 
Gerrit-Reviewer: Eike Rathke 
Gerrit-Reviewer: Fridrich Strba 

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


[Libreoffice-commits] core.git: vcl/aqua

2013-03-26 Thread Herbert Dürr
 vcl/aqua/source/dtrans/DataFlavorMapping.cxx |   60 ---
 vcl/aqua/source/dtrans/DataFlavorMapping.hxx |4 +
 2 files changed, 58 insertions(+), 6 deletions(-)

New commits:
commit 9fccdcaf810e543e28bca662300b5936ad7acf30
Author: Herbert Dürr 
Date:   Mon May 14 15:10:16 2012 +

(author=pl) fix for transporting OOo types through OSX system clipboard

Change-Id: Idc00fbc8ac1adb04c76ecb4b641cfa055f5901b8

diff --git a/vcl/aqua/source/dtrans/DataFlavorMapping.cxx 
b/vcl/aqua/source/dtrans/DataFlavorMapping.cxx
index 23dbfd4..b030ac8 100644
--- a/vcl/aqua/source/dtrans/DataFlavorMapping.cxx
+++ b/vcl/aqua/source/dtrans/DataFlavorMapping.cxx
@@ -50,6 +50,8 @@ using ::rtl::OString;
 
 namespace // private
 {
+  const Type CPPUTYPE_SEQINT8  = getCppuType((Sequence*)0);
+
   /* Determine whether or not a DataFlavor is valid.
*/
   bool isValidFlavor(const DataFlavor& aFlavor)
@@ -59,6 +61,22 @@ namespace // private
 return ((len > 0) && ((dtype == getCppuType((Sequence*)0)) || 
(dtype == getCppuType( (OUString*)0 ;
   }
 
+  OUString NSStringToOUString(NSString* cfString)
+  {
+BOOST_ASSERT(cfString && "Invalid parameter");
+
+const char* utf8Str = [cfString UTF8String];
+unsigned int len = rtl_str_getLength(utf8Str);
+
+return OUString(utf8Str, len, RTL_TEXTENCODING_UTF8);
+  }
+
+  NSString* OUStringToNSString(const OUString& ustring)
+  {
+OString utf8Str = OUStringToOString(ustring, RTL_TEXTENCODING_UTF8);
+return [NSString stringWithCString: utf8Str.getStr() encoding: 
NSUTF8StringEncoding];
+  }
+
   NSString* PBTYPE_SODX = 
@"application/x-openoffice-objectdescriptor-xml;windows_formatname=\"Star 
Object Descriptor (XML)\"";
   NSString* PBTYPE_SESX = 
@"application/x-openoffice-embed-source-xml;windows_formatname=\"Star Embed 
Source (XML)\"";
   NSString* PBTYPE_SLSDX = 
@"application/x-openoffice-linksrcdescriptor-xml;windows_formatname=\"Star Link 
Source Descriptor (XML)\"";
@@ -499,6 +517,16 @@ DataFlavorMapper::DataFlavorMapper()
 mrXMimeCntFactory = MimeContentTypeFactory::create( xContext );
 }
 
+DataFlavorMapper::~DataFlavorMapper()
+{
+// release potential NSStrings
+for( OfficeOnlyTypes::iterator it = maOfficeOnlyTypes.begin(); it != 
maOfficeOnlyTypes.end(); ++it )
+{
+[it->second release];
+it->second = nil;
+}
+}
+
 DataFlavor DataFlavorMapper::systemToOpenOfficeFlavor(NSString* 
systemDataFlavor) const
 {
   DataFlavor oOOFlavor;
@@ -514,22 +542,41 @@ DataFlavor 
DataFlavorMapper::systemToOpenOfficeFlavor(NSString* systemDataFlavor
 }
 } // for
 
-  return oOOFlavor;
+// look if this might be an internal type; if it comes in here it must have
+// been through openOfficeToSystemFlavor before, so it should then be in 
the map
+OUString aTryFlavor( NSStringToOUString( systemDataFlavor ) );
+if( maOfficeOnlyTypes.find( aTryFlavor ) != maOfficeOnlyTypes.end() )
+{
+oOOFlavor.MimeType = aTryFlavor;
+oOOFlavor.HumanPresentableName = OUString();
+oOOFlavor.DataType = CPPUTYPE_SEQINT8;
+}
+
+return oOOFlavor;
 }
 
 NSString* DataFlavorMapper::openOfficeToSystemFlavor(const DataFlavor& 
oOOFlavor) const
 {
-  NSString* sysFlavor = NULL;
+NSString* sysFlavor = NULL;
 
-  for (size_t i = 0; i < SIZE_FLAVOR_MAP; i++)
+for( size_t i = 0; i < SIZE_FLAVOR_MAP; ++i )
 {
-  if 
(oOOFlavor.MimeType.startsWith(OUString::createFromAscii(flavorMap[i].OOoFlavor)))
+   if 
(oOOFlavor.MimeType.startsWith(OUString::createFromAscii(flavorMap[i].OOoFlavor)))
 {
-  sysFlavor = flavorMap[i].SystemFlavor;
+sysFlavor = flavorMap[i].SystemFlavor;
 }
 }
 
-  return sysFlavor;
+if( ! sysFlavor )
+{
+OfficeOnlyTypes::const_iterator it = maOfficeOnlyTypes.find( 
oOOFlavor.MimeType );
+if( it == maOfficeOnlyTypes.end() )
+sysFlavor = maOfficeOnlyTypes[ oOOFlavor.MimeType ] = 
OUStringToNSString( oOOFlavor.MimeType );
+else
+sysFlavor = it->second;
+}
+
+return sysFlavor;
 }
 
 NSString* DataFlavorMapper::openOfficeImageToSystemFlavor(NSPasteboard* 
pPasteboard) const
@@ -668,6 +715,7 @@ NSArray* DataFlavorMapper::flavorSequenceToTypesArray(const 
com::sun::star::uno:
 
   if (str != NULL)
   {
+  [str retain];
   [array addObject: str];
   }
   }
diff --git a/vcl/aqua/source/dtrans/DataFlavorMapping.hxx 
b/vcl/aqua/source/dtrans/DataFlavorMapping.hxx
index 3efe8cf..7ef3640 100644
--- a/vcl/aqua/source/dtrans/DataFlavorMapping.hxx
+++ b/vcl/aqua/source/dtrans/DataFlavorMapping.hxx
@@ -29,6 +29,7 @@
 #import 
 #include 
 
+#include 
 #include 
 #include 
 
@@ -65,6 +66,7 @@ public:
  cannot be created.
*/
   DataFlavorMapper();
+  ~DataFlavorMapper();
 
 
   /* Map a system data flavor to an OpenOffice data flavor.
@@ -128,6 +130,8 @@ private:
 
 private:
   ::com::sun::st

[Libreoffice-commits] core.git: svl/inc

2013-03-26 Thread Michael Stahl
 svl/inc/svl/style.hrc |   20 ++--
 1 file changed, 10 insertions(+), 10 deletions(-)

New commits:
commit 5bd206efc74671ce4b9c1371d1cae635970f54bb
Author: Michael Stahl 
Date:   Tue Mar 26 13:55:55 2013 +0100

style.hrc: translate comments

Change-Id: I00766a327a99b6fd5eed8e0ba411f3f71f07e942

diff --git a/svl/inc/svl/style.hrc b/svl/inc/svl/style.hrc
index 83c7153..352cfe1 100644
--- a/svl/inc/svl/style.hrc
+++ b/svl/inc/svl/style.hrc
@@ -17,17 +17,17 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#ifndef _SFX_STYLE_HRC
-#define _SFX_STYLE_HRC
+#ifndef SFX_STYLE_HRC
+#define SFX_STYLE_HRC
 
-#define SFXSTYLEBIT_AUTO0x // automatisch; Flags kommen von der 
Applikation
-#define SFXSTYLEBIT_HIDDEN  0x0200 // benutzte Vorlage (als Suchmaske)
-#define SFXSTYLEBIT_HIERARCHY   0x1000 // hierarchical view - just for dialog
-#define SFXSTYLEBIT_READONLY0x2000 // benutzte Vorlage (als Suchmaske)
-#define SFXSTYLEBIT_USED0x4000 // benutzte Vorlage (als Suchmaske)
-#define SFXSTYLEBIT_USERDEF 0x8000 // benutzerdefinierte Vorlage
-#define SFXSTYLEBIT_ALL_VISIBLE 0xFDFF // alle Vorlagen
-#define SFXSTYLEBIT_ALL 0x // alle Vorlagen
+#define SFXSTYLEBIT_AUTO0x ///< automatic: flags from application
+#define SFXSTYLEBIT_HIDDEN  0x0200 ///< hidden styles (search mask)
+#define SFXSTYLEBIT_HIERARCHY   0x1000 ///< hierarchical view - just for dialog
+#define SFXSTYLEBIT_READONLY0x2000 ///< readonly styles (search mask)
+#define SFXSTYLEBIT_USED0x4000 ///< used styles (search mask)
+#define SFXSTYLEBIT_USERDEF 0x8000 ///< user defined styles (search mask)
+#define SFXSTYLEBIT_ALL_VISIBLE 0xFDFF ///< all styles
+#define SFXSTYLEBIT_ALL 0x ///< all styles
 
 #endif
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] dev-tools.git: scripts/esc-bug-stats.pl

2013-03-26 Thread Michael Meeks
 scripts/esc-bug-stats.pl |   90 +++
 1 file changed, 90 insertions(+)

New commits:
commit c2ca429a1a25adeeb65ae881a343ddac9c502ade
Author: Michael Meeks 
Date:   Tue Mar 26 13:03:20 2013 +

start to automate the number collection.

diff --git a/scripts/esc-bug-stats.pl b/scripts/esc-bug-stats.pl
new file mode 100755
index 000..9a7a152
--- /dev/null
+++ b/scripts/esc-bug-stats.pl
@@ -0,0 +1,90 @@
+#!/usr/bin/perl -w
+
+sub get_url($)
+{
+my $url = shift;
+my @lines;
+my $handle;
+open ($handle, "curl -s '$url' 2>&1 |") || die "can't exec curl: $!";
+while (<$handle>) {
+   push @lines, $_;
+}
+close ($handle);
+return @lines;
+}
+
+sub get_deps($)
+{
+my ($url) = @_;
+my @bugs = get_url($url);
+
+my $bug_count = -1;
+while (my $line = shift (@bugs)) {
+   if ($line =~ m/^\s*depends on\s*$/) {
+   $line = shift @bugs;
+#  print STDERR "Have depends on\n";
+   if ($line =~ m/^\s*(\d+)\s*$/) {
+   my $num = $1;
+   $line = shift @bugs;
+   $line = shift @bugs;
+   if ($line =~ m/bugs:/) {
+   $bug_count = $num;
+   last;
+   }
+   } else {
+   print STDERR "odd depends on follow-on: '$line'\n";
+   }
+   }
+}
+return $bug_count;
+}
+
+sub get_query($)
+{
+my ($url) = @_;
+my @bugs = get_url($url);
+
+my $bug_count = -1;
+while (my $line = shift (@bugs)) {
+   if ($line =~ m/(\d+) bugs found./) {
+   $bug_count = $1;
+   last;
+   }
+}
+return $bug_count;
+}
+
+my %bug_to_ver = (
+'4.1' => '60270',
+'4.0' => '54157',
+'3.6' => '6'
+);
+
+print "Querying for open MABs:\n";
+for my $ver (sort keys %bug_to_ver) {
+my $bug = $bug_to_ver{$ver};
+my $base_url = "https://bugs.freedesktop.org/showdependencytree.cgi?id="; . 
$bug;
+my $all = get_deps($base_url);
+my $open = get_deps($base_url . "&hide_resolved=1");
+$percent = ($open * 100.0) / $all;
+print "$ver: $open/$all - $percent%\n";
+}
+
+my ($all, $open);
+
+print "Querying for regressions:\n";
+my 
$regression_query="https://bugs.freedesktop.org/buglist.cgi?columnlist=bug_severity%2Cpriority%2Ccomponent%2Cop_sys%2Cassigned_to%2Cbug_status%2Cresolution%2Cshort_desc&keywords=regression%2C%20&keywords_type=allwords&list_id=267671&product=LibreOffice&query_format=advanced&order=bug_id&limit=0";;
+my 
$regression_open_query="https://bugs.freedesktop.org/buglist.cgi?keywords=regression%2C%20&keywords_type=allwords&list_id=267687&columnlist=bug_severity%2Cpriority%2Ccomponent%2Cop_sys%2Cassigned_to%2Cbug_status%2Cresolution%2Cshort_desc&resolution=---&query_based_on=Regressions&query_format=advanced&product=LibreOffice&known_name=Regressions&limit=0";;
+$all = get_query($regression_query);
+$open = get_query($regression_open_query);
+print "$open/$all\n";
+
+print "Querying for bibisection:\n";
+my $bibisect_query = 
"https://bugs.freedesktop.org/buglist.cgi?n2=1&f1=status_whiteboard&list_id=267679&o1=substring&resolution=---&resolution=FIXED&resolution=INVALID&resolution=WONTFIX&resolution=DUPLICATE&resolution=WORKSFORME&resolution=MOVED&resolution=NOTABUG&resolution=NOTOURBUG&query_based_on=BibisectedAll&o2=substring&query_format=advanced&f2=status_whiteboard&v1=bibisected&v2=bibisected35older&product=LibreOffice&known_name=BibisectedAll&limit=0";;
+my $bibisect_open_query = 
"https://bugs.freedesktop.org/buglist.cgi?n2=1&f1=status_whiteboard&list_id=267685&o1=substring&resolution=---&query_based_on=Bibisected&o2=substring&query_format=advanced&f2=status_whiteboard&v1=bibisected&v2=bibisected35older&product=LibreOffice&known_name=Bibisected&limit=0";;
+
+$all = get_query($bibisect_query);
+$open = get_query($bibisect_open_query);
+print "$open/$all\n";
+
+
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [Libreoffice-commits] core.git: make bridges work with clang on x64 linux

2013-03-26 Thread Peter Foley
On Tue, Mar 26, 2013 at 3:45 AM, Stephan Bergmann  wrote:
> What Linux version would that be?
It's a gentoo box.
I'm using clang 3.2 and llvm 3.2.
I've attached the gentoo package version info.
I'd be more than happy to try out a patch or something to try and fix
this properly.

Thanks,

Peter


i
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: opengrok feature request: correcting links to .hxx files

2013-03-26 Thread Christian Lohmaier
Hi Noel, *,

don't be afraid, no Zombie attacks, just happened to stumble upon this
with a different search :-)

On Sun, Dec 23, 2012 at 7:37 AM, Noel Grandin  wrote:
>
> Anybody feel like hacking on opengrok - I have a small request.
>
> The parsing for header file includes does not seem to know how to correctly
> grep our .hxx header files.
> [...]
>   #include "AsyncronousLink.hxx"
> [...]
> which incorrectly points to the file
> AsyncronousLink.h

This should be fixed with current versions of opengrok.
"Bug #16760: Generating incorrect links for C/C++ "#include"
statements if extension of a header file is not on the lexer's list"

So our copy of opengrok 0.9 should be replaced with 0.11.1
https://github.com/OpenGrok/OpenGrok/wiki/File-downloads

Needs regenerating the indexes again though (on that occasion, one
could enable leading wildcards as well)
And as they mention UI changes, maybe the LO-branding from
https://github.com/tdf/www-opengrok/ needs some little tweaking.

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


[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-03-26 Thread Zolnai Tamás
 vcl/inc/vcl/field.hxx|2 -
 vcl/source/control/field.cxx |   80 +++
 2 files changed, 45 insertions(+), 37 deletions(-)

New commits:
commit 84bac1799e528272ca808240508ca3f66272ee13
Author: Zolnai Tamás 
Date:   Mon Mar 25 13:02:01 2013 +0100

Kill double conversion from NumericBox

Change-Id: Id1161c40725f75ea0d7ec3f15a19ca14defd0a75
Reviewed-on: https://gerrit.libreoffice.org/3035
Reviewed-by: Eike Rathke 
Tested-by: Eike Rathke 

diff --git a/vcl/inc/vcl/field.hxx b/vcl/inc/vcl/field.hxx
index ec661c6..50888c6 100644
--- a/vcl/inc/vcl/field.hxx
+++ b/vcl/inc/vcl/field.hxx
@@ -170,7 +170,7 @@ protected:
 voidFieldLast();
 
 SAL_DLLPRIVATE void ImplLoadRes( const ResId& rResId );
-SAL_DLLPRIVATE sal_Bool ImplNumericReformat( const OUString& rStr, double& 
rValue, OUString& rOutStr );
+SAL_DLLPRIVATE sal_Bool ImplNumericReformat( const OUString& rStr, 
sal_Int64& rValue, OUString& rOutStr );
 SAL_DLLPRIVATE void ImplNewFieldValue( sal_Int64 nNewValue );
 SAL_DLLPRIVATE void ImplSetUserValue( sal_Int64 nNewValue, Selection* 
pNewSelection = NULL );
 
diff --git a/vcl/source/control/field.cxx b/vcl/source/control/field.cxx
index 69cb02e..3dead80 100644
--- a/vcl/source/control/field.cxx
+++ b/vcl/source/control/field.cxx
@@ -89,7 +89,7 @@ static sal_Bool ImplNumericProcessKeyInput( Edit*, const 
KeyEvent& rKEvt,
 
 // ---
 
-static sal_Bool ImplNumericGetValue( const OUString& rStr, double& rValue,
+static sal_Bool ImplNumericGetValue( const OUString& rStr, sal_Int64& rValue,
  sal_uInt16 nDecDigits, const 
LocaleDataWrapper& rLocaleDataWrappper,
  sal_Bool bCurrency = sal_False )
 {
@@ -196,7 +196,19 @@ static sal_Bool ImplNumericGetValue( const OUString& rStr, 
double& rValue,
 aStr  = aStr1.makeStringAndClear() + aStr2.makeStringAndClear();
 
 // check range
-double nValue = rtl::OUString(aStr).toDouble();
+sal_Int64 nValue = aStr.toInt64();
+if( nValue == 0 )
+{
+// check if string is equivalent to zero
+sal_Int16 nIndex = bNegative ? 1 : 0;
+while( aStr[nIndex] == '0' && nIndex < aStr.getLength() )
+++nIndex;
+if( nIndex < aStr.getLength() )
+{
+rValue = bNegative ? SAL_MIN_INT64 : SAL_MAX_INT64;
+return sal_True;
+}
+}
 if (bRound)
 {
 if ( !bNegative )
@@ -428,23 +440,22 @@ sal_Bool FormatterBase::IsEmptyFieldValue() const
 
 // ---
 
-sal_Bool NumericFormatter::ImplNumericReformat( const OUString& rStr, double& 
rValue,
+sal_Bool NumericFormatter::ImplNumericReformat( const OUString& rStr, 
sal_Int64& rValue,
 OUString& rOutStr )
 {
 if ( !ImplNumericGetValue( rStr, rValue, GetDecimalDigits(), 
ImplGetLocaleDataWrapper() ) )
 return sal_True;
 else
 {
-double nTempVal = rValue;
-// caution: precision loss in double cast
+sal_Int64 nTempVal = rValue;
 if ( nTempVal > mnMax )
-nTempVal = (double)mnMax;
+nTempVal = mnMax;
 else if ( nTempVal < mnMin )
-nTempVal = (double)mnMin;
+nTempVal = mnMin;
 
 if ( GetErrorHdl().IsSet() && (rValue != nTempVal) )
 {
-mnCorrectedValue = (sal_Int64)nTempVal;
+mnCorrectedValue = nTempVal;
 if ( !GetErrorHdl().Call( this ) )
 {
 mnCorrectedValue = 0;
@@ -454,7 +465,7 @@ sal_Bool NumericFormatter::ImplNumericReformat( const 
OUString& rStr, double& rV
 mnCorrectedValue = 0;
 }
 
-rOutStr = CreateFieldText( (sal_Int64)nTempVal );
+rOutStr = CreateFieldText( nTempVal );
 return sal_True;
 }
 }
@@ -627,17 +638,16 @@ sal_Int64 NumericFormatter::GetValue() const
 if ( !GetField() )
 return 0;
 
-double nTempValue;
+sal_Int64 nTempValue;
 
 if ( ImplNumericGetValue( GetField()->GetText(), nTempValue,
   GetDecimalDigits(), ImplGetLocaleDataWrapper() ) 
)
 {
-// caution: precision loss in double cast
 if ( nTempValue > mnMax )
-nTempValue = (double)mnMax;
+nTempValue = mnMax;
 else if ( nTempValue < mnMin )
-nTempValue = (double)mnMin;
-return (sal_Int64)nTempValue;
+nTempValue = mnMin;
+return nTempValue;
 }
 else
 return mnLastValue;
@@ -690,10 +700,9 @@ void NumericFormatter::Reformat()
 return;
 
 OUString aStr;
-// caution: precision loss in double cast
-double nTemp = (double)mnLastValue;
+sal_Int64 nTemp = mnLastValue;
 sal_Bool bOK = ImplNumericRefor

[PUSHED] Kill double conversion from NumericBox

2013-03-26 Thread Eike Rathke (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3035

Approvals:
  Eike Rathke: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id1161c40725f75ea0d7ec3f15a19ca14defd0a75
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Zolnai Tamás 
Gerrit-Reviewer: Eike Rathke 
Gerrit-Reviewer: Zolnai Tamás 

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


Re: [PATCH] fdo#51296:UI: Calc doesn't take care of security setting "Ctrl-click required to follow hyperlinks

2013-03-26 Thread Petr Mladek
On Mon, Mar 25, 2013 at 5:56 PM, Akash Shetye 
wrote:
>
> Petr thank you for your suggestions, I happened to overlook the "Ctrl-click
> to open hyperlink" prefix string to the tooltips in excitement oh having a
> patch :P. Now that I have taken a look at sc/ code, I find that the prefixed
> string is not hardcoded into the sw/ code. Also how sc/ and sw/ handle
> hyperlinks is very differently. I tried probing into the vcl part about
> fixedhyperlink.cxx to trace things back up from there but things aren't
> working well. Just knowing which part of code is doing the tooltip content
> setting in sc/ will be cool enough. Thanks for cc'ing the experts.

Yup, it needs some search. I am not much familiar with this code as
well. so, I suggest to compare the working code Writer with the code in
Calc.

In writer, it starts at
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/utlui/initui.src#n137
where the English string "STR_HYPERLINK_CLICK" is defined "%s-click to
open hyperlink".

Then you might see
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/utlui/initui.hrc#n51
, where we define some integer ID for the string STR_HYPERLINK_CLICK

The localized string is loaded at
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/utlui/initui.cxx#n157
and the "%s" is replaced by a real key code at
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/utlui/initui.cxx#n170

The string (variable aHyperlinkClick) is later used at
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/docvw/edtwin2.cxx#n220
http://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/docvw/SidebarTxtControl.cxx#n247

I think that edtwin2.cxx#n220 is the location that is responsible for
showing the tooltip when you hover over the hyperlink in Writer editor.
I think that SidebarTxtControl.cxx#n247 is just another place where you
could see similar tooltip but I am not sure where it is.


And now the fun is to find the related code in Calc. My guess is that it
might be somewhere around sc/source/ui/view. In each case, it should be
under sc/source/ui.

When I go back to sw/source/ui/docvw/edtwin2.cxx, the variable
"aHyperlinkClick" is assigned to "sTxt" which is later passed to
Help::ShowQuickHelp, so I would search for "ShowQuickHelp" and
"hyperlink" under sc/source/ui and try to fix where the hyperlink is
assigned to the tooltip string.

If you find this place, you need to:

+ define the string "%s-click to open hyperlink" in some
  Calc-specific .src file
+ define the related ID in the .hrc file
+ load the string as it was done in
  sw/source/ui/utlui/initui.cxx#n157
+ substiture the %s by the real key code
+ and finally add the string to the tooltip string if the 
  SvtSecurityOptions::E_CTRLCLICK_HYPERLINK option is set


I hope that the above makes sense and could help you.

Feel free to ask if you get lost. I am sure that Kohei or another Calc
expert could give you even more precise hints.


Best Regards,
Petr

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


Re: Trouble regarding fdo#42893

2013-03-26 Thread Michael Meeks

On Tue, 2013-03-26 at 17:20 +0530, Janit Anjaria wrote:
> I have coded the concept patch but i have been getting an error where
> the writer "hangs" and doesnt give an appropriate output.

Great :-) then you have some endless loop on the main thread.

> It would be great if someone could provide me help so i can complete
> the bug hence LO gets this feature up working asap.

So - do:

$ ps auwx | grep soffice.bin # note the pid of soffice.bin
gdb
attach 
...
-c
thread apply all backtrace

And (I imagine thread one) should be in the piece of code that is
hanging. Quite possibly you need to re-configure with --enable-symbols
and re-build just the piece that is hanging (initially) to get gdb to
show you much that is of use :-)

HTH,

Michael.

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

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


[Libreoffice-commits] core.git: 2 commits - svx/inc svx/source sw/source

2013-03-26 Thread Takeshi Abe
 svx/inc/svx/sdrmasterpagedescriptor.hxx|4 ++--
 svx/inc/svx/sdrpaintwindow.hxx |4 ++--
 svx/inc/svx/svdcrtv.hxx|6 +++---
 svx/inc/svx/svddrgv.hxx|3 +--
 svx/inc/svx/svdhdl.hxx |   12 ++--
 svx/source/sdr/contact/objectcontactofpageview.cxx |6 +++---
 svx/source/svdraw/charthelper.cxx  |   16 ++--
 svx/source/svdraw/clonelist.cxx|8 
 svx/source/svdraw/gradtrns.cxx |2 +-
 svx/source/svdraw/gradtrns.hxx |2 +-
 svx/source/svdraw/sdrmasterpagedescriptor.cxx  |4 ++--
 svx/source/svdraw/sdrpagewindow.cxx|6 +++---
 svx/source/svdraw/sdrpaintwindow.cxx   |4 ++--
 svx/source/svdraw/svdattr.cxx  |6 +++---
 svx/source/svdraw/svdcrtv.cxx  |   14 +++---
 svx/source/svdraw/svddrgv.cxx  |   14 +++---
 svx/source/svdraw/svdmrkv.cxx  |8 
 svx/source/svdraw/svdpntv.cxx  |4 ++--
 svx/source/svdraw/svdview.cxx  |6 +++---
 sw/source/ui/inc/view.hxx  |6 +++---
 sw/source/ui/uiview/viewdraw.cxx   |4 ++--
 sw/source/ui/uiview/viewport.cxx   |   14 +++---
 22 files changed, 70 insertions(+), 83 deletions(-)

New commits:
commit 0bd5a8d3f26590f467379042597f91be681fe25e
Author: Takeshi Abe 
Date:   Tue Mar 26 22:25:24 2013 +0900

sal_Bool to bool

Change-Id: I27a444a8313bff276e377ffb87458213f4e19707

diff --git a/svx/inc/svx/sdrmasterpagedescriptor.hxx 
b/svx/inc/svx/sdrmasterpagedescriptor.hxx
index 20658c1..1917a68 100644
--- a/svx/inc/svx/sdrmasterpagedescriptor.hxx
+++ b/svx/inc/svx/sdrmasterpagedescriptor.hxx
@@ -80,8 +80,8 @@ namespace sdr
 void SetVisibleLayers(const SetOfByte& rNew);
 
 // operators
-sal_Bool operator==(const MasterPageDescriptor& rCandidate) const;
-sal_Bool operator!=(const MasterPageDescriptor& rCandidate) const;
+bool operator==(const MasterPageDescriptor& rCandidate) const;
+bool operator!=(const MasterPageDescriptor& rCandidate) const;
 
 const SdrPageProperties* getCorrectSdrPageProperties() const;
 };
diff --git a/svx/inc/svx/sdrpaintwindow.hxx b/svx/inc/svx/sdrpaintwindow.hxx
index 5191f73..bf8620c 100644
--- a/svx/inc/svx/sdrpaintwindow.hxx
+++ b/svx/inc/svx/sdrpaintwindow.hxx
@@ -112,7 +112,7 @@ public:
 Rectangle GetVisibleArea() const;
 
 // Is OutDev a printer?
-sal_Bool OutputToPrinter() const { return (OUTDEV_PRINTER == 
mrOutputDevice.GetOutDevType()); }
+bool OutputToPrinter() const { return (OUTDEV_PRINTER == 
mrOutputDevice.GetOutDevType()); }
 
 // Is OutDev a window?
 sal_Bool OutputToWindow() const { return (OUTDEV_WINDOW == 
mrOutputDevice.GetOutDevType()); }
@@ -121,7 +121,7 @@ public:
 sal_Bool OutputToVirtualDevice() const { return (OUTDEV_VIRDEV == 
mrOutputDevice.GetOutDevType()); }
 
 // Is OutDev a recording MetaFile?
-sal_Bool OutputToRecordingMetaFile() const;
+bool OutputToRecordingMetaFile() const;
 
 // prepare PreRendering (evtl.)
 void PreparePreRenderDevice();
diff --git a/svx/inc/svx/svdcrtv.hxx b/svx/inc/svx/svdcrtv.hxx
index 5e42e5f..9a95ba1 100644
--- a/svx/inc/svx/svdcrtv.hxx
+++ b/svx/inc/svx/svdcrtv.hxx
@@ -114,13 +114,13 @@ public:
 void SetGluePointEditMode(sal_Bool bOn=sal_True) { 
SdrDragView::SetGluePointEditMode(bOn); CheckEdgeMode(); }
 
 // Determine whether a text tool is activated
-sal_Bool IsTextTool() const;
+bool IsTextTool() const;
 
 // Determine whether an object connector tool activated
-sal_Bool IsEdgeTool() const;
+bool IsEdgeTool() const;
 
 // Determine whether a measurement tool activated
-sal_Bool IsMeasureTool() const;
+bool IsMeasureTool() const;
 
 void SetCurrentObj(sal_uInt16 nIdent, sal_uInt32 nInvent=SdrInventor);
 void TakeCurrentObj(sal_uInt16& nIdent, sal_uInt32& nInvent) const  { 
nInvent=nAktInvent; nIdent=nAktIdent; }
diff --git a/svx/inc/svx/svddrgv.hxx b/svx/inc/svx/svddrgv.hxx
index 47ac8a8..1997dfb 100644
--- a/svx/inc/svx/svddrgv.hxx
+++ b/svx/inc/svx/svddrgv.hxx
@@ -139,8 +139,7 @@ public:
 sal_Bool IsInsertGluePoint() const { return bInsGluePoint; }
 
 // Interaktives einfuegen eines neuen Punktes. nIdx=0 => vor dem ersten 
Punkt.
-sal_Bool IsInsObjPointPossible() const;
-sal_Bool IsInsPointPossible() const { return IsInsObjPointPossible(); }
+bool IsInsObjPointPossible() const;
 sal_Bool BegInsObjPoint(const Point& rPnt, sal_Bool bNewObj) { return 
ImpBegInsObjPoint(sal_False, 0L, rPnt, bNewObj, 0L); }
 void MovInsObjPoint(const Point& rPnt) { MovDragObj(rPnt); }
 sal_Bool EndInsObjPoint(SdrCreateCmd eCmd)

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

2013-03-26 Thread Miklos Vajna
 sw/qa/extras/ooxmlimport/data/fdo59273.docx  |binary
 sw/qa/extras/ooxmlimport/data/n780645.docx   |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |   12 +
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   10 +--
 writerfilter/source/dmapper/DomainMapperTableManager.hxx |7 +
 writerfilter/source/dmapper/TablePropertiesHandler.cxx   |7 +
 writerfilter/source/ooxml/model.xml  |   19 ---
 7 files changed, 44 insertions(+), 11 deletions(-)

New commits:
commit 7329df74df134a77d078f47d5c8b70d54c5d1edb
Author: Miklos Vajna 
Date:   Tue Mar 26 11:15:59 2013 +0100

fdo#59273 import w:tblLayout of DOCX tables

Change-Id: I9af50a4c176168e2794eaa9b0e8eaeb6b9d06b13

diff --git a/sw/qa/extras/ooxmlimport/data/fdo59273.docx 
b/sw/qa/extras/ooxmlimport/data/fdo59273.docx
new file mode 100644
index 000..fcdfd77
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/fdo59273.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index a5b41e3..e3f1618 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -122,6 +122,7 @@ public:
 void testFdo61343();
 void testToolsLineNumbering();
 void testFdo60922();
+void testFdo59273();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX) && !defined(WNT)
@@ -195,6 +196,7 @@ void Test::run()
 {"fdo61343.docx", &Test::testFdo61343},
 {"tools-line-numbering.docx", &Test::testToolsLineNumbering},
 {"fdo60922.docx", &Test::testFdo60922},
+{"fdo59273.docx", &Test::testFdo59273},
 };
 header();
 for (unsigned int i = 0; i < SAL_N_ELEMENTS(aMethods); ++i)
@@ -1288,6 +1290,16 @@ void Test::testFdo60922()
 CPPUNIT_ASSERT_EQUAL(sal_Int32(100), 
getProperty(getRun(getParagraph(1), 1), "CharEscapementHeight"));
 }
 
+void Test::testFdo59273()
+{
+uno::Reference xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Reference 
xTables(xTablesSupplier->getTextTables( ), uno::UNO_QUERY);
+uno::Reference xTextTable(xTables->getByIndex(0), 
uno::UNO_QUERY);
+uno::Reference xTableRows(xTextTable->getRows(), 
uno::UNO_QUERY);
+// Was 9997, so the 4th column had ~zero width
+CPPUNIT_ASSERT_EQUAL(sal_Int16(7498), getProperty< 
uno::Sequence >(xTableRows->getByIndex(0), 
"TableColumnSeparators")[2].Position);
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index be4e05f..1119709 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -53,6 +53,7 @@ DomainMapperTableManager::DomainMapperTableManager(bool 
bOOXML) :
 m_bOOXML( bOOXML ),
 m_bPushCurrentWidth(false),
 m_bRowSizeTypeInserted(false),
+m_nLayoutType(0),
 m_pTablePropsHandler( new TablePropertiesHandler( bOOXML ) )
 {
 m_pTablePropsHandler->SetTableManager( this );
@@ -403,6 +404,7 @@ void DomainMapperTableManager::startLevel( )
 m_aTablePositions.push_back( pNewPositionHandler );
 m_nCell.push_back( 0 );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 // And push it back to the right level.
 if (oCurrentWidth)
@@ -416,6 +418,7 @@ void DomainMapperTableManager::endLevel( )
 m_aCellWidths.pop_back( );
 m_nCell.pop_back( );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 
 DomainMapperTableManager_Base_t::endLevel( );
@@ -556,12 +559,13 @@ void DomainMapperTableManager::endOfRowAction()
 #endif
 insertRowProps(pPropMap);
 }
-else if (pCellWidths->size() > 0)
+else if (pCellWidths->size() > 0 && m_nLayoutType == 
NS_ooxml::LN_Value_wordprocessingml_ST_TblLayout_fixed)
 {
 // If we're here, then the number of cells does not equal to the amount
 // defined by the grid, even after taking care of
 // gridSpan/gridBefore/gridAfter. Handle this by ignoring the grid and
-// providing the separators based on the provided cell widths.
+// providing the separators based on the provided cell widths, as long
+// as we have a fixed layout.
 uno::Sequence< text::TableColumnSeparator > 
aSeparators(pCellWidths->size() - 1);
 text::TableColumnSeparator* pSeparators = aSeparators.getArray();
 sal_Int16 nSum = 0;
@@ -602,7 +606,7 @@ void DomainMapperTableManager::endOfRowAction()
 
 void DomainMapperTableManager::clearData()
 {
-m_nRow = m_nCellBorderIndex = m_nHeaderRepeat = m_nTableWidth = 0;
+m_nRow = m_nCellBorderIndex = m_nHeaderRepeat = m_nTableWidth = 
m_nLayoutType = 0;
 m_sTableStyleName = OUString();
 m_pTableStyleTextProperies.reset();
 }
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.hxx 
b/writerfilter/source/dmapp

[Libreoffice-commits] core.git: 2 commits - logerrit

2013-03-26 Thread Bjoern Michaelsen
 logerrit |  424 ++-
 1 file changed, 234 insertions(+), 190 deletions(-)

New commits:
commit 15e9f568fdd73bd783e89283602b4017af373a39
Author: Bjoern Michaelsen 
Date:   Tue Mar 26 15:39:55 2013 +0100

add draft-submittal to logerrit and retab

Change-Id: I5f52d9c81c28c9a4b23eaf043e7b67b1fc6d82d9

diff --git a/logerrit b/logerrit
index 7700494..99d09ac 100755
--- a/logerrit
+++ b/logerrit
@@ -5,72 +5,81 @@ GERRITHOST=logerrit
 GERRITURL=ssh://$GERRITHOST/core
 
 get_SHA_for_change() {
-   SHA=`ssh ${GERRITHOST?} gerrit query --all-approvals change:$1|grep 
ref|tail -1|cut -d: -f2`
+SHA=`ssh ${GERRITHOST?} gerrit query --all-approvals change:$1|grep 
ref|tail -1|cut -d: -f2`
 }
 
 get_REVISION_for_change() {
-   REVISION=`ssh ${GERRITHOST?} gerrit query --all-approvals 
change:$1|grep revision|tail -1|cut -d: -f2`
+REVISION=`ssh ${GERRITHOST?} gerrit query --all-approvals change:$1|grep 
revision|tail -1|cut -d: -f2`
 }
 
 ask_tristate() {
-   case $1 in
-   [yY] | [yY][Ee][Ss] | [+] )
-   ANSWER=y
-   ;;
-   [nN] | [n|N][O|o] | [-] )
-   ANSWER=n
-   ;;
-   [] )
-   ANSWER=
-   ;;
-   * )
-   echo "Please answer with either +,-,y,n,yes,no or the 
empty string."
-   exit 1
-   ;;
-   esac
+case $1 in
+[yY] | [yY][Ee][Ss] | [+] )
+ANSWER=y
+;;
+[nN] | [n|N][O|o] | [-] )
+ANSWER=n
+;;
+[] )
+ANSWER=
+;;
+* )
+echo "Please answer with either +,-,y,n,yes,no or the empty 
string."
+exit 1
+;;
+esac
 
 }
 
 submit() {
-   BRANCH=$1
-   if test -z "$BRANCH"
-   then
-   BRANCH=`git symbolic-ref HEAD 2> /dev/null`
-   BRANCH="${BRANCH##refs/heads/}"
-   if test -z "$BRANCH"
-   then
-   echo "no branch specified, and could not guess 
the current branch"
-   exit 1
-   fi
-   echo "no branch specified, guessing current branch 
$BRANCH"
-   fi
-   git push $GERRITURL HEAD:refs/for/$BRANCH
+TYPE=$1
+BRANCH=$2
+if test -z "$BRANCH"
+then
+BRANCH=`git symbolic-ref HEAD 2> /dev/null`
+BRANCH="${BRANCH##refs/heads/}"
+if test -z "$BRANCH"
+then
+echo "no branch specified, and could not guess the current 
branch"
+exit 1
+fi
+echo "no branch specified, guessing current branch $BRANCH"
+fi
+git push $GERRITURL HEAD:refs/$TYPE/$BRANCH
 }
 
 case "$1" in
-   help|--help|"")
-   echo "Usage: ./logerrit subcommand [options]"
-   echo "simple and basic tool to interact with LibreOffice gerrit"
-   echo "subcommands:"
-   echo " setup   walking you though 
your gerrit setup"
-   echo " testtest your gerrit 
setup"
-   echo " --- for submitters:"
-   echo " submit [BRANCH] submit your change 
for review"
-   echo " resubmit [BRANCH]   create a new 
Change-Id and submit your change for review"
-   echo " (yes, this modifies 
your last commit)"
-   echo " nextchange [BRANCH] reset branch to the 
remote to start with the next change"
-   echo " --- for reviewers:"
-   echo " checkout CHANGEID   checkout the changes 
for review"
-   echo " pull CHANGEID   pull (and merge) the 
changes on current branch"
-   echo " cherry-pick CHANGEIDcherry-pick the 
change on current branch"
-   echo " patch CHANGEID  show the change as a 
patch"
-   echo " review [CHANGEID]   interactively review 
a change (current one if no changeid given)"
-   echo " query   query for changes 
for review on project core"
-   echo " "
-   echo "advanced users should consider using git review instead:"
-   echo "http://wiki.documentfoundation.org/Development/GitReview";
-   exit
-   ;;
+help|--help|"")
+echo "Usage: ./logerrit subcommand [options]"
+echo "simple and basic tool to interact with LibreOffice gerrit"
+echo "subcommands:"
+echo " setup   walking you though your 
gerrit

[Libreoffice-commits] core.git: Branch 'feature/formula-core-rework' - sc/inc sc/source

2013-03-26 Thread Kohei Yoshida
 sc/inc/dociter.hxx  |1 -
 sc/source/core/data/dociter.cxx |7 ---
 2 files changed, 8 deletions(-)

New commits:
commit e90d9b09e31565ec6b1488009b7075945b78ab22
Author: Kohei Yoshida 
Date:   Tue Mar 26 10:42:06 2013 -0400

This method no longer used. Good. Remove it.

Change-Id: I397dedd6c7388b350af0f93151fdf0160698acc7

diff --git a/sc/inc/dociter.hxx b/sc/inc/dociter.hxx
index d24c6ab..eb34633 100644
--- a/sc/inc/dociter.hxx
+++ b/sc/inc/dociter.hxx
@@ -76,7 +76,6 @@ public:
 boolGetFirst();
 boolGetNext();
 
-ScBaseCell* GetCell();
 ScCellValue GetCellValue() const;
 const ScPatternAttr*GetPattern();
 voidGetPos( SCCOL& rCol, SCROW& rRow, SCTAB& rTab );
diff --git a/sc/source/core/data/dociter.cxx b/sc/source/core/data/dociter.cxx
index 24fd2b3..9f7dd02 100644
--- a/sc/source/core/data/dociter.cxx
+++ b/sc/source/core/data/dociter.cxx
@@ -197,13 +197,6 @@ bool ScDocumentIterator::GetNext()
 return GetThis();
 }
 
-//
-
-ScBaseCell* ScDocumentIterator::GetCell()
-{
-return pCell;
-}
-
 ScCellValue ScDocumentIterator::GetCellValue() const
 {
 ScCellValue aRet;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - e5/dafaf285f6406e3eca404214fb9c60ff47a4f1

2013-03-26 Thread Michael Meeks
 e5/dafaf285f6406e3eca404214fb9c60ff47a4f1 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 7fe0eabffeca418b447aaf0c3edefcba26458f6f
Author: Michael Meeks 
Date:   Tue Mar 26 14:46:14 2013 +

Notes added by 'git notes add'

diff --git a/e5/dafaf285f6406e3eca404214fb9c60ff47a4f1 
b/e5/dafaf285f6406e3eca404214fb9c60ff47a4f1
new file mode 100644
index 000..af99c65
--- /dev/null
+++ b/e5/dafaf285f6406e3eca404214fb9c60ff47a4f1
@@ -0,0 +1 @@
+merged as: 9fccdcaf810e543e28bca662300b5936ad7acf30
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Please just tell people to do './logerrit setup' to get started, also: gerrit draft changes

2013-03-26 Thread Bjoern Michaelsen
Hi all,

during the Dresden Impress Sprint, I set up a few people for gerrit and
realized that there is some needless hassle with it: mainly to dig out the wiki
page for copy pasting the .ssh/config snipplet. So with:

 
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commit;h=646b992ee306f83fa7175cc957cba9dc2277155a

you can now tell people to run './logerrit setup' which should walk them
through the steps (and that also shows a link to the wiki page, should there be
need for more information.).

Also gerrit now supports drafts: That is uploading changes that are only
visible to those explicitly added as reviewers. I extended logerrit to have:

 ./logerrit submit-draft
 ./logerrit resubmit-draft

which are the same as the submit/resubmit command in all other ways.

HTH,

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


[PUSHED] fdo#39468: Translated German comments

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2871

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I85c8675881db9facdd543c02c846ec64642f8449
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: gerhard öttl 
Gerrit-Reviewer: Michael Stahl 
Gerrit-Reviewer: Petr Mladek 
Gerrit-Reviewer: Thomas Arnhold 

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


[PUSHED] Translate German comments, clean up some WS

2013-03-26 Thread Michael Meeks (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3057

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a6a99bb8efe8c3d10973c255040fbbf0f76f27d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Philipp Weissenbacher 
Gerrit-Reviewer: Michael Meeks 

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


Re: Please just tell people to do './logerrit setup' to get started, also: gerrit draft changes

2013-03-26 Thread Michael Meeks

On Tue, 2013-03-26 at 15:52 +0100, Bjoern Michaelsen wrote:
> during the Dresden Impress Sprint, I set up a few people for gerrit and
> realized that there is some needless hassle with it: mainly to dig out the 
> wiki
> page for copy pasting the .ssh/config snipplet. So with:

Deeply sexy :-)

> you can now tell people to run './logerrit setup' which should walk them

Would you violently object if I dumbed it down yet further:

if ~/.ssh/config is not there - just writing data into it.

if ~/.ssh is not there - mentioning ssh-keygen :-) and so on ?

Thanks !

Michael.

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

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


[Libreoffice-commits] core.git: logerrit

2013-03-26 Thread Miklos Vajna
 logerrit |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1ed4210d8e6cbf1b25cd1eb8666be6578cc45845
Author: Miklos Vajna 
Date:   Tue Mar 26 16:21:37 2013 +0100

logerrit: kill copy-paste

Change-Id: I2478ef0f886867c2d2247febf1b29b90fd32e5e7

diff --git a/logerrit b/logerrit
index 99d09ac..68c254e 100755
--- a/logerrit
+++ b/logerrit
@@ -88,7 +88,7 @@ case "$1" in
 echo
 echo "Note that you need to register additional email addresses, if 
you want to"
 echo "commit from them. Additional emails must be confirmed with 
repling to the"
-echo "invitation mail it sends you.  Which user name did you choose? 
Sweetshark"
+echo "invitation mail it sends you."
 echo
 read -p 'Which user name did you choose? ' GERRITUSER
 echo
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH libreoffice-4-0] fdo#59273 import w:tblLayout of DOCX tables

2013-03-26 Thread Miklos Vajna (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3058

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/58/3058/1

fdo#59273 import w:tblLayout of DOCX tables

Also add missing  to the bnc#780645 testcase
to avoid breaking the build.

(cherry picked from commits 7329df74df134a77d078f47d5c8b70d54c5d1edb and
4821d894f849a4939bbe6fb0a9a5ddd9e10b0467)

Conflicts:
writerfilter/source/dmapper/DomainMapperTableManager.cxx
writerfilter/source/dmapper/DomainMapperTableManager.hxx

Change-Id: I9af50a4c176168e2794eaa9b0e8eaeb6b9d06b13
---
A sw/qa/extras/ooxmlimport/data/fdo59273.docx
M sw/qa/extras/ooxmlimport/data/n780645.docx
M sw/qa/extras/ooxmlimport/ooxmlimport.cxx
M writerfilter/source/dmapper/DomainMapperTableManager.cxx
M writerfilter/source/dmapper/DomainMapperTableManager.hxx
M writerfilter/source/dmapper/TablePropertiesHandler.cxx
M writerfilter/source/ooxml/model.xml
7 files changed, 45 insertions(+), 11 deletions(-)



diff --git a/sw/qa/extras/ooxmlimport/data/fdo59273.docx 
b/sw/qa/extras/ooxmlimport/data/fdo59273.docx
new file mode 100644
index 000..fcdfd77
--- /dev/null
+++ b/sw/qa/extras/ooxmlimport/data/fdo59273.docx
Binary files differ
diff --git a/sw/qa/extras/ooxmlimport/data/n780645.docx 
b/sw/qa/extras/ooxmlimport/data/n780645.docx
index 12ffa38..35ad11f 100755
--- a/sw/qa/extras/ooxmlimport/data/n780645.docx
+++ b/sw/qa/extras/ooxmlimport/data/n780645.docx
Binary files differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 51ce241..aeda333 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -115,6 +115,7 @@
 void testFdo59638();
 void testFdo61343();
 void testFdo60922();
+void testFdo59273();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX) && !defined(WNT)
@@ -182,6 +183,7 @@
 {"fdo59638.docx", &Test::testFdo59638},
 {"fdo61343.docx", &Test::testFdo61343},
 {"fdo60922.docx", &Test::testFdo60922},
+{"fdo59273.docx", &Test::testFdo59273},
 };
 for (unsigned int i = 0; i < SAL_N_ELEMENTS(aMethods); ++i)
 {
@@ -1140,6 +1142,16 @@
 CPPUNIT_ASSERT_EQUAL(sal_Int32(100), 
getProperty(getRun(getParagraph(1), 1), "CharEscapementHeight"));
 }
 
+void Test::testFdo59273()
+{
+uno::Reference xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Reference 
xTables(xTablesSupplier->getTextTables( ), uno::UNO_QUERY);
+uno::Reference xTextTable(xTables->getByIndex(0), 
uno::UNO_QUERY);
+uno::Reference xTableRows(xTextTable->getRows(), 
uno::UNO_QUERY);
+// Was 9997, so the 4th column had ~zero width
+CPPUNIT_ASSERT_EQUAL(sal_Int16(7498), getProperty< 
uno::Sequence >(xTableRows->getByIndex(0), 
"TableColumnSeparators")[2].Position);
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index d81aea7..0298690 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -51,6 +51,7 @@
 m_nTableWidth(0),
 m_bOOXML( bOOXML ),
 m_bPushCurrentWidth(false),
+m_nLayoutType(0),
 m_pTablePropsHandler( new TablePropertiesHandler( bOOXML ) )
 {
 m_pTablePropsHandler->SetTableManager( this );
@@ -395,6 +396,7 @@
 m_aTablePositions.push_back( pNewPositionHandler );
 m_nCell.push_back( 0 );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 // And push it back to the right level.
 if (oCurrentWidth)
@@ -408,6 +410,7 @@
 m_aCellWidths.pop_back( );
 m_nCell.pop_back( );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 
 DomainMapperTableManager_Base_t::endLevel( );
@@ -548,12 +551,13 @@
 #endif
 insertRowProps(pPropMap);
 }
-else if (pCellWidths->size() > 0)
+else if (pCellWidths->size() > 0 && m_nLayoutType == 
NS_ooxml::LN_Value_wordprocessingml_ST_TblLayout_fixed)
 {
 // If we're here, then the number of cells does not equal to the amount
 // defined by the grid, even after taking care of
 // gridSpan/gridBefore/gridAfter. Handle this by ignoring the grid and
-// providing the separators based on the provided cell widths.
+// providing the separators based on the provided cell widths, as long
+// as we have a fixed layout.
 uno::Sequence< text::TableColumnSeparator > 
aSeparators(pCellWidths->size() - 1);
 text::TableColumnSeparator* pSeparators = aSeparators.getArray();
 sal_Int16 nSum = 0;
@@ -593,7 +597,7 @@
 
 void DomainMapperTableManager::clearData()
 {
-m_nRow = m_nCellBorderIndex = m_nHeaderRepeat = m_nTableWidth = 0;
+m_nRow = m_nCellBorderIndex = m_nHeaderRepeat = m_nTableWidth = 
m_nLayoutType = 0;
 m_sTableStyleName = OUString();
 m_pTab

[Libreoffice-commits] core.git: 6 commits - bin/module-deps.pl config_host.mk.in configure.ac connectivity/Library_postgresql-sdbc-impl.mk forms/Module_forms.mk postgresql/ExternalProject_postgresql.m

2013-03-26 Thread Peter Foley
 RepositoryModule_host.mk   |2 +-
 bin/module-deps.pl |2 +-
 config_host.mk.in  |2 ++
 configure.ac   |   11 +++
 connectivity/Library_postgresql-sdbc-impl.mk   |   13 -
 forms/Module_forms.mk  |2 --
 postgresql/ExternalProject_postgresql.mk   |3 +--
 solenv/bin/modules/installer/epmfile.pm|6 +++---
 solenv/bin/ooinstall   |2 +-
 solenv/gbuild/extensions/pre_MergedLibsList.mk |1 +
 solenv/gbuild/gbuild.mk|2 +-
 solenv/gbuild/partial_build.mk |7 ++-
 12 files changed, 36 insertions(+), 17 deletions(-)

New commits:
commit a5b669de8431377c43909e30d9e9d01a18a5b58a
Author: Peter Foley 
Date:   Tue Mar 26 11:23:02 2013 -0400

make linkoo work in separate builddir

Change-Id: I1c9ca490716fc0dc5951bd032d54a102541087c7

diff --git a/solenv/bin/ooinstall b/solenv/bin/ooinstall
index 31025b6..b9d52e5 100755
--- a/solenv/bin/ooinstall
+++ b/solenv/bin/ooinstall
@@ -130,7 +130,7 @@ if ($ENV{BUILD_TYPE} =~ m/ODK/) {
 print "Installer finished\n";
 
 if ($do_link && !$is_windows) {
-system("$ENV{SOLARENV}/bin/linkoo $path $ENV{SRC_ROOT}") &&
+system("$ENV{SOLARENV}/bin/linkoo $path $ENV{BUILDDIR}") &&
   die "linkoo failed: $!";
 }
 
commit 207e370a167b7856346c1bff0c735cd2688e2e6e
Author: Peter Foley 
Date:   Tue Mar 26 11:10:25 2013 -0400

make partial_build work in separate builddir

Change-Id: Ib5290a3e76170946ce3a8aac113b01d1ec54fac5

diff --git a/solenv/gbuild/partial_build.mk b/solenv/gbuild/partial_build.mk
index 9c7f3fb..8107bb2 100644
--- a/solenv/gbuild/partial_build.mk
+++ b/solenv/gbuild/partial_build.mk
@@ -1,8 +1,13 @@
 ifeq ($(gb_Side),)
 gb_Side:=host
 endif
+
+ifeq (,$(BUILDDIR))
+BUILDDIR := $(dir $(realpath $(firstword $(MAKEFILE_LIST../..
+endif
+
 ifeq ($(SOLARENV),)
-include $(dir $(realpath $(lastword 
$(MAKEFILE_LIST../../config_$(gb_Side).mk
+include $(BUILDDIR)/config_$(gb_Side).mk
 endif
 
 gb_PARTIAL_BUILD := T
commit 469bb100d690d03b627dce48253a796a8f8cce68
Author: Peter Foley 
Date:   Tue Mar 26 11:02:52 2013 -0400

avoid getcwd redefined warning

Change-Id: Idd788dad8c2b5e2689a17b571104d0fe04accfd1

diff --git a/solenv/bin/modules/installer/epmfile.pm 
b/solenv/bin/modules/installer/epmfile.pm
index 7ee6d13..4ea85b6 100644
--- a/solenv/bin/modules/installer/epmfile.pm
+++ b/solenv/bin/modules/installer/epmfile.pm
@@ -18,7 +18,7 @@
 
 package installer::epmfile;
 
-use Cwd;
+use Cwd qw();
 use installer::converter;
 use installer::exiter;
 use installer::files;
@@ -,7 +,7 @@ sub set_topdir_in_specfile
 {
 my ($changefile, $filename, $newepmdir) = @_;
 
-$newepmdir = cwd() . $installer::globals::separator . $newepmdir; # only 
absolute path allowed
+$newepmdir = Cwd::cwd() . $installer::globals::separator . $newepmdir; # 
only absolute path allowed
 
 # removing "%define _topdir", if existing
 
@@ -2070,7 +2070,7 @@ sub create_packages_without_epm
 
 if ( $rpmversion >= 4 )
 {
-my $dir = getcwd;
+my $dir = Cwd::getcwd;
 my $buildroot = $dir . "/" . $epmdir . "buildroot/";
 $buildrootstring = "--buildroot=$buildroot";
 mkdir($buildroot = $dir . "/" . $epmdir . "BUILD/");
commit ea360ff3ad9dac359f707cee5f0ca6548a7cd5d5
Author: Peter Foley 
Date:   Tue Mar 26 11:02:06 2013 -0400

add frm to Library_merged

Change-Id: I72fff5fe1fd26e618ffe89111471d95cd4d30852

diff --git a/RepositoryModule_host.mk b/RepositoryModule_host.mk
index e28c9ae..a2be8b8 100644
--- a/RepositoryModule_host.mk
+++ b/RepositoryModule_host.mk
@@ -79,7 +79,7 @@ $(eval $(call gb_Module_add_moduledirs,libreoffice,\
fileaccess \
filter \
$(call gb_Helper_optional,FONTCONFIG,fontconfig) \
-   forms \
+   $(call gb_Helper_optional,DBCONNECTIVITY,forms) \
formula \
fpicker \
framework \
diff --git a/forms/Module_forms.mk b/forms/Module_forms.mk
index 7480767..bd64ac1 100644
--- a/forms/Module_forms.mk
+++ b/forms/Module_forms.mk
@@ -28,7 +28,6 @@
 
 $(eval $(call gb_Module_Module,forms))
 
-ifneq (,$(filter DBCONNECTIVITY,$(BUILD_TYPE)))
 $(eval $(call gb_Module_add_targets,forms,\
 AllLangResTarget_frm \
 Library_frm \
@@ -38,6 +37,5 @@ $(eval $(call gb_Module_add_subsequentcheck_targets,forms,\
 JunitTest_forms_unoapi \
 JunitTest_forms_complex \
 ))
-endif
 
 # vim: set noet sw=4 ts=4:
diff --git a/solenv/gbuild/extensions/pre_MergedLibsList.mk 
b/solenv/gbuild/extensions/pre_MergedLibsList.mk
index 117c0df..2217014 100644
--- a/solenv/gbuild/extensions/pre_MergedLibsList.mk
+++ b/solenv/gbuild/extensions/pre_MergedLibsList.mk
@@ -45,6 +45,7 @@ gb_MERGEDLIBS := \
filterconfig \
for \
forui \
+   $(

Re: Please just tell people to do './logerrit setup' to get started, also: gerrit draft changes

2013-03-26 Thread Bjoern Michaelsen
Hi Michael,

On Tue, Mar 26, 2013 at 03:11:37PM +, Michael Meeks wrote:
>   Would you violently object if I dumbed it down yet further:
> 
>   if ~/.ssh/config is not there - just writing data into it.

No objection there, although the ~/.ssh/id_rsa path that it spits out is just a
guess right now.

>   if ~/.ssh is not there - mentioning ssh-keygen :-) and so on ?

ssh-keygen and all that jazz is already mentioned and linked on the gerrit
setup page where you upload your key (linking to githubs excellent tutorial),
so mentioning it again wont help much.

What might help though would be for './logerrit setup':
- to call ssh-keygen from the setup script itself, creating a 
~/.ssh/id_logerrit key
- then printing the public key for copy-paste in the gerrit setup page
- and then tweak ~/.ssh/config (after all we now know the path to the key for 
sure)

Best,

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


[Libreoffice-commits] core.git: 2 commits - sc/source sw/source

2013-03-26 Thread Lior Kaplan
 sc/source/ui/view/tabvwsha.cxx |   28 +++-
 sw/source/ui/wrtsh/move.cxx|7 +++
 sw/source/ui/wrtsh/select.cxx  |2 +-
 3 files changed, 19 insertions(+), 18 deletions(-)

New commits:
commit 447a924c115a86e96e91b24d744af70f97efde72
Author: Lior Kaplan 
Date:   Tue Mar 26 17:54:09 2013 +0200

Translate German comments to English

Change-Id: Iaedc226f0dfad234bc67100e2845e09cc2976687

diff --git a/sc/source/ui/view/tabvwsha.cxx b/sc/source/ui/view/tabvwsha.cxx
index 979591a..5d8e622 100644
--- a/sc/source/ui/view/tabvwsha.cxx
+++ b/sc/source/ui/view/tabvwsha.cxx
@@ -102,7 +102,7 @@ sal_Bool ScTabViewShell::GetFunction( String& rFuncStr, 
sal_uInt16 nErrCode )
 aStr += '0';
 else
 {
-//  Anzahl im Standardformat, die anderen nach Cursorposition
+// Number in the standard format, the other on the cursor 
position
 SvNumberFormatter* pFormatter = pDoc->GetFormatTable();
 sal_uInt32 nNumFmt = 0;
 if ( eFunc != SUBTOTAL_FUNC_CNT && eFunc != SUBTOTAL_FUNC_CNT2 
)
@@ -137,7 +137,7 @@ sal_Bool ScTabViewShell::GetFunction( String& rFuncStr, 
sal_uInt16 nErrCode )
 
 
 
-//  Funktionen, die je nach Selektion disabled sind
+//  Functions that are disabled, depending on the selection
 //  Default:
 //  SID_DELETE,
 //  SID_DELETE_CONTENTS,
@@ -223,8 +223,10 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
 
 case SID_SEARCH_OPTIONS:
 {
-sal_uInt16 nOptions = 0x;   // alles erlaubt
-// wenn ReadOnly, kein 
Ersetzen:
+// Anything goes
+sal_uInt16 nOptions = 0x;
+
+// No replacement if ReadOnly
 if (GetViewData()->GetDocShell()->IsReadOnly())
 nOptions &= ~( SEARCH_OPTIONS_REPLACE | 
SEARCH_OPTIONS_REPLACE_ALL );
 rSet.Put( SfxUInt16Item( nWhich, nOptions ) );
@@ -243,7 +245,7 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
 break;
 
 case SID_CURRENTTAB:
-//  Tabelle fuer Basic ist 1-basiert
+// Table for Basic is 1-based
 rSet.Put( SfxUInt16Item( nWhich, 
static_cast(GetViewData()->GetTabNo()) + 1 ) );
 break;
 
@@ -272,7 +274,7 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
 
 case FID_RESET_PRINTZOOM:
 {
-//  disablen, wenn schon Default eingestellt
+// disable if already set to default
 
 String aStyleName = pDoc->GetPageStyle( nTab );
 ScStyleSheetPool* pStylePool = pDoc->GetStyleSheetPool();
@@ -441,13 +443,13 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
 break;
 
 case SID_FORMATPAGE:
-//! bei geschuetzten Tabellen ???
+// in protected tables
 if ( pDocShell && ( pDocShell->IsReadOnly() || 
pDocShell->IsDocShared() ) )
 rSet.DisableItem( nWhich );
 break;
 
 case SID_PRINTPREVIEW:
-// Toggle-Slot braucht einen State
+// Toggle slot needs a State
 rSet.Put( SfxBoolItem( nWhich, false ) );
 break;
 
@@ -480,7 +482,7 @@ void ScTabViewShell::ExecuteCellFormatDlg( SfxRequest& 
rReq, sal_uInt16 nTabPage
 pOldAttrs->GetItemSet() );
 
 
-// Umrandungs-Items holen und in den Set packen:
+// Get border items and put them in the set:
 GetSelectionFrame( aLineOuter, aLineInner );
 //Fix border incorrect for RTL fdo#62399
 if( pDoc->IsLayoutRTL( GetViewData()->GetTabNo() ) )
@@ -502,7 +504,7 @@ void ScTabViewShell::ExecuteCellFormatDlg( SfxRequest& 
rReq, sal_uInt16 nTabPage
 
 pOldSet->Put( aLineInner );
 
-// NumberFormat Value aus Value und Language erzeugen und eintueten
+// Generate NumberFormat Value from Value and Language and box it.
 pOldSet->Put( SfxUInt32Item( ATTR_VALUE_FORMAT,
 pOldAttrs->GetNumberFormat( pDoc->GetFormatTable() ) ) );
 
@@ -708,9 +710,9 @@ void ScTabViewShell::UpdateInputHandlerCellAdjust( 
SvxCellHorJustify eJust )
 
 void ScTabViewShell::ExecuteSave( SfxRequest& rReq )
 {
-//  nur SID_SAVEDOC / SID_SAVEASDOC
+// only SID_SAVEDOC / SID_SAVEASDOC
 
-// Eingabe auf jeden Fall abschliessen, auch wenn eine Formel bearbeitet 
wird
+// Finish entering in any case, even if a formula is being processed
 SC_MOD()->InputEnterHandler();
 
 if ( GetViewData()->GetDocShell()->IsDocShared() )
@@ -718,7 +720,7 @@ void ScTabViewShell::ExecuteSave( SfxRequest& rReq )
 GetViewData()->GetDocShell()->SetDocumentModified();
 }
 
-// ansonsten normal weiter
+// otherwise a

[Libreoffice-commits] core.git: Branch 'feature/formula-core-rework' - sc/inc sc/source

2013-03-26 Thread Kohei Yoshida
 sc/inc/cell.hxx   |1 
 sc/inc/cellvalue.hxx  |   56 ++
 sc/inc/column.hxx |1 
 sc/inc/document.hxx   |   11 +
 sc/inc/table.hxx  |1 
 sc/source/core/data/cellvalue.cxx |  309 --
 sc/source/core/data/column3.cxx   |   12 +
 sc/source/core/data/document.cxx  |8 
 sc/source/core/data/table2.cxx|7 
 9 files changed, 326 insertions(+), 80 deletions(-)

New commits:
commit 0aa3d9c489effcc9ab01946ab9cea846bbab1c39
Author: Kohei Yoshida 
Date:   Tue Mar 26 12:00:43 2013 -0400

Add ScRefCellValue, which is ScCellValue without copied value.

It directly points to the original cell value instance.

Change-Id: I638ec8b931873d237871b6d8fa9f0e1277520d0f

diff --git a/sc/inc/cell.hxx b/sc/inc/cell.hxx
index 236ad43..0b78f98 100644
--- a/sc/inc/cell.hxx
+++ b/sc/inc/cell.hxx
@@ -175,6 +175,7 @@ public:
 
 inline void SetString( const rtl::OUString& rString ) { maString = 
rString; }
 inline constrtl::OUString& GetString() const { return maString; }
+const OUString* GetStringPtr() const { return &maString; }
 
 private:
 rtl::OUString   maString;
diff --git a/sc/inc/cellvalue.hxx b/sc/inc/cellvalue.hxx
index 9dd3a9a..7998898 100644
--- a/sc/inc/cellvalue.hxx
+++ b/sc/inc/cellvalue.hxx
@@ -19,7 +19,8 @@ class ScBaseCell;
 
 /**
  * Store arbitrary cell value of any kind.  It only stores cell value and
- * nothing else.
+ * nothing else.  It creates a copy of the original cell value, and manages
+ * the life cycle of the copied value.
  */
 struct SC_DLLPUBLIC ScCellValue
 {
@@ -78,6 +79,59 @@ struct SC_DLLPUBLIC ScCellValue
 void swap( ScCellValue& r );
 };
 
+/**
+ * This is very similar to ScCellValue, except that it points to the
+ * original value instead of copying it.  As such, don't hold an instance of
+ * this class any longer than necessary.
+ */
+struct SC_DLLPUBLIC ScRefCellValue
+{
+CellType meType;
+union {
+double mfValue;
+const OUString* mpString;
+const EditTextObject* mpEditText;
+ScFormulaCell* mpFormula;
+};
+
+ScRefCellValue();
+ScRefCellValue( double fValue );
+ScRefCellValue( const OUString* pString );
+ScRefCellValue( const EditTextObject* pEditText );
+ScRefCellValue( ScFormulaCell* pFormula );
+ScRefCellValue( const ScRefCellValue& r );
+~ScRefCellValue();
+
+void clear();
+
+/**
+ * Take cell value from specified position in specified document.
+ */
+void assign( ScDocument& rDoc, const ScAddress& rPos );
+
+/**
+ * TODO: Remove this later.
+ */
+void assign( ScBaseCell& rCell );
+
+/**
+ * Set cell value at specified position in specified document.
+ */
+void commit( ScDocument& rDoc, const ScAddress& rPos ) const;
+
+bool hasString() const;
+
+bool hasNumeric() const;
+
+bool isEmpty() const;
+
+bool equalsWithoutFormat( const ScRefCellValue& r ) const;
+
+ScRefCellValue& operator= ( const ScRefCellValue& r );
+
+void swap( ScRefCellValue& r );
+};
+
 #endif
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/inc/column.hxx b/sc/inc/column.hxx
index f25dc78..ac329b2 100644
--- a/sc/inc/column.hxx
+++ b/sc/inc/column.hxx
@@ -272,6 +272,7 @@ public:
 voidSetError( SCROW nRow, const sal_uInt16 nError);
 
 voidGetString( SCROW nRow, rtl::OUString& rString ) const;
+const OUString* GetStringCell( SCROW nRow ) const;
 voidGetInputString( SCROW nRow, rtl::OUString& rString ) const;
 double  GetValue( SCROW nRow ) const;
 const EditTextObject* GetEditText( SCROW nRow ) const;
diff --git a/sc/inc/document.hxx b/sc/inc/document.hxx
index 70c5620..892d18f 100644
--- a/sc/inc/document.hxx
+++ b/sc/inc/document.hxx
@@ -802,6 +802,17 @@ public:
 
 SC_DLLPUBLIC OUString GetString( SCCOL nCol, SCROW nRow, SCTAB nTab ) 
const;
 OUString GetString( const ScAddress& rPos ) const;
+
+/**
+ * Return a pointer to the string object stored in string cell.
+ *
+ * @param rPos cell position.
+ *
+ * @return pointer to the string object stored in string cell, or NULL if
+ * the cell at specified position is not a string cell. Note that
+ * it returns NULL even for a edit cell.
+ */
+const OUString* GetStringCell( const ScAddress& rPos ) const;
 SC_DLLPUBLIC void   GetInputString( SCCOL nCol, SCROW nRow, SCTAB 
nTab, String& rString );
 SC_DLLPUBLIC void   GetInputString( SCCOL nCol, SCROW nRow, SCTAB 
nTab, rtl::OUString& rString );
 sal_uInt16  GetStringForFormula( const ScAddress& rPos, 
rtl::OUString& rString );
diff --git a/sc/inc/table.hxx b/sc/inc/table.hxx
index 461da522..0a12ead 100644
--- a/sc/inc/table.hxx
+++ b/sc/inc/table.hxx
@@ -318,6 +318,7 @@ public:
 voidSetError( SCCOL nCol, SCROW nRow, sal

[Libreoffice-commits] core.git: apache-commons/ExternalProject_apache_commons_httpclient.mk apache-commons/Package_apache_commons_codec.mk apache-commons/Package_apache_commons_lang.mk apache-commons/

2013-03-26 Thread Pedro Giffuni
 apache-commons/ExternalProject_apache_commons_httpclient.mk |2 
 apache-commons/Package_apache_commons_codec.mk  |2 
 apache-commons/Package_apache_commons_lang.mk   |2 
 apache-commons/UnpackedTarball_apache_commons_httpclient.mk |5 
 apache-commons/patches/codec.patch  |   23 +-
 apache-commons/patches/httpclient.patch |  119 
 apache-commons/patches/httpclient_no_javadoc.patch  |   11 +
 download.lst|4 
 swext/Extension_wiki-publisher.mk   |4 
 9 files changed, 155 insertions(+), 17 deletions(-)

New commits:
commit adda54b5211916f9b442e8ac7d82cc861bdde54d
Author: Pedro Giffuni 
Date:   Wed May 9 22:22:11 2012 +

Update Apache Commons

Update commons-codec to version 1.6.
Update commons-lang to version 2.4.
Patch common-httpclient to avoid building issues
with OpenJDK-7.

These versions maintain ABI compatibility while
using features from newer Java versions and
have been tested on the FreeBSD port.

Change-Id: I6aec2b6d22fc3335ffbdda9558f475313ef53c85

diff --git a/apache-commons/ExternalProject_apache_commons_httpclient.mk 
b/apache-commons/ExternalProject_apache_commons_httpclient.mk
index 5b67c35..aaf1520 100644
--- a/apache-commons/ExternalProject_apache_commons_httpclient.mk
+++ b/apache-commons/ExternalProject_apache_commons_httpclient.mk
@@ -26,7 +26,7 @@ $(call 
gb_ExternalProject_get_state_target,apache_commons_httpclient,build) :
-f build.xml \

-Dbuild.label="build-$(LIBO_VERSION_MAJOR).$(LIBO_VERSION_MINOR).$(LIBO_VERSION_MICRO).$(LIBO_VERSION_PATCH)"
 \
-Dcommons-logging.jar="$(call 
gb_UnpackedTarball_get_dir,apache_commons_logging)/target/commons-logging-1.1.1-SNAPSHOT.jar"
 \
-   -Dcommons-codec.jar="$(call 
gb_UnpackedTarball_get_dir,apache_commons_codec)/dist/commons-codec-1.3.jar" \
+   -Dcommons-codec.jar="$(call 
gb_UnpackedTarball_get_dir,apache_commons_codec)/dist/commons-codec-1.6-SNAPSHOT.jar"
 \
$(if $(filter yes,$(JAVACISGCJ))\
,-Dbuild.compiler=gcj \
,-Dant.build.javac.source=$(JAVA_SOURCE_VER) \
diff --git a/apache-commons/Package_apache_commons_codec.mk 
b/apache-commons/Package_apache_commons_codec.mk
index aaf3b7f..65d302b 100644
--- a/apache-commons/Package_apache_commons_codec.mk
+++ b/apache-commons/Package_apache_commons_codec.mk
@@ -11,6 +11,6 @@ $(eval $(call 
gb_Package_Package,apache_commons_codec_inc,$(call gb_UnpackedTarb
 
 $(eval $(call 
gb_Package_use_external_project,apache_commons_codec_inc,apache_commons_codec))
 
-$(eval $(call 
gb_Package_add_file,apache_commons_codec_inc,bin/commons-codec-1.3.jar,dist/commons-codec-1.3.jar))
+$(eval $(call 
gb_Package_add_file,apache_commons_codec_inc,bin/commons-codec-1.6.jar,dist/commons-codec-1.6-SNAPSHOT.jar))
 
 # vim: set noet sw=4 ts=4:
diff --git a/apache-commons/Package_apache_commons_lang.mk 
b/apache-commons/Package_apache_commons_lang.mk
index 8f93b9be..a9c1eb3 100644
--- a/apache-commons/Package_apache_commons_lang.mk
+++ b/apache-commons/Package_apache_commons_lang.mk
@@ -11,6 +11,6 @@ $(eval $(call 
gb_Package_Package,apache_commons_lang_inc,$(call gb_UnpackedTarba
 
 $(eval $(call 
gb_Package_use_external_project,apache_commons_lang_inc,apache_commons_lang))
 
-$(eval $(call 
gb_Package_add_file,apache_commons_lang_inc,bin/commons-lang-2.3.jar,dist/commons-lang-2.3.jar))
+$(eval $(call 
gb_Package_add_file,apache_commons_lang_inc,bin/commons-lang-2.4.jar,dist/commons-lang-2.4.jar))
 
 # vim: set noet sw=4 ts=4:
diff --git a/apache-commons/UnpackedTarball_apache_commons_httpclient.mk 
b/apache-commons/UnpackedTarball_apache_commons_httpclient.mk
index a68c071..6139a94 100644
--- a/apache-commons/UnpackedTarball_apache_commons_httpclient.mk
+++ b/apache-commons/UnpackedTarball_apache_commons_httpclient.mk
@@ -11,4 +11,9 @@ $(eval $(call 
gb_UnpackedTarball_UnpackedTarball,apache_commons_httpclient))
 
 $(eval $(call 
gb_UnpackedTarball_set_tarball,apache_commons_httpclient,$(APACHE_COMMONS_HTTPCLIENT_TARBALL)))
 
+$(eval $(call gb_UnpackedTarball_add_patches,apache_commons_httpclient,\
+   apache-commons/patches/httpclient.patch \
+   apache-commons/patches/httpclient_no_javadoc.patch \
+))
+
 # vim: set noet sw=4 ts=4:
diff --git a/apache-commons/patches/codec.patch 
b/apache-commons/patches/codec.patch
index ffbcb57..7a9b489 100644
--- a/apache-commons/patches/codec.patch
+++ b/apache-commons/patches/codec.patch
@@ -1,17 +1,20 @@
 misc/build/commons-codec-1.3/build.xml-old 2008-01-31 15:28:58.0 
+0100
-+++ misc/build/commons-codec-1.3/build.xml 2008-01-31 15:29:21.0 
+0100
-@@ -90,14 +90,12 @@
+--- misc/build/commons-codec-1.6-src/build.xml-old 2012-05-05 
22:12:32.0 -0500
 misc/build/commo

[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - ef/867d46563915bce0439c2d6a741175baadf6d0

2013-03-26 Thread Fridrich Štrba
 ef/867d46563915bce0439c2d6a741175baadf6d0 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 440b487a0d1b34e690d50da506c4ad0d669e5ab4
Author: Fridrich Å trba 
Date:   Tue Mar 26 17:14:24 2013 +0100

Notes added by 'git notes add'

diff --git a/ef/867d46563915bce0439c2d6a741175baadf6d0 
b/ef/867d46563915bce0439c2d6a741175baadf6d0
new file mode 100644
index 000..719549e
--- /dev/null
+++ b/ef/867d46563915bce0439c2d6a741175baadf6d0
@@ -0,0 +1 @@
+merged as: adda54b5211916f9b442e8ac7d82cc861bdde54d
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[ABANDONED] Use an 8bpp virtual device (instead of a 1bpp one) in sw and...

2013-03-26 Thread Tor Lillqvist (via Code Review)
Tor Lillqvist has abandoned this change.

Change subject: Use an 8bpp virtual device (instead of a 1bpp one) in sw and sc
..


Patch Set 3: Abandoned

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: Ia34927cf728d4be05a31e88b7da78200d0b799ba
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Tor Lillqvist 
Gerrit-Reviewer: Christoph Brill 
Gerrit-Reviewer: Tor Lillqvist 

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


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 2e/d8ccfd4348df1e860477cdd093d780ca02e5da

2013-03-26 Thread Fridrich Štrba
 2e/d8ccfd4348df1e860477cdd093d780ca02e5da |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 1e53706339af59a1f4bdbb36da2f89dd18cb13c0
Author: Fridrich Å trba 
Date:   Tue Mar 26 17:16:08 2013 +0100

Notes added by 'git notes add'

diff --git a/2e/d8ccfd4348df1e860477cdd093d780ca02e5da 
b/2e/d8ccfd4348df1e860477cdd093d780ca02e5da
new file mode 100644
index 000..8ebbe55
--- /dev/null
+++ b/2e/d8ccfd4348df1e860477cdd093d780ca02e5da
@@ -0,0 +1 @@
+ignore: obsolete
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Fix template manager item selection behaviour with mouse.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3053


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icf63454973b403eea1cc583fce3a78c5594d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 
Gerrit-Reviewer: Bosdonnat Cedric 

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


[PUSHED] Syncronize thumbnails ids with SfxDocumentTemplates when mov...

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3054


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9ee17e00f769ca16f566b1397d09b76a09273912
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 
Gerrit-Reviewer: Bosdonnat Cedric 

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


[PUSHED] Merge TemplateView into TemplateAbstractView class.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3051


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iafc246eeff4c50c12ac9ebec9fe95dcc84991e9e
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 
Gerrit-Reviewer: Bosdonnat Cedric 

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


[PUSHED] Deselect thumbnailview items when clearing the view.

2013-03-26 Thread Bosdonnat Cedric (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3052


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8aa3f61d3ec7a8bf6195c5480b82d1dc1ecabc34
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rafael Dominguez 
Gerrit-Reviewer: Bosdonnat Cedric 

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


[Libreoffice-commits] core.git: 4 commits - config_host.mk.in connectivity/inc connectivity/source dbaccess/source i18npool/source l10ntools/source Makefile.in solenv/gbuild

2013-03-26 Thread Stephan Bergmann
 Makefile.in  |2 +-
 config_host.mk.in|1 -
 connectivity/inc/connectivity/CommonTools.hxx|3 ++-
 connectivity/inc/connectivity/FValue.hxx |1 +
 connectivity/source/commontools/CommonTools.cxx  |1 +
 dbaccess/source/filter/xml/xmlfilter.cxx |1 +
 dbaccess/source/ui/dlg/ConnectionPage.cxx|1 +
 i18npool/source/textconversion/textconversion_ko.cxx |4 +---
 l10ntools/source/cfglex.l|2 +-
 l10ntools/source/srclex.l|2 +-
 l10ntools/source/xrmlex.l|2 +-
 solenv/gbuild/platform/com_GCC_defs.mk   |4 ++--
 solenv/gbuild/platform/com_MSC_defs.mk   |4 ++--
 13 files changed, 15 insertions(+), 13 deletions(-)

New commits:
commit cd235e389d6ea2c9d9c8b15722659d9c3d5bf0a3
Author: Stephan Bergmann 
Date:   Tue Mar 26 17:21:14 2013 +0100

No (more/apparent) need for LIBO_WERROR

Change-Id: I8483047631f512513cae017fdbb2dee77ce0f52f

diff --git a/i18npool/source/textconversion/textconversion_ko.cxx 
b/i18npool/source/textconversion/textconversion_ko.cxx
index f20a1e6..d9fe16c 100644
--- a/i18npool/source/textconversion/textconversion_ko.cxx
+++ b/i18npool/source/textconversion/textconversion_ko.cxx
@@ -137,7 +137,6 @@ TextConversion_ko::getCharConversions(const OUString& 
aText, sal_Int32 nStartPos
 const sal_uInt16* (*getHanja2HangulIndex)() = (const sal_uInt16* (*)()) 
getFunctionBySymbol("getHanja2HangulIndex");
 const sal_Unicode* (*getHanja2HangulData)() = (const sal_Unicode* (*)()) 
getFunctionBySymbol("getHanja2HangulData");
 #else
-#ifdef LIBO_WERROR
 #pragma GCC diagnostic push
 #ifdef __clang__
 #pragma GCC diagnostic warning "-Wbool-conversions"
@@ -145,7 +144,6 @@ TextConversion_ko::getCharConversions(const OUString& 
aText, sal_Int32 nStartPos
 #pragma GCC diagnostic warning "-Waddress"
 #endif
 #endif
-#endif
 if (toHanja && getHangul2HanjaIndex && getHangul2HanjaIndexCount && 
getHangul2HanjaData) {
 ch = aText[nStartPos];
 const Hangul_Index *Hangul_ko = getHangul2HanjaIndex();
@@ -192,7 +190,7 @@ TextConversion_ko::getCharConversions(const OUString& 
aText, sal_Int32 nStartPos
 }
 delete[] newStr;
 }
-#if defined(DISABLE_DYNLOADING) && defined(LIBO_WERROR)
+#if defined(DISABLE_DYNLOADING)
 #pragma GCC diagnostic pop
 #endif
 return output;
diff --git a/solenv/gbuild/platform/com_GCC_defs.mk 
b/solenv/gbuild/platform/com_GCC_defs.mk
index d20b9f7..a695a2f 100644
--- a/solenv/gbuild/platform/com_GCC_defs.mk
+++ b/solenv/gbuild/platform/com_GCC_defs.mk
@@ -100,8 +100,8 @@ endif
 endif
 
 ifneq ($(EXTERNAL_WARNINGS_NOT_ERRORS),TRUE)
-gb_CFLAGS_WERROR := -Werror -DLIBO_WERROR
-gb_CXXFLAGS_WERROR := -Werror -DLIBO_WERROR
+gb_CFLAGS_WERROR := -Werror
+gb_CXXFLAGS_WERROR := -Werror
 endif
 
 ifeq ($(MERGELIBS),TRUE)
diff --git a/solenv/gbuild/platform/com_MSC_defs.mk 
b/solenv/gbuild/platform/com_MSC_defs.mk
index 2466422..e8488ef 100644
--- a/solenv/gbuild/platform/com_MSC_defs.mk
+++ b/solenv/gbuild/platform/com_MSC_defs.mk
@@ -233,8 +233,8 @@ gb_STDLIBS := \
advapi32.lib \
 
 ifneq ($(EXTERNAL_WARNINGS_NOT_ERRORS),TRUE)
-gb_CFLAGS_WERROR := -WX -DLIBO_WERROR
-gb_CXXFLAGS_WERROR := -WX -DLIBO_WERROR
+gb_CFLAGS_WERROR := -WX
+gb_CXXFLAGS_WERROR := -WX
 endif
 
 ifeq ($(MERGELIBS),TRUE)
commit 6ecf41eb2dcc4f15399cc8dc78547a517294781f
Author: Stephan Bergmann 
Date:   Tue Mar 26 17:21:01 2013 +0100

Typo __SINPRO_CC -> __SUNPRO_CC

Change-Id: I831abe80f9b4b160f4af10a16f4d45cc68fa3bb6

diff --git a/l10ntools/source/cfglex.l b/l10ntools/source/cfglex.l
index ea9e7b4..0ecb314 100644
--- a/l10ntools/source/cfglex.l
+++ b/l10ntools/source/cfglex.l
@@ -48,7 +48,7 @@
 #if (__GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 6))
 #pragma GCC diagnostic ignored "-Wunused-but-set-variable"
 #endif
-#elif defined __SINPRO_CC
+#elif defined __SUNPRO_CC
 #pragma disable_warn
 #elif defined _MSC_VER
 #pragma warning(push, 1)
diff --git a/l10ntools/source/srclex.l b/l10ntools/source/srclex.l
index 6716b1b..2826dce 100644
--- a/l10ntools/source/srclex.l
+++ b/l10ntools/source/srclex.l
@@ -49,7 +49,7 @@
 #if (__GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 6))
 #pragma GCC diagnostic ignored "-Wunused-but-set-variable"
 #endif
-#elif defined __SINPRO_CC
+#elif defined __SUNPRO_CC
 #pragma disable_warn
 #elif defined _MSC_VER
 #pragma warning(push, 1)
diff --git a/l10ntools/source/xrmlex.l b/l10ntools/source/xrmlex.l
index 7513edd..a2b5c0c 100644
--- a/l10ntools/source/xrmlex.l
+++ b/l10ntools/source/xrmlex.l
@@ -48,7 +48,7 @@
 #if (__GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 6))
 #pragma GCC diagnostic ignored "-Wunused-but-set-variable"
 #endif
-#elif defined __SINPRO_CC
+#elif defined __SUNPRO_CC
 #pragma disable_warn
 #elif defined _MSC_VER
 #pragma warning(p

[PATCH libreoffice-4-0] fix iint/iiint import from docx math

2013-03-26 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3059

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/59/3059/1

fix iint/iiint import from docx math

Use the correct LO names for the symbols.

Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29
---
M starmath/source/ooxmlimport.cxx
1 file changed, 2 insertions(+), 2 deletions(-)



diff --git a/starmath/source/ooxmlimport.cxx b/starmath/source/ooxmlimport.cxx
index 0b1beda..e3a48ca 100644
--- a/starmath/source/ooxmlimport.cxx
+++ b/starmath/source/ooxmlimport.cxx
@@ -530,10 +530,10 @@
 ret = "int";
 break;
 case MS_IINT:
-ret = "liint";
+ret = "iint";
 break;
 case MS_IIINT:
-ret = "liiint";
+ret = "iiint";
 break;
 case MS_LINT:
 ret = "lint";

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4da760fb84e6f9ca811e13061341f9a993342c29
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Luboš Luňák 

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


[PATCH libreoffice-4-0] m:nor and m:lit math docx tags lead to quoting

2013-03-26 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3060

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/60/3060/1

m:nor and m:lit math docx tags lead to quoting

The meaning should not be exactly the same, but it looks like there's
no better way to express this (or rather quoting means both).

Change-Id: I024ce6c89667a69f685bb84d768095100b0090b1
---
M starmath/source/ooxmlimport.cxx
1 file changed, 17 insertions(+), 5 deletions(-)



diff --git a/starmath/source/ooxmlimport.cxx b/starmath/source/ooxmlimport.cxx
index e3a48ca..648f808 100644
--- a/starmath/source/ooxmlimport.cxx
+++ b/starmath/source/ooxmlimport.cxx
@@ -570,11 +570,21 @@
 OUString SmOoxmlImport::handleR()
 {
 stream.ensureOpeningTag( M_TOKEN( r ));
-if( XmlStream::Tag rPr = stream.checkOpeningTag( OOX_TOKEN( doc, rPr )))
-{ // TODO
-//stream.checkOpeningTag( OOX_TOKEN( doc, rFonts ));
-//stream.ensureClosingTag( OOX_TOKEN( doc, rFonts ));
-stream.ensureClosingTag( OOX_TOKEN( doc, rPr ));
+bool normal = false;
+bool literal = false;
+if( XmlStream::Tag rPr = stream.checkOpeningTag( M_TOKEN( rPr )))
+{
+if( XmlStream::Tag litTag = stream.checkOpeningTag( M_TOKEN( lit )))
+{
+literal = litTag.attribute( M_TOKEN( val ), true );
+stream.ensureClosingTag( M_TOKEN( lit ));
+}
+if( XmlStream::Tag norTag = stream.checkOpeningTag( M_TOKEN( nor )))
+{
+normal = norTag.attribute( M_TOKEN( val ), true );
+stream.ensureClosingTag( M_TOKEN( nor ));
+}
+stream.ensureClosingTag( M_TOKEN( rPr ));
 }
 OUString text;
 while( !stream.atEnd() && stream.currentToken() != CLOSING( 
stream.currentToken()))
@@ -597,6 +607,8 @@
 }
 }
 stream.ensureClosingTag( M_TOKEN( r ));
+if( normal || literal )
+text = "\"" + text + "\"";
 return text.replaceAll("{", "\\{").replaceAll("}", "\\}");
 }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I024ce6c89667a69f685bb84d768095100b0090b1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Luboš Luňák 

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


[PATCH libreoffice-4-0] export quoted parts of math formulas properly to docx mathml

2013-03-26 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3061

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/61/3061/1

export quoted parts of math formulas properly to docx mathml

Change-Id: I7950ff78c23e5e6c58ffceaa9c2a2f52e3408b26
---
M starmath/source/ooxmlexport.cxx
1 file changed, 7 insertions(+), 0 deletions(-)



diff --git a/starmath/source/ooxmlexport.cxx b/starmath/source/ooxmlexport.cxx
index a918cb2..22f813c 100644
--- a/starmath/source/ooxmlexport.cxx
+++ b/starmath/source/ooxmlexport.cxx
@@ -75,6 +75,13 @@
 {
 m_pSerializer->startElementNS( XML_m, XML_r, FSEND );
 
+if( pNode->GetToken().eType == TTEXT ) // literal text (in quotes)
+{
+m_pSerializer->startElementNS( XML_m, XML_rPr, FSEND );
+m_pSerializer->singleElementNS( XML_m, XML_lit, FSEND );
+m_pSerializer->singleElementNS( XML_m, XML_nor, FSEND );
+m_pSerializer->endElementNS( XML_m, XML_rPr );
+}
 if( version == ECMA_DIALECT )
 { // HACK: MSOffice2007 does not import characters properly unless this 
font is explicitly given
 m_pSerializer->startElementNS( XML_w, XML_rPr, FSEND );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7950ff78c23e5e6c58ffceaa9c2a2f52e3408b26
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Luboš Luňák 

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


[Libreoffice-commits] core.git: cui/source cui/uiconfig cui/UI_cui.mk

2013-03-26 Thread Krisztian Pinter
 cui/UI_cui.mk  |1 
 cui/source/options/optfltr.cxx |   88 ++---
 cui/source/options/optfltr.hrc |   13 --
 cui/source/options/optfltr.hxx |   19 +-
 cui/source/options/optfltr.src |   94 --
 cui/uiconfig/ui/optfltrpage.ui |  260 +
 6 files changed, 311 insertions(+), 164 deletions(-)

New commits:
commit 2c1da7f081b581472df761384743c9ca223ba0a3
Author: Krisztian Pinter 
Date:   Sun Mar 24 12:50:00 2013 +0100

convert half of optfltr.src to .ui

Converted the "VBA Properties" half of optfltr.src to optfltrpage.ui.

Change-Id: Ie200377f2956aa243b1be859eb52972b79e9dc66
Reviewed-on: https://gerrit.libreoffice.org/2961
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 

diff --git a/cui/UI_cui.mk b/cui/UI_cui.mk
index fff3fda..c3b6fe7 100644
--- a/cui/UI_cui.mk
+++ b/cui/UI_cui.mk
@@ -37,6 +37,7 @@ $(eval $(call gb_UI_add_uifiles,cui,\
cui/uiconfig/ui/optaccessibilitypage \
cui/uiconfig/ui/optadvancedpage \
cui/uiconfig/ui/optappearancepage \
+   cui/uiconfig/ui/optfltrpage \
cui/uiconfig/ui/optfontspage \
cui/uiconfig/ui/optgeneralpage \
cui/uiconfig/ui/optmemorypage \
diff --git a/cui/source/options/optfltr.cxx b/cui/source/options/optfltr.cxx
index df2d57e..2271fab 100644
--- a/cui/source/options/optfltr.cxx
+++ b/cui/source/options/optfltr.cxx
@@ -39,23 +39,19 @@ enum MSFltrPg2_CheckBoxEntries {
 // ---
 
 OfaMSFilterTabPage::OfaMSFilterTabPage(Window* pParent, const SfxItemSet& rSet)
-: SfxTabPage( pParent, CUI_RES( RID_OFAPAGE_MSFILTEROPT ), rSet ),
-aMSWordGB   ( this, CUI_RES( GB_WORD) ),
-aWBasicCodeCB   ( this, CUI_RES( CB_WBAS_CODE ) ),
-aWBasicWbctblCB ( this, CUI_RES( CB_WBAS_WBCTBL ) ),
-aWBasicStgCB( this, CUI_RES( CB_WBAS_STG  ) ),
-aMSExcelGB  ( this, CUI_RES( GB_EXCEL ) ),
-aEBasicCodeCB   ( this, CUI_RES( CB_EBAS_CODE ) ),
-aEBasicExectblCB( this, CUI_RES( CB_EBAS_EXECTBL ) ),
-aEBasicStgCB( this, CUI_RES( CB_EBAS_STG  ) ),
-aMSPPointGB ( this, CUI_RES( GB_PPOINT) ),
-aPBasicCodeCB   ( this, CUI_RES( CB_PBAS_CODE ) ),
-aPBasicStgCB( this, CUI_RES( CB_PBAS_STG  ) )
+: SfxTabPage( pParent, "OptFltrPage", "cui/ui/optfltrpage.ui", rSet )
 {
-FreeResource();
-
-aWBasicCodeCB.SetClickHdl( LINK( this, OfaMSFilterTabPage, 
LoadWordBasicCheckHdl_Impl ) );
-aEBasicCodeCB.SetClickHdl( LINK( this, OfaMSFilterTabPage, 
LoadExcelBasicCheckHdl_Impl ) );
+get( aWBasicCodeCB  , "wo_basic");
+get( aWBasicWbctblCB, "wo_exec" );
+get( aWBasicStgCB   , "wo_saveorig" );
+get( aEBasicCodeCB  , "ex_basic");
+get( aEBasicExectblCB   , "ex_exec" );
+get( aEBasicStgCB   , "ex_saveorig" );
+get( aPBasicCodeCB  , "pp_basic");
+get( aPBasicStgCB   , "pp_saveorig" );
+
+aWBasicCodeCB->SetClickHdl( LINK( this, OfaMSFilterTabPage, 
LoadWordBasicCheckHdl_Impl ) );
+aEBasicCodeCB->SetClickHdl( LINK( this, OfaMSFilterTabPage, 
LoadExcelBasicCheckHdl_Impl ) );
 }
 
 OfaMSFilterTabPage::~OfaMSFilterTabPage()
@@ -64,13 +60,13 @@ OfaMSFilterTabPage::~OfaMSFilterTabPage()
 
 IMPL_LINK_NOARG(OfaMSFilterTabPage, LoadWordBasicCheckHdl_Impl)
 {
-aWBasicWbctblCB.Enable( aWBasicCodeCB.IsChecked() );
+aWBasicWbctblCB->Enable( aWBasicCodeCB->IsChecked() );
 return 0;
 }
 
 IMPL_LINK_NOARG(OfaMSFilterTabPage, LoadExcelBasicCheckHdl_Impl)
 {
-aEBasicExectblCB.Enable( aEBasicCodeCB.IsChecked() );
+aEBasicExectblCB->Enable( aEBasicCodeCB->IsChecked() );
 return 0;
 }
 
@@ -85,23 +81,23 @@ sal_Bool OfaMSFilterTabPage::FillItemSet( SfxItemSet& )
 SvtFilterOptions& rOpt = SvtFilterOptions::Get();
 
 sal_Bool bFlag;
-if( aWBasicCodeCB.GetSavedValue() != (bFlag = aWBasicCodeCB.IsChecked()))
+if( aWBasicCodeCB->GetSavedValue() != (bFlag = aWBasicCodeCB->IsChecked()))
 rOpt.SetLoadWordBasicCode( bFlag );
-if( aWBasicWbctblCB.GetSavedValue() != (bFlag = 
aWBasicWbctblCB.IsChecked()))
+if( aWBasicWbctblCB->GetSavedValue() != (bFlag = 
aWBasicWbctblCB->IsChecked()))
 rOpt.SetLoadWordBasicExecutable( bFlag );
-if( aWBasicStgCB.GetSavedValue() != (bFlag = aWBasicStgCB.IsChecked()))
+if( aWBasicStgCB->GetSavedValue() != (bFlag = aWBasicStgCB->IsChecked()))
 rOpt.SetLoadWordBasicStorage( bFlag );
 
-if( aEBasicCodeCB.GetSavedValue() != (bFlag = aEBasicCodeCB.IsChecked()))
+if( aEBasicCodeCB->GetSavedValue() != (bFlag = aEBasicCodeCB->IsChecked()))
 rOpt.SetLoadExcelBasicCode( bFlag );
-if( aEBasicExectblCB.GetSavedValue() != (bFlag = 
aEBasicExectblCB.IsChecked()))
+if( aEBasicExectblCB->GetSavedValue() != (bFlag = 
aEBasicExectblCB->IsChecked()))
 rOpt.SetLoadExcelBasicExecutable( bFlag );
-if( aEBasicStgCB.GetSavedValue() != (

[PUSHED] convert half of optfltr.src to .ui

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2961

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie200377f2956aa243b1be859eb52972b79e9dc66
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Krisztian Pinter 
Gerrit-Reviewer: LibreOffice gerrit bot 
Gerrit-Reviewer: Petr Mladek 

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


[Libreoffice-commits] core.git: Branch 'feature/formula-core-rework' - sc/inc sc/source

2013-03-26 Thread Kohei Yoshida
 sc/inc/chgtrack.hxx  |3 
 sc/inc/dociter.hxx   |   12 --
 sc/source/core/data/dociter.cxx  |  187 +--
 sc/source/core/tool/chgtrack.cxx |8 -
 sc/source/core/tool/interpr1.cxx |4 
 sc/source/core/tool/interpr5.cxx |2 
 6 files changed, 59 insertions(+), 157 deletions(-)

New commits:
commit 6a6f74a9d5fa61ccdd97bffd46c13fd799a8d52e
Author: Kohei Yoshida 
Date:   Tue Mar 26 12:35:36 2013 -0400

By using ScRefCellValue we can remove some duplicated code.

Change-Id: I75652172033e4ce00ac239e85d835c7cc9256fe2

diff --git a/sc/inc/chgtrack.hxx b/sc/inc/chgtrack.hxx
index b30cf1b..6b06c22 100644
--- a/sc/inc/chgtrack.hxx
+++ b/sc/inc/chgtrack.hxx
@@ -37,7 +37,6 @@
 
 class ScDocument;
 class ScFormulaCell;
-class ScCellIterator;
 
 enum ScChangeActionType
 {
@@ -800,7 +799,7 @@ public:
 rtl::OUString& rStr, ScDocument* pDoc, bool bFlag3D = false ) const;
 
 static ScChangeActionContentCellType GetContentCellType( const 
ScCellValue& rCell );
-static ScChangeActionContentCellType GetContentCellType( const 
ScCellIterator& rIter );
+static ScChangeActionContentCellType GetContentCellType( const 
ScRefCellValue& rIter );
 
 // NewCell
 bool IsMatrixOrigin() const;
diff --git a/sc/inc/dociter.hxx b/sc/inc/dociter.hxx
index eb34633..4f3ab26 100644
--- a/sc/inc/dociter.hxx
+++ b/sc/inc/dociter.hxx
@@ -24,6 +24,7 @@
 #include 
 #include "global.hxx"
 #include "scdllapi.h"
+#include "cellvalue.hxx"
 
 #include 
 
@@ -45,7 +46,6 @@ struct ScQueryParam;
 struct ScDBQueryParamInternal;
 struct ScDBQueryParamMatrix;
 class ScFormulaCell;
-class ScCellValue;
 
 class ScDocumentIterator// walk through all non-empty cells
 {
@@ -222,13 +222,7 @@ private:
 SCSIZE mnIndex;
 bool mbSubTotal;
 
-CellType meCurType;
-OUString maCurString;
-union {
-double mfCurValue;
-const EditTextObject* mpCurEditText; // points to the original.
-ScFormulaCell* mpCurFormula; // points to the original.
-};
+ScRefCellValue maCurCell;
 
 void init();
 bool getCurrent();
@@ -245,9 +239,11 @@ public:
 const ScFormulaCell* getFormulaCell() const;
 double getValue() const;
 ScCellValue getCellValue() const;
+ScRefCellValue getRefCellValue() const;
 
 bool hasString() const;
 bool hasNumeric() const;
+bool hasEmptyData() const;
 bool isEmpty() const;
 bool equalsWithoutFormat( const ScAddress& rPos ) const;
 
diff --git a/sc/source/core/data/dociter.cxx b/sc/source/core/data/dociter.cxx
index 9f7dd02..adb7787 100644
--- a/sc/source/core/data/dociter.cxx
+++ b/sc/source/core/data/dociter.cxx
@@ -943,9 +943,7 @@ ScCellIterator::ScCellIterator( ScDocument* pDoc, const 
ScRange& rRange, bool bS
 maStartPos(rRange.aStart),
 maEndPos(rRange.aEnd),
 mnIndex(0),
-mbSubTotal(bSTotal),
-meCurType(CELLTYPE_NONE),
-mfCurValue(0.0)
+mbSubTotal(bSTotal)
 {
 init();
 }
@@ -996,7 +994,7 @@ bool ScCellIterator::getCurrent()
 maCurPos.IncTab();
 if (maCurPos.Tab() > maEndPos.Tab())
 {
-meCurType = CELLTYPE_NONE;
+maCurCell.clear();
 return false; // Over and out
 }
 }
@@ -1021,26 +1019,26 @@ bool ScCellIterator::getCurrent()
 else
 {
 // Found it!
-meCurType = pCell->GetCellType();
-switch (meCurType)
+maCurCell.meType = pCell->GetCellType();
+switch (maCurCell.meType)
 {
 case CELLTYPE_VALUE:
-mfCurValue = static_cast(pCell)->GetValue();
+maCurCell.mfValue = static_cast(pCell)->GetValue();
 break;
 case CELLTYPE_STRING:
-maCurString = static_cast(pCell)->GetString();
+maCurCell.mpString = static_cast(pCell)->GetStringPtr();
 break;
 case CELLTYPE_EDIT:
-mpCurEditText = static_cast(pCell)->GetData();
+maCurCell.mpEditText = static_cast(pCell)->GetData();
 break;
 case CELLTYPE_FORMULA:
-mpCurFormula = static_cast(pCell);
+maCurCell.mpFormula = 
static_cast(pCell);
 break;
 default:
-meCurType = CELLTYPE_NONE;
+maCurCell.meType = CELLTYPE_NONE;
 }
 
-if (meCurType != CELLTYPE_NONE)
+if (maCurCell.meType != CELLTYPE_NONE)
 return true;
 

[Libreoffice-commits] core.git: sfx2/source

2013-03-26 Thread Stephan Bergmann
 sfx2/source/doc/templatedlg.cxx |   22 +++---
 1 file changed, 11 insertions(+), 11 deletions(-)

New commits:
commit de30e4c9164f5ad6e8f04218810ba997dc930410
Author: Stephan Bergmann 
Date:   Tue Mar 26 17:35:01 2013 +0100

quickfix

Change-Id: Ida36d96982d869553452c8240da7dec363cee4b4

diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 6899cf6..4250fe1 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -846,7 +846,7 @@ void SfxTemplateManagerDlg::OnTemplateImport ()
 
 if (aFiles.hasElements())
 {
-std::set::const_iterator pIter;
+std::set::const_iterator pIter;
 for (pIter = maSelFolders.begin(); pIter != maSelFolders.end(); 
++pIter)
 {
 OUString aTemplateList;
@@ -895,7 +895,7 @@ void SfxTemplateManagerDlg::OnTemplateExport()
 {
 sal_uInt16 i = 1;
 
-std::set::const_iterator pIter = 
maSelTemplates.begin();
+std::set::const_iterator pIter = 
maSelTemplates.begin();
 for (pIter = maSelTemplates.begin(); pIter != 
maSelTemplates.end(); ++pIter, ++i)
 {
 const TemplateSearchViewItem *pItem = static_cast(*pIter);
@@ -927,7 +927,7 @@ void SfxTemplateManagerDlg::OnTemplateExport()
 sal_uInt16 i = 1;
 sal_uInt16 nRegionItemId = maView->getCurRegionId() + 1;
 
-std::set::const_iterator pIter = 
maSelTemplates.begin();
+std::set::const_iterator pIter = 
maSelTemplates.begin();
 for (pIter = maSelTemplates.begin(); pIter != 
maSelTemplates.end(); ++pIter, ++i)
 {
 const TemplateViewItem *pItem = static_cast(*pIter);
@@ -1014,7 +1014,7 @@ void SfxTemplateManagerDlg::OnTemplateEdit ()
 aArgs[2].Value <<= UpdateDocMode::ACCORDING_TO_CONFIG;
 
 uno::Reference< XStorable > xStorable;
-std::set::const_iterator pIter;
+std::set::const_iterator pIter;
 for (pIter = maSelTemplates.begin(); pIter != maSelTemplates.end(); 
++pIter)
 {
 const TemplateViewItem *pItem = static_cast(*pIter);
@@ -1053,7 +1053,7 @@ void SfxTemplateManagerDlg::OnTemplateDelete ()
 
 if (mpSearchView->IsVisible())
 {
-std::set::const_iterator pIter;
+std::set::const_iterator 
pIter;
 for (pIter = maSelTemplates.begin(); pIter != maSelTemplates.end();)
 {
 const TemplateSearchViewItem *pItem =
@@ -1087,7 +1087,7 @@ void SfxTemplateManagerDlg::OnTemplateDelete ()
 }
 else
 {
-std::set::const_iterator pIter;
+std::set::const_iterator 
pIter;
 for (pIter = maSelTemplates.begin(); pIter != maSelTemplates.end();)
 {
 if 
(maView->removeTemplate((*pIter)->mnId,maView->getCurRegionId()+1))
@@ -1158,7 +1158,7 @@ void SfxTemplateManagerDlg::OnFolderDelete()
 
 OUString aFolderList;
 
-std::set::const_iterator pIter;
+std::set::const_iterator pIter;
 for (pIter = maSelFolders.begin(); pIter != maSelFolders.end();)
 {
 if (maView->removeRegion((*pIter)->mnId))
@@ -1238,7 +1238,7 @@ void SfxTemplateManagerDlg::OnTemplateSaveAs()
 }
 else
 {
-std::set::const_iterator pIter;
+std::set::const_iterator pIter;
 for (pIter = maSelFolders.begin(); pIter != 
maSelFolders.end(); ++pIter)
 {
 TemplateContainerItem *pItem = 
(TemplateContainerItem*)(*pIter);
@@ -1370,7 +1370,7 @@ void SfxTemplateManagerDlg::localMoveTo(sal_uInt16 
nMenuId)
 {
 OUString aTemplateList;
 
-std::set::const_iterator pIter;
+std::set::const_iterator pIter;
 for (pIter = maSelFolders.begin(); pIter != maSelFolders.end(); 
++pIter)
 {
 if (aTemplateList.isEmpty())
@@ -1414,7 +1414,7 @@ void SfxTemplateManagerDlg::remoteMoveTo(const sal_uInt16 
nMenuId)
 {
 OUString aTemplateList;
 
-std::set::const_iterator aIter;
+std::set::const_iterator 
aIter;
 for (aIter = maSelTemplates.begin(); aIter != maSelTemplates.end(); 
++aIter)
 {
 const TemplateSearchViewItem *pItem =
@@ -1470,7 +1470,7 @@ void SfxTemplateManagerDlg::localSearchMoveTo(sal_uInt16 
nMenuId)
 
 // Move templates to desired folder if for some reason move fails
 // try copying them.
-std::set::const_iterator aIter;
+std::set::const_iterator 
aIter;
 std::set aSelTemplates = 
maSelTemplates; //Copy to avoid invalidating an iterator
 
 for (aIter = aSelTemplates.begin(); aIter != aSelTemplates.end(); 
++aIter)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: ios/CustomTarget_Viewer_app.mk

2013-03-26 Thread Tor Lillqvist
 ios/CustomTarget_Viewer_app.mk |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 5a4c573506e4db1891cf873650eb3d154b334d52
Author: Tor Lillqvist 
Date:   Sat Mar 23 13:32:24 2013 +0200

Add images.zip

Change-Id: If764bb95ab29b8092b356fee644cfa5190ce3eeb

diff --git a/ios/CustomTarget_Viewer_app.mk b/ios/CustomTarget_Viewer_app.mk
index cd79db1..436bf46 100644
--- a/ios/CustomTarget_Viewer_app.mk
+++ b/ios/CustomTarget_Viewer_app.mk
@@ -84,6 +84,11 @@ $(SCRIPT_OUTPUT_FILE_0) : $(call 
gb_Executable_get_target,Viewer)
mkdir -p $(appdir)/program/resource
cp $(OUTDIR)/bin/*en-US.res $(appdir)/program/resource
 #
+# Artwork
+#
+   mkdir -p $(appdir)/share/config
+   cp -R $(OUTDIR)/bin/images_tango.zip $(appdir)/share/config/images.zip
+#
 # Set up rc, the "inifile". See getIniFileName_Impl(). 
 #
( \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: ios/experimental

2013-03-26 Thread Tor Lillqvist
 ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj |4 
 1 file changed, 4 insertions(+)

New commits:
commit 2672cbdd00667ecc71b3390a055ee25bb0b06d80
Author: Tor Lillqvist 
Date:   Sat Mar 23 13:36:13 2013 +0200

Add LOViewerWindow.[hm] to the project

Change-Id: I50ac91fd0e803b1b2322e5c7c25f7bf682a8208d

diff --git a/ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj 
b/ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj
index 0535a2b..bef9ae0 100644
--- a/ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj
+++ b/ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj
@@ -18,6 +18,8 @@
 /* End PBXBuildFile section */
 
 /* Begin PBXFileReference section */
+   BE08805B16FDB784000CED5C /* LOViewerWindow.h */ = {isa = 
PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = 
LOViewerWindow.h; sourceTree = ""; };
+   BE08805C16FDB784000CED5C /* LOViewerWindow.m */ = {isa = 
PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path 
= LOViewerWindow.m; sourceTree = ""; };
BE5A22551664F87600CF01C9 /* Viewer.app */ = {isa = 
PBXFileReference; explicitFileType = wrapper.application; includeInIndex = 0; 
path = Viewer.app; sourceTree = BUILT_PRODUCTS_DIR; };
BE5A22591664F87600CF01C9 /* UIKit.framework */ = {isa = 
PBXFileReference; lastKnownFileType = wrapper.framework; name = 
UIKit.framework; path = System/Library/Frameworks/UIKit.framework; sourceTree = 
SDKROOT; };
BE5A225B1664F87600CF01C9 /* Foundation.framework */ = {isa = 
PBXFileReference; lastKnownFileType = wrapper.framework; name = 
Foundation.framework; path = System/Library/Frameworks/Foundation.framework; 
sourceTree = SDKROOT; };
@@ -83,6 +85,8 @@
BE69AD3716958CA0001BE1CD /* Resources */,
BE5A22681664F87600CF01C9 /* 
LOViewerAppDelegate.h */,
BE5A22691664F87600CF01C9 /* 
LOViewerAppDelegate.mm */,
+   BE08805B16FDB784000CED5C /* LOViewerWindow.h */,
+   BE08805C16FDB784000CED5C /* LOViewerWindow.m */,
BE5A22601664F87600CF01C9 /* Supporting Files */,
);
path = Viewer;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 3 commits - sal/osl sc/source sfx2/inc sfx2/source vcl/coretext

2013-03-26 Thread Tor Lillqvist
 sal/osl/unx/file.cxx |   12 
 sc/source/ui/docshell/docsh.cxx  |8 ++--
 sc/source/ui/docshell/docsh4.cxx |9 +++--
 sc/source/ui/unoobj/confuno.cxx  |4 
 sc/source/ui/view/viewfunc.cxx   |2 ++
 sfx2/inc/sfx2/objsh.hxx  |8 +++-
 sfx2/source/doc/objmisc.cxx  |5 -
 sfx2/source/doc/objstor.cxx  |2 ++
 sfx2/source/doc/objxtor.cxx  |3 ++-
 sfx2/source/doc/sfxbasemodel.cxx |2 ++
 sfx2/source/view/viewfrm.cxx |6 --
 vcl/coretext/salgdi.cxx  |6 ++
 12 files changed, 58 insertions(+), 9 deletions(-)

New commits:
commit 83a486546f535b32a565d9215e7584c5572c2fb0
Author: Tor Lillqvist 
Date:   Sat Mar 23 17:17:20 2013 +0200

Shared documents make no sense for Android and iOS

(When I say documents, I mean spreadsheets. Only Calc has code for
this.)

Change-Id: I05fd19885633f9e250940678d33e2e4c82c31a9d

diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx
index 7a2aeba..045eaa5 100644
--- a/sc/source/ui/docshell/docsh.cxx
+++ b/sc/source/ui/docshell/docsh.cxx
@@ -635,6 +635,7 @@ void ScDocShell::Notify( SfxBroadcaster&, const SfxHint& 
rHint )
 {
 case SFX_EVENT_LOADFINISHED:
 {
+#if !defined(ANDROID) && !defined(IOS)
 // the readonly documents should not be opened in shared 
mode
 if ( HasSharedXMLFlagSet() && 
!SC_MOD()->IsInSharedDocLoading() && !IsReadOnly() )
 {
@@ -654,10 +655,12 @@ void ScDocShell::Notify( SfxBroadcaster&, const SfxHint& 
rHint )
 SetReadOnlyUI( sal_True );
 }
 }
+#endif
 }
 break;
 case SFX_EVENT_VIEWCREATED:
 {
+#if !defined(ANDROID) && !defined(IOS)
 if ( IsDocShared() && !SC_MOD()->IsInSharedDocLoading() )
 {
 ScAppOptions aAppOptions = SC_MOD()->GetAppOptions();
@@ -675,7 +678,7 @@ void ScDocShell::Notify( SfxBroadcaster&, const SfxHint& 
rHint )
 }
 }
 }
-
+#endif
 try
 {
 uno::Reference< uno::XComponentContext > xContext(
@@ -717,6 +720,7 @@ void ScDocShell::Notify( SfxBroadcaster&, const SfxHint& 
rHint )
 break;
 case SFX_EVENT_SAVEDOC:
 {
+#if !defined(ANDROID) && !defined(IOS)
 if ( IsDocShared() && !SC_MOD()->IsInSharedDocSaving() )
 {
 bool bSuccess = false;
@@ -900,7 +904,7 @@ void ScDocShell::Notify( SfxBroadcaster&, const SfxHint& 
rHint )
 if ( !bSuccess )
 SetError( ERRCODE_IO_ABORT, ::rtl::OUString( 
RTL_CONSTASCII_USTRINGPARAM( OSL_LOG_PREFIX ) ) ); // this error code will 
produce no error message, but will break the further saving process
 }
-
+#endif
 
 if (pSheetSaveData)
 pSheetSaveData->SetInSupportedSave(true);
diff --git a/sc/source/ui/docshell/docsh4.cxx b/sc/source/ui/docshell/docsh4.cxx
index 93d1ca0..42f7038 100644
--- a/sc/source/ui/docshell/docsh4.cxx
+++ b/sc/source/ui/docshell/docsh4.cxx
@@ -766,6 +766,7 @@ void ScDocShell::Execute( SfxRequest& rReq )
 
aDocument.SetChangeViewSettings(aChangeViewSet);
 }
 }
+#if !defined(ANDROID) && !defined(IOS)
 else if ( nSlot == SID_DOCUMENT_MERGE && IsDocShared() 
&& pChangeTrack )
 {
 sal_uLong nEnd = pChangeTrack->GetActionMax();
@@ -784,6 +785,7 @@ void ScDocShell::Execute( SfxRequest& rReq )
 PostPaintGridAll();
 }
 }
+#endif
 }
 pOtherDocSh->DoClose(); // delete passiert mit der Ref
 }
@@ -895,6 +897,7 @@ void ScDocShell::Execute( SfxRequest& rReq )
 }
 break;
 
+#if !defined(ANDROID) && !defined(IOS)
 case SID_SHARE_DOC:
 {
 ScViewData* pViewData = GetViewData();
@@ -1071,7 +1074,7 @@ void ScDocShell::Execute( SfxRequest& rReq )
 rReq.Done();
 }
 break;
-
+#endif
 case SID_OPEN_CALC:
 {
 SfxStringItem aApp(SID_DOC_SERVICE, 
rtl::OUString("com.sun.star.sheet.SpreadsheetDocument"));
@@ -2297,7 +2300,7 @@ IMPL_LINK( ScDocShell, DialogClosedHdl, 
sfx2::FileDialogHelper*, _pFileDlg )
 return 0;
 }
 
-//--
+#if !defined(ANDROID) && !defined(IOS)
 
 void ScDocShell::EnableSharedSettings( bool bEnable )
 {
@@ -2376,4 +2379,6 @@ 

[Libreoffice-commits] core.git: 2 commits - desktop/source framework/source sc/source sfx2/inc sfx2/source solenv/gbuild svl/Library_svl.mk sw/source vcl/source

2013-03-26 Thread Tor Lillqvist
 desktop/source/app/app.cxx|8 
++--
 desktop/source/app/cmdlineargs.cxx|2 +
 desktop/source/app/officeipcthread.cxx|   10 
++---
 desktop/source/app/userinstall.cxx|2 -
 desktop/source/deployment/manager/dp_activepackages.cxx   |   17 
--
 desktop/source/deployment/manager/dp_activepackages.hxx   |4 +-
 desktop/source/deployment/registry/configuration/dp_configuration.cxx |   13 
---
 desktop/source/deployment/registry/help/dp_help.cxx   |4 +-
 framework/source/services/autorecovery.cxx|4 ++
 sc/source/ui/docshell/docsh.cxx   |6 
+--
 sc/source/ui/docshell/docsh4.cxx  |6 
+--
 sc/source/ui/unoobj/confuno.cxx   |4 +-
 sc/source/ui/view/viewfunc.cxx|2 -
 sfx2/inc/sfx2/objsh.hxx   |2 -
 sfx2/source/doc/docfile.cxx   |   17 
+-
 sfx2/source/doc/objmisc.cxx   |2 -
 sfx2/source/doc/objstor.cxx   |2 -
 sfx2/source/doc/objxtor.cxx   |2 -
 sfx2/source/doc/sfxbasemodel.cxx  |2 -
 sfx2/source/view/viewfrm.cxx  |4 +-
 solenv/gbuild/Helper.mk   |4 ++
 solenv/gbuild/gbuild.mk   |3 +
 svl/Library_svl.mk|3 +
 sw/source/ui/uiview/viewport.cxx  |4 +-
 vcl/source/app/dbggui.cxx |2 -
 vcl/source/window/window.cxx  |2 -
 26 files changed, 82 insertions(+), 49 deletions(-)

New commits:
commit 5a5c8e253abbef5626dab70f8978275ac3c742f8
Author: Tor Lillqvist 
Date:   Sat Mar 23 19:06:58 2013 +0200

Check feature test macros instead of ANDROID and/or IOS

Pass -DLIBO_FEATURE_FOO to compiler for some elements in BUILD_TYPE.
BUILD_TYPE has at least two kinds of elements: Those that indicate
building a bundled copy of some 3rd-party library, and those that
indicate some specific feature of the LibreOffice platform or
build-time configuration choice. This is for the latter kind.

Change many of the checks for Android and/or iOS in the source code to
check LIBO_FEATURE_DESKTOP, LIBO_FEATURE_HELP or DISABLE_EXTENSIONS
instead, in cases where that is what is meant, not Android or iOS
specifically.

Change-Id: I2cd3f3bb99e953c7754dcea76a426f8f9d61e4db

diff --git a/desktop/source/app/app.cxx b/desktop/source/app/app.cxx
index d3e7b26..6a3e3de 100644
--- a/desktop/source/app/app.cxx
+++ b/desktop/source/app/app.cxx
@@ -1386,7 +1386,8 @@ int Desktop::Main()
 new DesktopContext( com::sun::star::uno::getCurrentContext() ) );
 
 CommandLineArgs& rCmdLineArgs = GetCommandLineArgs();
-#if !defined(ANDROID) && !defined(IOS)
+
+#ifdef LIBO_FEATURE_DESKTOP
 OUString aUnknown( rCmdLineArgs.GetUnknown() );
 if ( !aUnknown.isEmpty() )
 {
@@ -1450,7 +1451,8 @@ int Desktop::Main()
 // there is no other instance using our data files from a remote host
 RTL_LOGFILE_CONTEXT_TRACE( aLog, "desktop (lo119109) Desktop::Main -> 
Lockfile" );
 m_xLockfile.reset(new Lockfile);
-#if !defined(ANDROID) && !defined(IOS)
+
+#ifdef LIBO_FEATURE_DESKTOP
 if ( !rCmdLineArgs.IsHeadless() && !rCmdLineArgs.IsInvisible() &&
  !rCmdLineArgs.IsNoLockcheck() && !m_xLockfile->check( 
Lockfile_execWarning ))
 {
@@ -1537,7 +1539,7 @@ int Desktop::Main()
 */
 Application::GetDefaultDevice();
 
-#if !defined(ANDROID) && !defined(IOS)
+#ifdef LIBO_FEATURE_DESKTOP
 // Check if bundled or shared extensions were added /removed
 // and process those extensions (has to be done before checking
 // the extension dependencies!
diff --git a/desktop/source/app/cmdlineargs.cxx 
b/desktop/source/app/cmdlineargs.cxx
index 6ebe54e..b8cf153 100644
--- a/desktop/source/app/cmdlineargs.cxx
+++ b/desktop/source/app/cmdlineargs.cxx
@@ -442,7 +442,7 @@ bool CommandLineArgs::InterpretCommandLineParameter( const 
::rtl::OUString& aArg
 {
 m_nologo = true;
 }
-#if !defined(ANDROID) && !defined(IOS)
+#ifdef LIBO_FEATURE_DESKTOP
 else if ( oArg == "nolockcheck" )
 {
 m_nolockcheck = true;
diff --git a/desktop/source/app/officeipcthread.cxx 
b/desktop/source/app/officeipcthread.cxx
index af44990..071e2bb 100644
--- a/

[Libreoffice-commits] core.git: Branch 'libreoffice-4-0' - sw/qa writerfilter/source

2013-03-26 Thread Miklos Vajna
 sw/qa/extras/ooxmlimport/data/fdo59273.docx  |binary
 sw/qa/extras/ooxmlimport/data/n780645.docx   |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |   12 +
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   10 +--
 writerfilter/source/dmapper/DomainMapperTableManager.hxx |7 +
 writerfilter/source/dmapper/TablePropertiesHandler.cxx   |8 ++
 writerfilter/source/ooxml/model.xml  |   19 ---
 7 files changed, 45 insertions(+), 11 deletions(-)

New commits:
commit 0dd9158a1fe48dee192a954a8ea24c2efcd64a86
Author: Miklos Vajna 
Date:   Tue Mar 26 12:15:03 2013 +0100

fdo#59273 import w:tblLayout of DOCX tables

Also add missing  to the bnc#780645 testcase
to avoid breaking the build.

(cherry picked from commits 7329df74df134a77d078f47d5c8b70d54c5d1edb and
4821d894f849a4939bbe6fb0a9a5ddd9e10b0467)

Conflicts:
writerfilter/source/dmapper/DomainMapperTableManager.cxx
writerfilter/source/dmapper/DomainMapperTableManager.hxx

Change-Id: I9af50a4c176168e2794eaa9b0e8eaeb6b9d06b13
Reviewed-on: https://gerrit.libreoffice.org/3058
Reviewed-by: Luboš Luňák 
Tested-by: Luboš Luňák 

diff --git a/sw/qa/extras/ooxmlimport/data/fdo59273.docx 
b/sw/qa/extras/ooxmlimport/data/fdo59273.docx
new file mode 100644
index 000..fcdfd77
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/fdo59273.docx differ
diff --git a/sw/qa/extras/ooxmlimport/data/n780645.docx 
b/sw/qa/extras/ooxmlimport/data/n780645.docx
index 12ffa38..35ad11f 100755
Binary files a/sw/qa/extras/ooxmlimport/data/n780645.docx and 
b/sw/qa/extras/ooxmlimport/data/n780645.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 51ce241..aeda333 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -115,6 +115,7 @@ public:
 void testFdo59638();
 void testFdo61343();
 void testFdo60922();
+void testFdo59273();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX) && !defined(WNT)
@@ -182,6 +183,7 @@ void Test::run()
 {"fdo59638.docx", &Test::testFdo59638},
 {"fdo61343.docx", &Test::testFdo61343},
 {"fdo60922.docx", &Test::testFdo60922},
+{"fdo59273.docx", &Test::testFdo59273},
 };
 for (unsigned int i = 0; i < SAL_N_ELEMENTS(aMethods); ++i)
 {
@@ -1140,6 +1142,16 @@ void Test::testFdo60922()
 CPPUNIT_ASSERT_EQUAL(sal_Int32(100), 
getProperty(getRun(getParagraph(1), 1), "CharEscapementHeight"));
 }
 
+void Test::testFdo59273()
+{
+uno::Reference xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Reference 
xTables(xTablesSupplier->getTextTables( ), uno::UNO_QUERY);
+uno::Reference xTextTable(xTables->getByIndex(0), 
uno::UNO_QUERY);
+uno::Reference xTableRows(xTextTable->getRows(), 
uno::UNO_QUERY);
+// Was 9997, so the 4th column had ~zero width
+CPPUNIT_ASSERT_EQUAL(sal_Int16(7498), getProperty< 
uno::Sequence >(xTableRows->getByIndex(0), 
"TableColumnSeparators")[2].Position);
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index d81aea7..0298690 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -51,6 +51,7 @@ DomainMapperTableManager::DomainMapperTableManager(bool 
bOOXML) :
 m_nTableWidth(0),
 m_bOOXML( bOOXML ),
 m_bPushCurrentWidth(false),
+m_nLayoutType(0),
 m_pTablePropsHandler( new TablePropertiesHandler( bOOXML ) )
 {
 m_pTablePropsHandler->SetTableManager( this );
@@ -395,6 +396,7 @@ void DomainMapperTableManager::startLevel( )
 m_aTablePositions.push_back( pNewPositionHandler );
 m_nCell.push_back( 0 );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 // And push it back to the right level.
 if (oCurrentWidth)
@@ -408,6 +410,7 @@ void DomainMapperTableManager::endLevel( )
 m_aCellWidths.pop_back( );
 m_nCell.pop_back( );
 m_nTableWidth = 0;
+m_nLayoutType = 0;
 
 
 DomainMapperTableManager_Base_t::endLevel( );
@@ -548,12 +551,13 @@ void DomainMapperTableManager::endOfRowAction()
 #endif
 insertRowProps(pPropMap);
 }
-else if (pCellWidths->size() > 0)
+else if (pCellWidths->size() > 0 && m_nLayoutType == 
NS_ooxml::LN_Value_wordprocessingml_ST_TblLayout_fixed)
 {
 // If we're here, then the number of cells does not equal to the amount
 // defined by the grid, even after taking care of
 // gridSpan/gridBefore/gridAfter. Handle this by ignoring the grid and
-// providing the separators based on the provided cell widths.
+// providing the separators based on the provided cell widths, as long
+// as 

[PUSHED libreoffice-4-0] fdo#59273 import w:tblLayout of DOCX tables

2013-03-26 Thread via Code Review
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/3058

Approvals:
  Luboš Luňák: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9af50a4c176168e2794eaa9b0e8eaeb6b9d06b13
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Miklos Vajna 
Gerrit-Reviewer: Luboš Luňák 

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


[Libreoffice-commits] core.git: svx/source

2013-03-26 Thread K_Karthikeyan
 svx/source/table/tablecontroller.cxx |9 +++--
 1 file changed, 7 insertions(+), 2 deletions(-)

New commits:
commit 60570afbeaa058b6f8a9b878cc41b1535ef6e5f0
Author: K_Karthikeyan 
Date:   Sun Mar 24 09:18:31 2013 +0300

Resolves fdo#61540

On Insert before, the reference column whose size is going to be used for
newly created column(s) is wrong. As the new columns are inserted before the
reference column, the reference column moved to the new position by no., of 
new
columns i.e (earlier+newcolumns).

Change-Id: Ib52e3633aecb1220cdf709058391361376dc5f00
Reviewed-on: https://gerrit.libreoffice.org/2958
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 

diff --git a/svx/source/table/tablecontroller.cxx 
b/svx/source/table/tablecontroller.cxx
index 1cb3035..74c42cb 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -554,9 +554,14 @@ void SvxTableController::onInsert( sal_uInt16 nSId, const 
SfxItemSet* pArgs )
 
 for( sal_Int32 nOffset = 0; nOffset < nNewColumns; nOffset++ )
 {
-Reference< XPropertySet >( xCols->getByIndex( aEnd.mnCol + 
nOffset + 1 ), UNO_QUERY_THROW )->
+// Resolves fdo#61540
+// On Insert before, the reference column whose size is going 
to be
+// used for newly created column(s) is wrong. As the new 
columns are
+// inserted before the reference column, the reference column 
moved
+// to the new position by no., of new columns i.e 
(earlier+newcolumns).
+Reference< XPropertySet 
>(xCols->getByIndex(nNewStartColumn+nOffset), UNO_QUERY_THROW )->
 setPropertyValue( sSize,
-Reference< XPropertySet >( xCols->getByIndex( 
aStart.mnCol + nOffset ), UNO_QUERY_THROW )->
+Reference< XPropertySet >(xCols->getByIndex( 
bInsertAfter?nNewStartColumn-1:nNewStartColumn+nNewColumns ), UNO_QUERY_THROW 
)->
 getPropertyValue( sSize ) );
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Resolves fdo#61540

2013-03-26 Thread Petr Mladek (via Code Review)
Hi,

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

If you are interested in details, please visit

https://gerrit.libreoffice.org/2958

Approvals:
  Petr Mladek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib52e3633aecb1220cdf709058391361376dc5f00
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Karthikeyan Krishnamurthi 
Gerrit-Reviewer: Lior Kaplan 
Gerrit-Reviewer: Petr Mladek 

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


[Libreoffice-commits] core.git: 4 commits - vcl/coretext vcl/ios vcl/source

2013-03-26 Thread Tor Lillqvist
 vcl/coretext/salgdi.cxx  |6 +++---
 vcl/ios/iosinst.cxx  |   15 +--
 vcl/source/window/window.cxx |   14 +++---
 3 files changed, 15 insertions(+), 20 deletions(-)

New commits:
commit 59c83ffb05865a5e492c3204228e85ca006e2b5b
Author: Tor Lillqvist 
Date:   Sun Mar 24 00:22:15 2013 +0200

Don't exit intentionally

Change-Id: I1d4c7dbc8ed9254e92d097454704c811ba1532f1

diff --git a/vcl/ios/iosinst.cxx b/vcl/ios/iosinst.cxx
index 3ce3a65..b2178e9 100644
--- a/vcl/ios/iosinst.cxx
+++ b/vcl/ios/iosinst.cxx
@@ -236,21 +236,16 @@ void InitSalData()   {}
 void DeInitSalData() {}
 void InitSalMain()   {}
 
-void SalAbort( const rtl::OUString& rErrorText, bool bDumpCore )
+void SalAbort( const OUString& rErrorText, bool bDumpCore )
 {
-rtl::OUString aError( rErrorText );
-if( aError.isEmpty() )
-aError = rtl::OUString::createFromAscii("Unknown application error");
-
-if( bDumpCore )
-abort();
-else
-_exit(1);
+(void) bDumpCore;
+
+NSLog(@"SalAbort: %s", OUStringToOString(rErrorText, 
osl_getThreadTextEncoding()).getStr() );
 }
 
 const OUString& SalGetDesktopEnvironment()
 {
-static rtl::OUString aEnv( "android" );
+static OUString aEnv( "iOS" );
 return aEnv;
 }
 
commit d6869762a903d1614a9592508092f0721e5a4633
Author: Tor Lillqvist 
Date:   Sun Mar 24 00:10:07 2013 +0200

Try kCGImageAlphaNoneSkipLast

Change-Id: I127f450ae7c52f25033a6dbc34029545ea5ed07b

diff --git a/vcl/coretext/salgdi.cxx b/vcl/coretext/salgdi.cxx
index d2a0281..c7db660 100644
--- a/vcl/coretext/salgdi.cxx
+++ b/vcl/coretext/salgdi.cxx
@@ -285,7 +285,7 @@ bool SvpSalGraphics::CheckContext()
 basebmp::RawMemorySharedArray pixelBuffer = m_aDevice->getBuffer();
 
 mrContext = CGBitmapContextCreate(pixelBuffer.get(), size.getX(), 
size.getY(), 8, m_aDevice->getScanlineStride(),
-  CGColorSpaceCreateDeviceRGB(), 
kCGImageAlphaLast|kCGBitmapByteOrder32Little);
+  CGColorSpaceCreateDeviceRGB(), 
kCGImageAlphaNoneSkipLast);
 
 SAL_WARN_IF( mrContext == NULL, "vcl.ios", "CheckContext() failed" );
 
commit 9f1a3352b5cb04e877785af3110f56d2d3418eda
Author: Tor Lillqvist 
Date:   Sat Mar 23 23:22:31 2013 +0200

Improve debugging printout a bit

Change-Id: I1acc1b3f1e9d2cdcbba058b050f278e396f115c6

diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
index 459b917..81b002f 100644
--- a/vcl/source/window/window.cxx
+++ b/vcl/source/window/window.cxx
@@ -4223,11 +4223,11 @@ namespace
 pWindow = &i_rWindow;
 
 rtl::OStringBuffer aErrorString;
-aErrorString.append(char(13));
+aErrorString.append(' ');
 aErrorString.append(typeid( *pWindow ).name());
-aErrorString.append(" (window text: '");
+aErrorString.append(" (");
 aErrorString.append(rtl::OUStringToOString(pWindow->GetText(), 
RTL_TEXTENCODING_UTF8));
-aErrorString.append("')");
+aErrorString.append(")");
 return aErrorString.makeStringAndClear();
 }
 }
@@ -4377,7 +4377,7 @@ Window::~Window()
 aTempStr.append("Window (");
 aTempStr.append(rtl::OUStringToOString(GetText(),
RTL_TEXTENCODING_UTF8));
-aTempStr.append(") with living SystemWindow(s) destroyed: ");
+aTempStr.append(") with live SystemWindows destroyed: ");
 aTempStr.append(aErrorStr.toString());
 OSL_FAIL(aTempStr.getStr());
 // abort in non-pro version, this must be fixed!
@@ -4401,7 +4401,7 @@ Window::~Window()
 {
 rtl::OStringBuffer aTempStr( "Window (" );
 aTempStr.append(rtl::OUStringToOString(GetText(), 
RTL_TEXTENCODING_UTF8));
-aTempStr.append(") with living SystemWindow(s) destroyed: ");
+aTempStr.append(") with live SystemWindows destroyed: ");
 aTempStr.append(aErrorStr.toString());
 OSL_FAIL( aTempStr.getStr() );
 
GetpApp()->Abort(rtl::OStringToOUString(aTempStr.makeStringAndClear(), 
RTL_TEXTENCODING_UTF8));   // abort in non-pro version, this must be fixed!
@@ -4411,7 +4411,7 @@ Window::~Window()
 {
 rtl::OStringBuffer aTempStr("Window (");
 aTempStr.append(rtl::OUStringToOString(GetText(), 
RTL_TEXTENCODING_UTF8));
-aTempStr.append(") with living Child(s) destroyed: ");
+aTempStr.append(") with live children destroyed: ");
 pTempWin = mpWindowImpl->mpFirstChild;
 while ( pTempWin )
 {
@@ -4426,7 +4426,7 @@ Window::~Window()
 {
 rtl::OStringBuffer aTempStr("Window (");
 aTempStr.append(rtl::OUStringToOString(GetText(), 
RTL_TEXTENCODING_UTF8));
-aTempStr.append(") with living SystemWindow(s) destroyed: ");
+   

[Libreoffice-commits] core.git: 9 commits - basebmp/source ios/CustomTarget_Viewer_app.mk ios/experimental sal/inc sc/source tools/inc vcl/ios

2013-03-26 Thread Tor Lillqvist
 basebmp/source/bitmapdevice.cxx  |3 -
 ios/CustomTarget_Viewer_app.mk   |   12 
 ios/experimental/Viewer/Viewer.xcodeproj/project.pbxproj |   38 ++-
 ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm|6 +-
 ios/experimental/Viewer/Viewer/lo-viewer.mm  |4 +
 sal/inc/osl/detail/ios-bootstrap.h   |1 
 sc/source/ui/docshell/docsh4.cxx |3 -
 tools/inc/tools/gen.hxx  |7 +-
 vcl/ios/iosinst.cxx  |8 +++
 9 files changed, 75 insertions(+), 7 deletions(-)

New commits:
commit bcb57baaae2d7e6914ab4dd8eb5232aeacddbb72
Author: Tor Lillqvist 
Date:   Tue Mar 26 00:24:29 2013 +0200

Use the applicationFrame and not whole screen size

Change-Id: I33a67b4908759913e49608110cc2635cc50e54b1

diff --git a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm 
b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
index 7e7216f..271342b 100644
--- a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
+++ b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
@@ -26,7 +26,7 @@ static UIWindow *theWindow;
 (void) application;
 (void) launchOptions;
 
-CGRect bounds = [[UIScreen mainScreen] bounds];
+CGRect bounds = [[UIScreen mainScreen] applicationFrame];
 self.window = [[LOViewerWindow alloc] initWithFrame:bounds];
 
 theWindow = self.window;
commit 6d7d274151d18be9c806f72ac3ce10638e483bea
Author: Tor Lillqvist 
Date:   Mon Mar 25 20:29:28 2013 +0200

I think kCGImageAlphaNoneSkipLast is what we want

Change-Id: I19a5ab15650cef4ee834af63e19bea7807b77477

diff --git a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm 
b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
index 9f72b35..7e7216f 100644
--- a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
+++ b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
@@ -39,7 +39,7 @@ static UIWindow *theWindow;
 memset(pixelBuffer, 0xFF, nbytes);
 
 CGDataProviderRef provider = CGDataProviderCreateWithData( NULL, 
pixelBuffer, nbytes, NULL);
-image = CGImageCreate(bounds.size.width, bounds.size.height, 8, 32, 
bounds.size.width*4, CGColorSpaceCreateDeviceRGB(), kCGImageAlphaFirst, 
provider, NULL, false, kCGRenderingIntentDefault);
+image = CGImageCreate(bounds.size.width, bounds.size.height, 8, 32, 
bounds.size.width*4, CGColorSpaceCreateDeviceRGB(), kCGImageAlphaNoneSkipLast, 
provider, NULL, false, kCGRenderingIntentDefault);
 
 self.window.bounds = bounds;
 self.window.pixelBuffer = pixelBuffer;
commit 881814077515605ab4f816530b81709663de50a2
Author: Tor Lillqvist 
Date:   Mon Mar 25 19:52:30 2013 +0200

Introduce temporary lo_set_view_size() hack like on Android and call it

Change-Id: I44df0946f59d1b9a2a6ea935b3c2ea3c96c1260d

diff --git a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm 
b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
index f9678c4..9f72b35 100644
--- a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
+++ b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
@@ -45,6 +45,8 @@ static UIWindow *theWindow;
 self.window.pixelBuffer = pixelBuffer;
 self.window.image = image;
 
+lo_set_view_size(bounds.size.width, bounds.size.height);
+
 [self.window makeKeyAndVisible];
 
 NSThread* thread = [[NSThread alloc] initWithTarget:self
diff --git a/sal/inc/osl/detail/ios-bootstrap.h 
b/sal/inc/osl/detail/ios-bootstrap.h
index d5fd947..4738f28 100644
--- a/sal/inc/osl/detail/ios-bootstrap.h
+++ b/sal/inc/osl/detail/ios-bootstrap.h
@@ -42,6 +42,7 @@ void lo_damaged();
 /* 2) */
 
 void lo_runMain();
+void lo_set_view_size(int width, int height);
 void lo_render_windows(char *pixelBuffer, int width, int height);
 
 #ifdef __cplusplus
diff --git a/vcl/ios/iosinst.cxx b/vcl/ios/iosinst.cxx
index b2178e9..8e3137b 100644
--- a/vcl/ios/iosinst.cxx
+++ b/vcl/ios/iosinst.cxx
@@ -297,6 +297,14 @@ int IosSalSystem::ShowNativeDialog( const rtl::OUString& 
rTitle,
 }
 
 extern "C"
+void lo_set_view_size(int width, int height)
+{
+// Horrible
+viewWidth = width;
+viewHeight = height;
+}
+
+extern "C"
 void lo_render_windows(char *pixelBuffer, int width, int height)
 {
 // Hack: assume so far that we are asked to redraw the whole pixel buffer
commit 89d9b9e252f0584afb0e0d995c80c114697a7aa1
Author: Tor Lillqvist 
Date:   Mon Mar 25 17:55:19 2013 +0200

Log also corresponding destructions of BitmapDevice

Change-Id: Ic74032b430691215482172c3c5f834374d47873d

diff --git a/basebmp/source/bitmapdevice.cxx b/basebmp/source/bitmapdevice.cxx
index 24916f6..a6774d1 100644
--- a/basebmp/source/bitmapdevice.cxx
+++ b/basebmp/source/bitmapdevice.cxx
@@ -1090,6 +1090,7 @@ BitmapDevice::BitmapDevice( const basegfx::B2IBox&
   rBounds,
 BitmapDevice::~BitmapDevice()
 {
 // outline, because of internal Impl

Re: [PATCH] Init: fixed typo in the SystemDialog class.

2013-03-26 Thread Petr Mladek
Michael Meeks píše v Po 25. 03. 2013 v 13:28 +:
> On Mon, 2013-03-25 at 12:57 +0100, Javier Fernandez wrote:
> > Yes :) sorry about that. It was only for the first push; I had a big
> > commit before, but I thought it was easier to review it that way. I was
> > not sure what would be the result, in terms of gerrit process.
> 
>   Hey - that's no problem. I've merged ~everything to master - except for
> the changes that enable the new component - so (hopefully) it sits there
> without causing issues in master since it's not enabled yet.

I guess that you did not merge the following two commits by purpose:

   + https://gerrit.libreoffice.org/#/c/3027/
 Pack and register the Web wizard.

   + https://gerrit.libreoffice.org/#/c/3028/
 Ugly Hack: using our own WebConfigSet while the Topic stuff is not
 integrated.

I do not see any comments in gerrit. What are the plans here? ;-)

Best Regards,
Petr


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


[Libreoffice-commits] core.git: 9 commits - sc/source sw/source vcl/coretext vcl/headless vcl/inc vcl/ios vcl/source

2013-03-26 Thread Tor Lillqvist
 sc/source/core/data/documen8.cxx   |4 
 sw/source/core/doc/docnew.cxx  |4 
 vcl/coretext/salcoretextlayout.cxx |   17 -
 vcl/coretext/salgdi.cxx|   34 ++
 vcl/headless/svpbmp.cxx|1 +
 vcl/headless/svpgdi.cxx|   24 
 vcl/inc/vcl/sysdata.hxx|6 --
 vcl/inc/vcl/virdev.hxx |   28 +++-
 vcl/ios/iosinst.cxx|   16 
 vcl/source/gdi/virdev.cxx  |6 --
 10 files changed, 98 insertions(+), 42 deletions(-)

New commits:
commit 4ac0eff680a9fbf81dfd9e8a5772dee93bd0fb1a
Author: Tor Lillqvist 
Date:   Tue Mar 26 18:32:41 2013 +0200

Re-add change from 304cb6ab8fb159c883c42d2d42e82750fab4e4a7

Was accidentally reverted.

Change-Id: I1d62003cfab222664b7cf2053f640287910b2892

diff --git a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm 
b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
index 7e7216f..271342b 100644
--- a/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
+++ b/ios/experimental/Viewer/Viewer/LOViewerAppDelegate.mm
@@ -26,7 +26,7 @@ static UIWindow *theWindow;
 (void) application;
 (void) launchOptions;
 
-CGRect bounds = [[UIScreen mainScreen] bounds];
+CGRect bounds = [[UIScreen mainScreen] applicationFrame];
 self.window = [[LOViewerWindow alloc] initWithFrame:bounds];
 
 theWindow = self.window;
commit 92b973fadcf63409026237884e81a1e3f3e62660
Author: Tor Lillqvist 
Date:   Mon Mar 25 08:04:27 2013 +0200

Use an 8bpp virtual device (instead of a 1bpp one) in sw and sc for iOS

The reason why a 1bpp virtual device is used in these two places in sc
and sw is unclear to me. It causes complications on iOS as such a
bitmap gets passed to CGBitmapContextCreate() which does not accept
1bpp bitmaps. But let's keep the 1bpp for other platforms.

Change-Id: Ia34927cf728d4be05a31e88b7da78200d0b799ba

diff --git a/sc/source/core/data/documen8.cxx b/sc/source/core/data/documen8.cxx
index f535235..9a6c0fb 100644
--- a/sc/source/core/data/documen8.cxx
+++ b/sc/source/core/data/documen8.cxx
@@ -207,7 +207,11 @@ VirtualDevice* ScDocument::GetVirtualDevice_100th_mm()
 {
 if (!pVirtualDevice_100th_mm)
 {
+#ifdef IOS
+pVirtualDevice_100th_mm = new VirtualDevice( 8 );
+#else
 pVirtualDevice_100th_mm = new VirtualDevice( 1 );
+#endif
 
pVirtualDevice_100th_mm->SetReferenceDevice(VirtualDevice::REFDEV_MODE_MSO1);
 MapMode aMapMode( pVirtualDevice_100th_mm->GetMapMode() );
 aMapMode.SetMapUnit( MAP_100TH_MM );
diff --git a/sw/source/core/doc/docnew.cxx b/sw/source/core/doc/docnew.cxx
index 7ffacc3..c978716 100644
--- a/sw/source/core/doc/docnew.cxx
+++ b/sw/source/core/doc/docnew.cxx
@@ -698,7 +698,11 @@ SwDoc::~SwDoc()
 
 VirtualDevice& SwDoc::CreateVirtualDevice_() const
 {
+#ifdef IOS
+VirtualDevice* pNewVir = new VirtualDevice( 8 );
+#else
 VirtualDevice* pNewVir = new VirtualDevice( 1 );
+#endif
 
 pNewVir->SetReferenceDevice( VirtualDevice::REFDEV_MODE_MSO1 );
 
diff --git a/vcl/inc/vcl/virdev.hxx b/vcl/inc/vcl/virdev.hxx
index 9c8c4ed..5d8a70e 100644
--- a/vcl/inc/vcl/virdev.hxx
+++ b/vcl/inc/vcl/virdev.hxx
@@ -65,23 +65,41 @@ private:
 SAL_DLLPRIVATE bool ForceZeroExtleadBug() const
 { return ((meRefDevMode & REFDEV_FORCE_ZERO_EXTLEAD) != 0); }
 public:
+/** Create a virtual device of size 1x1
+
+@param nBitCount
+Bit depth of the generated virtual device. Use 0 here, to
+indicate: take default screen depth. Only 0, 1 and 8
+are allowed here, with 1 denoting binary mask and 8 a graylevel mask.
+ */
 VirtualDevice( sal_uInt16 nBitCount = 0 );
+
+/** Create a virtual device of size 1x1
+
+@param rCompDev
+The generated vdev will be compatible to this device.
+
+@param nBitCount
+Bit depth of the generated virtual device. Use 0 here, to
+indicate: take default screen depth. Only 0 and 1
+are allowed here, with 1 denoting binary mask.
+ */
 VirtualDevice( const OutputDevice& rCompDev,
sal_uInt16 nBitCount = 0 );
-/** Create a virtual device with alpha channel
+
+/** Create a virtual device  of size 1x1 with alpha channel
 
 @param rCompDev
-The generated vdev will be compatible to this device. By
-default, Application::GetDefaultDevice() is used here.
+The generated vdev will be compatible to this device.
 
 @param nBitCount
 Bit depth of the generated virtual device. Use 0 here, to
-indicate: take default screen depth. Currently, only 0 and 1
+indicate: take default screen depth. Only 0 and 1
 are allowed here, with 1 denoting binary mask.
 
 @param nAlphaBitCount

[Libreoffice-commits] core.git: lotuswordpro/source

2013-03-26 Thread Tor Lillqvist
 lotuswordpro/source/filter/lwpoleobject.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 700b6c1cbf4fee4044aa2b7179d3f1886ae928c6
Author: Tor Lillqvist 
Date:   Tue Mar 26 19:24:15 2013 +0200

Fix warning: unused variable 'sFormat' [loplugin]

Presumably the call to m_pObjStrm->QuickReadStringPtr() is still necessary
because of its side-effects (to update the stream pointer).

Change-Id: I69f24de4e956d5fefb4fc5382c0cc7e0173a5a0f

diff --git a/lotuswordpro/source/filter/lwpoleobject.cxx 
b/lotuswordpro/source/filter/lwpoleobject.cxx
index c818118..842e199 100644
--- a/lotuswordpro/source/filter/lwpoleobject.cxx
+++ b/lotuswordpro/source/filter/lwpoleobject.cxx
@@ -220,7 +220,7 @@ void LwpOleObject::Read()
 {
 m_pObjStrm->QuickReaduInt16();
 
-OUString sFormat = m_pObjStrm->QuickReadStringPtr();
+m_pObjStrm->QuickReadStringPtr();
 
 if (LwpFileHeader::m_nFileRevision < 0x000B)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


  1   2   3   >