[Libreoffice-commits] core.git: canvas/source cppcanvas/qa cui/source drawinglayer/source filter/source include/vcl sd/qa sd/source svtools/source svx/source sw/qa vcl/backendtest vcl/qa vcl/quartz vc

2018-02-05 Thread Chris Sherlock
 canvas/source/cairo/cairo_canvashelper.cxx|4 -
 cppcanvas/qa/extras/emfplus/emfplus.cxx   |2 
 cui/source/dialogs/colorpicker.cxx|2 
 cui/source/tabpages/tppattern.cxx |4 -
 drawinglayer/source/texture/texture3d.cxx |2 
 filter/source/msfilter/msdffimp.cxx   |4 -
 include/vcl/salbtype.hxx  |6 -
 sd/qa/unit/import-tests.cxx   |6 -
 sd/source/ui/slidesorter/view/SlsFramePainter.cxx |3 
 svtools/source/graphic/transformer.cxx|   74 ++-
 svx/source/dialog/dlgctrl.cxx |   18 +---
 sw/qa/extras/ooxmlexport/ooxmlexport10.cxx|6 -
 vcl/backendtest/outputdevice/common.cxx   |2 
 vcl/qa/cppunit/BitmapProcessorTest.cxx|2 
 vcl/qa/cppunit/BitmapTest.cxx |2 
 vcl/qa/cppunit/jpeg/JpegReaderTest.cxx|8 +-
 vcl/qa/cppunit/outdev.cxx |   16 ++--
 vcl/quartz/salbmp.cxx |6 -
 vcl/source/bitmap/BitmapProcessor.cxx |   16 ++--
 vcl/source/filter/igif/gifread.cxx|   33 
 vcl/source/gdi/bitmapex.cxx   |   13 +--
 vcl/source/gdi/pngread.cxx|   29 ---
 vcl/source/gdi/print2.cxx |   82 +++---
 vcl/unx/generic/gdi/gdiimpl.cxx   |4 -
 vcl/win/gdi/gdiimpl.cxx   |4 -
 25 files changed, 180 insertions(+), 168 deletions(-)

New commits:
commit 01e470f0ab49e458c57b0f28636a517b2bfb1b4a
Author: Chris Sherlock 
Date:   Sun Jan 21 04:48:50 2018 +1100

vcl: remove BitmapColor Color() operator

BitmapColor has a Color() operator. However, this is confusing and
tends to hide that the two classes aren't the same. I have converted this
to GetColor().

Change-Id: I0be2dcb3fc420e7be9c8d04330e7a3fe69a5412a
Reviewed-on: https://gerrit.libreoffice.org/48245
Reviewed-by: Stephan Bergmann 
Tested-by: Stephan Bergmann 

diff --git a/canvas/source/cairo/cairo_canvashelper.cxx 
b/canvas/source/cairo/cairo_canvashelper.cxx
index a25dac85acc6..bba59a15c475 100644
--- a/canvas/source/cairo/cairo_canvashelper.cxx
+++ b/canvas/source/cairo/cairo_canvashelper.cxx
@@ -470,7 +470,7 @@ namespace cairocanvas
 else
 nAlpha = data[ nOff + 3 ] = 255;
 #endif
-aColor = pBitmapReadAcc->GetPaletteColor( 
*pReadScan++ );
+aColor = 
pBitmapReadAcc->GetPaletteColor(*pReadScan++).GetColor();
 
 #ifdef OSL_BIGENDIAN
 data[ nOff++ ] = sal::static_int_cast(( nAlpha*( aColor.GetRed() ) )/255 );
@@ -611,7 +611,7 @@ namespace cairocanvas
 
 for( nX = 0; nX < nWidth; nX++ )
 {
-aColor = pBitmapReadAcc->GetColor( nY, nX );
+aColor = pBitmapReadAcc->GetColor( nY, nX 
).GetColor();
 
 // cairo need premultiplied color values
 // TODO(rodo) handle endianness
diff --git a/cppcanvas/qa/extras/emfplus/emfplus.cxx 
b/cppcanvas/qa/extras/emfplus/emfplus.cxx
index b5a32b1fca94..fbc55d2f2181 100644
--- a/cppcanvas/qa/extras/emfplus/emfplus.cxx
+++ b/cppcanvas/qa/extras/emfplus/emfplus.cxx
@@ -80,7 +80,7 @@ void Test::testFdo77229()
 Bitmap aBitmap = load("fdo77229.emf");
 Bitmap::ScopedReadAccess pAccess(aBitmap);
 // The green star was missing.
-Color aColor(pAccess->GetPixel(142, 140));
+Color aColor(pAccess->GetPixel(142, 140).GetColor());
 CPPUNIT_ASSERT_EQUAL(sal_uInt8(0), aColor.GetRed());
 CPPUNIT_ASSERT_EQUAL(sal_uInt8(0), aColor.GetBlue());
 CPPUNIT_ASSERT(aColor.GetGreen() == 0xfe || aColor.GetGreen() == 0xff);
diff --git a/cui/source/dialogs/colorpicker.cxx 
b/cui/source/dialogs/colorpicker.cxx
index 687e44677825..6492ad558a3e 100644
--- a/cui/source/dialogs/colorpicker.cxx
+++ b/cui/source/dialogs/colorpicker.cxx
@@ -447,7 +447,7 @@ void ColorFieldControl::ShowPosition( const Point& rPos, 
bool bUpdate )
 if (pReadAccess != nullptr)
 {
 // mpBitmap always has a bit count of 24 => use of GetPixel(...) 
is safe
-maColor = pReadAccess->GetPixel(nY, nX);
+maColor = pReadAccess->GetPixel(nY, nX).GetColor();
 Bitmap::ReleaseAccess(pReadAccess);
 pReadAccess = nullptr;
 }
diff --git a/cui/source/tabpages/tppattern.cxx 
b/cui/source/tabpages/tppattern.cxx
index 5422e9c0334f..141bd052cc66 100644
--- a/cui/source/tabpages/tppattern.cxx
+++ b/cui/source/tabpages/tppattern.cxx
@@ -313,8 +313,8 @@ IMPL_LINK_NOARG(SvxPatternTabPage, ChangePatternHdl_Impl, 
ValueSet*, void)
 
 
m_pCtlPixel->SetXBitmap(pGraphicObject->GetGraphic().GetBitmapEx

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3' - cui/source include/vcl

2018-02-05 Thread Tamás Zolnai
 cui/source/tabpages/backgrnd.cxx |   22 ++
 include/vcl/window.hxx   |2 +-
 2 files changed, 23 insertions(+), 1 deletion(-)

New commits:
commit 81e6d59738c67e2050428bce9c0f490eebacb7f2
Author: Tamás Zolnai 
Date:   Fri Feb 2 19:10:19 2018 +0100

lokdialog: Color/font picker in dialogs don't show preview

Invalidation needs to be passed upward to the right parent window.
It should work similar to other dialog items, but this preview class
is not derived from the Control class.

Reviewed-on: https://gerrit.libreoffice.org/49101
Tested-by: Jenkins 
Reviewed-by: Tamás Zolnai 
(cherry picked from commit 974ea04ddd1b82e5d5c0171bd8ad29310e3f2bd7)

Change-Id: I5fb2b6438b8be92d55609c8d3c25110b49d0e6d7
Reviewed-on: https://gerrit.libreoffice.org/49160
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx
index 43d8fdd2eddb..2554da1581b7 100644
--- a/cui/source/tabpages/backgrnd.cxx
+++ b/cui/source/tabpages/backgrnd.cxx
@@ -23,6 +23,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -51,6 +52,7 @@
 #include 
 #include 
 #include 
+#include 
 
 using namespace css;
 
@@ -160,6 +162,7 @@ protected:
 virtual voidPaint( vcl::RenderContext& /*rRenderContext*/, const 
Rectangle& rRect ) override;
 virtual voidDataChanged( const DataChangedEvent& rDCEvt ) override;
 virtual voidResize() override;
+virtual voidLogicInvalidate(const Rectangle* pRectangle) override;
 
 private:
 
@@ -321,6 +324,25 @@ void BackgroundPreviewImpl::DataChanged( const 
DataChangedEvent& rDCEvt )
 Window::DataChanged( rDCEvt );
 }
 
+
+void BackgroundPreviewImpl::LogicInvalidate(const Rectangle* /*pRectangle*/)
+{
+// Invalidate the container dialog or floating window
+// The code is same as in Control::LogicInvalidate() method
+if (comphelper::LibreOfficeKit::isActive() && 
!comphelper::LibreOfficeKit::isDialogPainting())
+{
+if (VclPtr pParent = GetParentWithLOKNotifier())
+{
+// invalidate the complete floating window for now
+if (pParent->ImplIsFloatingWindow())
+return pParent->LogicInvalidate(nullptr);
+
+const Rectangle aRect(Point(GetOutOffXPixel(), GetOutOffYPixel()), 
Size(GetOutputWidthPixel(), GetOutputHeightPixel()));
+pParent->LogicInvalidate(&aRect);
+}
+}
+}
+
 #define HDL(hdl) LINK(this,SvxBackgroundTabPage,hdl)
 
 SvxBackgroundTabPage::SvxBackgroundTabPage(vcl::Window* pParent, const 
SfxItemSet& rCoreSet)
diff --git a/include/vcl/window.hxx b/include/vcl/window.hxx
index 7f15b5dac267..dcdaddc75c12 100644
--- a/include/vcl/window.hxx
+++ b/include/vcl/window.hxx
@@ -560,7 +560,7 @@ public:
 
 SAL_DLLPRIVATE bool ImplIsWindowOrChild( const 
vcl::Window* pWindow, bool bSystemWindow = false ) const;
 SAL_DLLPRIVATE bool ImplIsChild( const vcl::Window* 
pWindow, bool bSystemWindow = false ) const;
-SAL_DLLPRIVATE bool ImplIsFloatingWindow() const;
+   bool ImplIsFloatingWindow() const;
 SAL_DLLPRIVATE bool ImplIsPushButton() const;
 SAL_DLLPRIVATE bool ImplIsSplitter() const;
 SAL_DLLPRIVATE bool ImplIsOverlapWindow() const;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: uitest/manual_tests

2018-02-05 Thread Saurav Chirania
 uitest/manual_tests/more_calc.py |   52 +++
 1 file changed, 52 insertions(+)

New commits:
commit b57ad983e6ed5d976903b1d6588b472bb3051e7d
Author: Saurav Chirania 
Date:   Sat Feb 3 01:58:42 2018 +0530

tdf#107501 automatize EN-8 manual test, related tdf#84810

This test is to check that paste special combined
with some options and link is ok.

Change-Id: Ifffca8033b195d64d310d9d063ee95bbc8e8ad5d
Reviewed-on: https://gerrit.libreoffice.org/49164
Reviewed-by: Markus Mohrhard 
Tested-by: Markus Mohrhard 

diff --git a/uitest/manual_tests/more_calc.py b/uitest/manual_tests/more_calc.py
new file mode 100644
index ..08aeb654e449
--- /dev/null
+++ b/uitest/manual_tests/more_calc.py
@@ -0,0 +1,52 @@
+# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-offset: 4 -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+from uitest.framework import UITestCase
+
+from libreoffice.calc.document import get_cell_by_position
+from libreoffice.uno.propertyvalue import mkPropertyValues
+
+from uitest.uihelper.calc import enter_text_to_cell
+
+
+class ManualCalcTests(UITestCase):
+def test_paste_special(self):
+# EN-8:Paste special with options
+# This test is to check that paste special combined with some options 
and link is ok.
+# Refers to tdf#84810
+
+self.ui_test.create_doc_in_start_center("calc")
+
+# Write text to cell A1
+xGridWin = self.xUITest.getTopFocusWindow().getChild("grid_window")
+enter_text_to_cell(xGridWin, "A1", "abcd")
+
+# Copy cell A1 to clipboard
+xGridWin.executeAction("SELECT", mkPropertyValues({"CELL": "A1"}))
+self.xUITest.executeCommand(".uno:Copy")
+
+# Set cursor to cell A3
+xGridWin.executeAction("SELECT", mkPropertyValues({"CELL": "A3"}))
+
+# Choose Paste Special Options and paste data
+self.ui_test.execute_dialog_through_command(".uno:PasteSpecial")
+xPasteSpecialDlg = self.xUITest.getTopFocusWindow()
+xAllChkBox = xPasteSpecialDlg.getChild("paste_all")
+xAllChkBox.executeAction("CLICK", tuple())
+xLinkChkBox = xPasteSpecialDlg.getChild("link")
+xLinkChkBox.executeAction("CLICK", tuple())
+xOkBtn = xPasteSpecialDlg.getChild("ok")
+self.ui_test.close_dialog_through_button(xOkBtn)
+
+# Assert successful paste
+document = self.ui_test.get_component()
+self.assertEqual(get_cell_by_position(document, 0, 0, 2).getString(), 
"abcd")
+self.ui_test.close_doc()
+
+# vim: set shiftwidth=4 softtabstop=4 expandtab:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sc/source/filter/excel/namebuff.cxx |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit ac3757b8becabc56081f9a454275ccc10bca2cda
Author: Johnny_M 
Date:   Sun Feb 4 00:48:57 2018 +0100

Translate German variable names

Akt -> Current in Excel filter (namebuff)

Change-Id: Idf36f8e19bb96b1d71edb49138c0ab36856c73ce
Reviewed-on: https://gerrit.libreoffice.org/49192
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/excel/namebuff.cxx 
b/sc/source/filter/excel/namebuff.cxx
index e08a46653666..b25edaa010dd 100644
--- a/sc/source/filter/excel/namebuff.cxx
+++ b/sc/source/filter/excel/namebuff.cxx
@@ -33,15 +33,15 @@
 sal_uInt32 StringHashEntry::MakeHashCode( const OUString& r )
 {
 sal_uInt32 n = 0;
-const sal_Unicode* pAkt = r.getStr();
-sal_UnicodecAkt = *pAkt;
+const sal_Unicode* pCurrent = r.getStr();
+sal_UnicodecCurrent = *pCurrent;
 
-while( cAkt )
+while( cCurrent )
 {
 n *= 70;
-n += static_cast(cAkt);
-pAkt++;
-cAkt = *pAkt;
+n += static_cast(cCurrent);
+pCurrent++;
+cCurrent = *pCurrent;
 }
 
 return n;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sc/source/filter/excel/read.cxx |  130 
 1 file changed, 65 insertions(+), 65 deletions(-)

New commits:
commit d757c695f9736c064735017c02380eb6257a7875
Author: Johnny_M 
Date:   Sun Feb 4 00:39:25 2018 +0100

Translate German variable names

Akt -> Current
Zustand -> STATE
Ende -> End

in Excel filter (read)

Change-Id: I4bbb59fb8de1b59ecb4c8fd1ce0765e654acd10f
Reviewed-on: https://gerrit.libreoffice.org/49193
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/excel/read.cxx b/sc/source/filter/excel/read.cxx
index b9d77a0b2b51..54fd40dea65a 100644
--- a/sc/source/filter/excel/read.cxx
+++ b/sc/source/filter/excel/read.cxx
@@ -74,7 +74,7 @@ ErrCode ImportExcel::Read()
 XclImpNameManager&  rNameMgr= GetNameManager();
 // call to GetCurrSheetDrawing() cannot be cached (changes in new sheets)
 
-enum Zustand {
+enum STATE {
 Z_BiffNull, // not a valid Biff-Format
 Z_Biff2,// Biff2: only one table
 
@@ -91,9 +91,9 @@ ErrCode ImportExcel::Read()
 Z_Biff5T,   // Biff5: a table itself
 Z_Biff5E,   // Biff5: between tables
 Z_Biffn0,   // all Biffs: skip table till next EOF
-Z_Ende };
+Z_End };
 
-Zustand eAkt = Z_BiffNull, ePrev = Z_BiffNull;
+STATE   eCurrent = Z_BiffNull, ePrev = Z_BiffNull;
 
 ErrCode eLastErr = ERRCODE_NONE;
 sal_uInt16  nOpcode;
@@ -107,9 +107,9 @@ ErrCode ImportExcel::Read()
 std::size_t nProgressBasePos = 0;
 std::size_t nProgressBaseSize = 0;
 
-for (; eAkt != Z_Ende; mnLastRecId = nOpcode)
+for (; eCurrent != Z_End; mnLastRecId = nOpcode)
 {
-if( eAkt == Z_Biff5E )
+if( eCurrent == Z_Biff5E )
 {
 sal_uInt16 nScTab = GetCurrScTab();
 if( nScTab < maSheetOffsets.size()  )
@@ -121,11 +121,11 @@ ErrCode ImportExcel::Read()
 if (!bValid)
 {
 // Safeguard ourselves from potential infinite loop.
-eAkt = Z_Ende;
+eCurrent = Z_End;
 }
 }
 else
-eAkt = Z_Ende;
+eCurrent = Z_End;
 }
 else
 aIn.StartNextRecord();
@@ -135,7 +135,7 @@ ErrCode ImportExcel::Read()
 if( !aIn.IsValid() )
 {
 // finalize table if EOF is missing
-switch( eAkt )
+switch( eCurrent )
 {
 case Z_Biff2:
 case Z_Biff3:
@@ -151,13 +151,13 @@ ErrCode ImportExcel::Read()
 break;
 }
 
-if( eAkt == Z_Ende )
+if( eCurrent == Z_End )
 break;
 
-if( eAkt != Z_Biff5TPre && eAkt != Z_Biff5WPre )
+if( eCurrent != Z_Biff5TPre && eCurrent != Z_Biff5WPre )
 pProgress->ProgressAbs( nProgressBaseSize + aIn.GetSvStreamPos() - 
nProgressBasePos );
 
-switch( eAkt )
+switch( eCurrent )
 {
 
 case Z_BiffNull:// --- Z_BiffNull -
@@ -176,7 +176,7 @@ ErrCode ImportExcel::Read()
 Bof2();
 if( pExcRoot->eDateiTyp == Biff2 )
 {
-eAkt = Z_Biff2;
+eCurrent = Z_Biff2;
 NewTable();
 }
 break;
@@ -184,7 +184,7 @@ ErrCode ImportExcel::Read()
 Bof3();
 if( pExcRoot->eDateiTyp == Biff3 )
 {
-eAkt = Z_Biff3;
+eCurrent = Z_Biff3;
 NewTable();
 }
 break;
@@ -192,17 +192,17 @@ ErrCode ImportExcel::Read()
 Bof4();
 if( pExcRoot->eDateiTyp == Biff4 )
 {
-eAkt = Z_Biff4;
+eCurrent = Z_Biff4;
 NewTable();
 }
 else if( pExcRoot->eDateiTyp == Biff4W )
-eAkt = Z_Biff4W;
+eCurrent = Z_Biff4W;
 break;
 case EXC_BIFF5:
 Bof5();
 if( pExcRoot->eDateiTyp == Biff5W )
 {
-eAkt = Z_Biff5WPre;
+eCurrent = Z_Biff5WPre;
 
 nBdshtTab = 0;

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

2018-02-05 Thread Johnny_M
 sc/source/filter/excel/tokstack.cxx |  198 ++--
 sc/source/filter/inc/tokstack.hxx   |   28 ++---
 2 files changed, 113 insertions(+), 113 deletions(-)

New commits:
commit f7fb78135c8747a8bca861b736960899dad295d4
Author: Johnny_M 
Date:   Sun Feb 4 01:11:18 2018 +0100

Translate German variable names

Akt -> Current in Excel filter (tokstack)

Change-Id: I1d7fe8e0116fa9db0df806f50f805e34f0955f56
Reviewed-on: https://gerrit.libreoffice.org/49197
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/excel/tokstack.cxx 
b/sc/source/filter/excel/tokstack.cxx
index 01cf12452ec0..1296f319bd6a 100644
--- a/sc/source/filter/excel/tokstack.cxx
+++ b/sc/source/filter/excel/tokstack.cxx
@@ -110,14 +110,14 @@ bool TokenPool::GrowId()
 
 bool TokenPool::CheckElementOrGrow()
 {
-// Last possible ID to be assigned somewhere is nElementAkt+1
-if (nElementAkt + 1 == nScTokenOff - 1)
+// Last possible ID to be assigned somewhere is nElementCurrent+1
+if (nElementCurrent + 1 == nScTokenOff - 1)
 {
-SAL_WARN("sc.filter","TokenPool::CheckElementOrGrow - last possible ID 
" << nElementAkt+1);
+SAL_WARN("sc.filter","TokenPool::CheckElementOrGrow - last possible ID 
" << nElementCurrent+1);
 return false;
 }
 
-if (nElementAkt >= nElement)
+if (nElementCurrent >= nElement)
 return GrowElement();
 
 return true;
@@ -176,9 +176,9 @@ bool TokenPool::GrowMatrix()
 
 bool TokenPool::GetElement( const sal_uInt16 nId )
 {
-if (nId >= nElementAkt)
+if (nId >= nElementCurrent)
 {
-SAL_WARN("sc.filter","TokenPool::GetElement - Id too large, " << nId 
<< " >= " << nElementAkt);
+SAL_WARN("sc.filter","TokenPool::GetElement - Id too large, " << nId 
<< " >= " << nElementCurrent);
 return false;
 }
 
@@ -343,11 +343,11 @@ bool TokenPool::GetElementRek( const sal_uInt16 nId )
 OSL_ENSURE(m_nRek <= nP_Id, "*TokenPool::GetElement(): recursion loops!?");
 #endif
 
-OSL_ENSURE( nId < nElementAkt, "*TokenPool::GetElementRek(): nId >= 
nElementAkt" );
+OSL_ENSURE( nId < nElementCurrent, "*TokenPool::GetElementRek(): nId >= 
nElementCurrent" );
 
-if (nId >= nElementAkt)
+if (nId >= nElementCurrent)
 {
-SAL_WARN("sc.filter", "*TokenPool::GetElementRek(): nId >= 
nElementAkt");
+SAL_WARN("sc.filter", "*TokenPool::GetElementRek(): nId >= 
nElementCurrent");
 #ifdef DBG_UTIL
 m_nRek--;
 #endif
@@ -372,33 +372,33 @@ bool TokenPool::GetElementRek( const sal_uInt16 nId )
 nCnt = 0;
 bRet = false;
 }
-sal_uInt16* pAkt = nCnt ? &pP_Id[ nFirstId ] : nullptr;
+sal_uInt16* pCurrent = nCnt ? &pP_Id[ nFirstId ] : nullptr;
 if (nCnt > nP_Id - nFirstId)
 {
 SAL_WARN("sc.filter", "TokenPool::GetElementRek: nCnt > nP_Id - 
nFirstId");
 nCnt = nP_Id - nFirstId;
 bRet = false;
 }
-for( ; nCnt > 0 ; nCnt--, pAkt++ )
+for( ; nCnt > 0 ; nCnt--, pCurrent++ )
 {
-assert(pAkt);
-if( *pAkt < nScTokenOff )
+assert(pCurrent);
+if( *pCurrent < nScTokenOff )
 {// recursion or not?
-if (*pAkt >= nElementAkt)
+if (*pCurrent >= nElementCurrent)
 {
-SAL_WARN("sc.filter", "TokenPool::GetElementRek: *pAkt >= 
nElementAkt");
+SAL_WARN("sc.filter", "TokenPool::GetElementRek: *pCurrent >= 
nElementCurrent");
 bRet = false;
 }
 else
 {
-if (pType[ *pAkt ] == T_Id)
-bRet = GetElementRek( *pAkt );
+if (pType[ *pCurrent ] == T_Id)
+bRet = GetElementRek( *pCurrent );
 else
-bRet = GetElement( *pAkt );
+bRet = GetElement( *pCurrent );
 }
 }
 else// elementary SC_Token
-pScToken->AddOpCode( static_cast( *pAkt - nScTokenOff 
) );
+pScToken->AddOpCode( static_cast( *pCurrent - 
nScTokenOff ) );
 }
 
 #ifdef DBG_UTIL
@@ -409,40 +409,40 @@ bool TokenPool::GetElementRek( const sal_uInt16 nId )
 
 void TokenPool::operator >>( TokenId& rId )
 {
-rId = static_cast( nElementAkt + 1 );
+rId = static_cast( nElementCurrent + 1 );
 
 if (!CheckElementOrGrow())
 return;
 
-pElement[ nElementAkt ] = nP_IdLast;// Start of Token-sequence
-pType[ nElementAkt ] = T_Id;// set Typeinfo
-pSize[ nElementAkt ] = nP_IdAkt - nP_IdLast;
-// write from nP_IdLast to nP_IdAkt-1 -> length of the sequence
+pElement[ nElementCurrent ] = nP_IdLast;// Start of Token-sequence
+pType[ nElementCurrent ] = T_Id;// set Typeinfo
+pSize[ nElementCurrent ] = nP_IdCurrent - nP_IdLast;
+// write from nP_IdLast to nP_IdCurrent-1 -> length of the sequence
 
-nElementAkt++;  // sta

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

2018-02-05 Thread Johnny_M
 sc/source/filter/lotus/lotread.cxx |   24 
 1 file changed, 12 insertions(+), 12 deletions(-)

New commits:
commit f8d7e734ec21f7e4a91289299bdd4d31b1bf4621
Author: Johnny_M 
Date:   Sat Feb 3 23:19:35 2018 +0100

Translate German variable names

Akt -> Current in Lotus filter

Change-Id: Icb13ea51c715df96223fd5b49f33d9321ef41d05
Reviewed-on: https://gerrit.libreoffice.org/49196
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/lotus/lotread.cxx 
b/sc/source/filter/lotus/lotread.cxx
index 165071c03931..23f9823629e1 100644
--- a/sc/source/filter/lotus/lotread.cxx
+++ b/sc/source/filter/lotus/lotread.cxx
@@ -47,7 +47,7 @@ ErrCode ImportLotus::Read()
 ErrCodeeRet = ERRCODE_NONE;
 //  ScFormulaCell   *pLastFormCell;
 
-STATE   eAkt = S_START;
+STATE   eCurrent = S_START;
 
 nTab = 0;
 nExtTab = -2;
@@ -57,28 +57,28 @@ ErrCode ImportLotus::Read()
 // start progressbar
 ScfStreamProgressBar aPrgrsBar( *pIn, pD->GetDocumentShell() );
 LotusContext &rContext = aConv.getContext();
-while( eAkt != S_END )
+while( eCurrent != S_END )
 {
 pIn->ReadUInt16( nOp ).ReadUInt16( nRecLen );
 
 if (!pIn->good() || nNextRec > SAL_MAX_UINT32 - nRecLen - 4)
 {
 eRet = SCERR_IMPORT_FORMAT;
-eAkt = S_END;
+eCurrent = S_END;
 if (!pIn->good())
 break;  // while
 }
 
 nNextRec += nRecLen + 4;
 
-switch( eAkt )
+switch( eCurrent )
 {
 
 case S_START:   // S_START
 if( nOp )
 {
 eRet = SCERR_IMPORT_UNKNOWN_WK;
-eAkt = S_END;
+eCurrent = S_END;
 }
 else
 {
@@ -87,16 +87,16 @@ ErrCode ImportLotus::Read()
 Bof();
 switch (rContext.pLotusRoot->eFirstType)
 {
-case Lotus123Typ::WK3: eAkt = S_WK3; break;
-case Lotus123Typ::WK4: eAkt = S_WK4; break;
+case Lotus123Typ::WK3: eCurrent = S_WK3; break;
+case Lotus123Typ::WK4: eCurrent = S_WK4; break;
 default:
 eRet = SCERR_IMPORT_UNKNOWN_WK;
-eAkt = S_END;
+eCurrent = S_END;
 }
 }
 else
 {
-eAkt = S_END;   // TODO: add here 
something for <= WK1!
+eCurrent = S_END;   // TODO: add here 
something for <= WK1!
 eRet = ErrCode(0x);
 }
 }
@@ -110,13 +110,13 @@ ErrCode ImportLotus::Read()
 switch( nOp )
 {
 case 0x0001:// EOF
-eAkt = S_FM3;
+eCurrent = S_FM3;
 nTab++;
 break;
 
 case 0x0002:// PASSWORD
 eRet = SCERR_IMPORT_FILEPASSWD;
-eAkt = S_END;
+eCurrent = S_END;
 break;
 
 case 0x0007:// COLUMNWIDTH
@@ -177,7 +177,7 @@ ErrCode ImportLotus::Read()
 else
 {
 eRet = SCERR_IMPORT_FORMAT;
-eAkt = S_END;
+eCurrent = S_END;
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sc/source/filter/dif/difimp.cxx |   78 
 sc/source/filter/inc/dif.hxx|2 -
 2 files changed, 40 insertions(+), 40 deletions(-)

New commits:
commit 2488768e402bcabd40c73e7c693ee2c1c9be2d69
Author: Johnny_M 
Date:   Sat Feb 3 22:56:56 2018 +0100

Translate German variable names

Akt -> Current in dif filter

Change-Id: Ibae58744a1d0b685b9df4d63b2977ec9b09745d0
Reviewed-on: https://gerrit.libreoffice.org/49189
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/dif/difimp.cxx b/sc/source/filter/dif/difimp.cxx
index c523b072ba02..349f8e4864d7 100644
--- a/sc/source/filter/dif/difimp.cxx
+++ b/sc/source/filter/dif/difimp.cxx
@@ -127,19 +127,19 @@ ErrCode ScFormatFilterPluginImpl::ScImportDif(SvStream& 
rIn, ScDocument* pDoc, c
 SCROW   nRowCnt = rInsPos.Row();
 DifAttrCacheaAttrCache;
 
-DATASET eAkt = D_UNKNOWN;
+DATASET eCurrent = D_UNKNOWN;
 
 ScSetStringParam aStrParam; // used to set string value without number 
detection.
 aStrParam.setTextInput();
 
-while( eAkt != D_EOD )
+while( eCurrent != D_EOD )
 {
-eAkt = aDifParser.GetNextDataset();
+eCurrent = aDifParser.GetNextDataset();
 
 aPrgrsBar.Progress();
 ScAddress aPos(nColCnt, nRowCnt, nBaseTab);
 
-switch( eAkt )
+switch( eCurrent )
 {
 case D_BOT:
 if( nColCnt < SCCOL_MAX )
@@ -434,30 +434,30 @@ bool DifParser::ReadNextLine( OUString& rStr )
 // a valid data record structure
 bool DifParser::LookAhead()
 {
-const sal_Unicode* pAktBuffer;
+const sal_Unicode* pCurrentBuffer;
 bool bValidStructure = false;
 
 OSL_ENSURE( aLookAheadLine.isEmpty(), "*DifParser::LookAhead(): LookAhead 
called twice in a row" );
 rIn.ReadUniOrByteStringLine( aLookAheadLine, rIn.GetStreamCharSet() );
 
-pAktBuffer = aLookAheadLine.getStr();
+pCurrentBuffer = aLookAheadLine.getStr();
 
-switch( *pAktBuffer )
+switch( *pCurrentBuffer )
 {
 case '-':   // Special Datatype
-pAktBuffer++;
+pCurrentBuffer++;
 
-if( Is1_0( pAktBuffer ) )
+if( Is1_0( pCurrentBuffer ) )
 {
 bValidStructure = true;
 }
 break;
 case '0':   // Numeric Data
-pAktBuffer++;
-if( *pAktBuffer == ',' )
+pCurrentBuffer++;
+if( *pCurrentBuffer == ',' )
 {
-pAktBuffer++;
-bValidStructure = ( GetNumberDataset(pAktBuffer) != 
D_SYNT_ERROR );
+pCurrentBuffer++;
+bValidStructure = ( GetNumberDataset(pCurrentBuffer) != 
D_SYNT_ERROR );
 }
 break;
 case '1':   // String Data
@@ -474,18 +474,18 @@ DATASET DifParser::GetNextDataset()
 {
 DATASET eRet = D_UNKNOWN;
 OUString aLine;
-const sal_Unicode* pAktBuffer;
+const sal_Unicode* pCurrentBuffer;
 
 ReadNextLine( aLine );
 
-pAktBuffer = aLine.getStr();
+pCurrentBuffer = aLine.getStr();
 
-switch( *pAktBuffer )
+switch( *pCurrentBuffer )
 {
 case '-':   // Special Datatype
-pAktBuffer++;
+pCurrentBuffer++;
 
-if( Is1_0( pAktBuffer ) )
+if( Is1_0( pCurrentBuffer ) )
 {
 ReadNextLine( aLine );
 if( IsBOT( aLine.getStr() ) )
@@ -495,17 +495,17 @@ DATASET DifParser::GetNextDataset()
 }
 break;
 case '0':   // Numeric Data
-pAktBuffer++;   // value in fVal, 2. line in aData
-if( *pAktBuffer == ',' )
+pCurrentBuffer++;   // value in fVal, 2. line in aData
+if( *pCurrentBuffer == ',' )
 {
-pAktBuffer++;
-eRet = GetNumberDataset(pAktBuffer);
+pCurrentBuffer++;
+eRet = GetNumberDataset(pCurrentBuffer);
 OUString aTmpLine;
 ReadNextLine( aTmpLine );
 if ( eRet == D_SYNT_ERROR )
 {   // for broken records write "#ERR: data" to cell
 OUStringBuffer aTmp("#ERR: ");
-aTmp.append(pAktBuffer).append(" (");
+aTmp.append(pCurrentBuffer).append(" (");
 aTmp.append(aTmpLine).append(')');
 aData = aTmp.makeStringAndClear();
 eRet = D_STRING;
@@ -588,30 +588,30 @@ const sal_Unicode* DifParser::ScanIntVal( const 
sal_Unicode* pStart, sal_uInt32&
 while (*pStart == ' ' || *pStart == '\t')
 ++pStart;
 
-sal_Unicode cAkt = *pStart;
+sal_Unicode cCurrent = *pStar

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

2018-02-05 Thread Johnny_M
 sc/source/filter/ftools/ftools.cxx |   16 
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit 3af813241cc8007bcc2077944e12af1a7dfac3b0
Author: Johnny_M 
Date:   Sun Feb 4 00:05:58 2018 +0100

Translate German variable names

Fakt -> Factor in ftools

Change-Id: I86f8d68015625c8ec7b3d260dbb3f734d2d7008e
Reviewed-on: https://gerrit.libreoffice.org/49195
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/ftools/ftools.cxx 
b/sc/source/filter/ftools/ftools.cxx
index 525b2eacebb2..f7d085a28b6c 100644
--- a/sc/source/filter/ftools/ftools.cxx
+++ b/sc/source/filter/ftools/ftools.cxx
@@ -67,25 +67,25 @@ SEEE  IMMM    
  
 */
 
 long double lfDouble;
-long double lfFakt = 256.0;
+long double lfFactor = 256.0;
 sal_uInt8 pDouble10[ 10 ];
 
 rStrm.ReadBytes(pDouble10, 10);// Intel-10 in pDouble10
 
 lfDouble  = static_cast< long double >( pDouble10[ 7 ] );   // Byte 7
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 6 ] );   // Byte 6
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 5 ] );   // Byte 5
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 4 ] );   // Byte 4
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 3 ] );   // Byte 3
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 2 ] );   // Byte 2
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 1 ] );   // Byte 1
-lfDouble *= lfFakt;
+lfDouble *= lfFactor;
 lfDouble += static_cast< long double >( pDouble10[ 0 ] );   // Byte 0
 
 //  For value 0.0 all bits are zero; pow(2.0,-16446) does not work with 
CSet compilers
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sc/source/filter/excel/fontbuff.cxx |   14 +++---
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit b4571ca497063448805bd707c12fcd584d576620
Author: Johnny_M 
Date:   Sun Feb 4 00:57:58 2018 +0100

Translate German variable names

Akt -> Current in Excel filter (fontbuff)

Change-Id: I7c969fc17e08efff7a446bf6e11ec0ce4c82
Reviewed-on: https://gerrit.libreoffice.org/49194
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/filter/excel/fontbuff.cxx 
b/sc/source/filter/excel/fontbuff.cxx
index 2a0a6c07d4f5..52461ffde7f6 100644
--- a/sc/source/filter/excel/fontbuff.cxx
+++ b/sc/source/filter/excel/fontbuff.cxx
@@ -42,16 +42,16 @@ void LotusFontBuffer::Fill( const sal_uInt8 nIndex, 
SfxItemSet& rItemSet )
 {
 sal_uInt8   nIntIndex = nIndex & 0x07;
 
-ENTRY*  pAkt = pData + nIntIndex;
+ENTRY*  pCurrent = pData + nIntIndex;
 
-if( pAkt->pFont )
-rItemSet.Put( *pAkt->pFont );
+if( pCurrent->pFont )
+rItemSet.Put( *pCurrent->pFont );
 
-if( pAkt->pHeight )
-rItemSet.Put( *pAkt->pHeight );
+if( pCurrent->pHeight )
+rItemSet.Put( *pCurrent->pHeight );
 
-if( pAkt->pColor )
-rItemSet.Put( *pAkt->pColor );
+if( pCurrent->pColor )
+rItemSet.Put( *pCurrent->pColor );
 
 if( nIndex & 0x08 )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sw/source/core/crsr/findtxt.cxx |   20 ++--
 1 file changed, 10 insertions(+), 10 deletions(-)

New commits:
commit f51ce842a6ac994f68bff453cf97d64dbc29b1ec
Author: Johnny_M 
Date:   Sun Feb 4 00:12:30 2018 +0100

Translate German variable names

Akt -> Current in crsr

Change-Id: Icdcba2c34c6b76e6a5232a526caeb77a294404ff
Reviewed-on: https://gerrit.libreoffice.org/49190
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sw/source/core/crsr/findtxt.cxx b/sw/source/core/crsr/findtxt.cxx
index 5ea33060dae6..cf98da007ba4 100644
--- a/sw/source/core/crsr/findtxt.cxx
+++ b/sw/source/core/crsr/findtxt.cxx
@@ -115,7 +115,7 @@ lcl_CleanStr(const SwTextNode& rNd, sal_Int32 const nStart, 
sal_Int32& rEnd,
 else
 break;
 
-const sal_Int32 nAkt = nStt - rArr.size();
+const sal_Int32 nCurrent = nStt - rArr.size();
 
 if ( bNewHint )
 {
@@ -141,17 +141,17 @@ lcl_CleanStr(const SwTextNode& rNd, sal_Int32 const 
nStart, sal_Int32& rEnd,
 // end (might be normal 0x7f).
 const bool bEmpty = pHt->Which() != RES_TXTATR_FIELD
 || (static_txtattr_cast(pHt)->GetFormatField().GetField()->ExpandField(true).isEmpty());
-if ( bEmpty && nStart == nAkt )
+if ( bEmpty && nStart == nCurrent )
 {
-rArr.push_back( nAkt );
+rArr.push_back( nCurrent );
 --rEnd;
-buf.remove(nAkt, 1);
+buf.remove(nCurrent, 1);
 }
 else
 {
 if ( bEmpty )
-aReplaced.push_back( nAkt );
-buf[nAkt] = '\x7f';
+aReplaced.push_back( nCurrent );
+buf[nCurrent] = '\x7f';
 }
 }
 break;
@@ -159,9 +159,9 @@ lcl_CleanStr(const SwTextNode& rNd, sal_Int32 const nStart, 
sal_Int32& rEnd,
 {
 if( bRemoveCommentAnchors )
 {
-rArr.push_back( nAkt );
+rArr.push_back( nCurrent );
 --rEnd;
-buf.remove( nAkt, 1 );
+buf.remove( nCurrent, 1 );
 }
 }
 break;
@@ -175,9 +175,9 @@ lcl_CleanStr(const SwTextNode& rNd, sal_Int32 const nStart, 
sal_Int32& rEnd,
 
 if ( bNewSoftHyphen )
 {
-rArr.push_back( nAkt );
+rArr.push_back( nCurrent );
 --rEnd;
-buf.remove(nAkt, 1);
+buf.remove(nCurrent, 1);
 ++nSoftHyphen;
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Johnny_M
 sw/source/uibase/dbui/dbmgr.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 2c6712b5fe7c0118d67887fa5d18affca8636c1b
Author: Johnny_M 
Date:   Sat Feb 3 23:25:09 2018 +0100

Translate German variable names

Akt -> Current in dbmgr

Change-Id: Ia44fc1bb424703b417b46ad68f64aa6ffc3d6ccf
Reviewed-on: https://gerrit.libreoffice.org/49191
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sw/source/uibase/dbui/dbmgr.cxx b/sw/source/uibase/dbui/dbmgr.cxx
index 43f6451fa1ef..225dbc2c50b0 100644
--- a/sw/source/uibase/dbui/dbmgr.cxx
+++ b/sw/source/uibase/dbui/dbmgr.cxx
@@ -611,8 +611,8 @@ static OUString  lcl_FindColumn(const OUString& 
sFormatStr,sal_uInt16  &nUsedPos
 nSeparator = 0xff;
 while(nUsedPos < nLen && nSeparator == 0xff)
 {
-sal_Unicode cAkt = sFormatStr[nUsedPos];
-switch(cAkt)
+sal_Unicode cCurrent = sFormatStr[nUsedPos];
+switch(cCurrent)
 {
 case ',':
 nSeparator = DB_SEP_SPACE;
@@ -627,7 +627,7 @@ static OUString  lcl_FindColumn(const OUString& 
sFormatStr,sal_uInt16  &nUsedPos
 nSeparator = DB_SEP_NEWLINE;
 break;
 default:
-sReturn += OUStringLiteral1(cAkt);
+sReturn += OUStringLiteral1(cCurrent);
 }
 nUsedPos++;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Caolán McNamara
 stoc/source/invocation/invocation.cxx |   18 --
 svx/source/table/tablelayouter.cxx|2 +-
 2 files changed, 1 insertion(+), 19 deletions(-)

New commits:
commit 914a35e8467252da43f864c86b0a820d8c0bd38e
Author: Caolán McNamara 
Date:   Sun Feb 4 20:45:30 2018 +

ofz#4949 Integer-overflow

Change-Id: Ib8928108c987989ab849e8d7e1a85078f6d93050
Reviewed-on: https://gerrit.libreoffice.org/49216
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/svx/source/table/tablelayouter.cxx 
b/svx/source/table/tablelayouter.cxx
index 9a31f56288cc..9bca986844c3 100644
--- a/svx/source/table/tablelayouter.cxx
+++ b/svx/source/table/tablelayouter.cxx
@@ -798,7 +798,7 @@ void TableLayouter::LayoutTableHeight( tools::Rectangle& 
rArea, bool bFit )
 
 // now scale if wanted and needed
 if( bFit && nCurrentHeight != rArea.getHeight() )
-distribute( maRows, rArea.getHeight() - nCurrentHeight );
+distribute(maRows, o3tl::saturating_sub(rArea.getHeight(), 
nCurrentHeight));
 
 // last step, update left edges
 sal_Int32 nNewHeight = 0;
commit 52030a3a65657dc5a3aed3208b25b33df7277e1b
Author: Tor Lillqvist 
Date:   Mon Feb 5 10:57:07 2018 +0200

Remove code that has been commented out since initial import in 2000

Change-Id: I42aaed79e88b6cb8bb0b93aa49555aad45050fe7

diff --git a/stoc/source/invocation/invocation.cxx 
b/stoc/source/invocation/invocation.cxx
index f639e55bdee8..f564425c4202 100644
--- a/stoc/source/invocation/invocation.cxx
+++ b/stoc/source/invocation/invocation.cxx
@@ -357,10 +357,6 @@ void Invocation_Impl::setMaterial( const Any& rMaterial )
 _xNameContainer.set( _xDirect, UNO_QUERY );
 _xENDirect.set( _xDirect, UNO_QUERY );
 _xDirect2.set( _xDirect, UNO_QUERY );
-
-// only once!!!
-//_xIntrospectionAccess = XIntrospectionAccessRef();
-//_xPropertySet = XPropertySetRef();
 }
 else
 {
@@ -417,20 +413,6 @@ void Invocation_Impl::setMaterial( const Any& rMaterial )
 _xENIntrospection.set( _xIntrospectionAccess, UNO_QUERY );
 }
 }
-/* only once !!!
-_xDirect = XInvocationRef();
-if( !_xIntrospectionAccess.is() )
-{
-// reset
-_xElementAccess = XElementAccessRef();
-_xEnumerationAccess = XEnumerationAccessRef();
-_xIndexAccess   = XIndexAccessRef();
-_xIndexContainer= XIndexContainerRef();
-_xNameAccess= XNameAccessRef();
-_xNameContainer = XNameContainerRef();
-_xPropertySet   = XPropertySetRef();
-}
-*/
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] dev-tools.git: esc-reporting/esc-collect.py

2018-02-05 Thread Xisco Fauli
 esc-reporting/esc-collect.py |6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 7cc66463acfb5e5bea972fe10fcfeda1af1ee023
Author: Xisco Fauli 
Date:   Mon Feb 5 10:11:16 2018 +0100

esc-collect: Add latest versions

diff --git a/esc-reporting/esc-collect.py b/esc-reporting/esc-collect.py
index 491fae4..425175a 100755
--- a/esc-reporting/esc-collect.py
+++ b/esc-reporting/esc-collect.py
@@ -275,7 +275,9 @@ def do_ESC_MAB_UPDATE(bz):
 url = bz + '&f1=version&o1=regexp&priority=highest&v1=^'
 rawList = {}
 
-series = {'5.3' : '5.3',
+series = {'6.0' : '6.0',
+  '5.4' : '5.4',
+  '5.3' : '5.3',
   '5.2' : '5.2',
   '5.1' : '5.1',
   '5.0' : '5.0',
@@ -682,7 +684,7 @@ def get_crash(cfg):
 dirList = 
util_load_url('http://dev-builds.libreoffice.org/crashtest/?C=M&O=D', 
useRaw=True)
 # find newest entry by using sort - in nginx' fancyindex first row is 
parent-directory
 # the second ones is most recent dir that was created. Only regular 
entries have a title
-# attribute though, so use that as a shortcut, skip 
+# attribute though, so use that as a shortcut, skip
 inx = dirList.find('title="', 0)
 if inx == -1:
print("ERROR: http://dev-builds.libreoffice.org/crashtest/?C=M&O=D not 
showing DIR list")
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Jim Raykowski
 include/sfx2/sidebar/SidebarDockingWindow.hxx |1 +
 sfx2/source/sidebar/SidebarDockingWindow.cxx  |9 +
 2 files changed, 10 insertions(+)

New commits:
commit bcfc348663b04834fe8e6d6315f647be1595469a
Author: Jim Raykowski 
Date:   Sun Feb 4 14:12:32 2018 -0900

tdf#115434 Fix key press misbehaviors in sidebar

This patch also is for tdf#98505 and tdf#84657

Change-Id: I6bc125ace035b2c8126f013a63bcc9498b06e08c
Reviewed-on: https://gerrit.libreoffice.org/49219
Tested-by: Jenkins 
Reviewed-by: Samuel Mehrbrodt 

diff --git a/include/sfx2/sidebar/SidebarDockingWindow.hxx 
b/include/sfx2/sidebar/SidebarDockingWindow.hxx
index 0a0a55c76c5e..3467df71c7b3 100644
--- a/include/sfx2/sidebar/SidebarDockingWindow.hxx
+++ b/include/sfx2/sidebar/SidebarDockingWindow.hxx
@@ -36,6 +36,7 @@ public:
  vcl::Window* pParent, WinBits nBits);
 virtual ~SidebarDockingWindow() override;
 virtual void dispose() override;
+virtual bool EventNotify(NotifyEvent& rEvent) override;
 
 using SfxDockingWindow::Close;
 
diff --git a/sfx2/source/sidebar/SidebarDockingWindow.cxx 
b/sfx2/source/sidebar/SidebarDockingWindow.cxx
index 84a2834e5a34..05e7a3ab41a9 100644
--- a/sfx2/source/sidebar/SidebarDockingWindow.cxx
+++ b/sfx2/source/sidebar/SidebarDockingWindow.cxx
@@ -104,6 +104,15 @@ SfxChildAlignment SidebarDockingWindow::CheckAlignment (
 }
 }
 
+bool SidebarDockingWindow::EventNotify(NotifyEvent& rEvent)
+{
+MouseNotifyEvent nType = rEvent.GetType();
+if (MouseNotifyEvent::KEYINPUT == nType)
+return true;
+
+return Window::EventNotify(rEvent);
+}
+
 } } // end of namespace sfx2::sidebar
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Michael Stahl
 sw/source/core/bastyp/swcache.cxx |   17 +++--
 1 file changed, 11 insertions(+), 6 deletions(-)

New commits:
commit 3c280b7235967b28905220f72333c7029911fb82
Author: Michael Stahl 
Date:   Fri Feb 2 21:24:37 2018 +0100

sw: convert some SAL_WARN to assert in SwCache

Change-Id: Id148253a2f2c60912cbfbe9628f5ddec56a48952
Reviewed-on: https://gerrit.libreoffice.org/49165
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 

diff --git a/sw/source/core/bastyp/swcache.cxx 
b/sw/source/core/bastyp/swcache.cxx
index 8d8b5997ff9a..282e5c6d4307 100644
--- a/sw/source/core/bastyp/swcache.cxx
+++ b/sw/source/core/bastyp/swcache.cxx
@@ -29,11 +29,14 @@
 void SwCache::Check()
 {
 if ( !m_pRealFirst )
+{
+assert(m_pFirst == nullptr && m_pLast == nullptr);
 return;
+}
 
 // consistency check
-SAL_WARN_IF( m_pLast->GetNext(), "sw.core", "Last but not last." );
-SAL_WARN_IF( m_pRealFirst->GetPrev(), "sw.core", "First but not first." );
+assert(m_pLast->GetNext() == nullptr);
+assert(m_pRealFirst->GetPrev() == nullptr);
 sal_uInt16 nCnt = 0;
 bool bFirstFound = false;
 SwCacheObj *pObj = m_pRealFirst;
@@ -44,17 +47,19 @@ void SwCache::Check()
 SwCacheObj *pTmp = m_pLast;
 while ( pTmp && pTmp != pObj )
 pTmp = pTmp->GetPrev();
-SAL_WARN_IF( !pTmp, "sw.core", "Object not found." );
+assert(pTmp && "Object not found.");
 
 ++nCnt;
 if ( pObj == m_pFirst )
 bFirstFound = true;
 if ( !pObj->GetNext() )
-SAL_WARN_IF( pObj != m_pLast, "sw.core", "Last not Found." );
+{
+assert(pObj == m_pLast);
+}
 pObj = pObj->GetNext();
-SAL_WARN_IF(pObj == pOldRealFirst, "sw.core", "Recursion in SwCache.");
+assert(pObj != pOldRealFirst);
 }
-SAL_WARN_IF( !bFirstFound, "sw.core", "First not Found." );
+assert(bFirstFound);
 SAL_WARN_IF( nCnt + m_aFreePositions.size() != size(), "sw.core", "Lost 
Chain." );
 SAL_WARN_IF(
 size() == m_nCurMax && m_nCurMax != m_aFreePositions.size() + nCnt, 
"sw.core",
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 3 commits - ios/CustomTarget_iOS_setup.mk ios/LibreOfficeLight

2018-02-05 Thread jan Iversen
 ios/CustomTarget_iOS_setup.mk  
 |1 
 ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
 |3 
 
ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Debug.xcscheme
   |   93 ++
 
ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Release.xcscheme
 |   93 ++
 4 files changed, 187 insertions(+), 3 deletions(-)

New commits:
commit f17d0ff5234d88816198becb0f3d8ea7f00f7eb0
Author: jan Iversen 
Date:   Mon Feb 5 11:09:43 2018 +0100

iOS, correctly build release/debug versions

Added 2 schemes Debug/Release as shared to make sure they are
available. Not having them as shared and cleaning causes them to be
"forgotten"

Change-Id: I4cb66c7088f770c7cda592994f3610e98fb1b7f0

diff --git 
a/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Debug.xcscheme
 
b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Debug.xcscheme
new file mode 100644
index ..b860af5cc7e5
--- /dev/null
+++ 
b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Debug.xcscheme
@@ -0,0 +1,93 @@
+
+
+   
+  
+ 
+
+
+ 
+  
+   
+   
+  
+  
+  
+ 
+ 
+  
+  
+  
+   
+   
+  
+ 
+ 
+  
+  
+  
+   
+   
+  
+ 
+ 
+  
+   
+   
+   
+   
+   
+
diff --git 
a/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Release.xcscheme
 
b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Release.xcscheme
new file mode 100644
index ..4861ceef6913
--- /dev/null
+++ 
b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/xcshareddata/xcschemes/Release.xcscheme
@@ -0,0 +1,93 @@
+
+
+   
+  
+ 
+
+
+ 
+  
+   
+   
+  
+  
+  
+ 
+ 
+  
+  
+  
+   
+   
+  
+ 
+ 
+  
+  
+  
+   
+   
+  
+ 
+ 
+  
+   
+   
+   
+   
+   
+
commit 1ed8f225ab0328055dbb6724060d21584b393bd5
Author: jan Iversen 
Date:   Mon Feb 5 11:06:22 2018 +0100

iOS, removed OTHER_LDFLAGS

Change-Id: Ib07c1c4823deda7fa4ba389e9679f6eff99691d8

diff --git a/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj 
b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
index c85096aa1a60..17263df41d76 100644
--- a/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
+++ b/ios/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
@@ -494,7 +494,6 @@
MACH_O_TYPE = mh_execute;
MTL_ENABLE_DEBUG_INFO = NO;
ONLY_ACTIVE_ARCH = YES;
-   OTHER_LDFLAGS = "";
SDKROOT = iphoneos;
SWIFT_OBJC_BRIDGING_HEADER = 
"lokit-Bridging-Header.h";
SWIFT_OPTIMIZATION_LEVEL = "-Owholemodule";
@@ -529,7 +528,6 @@
LD_RUNPATH_SEARCH_PATHS = "$(inherited) 
@executable_path/Frameworks";
LIBRARY_SEARCH_PATHS = "";
ONLY_ACTIVE_ARCH = YES;
-   OTHER_LDFLAGS = "$(LINK_LDFLAGS)";
PRODUCT_BUNDLE_IDENTIFIER = 
com.jani.Editors.LibreOfficeLight;
PRODUCT_NAME = "$(TARGET_NAME)";
SWIFT_OBJC_BRIDGING_HEADER = 
"LibreOfficeLight/lokit-Bridging-Header.h";
@@ -563,7 +561,6 @@
LD_RUNPATH_SEARCH_PATHS = "$(inherited) 
@executable_path/Frameworks";
LIBRARY_SEARCH_PATHS = "";
ONLY_ACTIVE_ARCH = YES;
-   OTHER_LDFLAGS = "$(LINK_LDFLAGS)";
PRODUCT_BUNDLE_IDENTIFIER = 
com.jani.Editors.LibreOfficeLight;
PRODUCT_NAME = "$(TARGET_NAME)";
SWIFT_OBJC_BRIDGING_HEADER = 
"LibreOfficeLight/lokit-Bridging-Header.h";
commit 8258b425e85bc360f67d776a0196a7a091e7975f
Author: jan Iversen 
Date:   Mon Feb 5 09:14:01 2018 +0100

iOS, allow x86 release builds

Added dir for x86_64 release builds

Change-Id: If8c5e775ecd1283d9386a764441019c2423170f9

diff --git a/ios/CustomTarget_iOS_setup.mk b/ios/CustomTarget_iOS_setup.mk
index 0ca73a345ffb..8fea759249e9 100644
--- a/ios/CustomTarget_iOS_setup.mk
+++ b/ios/CustomTarget_iOS_setup.mk
@@ -14,6 +14,7 @@ IOSRES  = $(IOSGEN)/resources
 IOSDIRS = $(IOSGEN) \
   $(IOSGEN)/Debug_x86_64 \
   $(IOSGEN)/Debug_arm64 \
+  $(IOSGEN)/Release_x86_64 \
   $(IOSGEN)/Release_arm64 \

[Libreoffice-commits] core.git: framework/inc framework/source include/comphelper include/filter include/unotools lingucomponent/source stoc/source vcl/source

2018-02-05 Thread Manuj Vashist
 framework/inc/jobs/helponstartup.hxx |2 --
 framework/inc/jobs/shelljob.hxx  |2 --
 framework/source/inc/loadenv/targethelper.hxx|2 --
 include/comphelper/processfactory.hxx|4 
 include/comphelper/streamsection.hxx |3 ---
 include/filter/msfilter/util.hxx |3 ---
 include/unotools/streamhelper.hxx|3 ---
 include/unotools/streamsection.hxx   |3 ---
 lingucomponent/source/languageguessing/guess.hxx |3 ---
 lingucomponent/source/languageguessing/simpleguesser.hxx |3 ---
 stoc/source/corereflection/lrucache.hxx  |1 -
 vcl/source/fontsubset/ttcr.cxx   |3 ---
 12 files changed, 32 deletions(-)

New commits:
commit 68d9583db4226a61ba23398928b9416c5d8745c8
Author: Manuj Vashist 
Date:   Mon Feb 5 15:27:22 2018 +0530

tdf#108523 Removed @author annotations.

Change-Id: I6a60128c413beab6cabb857c073607e15ddbc98f
Reviewed-on: https://gerrit.libreoffice.org/49215
Reviewed-by: Samuel Mehrbrodt 
Tested-by: Samuel Mehrbrodt 

diff --git a/framework/inc/jobs/helponstartup.hxx 
b/framework/inc/jobs/helponstartup.hxx
index 4550f962e2e7..1fc65b4dd6f6 100644
--- a/framework/inc/jobs/helponstartup.hxx
+++ b/framework/inc/jobs/helponstartup.hxx
@@ -35,8 +35,6 @@ namespace framework{
 /** @short  implements a job component, which handle the special
 feature to show a suitable help page for every (visible!)
 loaded document.
-
-@author as96863
  */
 class HelpOnStartup : public ::cppu::WeakImplHelper< 
css::lang::XServiceInfo,css::lang::XEventListener,css::task::XJob >
 {
diff --git a/framework/inc/jobs/shelljob.hxx b/framework/inc/jobs/shelljob.hxx
index 4ff90039cac9..a25eb448ceb8 100644
--- a/framework/inc/jobs/shelljob.hxx
+++ b/framework/inc/jobs/shelljob.hxx
@@ -38,8 +38,6 @@ namespace framework{
 registered for. Further there is a generic
 way to configure the shell command and it's list
 of arguments.
-
-@author as96863
  */
 class ShellJob : public ::cppu::WeakImplHelper< 
css::lang::XServiceInfo,css::task::XJob >
 {
diff --git a/framework/source/inc/loadenv/targethelper.hxx 
b/framework/source/inc/loadenv/targethelper.hxx
index afa968645f66..b35caf80d04d 100644
--- a/framework/source/inc/loadenv/targethelper.hxx
+++ b/framework/source/inc/loadenv/targethelper.hxx
@@ -28,8 +28,6 @@ namespace framework{
 
 /** @short  can be used to detect, if a target name (used e.g. for 
XFrame.findFrame())
 has a special meaning or can be used as normal frame name (e.g. 
for XFrame.setName()).
-
-@author as96863
  */
 class TargetHelper
 {
diff --git a/include/comphelper/processfactory.hxx 
b/include/comphelper/processfactory.hxx
index bf4645124c12..51ba33754a98 100644
--- a/include/comphelper/processfactory.hxx
+++ b/include/comphelper/processfactory.hxx
@@ -32,8 +32,6 @@ namespace comphelper
 
 /**
  * This function set the process service factory.
- *
- * @author Juergen Schmidt
  */
 COMPHELPER_DLLPUBLIC void setProcessServiceFactory(const css::uno::Reference< 
css::lang::XMultiServiceFactory >& xSMgr);
 
@@ -41,8 +39,6 @@ COMPHELPER_DLLPUBLIC void setProcessServiceFactory(const 
css::uno::Reference< cs
  * This function gets the process service factory.
  *
  * If no service factory is set the function throws a RuntimeException.
- *
- * @author Juergen Schmidt
  */
 COMPHELPER_DLLPUBLIC css::uno::Reference< css::lang::XMultiServiceFactory > 
getProcessServiceFactory();
 
diff --git a/include/comphelper/streamsection.hxx 
b/include/comphelper/streamsection.hxx
index 4a53c85a9f8f..6c52892fde6b 100644
--- a/include/comphelper/streamsection.hxx
+++ b/include/comphelper/streamsection.hxx
@@ -31,9 +31,6 @@ namespace comphelper
 /** implements handling for compatibly reading/writing data from/into an 
input/output stream.
 data written in a block secured by this class should be readable by older 
versions which
 use the same mechanism.
-
-@author Frank Schoenheit
-@since  00/26/05
 */
 
 class COMPHELPER_DLLPUBLIC OStreamSection
diff --git a/include/filter/msfilter/util.hxx b/include/filter/msfilter/util.hxx
index 8895a4181bec..50252fc6191a 100644
--- a/include/filter/msfilter/util.hxx
+++ b/include/filter/msfilter/util.hxx
@@ -37,9 +37,6 @@ MSFILTER_DLLPUBLIC rtl_TextEncoding 
getBestTextEncodingFromLocale(const css::lan
 MSFILTER_DLLPUBLIC sal_uInt32 BGRToRGB(sal_uInt32 nColour);
 
 /** Convert from DTTM to Writer's DateTime
-
-  @author
-  mailto:mma...@openoffice.org";>Martin Maher
 {
diff --git a/include/unotools/streamsection.hxx 
b/include/unotools/streamsection.hxx
index ad45b4e12496..74626d2bed4e 100644
--- a/include/unotools/streamsection.hxx
+++ b/include/unotools/streamsection.hxx
@@ -30,9 +30,6 @@ namespace utl
 /** implements handling for compat

Re: Crash test update

2018-02-05 Thread Caolán McNamara
On Thu, 2018-02-01 at 21:26 +, Crashtest VM wrote:
> New crashtest update available at http://dev-builds.libreoffice.org/c
> rashtest/4696d3728f0aba1087001bc543fc0867dd0ebdda/

The threaded xml import stuff is now gone, all that remains is the more
recent assert from...

commit faa0305ba3d0dc698fce4915d4f3a1fb52422380
Author: Eike Rathke 
Date:   Thu Jan 25 13:20:27 2018 +0100

Prepare CheckLinkFormulaNeedingCheck() to use either RPN or tokenized
code
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [libreoffice-design] Re: Minutes of the design meeting 2018-Jan-31

2018-02-05 Thread Pedro Rosmaninho
Working on papercut bugs is somewhat crucial.
If the user experience with each module of Libre Office demands that the
user has to use work-arounds to get work done in LO then he will just use
other officesuites where he won't face those same issues.
Getting someone to fix those issues would be crucial.

An example in Calc like the one Renon posted in Impress is this one:

https://bugs.documentfoundation.org/show_bug.cgi?id=53300

A bug that exists since Open Office days... With LO and the TDF constantly
bragging about having cleaned up so many bugs in LO it's unforgivable that
UX bugs like this one are still lingering from Open Office days.



On Sat, Feb 3, 2018 at 7:58 PM, Michel RENON 
wrote:

> Hi,
>
> Le 31/01/2018 à 21:48, Heiko Tietze a écrit :
>
>> [...]
>> GSoC
>>
>>   * Full list of ideas https://pad.documentfoundation.org/p/UX-GSoC_Ideas
>>   * Public list https://wiki.documentfoundatio
>> n.org/Development/GSoC/Ideas#User_Interface
>>
>
> Here are some suggestions based on my recent experience in teaching
> LibreOffice and writing a python macro :
>
>
>
> Impress : work on "papercut" bugs
> --
>
>
> 1 - Animations sets for a text box with several paragraphs don't affect
> new paragraphs (not inherited)
> https://bugs.documentfoundation.org/show_bug.cgi?id=57581
>
> This is the most annoying problem that new users face :
>   - they create a slide with few lines of text,
>   - they add an animation on the text, to make it appear line by line
>   - they test : it works correctly
>   - they add one line of text (inside or at the end)
>   - they test and boom... the animation is "broken"
>   - they make a pause, then... close impress and open powerpoint...
>
>
>
> 2 - Editing: Text spacing changes when click in textbox
> https://bugs.documentfoundation.org/show_bug.cgi?id=89060
>
> It is a subtle but very annoying visual effect.
>
>
>
>
>
>
> Python macros
> 
>
>
> I had to create a macro embedded in a file.
> As I had not written basic for a long time, I tried to write it in python.
>
> The fist problem is that on Ubuntu, it is necessary to install a specific
> package 'libreoffice-script-provider-python'
>
>
> The second problem is that, by default, there is no tool to integrate a
> python macro in a document.
> I found the APSO extension [1] that allows that and is very useful.
>
> I found some difficult points in writing python macro with that extension :
>   - after modifying a python script, you have to close and re-open the odf
> file
>   - some inline documentation is missing : about API, and some sample code
>   - the context (available via XSCRIPTCONTEXT) is missing some API,
> compared to basic
>
>
>
>
>
> So here are my suggestions for GSoC :
>   - allow an easy activation of python support : a simple button would
> download and install all necessary packages
>   - integrate the functionality of APSO (to allow embedding/managing
> python script in odf file)
>   - remove the need to re-open the odf file before using a new version of
> python script
>
>
>
> Other points can be done directly in the current extension :
>   - add some inline documentation about API [2]
>   - add inline documentation on both ways to use python script :
> - an external script that connects to LibreOffice
> - an internal script that get called by menu/button
>   - allow to create scripts empty or with comments (that provide minimum
> documentation and good practice)
>   - add a library of sample code (a list of "how-to ?"), that can be
> easily copied in script files
>
>
> Adding API to the context (available via XSCRIPTCONTEXT) can be done by
> working on LibreOffice code :
> /usr/lib/libreoffice/program/pythonscript/py
>
>
>
>
>
>
> Cheers,
>
> Michel
>
>
>
>
>
>
> [1] https://extensions.libreoffice.org/extensions/apso-
> alternative-script-organizer-for-python
>
> [2] based on very useful page https://wiki.openoffice.org/wi
> ki/Python/Transfer_from_Basic_to_Python
>
>
> --
> To unsubscribe e-mail to: design+unsubscr...@global.libreoffice.org
> Problems? https://www.libreoffice.org/get-help/mailing-lists/how-to-un
> subscribe/
> Posting guidelines + more: https://wiki.documentfoundation.org/Netiquette
> List archive: https://listarchives.libreoffice.org/global/design/
> All messages sent to this list will be publicly archived and cannot be
> deleted
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: Branch 'libreoffice-6-0' - vcl/source

2018-02-05 Thread Caolán McNamara
 vcl/source/filter/jpeg/jpegc.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit cd855591f49d688e116c3da63d913315c62028c1
Author: Caolán McNamara 
Date:   Sat Feb 3 16:39:12 2018 +

tdf#115416 allow build with classic libjpeg

Change-Id: I96252e1a78dbc52d21095e11b7bdb0807e5d76b6
Reviewed-on: https://gerrit.libreoffice.org/49205
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 

diff --git a/vcl/source/filter/jpeg/jpegc.cxx b/vcl/source/filter/jpeg/jpegc.cxx
index e120fc304228..1210cb34ad53 100644
--- a/vcl/source/filter/jpeg/jpegc.cxx
+++ b/vcl/source/filter/jpeg/jpegc.cxx
@@ -244,6 +244,7 @@ void ReadJPEG(JpegStuff& rContext, JPEGReader* pJPEGReader, 
void* pInputStream,
 eScanlineFormat = ScanlineFormat::N8BitPal;
 nPixelSize = 1;
 }
+#if defined(JCS_EXTENSIONS)
 else if (eFinalFormat == ScanlineFormat::N32BitTcBgra)
 {
 best_out_color_space = JCS_EXT_BGRA;
@@ -262,7 +263,7 @@ void ReadJPEG(JpegStuff& rContext, JPEGReader* pJPEGReader, 
void* pInputStream,
 eScanlineFormat = eFinalFormat;
 nPixelSize = 4;
 }
-
+#endif
 if (rContext.cinfo.jpeg_color_space == JCS_YCCK)
 rContext.cinfo.out_color_space = JCS_CMYK;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: connectivity/source dbaccess/source include/connectivity

2018-02-05 Thread Noel Grandin
 connectivity/source/commontools/TTableHelper.cxx |   20 ++--
 connectivity/source/drivers/ado/ATable.cxx   |   22 ++---
 connectivity/source/drivers/component/CTable.cxx |6 -
 connectivity/source/drivers/dbase/DTable.cxx |  100 +++
 connectivity/source/drivers/evoab2/NTable.cxx|6 -
 connectivity/source/drivers/file/FTable.cxx  |6 -
 connectivity/source/drivers/firebird/Table.cxx   |4 
 connectivity/source/drivers/flat/ETable.cxx  |6 -
 connectivity/source/drivers/hsqldb/HTable.cxx|   12 +-
 connectivity/source/drivers/macab/MacabTable.cxx |6 -
 connectivity/source/drivers/mysql/YTable.cxx |   12 +-
 connectivity/source/sdbcx/VTable.cxx |   33 ++-
 dbaccess/source/core/api/table.cxx   |6 -
 include/connectivity/sdbcx/VTable.hxx|7 -
 14 files changed, 119 insertions(+), 127 deletions(-)

New commits:
commit 5c7b954935369cbbd22a6f43be63ac7c0ddbbfdc
Author: Noel Grandin 
Date:   Mon Jan 29 13:28:44 2018 +0200

loplugin:useuniqueptr in OTable

use rtl::Reference here since these are reference counted data
structures, and we hand out references to me via our API

Change-Id: I8f69dde2db08519cb3d9de20b4ad54462e3ecc98
Reviewed-on: https://gerrit.libreoffice.org/49180
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/connectivity/source/commontools/TTableHelper.cxx 
b/connectivity/source/commontools/TTableHelper.cxx
index b94ab42d4617..7bb03a90da3c 100644
--- a/connectivity/source/commontools/TTableHelper.cxx
+++ b/connectivity/source/commontools/TTableHelper.cxx
@@ -299,10 +299,10 @@ void OTableHelper::refreshColumns()
 );
 }
 
-if(m_pColumns)
-m_pColumns->reFill(aVector);
+if(m_xColumns)
+m_xColumns->reFill(aVector);
 else
-m_pColumns  = createColumns(aVector);
+m_xColumns = createColumns(aVector);
 }
 
 const ColumnDesc* OTableHelper::getColumnDescription(const OUString& _sName) 
const
@@ -424,10 +424,10 @@ void OTableHelper::refreshKeys()
 {
 refreshPrimaryKeys(aNames);
 refreshForeignKeys(aNames);
-m_pKeys = createKeys(aNames);
+m_xKeys = createKeys(aNames);
 } // if(!isNew())
-else if (!m_pKeys )
-m_pKeys = createKeys(aNames);
+else if (!m_xKeys )
+m_xKeys = createKeys(aNames);
 /*if(m_pKeys)
 m_pKeys->reFill(aVector);
 else*/
@@ -469,10 +469,10 @@ void OTableHelper::refreshIndexes()
 }
 }
 
-if(m_pIndexes)
-m_pIndexes->reFill(aVector);
+if(m_xIndexes)
+m_xIndexes->reFill(aVector);
 else
-m_pIndexes  = createIndexes(aVector);
+m_xIndexes = createIndexes(aVector);
 }
 
 OUString OTableHelper::getRenameStart() const
@@ -549,7 +549,7 @@ void SAL_CALL OTableHelper::alterColumnByIndex( sal_Int32 
index, const Reference
 );
 
 Reference< XPropertySet > xOld(
-m_pColumns->getByIndex(index), css::uno::UNO_QUERY);
+m_xColumns->getByIndex(index), css::uno::UNO_QUERY);
 if(xOld.is())
 
alterColumnByName(getString(xOld->getPropertyValue(OMetaConnection::getPropMap().getNameByIndex(PROPERTY_ID_NAME))),descriptor);
 }
diff --git a/connectivity/source/drivers/ado/ATable.cxx 
b/connectivity/source/drivers/ado/ATable.cxx
index c1688a152cc4..0c028794a6c3 100644
--- a/connectivity/source/drivers/ado/ATable.cxx
+++ b/connectivity/source/drivers/ado/ATable.cxx
@@ -84,10 +84,10 @@ void OAdoTable::refreshColumns()
 aColumns.fillElementNames(aVector);
 }
 
-if(m_pColumns)
-m_pColumns->reFill(aVector);
+if(m_xColumns)
+m_xColumns->reFill(aVector);
 else
-m_pColumns = new 
OColumns(*this,m_aMutex,aVector,aColumns,isCaseSensitive(),m_pCatalog->getConnection());
+m_xColumns = new 
OColumns(*this,m_aMutex,aVector,aColumns,isCaseSensitive(),m_pCatalog->getConnection());
 }
 
 void OAdoTable::refreshKeys()
@@ -101,10 +101,10 @@ void OAdoTable::refreshKeys()
 aKeys.fillElementNames(aVector);
 }
 
-if(m_pKeys)
-m_pKeys->reFill(aVector);
+if(m_xKeys)
+m_xKeys->reFill(aVector);
 else
-m_pKeys = new 
OKeys(*this,m_aMutex,aVector,aKeys,isCaseSensitive(),m_pCatalog->getConnection());
+m_xKeys = new 
OKeys(*this,m_aMutex,aVector,aKeys,isCaseSensitive(),m_pCatalog->getConnection());
 }
 
 void OAdoTable::refreshIndexes()
@@ -118,10 +118,10 @@ void OAdoTable::refreshIndexes()
 aIndexes.fillElementNames(aVector);
 }
 
-if(m_pIndexes)
-m_pIndexes->reFill(aVector);
+if(m_xIndexes)
+m_xIndexes->reFill(aVector);
 else
-m_pIndexes = new 
OIndexes(*this,m_aMutex,aVector,aIndexes,isCaseSensitive(),m_pCatalog->getConnection());
+m_xIndexes = new 
OIndexes(*this,m_aMutex,aVector,aIndexes,isCaseSensitive(),m_pCatalog->getConnection());
 }
 
 Sequence< sal_Int8 > OAdoTable::getUnoTunnelImplementationI

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

2018-02-05 Thread Noel Grandin
 include/svx/e3dsceneupdater.hxx |3 ++-
 svx/source/engine3d/e3dsceneupdater.cxx |7 ++-
 2 files changed, 4 insertions(+), 6 deletions(-)

New commits:
commit b05a3198f6c1d4713a871c46610dd48bf3979e8a
Author: Noel Grandin 
Date:   Tue Jan 30 10:25:15 2018 +0200

loplugin:useuniqueptr in E3DModifySceneSnapRectUpdater

Change-Id: Ied2cdfa314f86ff3a25cecd5c13c03d3428966f0
Reviewed-on: https://gerrit.libreoffice.org/49207
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/include/svx/e3dsceneupdater.hxx b/include/svx/e3dsceneupdater.hxx
index ce51518ee689..ad01d667b4f5 100644
--- a/include/svx/e3dsceneupdater.hxx
+++ b/include/svx/e3dsceneupdater.hxx
@@ -21,6 +21,7 @@
 #define INCLUDED_SVX_E3DSCENEUPDATER_HXX
 
 #include 
+#include 
 
 class SdrObject;
 class E3dScene;
@@ -53,7 +54,7 @@ class SVX_DLLPUBLIC E3DModifySceneSnapRectUpdater
 
 // the 3d transformation stack at the time of construction, valid when
 // mpScene is not zero
-drawinglayer::geometry::ViewInformation3D*  mpViewInformation3D;
+std::unique_ptr  
mpViewInformation3D;
 
 public:
 // the constructor evaluates and sets the members at construction time
diff --git a/svx/source/engine3d/e3dsceneupdater.cxx 
b/svx/source/engine3d/e3dsceneupdater.cxx
index a77e7badd5a6..bc696daebafb 100644
--- a/svx/source/engine3d/e3dsceneupdater.cxx
+++ b/svx/source/engine3d/e3dsceneupdater.cxx
@@ -51,7 +51,7 @@ 
E3DModifySceneSnapRectUpdater::E3DModifySceneSnapRectUpdater(const SdrObject* pO
 else
 {
 // secure current 3D transformation stack
-mpViewInformation3D = new 
drawinglayer::geometry::ViewInformation3D(rVCScene.getViewInformation3D(aAllContentRange));
+mpViewInformation3D.reset( new 
drawinglayer::geometry::ViewInformation3D(rVCScene.getViewInformation3D(aAllContentRange))
 );
 }
 }
 }
@@ -83,8 +83,7 @@ 
E3DModifySceneSnapRectUpdater::~E3DModifySceneSnapRectUpdater()
 mpViewInformation3D->getDeviceToView(),
 mpViewInformation3D->getViewTime(),
 mpViewInformation3D->getExtendedInformationSequence());
-delete mpViewInformation3D;
-mpViewInformation3D = pNew;
+mpViewInformation3D.reset(pNew);
 }
 
 // transform content range to scene-relative coordinates using old 
3d transformation stack
@@ -111,8 +110,6 @@ 
E3DModifySceneSnapRectUpdater::~E3DModifySceneSnapRectUpdater()
 }
 }
 }
-
-delete mpViewInformation3D;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Weekly QA Report (W05-2018)

2018-02-05 Thread Xisco Fauli
Hello,

What have happened in QA in the last 7 days?

  * 163 bugs have been created, of which, 82 are still unconfirmed (
Total Unconfirmed bugs: 465 )
        + Created bugs: http://tinyurl.com/ybf95tyh
        + Still unconfirmed bugs: http://tinyurl.com/yb7wmdp9

  * 1259 comments have been written by 226 users.

  * 80 new users have signed up to Bugzilla.

== STATUSES CHANGED ==
  * 20 bugs have been changed to 'ASSIGNED'.
        + Link: http://tinyurl.com/y8a3f4on
        + Done by: Xisco Faulí ( 9 ), Julien Nabet ( 2 ), László Németh
( 2 ), Vasily Melenchuk (CIB) ( 1 ), Buovjaga ( 1 ), Heiko Tietze ( 1 ),
Szymon Kłos ( 1 ), Kevin Suo ( 1 ), Yousuf Philips (jay) ( 1 ), Henry
Castro ( 1 )

  * 2 bugs have been changed to 'CLOSED'.
        + Link: http://tinyurl.com/ycdztwhu
        + Done by: Alex Thurgood ( 1 ), Harald Koester ( 1 )

  * 19 bugs have been changed to 'NEEDINFO'.
        + Link: http://tinyurl.com/ycyy3jro
        + Done by: Xisco Faulí ( 7 ), Timur ( 3 ), Buovjaga ( 2 ),
Julien Nabet ( 2 ), m.a.riosv ( 2 ), Heiko Tietze ( 1 ), Alex Thurgood (
1 ), Dieter Praas ( 1 )

  * 62 bugs have been changed to 'NEW'.
        + Link: http://tinyurl.com/ydxk98zj
        + Done by: Buovjaga ( 14 ), Xisco Faulí ( 10 ), Heiko Tietze ( 6
), Dieter Praas ( 5 ), Aron Budea ( 5 ), V Stuart Foote ( 3 ), Julien
Nabet ( 3 ), Telesto ( 2 ), MM ( 2 ), Alex Thurgood ( 2 ), Miklos Vajna
( 1 ), Thorsten Behrens (CIB) ( 1 ), Regina Henschel ( 1 ), Pranav Kant
( 1 ), Yousuf Philips (jay) ( 1 ), m.a.riosv ( 1 ), kurt.pfeifle ( 1 ),
Diarmuid O'Briain ( 1 ), tommy27 ( 1 ), Alex ARNAUD ( 1 )

  * 3 bugs have been changed to 'REOPENED'.
        + Link: http://tinyurl.com/y85neufo
        + Done by: Kyle White ( 1 ), Cor Nouws ( 1 ), bordfeldt ( 1 )

  * 44 bugs have been changed to 'RESOLVED DUPLICATE'.
        + Link: http://tinyurl.com/yd7ygxms
        + Done by: Xisco Faulí ( 9 ), V Stuart Foote ( 7 ), Buovjaga ( 7
), Aron Budea ( 4 ), Alex Thurgood ( 3 ), Telesto ( 2 ), Julien Nabet (
2 ), Timur ( 2 ), Heiko Tietze ( 1 ), Pranav Kant ( 1 ), Yousuf Philips
(jay) ( 1 ), m.a.riosv ( 1 ), Luke ( 1 ), Khaled Hosny ( 1 ), Gabor
Kelemen ( 1 ), Cor Nouws ( 1 )

  * 34 bugs have been changed to 'RESOLVED FIXED'.
        + Link: http://tinyurl.com/ya29u4fh
        + Done by: Aron Budea ( 5 ), Thorsten Behrens (CIB) ( 3 ),
Michael Stahl ( 3 ), Heiko Tietze ( 2 ), Szymon Kłos ( 2 ), Jim
Raykowski ( 2 ), Adolfo Jayme ( 2 ), Caolán McNamara ( 2 ), Tamás Zolnai
( 1 ), Miklos Vajna ( 1 ), Vasily Melenchuk (CIB) ( 1 ), Buovjaga ( 1 ),
Julien Nabet ( 1 ), Serge Krot (CIB) ( 1 ), rolywhittellwebb ( 1 ),
Pranav Kant ( 1 ), Yousuf Philips (jay) ( 1 ), Noel Grandin ( 1 ),
Federico Mena-Quintero ( 1 ), Serg Bormant ( 1 ), Katarina Behrens (CIB)
( 1 )

  * 93 bugs have been changed to 'RESOLVED INSUFFICIENTDATA'.
        + Link: http://tinyurl.com/yaass4w7
        + Done by: qa-admin ( 91 ), Volga ( 1 ), raal ( 1 )

  * 1 bug has been changed to 'RESOLVED INVALID'.
        + Link: http://tinyurl.com/ydbavhen
        + Done by: Beuss ( 1 )

  * 1 bug has been changed to 'RESOLVED MOVED'.
        + Link: http://tinyurl.com/ydadhzck
        + Done by: Xisco Faulí ( 1 )

  * 5 bugs have been changed to 'RESOLVED NOTABUG'.
        + Link: http://tinyurl.com/yaqvcyqd
        + Done by: Xavier Van Wijmeersch ( 1 ), V Stuart Foote ( 1 ),
Mike Kaganski ( 1 ), m.a.riosv ( 1 ), Khaled Hosny ( 1 )

  * 10 bugs have been changed to 'RESOLVED NOTOURBUG'.
        + Link: http://tinyurl.com/yczd2ytc
        + Done by: Khaled Hosny ( 4 ), Buovjaga ( 3 ), Alex Thurgood ( 2
), Pranav Kant ( 1 )

  * 3 bugs have been changed to 'RESOLVED WONTFIX'.
        + Link: http://tinyurl.com/yae2vhhs
        + Done by: Heiko Tietze ( 2 ), Khaled Hosny ( 1 )

  * 25 bugs have been changed to 'RESOLVED WORKSFORME'.
        + Link: http://tinyurl.com/yb9qqhjb
        + Done by: Xisco Faulí ( 3 ), Cor Nouws ( 3 ), Aron Budea ( 2 ),
V Stuart Foote ( 1 ), Buovjaga ( 1 ), Heiko Tietze ( 1 ), Telesto ( 1 ),
Rob Westein ( 1 ), Regina Henschel ( 1 ), qa-admin ( 1 ), Martin
Srebotnjak ( 1 ), Michael Meeks ( 1 ), Marco ( 1 ), Matthias Basler ( 1
), Jos van den Oever ( 1 ), Jean-Baptiste Faure ( 1 ), Alex Thurgood ( 1
), Douglas Fiedler ( 1 ), Alex ARNAUD ( 1 ), Howard Johnson ( 1 )

  * 22 bugs have been changed to 'UNCONFIRMED'.
        + Link: http://tinyurl.com/y8s75je6
        + Done by: Xisco Faulí ( 2 ), Telesto ( 2 ), Volker Lenhardt ( 1
), u.fuchs ( 1 ), Sywey_Wang ( 1 ), Paul Menzel ( 1 ), Massimo ( 1 ),
perie_gut ( 1 ), Marvin Scott ( 1 ), Oliver Ostertag ( 1 ), Janne
Korkkula ( 1 ), Jo-Jo ( 1 ), harriet anders ( 1 ), Gerrit Boom ( 1 ),
David F Smith ( 1 ), cr...@arno.com ( 1 ), Daniel Caron ( 1 ),
c.bessiere ( 1 ), Bruce Fleming ( 1 ), drawbar ( 1 )

  * 5 bugs have been changed to 'VERIFIED FIXED'.
        + Link: http://tinyurl.com/y9wv3fys
        + Done by: Jacques Guilleron ( 2 ), Xisco Faulí ( 1 ), Buovjaga
( 1 ), Aron Budea ( 1 )

== KEYWORDS ADDED ==
  * 'acc

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

2018-02-05 Thread Caolán McNamara
 vcl/qa/cppunit/graphicfilter/data/wmf/fail/ofz5942-1.wmf |binary
 vcl/source/gdi/sallayout.cxx |6 ++
 2 files changed, 6 insertions(+)

New commits:
commit b189726c4f61a5d28a3b87112e7a85174b5573aa
Author: Caolán McNamara 
Date:   Mon Feb 5 09:02:39 2018 +

ofz#5942 Out of memory

when missing a glyph in a grapheme we set the whole grapheme as missing.  
There
can be a huge long grapheme and iterating through each missing codepoint
results in the whole range pushed back repeatedly as missing.

Change-Id: Ib7c283389ee2357eb3b4264c78e558955ebad1eb
Reviewed-on: https://gerrit.libreoffice.org/49223
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/vcl/qa/cppunit/graphicfilter/data/wmf/fail/ofz5942-1.wmf 
b/vcl/qa/cppunit/graphicfilter/data/wmf/fail/ofz5942-1.wmf
new file mode 100644
index ..f9a72867c593
Binary files /dev/null and 
b/vcl/qa/cppunit/graphicfilter/data/wmf/fail/ofz5942-1.wmf differ
diff --git a/vcl/source/gdi/sallayout.cxx b/vcl/source/gdi/sallayout.cxx
index 07b65564d6c2..a282a0248621 100644
--- a/vcl/source/gdi/sallayout.cxx
+++ b/vcl/source/gdi/sallayout.cxx
@@ -270,6 +270,12 @@ void ImplLayoutRuns::AddRun( int nCharPos0, int nCharPos1, 
bool bRTL )
 nCharPos1 = nTemp;
 }
 
+if (maRuns.size() >= 2 && nCharPos0 == maRuns[maRuns.size() - 2] && 
nCharPos1 == maRuns[maRuns.size() - 1])
+{
+//this run is the same as the last
+return;
+}
+
 // append new run
 maRuns.push_back( nCharPos0 );
 maRuns.push_back( nCharPos1 );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Noel Grandin
 sw/source/filter/ww8/docxattributeoutput.cxx |2 +-
 sw/source/filter/ww8/rtfattributeoutput.cxx  |5 ++---
 sw/source/filter/ww8/wrtww8.hxx  |6 +-
 sw/source/filter/ww8/wrtww8gr.cxx|3 +--
 sw/source/filter/ww8/ww8atr.cxx  |3 +--
 sw/source/filter/ww8/ww8par.cxx  |2 +-
 sw/source/filter/ww8/ww8par.hxx  |4 
 7 files changed, 15 insertions(+), 10 deletions(-)

New commits:
commit cfd78dadbb461d8364e9294094e38a03732cabc1
Author: Noel Grandin 
Date:   Mon Feb 5 11:10:50 2018 +0200

TypedWhichId in sw(2)

Change-Id: I68816c196f36d92fcd1ed58826368bbfe9aa52ce
Reviewed-on: https://gerrit.libreoffice.org/49224
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index e24b02e9dd55..c4143c91a8c1 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -6757,7 +6757,7 @@ void DocxAttributeOutput::CharEscapement( const 
SvxEscapementItem& rEscapement )
 m_pSerializer->singleElementNS( XML_w, XML_vertAlign,
FSNS( XML_w, XML_val ), sIss.getStr(), FSEND );
 
-const SvxFontHeightItem& rItem = static_cast(m_rExport.GetItem(RES_CHRATR_FONTSIZE));
+const SvxFontHeightItem& rItem = m_rExport.GetItem(RES_CHRATR_FONTSIZE);
 if (sIss.isEmpty() || sIss.match(OString("baseline")))
 {
 long nHeight = rItem.GetHeight();
diff --git a/sw/source/filter/ww8/rtfattributeoutput.cxx 
b/sw/source/filter/ww8/rtfattributeoutput.cxx
index fd5df7d44011..88c70e14532d 100644
--- a/sw/source/filter/ww8/rtfattributeoutput.cxx
+++ b/sw/source/filter/ww8/rtfattributeoutput.cxx
@@ -2229,8 +2229,7 @@ void RtfAttributeOutput::CharEscapement(const 
SvxEscapementItem& rEscapement)
 
 const char* pUpDn;
 
-SwTwips nH
-= static_cast(m_rExport.GetItem(RES_CHRATR_FONTSIZE)).GetHeight();
+SwTwips nH = m_rExport.GetItem(RES_CHRATR_FONTSIZE).GetHeight();
 
 if (0 < rEscapement.GetEsc())
 pUpDn = OOO_STRING_SVTOOLS_RTF_UP;
@@ -2752,7 +2751,7 @@ void RtfAttributeOutput::ParaTabStop(const 
SvxTabStopItem& rTabStop)
 // Tabs are absolute by default.
 if (m_rExport.m_pDoc->getIDocumentSettingAccess().get(
 DocumentSettingId::TABS_RELATIVE_TO_INDENT))
-nOffset = static_cast(m_rExport.GetItem(RES_LR_SPACE)).GetTextLeft();
+nOffset = m_rExport.GetItem(RES_LR_SPACE).GetTextLeft();
 
 for (sal_uInt16 n = 0; n < rTabStop.Count(); n++)
 {
diff --git a/sw/source/filter/ww8/wrtww8.hxx b/sw/source/filter/ww8/wrtww8.hxx
index 2b39a7d887d5..5a52a2aa852f 100644
--- a/sw/source/filter/ww8/wrtww8.hxx
+++ b/sw/source/filter/ww8/wrtww8.hxx
@@ -612,6 +612,10 @@ public:
 }
 
 const SfxPoolItem& GetItem( sal_uInt16 nWhich ) const;
+template const T& GetItem( TypedWhichId nWhich ) const
+{
+return static_cast(GetItem(sal_uInt16(nWhich)));
+}
 
 /// Find the reference.
 bool HasRefToObject( sal_uInt16 nTyp, const OUString* pName, sal_uInt16 
nSeqNo );
@@ -1405,7 +1409,7 @@ public:
 
 virtual const SfxPoolItem* HasTextItem( sal_uInt16 nWhich ) const = 0;
 virtual const SfxPoolItem& GetItem( sal_uInt16 nWhich ) const = 0;
-};
+ };
 
 /// Used to export formatted text associated to drawings.
 class MSWord_SdrAttrIter : public MSWordAttrIter
diff --git a/sw/source/filter/ww8/wrtww8gr.cxx 
b/sw/source/filter/ww8/wrtww8gr.cxx
index 31f4a501a019..d0b47f658c7a 100644
--- a/sw/source/filter/ww8/wrtww8gr.cxx
+++ b/sw/source/filter/ww8/wrtww8gr.cxx
@@ -420,8 +420,7 @@ void WW8Export::OutGrf(const ww8::Frame &rFrame)
 SwTwips nHeight = rFlyFormat.GetFrameSize().GetHeight();
 nHeight/=20; //nHeight was in twips, want it in half points, 
but
  //then half of total height.
-long nFontHeight = static_cast(
-GetItem(RES_CHRATR_FONTSIZE)).GetHeight();
+long nFontHeight = GetItem(RES_CHRATR_FONTSIZE).GetHeight();
 nHeight-=nFontHeight/20;
 
 Set_UInt16( pArr, NS_sprm::sprmCHpsPos );
diff --git a/sw/source/filter/ww8/ww8atr.cxx b/sw/source/filter/ww8/ww8atr.cxx
index cd8930647624..2c187e610522 100644
--- a/sw/source/filter/ww8/ww8atr.cxx
+++ b/sw/source/filter/ww8/ww8atr.cxx
@@ -1387,8 +1387,7 @@ void WW8AttributeOutput::CharEscapement( const 
SvxEscapementItem& rEscapement )
 
 if ( 0 == b || 0xFF == b )
 {
-long nHeight = static_cast(m_rWW8Export.GetItem(
-RES_CHRATR_FONTSIZE )).GetHeight();
+long nHeight = m_rWW8Export.GetItem( RES_CHRATR_FONTSIZE ).GetHeight();
 m_rWW8Export.InsUInt16( NS_sprm::sprmCHpsPos );
 
 m_rWW8Export.InsUInt16( static_cast(( nHeight * nEsc + 500 ) / 
1000 ));
diff --git a/sw/source/filter/ww8/ww8par.cxx b/sw/source/filter/ww8/ww8par.

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

2018-02-05 Thread Chris Sherlock
 chart2/source/view/charttypes/GL3DBarChart.cxx |5 -
 chart2/source/view/inc/GL3DRenderer.hxx|4 
 chart2/source/view/main/DummyXShape.cxx|4 
 chart2/source/view/main/OpenGLRender.hxx   |   13 -
 4 files changed, 26 deletions(-)

New commits:
commit 16ea77bc4d2218b68d40708f4b8b4a4418a84d05
Author: Chris Sherlock 
Date:   Sun Jan 7 00:14:48 2018 +1100

chart2: remove dead code

Change-Id: I67bc3785eb0fd2030626becd1ffb6a80bd97d861
Reviewed-on: https://gerrit.libreoffice.org/47491
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/chart2/source/view/charttypes/GL3DBarChart.cxx 
b/chart2/source/view/charttypes/GL3DBarChart.cxx
index 35cc5f572a69..946b28d05261 100644
--- a/chart2/source/view/charttypes/GL3DBarChart.cxx
+++ b/chart2/source/view/charttypes/GL3DBarChart.cxx
@@ -54,13 +54,8 @@ const float TEXT_HEIGHT = 10.0f;
 float DEFAULT_CAMERA_HEIGHT = 500.0f;
 const sal_uInt32 ID_STEP = 10;
 
-#if 0
-const float BAR_SIZE_X = 15.0f;
-const float BAR_SIZE_Y = 15.0f;
-#else
 const float BAR_SIZE_X = 30.0f;
 const float BAR_SIZE_Y = 5.0f;
-#endif
 const float BAR_DISTANCE_X = 5.0f;
 const float BAR_DISTANCE_Y = 5.0;
 
diff --git a/chart2/source/view/inc/GL3DRenderer.hxx 
b/chart2/source/view/inc/GL3DRenderer.hxx
index 19b6996b5d8f..141621c1bc70 100644
--- a/chart2/source/view/inc/GL3DRenderer.hxx
+++ b/chart2/source/view/inc/GL3DRenderer.hxx
@@ -406,11 +406,7 @@ private:
 glm::mat4 m_3DMVP;
 
 GLuint m_3DUBOBuffer;
-#if 0
-GLint m_3DLightBlockIndex;
 
-GLint m_3DMaterialBlockIndex;
-#endif
 GLint m_3DActualSizeLight;
 
 GLuint m_NormalBuffer;
diff --git a/chart2/source/view/main/DummyXShape.cxx 
b/chart2/source/view/main/DummyXShape.cxx
index 800f9cea2262..99752858c93d 100644
--- a/chart2/source/view/main/DummyXShape.cxx
+++ b/chart2/source/view/main/DummyXShape.cxx
@@ -1088,11 +1088,7 @@ void DummyChart::render()
 
 SAL_INFO("chart2.opengl", "render chart");
 m_GLRender.prepareToRender();
-#if 0
-m_GLRender.renderDebug();
-#else
 DummyXShapes::render();
-#endif
 }
 
 void DummyChart::clear()
diff --git a/chart2/source/view/main/OpenGLRender.hxx 
b/chart2/source/view/main/OpenGLRender.hxx
index 025b83297a09..d8f81a0f1799 100644
--- a/chart2/source/view/main/OpenGLRender.hxx
+++ b/chart2/source/view/main/OpenGLRender.hxx
@@ -109,10 +109,6 @@ public:
 void SetBackGroundColor(sal_uInt32 color1, sal_uInt32 color2, 
css::drawing::FillStyle fillstyle);
 
 private:
-#if 0
-//for performance
-double m_dFreq;
-#endif
 int m_iWidth;
 
 int m_iHeight;
@@ -128,22 +124,13 @@ private:
 glm::mat4 m_Model;
 // Our ModelViewProjection : multiplication of our 3 matrices
 glm::mat4 m_MVP;
-#if 0
-GLint m_RenderProID;
-#endif
 
 GLuint m_VertexBuffer;
 
 GLuint m_ColorBuffer;
 
 GLint m_MatrixID;
-#if 0
-GLint m_RenderVertexID;
 
-GLint m_RenderTexCoordID;
-
-GLint m_RenderTexID;
-#endif
 GLuint m_RenderVertexBuf;
 
 GLuint m_RenderTexCoordBuf;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-4' - sc/qa

2018-02-05 Thread Michael Stahl
 sc/qa/unit/functions_test.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit d78a70d34ac5eac0b9bd831d7cd5777a55d906ac
Author: Michael Stahl 
Date:   Fri Feb 2 14:13:35 2018 +0100

sc: disable "forecast.ets.add.fods" test file on Macs

This sometimes fails, since months, and nobody has fixed it.
Let's improve the reliability of CI a bit.

Change-Id: Ic4d2e085151855d35804f8ed6ebca04366815c0b
Reviewed-on: https://gerrit.libreoffice.org/49144
Reviewed-by: Eike Rathke 
Tested-by: Jenkins 
(cherry picked from commit 00a6f0908a73a80fb0521e1f4cdc99647e43c126)
Reviewed-on: https://gerrit.libreoffice.org/49228

diff --git a/sc/qa/unit/functions_test.cxx b/sc/qa/unit/functions_test.cxx
index ac9a430a96e7..e48c209e1974 100644
--- a/sc/qa/unit/functions_test.cxx
+++ b/sc/qa/unit/functions_test.cxx
@@ -40,6 +40,10 @@ bool FunctionsTest::load(const OUString& rFilter, const 
OUString& rURL,
 
 ScDocument& rDoc = xDocShRef->GetDocument();
 
+#ifdef __APPLE__
+// FIXME tends to fail a lot
+if (!rURL.endsWith("forecast.ets.add.fods"))
+#endif
 CPPUNIT_ASSERT_DOUBLES_EQUAL(1.0, rDoc.GetValue(1, 2, 0), 1e-14);
 
 xDocShRef->DoClose();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: bin/list-uitest.py

2018-02-05 Thread Zdeněk Crhonek
 bin/list-uitest.py |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit ad32bcdfa1cecb28abe934885df62bd6a49aa978
Author: Zdeněk Crhonek 
Date:   Wed Jan 3 20:14:24 2018 +0100

uitest - add folder uitest/manual_tests to script

script for page https://wiki.documentfoundation.org/ListUiTests

Change-Id: I0081a4e1229105ef7387e70d76a5670a4f5de691
Reviewed-on: https://gerrit.libreoffice.org/47340
Reviewed-by: Xisco Faulí 
Tested-by: Xisco Faulí 

diff --git a/bin/list-uitest.py b/bin/list-uitest.py
index baecc41b89a0..8924a1f5daec 100755
--- a/bin/list-uitest.py
+++ b/bin/list-uitest.py
@@ -12,7 +12,7 @@ import datetime
 def analyze_file(filename):
 class_name = ""
 method_list = []
-with open(filename) as fh:
+with open(filename, encoding='utf-8') as fh:
 for line in fh:
 if line.lstrip().startswith('class '):
 class_name = line.lstrip().split(" ")[1].split("(")[0]
@@ -51,7 +51,8 @@ def main():
 'Calc' : ['../uitest/calc_tests', '../sc/qa/uitest/'],
 'Impress' : ['../uitest/impress_tests/'],
 'Math': ['../uitest/math_tests/'],
-'Draw': ['']}
+'Draw': [''],
+'Manual_tests': ['../uitest/manual_tests/']}
 
 print('{{TopMenu}}')
 print('{{Menu}}')
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: wizards/com wizards/Pyuno_commonwizards.mk

2018-02-05 Thread Xisco Fauli
 wizards/Pyuno_commonwizards.mk|5 
 wizards/com/sun/star/wizards/common/Configuration.py  |   36 ---
 wizards/com/sun/star/wizards/common/FileAccess.py |  117 --
 wizards/com/sun/star/wizards/common/ListModel.py  |4 
 wizards/com/sun/star/wizards/common/SystemDialog.py   |   42 ---
 wizards/com/sun/star/wizards/common/UCB.py|   33 --
 wizards/com/sun/star/wizards/common/XMLHelper.py  |   34 --
 wizards/com/sun/star/wizards/common/XMLProvider.py|   24 --
 wizards/com/sun/star/wizards/document/OfficeDocument.py   |   63 -
 wizards/com/sun/star/wizards/letter/LetterWizardDialogImpl.py |   22 -
 wizards/com/sun/star/wizards/text/TextDocument.py |   25 --
 wizards/com/sun/star/wizards/ui/DocumentPreview.py|5 
 wizards/com/sun/star/wizards/ui/UnoDialog.py  |   24 --
 wizards/com/sun/star/wizards/ui/UnoDialog2.py |   37 ---
 wizards/com/sun/star/wizards/ui/WizardDialog.py   |   19 -
 wizards/com/sun/star/wizards/ui/event/CommonListener.py   |   50 
 wizards/com/sun/star/wizards/ui/event/DataAware.py|   27 --
 wizards/com/sun/star/wizards/ui/event/EventListenerList.py|3 
 wizards/com/sun/star/wizards/ui/event/ListDataEvent.py|   36 ---
 wizards/com/sun/star/wizards/ui/event/ListDataListener.py |   32 --
 wizards/com/sun/star/wizards/ui/event/ListModelBinder.py  |   42 ---
 wizards/com/sun/star/wizards/ui/event/SimpleDataAware.py  |   33 --
 wizards/com/sun/star/wizards/ui/event/Task.py |   38 ---
 wizards/com/sun/star/wizards/ui/event/TaskEvent.py|6 
 wizards/com/sun/star/wizards/ui/event/TaskListener.py |4 
 wizards/com/sun/star/wizards/ui/event/UnoDataAware.py |6 
 26 files changed, 28 insertions(+), 739 deletions(-)

New commits:
commit 2029a593f2ff5863b81ba6abc5e466473c475b48
Author: Xisco Fauli 
Date:   Thu Feb 1 15:40:51 2018 +0100

Removed unused code in python wizards

This code got unused after 5b3022860abfe7eefd2e25b60b5b2a563b42b3c0

Change-Id: I6c198ab359f8468a9f4d9041b9bc4feb88eedd74
Reviewed-on: https://gerrit.libreoffice.org/49109
Tested-by: Jenkins 
Reviewed-by: Xisco Faulí 

diff --git a/wizards/Pyuno_commonwizards.mk b/wizards/Pyuno_commonwizards.mk
index 578555c6c5eb..1c139950d963 100644
--- a/wizards/Pyuno_commonwizards.mk
+++ b/wizards/Pyuno_commonwizards.mk
@@ -25,8 +25,6 @@ $(eval $(call gb_Pyuno_add_files,commonwizards,wizards,\
 common/SystemDialog.py \
 common/IRenderer.py \
 common/UCB.py \
-common/XMLHelper.py \
-common/XMLProvider.py \
 common/ListModel.py \
 common/__init__.py \
 document/OfficeDocument.py \
@@ -50,12 +48,9 @@ $(eval $(call gb_Pyuno_add_files,commonwizards,wizards,\
 ui/event/ListModelBinder.py \
 ui/event/RadioDataAware.py \
 ui/event/UnoDataAware.py \
-ui/event/SimpleDataAware.py \
 ui/event/Task.py \
 ui/event/TaskEvent.py \
 ui/event/TaskListener.py \
-ui/event/ListDataEvent.py \
-ui/event/ListDataListener.py \
 ui/event/EventListenerList.py \
 ui/event/__init__.py \
 ))
diff --git a/wizards/com/sun/star/wizards/common/Configuration.py 
b/wizards/com/sun/star/wizards/common/Configuration.py
index e63d20827542..91274e0b288c 100644
--- a/wizards/com/sun/star/wizards/common/Configuration.py
+++ b/wizards/com/sun/star/wizards/common/Configuration.py
@@ -54,47 +54,11 @@ class Configuration(object):
 def getNode(self, name, parent):
 return parent.getByName(name)
 
-# This method creates a new configuration node and adds it
-# to the given view. Note that if a node with the given name
-# already exists it will be completely removed from
-# the configuration.
-# @param configView
-# @param name
-# @return the new created configuration node.
-# @throws com.sun.star.lang.WrappedTargetException
-# @throws ElementExistException
-# @throws NoSuchElementException
-# @throws com.sun.star.uno.Exception
-@classmethod
-def addConfigNode(self, configView, name):
-try:
-node = configView.getByName(name)
-except Exception:
-node = None
-if (node is not None):
-return node
-else:
-# create a new detached set element (instance of 
DataSourceDescription)
-# the new element is the result !
-node = configView.createInstance()
-# insert it - this also names the element
-configView.insertByName(name, node)
-return node
-
-@classmethod
-def removeNode(self, configView, name):
-if (configView.hasByName(name)):
-configView.removeByName(name)
-
 @classmethod
 def commit(self, configView):
 configView.commitChanges()
 
 @classmethod

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

2018-02-05 Thread Caolán McNamara
 sw/qa/core/data/html/fail/ofz5909-1.html |1 +
 sw/source/filter/html/htmltab.cxx|   11 +--
 2 files changed, 10 insertions(+), 2 deletions(-)

New commits:
commit 5090d575117fbe6ced75dc45c514f8d990523251
Author: Caolán McNamara 
Date:   Sun Feb 4 21:02:37 2018 +

ofz#5909 Null-dereference READ

Change-Id: I830d3e8bda517e7681cea7481c32457486ced693
Reviewed-on: https://gerrit.libreoffice.org/49218
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/qa/core/data/html/fail/ofz5909-1.html 
b/sw/qa/core/data/html/fail/ofz5909-1.html
new file mode 100644
index ..9511b2bc1eac
--- /dev/null
+++ b/sw/qa/core/data/html/fail/ofz5909-1.html
@@ -0,0 +1 @@
+
\ No newline at end of file
diff --git a/sw/source/filter/html/htmltab.cxx 
b/sw/source/filter/html/htmltab.cxx
index b1b273ddc531..fa9a8a6e9f87 100644
--- a/sw/source/filter/html/htmltab.cxx
+++ b/sw/source/filter/html/htmltab.cxx
@@ -2296,6 +2296,12 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 
nAbsAvail,
 break;
 }
 
+if (!m_pSwTable)
+{
+SAL_WARN("sw.html", "no table");
+return;
+}
+
 // get the table format and adapt it
 SwFrameFormat *pFrameFormat = m_pSwTable->GetFrameFormat();
 pFrameFormat->SetFormatAttr( SwFormatHoriOrient(0, eHoriOri) );
@@ -5196,8 +5202,9 @@ std::shared_ptr 
SwHTMLParser::BuildTable(SvxAdjust eParentAdjust,
 m_bUpperSpace = true;
 SetTextCollAttrs();
 
-m_nParaCnt = m_nParaCnt - std::min(m_nParaCnt,
-pTCntxt->GetTableNode()->GetTable().GetTabSortBoxes().size());
+SwTableNode* pTableNode = pTCntxt->GetTableNode();
+size_t nTableBoxSize = pTableNode ? 
pTableNode->GetTable().GetTabSortBoxes().size() : 0;
+m_nParaCnt = m_nParaCnt - std::min(m_nParaCnt, nTableBoxSize);
 
 // Jump to a table if needed
 if( JUMPTO_TABLE == m_eJumpTo && m_xTable->GetSwTable() &&
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-4' - ucbhelper/source

2018-02-05 Thread David Tardon
 ucbhelper/source/provider/contentinfo.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5c77726f88d1a04616bb177505a7733b0e008f5c
Author: David Tardon 
Date:   Sun Nov 5 20:41:03 2017 +0100

Related: rhbz#304121 one slash is enough

(cherry picked from commit 1bacce999d7dad91b1f87baaf09b08e6b4629fad)

Change-Id: I35fbca7592f3e17a071df567f691c62fe9232fc1
Reviewed-on: https://gerrit.libreoffice.org/49241
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 

diff --git a/ucbhelper/source/provider/contentinfo.cxx 
b/ucbhelper/source/provider/contentinfo.cxx
index 2ad3edc29d5d..cdbae40880fc 100644
--- a/ucbhelper/source/provider/contentinfo.cxx
+++ b/ucbhelper/source/provider/contentinfo.cxx
@@ -23,7 +23,7 @@
 
  */
 #include 
-#include 
+#include 
 #include 
 
 #include "osl/mutex.hxx"
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'private/swe/libreoffice-5-2+backports' - sc/source

2018-02-05 Thread Katarina Behrens
 sc/source/ui/inc/linkarea.hxx  |2 +-
 sc/source/ui/miscdlgs/linkarea.cxx |5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

New commits:
commit 91ec9510e5b28aa2214d735faf11197b931c72b3
Author: Katarina Behrens 
Date:   Thu Feb 1 12:08:34 2018 +0100

Don't recycle DocumentInserter, create a new one every time

After all, everyone else using DocumentInserter across the codebase
does the same.

For KDE4, an attempt to reuse the same filepicker dialog sporadically
crashes in the depths of (old-ish, possibly buggy) libQt due to nested
event loop and/or recursive repaints.

(crash scenario: Calc > Sheet > Link to external data > Browse > 
double-click
a file > Browse again > Kaboom!)

Reviewed-on: https://gerrit.libreoffice.org/49082
Tested-by: Jenkins 
Reviewed-by: Katarina Behrens 
(cherry picked from commit a74590e4b4475253bd7752738e570cdafe45e674)

Change-Id: Ib494203abd07873919b8394a442f09c5329d237c

diff --git a/sc/source/ui/inc/linkarea.hxx b/sc/source/ui/inc/linkarea.hxx
index a9a8b575d6e7..0d0e4e245b18 100644
--- a/sc/source/ui/inc/linkarea.hxx
+++ b/sc/source/ui/inc/linkarea.hxx
@@ -47,7 +47,7 @@ private:
 VclPtr m_pBtnOk;
 
 ScDocShell* pSourceShell;
-sfx2::DocumentInserter* pDocInserter;
+std::unique_ptr pDocInserter;
 
 SfxObjectShellRef   aSourceRef;
 
diff --git a/sc/source/ui/miscdlgs/linkarea.cxx 
b/sc/source/ui/miscdlgs/linkarea.cxx
index 143799d52ce3..0a475402dd12 100644
--- a/sc/source/ui/miscdlgs/linkarea.cxx
+++ b/sc/source/ui/miscdlgs/linkarea.cxx
@@ -88,9 +88,8 @@ short ScLinkedAreaDlg::Execute()
 
 IMPL_LINK_NOARG_TYPED(ScLinkedAreaDlg, BrowseHdl, Button*, void)
 {
-if ( !pDocInserter )
-pDocInserter = new sfx2::DocumentInserter(
-OUString::createFromAscii( ScDocShell::Factory().GetShortName() ) 
);
+pDocInserter.reset( new sfx2::DocumentInserter(
+OUString::createFromAscii( ScDocShell::Factory().GetShortName() ) 
) );
 pDocInserter->StartExecuteModal( LINK( this, ScLinkedAreaDlg, 
DialogClosedHdl ) );
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [libreoffice-design] Re: Minutes of the design meeting 2018-Jan-31

2018-02-05 Thread Adolfo Jayme Barrientos
Pedro, I think you’ve been here enough time to realize that, as many
similar libre software projects, this project is volunteer-driven, and
that waving around your pet bugs without giving some support
(financial or otherwise) yourself is borderline offensive for those
who are contributing their free time… People are free to choose what
they will work on… And no, the age of a bug does not automatically
increase its importance. Just sayin’.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-3-0' - common/Util.cpp

2018-02-05 Thread Miklos Vajna
 common/Util.cpp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 53384000a26222b5ba7189df43fa40022e53e153
Author: Miklos Vajna 
Date:   Tue Jan 30 15:50:10 2018 +0100

common: avoid pointless copy in Util

loolforkit is never able to get the arguments passed to it without this
patch and nothing starts.

Change-Id: I40097203eba6fede78a17caf057452f4704b2bd4
(cherry picked from commit 222399e99a2cec31bf6414252765098467629f70)
Reviewed-on: https://gerrit.libreoffice.org/49243
Reviewed-by: pranavk 
Tested-by: pranavk 

diff --git a/common/Util.cpp b/common/Util.cpp
index 357f6da8..8c7792c9 100644
--- a/common/Util.cpp
+++ b/common/Util.cpp
@@ -173,7 +173,7 @@ namespace Util
 
 std::vector params;
 params.push_back(const_cast(cmd.c_str()));
-for (auto i : args)
+for (const auto& i : args)
 params.push_back(const_cast(i.c_str()));
 params.push_back(nullptr);
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3' - cui/source include/svx sd/source

2018-02-05 Thread Caolán McNamara
 cui/source/inc/backgrnd.hxx  |1 
 cui/source/tabpages/backgrnd.cxx |  104 ++-
 include/svx/flagsdef.hxx |   11 ++--
 sd/source/ui/dlg/dlgchar.cxx |2 
 sd/source/ui/func/fuchar.cxx |   26 -
 5 files changed, 91 insertions(+), 53 deletions(-)

New commits:
commit b0958ff8bf86b86682131b4da99da701a4a72adb
Author: Caolán McNamara 
Date:   Fri Aug 11 11:51:09 2017 +0100

Resolves: tdf#110077 move the SvxBackgroundColorItem<->SvxBrushItem 
conversion

into the dialog itself

Reviewed-on: https://gerrit.libreoffice.org/41029
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 
(cherry picked from commit 6284804c2ccb3ad8bb6e1c1dac8d622f66b8c07d)

Reviewed-on: https://gerrit.libreoffice.org/41168
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 
(cherry picked from commit a407285747c65894bc858c4aa722a24e57caca31)

Change-Id: I83db9f02145a4927257dd5691f82dad1a11eaea6

diff --git a/cui/source/inc/backgrnd.hxx b/cui/source/inc/backgrnd.hxx
index d2663f3cd27e..cd7ab4ffec35 100644
--- a/cui/source/inc/backgrnd.hxx
+++ b/cui/source/inc/backgrnd.hxx
@@ -107,6 +107,7 @@ private:
 boolbIsGraphicValid : 1;
 boolbLinkOnly   : 1;
 boolbHighlighting   : 1;
+boolbCharBackColor  : 1;
 boolm_bColorSelected: 1;
 Graphic aBgdGraphic;
 OUStringaBgdGraphicPath;
diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx
index 2554da1581b7..ed04e696199c 100644
--- a/cui/source/tabpages/backgrnd.cxx
+++ b/cui/source/tabpages/backgrnd.cxx
@@ -39,6 +39,7 @@
 #include 
 
 #include 
+#include 
 #include "backgrnd.hxx"
 
 #include 
@@ -352,6 +353,7 @@ SvxBackgroundTabPage::SvxBackgroundTabPage(vcl::Window* 
pParent, const SfxItemSe
 , bIsGraphicValid(false)
 , bLinkOnly(false)
 , bHighlighting(false)
+, bCharBackColor(false)
 , m_bColorSelected(false)
 , pPageImpl(new SvxBackgroundPage_Impl)
 , pImportDlg(nullptr)
@@ -482,7 +484,7 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 
 
 // get and evaluate Input-BrushItem
-const SvxBrushItem* pBgdAttr = nullptr;
+bool bBrushItemSet = false;
 sal_uInt16 nSlot = SID_ATTR_BRUSH;
 const SfxPoolItem* pItem;
 sal_uInt16 nDestValue = USHRT_MAX;
@@ -510,20 +512,39 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 {
 nSlot = SID_ATTR_BRUSH_CHAR;
 }
+else if( bCharBackColor )
+{
+nSlot = SID_ATTR_CHAR_BACK_COLOR;
+}
+
 //#73# the destination item is missing when the parent style has been 
changed
 if(USHRT_MAX == nDestValue && m_pTblLBox->IsVisible())
 nDestValue = 0;
-sal_uInt16 nWhich = GetWhich( nSlot );
+sal_uInt16 nWhich = GetWhich(nSlot);
+SvxBrushItem aBgdAttr(nWhich);
 
-if ( rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT )
-pBgdAttr = static_cast(&( rSet->Get( nWhich ) ));
+if (rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT)
+{
+if (!bCharBackColor)
+aBgdAttr = static_cast(rSet->Get(nWhich));
+else
+{
+// EE_CHAR_BKGCOLOR is SvxBackgroundColorItem, but char background 
tabpage
+// can only work with SvxBrushItems
+// extract Color out of SvxBackColorItem
+Color aBackColor = static_cast(rSet->Get(nWhich)).GetValue();
+// make new SvxBrushItem with this Color
+aBgdAttr = SvxBrushItem(aBackColor, SID_ATTR_BRUSH_CHAR);
+}
+bBrushItemSet = true;
+}
 
 m_pBtnTile->Check();
 
-if ( pBgdAttr )
+if (bBrushItemSet)
 {
-FillControls_Impl(*pBgdAttr, aUserData);
-aBgdColor = const_cast(pBgdAttr)->GetColor();
+FillControls_Impl(aBgdAttr, aUserData);
+aBgdColor = aBgdAttr.GetColor();
 }
 else
 {
@@ -558,22 +579,22 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 nWhich = GetWhich( SID_ATTR_BRUSH );
 if ( rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT )
 {
-pBgdAttr = static_cast(&( rSet->Get( 
nWhich ) ));
-pTableBck_Impl->pCellBrush = new SvxBrushItem(*pBgdAttr);
+aBgdAttr = static_cast(rSet->Get(nWhich));
+pTableBck_Impl->pCellBrush = new SvxBrushItem(aBgdAttr);
 }
 pTableBck_Impl->nCellWhich = nWhich;
 
 if ( rSet->GetItemState( SID_ATTR_BRUSH_ROW, false ) >= 
SfxItemState::DEFAULT )
 {
-pBgdAttr = static_cast(&( rSet->Get( 
SID_ATTR_BRUSH_ROW ) ));
-pTableBck_Impl->pRowBrush = new SvxBrushItem(*pBgdAttr);
+aBgdAttr = static_cast(rSet->Get(SID_ATTR_BRUSH_ROW));
+pTableBck_Impl->pRowBrush = new SvxBrushItem(aBgdAttr);
 

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

2018-02-05 Thread Noel Grandin
 include/svx/svxids.hrc |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 96335019cdd7f7e611c29206eb9706556dd53c78
Author: Noel Grandin 
Date:   Mon Feb 5 12:58:43 2018 +0200

fix ubsan after "TypedWhichId in svx part 1"

 noelgrandin, SID_ATTR_PAGE_EXT1 is used both as an SfxBoolItem
(e.g., sd/source/ui/func/fupage.cxx) and as an SfxStringItem (e.g.,
sw/source/uibase/utlui/uitool.cxx)

commit 24158311c115c2db6dd05a751f75a5c084e2c0d1
Author: Noel Grandin 
Date:   Wed Dec 20 13:30:21 2017 +0200

TypedWhichId in svx part 1

Change-Id: If4e75470449d8f967f5f6818067edd7c24bc19a1
Reviewed-on: https://gerrit.libreoffice.org/49230
Reviewed-by: Stephan Bergmann 
Tested-by: Jenkins 

diff --git a/include/svx/svxids.hrc b/include/svx/svxids.hrc
index 93946c20a892..1d455dcef735 100644
--- a/include/svx/svxids.hrc
+++ b/include/svx/svxids.hrc
@@ -207,7 +207,8 @@ class SvxLineItem;
 #define SID_ATTR_PAGE_MAXSIZE   
TypedWhichId( SID_SVX_START + 52 )
 #define SID_ATTR_PAGE_ORIENTATION   
TypedWhichId( SID_SVX_START + 53 )
 #define SID_ATTR_PAGE_PAPERBIN  
TypedWhichId( SID_SVX_START + 54 )
-#define SID_ATTR_PAGE_EXT1  
TypedWhichId( SID_SVX_START + 55 )
+// this is used both as a SfxBoolItem(sd/source/ui/func/fupage.cxx) and as a 
SfxStringItem(sw/source/uibase/utlui/uitool.cxx)
+#define SID_ATTR_PAGE_EXT1  ( SID_SVX_START + 55 )
 #define SID_ATTR_PAGE_EXT2  
TypedWhichId( SID_SVX_START + 56 )
 #define SID_ATTR_PAGE_HEADERSET 
TypedWhichId( SID_SVX_START + 57 )
 #define SID_ATTR_PAGE_FOOTERSET 
TypedWhichId( SID_SVX_START + 58 )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: loleaflet/src

2018-02-05 Thread Marco Cecchetti
 loleaflet/src/control/Control.Header.js |6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit fec429ecdcb7de36cf65e1aa27d2e233a48a32ec
Author: Marco Cecchetti 
Date:   Fri Feb 2 12:36:24 2018 +0100

loleaflet: calc: double click on resize area freezes header handling

Change-Id: I05d758ed0cb6cafa5ab14418b82f23b607e4ea95
Reviewed-on: https://gerrit.libreoffice.org/49238
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/loleaflet/src/control/Control.Header.js 
b/loleaflet/src/control/Control.Header.js
index 844e340e..9a97a254 100644
--- a/loleaflet/src/control/Control.Header.js
+++ b/loleaflet/src/control/Control.Header.js
@@ -396,6 +396,10 @@ L.Control.Header = L.Control.extend({
if (!this._hitResizeArea)
return;
 
+   var rect = this.getHeaderEntryBoundingClientRect();
+   if (!rect)
+   return;
+
L.DomUtil.disableImageDrag();
L.DomUtil.disableTextSelection();
 
@@ -408,7 +412,7 @@ L.Control.Header = L.Control.extend({
L.DomEvent.on(document, 'mousemove', 
this._onMouseMoveForDragging, this);
L.DomEvent.on(document, 'mouseup', this._onMouseUp, this);
 
-   var rect = this.getHeaderEntryBoundingClientRect();
+
this._start = new L.Point(rect.left, rect.top);
this._offset = new L.Point(rect.right - e.clientX, rect.bottom 
- e.clientY);
this._item = target;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-3-0' - loleaflet/src

2018-02-05 Thread Marco Cecchetti
 loleaflet/src/control/Control.Header.js |6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit 31d2fc7cb079fc28674d8de8b8770c1e0e923047
Author: Marco Cecchetti 
Date:   Fri Feb 2 12:36:24 2018 +0100

loleaflet: calc: double click on resize area freezes header handling

Change-Id: I05d758ed0cb6cafa5ab14418b82f23b607e4ea95
Reviewed-on: https://gerrit.libreoffice.org/49239
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/loleaflet/src/control/Control.Header.js 
b/loleaflet/src/control/Control.Header.js
index ad8edec3..22e176da 100644
--- a/loleaflet/src/control/Control.Header.js
+++ b/loleaflet/src/control/Control.Header.js
@@ -396,6 +396,10 @@ L.Control.Header = L.Control.extend({
if (!this._hitResizeArea)
return;
 
+   var rect = this.getHeaderEntryBoundingClientRect();
+   if (!rect)
+   return;
+
L.DomUtil.disableImageDrag();
L.DomUtil.disableTextSelection();
 
@@ -408,7 +412,7 @@ L.Control.Header = L.Control.extend({
L.DomEvent.on(document, 'mousemove', 
this._onMouseMoveForDragging, this);
L.DomEvent.on(document, 'mouseup', this._onMouseUp, this);
 
-   var rect = this.getHeaderEntryBoundingClientRect();
+
this._start = new L.Point(rect.left, rect.top);
this._offset = new L.Point(rect.right - e.clientX, rect.bottom 
- e.clientY);
this._item = target;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: bin/check-elf-dynamic-objects postprocess/CustomTarget_check_dynamic_objects.mk

2018-02-05 Thread Michael Stahl
 bin/check-elf-dynamic-objects |7 +--
 postprocess/CustomTarget_check_dynamic_objects.mk |3 +--
 2 files changed, 6 insertions(+), 4 deletions(-)

New commits:
commit 49696161be136a274d23b5704f146e0421aeb7fe
Author: Michael Stahl 
Date:   Wed Nov 29 21:23:44 2017 +0100

postprocess: enable the check-elf-dynamic-objects for real

Change-Id: Ie08d3dfc6306c18da7b1a5c000a75a18917c1f70
Reviewed-on: https://gerrit.libreoffice.org/45529
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 

diff --git a/bin/check-elf-dynamic-objects b/bin/check-elf-dynamic-objects
index a066d8f55eb9..3561b4109c78 100755
--- a/bin/check-elf-dynamic-objects
+++ b/bin/check-elf-dynamic-objects
@@ -191,8 +191,11 @@ else
 wait
 
 if [ -s check_elf.out ] ; then
-   cat check_elf.out
-   status=1
+cat check_elf.out
+# FIXME known broken, ignore for now
+if grep -v 'libgpgmepp.so.6\|libgpgme.so.11\|libassuan.so.0' 
check_elf.out >/dev/null; then
+status=1
+fi
 fi
 rm check_elf.out
 fi
diff --git a/postprocess/CustomTarget_check_dynamic_objects.mk 
b/postprocess/CustomTarget_check_dynamic_objects.mk
index 53baa2bfd92f..aedf589eb51e 100644
--- a/postprocess/CustomTarget_check_dynamic_objects.mk
+++ b/postprocess/CustomTarget_check_dynamic_objects.mk
@@ -19,7 +19,6 @@ $(call 
gb_CustomTarget_get_workdir,postprocess/check_dynamic_objects)/check.done
$(call gb_Postprocess_get_target,AllPackages)
$(call gb_Output_announce,$(subst $(WORKDIR)/,,$@),$(true),CHK,2)
# currently we only check ELF objects
-   # FIXME fails currently, disabled
-   $(SRCDIR)/bin/check-elf-dynamic-objects -p || true
+   $(SRCDIR)/bin/check-elf-dynamic-objects -p
 
 # vim: set noet sw=4 ts=4:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Changes to 'distro/collabora/collabora-online-3-0-0'

2018-02-05 Thread Andras Timar
New branch 'distro/collabora/collabora-online-3-0-0' available with the 
following commits:
commit 8a1761aba1240fea1c6be64a96f92c6100c9f49d
Author: Andras Timar 
Date:   Mon Feb 5 16:54:05 2018 +0100

Bump package version to 3.0.0-8

Change-Id: Icac8464763e1ad922e6f8aa8f0f9fc3945cd11e6

commit d30e098804fbb1ba5cefc91303149dbea53095e9
Author: Marco Cecchetti 
Date:   Fri Feb 2 12:36:24 2018 +0100

loleaflet: calc: double click on resize area freezes header handling

Change-Id: I05d758ed0cb6cafa5ab14418b82f23b607e4ea95
Reviewed-on: https://gerrit.libreoffice.org/49239
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 
(cherry picked from commit 31d2fc7cb079fc28674d8de8b8770c1e0e923047)

commit 950ee3d05a2a638591f4aa954cef31d1c0282742
Author: Pranav Kant 
Date:   Fri Feb 2 21:37:58 2018 +0530

loleaflet: Don't try to translate command names

Command names are sent to loleaflet in already translated form when
there are appropriate langpacks installed in the system.

Only translate when we can.

Change-Id: I92dc5865da85c76bd5a2529b0e6df994937c4c28
(cherry picked from commit 34aed93d839f99f46874ae744fa95c269f086d9e)
Reviewed-on: https://gerrit.libreoffice.org/49157
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 
(cherry picked from commit 198f3e7356684b97a658ab51a7553ca8319dd761)

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


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

2018-02-05 Thread Michael Stahl
 sw/qa/extras/ww8export/ww8export.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b2a053160c4c8eb0acd66b74c52dbda23a654bda
Author: Michael Stahl 
Date:   Mon Feb 5 16:53:03 2018 +0100

sw: disable testTableKeep on WNT

Fails from time to time, like so:

https://tinderbox.libreoffice.org/cgi-bin/gunzip.cgi?tree=MASTER&brief-log=1517842001.29090#err2

Change-Id: Ia33193fbfa05da58d31bcc5008beea32c1feb2fe

diff --git a/sw/qa/extras/ww8export/ww8export.cxx 
b/sw/qa/extras/ww8export/ww8export.cxx
index fab47934835f..f63260bcc056 100644
--- a/sw/qa/extras/ww8export/ww8export.cxx
+++ b/sw/qa/extras/ww8export/ww8export.cxx
@@ -1293,7 +1293,7 @@ DECLARE_WW8EXPORT_TEST(testCommentExport, 
"comment-export.odt")
 }
 }
 
-#if !defined(MACOSX)
+#if !defined(MACOSX) && !defined(_WIN32)
 #if !TEST_FONTS_MISSING
 DECLARE_WW8EXPORT_TEST(testTableKeep, "tdf91083.odt")
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: wsd/LOOLWSD.cpp

2018-02-05 Thread Miklos Vajna
 wsd/LOOLWSD.cpp |4 
 1 file changed, 4 deletions(-)

New commits:
commit 0a2df21501e66c35aeb7ae721ec9bdbaa04e198f
Author: Miklos Vajna 
Date:   Mon Feb 5 16:45:23 2018 +0100

wsd: remove unused using declarations in LOOLWSD

Change-Id: I6b656a1446b015430068e4b172cf98fe746f26d1

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 34482638..6b6696d8 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -134,11 +134,7 @@ using Poco::Net::HTTPResponse;
 using Poco::Net::MessageHeader;
 using Poco::Net::NameValueCollection;
 using Poco::Net::PartHandler;
-using Poco::Net::SocketAddress;
 using Poco::Path;
-#ifndef KIT_IN_PROCESS
-using Poco::Pipe;
-#endif
 using Poco::Process;
 using Poco::StreamCopier;
 using Poco::StringTokenizer;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3-desktop' - cui/source include/svx sd/source

2018-02-05 Thread Caolán McNamara
 cui/source/inc/backgrnd.hxx  |1 
 cui/source/tabpages/backgrnd.cxx |  104 ++-
 include/svx/flagsdef.hxx |   11 ++--
 sd/source/ui/dlg/dlgchar.cxx |2 
 sd/source/ui/func/fuchar.cxx |   26 -
 5 files changed, 91 insertions(+), 53 deletions(-)

New commits:
commit 8a322f65715af84f3a2dc089700671b9172f
Author: Caolán McNamara 
Date:   Fri Aug 11 11:51:09 2017 +0100

Resolves: tdf#110077 move the SvxBackgroundColorItem<->SvxBrushItem 
conversion

into the dialog itself

Reviewed-on: https://gerrit.libreoffice.org/41029
Tested-by: Jenkins 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 
(cherry picked from commit 6284804c2ccb3ad8bb6e1c1dac8d622f66b8c07d)

Reviewed-on: https://gerrit.libreoffice.org/41168
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 
(cherry picked from commit a407285747c65894bc858c4aa722a24e57caca31)

Change-Id: I83db9f02145a4927257dd5691f82dad1a11eaea6
(cherry picked from commit b0958ff8bf86b86682131b4da99da701a4a72adb)

diff --git a/cui/source/inc/backgrnd.hxx b/cui/source/inc/backgrnd.hxx
index d2663f3cd27e..cd7ab4ffec35 100644
--- a/cui/source/inc/backgrnd.hxx
+++ b/cui/source/inc/backgrnd.hxx
@@ -107,6 +107,7 @@ private:
 boolbIsGraphicValid : 1;
 boolbLinkOnly   : 1;
 boolbHighlighting   : 1;
+boolbCharBackColor  : 1;
 boolm_bColorSelected: 1;
 Graphic aBgdGraphic;
 OUStringaBgdGraphicPath;
diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx
index 43d8fdd2eddb..9f396624c830 100644
--- a/cui/source/tabpages/backgrnd.cxx
+++ b/cui/source/tabpages/backgrnd.cxx
@@ -38,6 +38,7 @@
 #include 
 
 #include 
+#include 
 #include "backgrnd.hxx"
 
 #include 
@@ -330,6 +331,7 @@ SvxBackgroundTabPage::SvxBackgroundTabPage(vcl::Window* 
pParent, const SfxItemSe
 , bIsGraphicValid(false)
 , bLinkOnly(false)
 , bHighlighting(false)
+, bCharBackColor(false)
 , m_bColorSelected(false)
 , pPageImpl(new SvxBackgroundPage_Impl)
 , pImportDlg(nullptr)
@@ -460,7 +462,7 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 
 
 // get and evaluate Input-BrushItem
-const SvxBrushItem* pBgdAttr = nullptr;
+bool bBrushItemSet = false;
 sal_uInt16 nSlot = SID_ATTR_BRUSH;
 const SfxPoolItem* pItem;
 sal_uInt16 nDestValue = USHRT_MAX;
@@ -488,20 +490,39 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 {
 nSlot = SID_ATTR_BRUSH_CHAR;
 }
+else if( bCharBackColor )
+{
+nSlot = SID_ATTR_CHAR_BACK_COLOR;
+}
+
 //#73# the destination item is missing when the parent style has been 
changed
 if(USHRT_MAX == nDestValue && m_pTblLBox->IsVisible())
 nDestValue = 0;
-sal_uInt16 nWhich = GetWhich( nSlot );
+sal_uInt16 nWhich = GetWhich(nSlot);
+SvxBrushItem aBgdAttr(nWhich);
 
-if ( rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT )
-pBgdAttr = static_cast(&( rSet->Get( nWhich ) ));
+if (rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT)
+{
+if (!bCharBackColor)
+aBgdAttr = static_cast(rSet->Get(nWhich));
+else
+{
+// EE_CHAR_BKGCOLOR is SvxBackgroundColorItem, but char background 
tabpage
+// can only work with SvxBrushItems
+// extract Color out of SvxBackColorItem
+Color aBackColor = static_cast(rSet->Get(nWhich)).GetValue();
+// make new SvxBrushItem with this Color
+aBgdAttr = SvxBrushItem(aBackColor, SID_ATTR_BRUSH_CHAR);
+}
+bBrushItemSet = true;
+}
 
 m_pBtnTile->Check();
 
-if ( pBgdAttr )
+if (bBrushItemSet)
 {
-FillControls_Impl(*pBgdAttr, aUserData);
-aBgdColor = const_cast(pBgdAttr)->GetColor();
+FillControls_Impl(aBgdAttr, aUserData);
+aBgdColor = aBgdAttr.GetColor();
 }
 else
 {
@@ -536,22 +557,22 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 nWhich = GetWhich( SID_ATTR_BRUSH );
 if ( rSet->GetItemState( nWhich, false ) >= SfxItemState::DEFAULT )
 {
-pBgdAttr = static_cast(&( rSet->Get( 
nWhich ) ));
-pTableBck_Impl->pCellBrush = new SvxBrushItem(*pBgdAttr);
+aBgdAttr = static_cast(rSet->Get(nWhich));
+pTableBck_Impl->pCellBrush = new SvxBrushItem(aBgdAttr);
 }
 pTableBck_Impl->nCellWhich = nWhich;
 
 if ( rSet->GetItemState( SID_ATTR_BRUSH_ROW, false ) >= 
SfxItemState::DEFAULT )
 {
-pBgdAttr = static_cast(&( rSet->Get( 
SID_ATTR_BRUSH_ROW ) ));
-pTableBck_Impl->pRowBrush = new SvxBrushItem(*pBgdAttr);
+aBgdAttr = static_cast(rSet->Get(SID_ATTR_BRUSH_ROW));
+

[Libreoffice-commits] online.git: Changes to 'refs/tags/CODE-3.0.0.4'

2018-02-05 Thread Andras Timar
Tag 'CODE-3.0.0.4' created by Andras Timar  at 
2018-02-05 15:54 +

CODE-3.0.0.4

Changes since CODE-3.0.0.3-2:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Changes to 'refs/tags/3.0-final'

2018-02-05 Thread Andras Timar
Tag '3.0-final' created by Andras Timar  at 
2018-02-05 15:55 +

3.0-final

Changes since CODE-3.0.0.3-2:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: icon-themes/sifr icon-themes/sifr_dark

2018-02-05 Thread Matthias Freund
 icon-themes/sifr/cmd/32/backcolor.png  |binary
 icon-themes/sifr/cmd/32/bold.png   |binary
 icon-themes/sifr/cmd/lc_bold.png   |binary
 icon-themes/sifr/cmd/sc_backcolor.png  |binary
 icon-themes/sifr/cmd/sc_bold.png   |binary
 icon-themes/sifr_dark/cmd/32/backcolor.png |binary
 icon-themes/sifr_dark/cmd/32/bold.png  |binary
 icon-themes/sifr_dark/cmd/lc_backcolor.png |binary
 icon-themes/sifr_dark/cmd/lc_bold.png  |binary
 icon-themes/sifr_dark/cmd/sc_backcolor.png |binary
 icon-themes/sifr_dark/cmd/sc_bold.png  |binary
 11 files changed

New commits:
commit 2cde59c5441d9195e016f8f0b2da697fc4ae2a9a
Author: Matthias Freund 
Date:   Sun Feb 4 21:51:29 2018 +0100

tdf#75256 Sifr update 32px for bold and backcolor

Change-Id: Ibd6dc50925ad6188292e38c87f5ffbbf10792993
Reviewed-on: https://gerrit.libreoffice.org/49217
Tested-by: Jenkins 
Reviewed-by: Matthias Freund 

diff --git a/icon-themes/sifr/cmd/32/backcolor.png 
b/icon-themes/sifr/cmd/32/backcolor.png
new file mode 100644
index ..1ab5a920775d
Binary files /dev/null and b/icon-themes/sifr/cmd/32/backcolor.png differ
diff --git a/icon-themes/sifr/cmd/32/bold.png b/icon-themes/sifr/cmd/32/bold.png
new file mode 100644
index ..a1d742570fff
Binary files /dev/null and b/icon-themes/sifr/cmd/32/bold.png differ
diff --git a/icon-themes/sifr/cmd/lc_bold.png b/icon-themes/sifr/cmd/lc_bold.png
index 9ed70b4c4be6..f2e1706d03d9 100644
Binary files a/icon-themes/sifr/cmd/lc_bold.png and 
b/icon-themes/sifr/cmd/lc_bold.png differ
diff --git a/icon-themes/sifr/cmd/sc_backcolor.png 
b/icon-themes/sifr/cmd/sc_backcolor.png
index 9f000129e1a1..f12aee1b8d17 100644
Binary files a/icon-themes/sifr/cmd/sc_backcolor.png and 
b/icon-themes/sifr/cmd/sc_backcolor.png differ
diff --git a/icon-themes/sifr/cmd/sc_bold.png b/icon-themes/sifr/cmd/sc_bold.png
index ce7bc4eecadb..368a8cdd5692 100644
Binary files a/icon-themes/sifr/cmd/sc_bold.png and 
b/icon-themes/sifr/cmd/sc_bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/32/backcolor.png 
b/icon-themes/sifr_dark/cmd/32/backcolor.png
new file mode 100644
index ..a470fc676cf8
Binary files /dev/null and b/icon-themes/sifr_dark/cmd/32/backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/32/bold.png 
b/icon-themes/sifr_dark/cmd/32/bold.png
new file mode 100644
index ..20c364464ffa
Binary files /dev/null and b/icon-themes/sifr_dark/cmd/32/bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/lc_backcolor.png 
b/icon-themes/sifr_dark/cmd/lc_backcolor.png
index 5c7e2095afb5..a5badadc6fb1 100644
Binary files a/icon-themes/sifr_dark/cmd/lc_backcolor.png and 
b/icon-themes/sifr_dark/cmd/lc_backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/lc_bold.png 
b/icon-themes/sifr_dark/cmd/lc_bold.png
index 98e43c23c147..50db6cf345d1 100644
Binary files a/icon-themes/sifr_dark/cmd/lc_bold.png and 
b/icon-themes/sifr_dark/cmd/lc_bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/sc_backcolor.png 
b/icon-themes/sifr_dark/cmd/sc_backcolor.png
index 737e08c4de53..08ecc326c8c3 100644
Binary files a/icon-themes/sifr_dark/cmd/sc_backcolor.png and 
b/icon-themes/sifr_dark/cmd/sc_backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/sc_bold.png 
b/icon-themes/sifr_dark/cmd/sc_bold.png
index 148f88b4f609..6dab8dcd6fe5 100644
Binary files a/icon-themes/sifr_dark/cmd/sc_bold.png and 
b/icon-themes/sifr_dark/cmd/sc_bold.png differ
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread jan Iversen
 ios/CustomTarget_iOS_setup.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 2db0c3e04348d3096dff72c568c461be3cb1a0c6
Author: jan Iversen 
Date:   Mon Feb 5 19:37:43 2018 +0100

iOS, add tango theme to avoid vcl problem.

Change-Id: I73bee8ce04ec6234187224d03f0e081f09037e72

diff --git a/ios/CustomTarget_iOS_setup.mk b/ios/CustomTarget_iOS_setup.mk
index 8fea759249e9..eb102f669bf3 100644
--- a/ios/CustomTarget_iOS_setup.mk
+++ b/ios/CustomTarget_iOS_setup.mk
@@ -66,6 +66,7 @@ $(IOSGEN)/native-code.h: $(BUILDDIR)/config_host.mk \
cp $(INSTDIR)/share/filter/oox-drawingml-cs-presets 
$(IOSRES)/share/filter
cp $(INSTDIR)/share/filter/vml-shape-types $(IOSRES)/share/filter
cp -R $(INSTDIR)/share/registry $(IOSRES)/share
+   cp $(INSTDIR)/share/config/*zip $(IOSRES)/share/config
 
# Set up rc (the "inifile", fundamentalrc, unorc, bootstraprc and 
versionrc.
(echo '[Bootstrap]' \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Tomaž Vajngerl
 include/svx/svdograf.hxx   |4 
 svx/source/svdraw/svdograf.cxx |  181 +++--
 2 files changed, 88 insertions(+), 97 deletions(-)

New commits:
commit 94411b5765427344d106e5cca12c032ca5a1ea6a
Author: Tomaž Vajngerl 
Date:   Mon Feb 5 10:02:03 2018 +0900

use unique_ptr to store GraphicObject for SdrGrafObj

Change-Id: I578156cd96e1e1400e93e96ace8a38759e3d1b2c
Reviewed-on: https://gerrit.libreoffice.org/49221
Reviewed-by: Tomaž Vajngerl 
Tested-by: Tomaž Vajngerl 

diff --git a/include/svx/svdograf.hxx b/include/svx/svdograf.hxx
index b4fd3baba256..f31ed858e575 100644
--- a/include/svx/svdograf.hxx
+++ b/include/svx/svdograf.hxx
@@ -94,8 +94,8 @@ private:
 OUString aFileName; // If it's a Link, the filename can be found in here
 OUString aReferer;
 OUString aFilterName;
-GraphicObject* pGraphic; // In order to speed up output of bitmaps, 
especially rotated ones
-GraphicObject* mpReplacementGraphic;
+std::unique_ptr mpGraphicObject; // In order to speed up 
output of bitmaps, especially rotated ones
+std::unique_ptr mpReplacementGraphicObject;
 SdrGraphicLink* pGraphicLink; // And here a pointer for linked graphics
 bool bMirrored:1; // True: the graphic is horizontal, which means it's 
mirrored along the y-axis
 
diff --git a/svx/source/svdraw/svdograf.cxx b/svx/source/svdraw/svdograf.cxx
index 7297beb2c914..29dffa90fd3b 100644
--- a/svx/source/svdraw/svdograf.cxx
+++ b/svx/source/svdraw/svdograf.cxx
@@ -282,10 +282,10 @@ sdr::contact::ViewContact* 
SdrGrafObj::CreateObjectSpecificViewContact()
 
 void SdrGrafObj::onGraphicChanged()
 {
-if (!pGraphic || pGraphic->IsSwappedOut()) // don't force swap-in for this
+if (!mpGraphicObject || mpGraphicObject->IsSwappedOut()) // don't force 
swap-in for this
 return;
 
-const VectorGraphicDataPtr& rVectorGraphicDataPtr = 
pGraphic->GetGraphic().getVectorGraphicData();
+const VectorGraphicDataPtr& rVectorGraphicDataPtr = 
mpGraphicObject->GetGraphic().getVectorGraphicData();
 
 if (!rVectorGraphicDataPtr.get())
 return;
@@ -328,15 +328,14 @@ void SdrGrafObj::onGraphicChanged()
 
 SdrGrafObj::SdrGrafObj()
 : SdrRectObj()
-,  pGraphicLink(nullptr)
-,  bMirrored(false)
-,  mbIsSignatureLine(false)
-,  mbIsSignatureLineShowSignDate(true)
-,  mbIsSignatureLineCanAddComment(false)
-{
-pGraphic = new GraphicObject;
-mpReplacementGraphic = nullptr;
-pGraphic->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
+, mpGraphicObject(new GraphicObject)
+, pGraphicLink(nullptr)
+, bMirrored(false)
+, mbIsSignatureLine(false)
+, mbIsSignatureLineShowSignDate(true)
+, mbIsSignatureLineCanAddComment(false)
+{
+mpGraphicObject->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
 onGraphicChanged();
 
 // #i118485# Shear allowed and possible now
@@ -353,17 +352,16 @@ SdrGrafObj::SdrGrafObj()
 mbSupportTextIndentingOnLineWidthChange = false;
 }
 
-SdrGrafObj::SdrGrafObj(const Graphic& rGrf, const tools::Rectangle& rRect)
+SdrGrafObj::SdrGrafObj(const Graphic& rGraphic, const tools::Rectangle& rRect)
 : SdrRectObj(rRect)
+, mpGraphicObject(new GraphicObject(rGraphic))
 , pGraphicLink(nullptr)
 , bMirrored(false)
 , mbIsSignatureLine(false)
 , mbIsSignatureLineShowSignDate(true)
 , mbIsSignatureLineCanAddComment(false)
 {
-pGraphic = new GraphicObject( rGrf );
-mpReplacementGraphic = nullptr;
-pGraphic->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
+mpGraphicObject->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
 onGraphicChanged();
 
 // #i118485# Shear allowed and possible now
@@ -380,17 +378,16 @@ SdrGrafObj::SdrGrafObj(const Graphic& rGrf, const 
tools::Rectangle& rRect)
 mbSupportTextIndentingOnLineWidthChange = false;
 }
 
-SdrGrafObj::SdrGrafObj( const Graphic& rGrf )
+SdrGrafObj::SdrGrafObj(const Graphic& rGraphic)
 : SdrRectObj()
+, mpGraphicObject(new GraphicObject(rGraphic))
 , pGraphicLink(nullptr)
 , bMirrored(false)
 , mbIsSignatureLine(false)
 , mbIsSignatureLineShowSignDate(true)
 , mbIsSignatureLineCanAddComment(false)
 {
-pGraphic = new GraphicObject( rGrf );
-mpReplacementGraphic = nullptr;
-pGraphic->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
+mpGraphicObject->SetSwapStreamHdl( LINK(this, SdrGrafObj, ImpSwapHdl) );
 onGraphicChanged();
 
 // #i118485# Shear allowed and possible now
@@ -409,18 +406,15 @@ SdrGrafObj::SdrGrafObj( const Graphic& rGrf )
 
 SdrGrafObj::~SdrGrafObj()
 {
-delete pGraphic;
-delete mpReplacementGraphic;
 ImpDeregisterLink();
 }
 
-void SdrGrafObj::SetGraphicObject( const GraphicObject& rGrfObj )
+void SdrGrafObj::SetGraphicObject(const GraphicObject& rGraphicObject)
 {
-*pGraphic = rGrfObj;
-delete mpReplacementGraphic;
-mpReplacementGraphic = nullptr;
-pGraph

[Libreoffice-commits] core.git: Branch 'aoo/trunk' - toolkit/inc toolkit/source

2018-02-05 Thread Don Lewis
 toolkit/inc/layout/layout.hxx|2 
 toolkit/source/layout/vcl/wbutton.cxx|   72 
 toolkit/source/layout/vcl/wcontainer.cxx |8 -
 toolkit/source/layout/vcl/wfield.cxx |   87 +--
 toolkit/source/layout/vcl/wrapper.cxx|  136 ++-
 toolkit/source/layout/vcl/wrapper.hxx|6 -
 6 files changed, 150 insertions(+), 161 deletions(-)

New commits:
commit 837bd0e46c8cb64362614bb6c548addee73f80fa
Author: Don Lewis 
Date:   Mon Feb 5 18:20:47 2018 +

A number of methods in main/toolkit/source/layout/vcl/* use either

"!this" or "this" in a boolean context in order to detect being
called with a null object and exit early.  In a valid C++ program
"this" will never be null, so a compiler could legally optimize out
this comparision, which could potentially result in the method
dereferencing a null pointer later in the code.  This situation
could only happen if the caller is using a null object pointer to
call the method or is using a object ref to call the method that
was generated by dereferencing a null pointer, neither of which is
valid.  Resolve this by moving the checks out of the method and
into the caller.

Make this easier by changing the getImpl() method to return the
private *Impl pointer directly instead of deferencing the pointer
and returning a ref.  The latter is invalid if the pointer is null.
This allows GetImpl() to be called in a boolean contect to peform
the check.  It also allows a number of instances of "&getImpl()"
calls in a boolean context to be fixed by changing them to "getImpl()".
The address of a ref will never be zero in a valid C++ program, so
the compiler could potentially optimize out those checks.

There does not appear to be any need for Control and ComboBox to
use customized versions of GetImpl() since these appear to behave
identically to the versions generated by the canned macro, so switch
them back to the macro version.

This commit should result in no functional changes.

It seems like all of these checks for a null implementation should
not be necessary, but that requires further investigation.

diff --git a/toolkit/inc/layout/layout.hxx b/toolkit/inc/layout/layout.hxx
index 01dbd2228610..69e38af00d55 100644
--- a/toolkit/inc/layout/layout.hxx
+++ b/toolkit/inc/layout/layout.hxx
@@ -102,7 +102,7 @@ public:
 t( Window *parent, WinBits nStyle = defaultWinBit ); \
 t (Window *parent, ResId const &res)
 #define DECL_GET_IMPL(t) \
-inline t##Impl &getImpl() const
+inline t##Impl *getImpl() const
 
 #define DECL_GET_WINDOW( cls ) ::cls* Get##cls() const
 #define IMPL_GET_WINDOW( cls ) ::cls* cls::Get##cls() const { return 
dynamic_cast< ::cls*>( GetWindow() ); }
diff --git a/toolkit/source/layout/vcl/wbutton.cxx 
b/toolkit/source/layout/vcl/wbutton.cxx
index c769bdca736e..afdeca8a957b 100644
--- a/toolkit/source/layout/vcl/wbutton.cxx
+++ b/toolkit/source/layout/vcl/wbutton.cxx
@@ -140,35 +140,36 @@ String Button::GetStandardText (sal_uInt16 button_type)
 
 void Button::SetText( OUString const& rStr )
 {
-if ( !getImpl().mxButton.is() )
+if ( !getImpl()->mxButton.is() )
 return;
-getImpl().mxButton->setLabel( rStr );
+getImpl()->mxButton->setLabel( rStr );
 }
 
 void Button::SetClickHdl( const Link& link )
 {
-if (&getImpl () && getImpl().mxButton.is ())
-getImpl().SetClickHdl( link );
+if (getImpl() && getImpl()->mxButton.is ())
+getImpl()->SetClickHdl( link );
 }
 
 Link& Button::GetClickHdl ()
 {
-return getImpl().GetClickHdl ();
+return getImpl()->GetClickHdl ();
 }
 
 bool Button::SetModeImage (Image const& image)
 {
-return getImpl().SetModeImage (image.getImpl().mxGraphic);
+return getImpl() || getImpl()->SetModeImage (image.getImpl().mxGraphic);
 }
 
 bool Button::SetModeImage (::Image const& image, BmpColorMode mode)
 {
-return GetButton ()->SetModeImage (image, mode);
+return !GetButton () || GetButton ()->SetModeImage (image, mode);
 }
 
 void Button::SetImageAlign( ImageAlign eAlign )
 {
-getImpl().setProperty( "ImageAlign", uno::Any( (sal_Int16) eAlign ) );
+if ( getImpl() )
+getImpl()->setProperty( "ImageAlign", uno::Any( (sal_Int16) eAlign ) );
 }
 
 void Button::Click()
@@ -229,14 +230,15 @@ PushButton::~PushButton ()
 
 void PushButton::Check( bool bCheck )
 {
-getImpl().setProperty( "State", uno::Any( (sal_Int16) !!bCheck ) );
+if ( getImpl() )
+getImpl()->setProperty( "State", uno::Any( (sal_Int16) !!bCheck ) );
 // XButton doesn't have explicit toggle event
-getImpl().fireToggle();
+getImpl()->fireToggle();
 }
 
 bool PushButton::IsChecked() const
 {
-return !!( getImpl().getProperty( "State" ).get< sal_Int16 >() );
+return !!( getImpl() && getImpl()->getProperty( "State" ).get< sal_Int16 
>() );

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

2018-02-05 Thread Miklos Vajna
 sw/inc/docufld.hxx|   33 +++
 sw/source/core/fields/docufld.cxx |  168 +++---
 2 files changed, 101 insertions(+), 100 deletions(-)

New commits:
commit 3e6b60b27fe2eb4c8d38aa4ccafaf166ebbfd76d
Author: Miklos Vajna 
Date:   Mon Feb 5 10:09:01 2018 +0100

sw: prefix members of SwHiddenParaField, SwHiddenTextField and 
SwJumpEditField

Change-Id: I23a5ea2d85f5f22f48a2c069d84929b4d70f62a0
Reviewed-on: https://gerrit.libreoffice.org/49225
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins 

diff --git a/sw/inc/docufld.hxx b/sw/inc/docufld.hxx
index bcd44403e1ef..3f1f6b7c494c 100644
--- a/sw/inc/docufld.hxx
+++ b/sw/inc/docufld.hxx
@@ -277,28 +277,28 @@ public:
 
 class SwHiddenTextFieldType : public SwFieldType
 {
-bool bHidden;
+bool m_bHidden;
 public:
 SwHiddenTextFieldType(bool bSetHidden = true);
 
 virtual SwFieldType*Copy() const override;
 
 voidSetHiddenFlag( bool bSetHidden );
-bool GetHiddenFlag() const { return bHidden; }
+bool GetHiddenFlag() const { return m_bHidden; }
 };
 
 class SW_DLLPUBLIC SwHiddenTextField : public SwField
 {
-OUString aTRUEText; ///< Text if condition true.
-OUString aFALSEText;///< If condition false.
-OUString aContent; ///< Evaluated DB-text.
+OUString m_aTRUEText; ///< Text if condition true.
+OUString m_aFALSEText; ///< If condition false.
+OUString m_aContent; ///< Evaluated DB-text.
 
-OUString aCond;///< Condition.
-sal_uInt16  nSubType;
+OUString m_aCond; ///< Condition.
+sal_uInt16 m_nSubType;
 
-boolbCanToggle : 1; ///< Can field be toggled alone?
-boolbIsHidden  : 1; ///< Is it not visible?
-boolbValid : 1; ///< Is DB-field evaluated?
+bool m_bCanToggle : 1; ///< Can field be toggled alone?
+bool m_bIsHidden : 1; ///< Is it not visible?
+bool m_bValid : 1; ///< Is DB-field evaluated?
 
 virtual OUStringExpand() const override;
 virtual SwField*Copy() const override;
@@ -321,7 +321,7 @@ public:
 
 voidEvaluate(SwDoc*);
 
-void SetValue(bool bHidden)  { bIsHidden = bHidden; }
+void SetValue(bool bHidden)  { m_bIsHidden = bHidden; }
 static OUString GetColumnName(const OUString& rName);
 static OUString GetDBName(const OUString& rName, SwDoc *pDoc);
 
@@ -355,8 +355,8 @@ public:
 
 class SwHiddenParaField : public SwField
 {
-OUString   aCond;
-boolbIsHidden:1;
+OUString m_aCond;
+bool m_bIsHidden;
 public:
 /// Direct input, delete old value.
 SwHiddenParaField(SwHiddenParaFieldType*, const OUString& rCond);
@@ -364,8 +364,8 @@ public:
 virtual OUStringExpand() const override;
 virtual SwField*Copy() const override;
 
-voidSetHidden(bool bHidden) { bIsHidden = bHidden; }
-boolIsHidden() const{ return bIsHidden;}
+voidSetHidden(bool bHidden) { m_bIsHidden = bHidden; }
+boolIsHidden() const{ return m_bIsHidden;}
 
 /// Query, set condition.
 virtual OUStringGetPar1() const override;
@@ -643,7 +643,8 @@ public:
 
 class SwJumpEditField : public SwField
 {
-OUString sText, sHelp;
+OUString m_sText;
+OUString m_sHelp;
 public:
 SwJumpEditField( SwJumpEditFieldType*, sal_uInt32 nFormat,
  const OUString& sText, const OUString& sHelp );
diff --git a/sw/source/core/fields/docufld.cxx 
b/sw/source/core/fields/docufld.cxx
index fa0e3edc807a..174eb02afaab 100644
--- a/sw/source/core/fields/docufld.cxx
+++ b/sw/source/core/fields/docufld.cxx
@@ -1252,20 +1252,20 @@ bool SwDocInfoField::PutValue( const uno::Any& rAny, 
sal_uInt16 nWhichId )
 }
 
 SwHiddenTextFieldType::SwHiddenTextFieldType( bool bSetHidden )
-: SwFieldType( SwFieldIds::HiddenText ), bHidden( bSetHidden )
+: SwFieldType( SwFieldIds::HiddenText ), m_bHidden( bSetHidden )
 {
 }
 
 SwFieldType* SwHiddenTextFieldType::Copy() const
 {
-return new SwHiddenTextFieldType( bHidden );
+return new SwHiddenTextFieldType( m_bHidden );
 }
 
 void SwHiddenTextFieldType::SetHiddenFlag( bool bSetHidden )
 {
-if( bHidden != bSetHidden )
+if( m_bHidden != bSetHidden )
 {
-bHidden = bSetHidden;
+m_bHidden = bSetHidden;
 UpdateFields();   // notify all HiddenTexts
 }
 }
@@ -1276,26 +1276,26 @@ SwHiddenTextField::SwHiddenTextField( 
SwHiddenTextFieldType* pFieldType,
 const OUString& rStr,
 boolbHidden,
 sal_uInt16  nSub) :
-SwField( pFieldType ), aCond(rCond), nSubType(nSub),
-bCanToggle(bConditional), bIsHidden(bHidden), bValid(false)
+SwField( pFieldType ), m_aCond(rCond), m

Re: Minutes of the design meeting 2018-Jan-31

2018-02-05 Thread Michel RENON

Hi,

Le 31/01/2018 à 21:48, Heiko Tietze a écrit :

[...]
GSoC

  * Full list of ideas https://pad.documentfoundation.org/p/UX-GSoC_Ideas
  * Public list 
https://wiki.documentfoundation.org/Development/GSoC/Ideas#User_Interface


Here are some suggestions based on my recent experience in teaching 
LibreOffice and writing a python macro :




Impress : work on "papercut" bugs
--


1 - Animations sets for a text box with several paragraphs don't affect 
new paragraphs (not inherited)

https://bugs.documentfoundation.org/show_bug.cgi?id=57581

This is the most annoying problem that new users face :
  - they create a slide with few lines of text,
  - they add an animation on the text, to make it appear line by line
  - they test : it works correctly
  - they add one line of text (inside or at the end)
  - they test and boom... the animation is "broken"
  - they make a pause, then... close impress and open powerpoint...



2 - Editing: Text spacing changes when click in textbox
https://bugs.documentfoundation.org/show_bug.cgi?id=89060

It is a subtle but very annoying visual effect.






Python macros



I had to create a macro embedded in a file.
As I had not written basic for a long time, I tried to write it in python.

The fist problem is that on Ubuntu, it is necessary to install a 
specific package 'libreoffice-script-provider-python'



The second problem is that, by default, there is no tool to integrate a 
python macro in a document.

I found the APSO extension [1] that allows that and is very useful.

I found some difficult points in writing python macro with that extension :
  - after modifying a python script, you have to close and re-open the 
odf file

  - some inline documentation is missing : about API, and some sample code
  - the context (available via XSCRIPTCONTEXT) is missing some API, 
compared to basic






So here are my suggestions for GSoC :
  - allow an easy activation of python support : a simple button would 
download and install all necessary packages
  - integrate the functionality of APSO (to allow embedding/managing 
python script in odf file)
  - remove the need to re-open the odf file before using a new version 
of python script




Other points can be done directly in the current extension :
  - add some inline documentation about API [2]
  - add inline documentation on both ways to use python script :
- an external script that connects to LibreOffice
- an internal script that get called by menu/button
  - allow to create scripts empty or with comments (that provide 
minimum documentation and good practice)
  - add a library of sample code (a list of "how-to ?"), that can be 
easily copied in script files



Adding API to the context (available via XSCRIPTCONTEXT) can be done by 
working on LibreOffice code :

/usr/lib/libreoffice/program/pythonscript/py






Cheers,

Michel






[1] 
https://extensions.libreoffice.org/extensions/apso-alternative-script-organizer-for-python


[2] based on very useful page 
https://wiki.openoffice.org/wiki/Python/Transfer_from_Basic_to_Python

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


[Libreoffice-commits] core.git: Branch 'private/swe/libreoffice-5-2+backports' - cui/source

2018-02-05 Thread Eike Rathke
 cui/source/inc/backgrnd.hxx  |1 +
 cui/source/tabpages/backgrnd.cxx |   12 ++--
 2 files changed, 7 insertions(+), 6 deletions(-)

New commits:
commit f9a0f6bad57949c508024a7fcbca1f783788ec03
Author: Eike Rathke 
Date:   Tue Jun 20 23:56:44 2017 +0200

Resolves: tdf#105740 select without selecting.. tdf#101588 b0rk

The initial color, if determined, must result in being selected. 
Differentiate
between pre-selected and user-selected.

Regression of

commit 429be969b39867d6d9f36978c9ac15b601e78ba6
Date:   Mon Dec 12 21:24:20 2016 +0100

Change-Id: I928af131d03c61a8e474e5cd356aea8b766112fa
Reviewed-on: https://gerrit.libreoffice.org/49256
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/cui/source/inc/backgrnd.hxx b/cui/source/inc/backgrnd.hxx
index e2b8d84e5ca1..06497440b7c7 100644
--- a/cui/source/inc/backgrnd.hxx
+++ b/cui/source/inc/backgrnd.hxx
@@ -108,6 +108,7 @@ private:
 boolbIsGraphicValid : 1;
 boolbLinkOnly   : 1;
 boolbHighlighting   : 1;
+boolm_bColorSelected: 1;
 Graphic aBgdGraphic;
 OUStringaBgdGraphicPath;
 OUStringaBgdGraphicFilter;
diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx
index 1f34633e6fcd..59df9a96c345 100644
--- a/cui/source/tabpages/backgrnd.cxx
+++ b/cui/source/tabpages/backgrnd.cxx
@@ -330,6 +330,7 @@ SvxBackgroundTabPage::SvxBackgroundTabPage(vcl::Window* 
pParent, const SfxItemSe
 , bIsGraphicValid(false)
 , bLinkOnly(false)
 , bHighlighting(false)
+, m_bColorSelected(false)
 , pPageImpl(new SvxBackgroundPage_Impl)
 , pImportDlg(nullptr)
 , pTableBck_Impl(nullptr)
@@ -447,6 +448,8 @@ void SvxBackgroundTabPage::Reset( const SfxItemSet* rSet )
 return;
 }
 
+m_bColorSelected = false;
+
 // condition of the preview button is persistent due to UserData
 OUString aUserData = GetUserData();
 m_pBtnPreview->Check( !aUserData.isEmpty() && '1' == aUserData[0] );
@@ -681,7 +684,7 @@ bool SvxBackgroundTabPage::FillItemSet( SfxItemSet* 
rCoreSet )
 {
 // Brush-treatment:
 if ( rOldItem.GetColor() != aBgdColor ||
- (SfxItemState::DEFAULT >= eOldItemState && 
!m_pBackgroundColorSet->IsNoSelection()))
+ (SfxItemState::DEFAULT >= eOldItemState && 
m_bColorSelected))
 {
 bModified = true;
 rCoreSet->Put( SvxBrushItem( aBgdColor, nWhich ) );
@@ -1135,6 +1138,7 @@ IMPL_LINK_NOARG_TYPED(SvxBackgroundTabPage, 
BackgroundColorHdl_Impl, ValueSet*,
 sal_uInt16 nItemId = m_pBackgroundColorSet->GetSelectItemId();
 Color aColor = nItemId ? ( m_pBackgroundColorSet->GetItemColor( nItemId ) 
) : Color( COL_TRANSPARENT );
 aBgdColor = aColor;
+m_bColorSelected = true;
 m_pPreviewWin1->NotifyChange( aBgdColor );
 }
 
@@ -1421,13 +1425,9 @@ void SvxBackgroundTabPage::FillControls_Impl( const 
SvxBrushItem& rBgdAttr,
 }
 else
 {
-bool bNoSelection = m_pBackgroundColorSet->IsNoSelection();
 m_pBackgroundColorSet->SelectItem( nCol );
-m_pBackgroundColorSet->SaveValue();
-// The actual selection is user set, not what we preset.
-if (bNoSelection)
-m_pBackgroundColorSet->SetNoSelection();
 }
+m_pBackgroundColorSet->SaveValue();
 
 m_pPreviewWin1->NotifyChange( aBgdColor );
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Caolán McNamara
 lotuswordpro/source/filter/lwpfribtable.cxx |   20 ++--
 1 file changed, 10 insertions(+), 10 deletions(-)

New commits:
commit d525fe37700afd39ab1ced7c28476229bc5751dd
Author: Caolán McNamara 
Date:   Mon Feb 5 20:56:29 2018 +

ofz#6024 Direct-leak

Change-Id: I43959dea19860bc0d34c4623fc86ccb3ce069189

diff --git a/lotuswordpro/source/filter/lwpfribtable.cxx 
b/lotuswordpro/source/filter/lwpfribtable.cxx
index 67135fff4603..f675fd77e5d1 100644
--- a/lotuswordpro/source/filter/lwpfribtable.cxx
+++ b/lotuswordpro/source/filter/lwpfribtable.cxx
@@ -144,9 +144,9 @@ void LwpFribTable::XFConvert(XFContentContainer* pCont)
 sChangeID = pChangeMgr->GetChangeID(this);
 if (!sChangeID.isEmpty() && pXFContentContainer)
 {
-XFChangeStart* pChangeStart = new XFChangeStart;
-pChangeStart->SetChangeID(sChangeID);
-pXFContentContainer->Add(pChangeStart);
+rtl::Reference xChangeStart(new XFChangeStart);
+xChangeStart->SetChangeID(sChangeID);
+pXFContentContainer->Add(xChangeStart.get());
 }
 }
 
@@ -157,19 +157,19 @@ void LwpFribTable::XFConvert(XFContentContainer* pCont)
 {
 if (!sChangeID.isEmpty() && pXFContentContainer)
 {
-XFChangeEnd* pChangeEnd = new XFChangeEnd;
-pChangeEnd->SetChangeID(sChangeID);
-pXFContentContainer->Add(pChangeEnd);
+rtl::Reference xChangeEnd(new XFChangeEnd);
+xChangeEnd->SetChangeID(sChangeID);
+pXFContentContainer->Add(xChangeEnd.get());
 }
 }
 
 if(LwpLayoutRelativityGuts::LAY_INLINE_NEWLINE == nType
 && HasNextFrib())
 {
-XFParagraph* pXFPara = new XFParagraph();
-pXFPara->SetStyleName(m_StyleName);
-m_pPara->AddXFContent(pXFPara);
-m_pPara->GetFribs().SetXFPara(pXFPara);
+rtl::Reference xXFPara(new XFParagraph);
+xXFPara->SetStyleName(m_StyleName);
+m_pPara->AddXFContent(xXFPara.get());
+m_pPara->GetFribs().SetXFPara(xXFPara.get());
 }
 
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - b4/027b74a699d7a8070d0669e780b7fbe4d03807

2018-02-05 Thread Caolán McNamara
 b4/027b74a699d7a8070d0669e780b7fbe4d03807 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit ba529fd197fd2e99d3f549868d70c64565e447d8
Author: Caolán McNamara 
Date:   Mon Feb 5 21:00:39 2018 +

Notes added by 'git notes add'

diff --git a/b4/027b74a699d7a8070d0669e780b7fbe4d03807 
b/b4/027b74a699d7a8070d0669e780b7fbe4d03807
new file mode 100644
index ..75dcc599ca90
--- /dev/null
+++ b/b4/027b74a699d7a8070d0669e780b7fbe4d03807
@@ -0,0 +1 @@
+prefer: 6d3b26829b5f30ea934249ef88eb10b2507969f1
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: officecfg/util

2018-02-05 Thread Stephan Bergmann
 officecfg/util/component-conf.gen |   79 --
 officecfg/util/component-ldif.gen |   76 -
 officecfg/util/component-map.gen  |   87 --
 3 files changed, 242 deletions(-)

New commits:
commit 0f7825e7f32b5e62ab70d0b4856fe0aa16594fed
Author: Stephan Bergmann 
Date:   Mon Feb 5 16:29:52 2018 +0100

Remove unused officecfg/util/component-*.gen files

...that appear to never have been used for anything that would still be 
relevant
for LO today at least, and have effectively never been modified since their
introduction with 0ff65e7aa039b442e61080aae50e4ef629f8ff4c "#97750# property
mapping for LDAP DS" (and seem to be about some other LDAP integration than
what is provided by officecfg/registry/schema/oo-*.xcd.sample)

Change-Id: I36f92dbb187ff6966e0b89152f3dd3eeb128662e
Reviewed-on: https://gerrit.libreoffice.org/49248
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/officecfg/util/component-conf.gen 
b/officecfg/util/component-conf.gen
deleted file mode 100755
index 846e765521a6..
--- a/officecfg/util/component-conf.gen
+++ /dev/null
@@ -1,79 +0,0 @@
-: # -*- perl -*-
-#
-# This file is part of the LibreOffice project.
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-#
-# This file incorporates work covered by the following license notice:
-#
-#   Licensed to the Apache Software Foundation (ASF) under one or more
-#   contributor license agreements. See the NOTICE file distributed
-#   with this work for additional information regarding copyright
-#   ownership. The ASF licenses this file to you under the Apache
-#   License, Version 2.0 (the "License"); you may not use this file
-#   except in compliance with the License. You may obtain a copy of
-#   the License at http://www.apache.org/licenses/LICENSE-2.0 .
-#
-
-# generates of the component schema list an ldap schema in slapd format
-
-eval 'exec perl -wS $0 ${1+"$@"}'
-   if 0;
-
-#creating the output file
-open(OUTFILE, ">$ARGV[0]") or die "can't open >$ARGV[0]";
-
-#open the makefile
-open(INFILE, "makefile.mk") or die "can't open makefile.mk";
-
-$inxcs=0;
-
-#search all schemas in the makefile except userprofile and format them as e.g 
org.openoffice.Inet
-while() {
-   tr/\r\n//d;
-
-   if (/^\s*XCSFILES/) {
-   $inxcs++;
-   }
-   next unless $inxcs;
-
-   if ($inxcs) {
-
-   $inxcs=0 unless /\\$/;
-
-   next if (/^\s*XCSFILES/);
-   next if (/UserProfile/);
-
-   s/^\s+//;
-   s/\s*\\$//;
-   s/\.xcs.*$//;
-   s#\$/#.#g;
-
-   push(@comp_names, $_);
-   }
-}
-close(INFILE);
-
-# generate the schema in slapd format
-
-foreach (@comp_names) {
-
-   s#org.openoffice.#oo-#g;
-   s#\.#-#g;
-
-   $myLdapName=lc $_;
-
-   print OUTFILE "attribute $myLdapName-attr $myLdapName-attr-oid cis 
single\n";
-   print OUTFILE "objectclass $myLdapName-class\n";
-   print OUTFILE "\toid\n";
-   print OUTFILE "\t\t$myLdapName-class-oid\n";
-   print OUTFILE "\tsuperior top\n";
-   print OUTFILE "\tallows\n";
-   print OUTFILE "\t$myLdapName-attr\n";
-}
-
-close(OUTFILE);
-
-exit 0;
diff --git a/officecfg/util/component-ldif.gen 
b/officecfg/util/component-ldif.gen
deleted file mode 100755
index db26e31a35ee..
--- a/officecfg/util/component-ldif.gen
+++ /dev/null
@@ -1,76 +0,0 @@
-: # -*- perl -*-
-#
-# This file is part of the LibreOffice project.
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-#
-# This file incorporates work covered by the following license notice:
-#
-#   Licensed to the Apache Software Foundation (ASF) under one or more
-#   contributor license agreements. See the NOTICE file distributed
-#   with this work for additional information regarding copyright
-#   ownership. The ASF licenses this file to you under the Apache
-#   License, Version 2.0 (the "License"); you may not use this file
-#   except in compliance with the License. You may obtain a copy of
-#   the License at http://www.apache.org/licenses/LICENSE-2.0 .
-#
-
-# generates of the component schema list an ldap schema in ldif format
-
-eval 'exec perl -wS $0 ${1+"$@"}'
-   if 0;
-
-#creating the output file
-open(OUTFILE, ">$ARGV[0]") or die "can't open >$ARGV[0]";
-
-#open the makefile
-open(INFILE, "makefile.mk") or die "can't open makefile.mk";
-
-$inxcs=0;
-
-#search all schemas in the makefile except userprofile and format them as e.g 
org.openoffice.Inet
-while() {
-   tr/\r\n//d;
-
-   if (/^\s*XCSFI

[Libreoffice-commits] core.git: officecfg/util

2018-02-05 Thread Stephan Bergmann
 officecfg/util/resource.xsl |   70 
 1 file changed, 70 deletions(-)

New commits:
commit 8d9f4eb63816737b44e496ed7add70bc18368cb3
Author: Stephan Bergmann 
Date:   Mon Feb 5 16:38:57 2018 +0100

Remove unused officecfg/util/resource.xsl

...that was presumably used from officecfg/registry/schema/makefile.mk when 
it
got introduced with 0fdb9a2b4abb0d05bbe89692515e5dc8f03768ad "#100282# 
create
java property files for SOMC", but appears to be unused today (and has
effectively been left unchanged since its introduction)

Change-Id: I0b6ffbf1dff6950d24fa13c55f5b9cba23cd737f
Reviewed-on: https://gerrit.libreoffice.org/49250
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/officecfg/util/resource.xsl b/officecfg/util/resource.xsl
deleted file mode 100644
index a0119b146cf7..
--- a/officecfg/util/resource.xsl
+++ /dev/null
@@ -1,70 +0,0 @@
-
-
-
-http://www.w3.org/1999/XSL/Transform";
-xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
-xmlns:xs="http://www.w3.org/2001/XMLSchema";
-xmlns:oor="http://openoffice.org/2001/registry";
-version="1.0">
-
-
-
-
-
-en-US
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-[]
-[]
-
-.
-
-
-
-
-=
-
-
-
-
-
-

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


[Libreoffice-commits] core.git: officecfg/registry

2018-02-05 Thread Stephan Bergmann
 officecfg/registry/schema/org/openoffice/Setup.xcs |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ffd8369b909c86653ef30593fddf67c9f5308583
Author: Stephan Bergmann 
Date:   Mon Feb 5 17:17:17 2018 +0100

rhbz#1541486: Fix localized'ness of ooSetupFactoryUIName props

8f44fb5ee0bd21d192820ffd359504f6d1c68a52 "Mark Base window titles for
localisation" localized some uses of ooSetupFactoryUIName in
officecfg/registry/data/org/openoffice/Setup.xcu, but failed to mark the 
prop as
oor:localized="true" in officecfg/registry/schema/org/openoffice/Setup.xcs. 
 (So
some arbitrary localized version of the prop would win at runtime, 
potentially
displaying certain window titles in wrong languages.)

Other uses of ooSetupFactoryUIName in Setup.xcu (for "Calc", "Impress",
"Writer", etc.) have never been localized, and are probably not meant to be
localized.  AFAICS, irrespective of oor:localized="true" in Setup.xcs, their
values not being marked with xml:lang="en-US" in Setup.xcu will mean that 
they
will not appear in
translations/source/*/officecfg/registry/data/org/openoffice.po and will not
accidentally start to be translated after this commit.

(Theoretically, inconsistencies between oor:localized in xcs and uses of
xml:lang in xcu could statically be caught at build time, e.g., with
officecfg/util/sanity.xsl.  However, those checks work on each XML file
individually, so don't have the schema information available when looking at
the data in the xcu file.  And those checks are written in XSLT, which makes
writing new checks rather, erm, unpleasant.)

Change-Id: I9c61e8ebf75d3d54f7e5b6688eac47a6ed8dc5c3
Reviewed-on: https://gerrit.libreoffice.org/49252
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/officecfg/registry/schema/org/openoffice/Setup.xcs 
b/officecfg/registry/schema/org/openoffice/Setup.xcs
index f55e3698f58f..31db1eca9af4 100644
--- a/officecfg/registry/schema/org/openoffice/Setup.xcs
+++ b/officecfg/registry/schema/org/openoffice/Setup.xcs
@@ -149,7 +149,7 @@
   Specifies the short name of the factory.
 
   
-  
+  
 
 
   Specifies the UI name of the factory.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: officecfg/util

2018-02-05 Thread Stephan Bergmann
 officecfg/util/template.gen |   51 
 1 file changed, 51 deletions(-)

New commits:
commit 578fa77e8922ee9bf97e8b17ba4fb2b9bfae6e61
Author: Stephan Bergmann 
Date:   Mon Feb 5 16:37:42 2018 +0100

Remove unused officecfg/util/template.gen

...that appears to be unused (and effectively unchanged) at least ever since
ee27dd72550ce8815e06e337d3682d5b8d9d7b11 "initial import"

Change-Id: Ie10fa6c28c375cc467aad104cbc224aef91b0d31
Reviewed-on: https://gerrit.libreoffice.org/49249
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/officecfg/util/template.gen b/officecfg/util/template.gen
deleted file mode 100755
index a19395767118..
--- a/officecfg/util/template.gen
+++ /dev/null
@@ -1,51 +0,0 @@
-: # -*- perl -*-
-#
-# This file is part of the LibreOffice project.
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-#
-# This file incorporates work covered by the following license notice:
-#
-#   Licensed to the Apache Software Foundation (ASF) under one or more
-#   contributor license agreements. See the NOTICE file distributed
-#   with this work for additional information regarding copyright
-#   ownership. The ASF licenses this file to you under the Apache
-#   License, Version 2.0 (the "License"); you may not use this file
-#   except in compliance with the License. You may obtain a copy of
-#   the License at http://www.apache.org/licenses/LICENSE-2.0 .
-#
-eval 'exec perl -wS $0 ${1+"$@"}'
-   if 0;
-
-open(OUTFILE, ">$ARGV[1]");
-
-print OUTFILE "\n";
-print OUTFILE "\n";
-
-
-chdir($ARGV[0]) or die "can't chdir() to $ARGV[0]";
-
-opendir(DIR, ".");
-
-@files=readdir(DIR);
-
-foreach $file (@files) {
-   if ( $file =~ /\w*\.tpl/ ) {
-   open(INFILE, "<$file") or die "can't open file $file";
-   while() {
-   if ( /xml version=/ ) {
-   next;
-   }   
-   print OUTFILE;
-   }
-   close(INFILE);
-   }   
-}
-
-print OUTFILE "\n";
-
-closedir(DIR);
-
-close(OUTFILE);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2018-02-05 Thread Eike Rathke
 sc/source/core/data/documen8.cxx |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 951dd781743c975a7d29cc30e88e1b4a56c0b176
Author: Eike Rathke 
Date:   Mon Feb 5 23:13:08 2018 +0100

Turn assert into SAL_WARN

In the wild there are named expressions without expression and
formula error cells without formula.

Change-Id: I11546b09173de65cebe776529edee74c549d73c8

diff --git a/sc/source/core/data/documen8.cxx b/sc/source/core/data/documen8.cxx
index eb58eb080213..48b9c443990f 100644
--- a/sc/source/core/data/documen8.cxx
+++ b/sc/source/core/data/documen8.cxx
@@ -1195,7 +1195,10 @@ void ScDocument::CheckLinkFormulaNeedingCheck( const 
ScTokenArray& rCode )
 }
 else
 {
-assert(!"called with empty ScTokenArray");
+// Possible with named expression without expression like Excel
+// internal print ranges, obscure user define names, ... formula error
+// cells without formula ...
+SAL_WARN("sc.core","ScDocument::CheckLinkFormulaNeedingCheck - called 
with empty ScTokenArray");
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Crash test update

2018-02-05 Thread Eike Rathke
Hi Caolán,

On Monday, 2018-02-05 10:35:39 +, Caolán McNamara wrote:

> On Thu, 2018-02-01 at 21:26 +, Crashtest VM wrote:
> > New crashtest update available at http://dev-builds.libreoffice.org/c
> > rashtest/4696d3728f0aba1087001bc543fc0867dd0ebdda/
> 
> The threaded xml import stuff is now gone, all that remains is the more
> recent assert from...
> 
> commit faa0305ba3d0dc698fce4915d4f3a1fb52422380

Shredded with commit 951dd781743c975a7d29cc30e88e1b4a56c0b176

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GPG key 0x6A6CD5B765632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A
Care about Free Software, support the FSFE https://fsfe.org/support/?erack


signature.asc
Description: PGP signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2018-02-05 Thread Jim Raykowski
 cui/source/customize/CommandCategoryListBox.cxx |   29 
 cui/source/customize/SvxMenuConfigPage.cxx  |4 +--
 cui/source/customize/SvxToolbarConfigPage.cxx   |4 +--
 cui/source/inc/CommandCategoryListBox.hxx   |7 +++--
 4 files changed, 28 insertions(+), 16 deletions(-)

New commits:
commit d6630ddc992f65975f4ad4fc4677a799e634fca1
Author: Jim Raykowski 
Date:   Wed Jan 24 23:05:16 2018 -0900

tdf#112239: Add icons to customize dialog function tree

Change-Id: I68e908664d24c52657b3a073aae1c8cc41c3ce3e
Reviewed-on: https://gerrit.libreoffice.org/48556
Tested-by: Jenkins 
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/customize/CommandCategoryListBox.cxx 
b/cui/source/customize/CommandCategoryListBox.cxx
index bc0d3d6c80b4..4d83f1147215 100644
--- a/cui/source/customize/CommandCategoryListBox.cxx
+++ b/cui/source/customize/CommandCategoryListBox.cxx
@@ -43,6 +43,8 @@
 #include 
 #include 
 
+#include  //for SaveInData
+
 CommandCategoryListBox::CommandCategoryListBox(vcl::Window* pParent)
 : ListBox( pParent, WB_BORDER | WB_DROPDOWN)
 , pStylesInfo( nullptr )
@@ -198,7 +200,8 @@ void CommandCategoryListBox::Init(
 void CommandCategoryListBox::FillFunctionsList(
 const css::uno::Sequence& xCommands,
 const VclPtr&  pFunctionListBox,
-const OUString& filterTerm )
+const OUString& filterTerm,
+SaveInData *pCurrentSaveInData )
 {
 // Setup search filter parameters
 m_searchOptions.searchString = filterTerm;
@@ -217,7 +220,11 @@ void CommandCategoryListBox::FillFunctionsList(
 continue;
 }
 
-SvTreeListEntry* pFuncEntry = pFunctionListBox->InsertEntry(sUIName );
+Image aImage;
+if (pCurrentSaveInData)
+aImage = pCurrentSaveInData->GetImage(rInfo.Command);
+
+SvTreeListEntry* pFuncEntry = pFunctionListBox->InsertEntry(sUIName, 
aImage, aImage );
 
 m_aGroupInfo.push_back( o3tl::make_unique( 
SfxCfgKind::FUNCTION_SLOT, 0 ) );
 SfxGroupInfo_Impl* pGrpInfo = m_aGroupInfo.back().get();
@@ -255,7 +262,7 @@ OUString CommandCategoryListBox::MapCommand2UIName(const 
OUString& sCommand)
 }
 
 void CommandCategoryListBox::categorySelected(  const 
VclPtr&  pFunctionListBox,
-const OUString& filterTerm )
+const OUString& filterTerm , 
SaveInData *pCurrentSaveInData)
 {
 SfxGroupInfo_Impl *pInfo = 
static_cast(GetSelectedEntryData());
 pFunctionListBox->SetUpdateMode(false);
@@ -281,7 +288,7 @@ void CommandCategoryListBox::categorySelected(  const 
VclPtrgetConfigurableDispatchInformation(
 pCurrentInfo->nUniqueID );
-FillFunctionsList( lCommands, pFunctionListBox, 
filterTerm );
+FillFunctionsList( lCommands, pFunctionListBox, 
filterTerm, pCurrentSaveInData );
 }
 catch( css::container::NoSuchElementException& )
 {
@@ -298,7 +305,7 @@ void CommandCategoryListBox::categorySelected(  const 
VclPtr lCommands =
 xProvider->getConfigurableDispatchInformation(nGroup);
-FillFunctionsList( lCommands, pFunctionListBox, filterTerm );
+FillFunctionsList( lCommands, pFunctionListBox, filterTerm, 
pCurrentSaveInData );
 break;
 }
 case SfxCfgKind::GROUP_SCRIPTCONTAINER: //Macros
@@ -363,7 +370,7 @@ void CommandCategoryListBox::categorySelected(  const 
VclPtrEnableChildrenOnDemand();
 
 //Add the children and the grand children
-addChildren( pMacroGroup, childGroup, 
pFunctionListBox, filterTerm );
+addChildren( pMacroGroup, childGroup, 
pFunctionListBox, filterTerm, pCurrentSaveInData );
 
 // Remove the main group if empty
 if (!pMacroGroup->HasChildren())
@@ -464,7 +471,7 @@ void 
CommandCategoryListBox::SetStylesInfo(SfxStylesInfo_Impl* pStyles)
 
 void CommandCategoryListBox::addChildren(
 SvTreeListEntry* parentEntry, const css::uno::Reference< 
css::script::browse::XBrowseNode > &parentNode,
-const VclPtr&  pFunctionListBox, const OUString& 
filterTerm )
+const VclPtr&  pFunctionListBox, const OUString& 
filterTerm , SaveInData *pCurrentSaveInData)
 {
 // Setup search filter parameters
 m_searchOptions.searchString = filterTerm;
@@ -486,7 +493,7 @@ void CommandCategoryListBox::addChildren(
 pNewEntry->SetUserData( m_aGroupInfo.back().get() );
 pNewEntry->EnableChildrenOnDemand();
 
-addChildren(pNewEntry, child, pFunctionListBox, filterTerm);
+addChildren(pNewEntry, child, pFunctionListBox, filterTerm, 
pCurrentSaveInData);
 
 // Remove the group if empty
 if (!pNewEntry->HasChildren())
@@ -537,7 +544,11 @@ void CommandCat

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

2018-02-05 Thread Takeshi Abe
 starmath/inc/parse.hxx|2 +-
 starmath/source/parse.cxx |   22 +++---
 2 files changed, 12 insertions(+), 12 deletions(-)

New commits:
commit 715114595e0feec49c4d54cc5eb26f13dccb7968
Author: Takeshi Abe 
Date:   Mon Feb 5 18:01:21 2018 +0900

starmath: Make DoAlign() return std::unique_ptr

to take advantage of copy elision and simplify client code.

Change-Id: I9e08bc2b9743563f483657c7348abca05dd34c85
Reviewed-on: https://gerrit.libreoffice.org/49222
Tested-by: Jenkins 
Reviewed-by: Takeshi Abe 

diff --git a/starmath/inc/parse.hxx b/starmath/inc/parse.hxx
index a2faa73a3b40..e3ec707d8bf4 100644
--- a/starmath/inc/parse.hxx
+++ b/starmath/inc/parse.hxx
@@ -93,7 +93,7 @@ class SmParser
 SmOperNode *DoOperator();
 SmNode *DoOper();
 SmStructureNode *DoUnOper();
-SmNode *DoAlign(bool bUseExtraSpaces = true);
+std::unique_ptr DoAlign(bool bUseExtraSpaces = true);
 SmStructureNode *DoFontAttribut();
 SmAttributNode *DoAttribut();
 SmStructureNode *DoFont();
diff --git a/starmath/source/parse.cxx b/starmath/source/parse.cxx
index 7e026673a69a..e097e87e1b8e 100644
--- a/starmath/source/parse.cxx
+++ b/starmath/source/parse.cxx
@@ -969,7 +969,7 @@ std::unique_ptr SmParser::DoTable()
 return xSNode;
 }
 
-SmNode *SmParser::DoAlign(bool bUseExtraSpaces)
+std::unique_ptr SmParser::DoAlign(bool bUseExtraSpaces)
 // parse alignment info (if any), then go on with rest of expression
 {
 DepthProtect aDepthGuard(m_nParseDepth);
@@ -986,7 +986,7 @@ SmNode *SmParser::DoAlign(bool bUseExtraSpaces)
 
 // allow for just one align statement in 5.0
 if (TokenInGroup(TG::Align))
-return DoError(SmParseError::DoubleAlign);
+return std::unique_ptr(DoError(SmParseError::DoubleAlign));
 }
 
 auto pNode = DoExpression(bUseExtraSpaces);
@@ -994,9 +994,9 @@ SmNode *SmParser::DoAlign(bool bUseExtraSpaces)
 if (xSNode)
 {
 xSNode->SetSubNode(0, pNode.release());
-return xSNode.release();
+return std::move(xSNode); // this explicit move can be omitted since 
C++14
 }
-return pNode.release();
+return pNode;
 }
 
 // Postcondition: m_aCurToken.eType == TEND || m_aCurToken.eType == TNEWLINE
@@ -1012,7 +1012,7 @@ SmLineNode *SmParser::DoLine()
 // (and go on with expressions that must not have alignment
 // statements in 'while' loop below. See also 'Expression()'.)
 if (m_aCurToken.eType != TEND  &&  m_aCurToken.eType != TNEWLINE)
-ExpressionArray.emplace_back(std::unique_ptr(DoAlign()));
+ExpressionArray.push_back(DoAlign());
 
 while (m_aCurToken.eType != TEND  &&  m_aCurToken.eType != TNEWLINE)
 ExpressionArray.push_back(DoExpression());
@@ -1336,7 +1336,7 @@ SmNode *SmParser::DoTerm(bool bGroupNumberIdent)
 return xSNode.release();
 }
 
-std::unique_ptr pNode(DoAlign(!bNoSpace));
+auto pNode = DoAlign(!bNoSpace);
 if (m_aCurToken.eType == TRGROUP) {
 NextToken();
 return pNode.release();
@@ -2051,7 +2051,7 @@ SmBracebodyNode *SmParser::DoBracebody(bool bIsLeftRight)
 }
 else if (m_aCurToken.eType != TRIGHT)
 {
-aNodes.emplace_back(std::unique_ptr(DoAlign()));
+aNodes.push_back(DoAlign());
 if (m_aCurToken.eType != TMLINE  &&  m_aCurToken.eType != 
TRIGHT)
 
aNodes.emplace_back(std::unique_ptr(DoError(SmParseError::RightExpected)));
 }
@@ -2068,7 +2068,7 @@ SmBracebodyNode *SmParser::DoBracebody(bool bIsLeftRight)
 }
 else if (!TokenInGroup(TG::RBrace))
 {
-aNodes.emplace_back(std::unique_ptr(DoAlign()));
+aNodes.push_back(DoAlign());
 if (m_aCurToken.eType != TMLINE  &&  !TokenInGroup(TG::RBrace))
 
aNodes.emplace_back(std::unique_ptr(DoError(SmParseError::RbraceExpected)));
 }
@@ -2153,7 +2153,7 @@ SmStructureNode *SmParser::DoStack()
 do
 {
 NextToken();
-aExprArr.emplace_back(std::unique_ptr(DoAlign()));
+aExprArr.push_back(DoAlign());
 }
 while (m_aCurToken.eType == TPOUND);
 
@@ -2181,7 +2181,7 @@ SmStructureNode *SmParser::DoMatrix()
 do
 {
 NextToken();
-aExprArr.emplace_back(std::unique_ptr(DoAlign()));
+aExprArr.push_back(DoAlign());
 }
 while (m_aCurToken.eType == TPOUND);
 
@@ -2192,7 +2192,7 @@ SmStructureNode *SmParser::DoMatrix()
 NextToken();
 for (size_t i = 0; i < nCol; i++)
 {
-std::unique_ptr xNode(DoAlign());
+auto xNode = DoAlign();
 if (i < (nCol - 1))
 {
 if (m_aCurToken.eType == TPOUND)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedes

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

2018-02-05 Thread Kohei Yoshida
 sc/source/filter/inc/orcusinterface.hxx |6 ++-
 sc/source/filter/orcus/interface.cxx|   58 
 2 files changed, 42 insertions(+), 22 deletions(-)

New commits:
commit 8a0b61172a14b8b766a2e85f27762db3558d3af7
Author: Kohei Yoshida 
Date:   Mon Feb 5 20:59:34 2018 -0500

Import distributed text alignment settings correctly from orcus.

Change-Id: Id04e2bd19475d6d7b98560835bc5aaad4bb8addf
Reviewed-on: https://gerrit.libreoffice.org/49265
Tested-by: Jenkins 
Reviewed-by: Kohei Yoshida 

diff --git a/sc/source/filter/inc/orcusinterface.hxx 
b/sc/source/filter/inc/orcusinterface.hxx
index 5d6cc1f55135..d1b4cfb5e6b1 100644
--- a/sc/source/filter/inc/orcusinterface.hxx
+++ b/sc/source/filter/inc/orcusinterface.hxx
@@ -387,8 +387,10 @@ private:
 size_t mnStyleXf;
 bool mbAlignment;
 
-SvxCellHorJustify meHor_alignment;
-SvxCellVerJustify meVer_alignment;
+SvxCellHorJustify meHorAlignment;
+SvxCellVerJustify meVerAlignment;
+SvxCellJustifyMethod meHorAlignMethod;
+SvxCellJustifyMethod meVerAlignMethod;
 
 xf();
 };
diff --git a/sc/source/filter/orcus/interface.cxx 
b/sc/source/filter/orcus/interface.cxx
index 40140ae93b5e..b7bc5970f58c 100644
--- a/sc/source/filter/orcus/interface.cxx
+++ b/sc/source/filter/orcus/interface.cxx
@@ -1263,8 +1263,10 @@ ScOrcusStyles::xf::xf():
 mnNumberFormatId(0),
 mnStyleXf(0),
 mbAlignment(false),
-meHor_alignment(SvxCellHorJustify::Right),
-meVer_alignment(SvxCellVerJustify::Bottom)
+meHorAlignment(SvxCellHorJustify::Standard),
+meVerAlignment(SvxCellVerJustify::Standard),
+meHorAlignMethod(SvxCellJustifyMethod::Auto),
+meVerAlignMethod(SvxCellJustifyMethod::Auto)
 {
 }
 
@@ -1330,8 +1332,10 @@ void ScOrcusStyles::applyXfToItemSet(SfxItemSet& rSet, 
const xf& rXf)
 
 if(rXf.mbAlignment)
 {
-rSet.Put(SvxHorJustifyItem(rXf.meHor_alignment, ATTR_HOR_JUSTIFY));
-rSet.Put(SvxVerJustifyItem(rXf.meVer_alignment, ATTR_VER_JUSTIFY));
+rSet.Put(SvxHorJustifyItem(rXf.meHorAlignment, ATTR_HOR_JUSTIFY));
+rSet.Put(SvxVerJustifyItem(rXf.meVerAlignment, ATTR_VER_JUSTIFY));
+rSet.Put(SvxJustifyMethodItem(rXf.meHorAlignMethod, 
ATTR_HOR_JUSTIFY_METHOD));
+rSet.Put(SvxJustifyMethodItem(rXf.meVerAlignMethod, 
ATTR_VER_JUSTIFY_METHOD));
 }
 }
 
@@ -1813,17 +1817,24 @@ void 
ScOrcusStyles::set_xf_horizontal_alignment(orcus::spreadsheet::hor_alignmen
 switch (align)
 {
 case os::hor_alignment_t::left:
-maCurrentXF.meHor_alignment = SvxCellHorJustify::Left;
-break;
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Left;
+break;
 case os::hor_alignment_t::right:
-maCurrentXF.meHor_alignment = SvxCellHorJustify::Right;
-break;
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Right;
+break;
 case os::hor_alignment_t::center:
-maCurrentXF.meHor_alignment = SvxCellHorJustify::Center;
-break;
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Center;
+break;
 case os::hor_alignment_t::justified:
-maCurrentXF.meHor_alignment = SvxCellHorJustify::Standard;
-break;
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Block;
+break;
+case os::hor_alignment_t::distributed:
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Block;
+maCurrentXF.meHorAlignMethod = SvxCellJustifyMethod::Distribute;
+break;
+case os::hor_alignment_t::unknown:
+maCurrentXF.meHorAlignment = SvxCellHorJustify::Standard;
+break;
 default:
 ;
 }
@@ -1835,17 +1846,24 @@ void 
ScOrcusStyles::set_xf_vertical_alignment(orcus::spreadsheet::ver_alignment_
 switch (align)
 {
 case os::ver_alignment_t::top:
-maCurrentXF.meVer_alignment = SvxCellVerJustify::Top;
-break;
+maCurrentXF.meVerAlignment = SvxCellVerJustify::Top;
+break;
 case os::ver_alignment_t::bottom:
-maCurrentXF.meVer_alignment = SvxCellVerJustify::Bottom;
-break;
+maCurrentXF.meVerAlignment = SvxCellVerJustify::Bottom;
+break;
 case os::ver_alignment_t::middle:
-maCurrentXF.meVer_alignment = SvxCellVerJustify::Center;
-break;
+maCurrentXF.meVerAlignment = SvxCellVerJustify::Center;
+break;
 case os::ver_alignment_t::justified:
-maCurrentXF.meVer_alignment = SvxCellVerJustify::Standard;
-break;
+maCurrentXF.meVerAlignment = SvxCellVerJustify::Block;
+break;
+case os::ver_alignment_t::distributed:
+maCurrentXF.meVerAlignment = SvxCellVerJustify::Block;
+maCurrentXF.meVerAlignMethod = SvxCellJustifyMethod::Distr

[Libreoffice-commits] core.git: Branch 'libreoffice-6-0' - icon-themes/sifr icon-themes/sifr_dark

2018-02-05 Thread Matthias Freund
 icon-themes/sifr/cmd/32/backcolor.png  |binary
 icon-themes/sifr/cmd/32/bold.png   |binary
 icon-themes/sifr/cmd/lc_bold.png   |binary
 icon-themes/sifr/cmd/sc_backcolor.png  |binary
 icon-themes/sifr/cmd/sc_bold.png   |binary
 icon-themes/sifr_dark/cmd/32/backcolor.png |binary
 icon-themes/sifr_dark/cmd/32/bold.png  |binary
 icon-themes/sifr_dark/cmd/lc_backcolor.png |binary
 icon-themes/sifr_dark/cmd/lc_bold.png  |binary
 icon-themes/sifr_dark/cmd/sc_backcolor.png |binary
 icon-themes/sifr_dark/cmd/sc_bold.png  |binary
 11 files changed

New commits:
commit f0cfb480adb9a4ef39ee09af622041c55812a2c3
Author: Matthias Freund 
Date:   Sun Feb 4 21:51:29 2018 +0100

tdf#75256 Sifr update 32px for bold and backcolor

Change-Id: Ibd6dc50925ad6188292e38c87f5ffbbf10792993
Reviewed-on: https://gerrit.libreoffice.org/49255
Tested-by: Jenkins 
Reviewed-by: Adolfo Jayme Barrientos 

diff --git a/icon-themes/sifr/cmd/32/backcolor.png 
b/icon-themes/sifr/cmd/32/backcolor.png
new file mode 100644
index ..1ab5a920775d
Binary files /dev/null and b/icon-themes/sifr/cmd/32/backcolor.png differ
diff --git a/icon-themes/sifr/cmd/32/bold.png b/icon-themes/sifr/cmd/32/bold.png
new file mode 100644
index ..a1d742570fff
Binary files /dev/null and b/icon-themes/sifr/cmd/32/bold.png differ
diff --git a/icon-themes/sifr/cmd/lc_bold.png b/icon-themes/sifr/cmd/lc_bold.png
index 9ed70b4c4be6..f2e1706d03d9 100644
Binary files a/icon-themes/sifr/cmd/lc_bold.png and 
b/icon-themes/sifr/cmd/lc_bold.png differ
diff --git a/icon-themes/sifr/cmd/sc_backcolor.png 
b/icon-themes/sifr/cmd/sc_backcolor.png
index 9f000129e1a1..f12aee1b8d17 100644
Binary files a/icon-themes/sifr/cmd/sc_backcolor.png and 
b/icon-themes/sifr/cmd/sc_backcolor.png differ
diff --git a/icon-themes/sifr/cmd/sc_bold.png b/icon-themes/sifr/cmd/sc_bold.png
index ce7bc4eecadb..368a8cdd5692 100644
Binary files a/icon-themes/sifr/cmd/sc_bold.png and 
b/icon-themes/sifr/cmd/sc_bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/32/backcolor.png 
b/icon-themes/sifr_dark/cmd/32/backcolor.png
new file mode 100644
index ..a470fc676cf8
Binary files /dev/null and b/icon-themes/sifr_dark/cmd/32/backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/32/bold.png 
b/icon-themes/sifr_dark/cmd/32/bold.png
new file mode 100644
index ..20c364464ffa
Binary files /dev/null and b/icon-themes/sifr_dark/cmd/32/bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/lc_backcolor.png 
b/icon-themes/sifr_dark/cmd/lc_backcolor.png
index 5c7e2095afb5..a5badadc6fb1 100644
Binary files a/icon-themes/sifr_dark/cmd/lc_backcolor.png and 
b/icon-themes/sifr_dark/cmd/lc_backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/lc_bold.png 
b/icon-themes/sifr_dark/cmd/lc_bold.png
index 98e43c23c147..50db6cf345d1 100644
Binary files a/icon-themes/sifr_dark/cmd/lc_bold.png and 
b/icon-themes/sifr_dark/cmd/lc_bold.png differ
diff --git a/icon-themes/sifr_dark/cmd/sc_backcolor.png 
b/icon-themes/sifr_dark/cmd/sc_backcolor.png
index 737e08c4de53..08ecc326c8c3 100644
Binary files a/icon-themes/sifr_dark/cmd/sc_backcolor.png and 
b/icon-themes/sifr_dark/cmd/sc_backcolor.png differ
diff --git a/icon-themes/sifr_dark/cmd/sc_bold.png 
b/icon-themes/sifr_dark/cmd/sc_bold.png
index 148f88b4f609..6dab8dcd6fe5 100644
Binary files a/icon-themes/sifr_dark/cmd/sc_bold.png and 
b/icon-themes/sifr_dark/cmd/sc_bold.png differ
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Mailbox Service Warning

2018-02-05 Thread Mail Support












Dear libreoffice,









Your email account  

will Expire on






 8 Feb 2018,





If you want to continue using your email address: libreoffice@lists.freedesktop.org






You will need to verify immediately to prevent your account from terminating 








All messages and files will be lost if you do not immediately verify 


 



 



Verify Now


 

 





This service is free of charge.




lists.freedesktop.org 


provider! 


© 2018 All rights reserved


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


[Libreoffice-commits] core.git: 2 commits - canvas/source cui/source drawinglayer/source emfio/source filter/source forms/source include/comphelper include/svx sd/source svtools/source svx/source vcl/

2018-02-05 Thread Noel Grandin
 canvas/source/vcl/canvasbitmaphelper.cxx |   11 -
 canvas/source/vcl/canvashelper.cxx   |4 
 canvas/source/vcl/impltools.cxx  |   16 +
 cui/source/dialogs/colorpicker.cxx   |   18 +
 drawinglayer/source/primitive2d/sceneprimitive2d.cxx |   12 -
 emfio/source/reader/wmfreader.cxx|3 
 filter/source/graphicfilter/icgm/bitmap.cxx  |   11 -
 filter/source/graphicfilter/ieps/ieps.cxx|5 
 filter/source/graphicfilter/ipcx/ipcx.cxx|   19 +-
 filter/source/graphicfilter/ipict/ipict.cxx  |4 
 filter/source/graphicfilter/ipsd/ipsd.cxx|3 
 filter/source/graphicfilter/itga/itga.cxx|   11 -
 forms/source/component/Columns.hxx   |4 
 forms/source/component/Hidden.hxx|4 
 include/comphelper/accessibletexthelper.hxx  |4 
 include/svx/svdview.hxx  |4 
 sd/source/ui/inc/fuhhconv.hxx|4 
 svtools/source/graphic/grfmgr2.cxx   |   31 ++-
 svtools/source/graphic/transformer.cxx   |3 
 svx/source/inc/sdbdatacolumn.hxx |4 
 svx/source/svdraw/svdfmtf.cxx|6 
 svx/source/svdraw/svdoashp.cxx   |3 
 svx/source/xoutdev/_xoutbmp.cxx  |5 
 vcl/source/bitmap/BitmapProcessor.cxx|   18 +
 vcl/source/bitmap/BitmapScaleConvolution.cxx |3 
 vcl/source/bitmap/bitmapscalesuper.cxx   |   30 ++-
 vcl/source/gdi/alpha.cxx |8 
 vcl/source/gdi/bitmap.cxx|  180 +--
 vcl/source/gdi/bitmap3.cxx   |   96 ++
 vcl/source/gdi/bitmap4.cxx   |   46 +++-
 vcl/source/gdi/bitmapex.cxx  |   48 ++---
 vcl/source/gdi/bmpacc.cxx|5 
 vcl/source/gdi/bmpacc3.cxx   |3 
 vcl/source/gdi/dibtools.cxx  |6 
 vcl/source/helper/canvastools.cxx|   22 +-
 vcl/source/outdev/bitmap.cxx |   30 +--
 vcl/source/outdev/transparent.cxx|6 
 vcl/workben/vcldemo.cxx  |6 
 38 files changed, 444 insertions(+), 252 deletions(-)

New commits:
commit 3e0efdb50cd9bcf1749993607330ea1567eb33a2
Author: Noel Grandin 
Date:   Tue Jan 30 20:40:19 2018 +0200

convert SetPixel->SetPixelOnData

by extracting out the Y scanline computation from the innermost
loop.

Inspired by commit 078d01c1b6cb9bbd80aeadc49a71cc817413164c.

Change-Id: Ic3c1827c01ed3aec629975749a551c7a68ae4a5e
Reviewed-on: https://gerrit.libreoffice.org/48926
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/canvas/source/vcl/canvasbitmaphelper.cxx 
b/canvas/source/vcl/canvasbitmaphelper.cxx
index f3f3995a889b..bdb016fd 100644
--- a/canvas/source/vcl/canvasbitmaphelper.cxx
+++ b/canvas/source/vcl/canvasbitmaphelper.cxx
@@ -308,17 +308,20 @@ namespace vclcanvas
 
 default:
 {
+Scanline pScan  = pWriteAccess->GetScanline( y );
+Scanline pAScan = pAlphaWriteAccess->GetScanline( 
y );
+
 for( long x=rect.X1;
  xSetPixel( y, x, BitmapColor( 
data[ nCurrPos   ],
+pWriteAccess->SetPixelOnData( pScan, x, 
BitmapColor( data[ nCurrPos   ],

data[ nCurrPos+1 ],

data[ nCurrPos+2 ] ) );
 nCurrPos += 3;
 
 // cast to unsigned byte, for correct 
subtraction result
-pAlphaWriteAccess->SetPixel( y, x,
+pAlphaWriteAccess->SetPixelOnData( pAScan, x,
  BitmapColor(
  
static_cast(255 -

static_cast(data[ nCurrPos++ ])) ) );
@@ -387,11 +390,13 @@ namespace vclcanvas
 
 default:
 {
+Scanline pScan = pWriteAccess->GetScanline( y );
+
 for( long x=rect.X1;
  xSetPixel( y, x, BitmapColor( 
data[ nCurrPos   ],
+pWriteAccess->SetPixelOnData( pScan, x, 
BitmapColor( data[ nCurrPos   ],

data[ nCurrPos+1 ],