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

2017-01-06 Thread Henry Castro
 loleaflet/dist/loleaflet-help.html|1 +
 loleaflet/src/map/handler/Map.Keyboard.js |6 ++
 2 files changed, 7 insertions(+)

New commits:
commit 73cf6f71fbe1ccb1946958b06e8deab16176a82c
Author: Henry Castro 
Date:   Fri Jan 6 23:11:48 2017 -0400

tdf#103779: Help menu accessibility, part 2

Add a keyboard shortcut ctrl + h to display the
Keyboard shortcuts help

diff --git a/loleaflet/dist/loleaflet-help.html 
b/loleaflet/dist/loleaflet-help.html
index 2184635..bfed701 100644
--- a/loleaflet/dist/loleaflet-help.html
+++ b/loleaflet/dist/loleaflet-help.html
@@ -31,6 +31,7 @@
  Cut Ctrl + 
X 
  Paste as unformatted text Ctrl + Alt + Shift + V 
  Print (Download as PDF) Ctrl + P 
+ Display the Keyboard shortcuts help 
Ctrl + H 
 
 
 
diff --git a/loleaflet/src/map/handler/Map.Keyboard.js 
b/loleaflet/src/map/handler/Map.Keyboard.js
index 84c72d3..77302a8 100644
--- a/loleaflet/src/map/handler/Map.Keyboard.js
+++ b/loleaflet/src/map/handler/Map.Keyboard.js
@@ -398,6 +398,12 @@ L.Map.Keyboard = L.Handler.extend({
e.originalEvent.preventDefault();
}
 
+   if (e.originalEvent.ctrlKey && (e.originalEvent.key === 'h' || 
e.originalEvent.key === 'H')) {
+   map.showLOKeyboardHelp();
+   e.originalEvent.preventDefault();
+   return true;
+   }
+
if (e.originalEvent.ctrlKey && (e.originalEvent.key === 'z' || 
e.originalEvent.key === 'Z')) {
this._map._socket.sendMessage('uno .uno:Undo');
e.originalEvent.preventDefault();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Henry Castro
 include/vcl/outdev.hxx  |1 +
 svtools/source/control/ctrltool.cxx |9 -
 vcl/source/outdev/font.cxx  |5 +
 3 files changed, 14 insertions(+), 1 deletion(-)

New commits:
commit 0ad4b2570f06aa014f8446757931afa1f2305458
Author: Henry Castro 
Date:   Fri Dec 16 23:23:31 2016 -0400

lok: refill font list if empty

After preinit stage the font list is empty
when creating documents.

Reviewed-on: https://gerrit.libreoffice.org/32108
Reviewed-by: Michael Meeks 
Tested-by: Michael Meeks 

Conflicts:
sd/source/ui/docshell/docshel4.cxx
svtools/source/control/ctrltool.cxx

Change-Id: Ia6938d521626f4b4022f0f993e77c017e4932415
Reviewed-on: https://gerrit.libreoffice.org/32796
Tested-by: Jenkins 
Reviewed-by: Henry Castro 

diff --git a/include/vcl/outdev.hxx b/include/vcl/outdev.hxx
index a2130e0..2e7a5ae 100644
--- a/include/vcl/outdev.hxx
+++ b/include/vcl/outdev.hxx
@@ -1211,6 +1211,7 @@ public:
 int GetDevFontSizeCount( const vcl::Font& ) const;
 
 boolAddTempDevFont( const OUString& rFileURL, 
const OUString& rFontName );
+voidRefreshFontData( const bool bNewFontLists );
 
 FontMetric  GetFontMetric() const;
 FontMetric  GetFontMetric( const vcl::Font& rFont ) const;
diff --git a/svtools/source/control/ctrltool.cxx 
b/svtools/source/control/ctrltool.cxx
index fdbd04b..4a31b22 100644
--- a/svtools/source/control/ctrltool.cxx
+++ b/svtools/source/control/ctrltool.cxx
@@ -32,6 +32,7 @@
 #include 
 #include 
 #include 
+#include 
 
 // Standard Fontgroessen fuer scalierbare Fonts
 const sal_IntPtr FontList::aStdSizeAry[] =
@@ -248,7 +249,13 @@ void FontList::ImplInsertFonts(OutputDevice* pDevice, bool 
bInsertData)
 nType = FontListFontNameType::PRINTER;
 
 // inquire all fonts from the device
-int const n = pDevice->GetDevFontCount();
+int n = pDevice->GetDevFontCount();
+if (n == 0 && comphelper::LibreOfficeKit::isActive())
+{
+pDevice->RefreshFontData(true);
+n = pDevice->GetDevFontCount();
+}
+
 for (int i = 0; i < n; ++i)
 {
 FontMetric aFontMetric = pDevice->GetDevFont( i );
diff --git a/vcl/source/outdev/font.cxx b/vcl/source/outdev/font.cxx
index eba3374..572ac67 100644
--- a/vcl/source/outdev/font.cxx
+++ b/vcl/source/outdev/font.cxx
@@ -548,6 +548,11 @@ void OutputDevice::ImplClearFontData( const bool 
bNewFontLists )
 }
 }
 
+void OutputDevice::RefreshFontData( const bool bNewFontLists )
+{
+ImplRefreshFontData( bNewFontLists );
+}
+
 void OutputDevice::ImplRefreshFontData( const bool bNewFontLists )
 {
 //if ( GetOutDevType() == OUTDEV_PRINTER || mpPDFWriter )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 3 commits - basegfx/source basegfx/test include/basegfx sw/source

2017-01-06 Thread Caolán McNamara
 basegfx/source/color/bcolortools.cxx  |   11 ---
 basegfx/test/basegfx2d.cxx|   13 -
 include/basegfx/color/bcolortools.hxx |4 
 sw/source/core/draw/dcontact.cxx  |6 +++---
 4 files changed, 3 insertions(+), 31 deletions(-)

New commits:
commit 6af9609284e33c9a8501cbda9ef4c82f30a16b41
Author: Caolán McNamara 
Date:   Fri Jan 6 20:31:49 2017 +

coverity#1398369 Unchecked dynamic_cast

Change-Id: I33b5a2343eb7701a478c8d039d818e679471d4c0

diff --git a/sw/source/core/draw/dcontact.cxx b/sw/source/core/draw/dcontact.cxx
index 06e9f66..c0755c0 100644
--- a/sw/source/core/draw/dcontact.cxx
+++ b/sw/source/core/draw/dcontact.cxx
@@ -587,7 +587,7 @@ void SwFlyDrawContact::SwClientNotify(const SwModify& rMod, 
const SfxHint& rHint
 // This also needs to work when no layout exists. Thus, for
 // FlyFrames an alternative method is used now in that case.
 auto pFormat(dynamic_cast(&rMod));
-if(pFormat->Which() == RES_FLYFRMFMT && 
!pFormat->getIDocumentLayoutAccess().GetCurrentViewShell())
+if (pFormat && pFormat->Which() == RES_FLYFRMFMT && 
!pFormat->getIDocumentLayoutAccess().GetCurrentViewShell())
 pGetZOrdnerHint->m_rnZOrder = GetMaster()->GetOrdNum();
 }
 }
@@ -598,9 +598,9 @@ bool CheckControlLayer( const SdrObject *pObj )
 {
 if ( SdrInventor::FmForm == pObj->GetObjInventor() )
 return true;
-if (const SdrObjList *pObjLst = dynamic_cast(pObj))
+if (const SdrObjGroup *pObjGroup = dynamic_cast(pObj))
 {
-const SdrObjList *pLst = pObjLst->GetSubList();
+const SdrObjList *pLst = pObjGroup->GetSubList();
 for ( size_t i = 0; i < pLst->GetObjCount(); ++i )
 {
 if ( ::CheckControlLayer( pLst->GetObj( i ) ) )
commit fb92ec82caa96f2ddf4ab4bca8c51503ae9fba82
Author: Caolán McNamara 
Date:   Fri Jan 6 20:28:49 2017 +

dynamic cast followed by static cast

Change-Id: I425692130bd7ef6b8a9ebce3592c1976fa44dafc

diff --git a/sw/source/core/draw/dcontact.cxx b/sw/source/core/draw/dcontact.cxx
index 977841b..06e9f66 100644
--- a/sw/source/core/draw/dcontact.cxx
+++ b/sw/source/core/draw/dcontact.cxx
@@ -598,9 +598,9 @@ bool CheckControlLayer( const SdrObject *pObj )
 {
 if ( SdrInventor::FmForm == pObj->GetObjInventor() )
 return true;
-if ( dynamic_cast( pObj) !=  nullptr )
+if (const SdrObjList *pObjLst = dynamic_cast(pObj))
 {
-const SdrObjList *pLst = static_cast(pObj)->GetSubList();
+const SdrObjList *pLst = pObjLst->GetSubList();
 for ( size_t i = 0; i < pLst->GetObjCount(); ++i )
 {
 if ( ::CheckControlLayer( pLst->GetObj( i ) ) )
commit 9e7ff6a7e4415ce18b7a68c857bdb86f142a8e5d
Author: Caolán McNamara 
Date:   Fri Jan 6 20:36:56 2017 +

coverity#1398371 Useless call

Change-Id: I3614bb162cec889ee102e0284b6a3a33a98df470

diff --git a/basegfx/source/color/bcolortools.cxx 
b/basegfx/source/color/bcolortools.cxx
index 7e139ac..095ea50 100644
--- a/basegfx/source/color/bcolortools.cxx
+++ b/basegfx/source/color/bcolortools.cxx
@@ -189,17 +189,6 @@ namespace basegfx { namespace tools
 }
 }
 
-BColor rgb2ciexyz( const BColor& rRGBColor )
-{
-// from Poynton color faq, and SMPTE RP 177-1993, Derivation
-// of Basic Television Color Equations
-const double r=rRGBColor.getRed(), g=rRGBColor.getGreen(), 
b=rRGBColor.getBlue();
-return BColor(
-0.412453*r + 0.35758*g  + 0.180423*b,
-0.212671*r + 0.71516*g  + 0.072169*b,
-0.019334*r + 0.119193*g + 0.950227*b);
-}
-
 } } // end of namespace basegfx
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/basegfx/test/basegfx2d.cxx b/basegfx/test/basegfx2d.cxx
index 502f784..b0b27ef 100644
--- a/basegfx/test/basegfx2d.cxx
+++ b/basegfx/test/basegfx2d.cxx
@@ -1211,18 +1211,6 @@ public:
tools::rgb2hsv(BColor(.5,.25,.25)) == 
BColor(0,.5,.5));
 }
 
-void ciexyzTest()
-{
-tools::rgb2ciexyz(maWhite);
-tools::rgb2ciexyz(maBlack);
-tools::rgb2ciexyz(maRed);
-tools::rgb2ciexyz(maGreen);
-tools::rgb2ciexyz(maBlue);
-tools::rgb2ciexyz(maYellow);
-tools::rgb2ciexyz(maMagenta);
-tools::rgb2ciexyz(maCyan);
-}
-
 // Change the following lines only, if you add, remove or rename
 // member functions of the current class,
 // because these macros are need by auto register mechanism.
@@ -1230,7 +1218,6 @@ public:
 CPPUNIT_TEST_SUITE(bcolor);
 CPPUNIT_TEST(hslTest);
 CPPUNIT_TEST(hsvTest);
-CPPUNIT_TEST(ciexyzTest);
 CPPUNIT_TEST_SUITE_END();
 }; // class b2dvector
 
diff --git a/include/basegfx/color/bcolortools.hxx 
b/include/basegfx/color/bcolortools.hxx
index 2f4222f..64cdc95 100644
--- a/include/basegfx/color/bcolortools.hxx
+++ b/include/basegfx/color/bcolortools.hxx
@

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

2017-01-06 Thread Muhammet Kara
 accessibility/inc/helper/accessiblestrings.hrc|   23 +-
 accessibility/source/extended/accessibleeditbrowseboxcell.cxx |7 ++-
 accessibility/source/helper/accessiblestrings.src |   10 
 3 files changed, 28 insertions(+), 12 deletions(-)

New commits:
commit 92a1ad1f36b6d3cc13135a8c0805508933011577
Author: Muhammet Kara 
Date:   Tue Dec 27 00:04:44 2016 +0200

Localize return value of EditBrowseBoxTableCell::getAccessibleName

Change-Id: I040c1b9b8680a366819254e1d44b80115a339bbf
Reviewed-on: https://gerrit.libreoffice.org/32441
Tested-by: Jenkins 
Reviewed-by: Andras Timar 

diff --git a/accessibility/inc/helper/accessiblestrings.hrc 
b/accessibility/inc/helper/accessiblestrings.hrc
index 60c4392..f957ab9 100644
--- a/accessibility/inc/helper/accessiblestrings.hrc
+++ b/accessibility/inc/helper/accessiblestrings.hrc
@@ -30,21 +30,24 @@
 #define RID_STR_ACC_ACTION_DECLINE  ( RID_TK_ACC_START +4 )
 #define RID_STR_ACC_ACTION_INCBLOCK ( RID_TK_ACC_START +5 )
 #define RID_STR_ACC_ACTION_DECBLOCK ( RID_TK_ACC_START +6 )
-#define RID_STR_ACC_NAME_PREVIEW( RID_TK_ACC_START +7 )
+#define RID_STR_ACC_NAME_PREVIEW( RID_TK_ACC_START +7 )
 
 #define STR_SVT_ACC_ACTION_EXPAND   ( RID_TK_ACC_START +8 )
 #define STR_SVT_ACC_ACTION_COLLAPSE ( RID_TK_ACC_START +9 )
-#define STR_SVT_ACC_LISTENTRY_SELCTED_STATE ( RID_TK_ACC_START +10 )
+#define STR_SVT_ACC_LISTENTRY_SELCTED_STATE ( RID_TK_ACC_START +   10 )
 
-#define RID_STR_ACC_ACTION_CHECK( RID_TK_ACC_START +11 )
-#define RID_STR_ACC_ACTION_UNCHECK  ( RID_TK_ACC_START +12 )
-#define RID_STR_ACC_ACTION_DOUBLE_CLICK ( RID_TK_ACC_START +13 )
-#define RID_STR_ACC_SCROLLBAR_NAME_VERTICAL ( RID_TK_ACC_START +14 )
-#define RID_STR_ACC_SCROLLBAR_NAME_HORIZONTAL   ( RID_TK_ACC_START +15 
)
-#define RID_STR_ACC_PANEL_DESCRIPTION   ( RID_TK_ACC_START +16 )
+#define RID_STR_ACC_ACTION_CHECK( RID_TK_ACC_START +   11 )
+#define RID_STR_ACC_ACTION_UNCHECK  ( RID_TK_ACC_START +   12 )
+#define RID_STR_ACC_ACTION_DOUBLE_CLICK ( RID_TK_ACC_START +   13 )
+#define RID_STR_ACC_SCROLLBAR_NAME_VERTICAL ( RID_TK_ACC_START +   14 )
+#define RID_STR_ACC_SCROLLBAR_NAME_HORIZONTAL   ( RID_TK_ACC_START +   15 )
+#define RID_STR_ACC_PANEL_DESCRIPTION   ( RID_TK_ACC_START +   16 )
 
-#define RID_STR_ACC_NAME_BROWSEBUTTON   ( RID_TK_ACC_START + 100 )
-#define RID_STR_ACC_DESC_PANELDECL_TABBAR   ( RID_TK_ACC_START + 101 )
+#define RID_STR_ACC_COLUMN_NUM  ( RID_TK_ACC_START +   17 )
+#define RID_STR_ACC_ROW_NUM ( RID_TK_ACC_START +   18 )
+
+#define RID_STR_ACC_NAME_BROWSEBUTTON   ( RID_TK_ACC_START +  100 )
+#define RID_STR_ACC_DESC_PANELDECL_TABBAR   ( RID_TK_ACC_START +  101 )
 
 #endif  // ACCESSIBILITY_HELPER_ACCESSIBLESTRINGS_HRC_
 
diff --git a/accessibility/source/extended/accessibleeditbrowseboxcell.cxx 
b/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
index 7344acf..d816176 100644
--- a/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
+++ b/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
@@ -17,6 +17,9 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
+#include 
+#include 
+
 #include 
 #include 
 #include 
@@ -103,8 +106,8 @@ namespace accessibility
 {
 SolarMethodGuard aGuard( *this );
 
-// TODO: localize this!
-return "Column " + OUString::number(getColumnPos()-1) + ", Row " + 
OUString::number(getRowPos());
+return 
TK_RES_STRING(RID_STR_ACC_COLUMN_NUM).replaceAll("%COLUMNNUMBER", 
OUString::number(getColumnPos()-1)) + ", "
+   + TK_RES_STRING(RID_STR_ACC_ROW_NUM).replaceAll("%ROWNUMBER", 
OUString::number(getRowPos()));
 }
 
 css::uno::Reference< css::accessibility::XAccessibleRelationSet > SAL_CALL 
EditBrowseBoxTableCell::getAccessibleRelationSet() throw ( RuntimeException, 
std::exception )
diff --git a/accessibility/source/helper/accessiblestrings.src 
b/accessibility/source/helper/accessiblestrings.src
index 63e3067..1a82275 100644
--- a/accessibility/source/helper/accessiblestrings.src
+++ b/accessibility/source/helper/accessiblestrings.src
@@ -106,6 +106,16 @@ String RID_STR_ACC_PANEL_DESCRIPTION
 Text [ en-US ] = "Please press enter to go into child control for more 
operations";
 };
 
+String RID_STR_ACC_COLUMN_NUM
+{
+Text [ en-US ] = "Column %COLUMNNUMBER";
+};
+
+String RID_STR_ACC_ROW_NUM
+{
+Text [ en-US ] = "Row %ROWNUMBER";
+};
+
 String RID_STR_ACC_DESC_PANELDECL_TABBAR
 {
 Text [ en-US ] = "Panel Deck Tab Bar";
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Crash test update

2017-01-06 Thread Crashtest VM
New crashtest update available at 
http://dev-builds.libreoffice.org/crashtest/6b89bcf85a911d043c9d93e843be12e6f23adedd/


exportCrashes.csv
Description: Binary data


importCrash.csv
Description: Binary data


validationErrors.csv
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Bug TDF#99097 - Call for reflection

2017-01-06 Thread Pierre Lepage

Encoding of "N" and "NN" in StarBasic :
/libreoffice/basic/source/sbx/sbxscan.cxx
a) #define in lines 654 and 655.
b) Interpretation of "N" and "NN" in lines 743 to 754.

Hum. H and HH code for hours. There is a risk of confusion. I understand 
the difficulty of reassigning the N and NN codes. So why not go with two 
new codes? I propose I and II (second letter in the word Minute ...).


Pierre


Le 2017-01-06 à 14:58, Eike Rathke a écrit :

Hi Pierre,

On Friday, 2016-12-30 06:24:27 -0500, Pierre Lepage wrote:


StarBasic encodes "N" and "NN" as formats different
from those of LO, but similar to VBA, ie "N" for minutes without 0 in prefix
and "NN" for minutes With 0 in prefix.

Where does it do that? Could you provide a code pointer please?


The first solution appears as a crutch. Coding for an exception will not
resolve the source of confusion. Nevertheless, if this solution were to be
implemented, the algorithm would have to distinguish between two situations
of the codes: a) the codes are embedded in a time format string only in
which case the interpretation of the N and NN codes would be in conformity
To VBA, that is to say that N would code for minutes without 0 in prefix and
NN would code for minutes with 0 in prefix; B) otherwise the codes would be
interpreted as they are now, ie NN would code for the abbreviation of the
name of the day and N would be ignored.

I'd refrain from such diversion in the number formatter code..


The second solution appears more robust. The confusion would be definitively
eliminated. It remains that the historical reason behind the choice of the
letters NN and NNN rather than exclusively DDD and  to codify an
abbreviated and long day format is unknown to me. It is therefore possible
that old files based on the current interpretation of the NN, NNN and 
codes will require corrections to rely solely on the DDD and  codes.

Which also is not easily possible, because documents may use format
codes relying on the behavior, even worse use such format codes in
Calc's TEXT() spreadsheet function as argument.


I included a Calc file summarizing the format codes for dates and time. The
first tab is the current situation. The second tab is for the situation once
the second solution is implemented.

So, before going any further, I need to know the desired orientation for
LibreOffice.

I'd change the StarBasic code to pass on modified format codes, H for
N and HH for NN.

   Eike



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


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

2017-01-06 Thread Michael Stahl
 download.lst|3 ---
 sw/source/core/access/accfrmobj.cxx |   12 
 2 files changed, 8 insertions(+), 7 deletions(-)

New commits:
commit ade1d4c36053c2a7aed959e3dd09ebc1b8430ea8
Author: Michael Stahl 
Date:   Fri Jan 6 22:37:17 2017 +0100

tdf#105009 sw: fix a11y crash when removing drawing object

SwFrame::RemoveDrawObj() calls SwAnchoredObject::ChgAnchorFrame(0)
so the SwAnchoredDrawObj has no anchor frame later when
SwAccessibleMap::InvalidateCursorPosition() asks for the parent.

(regression from 76c549eb01dcb7b5bf28a271ce00e386f3d388ba)

Change-Id: Id55cb5fc41a4e37e863498265d1565e1621d508e

diff --git a/sw/source/core/access/accfrmobj.cxx 
b/sw/source/core/access/accfrmobj.cxx
index 3c6b3e5..9c8d515 100644
--- a/sw/source/core/access/accfrmobj.cxx
+++ b/sw/source/core/access/accfrmobj.cxx
@@ -379,10 +379,14 @@ const SwFrame* SwAccessibleChild::GetParent( const bool 
bInPagePreview ) const
 else
 {
 // In any other case the parent is the root frm
-if( bInPagePreview )
-pParent = pContact->GetAnchorFrame()->FindPageFrame();
-else
-pParent = pContact->GetAnchorFrame()->getRootFrame();
+SwFrame const*const pAnchor(pContact->GetAnchorFrame());
+if (pAnchor) // null if object removed from layout
+{
+if (bInPagePreview)
+pParent = pAnchor->FindPageFrame();
+else
+pParent = pAnchor->getRootFrame();
+}
 }
 }
 }
commit 2f1cf69153593ba61da672be72eb2ff4a5a7dad8
Author: Michael Stahl 
Date:   Fri Jan 6 21:05:57 2017 +0100

remove obsolete MOZ_ZIP_* variables

Change-Id: Ie83ff9e50534536326c78de1b6b717fe18b6dae1

diff --git a/download.lst b/download.lst
index 52b5210..c28b972 100644
--- a/download.lst
+++ b/download.lst
@@ -175,6 +175,3 @@ export ZLIB_TARBALL := zlib-1.2.8.tar.gz
 export ZMF_MD5SUM := c611df8664240de0276ab95670f413d8
 export ZMF_TARBALL := libzmf-0.0.1.tar.bz2
 
-export MOZ_ZIP_INC := $(OS)$(COM)$(if $(filter INTEL,$(CPUNAME)),I,X)inc.zip
-export MOZ_ZIP_LIB := $(OS)$(COM)$(if $(filter INTEL,$(CPUNAME)),I,X)lib.zip
-export MOZ_ZIP_RUNTIME := $(OS)$(COM)$(if $(filter 
INTEL,$(CPUNAME)),I,X)runtime.zip
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Khaled Hosny
 vcl/source/gdi/print.cxx |   34 +++---
 1 file changed, 11 insertions(+), 23 deletions(-)

New commits:
commit 6b737072279b137b0224a3bbe48907621a36f434
Author: Khaled Hosny 
Date:   Fri Jan 6 02:00:06 2017 +0200

Drop macros that are used only once

Change-Id: Ib75c0db614448f5297de9924335e2ee9f71f3a3f
Reviewed-on: https://gerrit.libreoffice.org/32766
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/vcl/source/gdi/print.cxx b/vcl/source/gdi/print.cxx
index a0b9218..50bc4ad 100644
--- a/vcl/source/gdi/print.cxx
+++ b/vcl/source/gdi/print.cxx
@@ -106,18 +106,6 @@ PrinterOptions::~PrinterOptions()
 {
 }
 
-#define PROPERTYNAME_REDUCETRANSPARENCY "ReduceTransparency"
-#define PROPERTYNAME_REDUCEDTRANSPARENCYMODE
"ReducedTransparencyMode"
-#define PROPERTYNAME_REDUCEGRADIENTS"ReduceGradients"
-#define PROPERTYNAME_REDUCEDGRADIENTMODE"ReducedGradientMode"
-#define PROPERTYNAME_REDUCEDGRADIENTSTEPCOUNT   
"ReducedGradientStepCount"
-#define PROPERTYNAME_REDUCEBITMAPS  "ReduceBitmaps"
-#define PROPERTYNAME_REDUCEDBITMAPMODE  "ReducedBitmapMode"
-#define PROPERTYNAME_REDUCEDBITMAPRESOLUTION
"ReducedBitmapResolution"
-#define PROPERTYNAME_REDUCEDBITMAPINCLUDESTRANSPARENCY  
"ReducedBitmapIncludesTransparency"
-#define PROPERTYNAME_CONVERTTOGREYSCALES"ConvertToGreyscales"
-#define PROPERTYNAME_PDFASSTANDARDPRINTJOBFORMAT
"PDFAsStandardPrintJobFormat"
-
 bool PrinterOptions::ReadFromConfig( bool i_bFile )
 {
 bool bSuccess = false;
@@ -155,27 +143,27 @@ bool PrinterOptions::ReadFromConfig( bool i_bFile )
 {
 sal_Int32 nValue = 0;
 bool  bValue = false;
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCETRANSPARENCY) >>= bValue )
+if( xSet->getPropertyValue("ReduceTransparency") >>= 
bValue )
 SetReduceTransparency( bValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCEDTRANSPARENCYMODE) >>= nValue )
+if( xSet->getPropertyValue("ReducedTransparencyMode") >>= 
nValue )
 SetReducedTransparencyMode( 
(PrinterTransparencyMode)nValue );
-if( xSet->getPropertyValue(PROPERTYNAME_REDUCEGRADIENTS) 
>>= bValue )
+if( xSet->getPropertyValue("ReduceGradients") >>= bValue )
 SetReduceGradients( bValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCEDGRADIENTMODE) >>= nValue )
+if( xSet->getPropertyValue("ReducedGradientMode") >>= 
nValue )
 SetReducedGradientMode( (PrinterGradientMode)nValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCEDGRADIENTSTEPCOUNT) >>= nValue )
+if( xSet->getPropertyValue("ReducedGradientStepCount") >>= 
nValue )
 SetReducedGradientStepCount( (sal_uInt16)nValue );
-if( xSet->getPropertyValue(PROPERTYNAME_REDUCEBITMAPS) >>= 
bValue )
+if( xSet->getPropertyValue("ReduceBitmaps") >>= bValue )
 SetReduceBitmaps( bValue );
-if( xSet->getPropertyValue(PROPERTYNAME_REDUCEDBITMAPMODE) 
>>= nValue )
+if( xSet->getPropertyValue("ReducedBitmapMode") >>= nValue 
)
 SetReducedBitmapMode( (PrinterBitmapMode)nValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCEDBITMAPRESOLUTION) >>= nValue )
+if( xSet->getPropertyValue("ReducedBitmapResolution") >>= 
nValue )
 SetReducedBitmapResolution( (sal_uInt16)nValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_REDUCEDBITMAPINCLUDESTRANSPARENCY) >>= 
bValue )
+if( 
xSet->getPropertyValue("ReducedBitmapIncludesTransparency") >>= bValue )
 SetReducedBitmapIncludesTransparency( bValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_CONVERTTOGREYSCALES) >>= bValue )
+if( xSet->getPropertyValue("ConvertToGreyscales") >>= 
bValue )
 SetConvertToGreyscales( bValue );
-if( 
xSet->getPropertyValue(PROPERTYNAME_PDFASSTANDARDPRINTJOBFORMAT) >>= bValue )
+if( xSet->getPropertyValue("PDFAsStandardPrintJobFormat") 
>>= bValue )
 SetPDFAsStandardPrintJobFormat( bValue );
 
 bSuccess = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Markus Mohrhard
 desktop/source/app/app.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit baa99b4e3a466495d0fa8fdef313230eec44c9ce
Author: Markus Mohrhard 
Date:   Thu Jan 5 22:35:52 2017 +0100

add the locale info to the crashreporter metadata

We have a few crashes that seem locale dependent. Especially the font
related problems.

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

diff --git a/desktop/source/app/app.cxx b/desktop/source/app/app.cxx
index 437ff51..90ace3c 100644
--- a/desktop/source/app/app.cxx
+++ b/desktop/source/app/app.cxx
@@ -2166,6 +2166,8 @@ void Desktop::OpenClients()
 }
 #if HAVE_FEATURE_BREAKPAD
 CrashReporter::writeCommonInfo();
+// write this information here to avoid depending on vcl in the crash 
reporter lib
+CrashReporter::AddKeyValue("Language", 
Application::GetSettings().GetLanguageTag().getBcp47());
 #endif
 
 RequestHandler::EnableRequests();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Caolán McNamara
 vcl/headless/svpbmp.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit a6b6980afa443b3d317de5f24d993f939073d73a
Author: Caolán McNamara 
Date:   Fri Jan 6 20:54:19 2017 +

oss-fuzz: make max sane bitmap allocation smaller

Change-Id: I0b924f615ed4652ce605c12d2d2e9ca2ad2befbf

diff --git a/vcl/headless/svpbmp.cxx b/vcl/headless/svpbmp.cxx
index d11f3b9..ee5a983 100644
--- a/vcl/headless/svpbmp.cxx
+++ b/vcl/headless/svpbmp.cxx
@@ -122,7 +122,7 @@ BitmapBuffer* ImplCreateDIB(
 }
 
 const size_t size = pDIB->mnScanlineSize * pDIB->mnHeight;
-if (size > SAL_MAX_INT32)
+if (size > SAL_MAX_INT32/2)
 {
 delete pDIB;
 return nullptr;
@@ -178,7 +178,7 @@ bool SvpSalBitmap::Create(const SalBitmap& rBmp)
 mpDIB = new BitmapBuffer( *rSalBmp.mpDIB );
 
 const size_t size = mpDIB->mnScanlineSize * mpDIB->mnHeight;
-if (size > SAL_MAX_INT32)
+if (size > SAL_MAX_INT32/2)
 {
 delete mpDIB;
 mpDIB = nullptr;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


New Defects reported by Coverity Scan for LibreOffice

2017-01-06 Thread scan-admin

Hi,

Please find the latest report on new defect(s) introduced to LibreOffice found 
with Coverity Scan.

3 new defect(s) introduced to LibreOffice found with Coverity Scan.
13 defect(s), reported by Coverity Scan earlier, were marked fixed in the 
recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 3 of 3 defect(s)


** CID 1398371:  Incorrect expression  (USELESS_CALL)
/basegfx/test/basegfx2d.cxx: 1223 in basegfx2d::bcolor::ciexyzTest()()



*** CID 1398371:  Incorrect expression  (USELESS_CALL)
/basegfx/test/basegfx2d.cxx: 1223 in basegfx2d::bcolor::ciexyzTest()()
1217 tools::rgb2ciexyz(maBlack);
1218 tools::rgb2ciexyz(maRed);
1219 tools::rgb2ciexyz(maGreen);
1220 tools::rgb2ciexyz(maBlue);
1221 tools::rgb2ciexyz(maYellow);
1222 tools::rgb2ciexyz(maMagenta);
>>> CID 1398371:  Incorrect expression  (USELESS_CALL)
>>> Calling "basegfx::tools::rgb2ciexyz(this->maCyan)" is only useful for 
>>> its return value, which is ignored.
1223 tools::rgb2ciexyz(maCyan);
1224 }
1225 
1226 // Change the following lines only, if you add, remove or rename
1227 // member functions of the current class,
1228 // because these macros are need by auto register mechanism.

** CID 1398370:  Memory - corruptions  (OVERRUN)
/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx: 386 in 
TiledRowColumnBar::docConfigureEvent(_GtkWidget *, _GdkEventConfigure *, void 
*)()



*** CID 1398370:  Memory - corruptions  (OVERRUN)
/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx: 386 in 
TiledRowColumnBar::docConfigureEvent(_GtkWidget *, _GdkEventConfigure *, void 
*)()
380 "\"HorizontalAlignment\":{"
381 "\"type\":\"unsigned short\", "
382 "\"value\":\"%d\""
383 "}"
384 "}";
385 
>>> CID 1398370:  Memory - corruptions  (OVERRUN)
>>> Allocating insufficient memory for the terminating null of the string.
386 char unoHorAlignArgsFormatted[strlen(unoHorAlignArgs)];
387 snprintf(unoHorAlignArgsFormatted, 
sizeof(unoHorAlignArgsFormatted), unoHorAlignArgs, 1);
388 lcl_registerToolItem(rWindow, rWindow.m_pLeftpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
389 snprintf(unoHorAlignArgsFormatted, 
sizeof(unoHorAlignArgsFormatted), unoHorAlignArgs, 2);
390 lcl_registerToolItem(rWindow, rWindow.m_pCenterpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
391 snprintf(unoHorAlignArgsFormatted, 
sizeof(unoHorAlignArgsFormatted), unoHorAlignArgs, 3);

** CID 1398369:  Null pointer dereferences  (FORWARD_NULL)
/sw/source/core/draw/dcontact.cxx: 589 in 
SwFlyDrawContact::SwClientNotify(const SwModify &, const SfxHint &)()



*** CID 1398369:  Null pointer dereferences  (FORWARD_NULL)
/sw/source/core/draw/dcontact.cxx: 589 in 
SwFlyDrawContact::SwClientNotify(const SwModify &, const SfxHint &)()
583 SwContact::SwClientNotify(rMod, rHint);
584 if(auto pGetZOrdnerHint = dynamic_cast(&rHint))
585 {
586 // #i11176#
587 // This also needs to work when no layout exists. Thus, for
588 // FlyFrames an alternative method is used now in that case.
>>> CID 1398369:  Null pointer dereferences  (FORWARD_NULL)
>>> Assigning: "pFormat" = "dynamic_cast (rMod)".
589 auto pFormat(dynamic_cast(&rMod));
590 if(pFormat->Which() == RES_FLYFRMFMT && 
!pFormat->getIDocumentLayoutAccess().GetCurrentViewShell())
591 pGetZOrdnerHint->m_rnZOrder = GetMaster()->GetOrdNum();
592 }
593 }
594 



To view the defects in Coverity Scan visit, 
https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRZBnDJeNb0HijxaS4JNJPxk3kpyAm2AYqo71yXmnOxB72ibeUH-2F-2F1Lhi9AZq3dRu-2F4-3D_g-2BrHdvqzaBa155F-2F8AmPhpJzY63UzWDisJV95WUBpGhqFw1ICExHG8aMaV2EoFpy8KKgQJetE0bvSegy2wEPKZOpuitNhTzMIun-2BLWgzpRkQ2zgqBpc2AvpeJSTeh-2Fjl-2FYy9avV7ZbFQ1kQllmVEAIO0RmdIfhEB6PVvF1gJQMN3VAWF92mpCIav5grL-2BoaG4g0hOUk1kWuhZFHxMEikAU-2BBx1E-2FOboBHL4lGe2rzyY-3D

To manage Coverity Scan email notifications for 
"libreoffice@lists.freedesktop.org", click 
https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRbVDbis712qZDP-2FA8y06Nq4k1FZJSDV-2FTHi5VQof9xGafB4oBwGYxuHHknceo2QLpCrZ44Ciy7AqBR2QyX6OCB5N5X-2B1MAElavPQhH6nLwozJzqOkt2k8uOkYf2ZoppNa9QVe0

Re: Bug TDF#99097 - Call for reflection

2017-01-06 Thread Eike Rathke
Hi Pierre,

On Friday, 2016-12-30 06:24:27 -0500, Pierre Lepage wrote:

> StarBasic encodes "N" and "NN" as formats different
> from those of LO, but similar to VBA, ie "N" for minutes without 0 in prefix
> and "NN" for minutes With 0 in prefix.

Where does it do that? Could you provide a code pointer please?

> The first solution appears as a crutch. Coding for an exception will not
> resolve the source of confusion. Nevertheless, if this solution were to be
> implemented, the algorithm would have to distinguish between two situations
> of the codes: a) the codes are embedded in a time format string only in
> which case the interpretation of the N and NN codes would be in conformity
> To VBA, that is to say that N would code for minutes without 0 in prefix and
> NN would code for minutes with 0 in prefix; B) otherwise the codes would be
> interpreted as they are now, ie NN would code for the abbreviation of the
> name of the day and N would be ignored.

I'd refrain from such diversion in the number formatter code..

> The second solution appears more robust. The confusion would be definitively
> eliminated. It remains that the historical reason behind the choice of the
> letters NN and NNN rather than exclusively DDD and  to codify an
> abbreviated and long day format is unknown to me. It is therefore possible
> that old files based on the current interpretation of the NN, NNN and 
> codes will require corrections to rely solely on the DDD and  codes.

Which also is not easily possible, because documents may use format
codes relying on the behavior, even worse use such format codes in
Calc's TEXT() spreadsheet function as argument.

> I included a Calc file summarizing the format codes for dates and time. The
> first tab is the current situation. The second tab is for the situation once
> the second solution is implemented.
> 
> So, before going any further, I need to know the desired orientation for
> LibreOffice.

I'd change the StarBasic code to pass on modified format codes, H for
N and HH for NN.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GPG key "ID" 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A
Better use 64-bit 0x6A6CD5B765632D3A here is why: https://evil32.com/
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] online.git: kit/Kit.cpp

2017-01-06 Thread Henry Castro
 kit/Kit.cpp |9 -
 1 file changed, 8 insertions(+), 1 deletion(-)

New commits:
commit 4641c922eac1c5f5d2ed2adefd8119011e7fdab5
Author: Henry Castro 
Date:   Fri Jan 6 15:49:33 2017 -0400

kit: use a second stage init after fork call

diff --git a/kit/Kit.cpp b/kit/Kit.cpp
index 62a183a..bfe75de 100644
--- a/kit/Kit.cpp
+++ b/kit/Kit.cpp
@@ -95,6 +95,7 @@ using namespace LOOLProtocol;
 // We only host a single document in our lifetime.
 class Document;
 static std::shared_ptr document;
+static LokHookFunction2* pInit = nullptr;
 
 namespace
 {
@@ -1594,7 +1595,7 @@ void lokit_main(const std::string& childRoot,
 auto kit = UnitKit::get().lok_init(instdir, userdir);
 if (!kit)
 {
-kit = lok_init_2(instdir, userdir);
+kit = (pInit ? pInit(instdir, userdir) : lok_init_2(instdir, 
userdir));
 }
 
 loKit = std::make_shared(kit);
@@ -1792,6 +1793,12 @@ bool globalPreinit(const std::string &loTemplate)
 return false;
 }
 
+pInit = reinterpret_cast(dlsym(handle, 
"libreofficekit_hook_2"));
+if (!pInit)
+{
+LOG_FTL("No libreofficekit_hook_2 symbol in " << loadedLibrary << ": " 
<< dlerror());
+}
+
 LOG_TRC("lok_preinit(" << loTemplate << "/program\", \"file:///user\")");
 if (preInit((loTemplate + "/program").c_str(), "file:///user") != 0)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - sc/source

2017-01-06 Thread Eike Rathke
 sc/source/filter/xml/xmlcelli.cxx |   21 -
 1 file changed, 16 insertions(+), 5 deletions(-)

New commits:
commit cb7ab49dde0ae21c472300f126cce4555a3e0ae5
Author: Eike Rathke 
Date:   Fri Jan 6 18:28:42 2017 +0100

recognize the broken "Err:xxx" written by 5.2 and earlier, tdf#105024 
related

... and handle same as "#ERRxxx!" if present.

Change-Id: I1ebb31d628b080c52b450a8fe624c20e9e1188b7
(cherry picked from commit 75d963bc7bb87429f304d29138c27178880c039a)
Reviewed-on: https://gerrit.libreoffice.org/32793
Reviewed-by: Eike Rathke 
Tested-by: Jenkins 

diff --git a/sc/source/filter/xml/xmlcelli.cxx 
b/sc/source/filter/xml/xmlcelli.cxx
index 3b38ac4..8fd7eda 100644
--- a/sc/source/filter/xml/xmlcelli.cxx
+++ b/sc/source/filter/xml/xmlcelli.cxx
@@ -1357,11 +1357,22 @@ void ScXMLTableRowCellContext::PutFormulaCell( const 
ScAddress& rCellPos )
 }
 else
 {
-OUString aFormulaNmsp = maFormula->second;
-if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
-aFormulaNmsp.clear();
-pCode->AssignXMLString( aText, aFormulaNmsp );
-rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+// 5.2 and earlier wrote broken "Err:xxx" as formula to designate
+// an error formula cell.
+if (aText.startsWithIgnoreAsciiCase("Err:") && aText.getLength() 
<= 9 &&
+((nError =
+  GetScImport().GetFormulaErrorConstant( "#ERR" + 
aText.copy(4) + "!")) != FormulaError::NONE))
+{
+pCode->SetCodeError(nError);
+}
+else
+{
+OUString aFormulaNmsp = maFormula->second;
+if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
+aFormulaNmsp.clear();
+pCode->AssignXMLString( aText, aFormulaNmsp );
+rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+}
 }
 
 ScFormulaCell* pNewCell = new ScFormulaCell(pDoc, rCellPos, pCode, 
eGrammar, MM_NONE);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Problems with Compiling

2017-01-06 Thread Eike Rathke
Hi Moritz,

On Wednesday, 2017-01-04 12:30:51 +, Moritz Staudinger wrote:

> I was able to clone the Project and the online version of it, but when i run 
> the make check command on github.com/Libreoffice/core , the sfx2_complex Unit 
> Test fail. Can I fix this easily?

You didn't say what fails nor what platform you're on..

As a side note, github might work and usually is in sync, but I'd
recommend to pull from git://anongit.freedesktop.org/libreoffice/core or
git://gerrit.libreoffice.org/core instead.

> Furthermore I would like to set the Menu to another language, is there a way 
> to do this? I tried
> 
> ./autogen.sh --without-help --with-lang=ALL , but when I start Office it is 
> still English.

The default UI language is selected according to your system settings,
or if for that no match is found then a fallback to English. Maybe you
need to explicitly select an UI language under
Tools->Options->LanguageSettings

Second, --with-lang=ALL can be quite some overhead to build for all
languages. Recommendable is a selective list, e.g. if you want a German
UI then --with-lang=de which still builds an en-US UI as well.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GPG key "ID" 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A
Better use 64-bit 0x6A6CD5B765632D3A here is why: https://evil32.com/
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] online.git: kit/Kit.cpp

2017-01-06 Thread Henry Castro
 kit/Kit.cpp |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 22b80e285baea2d9a801cfe808d2a95550bf5e22
Author: Henry Castro 
Date:   Fri Jan 6 15:35:57 2017 -0400

kit: remove unused typedef function pointer

diff --git a/kit/Kit.cpp b/kit/Kit.cpp
index 2b6eac1..62a183a 100644
--- a/kit/Kit.cpp
+++ b/kit/Kit.cpp
@@ -68,8 +68,6 @@
 #define LIB_SOFFICEAPP  "lib" "sofficeapp" ".so"
 #define LIB_MERGED  "lib" "mergedlo" ".so"
 
-typedef int (LokHookPreInit)  (const char *install_path, const char 
*user_profile_path);
-
 using Poco::Exception;
 using Poco::File;
 using Poco::JSON::Array;
___
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.1' - 4 commits - download.lst external/curl external/expat ucb/source

2017-01-06 Thread Michael Stahl
 download.lst|8 
 external/curl/ExternalPackage_curl.mk   |2 
 external/curl/ExternalProject_curl.mk   |2 
 external/curl/UnpackedTarball_curl.mk   |3 
 external/curl/curl-7.26.0_win-proxy.patch   |   43 -
 external/curl/curl-freebsd.patch.1  |   32 -
 external/curl/curl-msvc-schannel.patch.1|4 
 external/curl/curl-msvc.patch.1 |   12 
 external/expat/CVE-2015-1283-refix.patch.1  |   37 -
 external/expat/CVE-2016-0718-v2-2-1.patch.1 |  755 
 external/expat/UnpackedTarball_expat.mk |2 
 ucb/source/ucp/ftp/ftploaderthread.cxx  |2 
 12 files changed, 42 insertions(+), 860 deletions(-)

New commits:
commit 31b3cf5726d2d7fbaa5e08e7281c6123b6b54742
Author: Michael Stahl 
Date:   Wed Nov 2 11:48:21 2016 +0100

curl: upgrade to version 7.51.0

- fixes about a dozen CVEs
- tweak curl-7.26.0_win-proxy.patch:
  there is a "checksrc" thing now in curl that fails the build with:
warning: use of strtok is banned (BANNEDFUNC)
  so use strtok_s instead (which is MSVC's name for standard strtok_r)
- use Makefile.vc12 instead of Makefile.vc10
- remove strequal.obj from Makefile.vc12, it does not actually exist

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

diff --git a/download.lst b/download.lst
index 0da981a..2a4c12c 100755
--- a/download.lst
+++ b/download.lst
@@ -21,8 +21,8 @@ export COLLADA2GLTF_TARBALL := 
4b87018f7fff1d054939d19920b751a0-collada2gltf-mas
 export CPPUNIT_MD5SUM := d1c6bdd5a76c66d2c38331e2d287bc01
 export CPPUNIT_TARBALL := cppunit-1.13.2.tar.gz
 export CT2N_TARBALL := 
1f467e5bb703f12cbbb09d5cf67ecf4a-converttexttonumber-1-5-0.oxt
-export CURL_MD5SUM := 03940d7d4fcea1521fbbf07c1cf16f5e
-export CURL_TARBALL := curl-7.50.0.tar.gz
+export CURL_MD5SUM := 490e19a8ccd1f4a244b50338a0eb9456
+export CURL_TARBALL := curl-7.51.0.tar.gz
 export DBGHELP_DLL := 13fbc2e8b37ddf28181dd6d8081c2b8e-dbghelp.dll
 export EBOOK_MD5SUM := 6b48eda57914e6343efebc9381027b78
 export EBOOK_TARBALL := libe-book-0.1.2.tar.bz2
diff --git a/external/curl/ExternalProject_curl.mk 
b/external/curl/ExternalProject_curl.mk
index d91b61c..b2fc60d 100644
--- a/external/curl/ExternalProject_curl.mk
+++ b/external/curl/ExternalProject_curl.mk
@@ -79,7 +79,7 @@ else ifeq ($(COM),MSC)
 
 $(call gb_ExternalProject_get_state_target,curl,build):
$(call gb_ExternalProject_run,build,\
-   MAKEFLAGS= LIB="$(ILIB)" nmake -f Makefile.vc10 \
+   MAKEFLAGS= LIB="$(ILIB)" nmake -f Makefile.vc12 \
cfg=$(if 
$(MSVC_USE_DEBUG_RUNTIME),debug-dll,release-dll) \
EXCFLAGS="/EHa /Zc:wchar_t- /D_CRT_SECURE_NO_DEPRECATE 
/DUSE_WINDOWS_SSPI /D_USING_V110_SDK71_ $(SOLARINC)" $(if $(filter 
X86_64,$(CPUNAME)),MACHINE=X64) \
,lib)
diff --git a/external/curl/UnpackedTarball_curl.mk 
b/external/curl/UnpackedTarball_curl.mk
index fc9d3d2..63055a2 100644
--- a/external/curl/UnpackedTarball_curl.mk
+++ b/external/curl/UnpackedTarball_curl.mk
@@ -14,7 +14,7 @@ $(eval $(call 
gb_UnpackedTarball_set_tarball,curl,$(CURL_TARBALL),,curl))
 $(eval $(call gb_UnpackedTarball_set_patchlevel,curl,1))
 
 $(eval $(call gb_UnpackedTarball_fix_end_of_line,curl,\
-   lib/Makefile.vc10 \
+   lib/Makefile.vc12 \
 ))
 
 $(eval $(call gb_UnpackedTarball_add_patches,curl,\
diff --git a/external/curl/curl-7.26.0_win-proxy.patch 
b/external/curl/curl-7.26.0_win-proxy.patch
index 4c5327f..580479a 100644
--- a/external/curl/curl-7.26.0_win-proxy.patch
+++ b/external/curl/curl-7.26.0_win-proxy.patch
@@ -1,5 +1,5 @@
 curl-7.26.0/lib/Makefile.vc10
-+++ misc/build/curl-7.26.0/lib/Makefile.vc10
+--- curl-7.26.0/lib/Makefile.vc12
 misc/build/curl-7.26.0/lib/Makefile.vc12
 @@ -118,7 +118,7 @@
  WINSSLLIBS   = crypt32.lib
  ZLIBLIBSDLL  = zdll.lib
@@ -13,7 +13,7 @@
 +++ misc/build/curl-7.26.0/lib/url.c
 @@ -78,6 +78,10 @@
  bool curl_win32_idn_to_ascii(const char *in, char **out);
- #endif  /* USE_LIBIDN */
+ #endif  /* USE_LIBIDN2 */
  
 +#ifdef WIN32
 +#include 
@@ -52,7 +52,7 @@
  
  #ifndef CURL_DISABLE_HTTP
/* If proxy was not specified, we check for default proxy environment
-@@ -4613,7 +4633,63 @@
+@@ -4613,7 +4633,64 @@
 * For compatibility, the all-uppercase versions of these variables are
 * checked if the lowercase versions don't exist.
 */
@@ -83,9 +83,10 @@
 +  if(!check_noproxy(conn->host.name, no_proxy)) {
 +/* Look for the http proxy setting */
 +char* tok;
++char *saveptr;
 +
 +if(NULL != ieProxy) {
-+  tok = strtok(ieProxy, ";");
++  tok = strtok_s(ieProxy, ";", &saveptr);
 +  if(strchr(tok, '=') == NULL) {
 +proxy = strdup(ieProxy);
 +  }
@@ -95,7 +96,7 @@
 +/* We found HTTP proxy va

[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - 2 commits - formula/source sc/source

2017-01-06 Thread Eike Rathke
 formula/source/core/api/FormulaCompiler.cxx |5 +++--
 sc/source/core/data/formulacell.cxx |4 
 sc/source/filter/xml/xmlcelli.cxx   |   23 +--
 sc/source/filter/xml/xmlimprt.cxx   |6 +++---
 sc/source/filter/xml/xmlimprt.hxx   |2 +-
 5 files changed, 28 insertions(+), 12 deletions(-)

New commits:
commit 4164596f43132e7769057170a228d74118ee63c6
Author: Eike Rathke 
Date:   Fri Jan 6 17:46:02 2017 +0100

read single error constant formula as such, tdf#105024 related

... without creating a token so when writing again no leading '=' is 
prepended,
with which we can enable 5.2 to read such thing correctly, and when 
re-reading
in 5.3 it also doesn't lead to a "real" formula.

Change-Id: I26fbd20536436b49b781e2bbb5bba1dc6bafbb37
(cherry picked from commit 4fcbe16959c839bfacf745cfa554b234e639f794)
Reviewed-on: https://gerrit.libreoffice.org/32792
Reviewed-by: Eike Rathke 
Tested-by: Jenkins 

diff --git a/sc/source/core/data/formulacell.cxx 
b/sc/source/core/data/formulacell.cxx
index a3f14b4..bf6fc4e 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -1282,6 +1282,10 @@ void ScFormulaCell::CompileXML( 
sc::CompileFormulaContext& rCxt, ScProgress& rPr
 return ;
 }
 
+// Error constant formula cell stays as is.
+if (!pCode->GetLen() && pCode->GetCodeError() != FormulaError::NONE)
+return;
+
 // Compilation changes RPN count, remove and reinsert to FormulaTree if it
 // was in to update its count.
 bool bWasInFormulaTree = pDocument->IsInFormulaTree( this);
diff --git a/sc/source/filter/xml/xmlcelli.cxx 
b/sc/source/filter/xml/xmlcelli.cxx
index ddd8543..3b38ac4 100644
--- a/sc/source/filter/xml/xmlcelli.cxx
+++ b/sc/source/filter/xml/xmlcelli.cxx
@@ -1348,12 +1348,22 @@ void ScXMLTableRowCellContext::PutFormulaCell( const 
ScAddress& rCellPos )
 // temporary formula string as string tokens
 ScTokenArray *pCode = new ScTokenArray();
 
-OUString aFormulaNmsp = maFormula->second;
-if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
-aFormulaNmsp.clear();
-pCode->AssignXMLString( aText, aFormulaNmsp );
+// Check the special case of a single error constant without leading
+// '=' and create an error formula cell without tokens.
+FormulaError nError = GetScImport().GetFormulaErrorConstant(aText);
+if (nError != FormulaError::NONE)
+{
+pCode->SetCodeError(nError);
+}
+else
+{
+OUString aFormulaNmsp = maFormula->second;
+if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
+aFormulaNmsp.clear();
+pCode->AssignXMLString( aText, aFormulaNmsp );
+rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+}
 
-rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
 ScFormulaCell* pNewCell = new ScFormulaCell(pDoc, rCellPos, pCode, 
eGrammar, MM_NONE);
 SetFormulaCell(pNewCell);
 rDoc.setFormulaCell(rCellPos, pNewCell);
@@ -1466,7 +1476,8 @@ bool ScXMLTableRowCellContext::IsPossibleErrorString() 
const
 return false;
 else if(mbNewValueType && mbErrorValue)
 return true;
-return mbPossibleErrorCell || ( mbCheckWithCompilerForError && 
GetScImport().IsFormulaErrorConstant(*maStringValue) );
+return mbPossibleErrorCell || (mbCheckWithCompilerForError &&
+GetScImport().GetFormulaErrorConstant(*maStringValue) != 
FormulaError::NONE);
 }
 
 void ScXMLTableRowCellContext::EndElement()
diff --git a/sc/source/filter/xml/xmlimprt.cxx 
b/sc/source/filter/xml/xmlimprt.cxx
index d74f871..f1a818689 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -3407,12 +3407,12 @@ void ScXMLImport::ExtractFormulaNamespaceGrammar(
 reGrammar = eDefaultGrammar;
 }
 
-bool ScXMLImport::IsFormulaErrorConstant( const OUString& rStr ) const
+FormulaError ScXMLImport::GetFormulaErrorConstant( const OUString& rStr ) const
 {
 if (!mpComp)
-return false;
+return FormulaError::NONE;
 
-return mpComp->GetErrorConstant(rStr) > FormulaError::NONE;
+return mpComp->GetErrorConstant(rStr);
 }
 
 ScEditEngineDefaulter* ScXMLImport::GetEditEngine()
diff --git a/sc/source/filter/xml/xmlimprt.hxx 
b/sc/source/filter/xml/xmlimprt.hxx
index 317fad7..28f9ae9 100644
--- a/sc/source/filter/xml/xmlimprt.hxx
+++ b/sc/source/filter/xml/xmlimprt.hxx
@@ -1223,7 +1223,7 @@ public:
 const OUString& rAttrValue,
 bool bRestrictToExternalNmsp = false ) const;
 
-bool IsFormulaErrorConstant( const OUString& rStr ) const;
+FormulaError GetFormulaErrorConstant( const OUString& rStr ) const;
 
 ScEditEngineDefaulter* GetEditEngine();
 const ScXMLEditAttributeMap& GetEditAttributeMap() const;
com

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

2017-01-06 Thread Miklos Vajna
 filter/source/msfilter/svdfppt.cxx |   21 +
 sd/qa/unit/data/ppt/tdf105150.ppt  |binary
 sd/qa/unit/import-tests.cxx|   14 ++
 3 files changed, 35 insertions(+)

New commits:
commit b9648deb1d99cde1c085767433242d6e1b834e2d
Author: Miklos Vajna 
Date:   Fri Jan 6 17:44:11 2017 +0100

Related: tdf#105150 PPT import: handle mso_fillBackground fill type of 
shapes

Equivalent OOXML markup is . For now only handle the
case when the slide doesn't have an explicit background, and assume that
in that case we can default to white solid fill.

This partially restores the code removed in
6137b5f72f5ec491ea6bd6631a65484fa24d2973 (coverity#735432 Logically dead
code, 2014-03-25), so that ProcessData::aBackgroundColoredObjects is
actually read.

Change-Id: I9e7a9a41965db50638e9d9d1b2361af9fd7a98ca

diff --git a/filter/source/msfilter/svdfppt.cxx 
b/filter/source/msfilter/svdfppt.cxx
index b5e8a53..2fab16a 100644
--- a/filter/source/msfilter/svdfppt.cxx
+++ b/filter/source/msfilter/svdfppt.cxx
@@ -2898,6 +2898,27 @@ void SdrPowerPointImport::ImportPage( SdrPage* pRet, 
const PptSlidePersistEntry*
 break;
 }
 
+// Handle shapes where the fill matches the background
+// fill (mso_fillBackground).
+if (rSlidePersist.ePageKind == PPT_SLIDEPAGE)
+{
+if 
(!aProcessData.aBackgroundColoredObjects.empty())
+{
+if (!rSlidePersist.pBObj)
+{
+for (auto pObject : 
aProcessData.aBackgroundColoredObjects)
+{
+// The shape wants a background, but 
the slide doesn't have
+// one: default to white.
+SfxItemSet 
aNewSet(*pObject->GetMergedItemSet().GetPool());
+
aNewSet.Put(XFillStyleItem(css::drawing::FillStyle_SOLID));
+aNewSet.Put(XFillColorItem(OUString(), 
Color(COL_WHITE)));
+pObject->SetMergedItemSet(aNewSet);
+}
+}
+}
+}
+
 if ( rSlidePersist.pBObj )
 {
 // #i99386# transfer the attributes from the 
temporary BackgroundObject
diff --git a/sd/qa/unit/data/ppt/tdf105150.ppt 
b/sd/qa/unit/data/ppt/tdf105150.ppt
new file mode 100644
index 000..c832371
Binary files /dev/null and b/sd/qa/unit/data/ppt/tdf105150.ppt differ
diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx
index eb56ffb..dd5f2e0 100644
--- a/sd/qa/unit/import-tests.cxx
+++ b/sd/qa/unit/import-tests.cxx
@@ -135,6 +135,7 @@ public:
 void testTdf103477();
 void testTdf104445();
 void testTdf105150();
+void testTdf105150PPT();
 
 CPPUNIT_TEST_SUITE(SdImportTest);
 
@@ -194,6 +195,7 @@ public:
 CPPUNIT_TEST(testTdf103477);
 CPPUNIT_TEST(testTdf104445);
 CPPUNIT_TEST(testTdf105150);
+CPPUNIT_TEST(testTdf105150PPT);
 
 CPPUNIT_TEST_SUITE_END();
 };
@@ -1655,6 +1657,18 @@ void SdImportTest::testTdf105150()
 xDocShRef->DoClose();
 }
 
+void SdImportTest::testTdf105150PPT()
+{
+sd::DrawDocShellRef xDocShRef = 
loadURL(m_directories.getURLFromSrc("sd/qa/unit/data/ppt/tdf105150.ppt"), PPT);
+const SdrPage* pPage = GetPage(1, xDocShRef);
+const SdrObject* pObj = pPage->GetObj(1);
+// This was drawing::FillStyle_NONE, the shape's mso_fillBackground was
+// ignored when the slide didn't have an explicit background fill.
+auto& rFillStyleItem = dynamic_cast(pObj->GetMergedItem(XATTR_FILLSTYLE));
+CPPUNIT_ASSERT_EQUAL(drawing::FillStyle_SOLID, rFillStyleItem.GetValue());
+xDocShRef->DoClose();
+}
+
 void SdImportTest::testTdf104445()
 {
 // Extra bullets were added to the first shape
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - sc/source

2017-01-06 Thread Eike Rathke
 sc/source/core/data/formulacell.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 74c3abeb970015f72a081cbfea59007fd0cbb011
Author: Eike Rathke 
Date:   Fri Jan 6 16:55:32 2017 +0100

out of bounds string access

Change-Id: Iecbe24f90cefe34a447cd52a44967a4b09b4394b
(cherry picked from commit 00f17500520bcb2634146496adc8de9f4d28eea5)
Reviewed-on: https://gerrit.libreoffice.org/32787
Reviewed-by: Eike Rathke 
Tested-by: Jenkins 

diff --git a/sc/source/core/data/formulacell.cxx 
b/sc/source/core/data/formulacell.cxx
index 465f0b8..a3f14b4 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -1355,7 +1355,7 @@ void ScFormulaCell::CompileXML( 
sc::CompileFormulaContext& rCxt, ScProgress& rPr
 {
 if ( !pCode->GetLen() )
 {
-if ( aFormula[0] == '=' )
+if ( !aFormula.isEmpty() && aFormula[0] == '=' )
 pCode->AddBad( aFormula.copy( 1 ) );
 else
 pCode->AddBad( aFormula );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Eike Rathke
 formula/source/core/api/FormulaCompiler.cxx |5 ++--
 sc/source/core/data/formulacell.cxx |4 +++
 sc/source/filter/xml/xmlcelli.cxx   |   34 +++-
 sc/source/filter/xml/xmlimprt.cxx   |6 ++--
 sc/source/filter/xml/xmlimprt.hxx   |2 -
 5 files changed, 39 insertions(+), 12 deletions(-)

New commits:
commit 75d963bc7bb87429f304d29138c27178880c039a
Author: Eike Rathke 
Date:   Fri Jan 6 18:28:42 2017 +0100

recognize the broken "Err:xxx" written by 5.2 and earlier, tdf#105024 
related

... and handle same as "#ERRxxx!" if present.

Change-Id: I1ebb31d628b080c52b450a8fe624c20e9e1188b7

diff --git a/sc/source/filter/xml/xmlcelli.cxx 
b/sc/source/filter/xml/xmlcelli.cxx
index 3b38ac4..8fd7eda 100644
--- a/sc/source/filter/xml/xmlcelli.cxx
+++ b/sc/source/filter/xml/xmlcelli.cxx
@@ -1357,11 +1357,22 @@ void ScXMLTableRowCellContext::PutFormulaCell( const 
ScAddress& rCellPos )
 }
 else
 {
-OUString aFormulaNmsp = maFormula->second;
-if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
-aFormulaNmsp.clear();
-pCode->AssignXMLString( aText, aFormulaNmsp );
-rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+// 5.2 and earlier wrote broken "Err:xxx" as formula to designate
+// an error formula cell.
+if (aText.startsWithIgnoreAsciiCase("Err:") && aText.getLength() 
<= 9 &&
+((nError =
+  GetScImport().GetFormulaErrorConstant( "#ERR" + 
aText.copy(4) + "!")) != FormulaError::NONE))
+{
+pCode->SetCodeError(nError);
+}
+else
+{
+OUString aFormulaNmsp = maFormula->second;
+if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
+aFormulaNmsp.clear();
+pCode->AssignXMLString( aText, aFormulaNmsp );
+rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+}
 }
 
 ScFormulaCell* pNewCell = new ScFormulaCell(pDoc, rCellPos, pCode, 
eGrammar, MM_NONE);
commit 4fcbe16959c839bfacf745cfa554b234e639f794
Author: Eike Rathke 
Date:   Fri Jan 6 17:46:02 2017 +0100

read single error constant formula as such, tdf#105024 related

... without creating a token so when writing again no leading '=' is 
prepended,
with which we can enable 5.2 to read such thing correctly, and when 
re-reading
in 5.3 it also doesn't lead to a "real" formula.

Change-Id: I26fbd20536436b49b781e2bbb5bba1dc6bafbb37

diff --git a/sc/source/core/data/formulacell.cxx 
b/sc/source/core/data/formulacell.cxx
index 9398159..9ad3731 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -1282,6 +1282,10 @@ void ScFormulaCell::CompileXML( 
sc::CompileFormulaContext& rCxt, ScProgress& rPr
 return ;
 }
 
+// Error constant formula cell stays as is.
+if (!pCode->GetLen() && pCode->GetCodeError() != FormulaError::NONE)
+return;
+
 // Compilation changes RPN count, remove and reinsert to FormulaTree if it
 // was in to update its count.
 bool bWasInFormulaTree = pDocument->IsInFormulaTree( this);
diff --git a/sc/source/filter/xml/xmlcelli.cxx 
b/sc/source/filter/xml/xmlcelli.cxx
index ddd8543..3b38ac4 100644
--- a/sc/source/filter/xml/xmlcelli.cxx
+++ b/sc/source/filter/xml/xmlcelli.cxx
@@ -1348,12 +1348,22 @@ void ScXMLTableRowCellContext::PutFormulaCell( const 
ScAddress& rCellPos )
 // temporary formula string as string tokens
 ScTokenArray *pCode = new ScTokenArray();
 
-OUString aFormulaNmsp = maFormula->second;
-if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
-aFormulaNmsp.clear();
-pCode->AssignXMLString( aText, aFormulaNmsp );
+// Check the special case of a single error constant without leading
+// '=' and create an error formula cell without tokens.
+FormulaError nError = GetScImport().GetFormulaErrorConstant(aText);
+if (nError != FormulaError::NONE)
+{
+pCode->SetCodeError(nError);
+}
+else
+{
+OUString aFormulaNmsp = maFormula->second;
+if( eGrammar != formula::FormulaGrammar::GRAM_EXTERNAL )
+aFormulaNmsp.clear();
+pCode->AssignXMLString( aText, aFormulaNmsp );
+rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
+}
 
-rDoc.getDoc().IncXMLImportedFormulaCount( aText.getLength() );
 ScFormulaCell* pNewCell = new ScFormulaCell(pDoc, rCellPos, pCode, 
eGrammar, MM_NONE);
 SetFormulaCell(pNewCell);
 rDoc.setFormulaCell(rCellPos, pNewCell);
@@ -1466,7 +1476,8 @@ bool ScXMLTableRowCellContext::IsPossibleErrorString() 
const
 return false;
 else if(mbNewV

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

2017-01-06 Thread Michael Meeks
 vcl/source/window/window2.cxx |6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit d788d37a16d382d7ff32a19e8d65e5c605ac55dd
Author: Michael Meeks 
Date:   Fri Jan 6 15:23:30 2017 +

tdf#105151 - be more conservative about EndTracking / Tracking calls.

We really shouldn't be emitting these into widgets after they are
disposed - it serves only to confuse things. Restore behaviour to how
it was before the VclPtr change for this.

Change-Id: Ife666929f8aac0e4431a5e43970769692c11888c
Reviewed-on: https://gerrit.libreoffice.org/32785
Tested-by: Jenkins 
Reviewed-by: Michael Meeks 

diff --git a/vcl/source/window/window2.cxx b/vcl/source/window/window2.cxx
index b6f4ea4..1c8be91 100644
--- a/vcl/source/window/window2.cxx
+++ b/vcl/source/window/window2.cxx
@@ -365,7 +365,11 @@ void Window::EndTracking( TrackingEventFlags nFlags )
mpWindowImpl->mpFrameData->mnMouseCode,
mpWindowImpl->mpFrameData->mnMouseCode );
 TrackingEvent   aTEvt( aMEvt, nFlags | TrackingEventFlags::End );
-Tracking( aTEvt );
+// CompatTracking effectively
+if (!mpWindowImpl || mpWindowImpl->mbInDispose)
+return Window::Tracking( aTEvt );
+else
+return Tracking( aTEvt );
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: compilerplugins/clang helpcompiler/source vcl/source

2017-01-06 Thread Stephan Bergmann
 compilerplugins/clang/charrightshift.cxx |   57 +++
 helpcompiler/source/HelpLinker.cxx   |2 -
 vcl/source/fontsubset/cff.cxx|2 -
 3 files changed, 59 insertions(+), 2 deletions(-)

New commits:
commit a7d554f3b24d582919656233476c89a6d74cb207
Author: Stephan Bergmann 
Date:   Fri Jan 6 18:15:24 2017 +0100

New loplugin:charrightshift

Change-Id: Ib645fb11004bc0fe05c9c416ae72b0ae56c23a15

diff --git a/compilerplugins/clang/charrightshift.cxx 
b/compilerplugins/clang/charrightshift.cxx
new file mode 100644
index 000..c1bf951
--- /dev/null
+++ b/compilerplugins/clang/charrightshift.cxx
@@ -0,0 +1,57 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; 
fill-column: 100 -*- */
+/*
+ * 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/.
+ */
+
+#include "check.hxx"
+#include "plugin.hxx"
+
+namespace {
+
+class CharRightShift:
+public RecursiveASTVisitor, public loplugin::Plugin
+{
+public:
+explicit CharRightShift(InstantiationData const & data): Plugin(data) {}
+
+void run() override
+{ TraverseDecl(compiler.getASTContext().getTranslationUnitDecl()); }
+
+bool VisitBinShr(BinaryOperator const * expr) {
+if (ignoreLocation(expr)) {
+return true;
+}
+auto t = expr->getLHS()->IgnoreParenImpCasts()->getType();
+if (!loplugin::TypeCheck(t).Char()) {
+return true;
+}
+if (!expr->getRHS()->getType()->isBuiltinType()) {
+//TODO: in which case the expression should be an
+// CXXOperatorCallExpr instead of a BinaryOperator? but at least
+// recent Clang trunk reports
+//
+//  '(' >> orExpression
+//
+// (connectivity/source/commontools/RowFunctionParser.cxx, the RHS
+// being of type boost::spirit::rule) here
+return true;
+}
+report(
+DiagnosticsEngine::Warning,
+("right shift of %0 is implementation-defined when 'char' is 
signed"
+ " and value is negative"),
+expr->getLHS()->getExprLoc())
+<< t << expr->getSourceRange();
+return true;
+}
+};
+
+loplugin::Plugin::Registration X("charrightshift");
+
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s 
cinkeys+=0=break: */
diff --git a/helpcompiler/source/HelpLinker.cxx 
b/helpcompiler/source/HelpLinker.cxx
index 66ad44b..cfff46a 100644
--- a/helpcompiler/source/HelpLinker.cxx
+++ b/helpcompiler/source/HelpLinker.cxx
@@ -216,7 +216,7 @@ namespace URLEncoder
 result += c;
 else {
 result += '%';
-result += hex[c >> 4];
+result += hex[static_cast(c) >> 4];
 result += hex[c & 0xf];
 }
 }
diff --git a/vcl/source/fontsubset/cff.cxx b/vcl/source/fontsubset/cff.cxx
index cca14ed..a50801f 100644
--- a/vcl/source/fontsubset/cff.cxx
+++ b/vcl/source/fontsubset/cff.cxx
@@ -1777,7 +1777,7 @@ inline void Type1Emitter::emitAllHex()
 char* pOut = aHexBuf;
 while( (p < mpPtr) && (pOut < aHexBuf+sizeof(aHexBuf)-4)) {
 // convert each byte to hex
-char cNibble = (*p >> 4) & 0x0F;
+char cNibble = (static_cast(*p) >> 4) & 0x0F;
 cNibble += (cNibble < 10) ? '0' : 'A'-10;
 *(pOut++) = cNibble;
 cNibble = *(p++) & 0x0F;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Miklos Vajna
 oox/source/ppt/pptshapegroupcontext.cxx |   10 +-
 sd/qa/unit/data/pptx/tdf105150.pptx |binary
 sd/qa/unit/import-tests.cxx |   14 ++
 3 files changed, 23 insertions(+), 1 deletion(-)

New commits:
commit 59339dec1ce56213dc74a06af2f0d35ac1c534d7
Author: Miklos Vajna 
Date:   Fri Jan 6 16:55:22 2017 +0100

tdf#105150 PPTX import: try harder to handle 

The bugdoc has two shapes. The red rectangle is supposed to be covered
by the other shape, but it wasn't as the fill style was set to none. The
reason for that was we ignored useBgFill, unless the slide had an
explicit fill style. Assume that in that case a white solid fill is the
default.

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

diff --git a/oox/source/ppt/pptshapegroupcontext.cxx 
b/oox/source/ppt/pptshapegroupcontext.cxx
index 57f0f912..5208bd4 100644
--- a/oox/source/ppt/pptshapegroupcontext.cxx
+++ b/oox/source/ppt/pptshapegroupcontext.cxx
@@ -101,7 +101,15 @@ ContextHandlerRef PPTShapeGroupContext::onCreateContext( 
sal_Int32 aElementToken
 std::shared_ptr pShape( new PPTShape( meShapeLocation, 
"com.sun.star.drawing.CustomShape" ) );
 if( rAttribs.getBool( XML_useBgFill, false ) )
 {
-const oox::drawingml::FillPropertiesPtr 
pBackgroundPropertiesPtr = mpSlidePersistPtr->getBackgroundProperties();
+oox::drawingml::FillPropertiesPtr pBackgroundPropertiesPtr = 
mpSlidePersistPtr->getBackgroundProperties();
+if (!pBackgroundPropertiesPtr)
+{
+// The shape wants a background, but the slide doesn't have
+// one: default to white.
+pBackgroundPropertiesPtr.reset(new 
oox::drawingml::FillProperties());
+pBackgroundPropertiesPtr->moFillType = XML_solidFill;
+pBackgroundPropertiesPtr->maFillColor.setSrgbClr(0xFF);
+}
 if ( pBackgroundPropertiesPtr ) {
 pShape->getFillProperties().assignUsed( 
*pBackgroundPropertiesPtr );
 }
diff --git a/sd/qa/unit/data/pptx/tdf105150.pptx 
b/sd/qa/unit/data/pptx/tdf105150.pptx
new file mode 100644
index 000..48a2e01
Binary files /dev/null and b/sd/qa/unit/data/pptx/tdf105150.pptx differ
diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx
index f4faf00..eb56ffb 100644
--- a/sd/qa/unit/import-tests.cxx
+++ b/sd/qa/unit/import-tests.cxx
@@ -134,6 +134,7 @@ public:
 void testTdf104201();
 void testTdf103477();
 void testTdf104445();
+void testTdf105150();
 
 CPPUNIT_TEST_SUITE(SdImportTest);
 
@@ -192,6 +193,7 @@ public:
 CPPUNIT_TEST(testTdf104201);
 CPPUNIT_TEST(testTdf103477);
 CPPUNIT_TEST(testTdf104445);
+CPPUNIT_TEST(testTdf105150);
 
 CPPUNIT_TEST_SUITE_END();
 };
@@ -1641,6 +1643,18 @@ void SdImportTest::testTdf103477()
 xDocShRef->DoClose();
 }
 
+void SdImportTest::testTdf105150()
+{
+sd::DrawDocShellRef xDocShRef = 
loadURL(m_directories.getURLFromSrc("sd/qa/unit/data/pptx/tdf105150.pptx"), 
PPTX);
+const SdrPage* pPage = GetPage(1, xDocShRef);
+const SdrObject* pObj = pPage->GetObj(1);
+auto& rFillStyleItem = dynamic_cast(pObj->GetMergedItem(XATTR_FILLSTYLE));
+// This was drawing::FillStyle_NONE,  was ignored when
+// the slide didn't have an explicit background fill.
+CPPUNIT_ASSERT_EQUAL(drawing::FillStyle_SOLID, rFillStyleItem.GetValue());
+xDocShRef->DoClose();
+}
+
 void SdImportTest::testTdf104445()
 {
 // Extra bullets were added to the first shape
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - sc/source

2017-01-06 Thread Caolán McNamara
 sc/source/ui/condformat/condformatdlg.cxx |8 
 sc/source/ui/inc/condformatdlg.hxx|1 +
 2 files changed, 9 insertions(+)

New commits:
commit 9341c166d6b2971e637ba9d17f7be68f5bb852f3
Author: Caolán McNamara 
Date:   Fri Jan 6 16:18:37 2017 +

Resolves: tdf#105061 Only one text box shown in case of 'between'

non-layout ScCondFormatList widget not responding to layout child change

Change-Id: I7edd4fc54de9e71fb86b47a9b182bfc22fa4af25
(cherry picked from commit ac561ab34ea9d26243fe3791f624c440dcf7ae6d)

diff --git a/sc/source/ui/condformat/condformatdlg.cxx 
b/sc/source/ui/condformat/condformatdlg.cxx
index 22bd55a..b6e2191 100644
--- a/sc/source/ui/condformat/condformatdlg.cxx
+++ b/sc/source/ui/condformat/condformatdlg.cxx
@@ -157,6 +157,14 @@ void ScCondFormatList::Resize()
 RecalcAll();
 }
 
+void ScCondFormatList::queue_resize(StateChangedType eReason)
+{
+Control::queue_resize(eReason);
+if (!mpDialogParent) //detects that this is during dispose
+return;
+RecalcAll();
+}
+
 ScConditionalFormat* ScCondFormatList::GetConditionalFormat() const
 {
 if(maEntries.empty())
diff --git a/sc/source/ui/inc/condformatdlg.hxx 
b/sc/source/ui/inc/condformatdlg.hxx
index 62f3758..a79ae0d 100644
--- a/sc/source/ui/inc/condformatdlg.hxx
+++ b/sc/source/ui/inc/condformatdlg.hxx
@@ -66,6 +66,7 @@ public:
 condformat::dialog::ScCondFormatDialogType eType);
 
 virtual Size GetOptimalSize() const override;
+virtual void queue_resize(StateChangedType eReason = 
StateChangedType::Layout) override;
 virtual void Resize() override;
 
 ScConditionalFormat* GetConditionalFormat() const;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Caolán McNamara
 external/breakpad/README  |   22 +-
 sc/source/ui/condformat/condformatdlg.cxx |8 
 sc/source/ui/inc/condformatdlg.hxx|1 +
 3 files changed, 30 insertions(+), 1 deletion(-)

New commits:
commit ac561ab34ea9d26243fe3791f624c440dcf7ae6d
Author: Caolán McNamara 
Date:   Fri Jan 6 16:18:37 2017 +

Resolves: tdf#105061 Only one text box shown in case of 'between'

non-layout ScCondFormatList widget not responding to layout child change

Change-Id: I7edd4fc54de9e71fb86b47a9b182bfc22fa4af25

diff --git a/sc/source/ui/condformat/condformatdlg.cxx 
b/sc/source/ui/condformat/condformatdlg.cxx
index 22bd55a..b6e2191 100644
--- a/sc/source/ui/condformat/condformatdlg.cxx
+++ b/sc/source/ui/condformat/condformatdlg.cxx
@@ -157,6 +157,14 @@ void ScCondFormatList::Resize()
 RecalcAll();
 }
 
+void ScCondFormatList::queue_resize(StateChangedType eReason)
+{
+Control::queue_resize(eReason);
+if (!mpDialogParent) //detects that this is during dispose
+return;
+RecalcAll();
+}
+
 ScConditionalFormat* ScCondFormatList::GetConditionalFormat() const
 {
 if(maEntries.empty())
diff --git a/sc/source/ui/inc/condformatdlg.hxx 
b/sc/source/ui/inc/condformatdlg.hxx
index 62f3758..a79ae0d 100644
--- a/sc/source/ui/inc/condformatdlg.hxx
+++ b/sc/source/ui/inc/condformatdlg.hxx
@@ -66,6 +66,7 @@ public:
 condformat::dialog::ScCondFormatDialogType eType);
 
 virtual Size GetOptimalSize() const override;
+virtual void queue_resize(StateChangedType eReason = 
StateChangedType::Layout) override;
 virtual void Resize() override;
 
 ScConditionalFormat* GetConditionalFormat() const;
commit abf38a157eabe867fb23a85901e149a50e6782cd
Author: Michael Stahl 
Date:   Fri Jan 6 17:15:17 2017 +0100

breakpad: document how to read mini-dumps locally

Change-Id: I888930079fc93c609769ac31a025c410b9adad96

diff --git a/external/breakpad/README b/external/breakpad/README
index 41fbde6..dfd2bf9 100644
--- a/external/breakpad/README
+++ b/external/breakpad/README
@@ -1 +1,21 @@
-Render glTF 3D models.
+Google breakpad crash-reporting library
+
+https://chromium.googlesource.com/breakpad/breakpad
+
+When this is enabled and soffice.bin crashes, a "mini-dump" file is written
+as "instdir/crash/*.dmp".
+
+There is an UI to upload the mini-dump to a TDF server but of course
+that only makes sense if the server has symbols available that match
+the build, which is not the case if you have built LO yourself.
+
+If you want to get the backtrace from local mini-dump files:
+
+* with Visual Studio:
+  1. open the *.dmp file from the Visual Studio IDE File->Open->File
+  2. then click "Debug Native Only"
+
+* otherwise:
+  1. run "make symbols" to extract the debuginfo from the binaries
+  2. run "workdir/UnpackedTarball/breakpad/src/processor/minidump_stackwalk 
foo.dmp workdir/symbols"
+
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Eike Rathke
 sc/source/core/data/formulacell.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 00f17500520bcb2634146496adc8de9f4d28eea5
Author: Eike Rathke 
Date:   Fri Jan 6 16:55:32 2017 +0100

out of bounds string access

Change-Id: Iecbe24f90cefe34a447cd52a44967a4b09b4394b

diff --git a/sc/source/core/data/formulacell.cxx 
b/sc/source/core/data/formulacell.cxx
index 06fc6b1..9398159 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -1355,7 +1355,7 @@ void ScFormulaCell::CompileXML( 
sc::CompileFormulaContext& rCxt, ScProgress& rPr
 {
 if ( !pCode->GetLen() )
 {
-if ( aFormula[0] == '=' )
+if ( !aFormula.isEmpty() && aFormula[0] == '=' )
 pCode->AddBad( aFormula.copy( 1 ) );
 else
 pCode->AddBad( aFormula );
___
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.1' - sw/qa writerfilter/source

2017-01-06 Thread Miklos Vajna
 sw/qa/extras/ooxmlimport/data/tdf105143.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |8 
 writerfilter/source/filter/WriterFilter.cxx  |1 +
 3 files changed, 9 insertions(+)

New commits:
commit 3753e2010d0acd5f4ff25e2cb75a33cb580d310f
Author: Miklos Vajna 
Date:   Fri Jan 6 13:49:06 2017 +0100

tdf#105143 DOCX import: enable DoNotCaptureDrawObjsOnPage layout compat 
option

Because that's what Word does to show only part of the shape in the
bugdoc.

Conflicts:
writerfilter/source/filter/WriterFilter.cxx

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

diff --git a/sw/qa/extras/ooxmlimport/data/tdf105143.docx 
b/sw/qa/extras/ooxmlimport/data/tdf105143.docx
new file mode 100644
index 000..aa4bf40
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf105143.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 097b749..7168d17 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -1740,6 +1740,14 @@ DECLARE_OOXMLIMPORT_TEST(testTdf105127, "tdf105127.docx")
 CPPUNIT_ASSERT_EQUAL(static_cast(3257), 
aPolyPolygon.Coordinates[0][0].Y);
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf105143, "tdf105143.docx")
+{
+OUString aTop = 
parseDump("/root/page/body/txt/anchored/SwAnchoredDrawObject/bounds", "top");
+// This was 6272, i.e. the shape was moved up (incorrect position) to be
+// inside the page rectangle.
+CPPUNIT_ASSERT_EQUAL(OUString("6674"), aTop);
+}
+
 DECLARE_OOXMLIMPORT_TEST(testfdo76583, "fdo76583.docx")
 {
 // The problem was that the floating table was imported as a non-floating 
one.
diff --git a/writerfilter/source/filter/WriterFilter.cxx 
b/writerfilter/source/filter/WriterFilter.cxx
index 1ab28c2..0b95b45 100644
--- a/writerfilter/source/filter/WriterFilter.cxx
+++ b/writerfilter/source/filter/WriterFilter.cxx
@@ -300,6 +300,7 @@ void WriterFilter::setTargetDocument(const uno::Reference< 
lang::XComponent >& x
 xSettings->setPropertyValue("BackgroundParaOverDrawings", 
uno::makeAny(sal_True));
 xSettings->setPropertyValue("TabOverMargin", uno::makeAny(sal_True));
 xSettings->setPropertyValue("PropLineSpacingShrinksFirstLine", 
uno::makeAny(sal_True));
+xSettings->setPropertyValue("DoNotCaptureDrawObjsOnPage", 
uno::makeAny(sal_True));
 }
 
 void WriterFilter::setSourceDocument(const uno::Reference< lang::XComponent >& 
xDoc) throw (lang::IllegalArgumentException, uno::RuntimeException, 
std::exception)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Miklos Vajna
 sw/qa/extras/ooxmlimport/data/tdf105143.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |8 
 writerfilter/source/filter/WriterFilter.cxx  |1 +
 3 files changed, 9 insertions(+)

New commits:
commit af313fc149f80adb0f1680ca20e19745ccb7fede
Author: Miklos Vajna 
Date:   Fri Jan 6 13:49:06 2017 +0100

tdf#105143 DOCX import: enable DoNotCaptureDrawObjsOnPage layout compat 
option

Because that's what Word does to show only part of the shape in the
bugdoc.

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

diff --git a/sw/qa/extras/ooxmlimport/data/tdf105143.docx 
b/sw/qa/extras/ooxmlimport/data/tdf105143.docx
new file mode 100644
index 000..aa4bf40
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf105143.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 0acf8f0..2d1c66b 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -699,6 +699,14 @@ DECLARE_OOXMLIMPORT_TEST(testTdf105127, "tdf105127.docx")
 CPPUNIT_ASSERT_EQUAL(static_cast(3257), 
aPolyPolygon.Coordinates[0][0].Y);
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf105143, "tdf105143.docx")
+{
+OUString aTop = 
parseDump("/root/page/body/txt/anchored/SwAnchoredDrawObject/bounds", "top");
+// This was 6272, i.e. the shape was moved up (incorrect position) to be
+// inside the page rectangle.
+CPPUNIT_ASSERT_EQUAL(OUString("6674"), aTop);
+}
+
 DECLARE_OOXMLIMPORT_TEST(testfdo76583, "fdo76583.docx")
 {
 // The problem was that the floating table was imported as a non-floating 
one.
diff --git a/writerfilter/source/filter/WriterFilter.cxx 
b/writerfilter/source/filter/WriterFilter.cxx
index 3826b09..e247000 100644
--- a/writerfilter/source/filter/WriterFilter.cxx
+++ b/writerfilter/source/filter/WriterFilter.cxx
@@ -298,6 +298,7 @@ void WriterFilter::setTargetDocument(const uno::Reference< 
lang::XComponent >& x
 xSettings->setPropertyValue("TabOverMargin", uno::makeAny(true));
 xSettings->setPropertyValue("TreatSingleColumnBreakAsPageBreak", 
uno::makeAny(true));
 xSettings->setPropertyValue("PropLineSpacingShrinksFirstLine", 
uno::makeAny(true));
+xSettings->setPropertyValue("DoNotCaptureDrawObjsOnPage", 
uno::makeAny(true));
 }
 
 void WriterFilter::setSourceDocument(const uno::Reference< lang::XComponent >& 
xDoc) throw (lang::IllegalArgumentException, uno::RuntimeException, 
std::exception)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - oox/source sw/qa

2017-01-06 Thread Miklos Vajna
 oox/source/vml/vmlshape.cxx  |   22 ++
 sw/qa/extras/ooxmlimport/data/tdf105127.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |7 +++
 3 files changed, 29 insertions(+)

New commits:
commit 86847b02c581e908b5aae1cb13be7dba0e40e180
Author: Miklos Vajna 
Date:   Fri Jan 6 09:42:20 2017 +0100

tdf#105127 VML import: handle 

I don't see an easy way to implement this via UNO, so use the internal
API. As to the internal API usage,
SdrEditView::MirrorMarkedObjVertical() (for UI) and
SvxMSDffManager::ImportShape() (for WW8 import) are example client code.

Change-Id: I9bf27788db32fd35d6b56e0f1a240c4b7abc5604
(cherry picked from commit ac03883cd66a2d58c17c8dac555a053586e46625)
Reviewed-on: https://gerrit.libreoffice.org/32778
Tested-by: Jenkins 
Reviewed-by: Jan Holesovsky 

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index c978c39..0b68675 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -1058,6 +1058,28 @@ Reference< XShape > BezierShape::implConvertAndInsert( 
const Reference< XShapes
 aPropSet.setProperty( PROP_PolyPolygonBezier, aBezierCoords );
 }
 
+// Handle horizontal and vertical flip.
+if (!maTypeModel.maFlip.isEmpty())
+{
+if (SdrObject* pShape = GetSdrObjectFromXShape(xShape))
+{
+if (maTypeModel.maFlip.startsWith("x"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setY(aPoint2.getY() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+if (maTypeModel.maFlip.endsWith("y"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setX(aPoint2.getX() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+}
+}
+
 // Hacky way of ensuring the shape is correctly sized/positioned
 xShape->setSize( awt::Size( rShapeRect.Width, rShapeRect.Height ) );
 xShape->setPosition( awt::Point( rShapeRect.X, rShapeRect.Y ) );
diff --git a/sw/qa/extras/ooxmlimport/data/tdf105127.docx 
b/sw/qa/extras/ooxmlimport/data/tdf105127.docx
new file mode 100644
index 000..3cdfa7f
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf105127.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index b1ccafc..0acf8f0 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -692,6 +692,13 @@ DECLARE_OOXMLIMPORT_TEST(testBnc779620, "bnc779620.docx")
 lcl_countTextFrames( mxComponent, 1 );
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf105127, "tdf105127.docx")
+{
+auto aPolyPolygon = 
getProperty(getShape(1), "PolyPolygonBezier");
+// This was 1910, the shape was rendered upside down.
+CPPUNIT_ASSERT_EQUAL(static_cast(3257), 
aPolyPolygon.Coordinates[0][0].Y);
+}
+
 DECLARE_OOXMLIMPORT_TEST(testfdo76583, "fdo76583.docx")
 {
 // The problem was that the floating table was imported as a non-floating 
one.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Caolán McNamara
 vcl/headless/svpbmp.cxx |   12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

New commits:
commit 600f22fa3e718c36f3d8f47710df1dbbe0d0c4c2
Author: Caolán McNamara 
Date:   Fri Jan 6 13:46:53 2017 +

use same logic in both Creates

Change-Id: I49dab360c5f93e40f5a6e91fbe721935d795472c

diff --git a/vcl/headless/svpbmp.cxx b/vcl/headless/svpbmp.cxx
index e7d367a..d11f3b9 100644
--- a/vcl/headless/svpbmp.cxx
+++ b/vcl/headless/svpbmp.cxx
@@ -121,7 +121,7 @@ BitmapBuffer* ImplCreateDIB(
 pDIB->maPalette.SetEntryCount( nColors );
 }
 
-size_t size = pDIB->mnScanlineSize * pDIB->mnHeight;
+const size_t size = pDIB->mnScanlineSize * pDIB->mnHeight;
 if (size > SAL_MAX_INT32)
 {
 delete pDIB;
@@ -176,10 +176,18 @@ bool SvpSalBitmap::Create(const SalBitmap& rBmp)
 {
 // TODO: reference counting...
 mpDIB = new BitmapBuffer( *rSalBmp.mpDIB );
+
+const size_t size = mpDIB->mnScanlineSize * mpDIB->mnHeight;
+if (size > SAL_MAX_INT32)
+{
+delete mpDIB;
+mpDIB = nullptr;
+return false;
+}
+
 // TODO: get rid of this when BitmapBuffer gets copy constructor
 try
 {
-size_t size = mpDIB->mnScanlineSize * mpDIB->mnHeight;
 mpDIB->mpBits = new sal_uInt8[size];
 std::memcpy(mpDIB->mpBits, rSalBmp.mpDIB->mpBits, size);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: external/unixODBC

2017-01-06 Thread Michael Stahl
 external/unixODBC/inc/odbc/sql.h |   52 +++
 1 file changed, 26 insertions(+), 26 deletions(-)

New commits:
commit 2574757cf026cee4d45195a2f37ecde566411654
Author: Michael Stahl 
Date:   Fri Jan 6 12:49:42 2017 +0100

unixODBC: revert nonsensical trival change 622bb4bc

There is no point to trivial cleanups in external headers.

Change-Id: I9cb7ba135655e1426dcafd3a621ab4eaa7da1011

diff --git a/external/unixODBC/inc/odbc/sql.h b/external/unixODBC/inc/odbc/sql.h
index f426dca..6d1e160 100644
--- a/external/unixODBC/inc/odbc/sql.h
+++ b/external/unixODBC/inc/odbc/sql.h
@@ -468,12 +468,12 @@ extern "C" {
 
 /* SQL_ALTER_TABLE bitmasks */
 #if (ODBCVER >= 0x0200)
-#define SQL_AT_ADD_COLUMN   0x0001
-#define SQL_AT_DROP_COLUMN  0x0002
+#define SQL_AT_ADD_COLUMN   0x0001L
+#define SQL_AT_DROP_COLUMN  0x0002L
 #endif /* ODBCVER >= 0x0200 */
 
 #if (ODBCVER >= 0x0300)
-#define SQL_AT_ADD_CONSTRAINT   0x0008
+#define SQL_AT_ADD_CONSTRAINT   0x0008L
 
 /* The following bitmasks are ODBC extensions and defined in sqlext.h
 *#defineSQL_AT_COLUMN_SINGLE0x0020L
@@ -508,16 +508,16 @@ extern "C" {
 #define SQL_CB_PRESERVE 2
 
 /* SQL_FETCH_DIRECTION bitmasks */
-#define SQL_FD_FETCH_NEXT   0x0001
-#define SQL_FD_FETCH_FIRST  0x0002
-#define SQL_FD_FETCH_LAST   0x0004
-#define SQL_FD_FETCH_PRIOR  0x0008
-#define SQL_FD_FETCH_ABSOLUTE   0x0010
-#define SQL_FD_FETCH_RELATIVE   0x0020
+#define SQL_FD_FETCH_NEXT   0x0001L
+#define SQL_FD_FETCH_FIRST  0x0002L
+#define SQL_FD_FETCH_LAST   0x0004L
+#define SQL_FD_FETCH_PRIOR  0x0008L
+#define SQL_FD_FETCH_ABSOLUTE   0x0010L
+#define SQL_FD_FETCH_RELATIVE   0x0020L
 
 /* SQL_GETDATA_EXTENSIONS bitmasks */
-#define SQL_GD_ANY_COLUMN   0x0001
-#define SQL_GD_ANY_ORDER0x0002
+#define SQL_GD_ANY_COLUMN   0x0001L
+#define SQL_GD_ANY_ORDER0x0002L
 
 /* SQL_IDENTIFIER_CASE values */
 #define SQL_IC_UPPER1
@@ -530,20 +530,20 @@ extern "C" {
 
 
 #if (ODBCVER >= 0x0201)
-#define SQL_OJ_LEFT 0x0001
-#define SQL_OJ_RIGHT0x0002
-#define SQL_OJ_FULL 0x0004
-#define SQL_OJ_NESTED   0x0008
-#define SQL_OJ_NOT_ORDERED  0x0010
-#define SQL_OJ_INNER0x0020
-#define SQL_OJ_ALL_COMPARISON_OPS   0x0040
+#define SQL_OJ_LEFT 0x0001L
+#define SQL_OJ_RIGHT0x0002L
+#define SQL_OJ_FULL 0x0004L
+#define SQL_OJ_NESTED   0x0008L
+#define SQL_OJ_NOT_ORDERED  0x0010L
+#define SQL_OJ_INNER0x0020L
+#define SQL_OJ_ALL_COMPARISON_OPS   0x0040L
 #endif
 
 /* SQL_SCROLL_CONCURRENCY bitmasks */
-#define SQL_SCCO_READ_ONLY  0x0001
-#define SQL_SCCO_LOCK   0x0002
-#define SQL_SCCO_OPT_ROWVER 0x0004
-#define SQL_SCCO_OPT_VALUES 0x0008
+#define SQL_SCCO_READ_ONLY  0x0001L
+#define SQL_SCCO_LOCK   0x0002L
+#define SQL_SCCO_OPT_ROWVER 0x0004L
+#define SQL_SCCO_OPT_VALUES 0x0008L
 
 /* SQL_TXN_CAPABLE values */
 #define SQL_TC_NONE 0
@@ -553,13 +553,13 @@ extern "C" {
 #define SQL_TC_DDL_IGNORE   4
 
 /* SQL_TXN_ISOLATION_OPTION bitmasks */
-#define SQL_TXN_READ_UNCOMMITTED0x0001
+#define SQL_TXN_READ_UNCOMMITTED0x0001L
 #define SQL_TRANSACTION_READ_UNCOMMITTEDSQL_TXN_READ_UNCOMMITTED
-#define SQL_TXN_READ_COMMITTED  0x0002
+#define SQL_TXN_READ_COMMITTED  0x0002L
 #define SQL_TRANSACTION_READ_COMMITTED  SQL_TXN_READ_COMMITTED
-#define SQL_TXN_REPEATABLE_READ 0x0004
+#define SQL_TXN_REPEATABLE_READ 0x0004L
 #define SQL_TRANSACTION_REPEATABLE_READ SQL_TXN_REPEATABLE_READ
-#define SQL_TXN_SERIALIZABLE0x0008
+#define SQL_TXN_SERIALIZABLE0x0008L
 #define SQL_TRANSACTION_SERIALIZABLESQL_TXN_SERIALIZABLE
 
 /* SQL_NULL_COLLATION values */
___
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.1' - oox/source sw/qa

2017-01-06 Thread Miklos Vajna
 oox/source/vml/vmlshape.cxx  |   22 ++
 sw/qa/extras/ooxmlimport/data/tdf105127.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |7 +++
 3 files changed, 29 insertions(+)

New commits:
commit 552052d4fe59e4fc0449b8c419cb40582f5c34dc
Author: Miklos Vajna 
Date:   Fri Jan 6 09:42:20 2017 +0100

tdf#105127 VML import: handle 

I don't see an easy way to implement this via UNO, so use the internal
API. As to the internal API usage,
SdrEditView::MirrorMarkedObjVertical() (for UI) and
SvxMSDffManager::ImportShape() (for WW8 import) are example client code.

Change-Id: I9bf27788db32fd35d6b56e0f1a240c4b7abc5604
(cherry picked from commit ac03883cd66a2d58c17c8dac555a053586e46625)
Reviewed-on: https://gerrit.libreoffice.org/32781
Reviewed-by: Michael Meeks 
Tested-by: Michael Meeks 

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index 7473141..af02bac 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -1074,6 +1074,28 @@ Reference< XShape > BezierShape::implConvertAndInsert( 
const Reference< XShapes
 aPropSet.setProperty( PROP_PolyPolygonBezier, aBezierCoords );
 }
 
+// Handle horizontal and vertical flip.
+if (!maTypeModel.maFlip.isEmpty())
+{
+if (SdrObject* pShape = GetSdrObjectFromXShape(xShape))
+{
+if (maTypeModel.maFlip.startsWith("x"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setY(aPoint2.getY() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+if (maTypeModel.maFlip.endsWith("y"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setX(aPoint2.getX() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+}
+}
+
 // Hacky way of ensuring the shape is correctly sized/positioned
 xShape->setSize( awt::Size( rShapeRect.Width, rShapeRect.Height ) );
 xShape->setPosition( awt::Point( rShapeRect.X, rShapeRect.Y ) );
diff --git a/sw/qa/extras/ooxmlimport/data/tdf105127.docx 
b/sw/qa/extras/ooxmlimport/data/tdf105127.docx
new file mode 100644
index 000..3cdfa7f
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf105127.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 1a457c3..097b749 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -1733,6 +1733,13 @@ DECLARE_OOXMLIMPORT_TEST(testBnc779620, "bnc779620.docx")
 lcl_countTextFrames( mxComponent, 1 );
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf105127, "tdf105127.docx")
+{
+auto aPolyPolygon = 
getProperty(getShape(1), "PolyPolygonBezier");
+// This was 1910, the shape was rendered upside down.
+CPPUNIT_ASSERT_EQUAL(static_cast(3257), 
aPolyPolygon.Coordinates[0][0].Y);
+}
+
 DECLARE_OOXMLIMPORT_TEST(testfdo76583, "fdo76583.docx")
 {
 // The problem was that the floating table was imported as a non-floating 
one.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 connectivity/source/drivers/ado/AConnection.cxx |3 ---
 1 file changed, 3 deletions(-)

New commits:
commit b397c4b8cb9f1a0f2182f1d4b480bf77951aa68f
Author: Stephan Bergmann 
Date:   Fri Jan 6 12:34:15 2017 +0100

bSilent is unused

...ever since c25ec0608a167bcf1d891043f02273761c351701 "initial import"

Change-Id: Iccee867aa9722ced2f66d4d8b8911c5e191502c2

diff --git a/connectivity/source/drivers/ado/AConnection.cxx 
b/connectivity/source/drivers/ado/AConnection.cxx
index d5a2a60..15432c3 100644
--- a/connectivity/source/drivers/ado/AConnection.cxx
+++ b/connectivity/source/drivers/ado/AConnection.cxx
@@ -108,15 +108,12 @@ void OConnection::construct(const OUString& url,const 
Sequence< PropertyValue >&
 aDSN = aDSN.copy(7);
 
 sal_Int32 nTimeout = 20;
-bool bSilent = true;
 const PropertyValue *pIter  = info.getConstArray();
 const PropertyValue *pEnd   = pIter + info.getLength();
 for(;pIter != pEnd;++pIter)
 {
 if(pIter->Name == "Timeout")
 pIter->Value >>= nTimeout;
-else if(pIter->Name == "Silent")
-pIter->Value >>= bSilent;
 else if(pIter->Name == "user")
 pIter->Value >>= aUID;
 else if(pIter->Name == "password")
___
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-2-0' - 4 commits - loleaflet/dist loleaflet/main.js loleaflet/src

2017-01-06 Thread Henry Castro
 loleaflet/dist/toolbar/toolbar.js |   52 ++
 loleaflet/main.js |5 +++
 loleaflet/src/control/Toolbar.js  |9 ++
 3 files changed, 66 insertions(+)

New commits:
commit 02247243163b56478137d7b114bef4d014ea6a1b
Author: Henry Castro 
Date:   Wed Dec 28 13:44:49 2016 -0400

tdf#103642 Page refresh causes unwanted error message

(cherry picked from commit 9c64db3ad1d60c4de659d779a4fba17a298ed648)
Signed-off-by: Andras Timar 

diff --git a/loleaflet/main.js b/loleaflet/main.js
index bd9dd1d..27ca498 100644
--- a/loleaflet/main.js
+++ b/loleaflet/main.js
@@ -114,4 +114,9 @@ map.addControl(L.control.rowHeader());
 map.addControl(L.control.contextMenu());
 map.addControl(L.control.menubar());
 
+window.addEventListener('beforeunload', function () {
+   if (global.map && global.map._socket) {
+   global.map._socket.close();
+   }
+});
 //require('./dist/toolbar/toolbar');
commit 6ca3ec8d6ccec19bf7519d24e989245ae7f160e7
Author: Henry Castro 
Date:   Thu Dec 22 13:41:26 2016 -0400

tdf#103779: Help menu accessibility

(cherry picked from commit 1bede62b132e61fa847aaf066447c2670d14dbab)
Signed-off-by: Andras Timar 

diff --git a/loleaflet/src/control/Toolbar.js b/loleaflet/src/control/Toolbar.js
index 8ad990a..466d698 100644
--- a/loleaflet/src/control/Toolbar.js
+++ b/loleaflet/src/control/Toolbar.js
@@ -215,6 +215,8 @@ L.Map.include({

translatableContent[i].firstChild.nodeValue = 
translatableContent[i].firstChild.nodeValue.toLocaleString();
}
 
+   $('.vex-content').attr('tabindex', -1);
+   $('.vex-content').focus();
// workaround for 
https://github.com/HubSpot/vex/issues/43
$('.vex-overlay').css({ 
'pointer-events': 'none'});
$('.vex').click(function() {
@@ -223,6 +225,9 @@ L.Map.include({
$('.vex-content').click(function(e) {
e.stopPropagation();
});
+   },
+   beforeClose: function () {
+   map.focus();
}
});
});
@@ -245,6 +250,7 @@ L.Map.include({
contentCSS: { width: w + 'px'},
buttons: {},
afterOpen: function($vexContent) {
+   map.enable(false);
// workaround for 
https://github.com/HubSpot/vex/issues/43
$('.vex-overlay').css({ 'pointer-events': 
'none'});
$('.vex').click(function() {
@@ -253,6 +259,9 @@ L.Map.include({
$('.vex-content').click(function(e) {
e.stopPropagation();
});
+   },
+   beforeClose: function () {
+   map.enable(true);
}
});
}
commit 1e12ab5607615ceef1408861771836a838452323
Author: Henry Castro 
Date:   Wed Dec 21 19:50:04 2016 -0400

tdf#103775: Font color/Highlighting button issue

(cherry picked from commit 65bfc63ace567501912e4b6399ff8f81cc62516e)
Signed-off-by: Andras Timar 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index b864ee9..2d4160d 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -407,6 +407,28 @@ $(function () {
if (!L.DomUtil.get('fontcolorindicator')) {
var fontColorIndicator = 
L.DomUtil.create('div', 'font-color-indicator', 
L.DomUtil.get('tb_toolbar-up_item_fontcolor'));
fontColorIndicator.id = 'fontcolorindicator';
+   L.DomEvent.on(fontColorIndicator, 'mouseover', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).addClass('over');
+   });
+   L.DomEvent.on(fontColorIndicator, 'mouseout', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).removeClass('over');
+   });
+   L.DomEvent.on(fontColorIndicator, 'mousedown', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+  

[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - de/091ad1b8abb5d4fb2d7ea1cc2a90b6c1bad236

2017-01-06 Thread Caolán McNamara
 de/091ad1b8abb5d4fb2d7ea1cc2a90b6c1bad236 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 2040ccfd0c6a84b58e5a6f2faa01676d7503e0b5
Author: Caolán McNamara 
Date:   Fri Jan 6 11:06:06 2017 +

Notes added by 'git notes add'

diff --git a/de/091ad1b8abb5d4fb2d7ea1cc2a90b6c1bad236 
b/de/091ad1b8abb5d4fb2d7ea1cc2a90b6c1bad236
new file mode 100644
index 000..e4e62f3
--- /dev/null
+++ b/de/091ad1b8abb5d4fb2d7ea1cc2a90b6c1bad236
@@ -0,0 +1 @@
+prefer: 0e082ac24bfb4e6924539e4814901ce814880737
___
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' - f3/71d243a59ba61e1cb282c2f9363adac52a188e

2017-01-06 Thread Caolán McNamara
 f3/71d243a59ba61e1cb282c2f9363adac52a188e |1 +
 1 file changed, 1 insertion(+)

New commits:
commit cd0ca0f36eede5cb55fc660e63332b138639a1fd
Author: Caolán McNamara 
Date:   Fri Jan 6 11:04:14 2017 +

Notes added by 'git notes add'

diff --git a/f3/71d243a59ba61e1cb282c2f9363adac52a188e 
b/f3/71d243a59ba61e1cb282c2f9363adac52a188e
new file mode 100644
index 000..8ebbe55
--- /dev/null
+++ b/f3/71d243a59ba61e1cb282c2f9363adac52a188e
@@ -0,0 +1 @@
+ignore: obsolete
___
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' - a8/3a5905776f6192e741a222297eeda0819814b4

2017-01-06 Thread Caolán McNamara
 a8/3a5905776f6192e741a97eeda0819814b4 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit dc3ad1e78b43ac50631416ccc404c464e27069ac
Author: Caolán McNamara 
Date:   Fri Jan 6 11:03:50 2017 +

Notes added by 'git notes add'

diff --git a/a8/3a5905776f6192e741a97eeda0819814b4 
b/a8/3a5905776f6192e741a97eeda0819814b4
new file mode 100644
index 000..8ebbe55
--- /dev/null
+++ b/a8/3a5905776f6192e741a97eeda0819814b4
@@ -0,0 +1 @@
+ignore: obsolete
___
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' - 25/8c5336495c330e41bc259481ffa253589ff31c

2017-01-06 Thread Caolán McNamara
 25/8c5336495c330e41bc259481ffa253589ff31c |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 978bdf4ef1081f1ceccbfc4bccfb7179a53adcef
Author: Caolán McNamara 
Date:   Fri Jan 6 11:03:30 2017 +

Notes added by 'git notes add'

diff --git a/25/8c5336495c330e41bc259481ffa253589ff31c 
b/25/8c5336495c330e41bc259481ffa253589ff31c
new file mode 100644
index 000..8ebbe55
--- /dev/null
+++ b/25/8c5336495c330e41bc259481ffa253589ff31c
@@ -0,0 +1 @@
+ignore: obsolete
___
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' - 2d/dbf071356df6dba6954b8c0540793611a60560

2017-01-06 Thread Caolán McNamara
 2d/dbf071356df6dba6954b8c0540793611a60560 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 774f0247c9c015db995f8d3779c727d6b59b422d
Author: Caolán McNamara 
Date:   Fri Jan 6 11:03:09 2017 +

Notes added by 'git notes add'

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


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

2017-01-06 Thread Caolán McNamara
 vcl/source/filter/igif/gifread.cxx |   13 -
 1 file changed, 4 insertions(+), 9 deletions(-)

New commits:
commit 22e1fc5402c17c8459873e621f7630674d2b98f1
Author: Caolán McNamara 
Date:   Fri Jan 6 10:32:56 2017 +

lsan+wmffuzzer shows a circular reference leading to a leak

graphic gets a context set on it which has a shallow copy
of the graphic in it.

==37==ERROR: LeakSanitizer: detected memory leaks

Indirect leak of 1024 byte(s) in 1 object(s) allocated from:
#0 0x6170b0 in operator new[](unsigned long) 
/src/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:84
#1 0x91361b in BitmapPalette::BitmapPalette(unsigned short) 
/src/libreoffice/include/vcl/salbtype.hxx:467:56
#2 0x9029f0 in GIFReader::GIFReader(SvStream&) 
/src/libreoffice/vcl/source/filter/igif/gifread.cxx:114:7
#3 0x91648d in 
std::__1::__libcpp_compressed_pair_imp, 
GIFReader, 1u>::__libcpp_compressed_pair_imp&, 
SvStream&, 0ul, 0ul>(std::__1::piecewise_construct_t, 
std::__1::tuple&>, std::__1::tuple, 
std::__1::__tuple_indices<0ul>, std::__1::__tuple_indices<0ul>) 
/usr/local/bin/../include/c++/v1/memory:2173:15
#4 0x91648d in 
std::__1::__compressed_pair, 
GIFReader>::__compressed_pair&, 
SvStream&>(std::__1::piecewise_construct_t, 
std::__1::tuple&>, std::__1::tuple) 
/usr/local/bin/../include/c++/v1/memory:2330
#5 0x91648d in std::__1::__shared_ptr_emplace 
>::__shared_ptr_emplace(std::__1::allocator, SvStream&) 
/usr/local/bin/../include/c++/v1/memory:3827
#6 0x91648d in std::__1::shared_ptr 
std::__1::shared_ptr::make_shared(SvStream&) 
/usr/local/bin/../include/c++/v1/memory:4443
#7 0x91284b in 
_ZNSt3__111make_sharedI9GIFReaderJR8SvStreamEEENS_9enable_ifIXntsr8is_arrayIT_EE5valueENS_10shared_ptrIS5_EEE4typeEDpOT0_
 /usr/local/bin/../include/c++/v1/memory:4807:12
#8 0x91284b in ImportGIF(SvStream&, Graphic&) 
/src/libreoffice/vcl/source/filter/igif/gifread.cxx:889
#9 0x61c85c in LLVMFuzzerTestOneInput 
/src/libreoffice/vcl/workben/giffuzzer.cxx:18:11
#10 0x575ef58 in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, 
unsigned long) /src/libfuzzer/FuzzerLoop.cpp:546:13
#11 0x575fcb4 in fuzzer::Fuzzer::RunOne(unsigned char const*, unsigned 
long) /src/libfuzzer/FuzzerLoop.cpp:497:3
#12 0x577e43a in fuzzer::Fuzzer::RunOne(std::__1::vector > const&) 
/src/libfuzzer/FuzzerInternal.h:119:41
#13 0x575e5ad in 
fuzzer::Fuzzer::ShuffleAndMinimize(std::__1::vector >, 
std::__1::allocator > > >*) /src/libfuzzer/FuzzerLoop.cpp:476:30
#14 0x5708588 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned 
char const*, unsigned long)) /src/libfuzzer/FuzzerDriver.cpp:534:6
#15 0x56fb3c8 in main /src/libfuzzer/FuzzerMain.cpp:20:10
#16 0x7fb5f13da82f in __libc_start_main 
(/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

Change-Id: I1384f4ced094e79a27e7d15b569c88f129cd115a

diff --git a/vcl/source/filter/igif/gifread.cxx 
b/vcl/source/filter/igif/gifread.cxx
index 4cd2994..3789ebb 100644
--- a/vcl/source/filter/igif/gifread.cxx
+++ b/vcl/source/filter/igif/gifread.cxx
@@ -49,7 +49,6 @@ class SvStream;
 
 class GIFReader : public GraphicReader
 {
-Graphic aImGraphic;
 Animation   aAnimation;
 Bitmap  aBmp8;
 Bitmap  aBmp1;
@@ -103,10 +102,9 @@ class GIFReader : public GraphicReader
 public:
 
 ReadState   ReadGIF( Graphic& rGraphic );
-const Graphic&  GetIntermediateGraphic();
+Graphic GetIntermediateGraphic();
 
 explicitGIFReader( SvStream& rStm );
-virtual ~GIFReader() override;
 };
 
 GIFReader::GIFReader( SvStream& rStm )
@@ -145,11 +143,6 @@ GIFReader::GIFReader( SvStream& rStm )
 ClearImageExtensions();
 }
 
-GIFReader::~GIFReader()
-{
-aImGraphic.SetContext( nullptr );
-}
-
 void GIFReader::ClearImageExtensions()
 {
 nGCDisposalMethod = 0;
@@ -651,8 +644,10 @@ void GIFReader::CreateNewBitmaps()
 }
 }
 
-const Graphic& GIFReader::GetIntermediateGraphic()
+Graphic GIFReader::GetIntermediateGraphic()
 {
+Graphic aImGraphic;
+
 // only create intermediate graphic, if data is available
 // but graphic still not completely read
 if ( bImGraphicReady && !aAnimation.Count() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] help.git: source/text

2017-01-06 Thread Olivier
 source/text/swriter/01/title_page.xhp |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 65af088bf1a712cd234e0e97e2820c44f6327f81
Author: Olivier 
Date:   Fri Jan 6 08:53:44 2017 -0200

Better wording for title pages help page

Change-Id: Idb7643131803708206c04382d7b2520610216ad1
Reviewed-on: https://gerrit.libreoffice.org/32780
Reviewed-by: Olivier Hallot 
Tested-by: Olivier Hallot 

diff --git a/source/text/swriter/01/title_page.xhp 
b/source/text/swriter/01/title_page.xhp
index f442f5f..7be6d9d 100644
--- a/source/text/swriter/01/title_page.xhp
+++ b/source/text/swriter/01/title_page.xhp
@@ -49,7 +49,7 @@
 
 Many documents, such as letters and reports, have a first page 
that is different from the other pages in the document. For example, the first 
page of a letterhead typically has a different header or the first page of a 
report might have no header or footer, while the other pages do. This is simple 
to achieve with %PRODUCTNAME Writer.
 Page header and footer, numbering, margins and orientation are 
some of the properties that belongs to page styles. %PRODUCTNAME Writer let 
your to insert a blank title page at any point of your document, or format an 
existing page like a title page, by inserting a page break followed by a page 
style of your choice or by changing the page style at cursor 
position.
-Note: 
Using the Default (or any other) page style for your document, you can add a 
header/footer as you wish to the first page by deselecting the Same content on first page option on the header/footer 
pages in the Page Style dialog, and then adding the header/footer. 
You can then add a different header/footer to the other pages of the 
document.
+Note: 
Using the Default (or any other) page style for your document, you can add a 
header or footer as you wish to the first page by deselecting the Same content on first page option on the header/footer 
tabs in the Page Style dialog, and then adding the header or 
footer. You can then add a different header or footer to the other pages of the 
document.
 
 To convert the first page of the document into a title 
page
 
@@ -64,7 +64,7 @@
 Select Converting existing pages to title pages 

   
   
-Select the style of the title page in the Edit 
page property area
+Select the style of the title page in the Edit Page 
Properties area
   
 
 Note: By 
default, %PRODUCTNAME selects the First page page 
style.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2017-01-06 Thread Olivier
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0a4c1ed2e8b1f443a12a185b2da5ceea46a1038c
Author: Olivier 
Date:   Fri Jan 6 08:53:44 2017 -0200

Updated core
Project: help  65af088bf1a712cd234e0e97e2820c44f6327f81

Better wording for title pages help page

Change-Id: Idb7643131803708206c04382d7b2520610216ad1
Reviewed-on: https://gerrit.libreoffice.org/32780
Reviewed-by: Olivier Hallot 
Tested-by: Olivier Hallot 

diff --git a/helpcontent2 b/helpcontent2
index d2ff85a..65af088 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit d2ff85a3942838ae72e4d7abced5e94e6d275225
+Subproject commit 65af088bf1a712cd234e0e97e2820c44f6327f81
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: News on Xcode IDE, libre office now loads as 1 project, with modules etc.

2017-01-06 Thread Alexander Thurgood
Le 05/01/2017 à 19:23, jan iversen a écrit :

Hi Jan,

Thanks for your input, I'll try the make gbuildtojson again and send you
the zipped output from workdir/gbuildjson as requested.


Alex


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


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

2017-01-06 Thread Pranav Kant
 libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx |   22 +++-
 1 file changed, 8 insertions(+), 14 deletions(-)

New commits:
commit 29f697711e06b88fb8387518e7f8440cf49f0933
Author: Pranav Kant 
Date:   Fri Jan 6 15:57:43 2017 +0530

gtktiledviewer: Lets avoid using manual buffer sizes

Change-Id: I76e70ccb5b1f40193a1eda1cbca19b3444168dae

diff --git a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx 
b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
index 3382987..3c774c1 100644
--- a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
+++ b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
@@ -375,23 +375,17 @@ gboolean TiledRowColumnBar::docConfigureEvent(GtkWidget* 
pDocView, GdkEventConfi
 gtk_widget_show(rWindow.m_pFormulabarEntry);
 
 // Change horizontal alignment uno commands for spreadsheet
-const char* unoHorAlignArgs =
+const std::string argsPrefix =
 "{"
 "\"HorizontalAlignment\":{"
 "\"type\":\"unsigned short\", "
-"\"value\":\"%d\""
-"}"
-"}";
-
-char unoHorAlignArgsFormatted[strlen(unoHorAlignArgs)];
-snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 1);
-lcl_registerToolItem(rWindow, rWindow.m_pLeftpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
-snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 2);
-lcl_registerToolItem(rWindow, rWindow.m_pCenterpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
-snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 3);
-lcl_registerToolItem(rWindow, rWindow.m_pRightpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
-snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 4);
-lcl_registerToolItem(rWindow, rWindow.m_pJustifypara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
+"\"value\":\"";
+const std::string argsSuffix = "\"}}";
+
+lcl_registerToolItem(rWindow, rWindow.m_pLeftpara, 
".uno:HorizontalAlignment", argsPrefix + std::to_string(1) + argsSuffix);
+lcl_registerToolItem(rWindow, rWindow.m_pCenterpara, 
".uno:HorizontalAlignment", argsPrefix + std::to_string(2) + argsSuffix);
+lcl_registerToolItem(rWindow, rWindow.m_pRightpara, 
".uno:HorizontalAlignment", argsPrefix + std::to_string(3) + argsSuffix);
+lcl_registerToolItem(rWindow, rWindow.m_pJustifypara, 
".uno:HorizontalAlignment", argsPrefix + std::to_string(4) + argsSuffix);
 }
 
 return TRUE;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: canvas/source connectivity/source extensions/source fpicker/source winaccessibility/source

2017-01-06 Thread Stephan Bergmann
 canvas/source/directx/dx_spritecanvas.cxx   |2 +-
 connectivity/source/drivers/ado/AConnection.cxx |2 +-
 extensions/source/ole/unoconversionutilities.hxx|2 +-
 fpicker/source/win32/filepicker/VistaFilePickerImpl.cxx |2 +-
 winaccessibility/source/UAccCOM/MAccessible.cxx |2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit c38aa3b734a0176eedb41d8c2c40f5ac293c11ba
Author: Stephan Bergmann 
Date:   Fri Jan 6 11:19:42 2017 +0100

loplugin:salbool

Change-Id: I7dfd48a04e2dff1391d10acb75180711b403af9b

diff --git a/canvas/source/directx/dx_spritecanvas.cxx 
b/canvas/source/directx/dx_spritecanvas.cxx
index 291c19f..16d7287 100644
--- a/canvas/source/directx/dx_spritecanvas.cxx
+++ b/canvas/source/directx/dx_spritecanvas.cxx
@@ -89,7 +89,7 @@ namespace dxcanvas
 awt::Rectangle aRect;
 maArguments[2] >>= aRect;
 
-sal_Bool bIsFullscreen( false );
+bool bIsFullscreen( false );
 maArguments[3] >>= bIsFullscreen;
 
 // setup helper
diff --git a/connectivity/source/drivers/ado/AConnection.cxx 
b/connectivity/source/drivers/ado/AConnection.cxx
index a61e12d..d5a2a60 100644
--- a/connectivity/source/drivers/ado/AConnection.cxx
+++ b/connectivity/source/drivers/ado/AConnection.cxx
@@ -108,7 +108,7 @@ void OConnection::construct(const OUString& url,const 
Sequence< PropertyValue >&
 aDSN = aDSN.copy(7);
 
 sal_Int32 nTimeout = 20;
-sal_Bool bSilent = true;
+bool bSilent = true;
 const PropertyValue *pIter  = info.getConstArray();
 const PropertyValue *pEnd   = pIter + info.getLength();
 for(;pIter != pEnd;++pIter)
diff --git a/extensions/source/ole/unoconversionutilities.hxx 
b/extensions/source/ole/unoconversionutilities.hxx
index 1462997..430064f 100644
--- a/extensions/source/ole/unoconversionutilities.hxx
+++ b/extensions/source/ole/unoconversionutilities.hxx
@@ -794,7 +794,7 @@ void UnoConversionUtilities::anyToVariant(VARIANT* 
pVariant, const Any& rAny)
 }
 case TypeClass_BOOLEAN:
 {
-sal_Bool value;
+bool value;
 if (rAny >>= value)
 {
 pVariant->vt = VT_BOOL;
diff --git a/fpicker/source/win32/filepicker/VistaFilePickerImpl.cxx 
b/fpicker/source/win32/filepicker/VistaFilePickerImpl.cxx
index e91d3fe..b4c1e9b 100644
--- a/fpicker/source/win32/filepicker/VistaFilePickerImpl.cxx
+++ b/fpicker/source/win32/filepicker/VistaFilePickerImpl.cxx
@@ -1074,7 +1074,7 @@ void VistaFilePickerImpl::impl_sta_SetControlValue(const 
RequestRef& rRequest)
 //case 
css::ui::dialogs::ExtendedFilePickerElementIds::CHECKBOX_PREVIEW : // can be 
ignored ... preview is supported native now !
 case 
css::ui::dialogs::ExtendedFilePickerElementIds::CHECKBOX_SELECTION :
 {
-sal_Bool bValue   = false;
+bool bValue   = false;
aValue >>= bValue;
 iCustom->SetCheckButtonState(nId, bValue);
 }
diff --git a/winaccessibility/source/UAccCOM/MAccessible.cxx 
b/winaccessibility/source/UAccCOM/MAccessible.cxx
index fd495a1..b832c53 100644
--- a/winaccessibility/source/UAccCOM/MAccessible.cxx
+++ b/winaccessibility/source/UAccCOM/MAccessible.cxx
@@ -2678,7 +2678,7 @@ void CMAccessible::get_OLECHARFromAny(Any& pAny, OLECHAR* 
pChar)
 }
 case TypeClass_BOOLEAN:
 {
-sal_Bool val;
+bool val;
 pAny >>= val;
 swprintf( pChar, L"%d", val);
 break;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 desktop/source/app/cmdlinehelp.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 4654e312cfcd84d64bc844c56b3fab8594a76888
Author: Stephan Bergmann 
Date:   Fri Jan 6 11:18:26 2017 +0100

loplugin:nullptr

Change-Id: I279e65c2ec5fa7b3b06f01e7ef4e50ed822fcd03

diff --git a/desktop/source/app/cmdlinehelp.cxx 
b/desktop/source/app/cmdlinehelp.cxx
index 89ef21a..6498ec8 100644
--- a/desktop/source/app/cmdlinehelp.cxx
+++ b/desktop/source/app/cmdlinehelp.cxx
@@ -180,7 +180,7 @@ namespace desktop
 HANDLE hIn = GetStdHandle(STD_INPUT_HANDLE),
 hOut = GetStdHandle(STD_OUTPUT_HANDLE),
 hErr = GetStdHandle(STD_ERROR_HANDLE);
-if (hOut == NULL) // application does not have associated 
standard handles
+if (hOut == nullptr) // application does not have associated 
standard handles
 {
 STARTUPINFOA aStartupInfo;
 aStartupInfo.cb = sizeof(aStartupInfo);
___
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-2-0' - loleaflet/dist

2017-01-06 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   16 +++-
 1 file changed, 11 insertions(+), 5 deletions(-)

New commits:
commit e2ac447b36b70c372ec3e6694ae0f20f09f2e83c
Author: Pranav Kant 
Date:   Fri Jan 6 14:42:27 2017 +0530

loleaflet: Make id of these toolbar items consistent with others

We assume that id of the toolbar item is lowercase(unocommand),
so lets keep it consistent too for alignment buttons.

This also makes alignment buttons toggle except for spreadsheet
where alignment of cell text happens via a different uno command,
HorizontalAlignment.

Change-Id: Ibacf06bd99435cd24b27c7870173a2519b167af1
(cherry picked from commit b22e52aa864c09b02e9e201367d231b442254aef)
Reviewed-on: https://gerrit.libreoffice.org/32776
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 8b04688..d00dbb7 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -367,10 +367,10 @@ $(function () {
{type: 'html',  id: 'backcolor-html', html: ''},
{type: 'button',  id: 'backcolor', img: 'backcolor', 
hint: _('Highlighting')},
{type: 'break'},
-   {type: 'button',  id: 'alignleft',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
-   {type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
-   {type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
-   {type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'button',  id: 'leftpara',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
+   {type: 'button',  id: 'centerpara',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
+   {type: 'button',  id: 'rightpara',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
+   {type: 'button',  id: 'justifypara',  img: 
'alignblock', hint: _('Justified'), uno: 'JustifyPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"4"}}'},
{type: 'break',  id: 'wraptextseparator'},
{type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
{type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
@@ -587,6 +587,11 @@ function toLocalePattern (pattern, regex, text, sub1, 
sub2) {
return text;
 }
 
+function unoCmdToToolbarId(commandname)
+{
+   return commandname.toLowerCase().substr(5);
+}
+
 function selectItem(item, func)
 {
var index = -1;
@@ -880,6 +885,7 @@ map.on('commandstatechanged', function (e) {
var found = false;
var value, color, div;
var matches;
+
if (commandName === '.uno:AssignLayout') {
$('.styles-select').val(state).trigger('change');
} else if (commandName === '.uno:StyleApply') {
@@ -1023,7 +1029,7 @@ map.on('commandstatechanged', function (e) {
$('#PageStatus').html(state ? state : 
' ');
}
 
-   var id = commandName.toLowerCase().substr(5);
+   var id = unoCmdToToolbarId(commandName);
if (typeof formatButtons[id] !== 'undefined') {
if (state === 'true') {
toolbar.enable(id);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 desktop/source/app/cmdlinehelp.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit b12b4fdaf63dd4ae350cf95b5d89a9dbad107abd
Author: Stephan Bergmann 
Date:   Fri Jan 6 11:18:02 2017 +0100

Work around clang-cl -Werror,-Wmissing-field-initializers

...the same way as in other uses of GetStartupInfo

Change-Id: Ieeadf028d15c915df32e055e3725513b2047da4d

diff --git a/desktop/source/app/cmdlinehelp.cxx 
b/desktop/source/app/cmdlinehelp.cxx
index 26ce708..89ef21a 100644
--- a/desktop/source/app/cmdlinehelp.cxx
+++ b/desktop/source/app/cmdlinehelp.cxx
@@ -182,7 +182,8 @@ namespace desktop
 hErr = GetStdHandle(STD_ERROR_HANDLE);
 if (hOut == NULL) // application does not have associated 
standard handles
 {
-STARTUPINFOA aStartupInfo{ sizeof(aStartupInfo) };
+STARTUPINFOA aStartupInfo;
+aStartupInfo.cb = sizeof(aStartupInfo);
 GetStartupInfoA(&aStartupInfo);
 if ((aStartupInfo.dwFlags & STARTF_USESTDHANDLES) == 
STARTF_USESTDHANDLES)
 {
___
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-2-0' - loleaflet/dist

2017-01-06 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   13 +++--
 1 file changed, 7 insertions(+), 6 deletions(-)

New commits:
commit ae5f4db452d3b82fb8b20b5bf0df44e67ac3d50a
Author: Pranav Kant 
Date:   Tue Dec 20 15:53:20 2016 +0530

loleaflet: Separator before 'wrap text' in calc

Change-Id: If1f4536dec24238ff1476e9e7d9ab58207d86495
(cherry picked from commit 00b5ca590bea868f9930e41823138c860c60a845)
Reviewed-on: https://gerrit.libreoffice.org/32775
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index d2f90be..8b04688 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -371,6 +371,7 @@ $(function () {
{type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
{type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
{type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'break',  id: 'wraptextseparator'},
{type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
{type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
{type: 'break',   id: 'break-toggle'},
@@ -831,8 +832,8 @@ map.on('doclayerinit', function () {
]);
break;
case 'text':
-   toolbarUp.remove('wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
-   toolbarUpMore.remove('wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
+   toolbarUp.remove('wraptextseparator', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('wraptextseparator', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'StatePageNumber',
@@ -851,8 +852,8 @@ map.on('doclayerinit', function () {
case 'presentation':
var presentationToolbar = w2ui['presentation-toolbar'];
presentationToolbar.show('presentation', 'presentationbreak', 
'insertpage', 'duplicatepage', 'deletepage');
-   toolbarUp.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
-   toolbarUpMore.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUp.remove('insertannotation', 'wraptextseparator', 
'wraptext', 'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('insertannotation', 'wraptextseparator', 
'wraptext', 'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'PageStatus',
@@ -860,8 

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

2017-01-06 Thread Pranav Kant
 libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx |   19 +++
 1 file changed, 19 insertions(+)

New commits:
commit 6ccd698371a8e88ea06ed0e144171dd1b6236aa7
Author: Pranav Kant 
Date:   Fri Jan 6 14:27:27 2017 +0530

gtktiledviewer: Override alignment buttons uno commands for calc

Use .uno:HorizontalAlignment instead of .uno:XXXPara for calc

Change-Id: I5af4fe62dc3075af4ba779499d062e6d91f21ecf
Reviewed-on: https://gerrit.libreoffice.org/32773
Reviewed-by: pranavk 
Tested-by: pranavk 

diff --git a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx 
b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
index 2f75a65..3382987 100644
--- a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
+++ b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
@@ -373,6 +373,25 @@ gboolean TiledRowColumnBar::docConfigureEvent(GtkWidget* 
pDocView, GdkEventConfi
 gtk_widget_show(rWindow.m_pColumnBar->m_pDrawingArea);
 gtk_widget_queue_draw(rWindow.m_pColumnBar->m_pDrawingArea);
 gtk_widget_show(rWindow.m_pFormulabarEntry);
+
+// Change horizontal alignment uno commands for spreadsheet
+const char* unoHorAlignArgs =
+"{"
+"\"HorizontalAlignment\":{"
+"\"type\":\"unsigned short\", "
+"\"value\":\"%d\""
+"}"
+"}";
+
+char unoHorAlignArgsFormatted[strlen(unoHorAlignArgs)];
+snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 1);
+lcl_registerToolItem(rWindow, rWindow.m_pLeftpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
+snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 2);
+lcl_registerToolItem(rWindow, rWindow.m_pCenterpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
+snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 3);
+lcl_registerToolItem(rWindow, rWindow.m_pRightpara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
+snprintf(unoHorAlignArgsFormatted, sizeof(unoHorAlignArgsFormatted), 
unoHorAlignArgs, 4);
+lcl_registerToolItem(rWindow, rWindow.m_pJustifypara, 
".uno:HorizontalAlignment", std::string(unoHorAlignArgsFormatted));
 }
 
 return TRUE;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Pranav Kant
 libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx |   21 +++-
 1 file changed, 12 insertions(+), 9 deletions(-)

New commits:
commit 3ab6e2783bf7142729924cb95ca730404d0c54e1
Author: Pranav Kant 
Date:   Fri Jan 6 14:26:53 2017 +0530

gtktiledviewer: Support for posting arguments to uno commands

Change-Id: I5db217b525d0f976ef6a7840e370f8f6a21d60cb
Reviewed-on: https://gerrit.libreoffice.org/32772
Reviewed-by: pranavk 
Tested-by: pranavk 

diff --git a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx 
b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
index 27c8e43..2f75a65 100644
--- a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
+++ b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
@@ -121,6 +121,7 @@ public:
 GtkWidget* m_pFormulabarEntry;
 GtkWidget* m_pScrolledWindow;
 std::map m_aToolItemCommandNames;
+std::map m_aToolItemCommandArgs;
 std::map m_aCommandNameToolItems;
 /// Sensitivity (enabled or disabled) or each tool item, ignoring edit
 /// state.
@@ -209,6 +210,14 @@ static std::string getNextAuthor()
 return std::string(pPasswd->pw_gecos) + " #" + std::to_string(++nCounter);
 }
 
+static void lcl_registerToolItem(TiledWindow& rWindow, GtkToolItem* pItem, 
const std::string& rName, const std::string& rArgs = "")
+{
+rWindow.m_aToolItemCommandNames[pItem] = rName;
+rWindow.m_aToolItemCommandArgs[pItem] = rArgs;
+rWindow.m_aCommandNameToolItems[rName] = pItem;
+rWindow.m_aToolItemSensitivities[pItem] = true;
+}
+
 TiledRowColumnBar::TiledRowColumnBar(TiledBarType eType)
 : m_pDrawingArea(gtk_drawing_area_new()),
 m_nSizePixel(0),
@@ -396,13 +405,6 @@ gboolean TiledCornerButton::drawImpl(GtkWidget* 
/*pWidget*/, cairo_t* pCairo)
 return FALSE;
 }
 
-static void lcl_registerToolItem(TiledWindow& rWindow, GtkToolItem* pItem, 
const std::string& rName)
-{
-rWindow.m_aToolItemCommandNames[pItem] = rName;
-rWindow.m_aCommandNameToolItems[rName] = pItem;
-rWindow.m_aToolItemSensitivities[pItem] = true;
-}
-
 const float fZooms[] = { 0.25, 0.5, 0.75, 1.0, 1.5, 2.0, 3.0, 5.0 };
 
 static void iterateUnoParams(GtkWidget* pWidget, gpointer userdata)
@@ -1332,12 +1334,13 @@ static void toggleToolItem(GtkWidget* pWidget, gpointer 
/*pData*/)
 LOKDocView* pLOKDocView = LOK_DOC_VIEW(rWindow.m_pDocView);
 GtkToolItem* pItem = GTK_TOOL_ITEM(pWidget);
 const std::string& rString = rWindow.m_aToolItemCommandNames[pItem];
-g_info("toggleToolItem: lok_doc_view_post_command('%s')", 
rString.c_str());
+const std::string& rArguments = rWindow.m_aToolItemCommandArgs[pItem];
+g_info("toggleToolItem: lok_doc_view_post_command('%s %s')", 
rString.c_str(), rArguments.c_str());
 
 // notify about the finished Save
 gboolean bNotify = (rString == ".uno:Save");
 
-lok_doc_view_post_command(pLOKDocView, rString.c_str(), 
/*pArguments=*/nullptr, bNotify);
+lok_doc_view_post_command(pLOKDocView, rString.c_str(), 
rArguments.c_str(), bNotify);
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Miklos Vajna
 oox/source/vml/vmlshape.cxx  |   22 ++
 sw/qa/extras/ooxmlimport/data/tdf105127.docx |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |7 +++
 3 files changed, 29 insertions(+)

New commits:
commit ac03883cd66a2d58c17c8dac555a053586e46625
Author: Miklos Vajna 
Date:   Fri Jan 6 09:42:20 2017 +0100

tdf#105127 VML import: handle 

I don't see an easy way to implement this via UNO, so use the internal
API. As to the internal API usage,
SdrEditView::MirrorMarkedObjVertical() (for UI) and
SvxMSDffManager::ImportShape() (for WW8 import) are example client code.

Change-Id: I9bf27788db32fd35d6b56e0f1a240c4b7abc5604

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index c978c39..0b68675 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -1058,6 +1058,28 @@ Reference< XShape > BezierShape::implConvertAndInsert( 
const Reference< XShapes
 aPropSet.setProperty( PROP_PolyPolygonBezier, aBezierCoords );
 }
 
+// Handle horizontal and vertical flip.
+if (!maTypeModel.maFlip.isEmpty())
+{
+if (SdrObject* pShape = GetSdrObjectFromXShape(xShape))
+{
+if (maTypeModel.maFlip.startsWith("x"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setY(aPoint2.getY() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+if (maTypeModel.maFlip.endsWith("y"))
+{
+Point aCenter(pShape->GetSnapRect().Center());
+Point aPoint2(aCenter);
+aPoint2.setX(aPoint2.getX() + 1);
+pShape->NbcMirror(aCenter, aPoint2);
+}
+}
+}
+
 // Hacky way of ensuring the shape is correctly sized/positioned
 xShape->setSize( awt::Size( rShapeRect.Width, rShapeRect.Height ) );
 xShape->setPosition( awt::Point( rShapeRect.X, rShapeRect.Y ) );
diff --git a/sw/qa/extras/ooxmlimport/data/tdf105127.docx 
b/sw/qa/extras/ooxmlimport/data/tdf105127.docx
new file mode 100644
index 000..3cdfa7f
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/tdf105127.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index b1ccafc..0acf8f0 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -692,6 +692,13 @@ DECLARE_OOXMLIMPORT_TEST(testBnc779620, "bnc779620.docx")
 lcl_countTextFrames( mxComponent, 1 );
 }
 
+DECLARE_OOXMLIMPORT_TEST(testTdf105127, "tdf105127.docx")
+{
+auto aPolyPolygon = 
getProperty(getShape(1), "PolyPolygonBezier");
+// This was 1910, the shape was rendered upside down.
+CPPUNIT_ASSERT_EQUAL(static_cast(3257), 
aPolyPolygon.Coordinates[0][0].Y);
+}
+
 DECLARE_OOXMLIMPORT_TEST(testfdo76583, "fdo76583.docx")
 {
 // The problem was that the floating table was imported as a non-floating 
one.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Libreoffice service fails in OBS (openSUSE)

2017-01-06 Thread Tomáš Chvátal
2017-01-05 22:11 GMT+01:00 Miklos Vajna :

> Hi,
>
> On Thu, Jan 05, 2017 at 11:11:31AM +0100, Tomáš Chvátal <
> tomas.chva...@gmail.com> wrote:
> > It always reports the same prior the crash, see the attached file.
> > Everytime I trigger it the result is the same.
>
> I meant something like this:
>
> diff --git a/postprocess/qa/services.cxx b/postprocess/qa/services.cxx
> index 6e95c64..ea451c0 100644
> --- a/postprocess/qa/services.cxx
> +++ b/postprocess/qa/services.cxx
> @@ -260,6 +260,9 @@ void Test::test() {
>  == blacklist.end())
>  {
>  if (i.second.constructors.empty()) {
> +std::cout
> +<< "trying to instantiate implementation \""
> +<< i.first << "\"\n";
>  if (i.second.accumulationBased) {
>  createInstance(
>  i.first, false, i.first, i.second.serviceNames,
> &comps);
>
> I'm afraid your log only contains what is the last service that's not
> instantiated before the crash, but that may be irrelevant.
>
>
Ah now i get it. Well it happens always on the same test:

[  649s] trying to instantiate implementation
"com.sun.star.wizards.agenda.CallWizard"
[  649s] unknown:0:(anonymous namespace)::Test::test
[  649s] uncaught exception of type std::bad_alloc
[  649s] - std::bad_alloc

Cheers

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


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

2017-01-06 Thread Stephan Bergmann
 connectivity/source/drivers/odbc/OConnection.cxx |   37 +++
 1 file changed, 12 insertions(+), 25 deletions(-)

New commits:
commit c2b24cd6fbcc70e279b5c4c742911453dafe02a0
Author: Stephan Bergmann 
Date:   Fri Jan 6 10:53:34 2017 +0100

Use string literals directly

Change-Id: Ia4ae5c01ae7eb976a0e9638f8bc8fe7ebab8e2cb

diff --git a/connectivity/source/drivers/odbc/OConnection.cxx 
b/connectivity/source/drivers/odbc/OConnection.cxx
index 66f3146..9e0b3d1 100644
--- a/connectivity/source/drivers/odbc/OConnection.cxx
+++ b/connectivity/source/drivers/odbc/OConnection.cxx
@@ -191,68 +191,55 @@ SQLRETURN OConnection::Construct(const OUString& 
url,const Sequence< PropertyVal
 OUString aDSN("DSN="), aUID, aPWD, aSysDrvSettings;
 aDSN += url.copy(nLen+1);
 
-const char* pUser   = "user";
-const char* pTimeout= "Timeout";
-const char* pSilent = "Silent";
-const char* pPwd= "password";
-const char* pUseCatalog = "UseCatalog";
-const char* pSysDrv = "SystemDriverSettings";
-const char* pCharSet= "CharSet";
-const char* pParaName   = "ParameterNameSubstitution";
-const char* pPrivName   = "IgnoreDriverPrivileges";
-const char* pVerColName = "PreventGetVersionColumns";   // #i60273#
-const char* pRetrieving = "IsAutoRetrievingEnabled";
-const char* pRetriStmt  = "AutoRetrievingStatement";
-
 sal_Int32 nTimeout = 20;
 bool bSilent = true;
 const PropertyValue *pBegin = info.getConstArray();
 const PropertyValue *pEnd   = pBegin + info.getLength();
 for(;pBegin != pEnd;++pBegin)
 {
-if( pBegin->Name.equalsAscii(pTimeout))
+if( pBegin->Name == "Timeout")
 OSL_VERIFY( pBegin->Value >>= nTimeout );
-else if( pBegin->Name.equalsAscii(pSilent))
+else if( pBegin->Name == "Silent")
 OSL_VERIFY( pBegin->Value >>= bSilent );
-else if( pBegin->Name.equalsAscii(pPrivName))
+else if( pBegin->Name == "IgnoreDriverPrivileges")
 OSL_VERIFY( pBegin->Value >>= m_bIgnoreDriverPrivileges );
-else if( pBegin->Name.equalsAscii(pVerColName))
+else if( pBegin->Name == "PreventGetVersionColumns")
 OSL_VERIFY( pBegin->Value >>= m_bPreventGetVersionColumns );
-else if( pBegin->Name.equalsAscii(pParaName))
+else if( pBegin->Name == "ParameterNameSubstitution")
 OSL_VERIFY( pBegin->Value >>= m_bParameterSubstitution );
-else if( pBegin->Name.equalsAscii(pRetrieving))
+else if( pBegin->Name == "IsAutoRetrievingEnabled")
 {
 bool bAutoRetrievingEnabled = false;
 OSL_VERIFY( pBegin->Value >>= bAutoRetrievingEnabled );
 enableAutoRetrievingEnabled(bAutoRetrievingEnabled);
 }
-else if( pBegin->Name.equalsAscii(pRetriStmt))
+else if( pBegin->Name == "AutoRetrievingStatement")
 {
 OUString sGeneratedValueStatement;
 OSL_VERIFY( pBegin->Value >>= sGeneratedValueStatement );
 setAutoRetrievingStatement(sGeneratedValueStatement);
 }
-else if( pBegin->Name.equalsAscii(pUser))
+else if( pBegin->Name == "user")
 {
 OSL_VERIFY( pBegin->Value >>= aUID );
 aDSN = aDSN + ";UID=" + aUID;
 }
-else if( pBegin->Name.equalsAscii(pPwd))
+else if( pBegin->Name == "password")
 {
 OSL_VERIFY( pBegin->Value >>= aPWD );
 aDSN = aDSN + ";PWD=" + aPWD;
 }
-else if( pBegin->Name.equalsAscii(pUseCatalog))
+else if( pBegin->Name == "UseCatalog")
 {
  OSL_VERIFY( pBegin->Value >>= m_bUseCatalog );
 }
-else if( pBegin->Name.equalsAscii(pSysDrv))
+else if( pBegin->Name == "SystemDriverSettings")
 {
 OSL_VERIFY( pBegin->Value >>= aSysDrvSettings );
 aDSN += ";";
 aDSN += aSysDrvSettings;
 }
-else if( pBegin->Name.equalsAscii(pCharSet))
+else if( pBegin->Name == "CharSet")
 {
 OUString sIanaName;
 OSL_VERIFY( pBegin->Value >>= sIanaName );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - solenv/qa

2017-01-06 Thread Christian Lohmaier
 solenv/qa/python/gbuildtoide.py |9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

New commits:
commit ed11f8eb344fb70d17a76f15365f181a662bcc98
Author: Christian Lohmaier 
Date:   Thu Nov 24 11:21:32 2016 +0100

gbuildtoide: builddir is not necessarily same as srcdir

Change-Id: Ied9f9abcc75f2edcd518ac247907f573aa21f35e
Reviewed-on: https://gerrit.libreoffice.org/32405
Reviewed-by: Christian Lohmaier 
Tested-by: Jenkins 

diff --git a/solenv/qa/python/gbuildtoide.py b/solenv/qa/python/gbuildtoide.py
index 8ff87ba..7b2a2c4 100644
--- a/solenv/qa/python/gbuildtoide.py
+++ b/solenv/qa/python/gbuildtoide.py
@@ -98,20 +98,23 @@ class CheckGbuildToIdeModules(unittest.TestCase):
 self.tempwork = tempfile.mkdtemp()
 self.tempsrc = tempfile.mkdtemp()
 self.srcdir = os.environ['SRCDIR']
+self.builddir = os.environ['BUILDDIR']
 if os.environ['OS'] == 'WNT':
 self.tempworkmixed = self.tempwork.replace('\\','/')
 self.tempsrcmixed = self.tempsrc.replace('\\','/')
 self.srcdirnative = self.srcdir.replace('/','\\')
+self.builddirnative = self.builddir.replace('/','\\')
 else:
 self.tempworkmixed = self.tempwork
 self.tempsrcmixed = self.tempsrc
 self.srcdirnative = self.srcdir
-shutil.copyfile(os.path.join(self.srcdirnative, 'config_host.mk'), 
os.path.join(self.tempsrc, 'config_host.mk'))
-shutil.copyfile(os.path.join(self.srcdirnative, 
'config_host_lang.mk'), os.path.join(self.tempsrc, 'config_host_lang.mk'))
+self.builddirnative = self.builddir
+shutil.copyfile(os.path.join(self.builddirnative, 'config_host.mk'), 
os.path.join(self.tempsrc, 'config_host.mk'))
+shutil.copyfile(os.path.join(self.builddirnative, 
'config_host_lang.mk'), os.path.join(self.tempsrc, 'config_host_lang.mk'))
+shutil.copytree(os.path.join(self.builddirnative, 'config_host'),  
os.path.join(self.tempsrc, 'config_host'))
 shutil.copyfile(os.path.join(self.srcdirnative, 'Repository.mk'), 
os.path.join(self.tempsrc, 'Repository.mk'))
 shutil.copyfile(os.path.join(self.srcdirnative, 
'RepositoryExternal.mk'), os.path.join(self.tempsrc, 'RepositoryExternal.mk'))
 shutil.copyfile(os.path.join(self.srcdirnative, 'RepositoryFixes.mk'), 
os.path.join(self.tempsrc, 'RepositoryFixes.mk'))
-shutil.copytree(os.path.join(self.srcdirnative, 'config_host'),  
os.path.join(self.tempsrc, 'config_host'))
 print('copytree from _%s_ to _%s_' % (os.path.join(self.srcdirnative, 
'solenv').replace('\\', '#').replace('/', '!'), os.path.join(self.tempsrc, 
'solenv').replace('\\', '#').replace('/', '!')))
 shutil.copytree(os.path.join(self.srcdirnative, 'solenv'),  
os.path.join(self.tempsrc, 'solenv'))
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Laurent Balland-Poirier
 basic/qa/basic_coverage/test_abs_method.vb 
|   17 ++
 basic/qa/basic_coverage/test_array_method.vb   
|   20 ++
 basic/qa/basic_coverage/test_array_methods.vb  
|   33 ---
 basic/qa/basic_coverage/test_asc_method.vb 
|   15 +
 basic/qa/basic_coverage/test_atn_method.vb 
|   15 +
 basic/qa/basic_coverage/test_beep_method.vb
|   12 +
 basic/qa/basic_coverage/test_cbool_method.vb   
|   16 +
 basic/qa/basic_coverage/test_cbyte_method.vb   
|   16 +
 basic/qa/basic_coverage/test_ccur_method.vb
|   16 +
 basic/qa/basic_coverage/test_cdate_method.vb   
|   16 +
 basic/qa/basic_coverage/test_cdatetofromiso_methods.vb 
|   16 +
 basic/qa/basic_coverage/test_cdatetounodatecdatefromunodate_methods.vb 
|   18 ++
 basic/qa/basic_coverage/test_cdatetounodatetimecdatefromunodatetime_methods.vb 
|   18 ++
 basic/qa/basic_coverage/test_cdatetounotimecdatefromunotime_methods.vb 
|   18 ++
 basic/qa/basic_coverage/test_cdbl_method.vb
|   16 +
 basic/qa/basic_coverage/test_chdircurdir_methods.vb
|   12 +
 basic/qa/basic_coverage/test_choose_method.vb  
|   16 +
 basic/qa/basic_coverage/test_cint_method.vb
|   16 +
 basic/qa/basic_coverage/test_compatibilitymode_method.vb   
|   15 +
 basic/qa/basic_coverage/test_conversion_methods.vb 
|   49 -
 basic/qa/basic_coverage/test_converttofromurl_methods.vb   
|   15 +
 basic/qa/basic_coverage/test_cossin_methods.vb 
|   15 +
 basic/qa/basic_coverage/test_createobject_method.vb
|   21 ++
 basic/qa/basic_coverage/test_createunolistener_method.vb   
|   13 +
 basic/qa/basic_coverage/test_createunoservice_method.vb
|   13 +
 basic/qa/basic_coverage/test_createunostruct_method.vb 
|   13 +
 basic/qa/basic_coverage/test_createunovalue_method.vb  
|   15 +
 basic/qa/basic_coverage/test_csng_method.vb
|   16 +
 basic/qa/basic_coverage/test_cstr_method.vb
|   16 +
 basic/qa/basic_coverage/test_cvar_method.vb
|   16 +
 basic/qa/basic_coverage/test_cverr_method.vb   
|   16 +
 basic/qa/basic_coverage/test_datedateadddatediff_methods.vb
|   18 ++
 basic/qa/basic_coverage/test_datedatepartday_methods.vb
|   18 ++
 basic/qa/basic_coverage/test_datetime_methods.vb   
|   64 ---
 basic/qa/basic_coverage/test_dimarray_method.vb
|   18 ++
 basic/qa/basic_coverage/test_environ_method.vb 
|   12 +
 basic/qa/basic_coverage/test_equalunoobjects_method.vb 
|   25 ++
 basic/qa/basic_coverage/test_erl_method.vb 
|   22 ++
 basic/qa/basic_coverage/test_err_method.vb 
|   22 ++
 basic/qa/basic_coverage/test_falsetrue_method.vb   
|   16 +
 basic/qa/basic_coverage/test_fix_method.vb 
|   15 +
 basic/qa/basic_coverage/test_frac_method.vb
|   15 +
 basic/qa/basic_coverage/test_freefile_method.vb
|   15 +
 basic/qa/basic_coverage/test_freelibrary_method.vb 
|   12 +
 basic/qa/basic_coverage/test_getdefaultcontext_method.vb   
|   12 +
 basic/qa/basic_coverage/test_getdialogzoomfactorx_method.vb
|   15 +
 basic/qa/basic_coverage/test_getdialogzoomfactory_method.vb
|   15 +
 basic/qa/basic_coverage/test_getguitype_method.vb  
|   15 +
 basic/qa/basic_coverage/test_getguiversion_method.vb   
|   15 +
 basic/qa/basic_coverage/test_getpathseparator_method.vb
|   15 +
 basic/qa/basic_coverage/test_getprocessservicemanager_method.vb
|   12 +
 basic/qa/basic_coverage/test_getsolarversion_method.vb 
|   15 +
 basic/qa/basic_coverage/test_getsystemticks_method.vb  
|   15 +
 basic/qa/basic_coverage/test_getsystemtype_method.vb   
|   15 +
 basic/qa/basic_coverage/test_hasunointerfaces_method.vb
|   16 +
 basic/qa/basic_coverage/

[Libreoffice-commits] core.git: writerperfect/qa xmloff/qa

2017-01-06 Thread Miklos Vajna
 writerperfect/qa/unit/DirectoryStreamTest.cxx |2 +-
 writerperfect/qa/unit/WPXSvStreamTest.cxx |   24 
 xmloff/qa/unit/uxmloff.cxx|6 +++---
 3 files changed, 16 insertions(+), 16 deletions(-)

New commits:
commit ff8f5e6255ed7b627fb42ecd9ef18293218f
Author: Miklos Vajna 
Date:   Fri Jan 6 08:33:59 2017 +0100

writerperfect, xmloff: fix loplugin:cppunitassertequals warnings

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

diff --git a/writerperfect/qa/unit/DirectoryStreamTest.cxx 
b/writerperfect/qa/unit/DirectoryStreamTest.cxx
index 79646ed..0b347ac 100644
--- a/writerperfect/qa/unit/DirectoryStreamTest.cxx
+++ b/writerperfect/qa/unit/DirectoryStreamTest.cxx
@@ -139,7 +139,7 @@ void lcl_testDataOperations(RVNGInputStream &rStream)
 CPPUNIT_ASSERT_EQUAL(-1, rStream.seek(0, librevenge::RVNG_SEEK_CUR));
 
 unsigned long numBytesRead = 0;
-CPPUNIT_ASSERT(nullptr == rStream.read(1, numBytesRead));
+CPPUNIT_ASSERT(!rStream.read(1, numBytesRead));
 CPPUNIT_ASSERT_EQUAL(0UL, numBytesRead);
 }
 
diff --git a/writerperfect/qa/unit/WPXSvStreamTest.cxx 
b/writerperfect/qa/unit/WPXSvStreamTest.cxx
index 16d8ebc..70ff441 100644
--- a/writerperfect/qa/unit/WPXSvStreamTest.cxx
+++ b/writerperfect/qa/unit/WPXSvStreamTest.cxx
@@ -197,7 +197,7 @@ void WPXSvStreamTest::testSeekSet()
 CPPUNIT_ASSERT(!pInput->isEnd());
 
 CPPUNIT_ASSERT_EQUAL(0, pInput->seek(nLen, RVNG_SEEK_SET));
-CPPUNIT_ASSERT(nLen == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), nLen);
 CPPUNIT_ASSERT(pInput->isEnd());
 
 // go back to the beginning
@@ -212,7 +212,7 @@ void WPXSvStreamTest::testSeekSet()
 CPPUNIT_ASSERT(!pInput->isEnd());
 
 CPPUNIT_ASSERT(0 != pInput->seek(nLen + 1, RVNG_SEEK_SET));
-CPPUNIT_ASSERT(nLen == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), nLen);
 CPPUNIT_ASSERT(pInput->isEnd());
 }
 
@@ -249,7 +249,7 @@ void WPXSvStreamTest::testSeekCur()
 CPPUNIT_ASSERT(!pInput->isEnd());
 
 CPPUNIT_ASSERT(0 != pInput->seek(nLen + 1, RVNG_SEEK_CUR));
-CPPUNIT_ASSERT(nLen == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), nLen);
 CPPUNIT_ASSERT(pInput->isEnd());
 }
 
@@ -264,11 +264,11 @@ void WPXSvStreamTest::testSeekEnd()
 
 // valid seeks
 CPPUNIT_ASSERT_EQUAL(0, pInput->seek(0, RVNG_SEEK_END));
-CPPUNIT_ASSERT(nLen == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), nLen);
 CPPUNIT_ASSERT(pInput->isEnd());
 
 CPPUNIT_ASSERT_EQUAL(0, pInput->seek(-1, RVNG_SEEK_END));
-CPPUNIT_ASSERT((nLen - 1) == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), (nLen - 1));
 CPPUNIT_ASSERT(!pInput->isEnd());
 
 CPPUNIT_ASSERT_EQUAL(0, pInput->seek(-nLen, RVNG_SEEK_END));
@@ -281,7 +281,7 @@ void WPXSvStreamTest::testSeekEnd()
 
 // invalid seeks
 CPPUNIT_ASSERT(0 != pInput->seek(1, RVNG_SEEK_END));
-CPPUNIT_ASSERT(nLen == pInput->tell());
+CPPUNIT_ASSERT_EQUAL(pInput->tell(), nLen);
 CPPUNIT_ASSERT(pInput->isEnd());
 
 CPPUNIT_ASSERT(0 != pInput->seek(-nLen - 1, RVNG_SEEK_END));
@@ -297,7 +297,7 @@ void WPXSvStreamTest::testStructured()
 assert(bool(pInput));
 
 CPPUNIT_ASSERT(pInput->isStructured());
-CPPUNIT_ASSERT(2 == pInput->subStreamCount());
+CPPUNIT_ASSERT_EQUAL(static_cast(2), 
pInput->subStreamCount());
 lcl_testSubStreams(pInput);
 
 // check for existing substream
@@ -318,7 +318,7 @@ void WPXSvStreamTest::testStructured()
 assert(bool(pInput));
 
 CPPUNIT_ASSERT(pInput->isStructured());
-CPPUNIT_ASSERT(9 == pInput->subStreamCount());
+CPPUNIT_ASSERT_EQUAL(static_cast(9), 
pInput->subStreamCount());
 lcl_testSubStreams(pInput);
 
 // check for existing substream
@@ -338,11 +338,11 @@ void WPXSvStreamTest::testStructured()
 const shared_ptr pInput(lcl_createStream());
 
 CPPUNIT_ASSERT(!pInput->isStructured());
-CPPUNIT_ASSERT(0 == pInput->subStreamCount());
+CPPUNIT_ASSERT_EQUAL(static_cast(0), 
pInput->subStreamCount());
 CPPUNIT_ASSERT(!pInput->existsSubStream("foo"));
-CPPUNIT_ASSERT(nullptr == pInput->getSubStreamByName("foo"));
-CPPUNIT_ASSERT(nullptr == pInput->getSubStreamById(42));
-CPPUNIT_ASSERT(nullptr == pInput->subStreamName(42));
+CPPUNIT_ASSERT(!pInput->getSubStreamByName("foo"));
+CPPUNIT_ASSERT(!pInput->getSubStreamById(42));
+CPPUNIT_ASSERT(!pInput->subStreamName(42));
 }
 }
 
diff --git a/xmloff/qa/unit/uxmloff.cxx b/xmloff/qa/unit/uxmloff.cxx
index 7fc606e..baae178 100644
--- a/xmloff/qa/unit/uxmloff.cxx
+++ b/xmloff/qa/unit/uxmloff.cxx
@@ -78,13 +78,13 @@ void Test::testAutoStylePool()
 // not that interesting but worth checking
 bool bHack = (getenv("LIBO_ONEWAY_STAB

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

2017-01-06 Thread Stephan Bergmann
 sw/source/filter/ww8/rtfsdrexport.cxx |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 032b5090729a19170856851215a2ea8ea8616e81
Author: Stephan Bergmann 
Date:   Fri Jan 6 10:30:55 2017 +0100

No need to check isEmpty() before clearing

Change-Id: I75b077cbc2d100964d4157220561d53c1b235199

diff --git a/sw/source/filter/ww8/rtfsdrexport.cxx 
b/sw/source/filter/ww8/rtfsdrexport.cxx
index 4397dff..d76a1f6 100644
--- a/sw/source/filter/ww8/rtfsdrexport.cxx
+++ b/sw/source/filter/ww8/rtfsdrexport.cxx
@@ -60,8 +60,7 @@ void RtfSdrExport::OpenContainer(sal_uInt16 nEscherContainer, 
int nRecInstance)
 if (nEscherContainer == ESCHER_SpContainer)
 {
 m_nShapeType = ESCHER_ShpInst_Nil;
-if (!m_aShapeStyle.isEmpty())
-m_aShapeStyle.setLength(0);
+m_aShapeStyle.setLength(0);
 m_aShapeStyle.ensureCapacity(200);
 m_aShapeProps.clear();
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   16 +++-
 1 file changed, 11 insertions(+), 5 deletions(-)

New commits:
commit 22b676f088b558d3ea79b14863755a931bd7c919
Author: Pranav Kant 
Date:   Fri Jan 6 14:42:27 2017 +0530

loleaflet: Make id of these toolbar items consistent with others

We assume that id of the toolbar item is lowercase(unocommand),
so lets keep it consistent too for alignment buttons.

This also makes alignment buttons toggle except for spreadsheet
where alignment of cell text happens via a different uno command,
HorizontalAlignment.

Change-Id: Ibacf06bd99435cd24b27c7870173a2519b167af1
Reviewed-on: https://gerrit.libreoffice.org/32774
Reviewed-by: pranavk 
Tested-by: pranavk 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 3ef80bd..4263580 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -383,10 +383,10 @@ $(function () {
{type: 'html',  id: 'backcolor-html', html: ''},
{type: 'button',  id: 'backcolor', img: 'backcolor', 
hint: _('Highlighting')},
{type: 'break'},
-   {type: 'button',  id: 'alignleft',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
-   {type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
-   {type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
-   {type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'button',  id: 'leftpara',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
+   {type: 'button',  id: 'centerpara',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
+   {type: 'button',  id: 'rightpara',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
+   {type: 'button',  id: 'justifypara',  img: 
'alignblock', hint: _('Justified'), uno: 'JustifyPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"4"}}'},
{type: 'break',  id: 'wraptextseparator'},
{type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
{type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
@@ -606,6 +606,11 @@ function toLocalePattern (pattern, regex, text, sub1, 
sub2) {
return text;
 }
 
+function unoCmdToToolbarId(commandname)
+{
+   return commandname.toLowerCase().substr(5);
+}
+
 function selectItem(item, func)
 {
var index = -1;
@@ -898,6 +903,7 @@ map.on('commandstatechanged', function (e) {
var found = false;
var value, color, div;
var matches;
+
if (commandName === '.uno:AssignLayout') {
$('.styles-select').val(state).trigger('change');
} else if (commandName === '.uno:StyleApply') {
@@ -1041,7 +1047,7 @@ map.on('commandstatechanged', function (e) {
$('#PageStatus').html(state ? state : 
' ');
}
 
-   var id = commandName.toLowerCase().substr(5);
+   var id = unoCmdToToolbarId(commandName);
if (typeof formatButtons[id] !== 'undefined') {
if (state === 'true') {
toolbar.enable(id);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2017-01-06 Thread Olivier
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 9e6132b985b7cb8acebe4fdb2322b25931abde3e
Author: Olivier 
Date:   Thu Jan 5 18:56:27 2017 -0200

Updated core
Project: help  d2ff85a3942838ae72e4d7abced5e94e6d275225

Typos and tweaks in Help pages from translation

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

diff --git a/helpcontent2 b/helpcontent2
index 5dab8d0..d2ff85a 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 5dab8d006d59a1187da8afae151c48f24080764f
+Subproject commit d2ff85a3942838ae72e4d7abced5e94e6d275225
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] help.git: source/text

2017-01-06 Thread Olivier
 source/text/scalc/01/04060102.xhp |4 ++--
 source/text/scalc/01/func_aggregate.xhp   |2 +-
 source/text/swriter/classificationbar.xhp |6 +++---
 source/text/swriter/guide/protection.xhp  |2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit d2ff85a3942838ae72e4d7abced5e94e6d275225
Author: Olivier 
Date:   Thu Jan 5 18:56:27 2017 -0200

Typos and tweaks in Help pages from translation

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

diff --git a/source/text/scalc/01/04060102.xhp 
b/source/text/scalc/01/04060102.xhp
index 62550e7..e99aa7b 100644
--- a/source/text/scalc/01/04060102.xhp
+++ b/source/text/scalc/01/04060102.xhp
@@ -165,11 +165,11 @@
 
 
 
- 
 

+  
 
 
 
- 
 

+  
 
 
 
diff --git a/source/text/scalc/01/func_aggregate.xhp 
b/source/text/scalc/01/func_aggregate.xhp
index c934650..5f77dae 100644
--- a/source/text/scalc/01/func_aggregate.xhp
+++ b/source/text/scalc/01/func_aggregate.xhp
@@ -281,7 +281,7 @@
 Array – obligatory argument. The array can be 
specified by the boundaries of the range, the name of the named range or the 
column label.
 For using 
column labels “Automatically find columns and rows labels” function needs 
to be enabled.
 k – obligatory argument for the following 
functions: LARGE, SMALL, PERCENTILE.INC, QUARTILE.INC, PERCENTILE.EXC, 
QUARTILE.EXC. It is a numeric argument, which must correspond to the second 
argument of these functions.
-If the 
k argument is necessary, but not specified, the function returns 
the error Err:511.If the Function and/or Option 
arguments specified not correctly, the function returns the error 
Err:502.
+If the 
k argument is necessary, but not specified, the function returns 
the error Err:511.If the Function and/or Option 
arguments are not specified correctly, the function returns the error 
Err:502.
 
 Examples
 
diff --git a/source/text/swriter/classificationbar.xhp 
b/source/text/swriter/classificationbar.xhp
index 82e929b..9f8fddf 100644
--- a/source/text/swriter/classificationbar.xhp
+++ b/source/text/swriter/classificationbar.xhp
@@ -89,7 +89,7 @@
   classification levels;customizing
 
 
-%PRODUCTNAME allows customization of the levels of 
classification for your business. To customize the number and the name of the 
levels, copy the file example.xml located in Tools -> Options -> LibreOffice -> Paths -> 
Classification into a local folder and edit the contents.
+%PRODUCTNAME allows customization of the levels of 
classification for your business. To customize the number and the name of the 
levels, copy the file example.xml located in Tools - Options - LibreOffice - Paths - Classification 
into a local folder and edit the contents.
 Use the 
file with your %PRODUCTNAME locale in the name as example.
 Save the file and make the classification path above access 
the file.
 Your 
system administrator can place the file in a network folder and make all users 
access the classification settings file.
@@ -109,10 +109,10 @@
 
 
 The Classification bar contains tools to help secure 
document handling.
-The 
Classification toolbar contains listboxes to help in selecting the security of 
the document, according to the BAF category policy 
and BAILS levels. %PRODUCTNAME will add custom 
fields in the document properties (File -> 
Properties, Custom fields tab) to store the classification policy as 
document metadata.
+The 
Classification toolbar contains listboxes to help in selecting the security of 
the document, according to the BAF category policy 
and BAILS levels. %PRODUCTNAME will add custom 
fields in the document properties (File - 
Properties, Custom fields tab) to store the classification policy as 
document metadata.
 
 
-Go 
to menu View -> Toolbars and select Classification
+Go 
to menu View - Toolbars and select Classification
 
 
 
diff --git a/source/text/swriter/guide/protection.xhp 
b/source/text/swriter/guide/protection.xhp
index 7c49682..8c28d5b 100644
--- a/source/text/swriter/guide/protection.xhp
+++ b/source/text/swriter/guide/protection.xhp
@@ -98,7 +98,7 @@
 
 
 Protection of the whole %PRODUCTNAME 
Writer document from changes.
-You can 
protect the contents of %PRODUCTNAME Writer 
document from changes, if it is one form follow file formats: .doc, .docx, 
.odt, .ott.
+You can 
protect the contents of %PRODUCTNAME Writer 
document from changes, if it is one  of the following file formats: .doc, 
.docx, .odt, .ott.
 This 
protection is not intended to be a secure protection. It is just a switch to 
protect the contents against accidental changes in the document.
 To 
enable the protection of the whole document, go to Tools - Options... - 
Writer - Compatibility and choose Protect form.
 To 
disable the protection of the whole document, go to Tools - Options... - 
Writer - Compatibility and uncheck Protect for

[Libreoffice-commits] core.git: Branch 'libreoffice-5-2' - download.lst

2017-01-06 Thread Thorsten Behrens
 download.lst |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 11913e7f3d9ee9e90e6131f67584bb60f67aa577
Author: Thorsten Behrens 
Date:   Fri Jan 6 02:47:23 2017 +0100

libpng: update to release 1.6.28

Change-Id: Ie22e5eb6656ef1d2e7903011ea72a1aa26d4da7c
Reviewed-on: https://gerrit.libreoffice.org/32768
Tested-by: Jenkins 
Reviewed-by: Thorsten Behrens 
Reviewed-on: https://gerrit.libreoffice.org/32769
Reviewed-by: Andras Timar 

diff --git a/download.lst b/download.lst
index 700c4dd..3662491 100644
--- a/download.lst
+++ b/download.lst
@@ -125,8 +125,8 @@ export OWNCLOUD_ANDROID_LIB_TARBALL := 
owncloud-android-library-0.9.4-no-binary-
 export PAGEMAKER_MD5SUM := 5c4985a68be0b79d3f809da5e12b143c
 export PAGEMAKER_TARBALL := libpagemaker-0.0.3.tar.bz2
 export PIXMAN_TARBALL := c63f411b3ad147db2bcce1bf262a0e02-pixman-0.24.4.tar.bz2
-export PNG_MD5SUM := 65213080dd30a9b16193d9b83adc1ee9
-export PNG_TARBALL := libpng-1.6.24.tar.gz
+export PNG_MD5SUM := 897ccec1ebfb0922e83c2bfaa1be8748
+export PNG_TARBALL := libpng-1.6.28.tar.gz
 export POPPLER_MD5SUM := 38c758d84437378ec4f5aae9f875301d
 export POPPLER_TARBALL := poppler-0.46.0.tar.bz2
 export POSTGRESQL_TARBALL := 
c0b4799ea9850eae3ead14f0a60e9418-postgresql-9.2.1.tar.bz2
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 forms/source/xforms/model_ui.cxx |6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)

New commits:
commit 1f56cb61e2b991d2e9c7a639ec3b2f2994f93888
Author: Stephan Bergmann 
Date:   Fri Jan 6 10:01:05 2017 +0100

This apparently doesn't happen

The code has been like this ever since 
ed8c4fd992a4e2e43e65d05e6f06336bfe2fdba2
"INTEGRATION: CWS eforms2", but it doesn't make sense:  It apparently wants 
to
clear XCurrent to leave the for-loop early, but would acutally cause a null
deref instead.  So this branch apparently doesn't get executed routinely.  
Is it
impossible to be reached?  To play it safe, assume no.  But I can't find
anything substantiating the "we'll remove the slash below" claim, so just 
return
an empty string instead.

Change-Id: I18e05385bbef05e7682a2eb4d525cfee25d9a6f5

diff --git a/forms/source/xforms/model_ui.cxx b/forms/source/xforms/model_ui.cxx
index 80442cc..fa4be64 100644
--- a/forms/source/xforms/model_ui.cxx
+++ b/forms/source/xforms/model_ui.cxx
@@ -241,11 +241,7 @@ OUString Model::getDefaultBindingExpressionForNode(
 default:
 // unknown type? fail!
 OSL_FAIL( "unknown node type!" );
-xCurrent.set( nullptr );
-aBuffer.setLength(0);
-// we'll remove the slash below
-aBuffer.insert( 0, '/' );
-break;
+return OUString();
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Pranav Kant
 libreofficekit/source/gtk/lokdocview.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit de6e3d1e87d0e620bcef115605404a76800600a9
Author: Pranav Kant 
Date:   Fri Jan 6 14:33:36 2017 +0530

lokdocview: Handle insert key

Change-Id: Idb5ad17a3a94b9a6fba1113a759207c59acfb7bd

diff --git a/libreofficekit/source/gtk/lokdocview.cxx 
b/libreofficekit/source/gtk/lokdocview.cxx
index cb88bb1..0721080 100644
--- a/libreofficekit/source/gtk/lokdocview.cxx
+++ b/libreofficekit/source/gtk/lokdocview.cxx
@@ -749,6 +749,9 @@ signalKey (GtkWidget* pWidget, GdkEventKey* pEvent)
 case GDK_KEY_Page_Up:
 nKeyCode = com::sun::star::awt::Key::PAGEUP;
 break;
+case GDK_KEY_Insert:
+nKeyCode = com::sun::star::awt::Key::INSERT;
+break;
 case GDK_KEY_Shift_L:
 case GDK_KEY_Shift_R:
 if (pEvent->type == GDK_KEY_PRESS)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 basctl/source/basicide/baside2b.cxx |9 -
 1 file changed, 4 insertions(+), 5 deletions(-)

New commits:
commit 1426eb8a9a8cc150f2330135f28384e5193c6c2b
Author: Stephan Bergmann 
Date:   Fri Jan 6 09:38:06 2017 +0100

Clean up a usage of OUStringBuffer

Change-Id: I05205763502fb20e056cc80f50a37b93232e5743

diff --git a/basctl/source/basicide/baside2b.cxx 
b/basctl/source/basicide/baside2b.cxx
index 0468e12..035d9b0 100644
--- a/basctl/source/basicide/baside2b.cxx
+++ b/basctl/source/basicide/baside2b.cxx
@@ -2620,7 +2620,7 @@ TextView* CodeCompleteListBox::GetParentEditView()
 
 void CodeCompleteListBox::InsertSelectedEntry()
 {
-if( !aFuncBuffer.toString().isEmpty() )
+if( !aFuncBuffer.isEmpty() )
 {
 // if the user typed in something: remove, and insert
 GetParentEditView()->SetSelection( 
pCodeCompleteWindow->pParent->GetLastHighlightPortionTextSelection() );
@@ -2725,7 +2725,7 @@ void CodeCompleteListBox::KeyInput( const KeyEvent& 
rKeyEvt )
 HideAndRestoreFocus();
 break;
 case KEY_BACKSPACE: case KEY_DELETE:
-if( !aFuncBuffer.toString().isEmpty() )
+if( !aFuncBuffer.isEmpty() )
 {
 //if there was something inserted by tab: add it to 
aFuncBuffer
 TextSelection aSel( GetParentEditView()->GetSelection() );
@@ -2736,10 +2736,9 @@ void CodeCompleteListBox::KeyInput( const KeyEvent& 
rKeyEvt )
 
 if( !aTabInsertedStr.isEmpty() && aTabInsertedStr != 
aFuncBuffer.toString() )
 {
-aFuncBuffer.setLength(0);
-aFuncBuffer = aFuncBuffer.append(aTabInsertedStr);
+aFuncBuffer = aTabInsertedStr;
 }
-aFuncBuffer = 
aFuncBuffer.remove(aFuncBuffer.getLength()-1, 1);
+aFuncBuffer.remove(aFuncBuffer.getLength()-1, 1);
 SetMatchingEntries();
 }
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 forms/source/xforms/model_ui.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d289b56c655dae12a26316f40473c46377e18a44
Author: Stephan Bergmann 
Date:   Fri Jan 6 09:37:00 2017 +0100

Use OUString::boolean

Change-Id: I8989fcc18cd0f0b28ef6da5c3c3d7f87429b569b

diff --git a/forms/source/xforms/model_ui.cxx b/forms/source/xforms/model_ui.cxx
index 50cdf82..80442cc 100644
--- a/forms/source/xforms/model_ui.cxx
+++ b/forms/source/xforms/model_ui.cxx
@@ -857,7 +857,7 @@ static OUString lcl_serializeForDisplay( const 
Reference& xResult
 switch( xResult->getObjectType() )
 {
 case XPathObjectType_XPATH_BOOLEAN:
-return xResult->getBoolean() ? OUString("true") : OUString("false");
+return OUString::boolean(xResult->getBoolean());
 
 case XPathObjectType_XPATH_STRING:
 return "\"" + xResult->getString() + "\"";
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-01-06 Thread Stephan Bergmann
 forms/source/xforms/model_ui.cxx |   21 +
 1 file changed, 5 insertions(+), 16 deletions(-)

New commits:
commit 28180c40a6ad3e500156fa660fdf7ec810049629
Author: Stephan Bergmann 
Date:   Fri Jan 6 09:34:45 2017 +0100

No need for an OUStringBuffer here

Change-Id: I01697ac2b84f836ff0b3d15aa8d03042f229eacb

diff --git a/forms/source/xforms/model_ui.cxx b/forms/source/xforms/model_ui.cxx
index 5b75be5..50cdf82 100644
--- a/forms/source/xforms/model_ui.cxx
+++ b/forms/source/xforms/model_ui.cxx
@@ -853,29 +853,20 @@ static OUString lcl_serializeForDisplay( const 
Reference& xResult
 if( ! xResult.is() )
 return getResource( RID_STR_XFORMS_CANT_EVALUATE );
 
-
 // TODO: localize
-OUStringBuffer aBuffer;
-
 switch( xResult->getObjectType() )
 {
 case XPathObjectType_XPATH_BOOLEAN:
-aBuffer.append( xResult->getBoolean()
-? OUString("true")
-: OUString("false") );
-break;
+return xResult->getBoolean() ? OUString("true") : OUString("false");
 
 case XPathObjectType_XPATH_STRING:
-aBuffer.append("\"" + xResult->getString() + "\"");
-break;
+return "\"" + xResult->getString() + "\"";
 
 case XPathObjectType_XPATH_NODESET:
-aBuffer.append( lcl_serializeForDisplay( xResult->getNodeList() ) );
-break;
+return lcl_serializeForDisplay( xResult->getNodeList() );
 
 case XPathObjectType_XPATH_NUMBER:
-aBuffer.append( xResult->getDouble() );
-break;
+return OUString::number(xResult->getDouble());
 
 case XPathObjectType_XPATH_UNDEFINED:
 case XPathObjectType_XPATH_POINT:
@@ -885,10 +876,8 @@ static OUString lcl_serializeForDisplay( const 
Reference& xResult
 case XPathObjectType_XPATH_XSLT_TREE:
 default:
 // TODO: localized error message?
-break;
+return OUString();
 }
-
-return aBuffer.makeStringAndClear();
 }
 
 OUString Model::getResultForExpression(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits