[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - external/libabw

2014-01-21 Thread Fridrich Štrba
 external/libabw/UnpackedTarball_libabw.mk  |1 +
 external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 |   11 +++
 2 files changed, 12 insertions(+)

New commits:
commit 97269ea7b1a16bb0dd692307c903e63888545efa
Author: Fridrich Å trba fridrich.st...@bluewin.ch
Date:   Tue Jan 21 08:44:45 2014 +0100

Coverity: Potential crash (?)

Change-Id: I819b0df446cf95d511d6a7a5a65c381a2a837d00
(cherry picked from commit 7989850e9fc5fb2fb757ecf388b51bc3e96a0267)
Reviewed-on: https://gerrit.libreoffice.org/7562
Reviewed-by: David Tardon dtar...@redhat.com
Tested-by: David Tardon dtar...@redhat.com

diff --git a/external/libabw/UnpackedTarball_libabw.mk 
b/external/libabw/UnpackedTarball_libabw.mk
index c61f2d5..4f8b440 100644
--- a/external/libabw/UnpackedTarball_libabw.mk
+++ b/external/libabw/UnpackedTarball_libabw.mk
@@ -13,6 +13,7 @@ $(eval $(call 
gb_UnpackedTarball_set_tarball,libabw,$(ABW_TARBALL)))
 
 $(eval $(call gb_UnpackedTarball_add_patches,libabw,\
external/libabw/libabw-0.0.1-inttypes.patch.1 \
+   external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 
b/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1
new file mode 100644
index 000..fccaa8e
--- /dev/null
+++ b/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1
@@ -0,0 +1,11 @@
+--- a/src/lib/ABWContentCollector.cpp
 b/src/lib/ABWContentCollector.cpp
+@@ -643,7 +643,7 @@ void 
libabw::ABWContentCollector::collectHeaderFooter(const char *id, const char
+   if (!type)
+ m_ps-m_currentHeaderFooterId = -1;
+ 
+-  std::string sType(type);
++  std::string sType(type ? type : );
+   boost::trim(sType);
+   std::vectorstd::string strVec;
+   boost::algorithm::split(strVec, sType, boost::is_any_of(-), 
boost::token_compress_on);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-1' - sc/source

2014-01-21 Thread Eike Rathke
 sc/source/filter/xml/xmlexprt.cxx |8 +---
 1 file changed, 5 insertions(+), 3 deletions(-)

New commits:
commit 70c61a549184cf8611f7890ac31b0ba4c460cdec
Author: Eike Rathke er...@redhat.com
Date:   Mon Jan 20 23:15:14 2014 +0100

resolved fdo#70275 write correct office:string-value if text format applied

Change-Id: I52e5135139fdf88abe380b6e5ae60e2afb582289
Reviewed-on: https://gerrit.libreoffice.org/7555
Reviewed-by: David Tardon dtar...@redhat.com
Tested-by: David Tardon dtar...@redhat.com

diff --git a/sc/source/filter/xml/xmlexprt.cxx 
b/sc/source/filter/xml/xmlexprt.cxx
index 4f97260..fe0a33b 100644
--- a/sc/source/filter/xml/xmlexprt.cxx
+++ b/sc/source/filter/xml/xmlexprt.cxx
@@ -2882,13 +2882,15 @@ void ScXMLExport::WriteCell(ScMyCell aCell, sal_Int32 
nEqualCellCount)
 break;
 case table::CellContentType_TEXT :
 {
-GetCellText(aCell, aCellPos);
 OUString sFormula(lcl_GetRawString(pDoc, aCellPos));
+if (aCell.maBaseCell.isEmpty())
+aCell.maBaseCell.assign( *pDoc, aCellPos);
+OUString sCellString = aCell.maBaseCell.getString(pDoc);
 
GetNumberFormatAttributesExportHelper()-SetNumberFormatAttributes(
-sFormula, aCell.sStringValue, true, true);
+sCellString, sFormula, true, true);
 if( getDefaultVersion()  SvtSaveOptions::ODFVER_012 )
 
GetNumberFormatAttributesExportHelper()-SetNumberFormatAttributes(
-sFormula, aCell.sStringValue, false, true, 
XML_NAMESPACE_CALC_EXT);
+sCellString, sFormula, false, true, 
XML_NAMESPACE_CALC_EXT);
 }
 break;
 case table::CellContentType_FORMULA :
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Eike Rathke
 sc/source/core/tool/cellform.cxx |   10 ++
 1 file changed, 6 insertions(+), 4 deletions(-)

New commits:
commit a8de4627134ed8bf79a50c78a39f42fbc3be1051
Author: Eike Rathke er...@redhat.com
Date:   Mon Jan 20 19:21:55 2014 +0100

resolved fdo#73836 do not apply format twice to string

ScDocument::GetString() already returns the formatted string that then
was formatted through number formatter. Format raw string instead.

Change-Id: I806b4fc904d04169a6c1dbd4a3d00e2556eda7bd
(cherry picked from commit f6b9b73a1f87d1815ed8ab8f2494e5526a710bb9)
Reviewed-on: https://gerrit.libreoffice.org/7550
Reviewed-by: Kohei Yoshida libreoff...@kohei.us
Tested-by: Kohei Yoshida libreoff...@kohei.us
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com
Reviewed-by: David Tardon dtar...@redhat.com

diff --git a/sc/source/core/tool/cellform.cxx b/sc/source/core/tool/cellform.cxx
index 434043f..6a23ba2 100644
--- a/sc/source/core/tool/cellform.cxx
+++ b/sc/source/core/tool/cellform.cxx
@@ -144,14 +144,16 @@ OUString ScCellFormat::GetString(
 {
 case CELLTYPE_STRING:
 {
-OUString aCellString = rDoc.GetString(rPos);
-rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor, 
bUseStarFormat);
+ScRefCellValue aCell;
+aCell.assign(rDoc, rPos);
+rFormatter.GetOutputString(aCell.mpString-getString(), nFormat, 
aString, ppColor, bUseStarFormat);
 }
 break;
 case CELLTYPE_EDIT:
 {
-OUString aCellString = rDoc.GetString(rPos);
-rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor);
+ScRefCellValue aCell;
+aCell.assign(rDoc, rPos);
+rFormatter.GetOutputString(aCell.getString(rDoc), nFormat, 
aString, ppColor);
 }
 break;
 case CELLTYPE_VALUE:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Tor Lillqvist
 sd/source/ui/remotecontrol/AvahiNetworkService.cxx |   32 ++---
 1 file changed, 16 insertions(+), 16 deletions(-)

New commits:
commit 9d4a40f108e24422dc7aad60f4d17bff08449e90
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 10:00:48 2014 +0200

Use SAL_WARN instead of printing to stderr

Change-Id: I83f2b1d7fbe940917ef2c0b7be225c722fe9df05

diff --git a/sd/source/ui/remotecontrol/AvahiNetworkService.cxx 
b/sd/source/ui/remotecontrol/AvahiNetworkService.cxx
index 5b5fadf..d4d92cd 100644
--- a/sd/source/ui/remotecontrol/AvahiNetworkService.cxx
+++ b/sd/source/ui/remotecontrol/AvahiNetworkService.cxx
@@ -66,7 +66,7 @@ static void entry_group_callback(AvahiEntryGroup *g, 
AvahiEntryGroupState state,
 
 case AVAHI_ENTRY_GROUP_FAILURE :
 
-fprintf(stderr, Entry group failure: %s\n, 
avahi_strerror(avahi_client_errno(avahi_entry_group_get_client(g;
+SAL_WARN(sdremote.wifi, Entry group failure:   
avahi_strerror(avahi_client_errno(avahi_entry_group_get_client(g;
 
 /* Some kind of failure happened while we were registering our 
services */
 avahi_threaded_poll_quit(threaded_poll);
@@ -86,7 +86,7 @@ static void create_services(AvahiClient *c) {
 
 if (!group)
 if (!(group = avahi_entry_group_new(c, entry_group_callback, NULL))) {
-fprintf(stderr, avahi_entry_group_new() failed: %s\n, 
avahi_strerror(avahi_client_errno(c)));
+SAL_WARN(sdremote.wifi, avahi_entry_group_new() failed:   
avahi_strerror(avahi_client_errno(c)));
 avahiService-clear();
 }
 
@@ -109,7 +109,7 @@ static void create_services(AvahiClient *c) {
 char *n = 
avahi_alternative_service_name(avahiService-getName().c_str());
 avahiService-setName(n);
 
-fprintf(stderr, Service name collision, renaming service to 
'%s'\n, avahiService-getName().c_str());
+SAL_WARN(sdremote.wifi, Service name collision, renaming 
service to '  avahiService-getName()  ');
 
 avahi_entry_group_reset(group);
 
@@ -117,13 +117,13 @@ static void create_services(AvahiClient *c) {
 return;
 }
 
-fprintf(stderr, Failed to add _impressremote._tcp service: %s\n, 
avahi_strerror(ret));
+SAL_WARN(sdremote.wifi, Failed to add _impressremote._tcp 
service:   avahi_strerror(ret));
 avahiService-clear();
 }
 
 /* Tell the server to register the service */
 if ((ret = avahi_entry_group_commit(group))  0) {
-fprintf(stderr, Failed to commit entry group: %s\n, 
avahi_strerror(ret));
+SAL_WARN(sdremote.wifi, Failed to commit entry group:   
avahi_strerror(ret));
 avahiService-clear();
 }
 }
@@ -141,7 +141,7 @@ static void client_callback(AvahiClient *c, 
AvahiClientState state, AVAHI_GCC_UN
 create_services(c);
 break;
 case AVAHI_CLIENT_FAILURE:
-fprintf(stderr, Client failure: %s\n, 
avahi_strerror(avahi_client_errno(c)));
+SAL_WARN(sdremote.wifi, Client failure:   
avahi_strerror(avahi_client_errno(c)));
 avahiService-clear();
 break;
 case AVAHI_CLIENT_S_COLLISION:
@@ -165,27 +165,27 @@ void AvahiNetworkService::setup() {
int error = 0;
avahiService = this;
if (!(threaded_poll = avahi_threaded_poll_new())) {
- fprintf(stderr, avahi_threaded_poll_new '%s' failed.\n, avahi_strdup( 
avahiService-getName().c_str() ));
- return;
+   SAL_WARN(sdremote.wifi, avahi_threaded_poll_new '  
avahiService-getName()  ' failed);
+   return;
}
 
if (!(client = avahi_client_new(avahi_threaded_poll_get(threaded_poll), 
static_castAvahiClientFlags(0), client_callback, NULL, error))) {
- fprintf(stderr, avahi_client_new failed.\n);
- return;
+   SAL_WARN(sdremote.wifi, avahi_client_new failed);
+   return;
}
 
create_services(client);
 
/* Finally, start the event loop thread */
if (avahi_threaded_poll_start(threaded_poll)  0) {
- fprintf(stderr, avahi_threaded_poll_start failed.\n);
- return;
+   SAL_WARN(sdremote.wifi, avahi_threaded_poll_start failed);
+   return;
}
 }
 
 void AvahiNetworkService::clear() {
-  /* Call this when the app shuts down */
-  avahi_threaded_poll_stop(threaded_poll);
-  avahi_client_free(client);
-  avahi_threaded_poll_free(threaded_poll);
+/* Call this when the app shuts down */
+avahi_threaded_poll_stop(threaded_poll);
+avahi_client_free(client);
+avahi_threaded_poll_free(threaded_poll);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Zolnai Tamás
 sw/qa/extras/ooxmlimport/data/fdo73389.docx  |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |7 
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   23 +--
 3 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit 7e31667cd6d83d76b641462118ae05ec3c19f4b5
Author: Zolnai Tamás tamas.zol...@collabora.com
Date:   Mon Jan 20 13:44:46 2014 +0100

fdo#73389 Writer does not show a docx-document with nested table correctly

The table manager can work with more table simultaneously
and so m_aCellWidths contains more table's properties.Only one
item of it belongs to the current table (getCurrentCellwidths).
Regression from 74c5ed19f430327988194cdcd6bdff09591a93fa

(cherry picked from commit d0c383256ef72d5212d8e2db77582d0ebe417209)

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

Change-Id: I93efac0c004af1b2524c955ffb20c3ecd74a2920
Reviewed-on: https://gerrit.libreoffice.org/7543
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlimport/data/fdo73389.docx 
b/sw/qa/extras/ooxmlimport/data/fdo73389.docx
new file mode 100644
index 000..02b55f7
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/fdo73389.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 6d44d45..e9c5ad0 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -1563,6 +1563,13 @@ DECLARE_OOXMLIMPORT_TEST(testRPrChangeClosed, 
rprchange_closed.docx)
 CPPUNIT_ASSERT_EQUAL(false, hasProperty(getRun(getParagraph(2), 1), 
RedlineType));
 }
 
+DECLARE_OOXMLIMPORT_TEST(testFdo73389,fdo73389.docx)
+{
+uno::Referencetext::XTextTablesSupplier xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Referencecontainer::XIndexAccess 
xTables(xTablesSupplier-getTextTables(), uno::UNO_QUERY);
+// This was 9340, i.e. the width of the inner table was too large.
+CPPUNIT_ASSERT_EQUAL(sal_Int32(2842), 
getPropertysal_Int32(xTables-getByIndex(0), Width));
+}
 #endif
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index c4b24a7..cc7d119 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -142,24 +142,17 @@ bool DomainMapperTableManager::sprm(Sprm  rSprm)
 */
 bool bFixed = true;
 sal_Int32 nRowFixedWidth = 0;
-if (!m_aCellWidths.empty())
+IntVectorPtr pCellWidths = getCurrentCellWidths();
+// Step 1. Check whether all cells have fixed 
widths in the given row of table.
+for (std::vectorsal_Int32::const_iterator 
aValIter = pCellWidths-begin(); aValIter != pCellWidths-end(); ++aValIter)
 {
-// Step 1. Check whether all cells have fixed 
widths in the given row of table.
-::std::vector IntVectorPtr ::iterator itr;
-for (itr = m_aCellWidths.begin(); itr != 
m_aCellWidths.end(); ++itr )
+if (*aValIter == -1)
 {
-IntVectorPtr itrVal = (*itr);
-for 
(std::vectorsal_Int32::const_iterator aValIter = itrVal-begin(); aValIter != 
itrVal-end(); ++aValIter)
-{
-if (*aValIter == -1)
-{
-bFixed = false;
-break;
-}
-// Sum the width of cells to find the 
total width of given row
-nRowFixedWidth += (*aValIter);
-}
+bFixed = false;
+break;
 }
+// Sum the width of cells to find the total 
width of given row
+nRowFixedWidth += (*aValIter);
 }
 
 // Check whether the total width of given row is 
compared with the maximum value of rows (m_nMaxFixedWidth).
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 37/71f2a8da98fd1ef2ac40a18a2c104c3087b327

2014-01-21 Thread Caolán McNamara
 37/71f2a8da98fd1ef2ac40a18a2c104c3087b327 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit ebcee0f5d997d9a959e2cb030dadf4fdbfb74d2a
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 09:18:42 2014 +

Notes added by 'git notes add'

diff --git a/37/71f2a8da98fd1ef2ac40a18a2c104c3087b327 
b/37/71f2a8da98fd1ef2ac40a18a2c104c3087b327
new file mode 100644
index 000..3c68f67
--- /dev/null
+++ b/37/71f2a8da98fd1ef2ac40a18a2c104c3087b327
@@ -0,0 +1 @@
+prefer: 660d761bfeea81c0b76cf08b4c961a2ad8d09a06
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 33/d4279ad9ea0b51baa56430b5e4ba3a8a6ec70c

2014-01-21 Thread Caolán McNamara
 33/d4279ad9ea0b51baa56430b5e4ba3a8a6ec70c |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 1edb76154fd734ec71fdcea6b596b90211ea52b8
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 09:20:18 2014 +

Notes added by 'git notes add'

diff --git a/33/d4279ad9ea0b51baa56430b5e4ba3a8a6ec70c 
b/33/d4279ad9ea0b51baa56430b5e4ba3a8a6ec70c
new file mode 100644
index 000..f2971b9
--- /dev/null
+++ b/33/d4279ad9ea0b51baa56430b5e4ba3a8a6ec70c
@@ -0,0 +1 @@
+prefer: c8e9038b13c64df430fb4e3fecac65a3c507e3e8
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Michael Stahl
 drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 1d2ef1adfbee6ac1fe1fa2ba369d2cbd28dd8246
Author: Michael Stahl mst...@redhat.com
Date:   Fri Jan 17 15:17:55 2014 +0100

fdo#72530: drawinglayer: fix PDF export of lossless picture

Don't pass empty Rectangle to PDFExtOutDevData::EndGroup(), it will
result in early return from PDFWriterImpl::drawJPGBitmap() and no
picture in the PDF.  Passing in the same Rectangle for the 3rd and 4th
parameter will avoid cropping in PDFExtOutDevData.

(regression from 677df250becd5cde71c6310f647e78ac3763347e)

Change-Id: Icf110fe3e635319419fcacc7bc684c792a34d6ad
(cherry picked from commit c8e9038b13c64df430fb4e3fecac65a3c507e3e8)
Reviewed-on: https://gerrit.libreoffice.org/7502
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit 9c569b326c9d2d71132e722a1783e7e201c1bf3c)
Reviewed-on: https://gerrit.libreoffice.org/7503
Reviewed-by: Eike Rathke er...@redhat.com
Reviewed-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com

diff --git a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx 
b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
index f6aad73..94fc972 100644
--- a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
+++ b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
@@ -824,7 +824,8 @@ namespace drawinglayer
 sal_Int32(floor(aCurrentRange.getMinX())), 
sal_Int32(floor(aCurrentRange.getMinY())),
 sal_Int32(ceil(aCurrentRange.getMaxX())), 
sal_Int32(ceil(aCurrentRange.getMaxY(;
 const GraphicAttr rAttr = 
rGraphicPrimitive.getGraphicAttr();
-Rectangle aCropRect;
+// fdo#72530 don't pass empty Rectangle to EndGroup
+Rectangle aCropRect(aCurrentRect);
 
 if(rAttr.IsCropped())
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Stephan Bergmann
 include/svx/fmgridcl.hxx |2 +-
 include/svx/fmshell.hxx  |2 +-
 include/svx/gridctrl.hxx |2 +-
 include/svx/svdmrkv.hxx  |4 ++--
 sc/source/ui/view/tabview5.cxx   |2 +-
 svx/source/fmcomp/fmgridcl.cxx   |2 +-
 svx/source/fmcomp/gridctrl.cxx   |2 +-
 svx/source/form/fmshell.cxx  |2 +-
 svx/source/form/fmshimp.cxx  |2 +-
 svx/source/inc/fmshimp.hxx   |2 +-
 svx/source/svdraw/svdmrkv.cxx|2 +-
 svx/source/svdraw/svdmrkv1.cxx   |2 +-
 svx/source/unodraw/unopage.cxx   |4 ++--
 sw/source/core/frmedt/fecopy.cxx |4 ++--
 sw/source/ui/ribbar/conform.cxx  |2 +-
 sw/source/ui/shells/textdrw.cxx  |2 +-
 16 files changed, 19 insertions(+), 19 deletions(-)

New commits:
commit 7f9301e7eaff91cbe064d26e6b69b785dd9bb458
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 10:49:49 2014 +0100

warning C4805: '!=' : unsafe mix of type 'sal_Bool' and type 'bool'

Change-Id: I3d4f1ac0debf506e3c4f1ce9143cb8c1cf66925d

diff --git a/include/svx/svdmrkv.hxx b/include/svx/svdmrkv.hxx
index 30d5892..80518cc 100644
--- a/include/svx/svdmrkv.hxx
+++ b/include/svx/svdmrkv.hxx
@@ -363,7 +363,7 @@ public:
 sal_Bool IsMarkPoints() const { return (0L != mpMarkPointsOverlay); }
 
 // Select that additional handles are displayed permanently.
-void SetPlusHandlesAlwaysVisible(sal_Bool bOn);
+void SetPlusHandlesAlwaysVisible(bool bOn);
 sal_Bool IsPlusHandlesAlwaysVisible() const { return bPlusHdlAlways; }
 
 // Are Handles visible during TextEdit (in double size)?
diff --git a/svx/source/svdraw/svdmrkv1.cxx b/svx/source/svdraw/svdmrkv1.cxx
index 00a2da7..ba06592 100644
--- a/svx/source/svdraw/svdmrkv1.cxx
+++ b/svx/source/svdraw/svdmrkv1.cxx
@@ -272,7 +272,7 @@ const Rectangle SdrMarkView::GetMarkedPointsRect() const
 return aMarkedPointsRect;
 }
 
-void SdrMarkView::SetPlusHandlesAlwaysVisible(sal_Bool bOn)
+void SdrMarkView::SetPlusHandlesAlwaysVisible(bool bOn)
 { // TODO: Optimize HandlePaint!
 ForceUndirtyMrkPnt();
 if (bOn!=bPlusHdlAlways) {
commit 98fba37bb1e2c5fd10d57d20d6e139765eb69751
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 10:39:12 2014 +0100

Consistently let SetDesingMode functions take bool argument

...to avoid warning C4805: '!=' : unsafe mix of type 'bool' and type 
'sal_Bool'
in opertaion style errors.  Additionally, mark those functions that are 
virtual
overrides as such.

Change-Id: I64f919f7e2d8f6224303f9bac67f6270d58de014

diff --git a/include/svx/fmgridcl.hxx b/include/svx/fmgridcl.hxx
index 306023c..62ef3ae 100644
--- a/include/svx/fmgridcl.hxx
+++ b/include/svx/fmgridcl.hxx
@@ -159,7 +159,7 @@ protected:
 virtual void ColumnResized(sal_uInt16 nId);
 virtual void ColumnMoved(sal_uInt16 nId);
 virtual void DeleteSelectedRows();
-virtual void SetDesignMode(sal_Bool bMode);
+virtual void SetDesignMode(bool bMode) SAL_OVERRIDE;
 virtual void CellModified();
 virtual void HideColumn(sal_uInt16 nId);
 virtual void ShowColumn(sal_uInt16 nId);
diff --git a/include/svx/fmshell.hxx b/include/svx/fmshell.hxx
index 42d204d..f0e7eb5 100644
--- a/include/svx/fmshell.hxx
+++ b/include/svx/fmshell.hxx
@@ -162,7 +162,7 @@ public:
 ) const;
 
 sal_BoolIsDesignMode() const { return m_bDesignMode; }
-voidSetDesignMode( sal_Bool _bDesignMode );
+voidSetDesignMode( bool _bDesignMode );
 
 protected:
 void GetFormState(SfxItemSet rSet, sal_uInt16 nWhich);
diff --git a/include/svx/gridctrl.hxx b/include/svx/gridctrl.hxx
index 673f604..9c457b8 100644
--- a/include/svx/gridctrl.hxx
+++ b/include/svx/gridctrl.hxx
@@ -419,7 +419,7 @@ public:
 sal_uInt16 GetColumnIdFromViewPos( sal_uInt16 nPos ) const { return 
GetColumnId(nPos + 1); }
 sal_uInt16 GetColumnIdFromModelPos( sal_uInt16 nPos ) const;
 
-virtual void SetDesignMode(sal_Bool bMode);
+virtual void SetDesignMode(bool bMode);
 sal_Bool IsDesignMode() const {return m_bDesignMode;}
 sal_Bool IsOpen() const {return m_pSeekCursor != NULL;}
 
diff --git a/include/svx/svdmrkv.hxx b/include/svx/svdmrkv.hxx
index e53bf10..30d5892 100644
--- a/include/svx/svdmrkv.hxx
+++ b/include/svx/svdmrkv.hxx
@@ -222,7 +222,7 @@ public:
 void SetGluePointEditMode(sal_Bool bOn=sal_True) { 
SetEditMode(bOn?SDREDITMODE_GLUEPOINTEDIT:eEditMode0); }
 sal_Bool IsGluePointEditMode() const { return 
eEditMode==SDREDITMODE_GLUEPOINTEDIT; }
 
-void SetDesignMode(sal_Bool bOn=sal_True);
+void SetDesignMode(bool bOn = true);
 sal_Bool IsDesignMode() const { return bDesignMode; }
 
 void SetFrameDragSingles(sal_Bool bOn=sal_True) { SetFrameHandles(bOn); }
diff --git a/sc/source/ui/view/tabview5.cxx b/sc/source/ui/view/tabview5.cxx
index 681fcc6..87933b4 100644
--- a/sc/source/ui/view/tabview5.cxx
+++ b/sc/source/ui/view/tabview5.cxx
@@ -222,7 +222,7 @@ void 

[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - solenv/gbuild

2014-01-21 Thread Michael Stahl
 solenv/gbuild/UnoApi.mk   |3 +++
 solenv/gbuild/UnoApiTarget.mk |   13 +
 2 files changed, 16 insertions(+)

New commits:
commit e7ba6ad7c597fa808265938dbcf40017b8b3e5e0
Author: Michael Stahl mst...@redhat.com
Date:   Mon Jan 20 18:01:04 2014 +0100

gbuild: UnoApi: fix rebuilds after IDL changes

There used to be a dependency between .urd file and .hpp/.hdl files but
that was removed; introduce a dependency between .idl files and
.hpp/.hdl files to replace it so headers and stuff depending on them get
rebuilt.

Change-Id: Ibf1fe1db08c36ce42c392d27b794c2bc2a33738c
(cherry picked from commit 0190a70e79efa437e1516e635f4f670d7a35bb2d)
Reviewed-on: https://gerrit.libreoffice.org/7547
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/solenv/gbuild/UnoApi.mk b/solenv/gbuild/UnoApi.mk
index 5f40ee3..21ce4d6 100644
--- a/solenv/gbuild/UnoApi.mk
+++ b/solenv/gbuild/UnoApi.mk
@@ -68,6 +68,8 @@ endef
 
 define gb_UnoApi_add_idlfiles
 $(call gb_UnoApiTarget_add_idlfiles,$(1),$(2),$(3))
+$(call gb_UnoApiHeadersTarget_add_headerfiles,$(1),$(2),$(addsuffix .hpp,$(3)))
+$(call gb_UnoApiHeadersTarget_add_headerfiles,$(1),$(2),$(addsuffix .hdl,$(3)))
 $(foreach idl,$(3),$(call 
gb_UnoApi__add_idlfile,$(1),$(gb_UnoApiTarget_REG_$(1))/$(2)/$(idl)))
 
 endef
@@ -81,6 +83,7 @@ endef
 
 define gb_UnoApi_add_idlfiles_nohdl
 $(call gb_UnoApiTarget_add_idlfiles,$(1),$(2),$(3))
+$(call gb_UnoApiHeadersTarget_add_headerfiles,$(1),$(2),$(addsuffix .hpp,$(3)))
 $(foreach idl,$(3),$(call 
gb_UnoApi__add_idlfile,$(1),$(gb_UnoApiTarget_REG_$(1))/$(2)/$(idl)))
 
 endef
diff --git a/solenv/gbuild/UnoApiTarget.mk b/solenv/gbuild/UnoApiTarget.mk
index 28695b8..dbce2d4 100644
--- a/solenv/gbuild/UnoApiTarget.mk
+++ b/solenv/gbuild/UnoApiTarget.mk
@@ -202,6 +202,19 @@ $(call 
gb_UnoApiHeadersTarget_get_real_comprehensive_dir,$(1))/%.hpp :| \
 
 endef
 
+# ensure that idl change triggers the dummy rule to rebuild the headers
+# call gb_UnoApiHeadersTarget_add_headerfile,unoapi,headerfile
+define gb_UnoApiHeadersTarget_add_headerfile
+$(call gb_UnoApiHeadersTarget_get_dir,$(1))/$(2) : \
+   $(SRCDIR)/$(basename $(gb_UnoApiTarget_REG_$(1))/$(2)).idl
+
+endef
+
+# call gb_UnoApiHeadersTarget_add_headerfiles,unoapi,directory,headerfilenames
+define gb_UnoApiHeadersTarget_add_headerfiles
+$(foreach hdr,$(3),$(call 
gb_UnoApiHeadersTarget_add_headerfile,$(1),$(2)/$(hdr)))
+endef
+
 define gb_UnoApiHeadersTarget__use_api_for_target
 $(call gb_UnoApiHeadersTarget_get_$(3),$(1)) : $(call 
gb_UnoApiTarget_get_target,$(2))
 $(call gb_UnoApiHeadersTarget_get_$(3),$(1)) : UNOAPI_DEPS += -X$(call 
gb_UnoApiTarget_get_target,$(2))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: solenv/gbuild

2014-01-21 Thread Stephan Bergmann
 solenv/gbuild/UnoApi.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit f4ccd054a94de1290e01bd21d5d45f1720f7cb74
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 10:56:08 2014 +0100

idl files need to be packaged relative to INSTDIR, not INSTROOT

...so that on OS X they end up in LibreOffice*_SDK.app/idl/, not
LibreOffice.app/Contents/LibreOffice*_SDK.app/idl/.

Change-Id: Id001aed334a0e1d8f835ce797a17f1f67d1a3abb

diff --git a/solenv/gbuild/UnoApi.mk b/solenv/gbuild/UnoApi.mk
index 52028d5..d99195d 100644
--- a/solenv/gbuild/UnoApi.mk
+++ b/solenv/gbuild/UnoApi.mk
@@ -20,6 +20,7 @@ $(call gb_UnoApiTarget_UnoApiTarget,$(1),$(1))
 $(call gb_UnoApiHeadersTarget_UnoApiHeadersTarget,$(1))
 ifneq ($(gb_UnoApi_ENABLE_INSTALL),)
 $(call gb_Package_Package_internal,$(1)_idl,$(SRCDIR))
+$(call gb_Package_set_outdir,$(1)_idl,$(INSTDIR))
 endif
 
 $(call gb_UnoApi_get_target,$(1)) :| $(dir $(call 
gb_UnoApi_get_target,$(1))).dir
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-4.1' - sw/qa writerfilter/source

2014-01-21 Thread Zolnai Tamás
 sw/qa/extras/ooxmlimport/data/fdo73389.docx  |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |   11 +++
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   23 +--
 3 files changed, 19 insertions(+), 15 deletions(-)

New commits:
commit 90c19d92344879577b41098ecb5ce1b007600609
Author: Zolnai Tamás tamas.zol...@collabora.com
Date:   Tue Jan 21 11:08:15 2014 +0100

fdo#73389 Writer does not show a docx-document with nested table correctly

The table manager can work with more table simultaneously
and so m_aCellWidths contains more table's properties.Only one
item of it belongs to the current table (getCurrentCellwidths).
Regression from 74c5ed19f430327988194cdcd6bdff09591a93fa

(cherry picked from commit d0c383256ef72d5212d8e2db77582d0ebe417209)

Conflicts:
sw/qa/extras/ooxmlimport/ooxmlimport.cxx
writerfilter/source/dmapper/DomainMapperTableManager.cxx

Change-Id: I93efac0c004af1b2524c955ffb20c3ecd74a2920

diff --git a/sw/qa/extras/ooxmlimport/data/fdo73389.docx 
b/sw/qa/extras/ooxmlimport/data/fdo73389.docx
new file mode 100644
index 000..02b55f7
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/fdo73389.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index e9be826..12e4c2e 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -142,6 +142,7 @@ public:
 void testBnc780044Spacing();
 void testRPrChangeClosed();
 void testFdo65090();
+void testFdo73389();
 
 CPPUNIT_TEST_SUITE(Test);
 #if !defined(MACOSX)  !defined(WNT)
@@ -248,6 +249,7 @@ void Test::run()
 {bnc780044_spacing.docx, Test::testBnc780044Spacing},
 {rprchange_closed.docx,Test::testRPrChangeClosed},
 {fdo65090.docx, Test::testFdo65090},
+{fdo73389.docx, Test::testFdo73389},
 };
 header();
 for (unsigned int i = 0; i  SAL_N_ELEMENTS(aMethods); ++i)
@@ -1722,6 +1724,15 @@ void Test::testFdo65090()
 CPPUNIT_ASSERT_EQUAL(sal_Int32(0), getProperty 
uno::Sequencetext::TableColumnSeparator (xTableRows-getByIndex(0), 
TableColumnSeparators).getLength());
 }
 
+void Test::testFdo73389()
+{
+uno::Referencetext::XTextTablesSupplier xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Referencecontainer::XIndexAccess 
xTables(xTablesSupplier-getTextTables(), uno::UNO_QUERY);
+// This was 9340, i.e. the width of the inner table was too large.
+CPPUNIT_ASSERT_EQUAL(sal_Int32(2842), 
getPropertysal_Int32(xTables-getByIndex(0), Width));
+}
+
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index d2380a2..c2c0728 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -142,24 +142,17 @@ bool DomainMapperTableManager::sprm(Sprm  rSprm)
 */
 bool bFixed = true;
 sal_Int32 nRowFixedWidth = 0;
-if (!m_aCellWidths.empty())
+IntVectorPtr pCellWidths = getCurrentCellWidths();
+// Step 1. Check whether all cells hava fixed 
widths in the given row of table.
+for (std::vectorsal_Int32::const_iterator 
aValIter = pCellWidths-begin(); aValIter != pCellWidths-end(); ++aValIter)
 {
-// Step 1. Check whether all cells have fixed 
widths in the given row of table.
-::std::vector IntVectorPtr ::iterator itr;
-for (itr = m_aCellWidths.begin(); itr != 
m_aCellWidths.end(); itr ++)
+if (*aValIter == -1)
 {
-IntVectorPtr itrVal = (*itr);
-for 
(std::vectorsal_Int32::const_iterator aValIter = itrVal-begin(); aValIter != 
itrVal-end(); ++aValIter)
-{
-if (*aValIter == -1)
-{
-bFixed = false;
-break;
-}
-// Sum the width of cells to find the 
total width of given row
-nRowFixedWidth += (*aValIter);
-}
+bFixed = false;
+break;
 }
+// Sum the width of cells to find the total 
width of given row
+  

Re: New Developer

2014-01-21 Thread Miklos Vajna
Hi Manohar,

On Mon, Jan 20, 2014 at 11:12:17PM +0530, Manohar Kh khmanoh...@gmail.com 
wrote:
 manohar@crazyHorse:~/Development$ git clone git://
 anongit.freedesktop.org/libreoffice/core libreoffice
 Cloning into 'libreoffice'...
 fatal: unable to connect to anongit.freedesktop.org:
 anongit.freedesktop.org[0: 131.252.210.161]: errno=Connection timed out

In case you're behind a proxy, try https:

git clone https://gerrit.libreoffice.org/p/core.git

Miklos


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


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

2014-01-21 Thread Michael Stahl
 sw/source/core/crsr/findtxt.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 54a8a941c0c5f0a11c8c4fd40617bfdbb40c07ed
Author: Michael Stahl mst...@redhat.com
Date:   Fri Jan 17 23:09:50 2014 +0100

fdo#73660: sw: fix Find of words containing soft-hyphen

Soft hyphen was not detected if there is no hint (-1).

(regression from ebeff3f074dd94dce4ce6cc55abd0495103684bd)

Change-Id: I09108796a32429a01eff93f797a6de521ab1d08a
(cherry picked from commit 156a0235bd8ff06840cecad1e70a57ffc5b8e7e1)
Reviewed-on: https://gerrit.libreoffice.org/7511
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit 22183794b50d3d55094e1363755af74a10a4b555)
Reviewed-on: https://gerrit.libreoffice.org/7542
Reviewed-by: Eike Rathke er...@redhat.com
Reviewed-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com

diff --git a/sw/source/core/crsr/findtxt.cxx b/sw/source/core/crsr/findtxt.cxx
index 796da83..5567cfa 100644
--- a/sw/source/core/crsr/findtxt.cxx
+++ b/sw/source/core/crsr/findtxt.cxx
@@ -89,7 +89,9 @@ lcl_CleanStr(const SwTxtNode rNd, sal_Int32 const nStart, 
sal_Int32 rEnd,
 bNewHint = true;
 }
 // Check if next stop is a soft hyphen.
-else if (-1 != nSoftHyphen  nSoftHyphen  nHintStart  nSoftHyphen 
 nEnd)
+else if (   -1 != nSoftHyphen
+  (-1 == nHintStart || nSoftHyphen  nHintStart)
+  nSoftHyphen  nEnd)
 {
 nStt = nSoftHyphen;
 bNewSoftHyphen = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 45168] Add Option to crop Images in PDF Export

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45168

--- Comment #6 from Jorendc jore...@libreoffice.org ---
(In reply to comment #4)
 I am beginning work on this issue.

@Arthur: any progression on this one? Still working on it?
If not, please mind to reset the 'assigned to' field, so others can have a
look.

Kind regards,
Joren

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


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

2014-01-21 Thread Michael Stahl
 sw/source/core/text/txtfly.cxx|6 +++---
 sw/source/core/txtnode/fntcap.cxx |4 ++--
 sw/source/core/txtnode/swfont.cxx |8 
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit c445d40da8daa2f6ec0ae9930a06f50ab80cc85d
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 11:16:20 2014 +0100

fdo#68959: sw: fix painting of tab fill characters in small caps text

The member SwDrawTextInfo::pPos was changed from a pointer to some
externally provided Point to a real object; unfortunately there were
some functions which hold backups of the current pPos as a
reference/pointer and then restore it, and the restore is a no-op now.
So use real Points as backups.

(regression from f22006dc6ac34a35a060e15466cf6b2d2058617d)

Change-Id: I8470143ec494cb194a88ddcbd12d96867974278f

diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 928ee0b..ba51593 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -568,7 +568,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 }
 
 Point aPos( rInf.GetPos().X(), rInf.GetPos().Y() + rInf.GetAscent() );
-const Point rOld = rInf.GetPos();
+const Point aOldPos(rInf.GetPos());
 rInf.SetPos( aPos );
 
 if( !bOpaque )
@@ -577,7 +577,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawStretchText( rInf );
 else
 rInf.GetFont()-_DrawText( rInf );
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_False;
 }
 else if( !aRegion.empty() )
@@ -595,7 +595,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawText( rInf );
 }
 }
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_True;
 }
 
diff --git a/sw/source/core/txtnode/fntcap.cxx 
b/sw/source/core/txtnode/fntcap.cxx
index 8fd5775..b92c482 100644
--- a/sw/source/core/txtnode/fntcap.cxx
+++ b/sw/source/core/txtnode/fntcap.cxx
@@ -516,9 +516,9 @@ void SwSubFont::DrawStretchCapital( SwDrawTextInfo rInf )
 if( rInf.GetLen() == COMPLETE_STRING )
 rInf.SetLen( rInf.GetText().getLength() );
 
-const Point rOldPos = rInf.GetPos();
+const Point aOldPos = rInf.GetPos();
 const sal_uInt16 nCapWidth = (sal_uInt16)( GetCapitalSize( rInf ).Width() 
);
-rInf.SetPos( rOldPos );
+rInf.SetPos(aOldPos);
 
 rInf.SetDrawSpace( GetUnderline() != UNDERLINE_NONE ||
GetOverline()  != UNDERLINE_NONE ||
diff --git a/sw/source/core/txtnode/swfont.cxx 
b/sw/source/core/txtnode/swfont.cxx
index 7fdb145..8a58a50 100644
--- a/sw/source/core/txtnode/swfont.cxx
+++ b/sw/source/core/txtnode/swfont.cxx
@@ -1195,12 +1195,12 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 
 SwDigitModeModifier aDigitModeModifier( rInf.GetOut(), 
rInf.GetFont()-GetLanguage() );
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
 
@@ -1300,7 +1300,7 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
@@ -1325,13 +1325,13 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 
 rInf.ApplyAutoColor();
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 
 if( IsCapital() )
@@ -1382,7 +1382,7 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 sal_Int32 SwSubFont::_GetCrsrOfst( SwDrawTextInfo rInf )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 63962] Remove LibreOffice icon themes hardcoding

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=63962

--- Comment #9 from Michael Meeks michael.me...@collabora.com ---
For translation we should re-use the code from the gallery that parses .desktop
files - since we have code to generate translated theme names from that [ that
is if we want to translate those :-].

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


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

2014-01-21 Thread Michael Stahl
 sw/source/core/text/txtfly.cxx|6 +++---
 sw/source/core/txtnode/fntcap.cxx |4 ++--
 sw/source/core/txtnode/swfont.cxx |8 
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 7cdf5a6995a017e295e9b1099abc6dcb1987ae9e
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 11:16:20 2014 +0100

fdo#68959: sw: fix painting of tab fill characters in small caps text

The member SwDrawTextInfo::pPos was changed from a pointer to some
externally provided Point to a real object; unfortunately there were
some functions which hold backups of the current pPos as a
reference/pointer and then restore it, and the restore is a no-op now.
So use real Points as backups.

(regression from f22006dc6ac34a35a060e15466cf6b2d2058617d)

Change-Id: I8470143ec494cb194a88ddcbd12d96867974278f
(cherry picked from commit c445d40da8daa2f6ec0ae9930a06f50ab80cc85d)
Reviewed-on: https://gerrit.libreoffice.org/7568
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 54635d7..3a140b6 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -568,7 +568,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 }
 
 Point aPos( rInf.GetPos().X(), rInf.GetPos().Y() + rInf.GetAscent() );
-const Point rOld = rInf.GetPos();
+const Point aOldPos(rInf.GetPos());
 rInf.SetPos( aPos );
 
 if( !bOpaque )
@@ -577,7 +577,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawStretchText( rInf );
 else
 rInf.GetFont()-_DrawText( rInf );
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_False;
 }
 else if( !aRegion.empty() )
@@ -595,7 +595,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawText( rInf );
 }
 }
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_True;
 }
 
diff --git a/sw/source/core/txtnode/fntcap.cxx 
b/sw/source/core/txtnode/fntcap.cxx
index 9735b2c..3f827f2 100644
--- a/sw/source/core/txtnode/fntcap.cxx
+++ b/sw/source/core/txtnode/fntcap.cxx
@@ -536,9 +536,9 @@ void SwSubFont::DrawStretchCapital( SwDrawTextInfo rInf )
 if( rInf.GetLen() == STRING_LEN )
 rInf.SetLen( rInf.GetText().getLength() );
 
-const Point rOldPos = rInf.GetPos();
+const Point aOldPos = rInf.GetPos();
 const sal_uInt16 nCapWidth = (sal_uInt16)( GetCapitalSize( rInf ).Width() 
);
-rInf.SetPos( rOldPos );
+rInf.SetPos(aOldPos);
 
 rInf.SetDrawSpace( GetUnderline() != UNDERLINE_NONE ||
GetOverline()  != UNDERLINE_NONE ||
diff --git a/sw/source/core/txtnode/swfont.cxx 
b/sw/source/core/txtnode/swfont.cxx
index 977b6d2..8a4ed6d 100644
--- a/sw/source/core/txtnode/swfont.cxx
+++ b/sw/source/core/txtnode/swfont.cxx
@@ -849,12 +849,12 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 
 SwDigitModeModifier aDigitModeModifier( rInf.GetOut(), 
rInf.GetFont()-GetLanguage() );
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
 
@@ -955,7 +955,7 @@ static sal_Char const sDoubleSpace[] =   ;
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
@@ -980,13 +980,13 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 
 rInf.ApplyAutoColor();
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 
 if( IsCapital() )
@@ -1038,7 +1038,7 @@ static sal_Char const sDoubleSpace[] =   ;
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 /*
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Michael Stahl
 sw/source/core/text/txtfly.cxx|6 +++---
 sw/source/core/txtnode/fntcap.cxx |4 ++--
 sw/source/core/txtnode/swfont.cxx |8 
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit b038679c550d2a0178c1beca368a86598dd0c80b
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 11:16:20 2014 +0100

fdo#68959: sw: fix painting of tab fill characters in small caps text

The member SwDrawTextInfo::pPos was changed from a pointer to some
externally provided Point to a real object; unfortunately there were
some functions which hold backups of the current pPos as a
reference/pointer and then restore it, and the restore is a no-op now.
So use real Points as backups.

(regression from f22006dc6ac34a35a060e15466cf6b2d2058617d)

Change-Id: I8470143ec494cb194a88ddcbd12d96867974278f
(cherry picked from commit c445d40da8daa2f6ec0ae9930a06f50ab80cc85d)
Reviewed-on: https://gerrit.libreoffice.org/7566
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 1e66412..9f7d970 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -568,7 +568,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 }
 
 Point aPos( rInf.GetPos().X(), rInf.GetPos().Y() + rInf.GetAscent() );
-const Point rOld = rInf.GetPos();
+const Point aOldPos(rInf.GetPos());
 rInf.SetPos( aPos );
 
 if( !bOpaque )
@@ -577,7 +577,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawStretchText( rInf );
 else
 rInf.GetFont()-_DrawText( rInf );
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_False;
 }
 else if( !aRegion.empty() )
@@ -595,7 +595,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawText( rInf );
 }
 }
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_True;
 }
 
diff --git a/sw/source/core/txtnode/fntcap.cxx 
b/sw/source/core/txtnode/fntcap.cxx
index 0c31695..c435eda 100644
--- a/sw/source/core/txtnode/fntcap.cxx
+++ b/sw/source/core/txtnode/fntcap.cxx
@@ -523,9 +523,9 @@ void SwSubFont::DrawStretchCapital( SwDrawTextInfo rInf )
 if( rInf.GetLen() == STRING_LEN )
 rInf.SetLen( rInf.GetText().getLength() );
 
-const Point rOldPos = rInf.GetPos();
+const Point aOldPos = rInf.GetPos();
 const sal_uInt16 nCapWidth = (sal_uInt16)( GetCapitalSize( rInf ).Width() 
);
-rInf.SetPos( rOldPos );
+rInf.SetPos(aOldPos);
 
 rInf.SetDrawSpace( GetUnderline() != UNDERLINE_NONE ||
GetOverline()  != UNDERLINE_NONE ||
diff --git a/sw/source/core/txtnode/swfont.cxx 
b/sw/source/core/txtnode/swfont.cxx
index 2269f24..21c6601 100644
--- a/sw/source/core/txtnode/swfont.cxx
+++ b/sw/source/core/txtnode/swfont.cxx
@@ -1195,12 +1195,12 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 
 SwDigitModeModifier aDigitModeModifier( rInf.GetOut(), 
rInf.GetFont()-GetLanguage() );
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
 
@@ -1300,7 +1300,7 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
@@ -1325,13 +1325,13 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 
 rInf.ApplyAutoColor();
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 
 if( IsCapital() )
@@ -1382,7 +1382,7 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 xub_StrLen SwSubFont::_GetCrsrOfst( SwDrawTextInfo rInf )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Fridrich Štrba
 external/libabw/UnpackedTarball_libabw.mk  |1 +
 external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 |   11 +++
 2 files changed, 12 insertions(+)

New commits:
commit f14eddb9c8ba5c83e6699ec3f2ce57ea80cc8e8f
Author: Fridrich Å trba fridrich.st...@bluewin.ch
Date:   Tue Jan 21 08:44:45 2014 +0100

Coverity: Potential crash (?)

Change-Id: I819b0df446cf95d511d6a7a5a65c381a2a837d00
(cherry picked from commit 7989850e9fc5fb2fb757ecf388b51bc3e96a0267)
Reviewed-on: https://gerrit.libreoffice.org/7563
Reviewed-by: David Tardon dtar...@redhat.com
Tested-by: David Tardon dtar...@redhat.com
Reviewed-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/external/libabw/UnpackedTarball_libabw.mk 
b/external/libabw/UnpackedTarball_libabw.mk
index c61f2d5..4f8b440 100644
--- a/external/libabw/UnpackedTarball_libabw.mk
+++ b/external/libabw/UnpackedTarball_libabw.mk
@@ -13,6 +13,7 @@ $(eval $(call 
gb_UnpackedTarball_set_tarball,libabw,$(ABW_TARBALL)))
 
 $(eval $(call gb_UnpackedTarball_add_patches,libabw,\
external/libabw/libabw-0.0.1-inttypes.patch.1 \
+   external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1 
b/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1
new file mode 100644
index 000..fccaa8e
--- /dev/null
+++ b/external/libabw/libabw-0.0.1-stdstringfromnull.patch.1
@@ -0,0 +1,11 @@
+--- a/src/lib/ABWContentCollector.cpp
 b/src/lib/ABWContentCollector.cpp
+@@ -643,7 +643,7 @@ void 
libabw::ABWContentCollector::collectHeaderFooter(const char *id, const char
+   if (!type)
+ m_ps-m_currentHeaderFooterId = -1;
+ 
+-  std::string sType(type);
++  std::string sType(type ? type : );
+   boost::trim(sType);
+   std::vectorstd::string strVec;
+   boost::algorithm::split(strVec, sType, boost::is_any_of(-), 
boost::token_compress_on);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Eike Rathke
 sc/source/core/data/column.cxx |   11 ++-
 sc/source/core/data/table2.cxx |8 
 2 files changed, 14 insertions(+), 5 deletions(-)

New commits:
commit 9f27edb1de3a7ef7499fc0e4c0bdc36e3ae20ada
Author: Eike Rathke er...@redhat.com
Date:   Wed Jan 15 23:09:20 2014 +0100

resolved fdo#72293 correctly split grouped formulas and regroup

Grouped formulas need to be split at both edges of the deleted area,
regroup _after_ rows have been deleted.

Change-Id: I840806f30cef592e17b4cd3a33c81fd2a8309055
(cherry picked from commit 42f551d524a1df46f6a311d5897ac30bd8fc1aaf)
Reviewed-on: https://gerrit.libreoffice.org/7453
Reviewed-by: Kohei Yoshida libreoff...@kohei.us
Tested-by: Kohei Yoshida libreoff...@kohei.us
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx
index 145f61a..aafb314 100644
--- a/sc/source/core/data/column.cxx
+++ b/sc/source/core/data/column.cxx
@@ -2444,7 +2444,16 @@ bool ScColumn::UpdateReference( sc::RefUpdateContext 
rCxt, ScDocument* pUndoDoc
 {
 // Cells in this column is being shifted.  Split formula grouping at
 // the top and bottom boundaries before they get shifted.
-SCROW nSplitPos = rCxt.maRange.aStart.Row();
+// Also, for deleted rows split at the top of the deleted area to adapt
+// the affected group length.
+SCROW nSplitPos;
+if (rCxt.mnRowDelta  0)
+{
+nSplitPos = rCxt.maRange.aStart.Row() + rCxt.mnRowDelta;
+if (ValidRow(nSplitPos))
+aBounds.push_back(nSplitPos);
+}
+nSplitPos = rCxt.maRange.aStart.Row();
 if (ValidRow(nSplitPos))
 {
 aBounds.push_back(nSplitPos);
diff --git a/sc/source/core/data/table2.cxx b/sc/source/core/data/table2.cxx
index 30771ab..27233f7 100644
--- a/sc/source/core/data/table2.cxx
+++ b/sc/source/core/data/table2.cxx
@@ -239,16 +239,16 @@ void ScTable::DeleteRow(
 }
 }
 
-std::vectorSCCOL aRegroupCols;
-rRegroupCols.getColumns(nTab, aRegroupCols);
-std::for_each(aRegroupCols.begin(), aRegroupCols.end(), 
ColumnRegroupFormulaCells(aCol));
-
 {   // scope for bulk broadcast
 ScBulkBroadcast aBulkBroadcast( pDocument-GetBASM());
 for (SCCOL j=nStartCol; j=nEndCol; j++)
 aCol[j].DeleteRow( nStartRow, nSize );
 }
 
+std::vectorSCCOL aRegroupCols;
+rRegroupCols.getColumns(nTab, aRegroupCols);
+std::for_each(aRegroupCols.begin(), aRegroupCols.end(), 
ColumnRegroupFormulaCells(aCol));
+
 InvalidatePageBreaks();
 
 if (IsStreamValid())
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Kohei Yoshida
 sc/source/core/data/formulacell.cxx |   17 +
 1 file changed, 17 insertions(+)

New commits:
commit 41b9f71f6717e9e4a6395227406c73b73c2d3a35
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Thu Jan 16 12:49:14 2014 -0500

fdo#73655: Don't adjust formula tokens on non-top shared formula cells.

Only adjust tokens for non-shared formula cells and shared formuls cells
that are the top of their respective shared groups.

Change-Id: I9cd57653602c97504a802001947a28c4d03368a5
(cherry picked from commit 982a03d4e2baf4ca7e78c79014389d9b23e200d1)
Reviewed-on: https://gerrit.libreoffice.org/7483
Reviewed-by: Eike Rathke er...@redhat.com
Tested-by: Eike Rathke er...@redhat.com
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com

diff --git a/sc/source/core/data/formulacell.cxx 
b/sc/source/core/data/formulacell.cxx
index 721f268..66ef732 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -2799,6 +2799,8 @@ bool ScFormulaCell::UpdateReference(
 
 void ScFormulaCell::UpdateInsertTab( sc::RefUpdateInsertTabContext rCxt )
 {
+// Adjust tokens only when it's not grouped or grouped top cell.
+bool bAdjustCode = !mxGroup || mxGroup-mpTopCell == this;
 bool bPosChanged = (rCxt.mnInsertPos = aPos.Tab());
 pCode-Reset();
 if (pDocument-IsClipOrUndo() || !pCode-GetNextReferenceRPN())
@@ -2815,6 +2817,9 @@ void ScFormulaCell::UpdateInsertTab( 
sc::RefUpdateInsertTabContext rCxt )
 if (bPosChanged)
 aPos.IncTab(rCxt.mnSheets);
 
+if (!bAdjustCode)
+return;
+
 sc::RefUpdateResult aRes = pCode-AdjustReferenceOnInsertedTab(rCxt, 
aOldPos);
 if (aRes.mbNameModified)
 // Re-compile after new sheet(s) have been inserted.
@@ -2825,6 +2830,8 @@ void ScFormulaCell::UpdateInsertTab( 
sc::RefUpdateInsertTabContext rCxt )
 
 bool ScFormulaCell::UpdateDeleteTab( sc::RefUpdateDeleteTabContext rCxt )
 {
+// Adjust tokens only when it's not grouped or grouped top cell.
+bool bAdjustCode = !mxGroup || mxGroup-mpTopCell == this;
 bool bPosChanged = (aPos.Tab() = rCxt.mnDeletePos + rCxt.mnSheets);
 pCode-Reset();
 if (pDocument-IsClipOrUndo() || !pCode-GetNextReferenceRPN())
@@ -2840,6 +2847,9 @@ bool ScFormulaCell::UpdateDeleteTab( 
sc::RefUpdateDeleteTabContext rCxt )
 if (bPosChanged)
 aPos.IncTab(-1*rCxt.mnSheets);
 
+if (!bAdjustCode)
+return false;
+
 sc::RefUpdateResult aRes = pCode-AdjustReferenceOnDeletedTab(rCxt, 
aOldPos);
 if (aRes.mbNameModified)
 // Re-compile after sheet(s) have been deleted.
@@ -2850,6 +2860,9 @@ bool ScFormulaCell::UpdateDeleteTab( 
sc::RefUpdateDeleteTabContext rCxt )
 
 void ScFormulaCell::UpdateMoveTab( sc::RefUpdateMoveTabContext rCxt, SCTAB 
nTabNo )
 {
+// Adjust tokens only when it's not grouped or grouped top cell.
+bool bAdjustCode = !mxGroup || mxGroup-mpTopCell == this;
+
 pCode-Reset();
 if (!pCode-GetNextReferenceRPN() || pDocument-IsClipOrUndo())
 {
@@ -2863,6 +2876,10 @@ void ScFormulaCell::UpdateMoveTab( 
sc::RefUpdateMoveTabContext rCxt, SCTAB nTab
 aPos.SetTab(nTabNo);
 
 // no StartListeningTo because pTab[nTab] not yet correct!
+
+if (!bAdjustCode)
+return;
+
 sc::RefUpdateResult aRes = pCode-AdjustReferenceOnMovedTab(rCxt, aOldPos);
 if (aRes.mbNameModified)
 // Re-compile after sheet(s) have been deleted.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Eike Rathke
 sc/source/core/tool/cellform.cxx |   10 ++
 1 file changed, 6 insertions(+), 4 deletions(-)

New commits:
commit b00aee5aa776c99940d858a8b73557612a558d01
Author: Eike Rathke er...@redhat.com
Date:   Mon Jan 20 19:21:55 2014 +0100

resolved fdo#73836 do not apply format twice to string

ScDocument::GetString() already returns the formatted string that then
was formatted through number formatter. Format raw string instead.

(cherry picked from commit f6b9b73a1f87d1815ed8ab8f2494e5526a710bb9)

Change-Id: I806b4fc904d04169a6c1dbd4a3d00e2556eda7bd
Reviewed-on: https://gerrit.libreoffice.org/7554
Reviewed-by: Kohei Yoshida libreoff...@kohei.us
Tested-by: Kohei Yoshida libreoff...@kohei.us
Reviewed-by: David Tardon dtar...@redhat.com
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sc/source/core/tool/cellform.cxx b/sc/source/core/tool/cellform.cxx
index bc65f91..0893149 100644
--- a/sc/source/core/tool/cellform.cxx
+++ b/sc/source/core/tool/cellform.cxx
@@ -143,14 +143,16 @@ OUString ScCellFormat::GetString(
 {
 case CELLTYPE_STRING:
 {
-OUString aCellString = rDoc.GetString(rPos);
-rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor, 
bUseStarFormat);
+ScRefCellValue aCell;
+aCell.assign(rDoc, rPos);
+rFormatter.GetOutputString(aCell.getString(rDoc), nFormat, 
aString, ppColor, bUseStarFormat);
 }
 break;
 case CELLTYPE_EDIT:
 {
-OUString aCellString = rDoc.GetString(rPos);
-rFormatter.GetOutputString(aCellString, nFormat, aString, ppColor);
+ScRefCellValue aCell;
+aCell.assign(rDoc, rPos);
+rFormatter.GetOutputString(aCell.getString(rDoc), nFormat, 
aString, ppColor);
 }
 break;
 case CELLTYPE_VALUE:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Eike Rathke
 sc/source/ui/StatisticsDialogs/AnalysisOfVarianceDialog.cxx   |3 ++-
 sc/source/ui/StatisticsDialogs/DescriptiveStatisticsDialog.cxx|3 ++-
 sc/source/ui/StatisticsDialogs/ExponentialSmoothingDialog.cxx |3 ++-
 sc/source/ui/StatisticsDialogs/MatrixComparisonGenerator.cxx  |3 ++-
 sc/source/ui/StatisticsDialogs/MovingAverageDialog.cxx|3 ++-
 sc/source/ui/StatisticsDialogs/TableFillingAndNavigationTools.cxx |9 
++---
 sc/source/ui/inc/TableFillingAndNavigationTools.hxx   |8 
+---
 7 files changed, 21 insertions(+), 11 deletions(-)

New commits:
commit 7b6dc462bbacd858df6a2defebd1fd1caa4d1b64
Author: Eike Rathke er...@redhat.com
Date:   Mon Jan 13 18:02:21 2014 +0100

resolved fdo#73555 compile statistics templates using English grammar

The statistics templates' formula expressions are setup using English
function names, so compile the resulting expressions using an English
grammar merged with the current address conventions.

(cherry picked from commit 9570a27d3e2f56e9046a8bfacaafe9e3c9744ac5)

Conflicts:
sc/source/ui/StatisticsDialogs/TTestDialog.cxx
sc/source/ui/StatisticsDialogs/TableFillingAndNavigationTools.cxx

Change-Id: I7c782a42d007daeaaf99463beb8aa580c05c7363
Reviewed-on: https://gerrit.libreoffice.org/7413
Reviewed-by: Kohei Yoshida libreoff...@kohei.us
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sc/source/ui/StatisticsDialogs/AnalysisOfVarianceDialog.cxx 
b/sc/source/ui/StatisticsDialogs/AnalysisOfVarianceDialog.cxx
index cc35630..0dc9971 100644
--- a/sc/source/ui/StatisticsDialogs/AnalysisOfVarianceDialog.cxx
+++ b/sc/source/ui/StatisticsDialogs/AnalysisOfVarianceDialog.cxx
@@ -104,7 +104,8 @@ sal_Int16 ScAnalysisOfVarianceDialog::GetUndoNameId()
 
 ScRange ScAnalysisOfVarianceDialog::ApplyOutput(ScDocShell* pDocShell)
 {
-AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument);
+AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument,
+formula::FormulaGrammar::mergeToGrammar( 
formula::FormulaGrammar::GRAM_ENGLISH, mAddressDetails.eConv));
 FormulaTemplate aTemplate(mDocument, mAddressDetails);
 
 output.writeBoldString(SC_STRLOAD(RID_STATISTICS_DLGS, 
STR_ANOVA_SINGLE_FACTOR_LABEL));
diff --git a/sc/source/ui/StatisticsDialogs/DescriptiveStatisticsDialog.cxx 
b/sc/source/ui/StatisticsDialogs/DescriptiveStatisticsDialog.cxx
index d4601ff..1687773 100644
--- a/sc/source/ui/StatisticsDialogs/DescriptiveStatisticsDialog.cxx
+++ b/sc/source/ui/StatisticsDialogs/DescriptiveStatisticsDialog.cxx
@@ -83,7 +83,8 @@ sal_Int16 ScDescriptiveStatisticsDialog::GetUndoNameId()
 
 ScRange ScDescriptiveStatisticsDialog::ApplyOutput(ScDocShell* pDocShell)
 {
-AddressWalkerWriter aOutput(mOutputAddress, pDocShell, mDocument);
+AddressWalkerWriter aOutput(mOutputAddress, pDocShell, mDocument,
+formula::FormulaGrammar::mergeToGrammar( 
formula::FormulaGrammar::GRAM_ENGLISH, mAddressDetails.eConv));
 FormulaTemplate aTemplate(mDocument, mAddressDetails);
 
 boost::scoped_ptrDataRangeIterator pIterator;
diff --git a/sc/source/ui/StatisticsDialogs/ExponentialSmoothingDialog.cxx 
b/sc/source/ui/StatisticsDialogs/ExponentialSmoothingDialog.cxx
index be6de68..6b20afd 100644
--- a/sc/source/ui/StatisticsDialogs/ExponentialSmoothingDialog.cxx
+++ b/sc/source/ui/StatisticsDialogs/ExponentialSmoothingDialog.cxx
@@ -52,7 +52,8 @@ sal_Int16 ScExponentialSmoothingDialog::GetUndoNameId()
 
 ScRange ScExponentialSmoothingDialog::ApplyOutput(ScDocShell* pDocShell)
 {
-AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument);
+AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument,
+formula::FormulaGrammar::mergeToGrammar( 
formula::FormulaGrammar::GRAM_ENGLISH, mAddressDetails.eConv));
 FormulaTemplate aTemplate(mDocument, mAddressDetails);
 
 // Smoothing factor
diff --git a/sc/source/ui/StatisticsDialogs/MatrixComparisonGenerator.cxx 
b/sc/source/ui/StatisticsDialogs/MatrixComparisonGenerator.cxx
index e797b0c..a5e5001 100644
--- a/sc/source/ui/StatisticsDialogs/MatrixComparisonGenerator.cxx
+++ b/sc/source/ui/StatisticsDialogs/MatrixComparisonGenerator.cxx
@@ -74,7 +74,8 @@ sal_Int16 ScMatrixComparisonGenerator::GetUndoNameId()
 
 ScRange ScMatrixComparisonGenerator::ApplyOutput(ScDocShell* pDocShell)
 {
-AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument);
+AddressWalkerWriter output(mOutputAddress, pDocShell, mDocument,
+formula::FormulaGrammar::mergeToGrammar( 
formula::FormulaGrammar::GRAM_ENGLISH, mAddressDetails.eConv));
 FormulaTemplate aTemplate(mDocument, mAddressDetails);
 
 SCTAB inTab = mInputRange.aStart.Tab();
diff --git a/sc/source/ui/StatisticsDialogs/MovingAverageDialog.cxx 
b/sc/source/ui/StatisticsDialogs/MovingAverageDialog.cxx
index 

[Bug 63154] replace tools/solar.h macros with osl versions

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=63154

Commit Notification libreoffice-comm...@lists.freedesktop.org changed:

   What|Removed |Added

 Whiteboard|EasyHack DifficultyBeginner |EasyHack DifficultyBeginner
   |SkillCpp TopicCleanup   |SkillCpp TopicCleanup
   |target:4.1.0|target:4.1.0 target:4.3.0

--- Comment #34 from Commit Notification 
libreoffice-comm...@lists.freedesktop.org ---
Alexandre Vicenzi committed a patch related to this issue.
It has been pushed to master:

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

fdo#63154 Removed unused solar.h reference in oox



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

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


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

2014-01-21 Thread Alexandre Vicenzi
 oox/inc/pch/precompiled_oox.hxx |1 -
 oox/source/drawingml/shape.cxx  |5 -
 2 files changed, 6 deletions(-)

New commits:
commit 15913f9ae1b8753a1fab894667b7543e08210b04
Author: Alexandre Vicenzi vicenzi.alexan...@gmail.com
Date:   Mon Jan 20 23:03:01 2014 -0200

fdo#63154 Removed unused solar.h reference in oox

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

diff --git a/oox/inc/pch/precompiled_oox.hxx b/oox/inc/pch/precompiled_oox.hxx
index f997388..d2dae89 100644
--- a/oox/inc/pch/precompiled_oox.hxx
+++ b/oox/inc/pch/precompiled_oox.hxx
@@ -388,7 +388,6 @@
 #include tools/globname.hxx
 #include tools/mapunit.hxx
 #include tools/multisel.hxx
-#include tools/solar.h
 #include tools/stream.hxx
 #include tools/time.hxx
 #include unotools/fltrcfg.hxx
diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index c9889db..78b3f35 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -36,7 +36,6 @@
 #include oox/helper/graphichelper.hxx
 #include oox/helper/propertyset.hxx
 
-#include tools/solar.h
 #include tools/gen.hxx
 #include tools/mapunit.hxx
 #include editeng/unoprnms.hxx
@@ -80,8 +79,6 @@ using namespace ::com::sun::star::style;
 
 namespace oox { namespace drawingml {
 
-// 
-
 Shape::Shape( const sal_Char* pServiceName, bool bDefaultHeight )
 : mbIsChild( false )
 , mpLinePropertiesPtr( new LineProperties )
@@ -,8 +1108,6 @@ void Shape::putPropertiesToGrabBag( const Sequence 
PropertyValue  aProperties
 }
 }
 
-// 
-
 } }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 3 commits - odk/examples

2014-01-21 Thread Stephan Bergmann
 odk/examples/DevelopersGuide/OfficeBean/Makefile   |5 +
 odk/examples/DevelopersGuide/OfficeDev/DesktopEnvironment/Makefile |5 +
 odk/examples/DevelopersGuide/ProfUNO/Lifetime/object_lifetime.cxx  |2 +-
 3 files changed, 11 insertions(+), 1 deletion(-)

New commits:
commit 6f2fbc381de2c12f4ca9c2fc515c4f6860d07095
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 12:24:54 2014 +0100

DesktopEnvironment example does not work on OS X

...as nativelib/unix/nativeview.c uses X11.

Change-Id: Ifdc0206b77085db909155fb61c4a345f2813849d

diff --git a/odk/examples/DevelopersGuide/OfficeDev/DesktopEnvironment/Makefile 
b/odk/examples/DevelopersGuide/OfficeDev/DesktopEnvironment/Makefile
index b4e69cf..4227108 100644
--- a/odk/examples/DevelopersGuide/OfficeDev/DesktopEnvironment/Makefile
+++ b/odk/examples/DevelopersGuide/OfficeDev/DesktopEnvironment/Makefile
@@ -81,8 +81,13 @@ endif
 
 # Targets
 .PHONY: ALL
+ifeq $(OS) MACOSX
+ALL :
+   @printf 'This example does not work on Mac OS X\n'
+else
 ALL : $(SUBDIR) \
$(EXAMPLE_NAME)
+endif
 
 include $(SETTINGS)/stdtarget.mk
 
commit 3034b144d0062e9c4394b901aded43fec117ed11
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 12:22:35 2014 +0100

OfficeBean example does not work on OS X (where officebean.jar is not built)

Change-Id: I18c9d0fa0a74cbbdad43bd99d87dc6414a0264d2

diff --git a/odk/examples/DevelopersGuide/OfficeBean/Makefile 
b/odk/examples/DevelopersGuide/OfficeBean/Makefile
index 3326244..2dfc678 100644
--- a/odk/examples/DevelopersGuide/OfficeBean/Makefile
+++ b/odk/examples/DevelopersGuide/OfficeBean/Makefile
@@ -59,8 +59,13 @@ SDK_CLASSPATH = $(subst $(EMPTYSTRING) 
$(PATH_SEPARATOR),$(PATH_SEPARATOR),$(URE
 
 # Targets
 .PHONY: ALL
+ifeq $(OS) MACOSX
+ALL :
+   @printf 'This example does not work on Mac OS X\n'
+else
 ALL : \
 OOoBeanViewer
+endif
 
 include $(SETTINGS)/stdtarget.mk
 
commit fd95c8950c674ad0935f2583f8fb209d873a73b7
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 12:21:51 2014 +0100

Fix signature of main function

Change-Id: Id6a4968b15445f04e788eaa6ca950707d5311e66

diff --git a/odk/examples/DevelopersGuide/ProfUNO/Lifetime/object_lifetime.cxx 
b/odk/examples/DevelopersGuide/ProfUNO/Lifetime/object_lifetime.cxx
index c24e4e3..f92731d 100644
--- a/odk/examples/DevelopersGuide/ProfUNO/Lifetime/object_lifetime.cxx
+++ b/odk/examples/DevelopersGuide/ProfUNO/Lifetime/object_lifetime.cxx
@@ -62,7 +62,7 @@ void simple_object_creation_and_destruction()
 }
 
 
-int main( char * argv[] )
+int main()
 {
 simple_object_creation_and_destruction();
 return 0;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Luc Castermans
 vcl/inc/ilstbox.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1424b5c1de7806566297a74ac1d8611058c77f8e
Author: Luc Castermans luc.casterm...@gmail.com
Date:   Sat Jan 18 21:19:02 2014 +0100

Translated German comments

Change-Id: I63ca99a57a84a0c066a2abb1e65d8c04ae9cabfc
Reviewed-on: https://gerrit.libreoffice.org/7522
Reviewed-by: Chris Sherlock chris.sherloc...@gmail.com
Reviewed-by: Michael Meeks michael.me...@collabora.com
Tested-by: Michael Meeks michael.me...@collabora.com

diff --git a/vcl/inc/ilstbox.hxx b/vcl/inc/ilstbox.hxx
index 7f6a8df..af7910c 100644
--- a/vcl/inc/ilstbox.hxx
+++ b/vcl/inc/ilstbox.hxx
@@ -415,7 +415,7 @@ private:
 bool mbAutoHScroll : 1; // AutoHScroll an oder aus
 bool mbEdgeBlending : 1;
 
-LinkmaScrollHdl;// Weil der vom ImplListBoxWindow 
selbst benoetigt wird.
+LinkmaScrollHdl;// because it is needed by 
ImplListBoxWindow itself
 ::com::sun::star::uno::Reference ::com::sun::star::uno::XInterface  
mxDNDListenerContainer;
 
 protected:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Luc Castermans
 vcl/source/control/edit.cxx   |2 -
 vcl/source/filter/igif/decode.cxx |4 +-
 vcl/source/filter/ixbm/xbmread.cxx|   26 ++---
 vcl/source/filter/ixpm/xpmread.cxx|   66 +-
 vcl/source/filter/jpeg/JpegReader.cxx |4 +-
 vcl/source/filter/sgfbram.hxx |   66 +-
 vcl/source/filter/sgffilt.hxx |   16 
 7 files changed, 92 insertions(+), 92 deletions(-)

New commits:
commit 3007e015310c4a46e1249f69aa26e53dad9aafe2
Author: Luc Castermans luc.casterm...@gmail.com
Date:   Sun Jan 19 10:51:39 2014 +0100

Translated German comments vcl/source

Change-Id: I1143996ef86e35caa72cf2e60ae586940a10ce81
Reviewed-on: https://gerrit.libreoffice.org/7527
Reviewed-by: Michael Meeks michael.me...@collabora.com
Tested-by: Michael Meeks michael.me...@collabora.com

diff --git a/vcl/source/control/edit.cxx b/vcl/source/control/edit.cxx
index 3a5e579..2a17a04 100644
--- a/vcl/source/control/edit.cxx
+++ b/vcl/source/control/edit.cxx
@@ -2107,7 +2107,7 @@ void Edit::Command( const CommandEvent rCEvt )
 }
 else
 {
-// Paste nur, wenn Text im Clipboard
+// only paste if text available in clipboard
 sal_Bool bData = sal_False;
 uno::Reference datatransfer::clipboard::XClipboard  xClipboard = 
GetClipboard();
 if ( xClipboard.is() )
diff --git a/vcl/source/filter/igif/decode.cxx 
b/vcl/source/filter/igif/decode.cxx
index bf29328..fd92fba 100644
--- a/vcl/source/filter/igif/decode.cxx
+++ b/vcl/source/filter/igif/decode.cxx
@@ -153,7 +153,7 @@ bool GIFLZWDecompressor::ProcessOneCode()
 
 if ( !bEndOfBlock )
 {
-// Einen Code aus dem Eingabe-Buffer holen:
+// fetch code from input buffer
 nCode = sal::static_int_cast sal_uInt16 (
 ( (sal_uInt16) nInputBitsBuf )  ( ~( 0x  nCodeSize ) ));
 nInputBitsBuf = nCodeSize;
@@ -188,7 +188,7 @@ bool GIFLZWDecompressor::ProcessOneCode()
 
 nOldCode = nCode;
 
-// Zeichen(/-folge) des Codes nCode in den Ausgabe-Buffer schreiben:
+// write character(/-sequence) of code nCode in the output buffer:
 pE = pTable + nCode;
 do
 {
diff --git a/vcl/source/filter/ixbm/xbmread.cxx 
b/vcl/source/filter/ixbm/xbmread.cxx
index 7dfc333..e7c1ed1 100644
--- a/vcl/source/filter/ixbm/xbmread.cxx
+++ b/vcl/source/filter/ixbm/xbmread.cxx
@@ -80,7 +80,7 @@ void XBMReader::InitTable()
 pHexTable[(int)'e'] = 14;
 pHexTable[(int)'f'] = 15;
 pHexTable[(int)'x'] = 0;
-pHexTable[(int)' '] = -1;
+pHexTable[(int)' '] = -1;
 pHexTable[(int)','] = -1;
 pHexTable[(int)'}'] = -1;
 pHexTable[(int)'\n'] = -1;
@@ -143,22 +143,22 @@ long XBMReader::ParseDefine( const sal_Char* pDefine )
 char*   pTmp = (char*) pDefine;
 unsigned char   cTmp;
 
-// bis zum Ende gehen
+// move to end
 pTmp += ( strlen( pDefine ) - 1 );
 cTmp = *pTmp--;
 
-// letzte Ziffer suchen
+// search last digit
 while( pHexTable[ cTmp ] == -1 )
 cTmp = *pTmp--;
 
-// bis vor die Zahl laufen
+// move before number
 while( pHexTable[ cTmp ] != -1 )
 cTmp = *pTmp--;
 
-// auf Anfang der Zahl gehen
+// move to start of number
 pTmp += 2;
 
-// Hex lesen
+// read Hex
 if( ( pTmp[0] == '0' )  ( ( pTmp[1] == 'X' ) || ( pTmp[1] == 'x' ) ) )
 {
 pTmp += 2;
@@ -170,7 +170,7 @@ long XBMReader::ParseDefine( const sal_Char* pDefine )
 cTmp = *pTmp++;
 }
 }
-// Dezimal lesen
+// read decimal
 else
 {
 cTmp = *pTmp++;
@@ -203,7 +203,7 @@ bool XBMReader::ParseData( SvStream* pInStm, const OString 
aLastLine, XBMFormat
 {
 sal_Int32 nPos;
 
-// einfuehrende geschweifte Klammer loeschen
+// delete opening curly bracket
 if( (nPos = ( aLine = aLastLine ).indexOf('{') ) != -1 )
 aLine = aLine.copy(nPos + 1);
 
@@ -264,12 +264,12 @@ ReadState XBMReader::ReadXBM( Graphic rGraphic )
 ReadState   eReadState;
 sal_uInt8   cDummy;
 
-// sehen, ob wir _alles_ lesen koennen
+// check if we can read ALL
 rIStm.Seek( STREAM_SEEK_TO_END );
 rIStm  cDummy;
 
-// falls wir nicht alles lesen koennen
-// kehren wir zurueck und warten auf neue Daten
+// if we cannot read all
+// we returnn and wait for new data
 if ( rIStm.GetError() != ERRCODE_IO_PENDING )
 {
 rIStm.Seek( nLastPos );
@@ -284,8 +284,8 @@ ReadState XBMReader::ReadXBM( Graphic rGraphic )
 nWidth = nValue;
 aLine = FindTokenLine( rIStm, #define, _height );
 
-// Falls die Hoehe nicht folgt, suchen wir noch
-// einmal vom Anfang der Datei an
+// if height was not received, we search again
+// from start of 

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

2014-01-21 Thread Bjoern Michaelsen
 include/svtools/colorcfg.hxx   |1 
 svtools/source/config/accessibilityoptions.cxx |4 ++
 svtools/source/config/colorcfg.cxx |   36 -
 3 files changed, 39 insertions(+), 2 deletions(-)

New commits:
commit 23221e48495262d0384c9169a0d8a01db8a5dab5
Author: Bjoern Michaelsen bjoern.michael...@canonical.com
Date:   Thu Jan 16 21:42:47 2014 +0100

fdo#71511: in autodetected a11y HC mode, pull background color from theme

- ... and reload ColorConfig after tweaking relevant a11y settings
- dark theme default to high contrast
- as per fdo#35365, having a dark document background is inconvenient
  for non-a11y endusers
  - a11y standard require the (rather ugly) background for a11y
  - thus, when automatically detect high contract mode of operating
system
in Tools-Options-a11y is enabled, use the dark document
backgroundby default, otherwise use a white default

note that the autodetect HC option is somewhat broken anyway:
it resets the icon theme hard, so there are not simple roundtrips
enableing/disabling it for that, but those havent been there before
either.

(includes: Ia35a41717224dfb7437054bb885c61d7e0b189d7)

Change-Id: Id8ad1eb3d57b3708ac5a241092208e3a8d98ade0
Reviewed-on: https://gerrit.libreoffice.org/7484
Reviewed-by: Michael Meeks michael.me...@collabora.com
Tested-by: Michael Meeks michael.me...@collabora.com

diff --git a/include/svtools/colorcfg.hxx b/include/svtools/colorcfg.hxx
index 2872f4c..ddc8485 100644
--- a/include/svtools/colorcfg.hxx
+++ b/include/svtools/colorcfg.hxx
@@ -105,6 +105,7 @@ public:
 // instead of the automatic color
 ColorConfigValueGetColorValue(ColorConfigEntry eEntry, sal_Bool 
bSmart = sal_True)const;
 static ColorGetDefaultColor(ColorConfigEntry eEntry);
+void Reload();
 };
 
 class SVT_DLLPUBLIC EditableColorConfig
diff --git a/svtools/source/config/accessibilityoptions.cxx 
b/svtools/source/config/accessibilityoptions.cxx
index 4294e7c..84f0e1c 100644
--- a/svtools/source/config/accessibilityoptions.cxx
+++ b/svtools/source/config/accessibilityoptions.cxx
@@ -31,6 +31,8 @@
 
 #include svl/smplhint.hxx
 
+#include svtools/colorcfg.hxx
+
 #include vcl/settings.hxx
 #include vcl/svapp.hxx
 #include rtl/instance.hxx
@@ -386,7 +388,7 @@ void 
SvtAccessibilityOptions_Impl::SetAutoDetectSystemHC(sal_Bool bSet)
 {
 xNode-setPropertyValue(s_sAutoDetectSystemHC, 
css::uno::makeAny(bSet));
 ::comphelper::ConfigurationHelper::flush(m_xCfg);
-
+svtools::ColorConfig().Reload();
 bIsModified = sal_True;
 }
 }
diff --git a/svtools/source/config/colorcfg.cxx 
b/svtools/source/config/colorcfg.cxx
index 850bd93..a8e5ba8 100644
--- a/svtools/source/config/colorcfg.cxx
+++ b/svtools/source/config/colorcfg.cxx
@@ -23,7 +23,9 @@
 #include com/sun/star/uno/Sequence.hxx
 #include com/sun/star/lang/Locale.hpp
 #include com/sun/star/beans/PropertyValue.hpp
+#include comphelper/processfactory.hxx
 #include unotools/configitem.hxx
+#include unotools/confignode.hxx
 #include unotools/configpaths.hxx
 #include com/sun/star/uno/Sequence.h
 #include svl/poolitem.hxx
@@ -64,6 +66,7 @@ class ColorConfig_Impl : public utl::ConfigItem
 sal_Boolm_bEditMode;
 OUString   m_sIsVisible;
 OUString   m_sLoadedScheme;
+bool m_bAutoDetectSystemHC;
 
 uno::Sequence OUString GetPropertyNames(const OUString rScheme);
 public:
@@ -91,6 +94,7 @@ public:
 voidSetModified(){ConfigItem::SetModified();}
 void
ClearModified(){ConfigItem::ClearModified();}
 voidSettingsChanged();
+bool GetAutoDetectSystemHC() {return m_bAutoDetectSystemHC;}
 
 // #100822#
 DECL_LINK( DataChangedEventListener, VclWindowEvent* );
@@ -183,7 +187,8 @@ uno::Sequence OUString 
ColorConfig_Impl::GetPropertyNames(const OUString rSch
 ColorConfig_Impl::ColorConfig_Impl(sal_Bool bEditMode) :
 ConfigItem(Office.UI/ColorScheme),
 m_bEditMode(bEditMode),
-m_sIsVisible(/IsVisible)
+m_sIsVisible(/IsVisible),
+m_bAutoDetectSystemHC(true)
 {
 if(!m_bEditMode)
 {
@@ -237,6 +242,15 @@ void ColorConfig_Impl::Load(const OUString rScheme)
 if(pColorNames[nIndex].match(m_sIsVisible, 
pColorNames[nIndex].getLength() - m_sIsVisible.getLength()))
  m_aConfigValues[i / 2].bIsVisible = Any2Bool(pColors[nIndex++]);
 }
+// fdo#71511: check if we are running in a11y autodetect
+{
+utl::OConfigurationNode aNode = 
utl::OConfigurationTreeRoot::tryCreateWithComponentContext(comphelper::getProcessComponentContext(),OUString(org.openoffice.Office.Common/Accessibility)
 );
+if(aNode.isValid())
+{
+uno::Any aValue = 
aNode.getNodeValue(OUString(AutoDetectSystemHC));
+   

space between values and units in spinners, fdo#71688

2014-01-21 Thread Caolán McNamara
For 4.3 we are now, in the normal case, putting space between values and
units in spinners. e.g. now 10 mm instead of 10mm.
(http://en.wikipedia.org/wiki/ISO_31-0)

For the degree symbol we are not putting space between value and unit,
i.e. it remains 10°

I think those are hopefully uncontroversial.

The tricky case is the % symbol, because, while ISO-31-0 implies that
10 % is the preferred scientific format, English style guides prefer
10% and 10 % looks just plain wrong. It's a locale-dependent
formatting rule (http://en.wikipedia.org/wiki/Percent_sign)

So to resolve this we now have a function formatPercent which takes the
number and the locale and runs it through icu's PercentInstance to
format it according to that locale's formatting rule. Which gives e.g.
10 % for de and 10% for en. (http://site.icu-project.org/)

What this means is that:
a) In the English strings in .src files the source text should not have
a space between number and %. I think these are all fixed now.
b) You should check that the icu percent formatting rule is considered
correct for your language in the context of a office suite UI e.g. in
the spinner of view-zoom-zoom-variable
c) If you agree with icu's % formatting for your language then update
the translations for X% to be consistent with that formatting. Otherwise
we can fix icu or make a blacklist of languages which should be excluded
from the default icu percent formatting.

C.

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


[Libreoffice-commits] core.git: dbaccess/source dbaccess/uiconfig dbaccess/UIConfig_dbaccess.mk

2014-01-21 Thread Caolán McNamara
 dbaccess/UIConfig_dbaccess.mk  |1 
 dbaccess/source/ui/dlg/TablesSingleDlg.cxx |   15 ++---
 dbaccess/source/ui/inc/TablesSingleDlg.hxx |2 
 dbaccess/source/ui/inc/dbu_resource.hrc|1 
 dbaccess/uiconfig/ui/tablesfilterdialog.ui |   82 +
 5 files changed, 92 insertions(+), 9 deletions(-)

New commits:
commit 9eebbbcbf7a022e252c5f34f091f3e4b398271cc
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 09:33:35 2014 +

convert OTableSubscriptionDialog to SfxSingleTabDialog

Change-Id: I8335012139ccaf58bc18d0779e63a5bc388c9fb4

diff --git a/dbaccess/UIConfig_dbaccess.mk b/dbaccess/UIConfig_dbaccess.mk
index 702b9a2..7148268 100644
--- a/dbaccess/UIConfig_dbaccess.mk
+++ b/dbaccess/UIConfig_dbaccess.mk
@@ -19,6 +19,7 @@ $(eval $(call gb_UIConfig_add_uifiles,dbaccess, \
 dbaccess/uiconfig/ui/querypropertiesdialog  \
 dbaccess/uiconfig/ui/rowheightdialog \
 dbaccess/uiconfig/ui/sortdialog \
+dbaccess/uiconfig/ui/tablesfilterdialog \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/dbaccess/source/ui/dlg/TablesSingleDlg.cxx 
b/dbaccess/source/ui/dlg/TablesSingleDlg.cxx
index f470a54..f19080e 100644
--- a/dbaccess/source/ui/dlg/TablesSingleDlg.cxx
+++ b/dbaccess/source/ui/dlg/TablesSingleDlg.cxx
@@ -42,10 +42,11 @@ OTableSubscriptionDialog::OTableSubscriptionDialog(Window* 
pParent
 ,SfxItemSet* _pItems
 ,const Reference XComponentContext  _rxORB
 ,const ::com::sun::star::uno::Any _aDataSourceName)
-:SfxNoLayoutSingleTabDialog(pParent,DLG_TABLE_FILTER,_pItems)
-,m_pImpl( new ODbDataSourceAdministrationHelper( _rxORB, pParent, this ) )
-,m_bStopExecution(sal_False)
-,m_pOutSet(_pItems)
+: SfxSingleTabDialog(pParent, _pItems, TablesFilterDialog,
+dbaccess/ui/tablesfilterdialog.ui)
+, m_pImpl( new ODbDataSourceAdministrationHelper( _rxORB, pParent, this ) )
+, m_bStopExecution(sal_False)
+, m_pOutSet(_pItems)
 {
 DBG_CTOR(OTableSubscriptionDialog,NULL);
 m_pImpl-setDataSourceOrName(_aDataSourceName);
@@ -55,9 +56,9 @@ OTableSubscriptionDialog::OTableSubscriptionDialog(Window* 
pParent
 m_pImpl-translateProperties(xDatasource, *m_pOutSet);
 SetInputSet(m_pOutSet);
 
-OTableSubscriptionPage* pTabPage = new 
OTableSubscriptionPage(this,*m_pOutSet,this);
+OTableSubscriptionPage* pTabPage = new 
OTableSubscriptionPage(get_content_area(), *m_pOutSet, this);
 pTabPage-SetServiceFactory(_rxORB);
-SetTabPage(pTabPage);
+setTabPage(pTabPage);
 }
 
 OTableSubscriptionDialog::~OTableSubscriptionDialog()
@@ -71,7 +72,7 @@ short OTableSubscriptionDialog::Execute()
 short nRet = RET_CANCEL;
 if ( !m_bStopExecution )
 {
-nRet = SfxNoLayoutSingleTabDialog::Execute();
+nRet = SfxSingleTabDialog::Execute();
 if ( nRet == RET_OK )
 {
 m_pOutSet-Put(*GetOutputItemSet());
diff --git a/dbaccess/source/ui/inc/TablesSingleDlg.hxx 
b/dbaccess/source/ui/inc/TablesSingleDlg.hxx
index 14c4ec6..ba11843 100644
--- a/dbaccess/source/ui/inc/TablesSingleDlg.hxx
+++ b/dbaccess/source/ui/inc/TablesSingleDlg.hxx
@@ -40,7 +40,7 @@ namespace dbaui
 {
 class ODbDataSourceAdministrationHelper;
 // OTableSubscriptionDialog
-class OTableSubscriptionDialog : public SfxNoLayoutSingleTabDialog, public 
IItemSetHelper
+class OTableSubscriptionDialog : public SfxSingleTabDialog, public 
IItemSetHelper
 {
 ::std::auto_ptrODbDataSourceAdministrationHelper  m_pImpl;
 sal_Bool m_bStopExecution; // set when the dialog should not be 
executed
diff --git a/dbaccess/source/ui/inc/dbu_resource.hrc 
b/dbaccess/source/ui/inc/dbu_resource.hrc
index 2905821..bbe093e 100644
--- a/dbaccess/source/ui/inc/dbu_resource.hrc
+++ b/dbaccess/source/ui/inc/dbu_resource.hrc
@@ -98,7 +98,6 @@
 #define DLG_DATABASE_TYPE_CHANGERID_DIALOG_START + 28
 #define DLG_DATABASE_WIZARD RID_DIALOG_START + 29
 #define DLG_TEXT_CONNECTION_SETTINGSRID_DIALOG_START + 31
-#define DLG_TABLE_FILTERRID_DIALOG_START + 32
 //
 // tab pages
 
diff --git a/dbaccess/uiconfig/ui/tablesfilterdialog.ui 
b/dbaccess/uiconfig/ui/tablesfilterdialog.ui
new file mode 100644
index 000..b311189
--- /dev/null
+++ b/dbaccess/uiconfig/ui/tablesfilterdialog.ui
@@ -0,0 +1,82 @@
+?xml version=1.0 encoding=UTF-8?
+!-- Generated with glade 3.16.0 on Tue Jan 21 09:32:06 2014 --
+interface
+  !-- interface-requires gtk+ 3.0 --
+  object class=GtkDialog id=TablesFilterDialog
+property name=can_focusFalse/property
+property name=border_width6/property
+property name=title translatable=yesTables Filter/property
+property name=type_hintdialog/property
+child internal-child=vbox
+  object class=GtkBox id=dialog-vbox1
+property name=can_focusFalse/property
+property 

Re: Mac OS X 64-bit, libstdc++ vs. libc++

2014-01-21 Thread Stephan Bergmann

On 01/17/2014 10:02 AM, Stephan Bergmann wrote:

Now, the good news appears to be that Libc++ is specifically designed
so that you can use it in the same process as libstdc++.
[http://clang-developers.42468.n3.nabble.com/Adopting-libc-hitting-problems-on-Mac-OS-X-td4027011.html]
  That means that it hopefully works to stick with libstdc++ for LO 4.2
(and avoid any trouble switching this late in the release schedule),
switch to libc++ for LO 4.3 say, and expect any native extensions using
either of the two libraries to work with both flavors of LO.  (And,
given the assumption that Apple will eventually discontinue the support
of libstdc++, recommend that native extensions---which there are none
yet for Mac OS X 64-bit---use libc++.)


So I built an extension (the SDK's examples/cpp/complextoolbarcontrols) 
containing a native UNO component using libc++, and installed that into 
http://donate.libreoffice.org/home/dl/mac-x86_64/4.2.0/en-US/LibreOffice_4.2.0.2_MacOS_x86-64.dmg 
(using libstdc++), and everything works fine.  So I still think we are 
safe to keep LO 4.2.0 at libstdc++ and recommend libc++ for extensions.


Apropos SDK, doing the above made me aware that the SDK for Mac OS X 
64-bit is broken, in that its settings/settings.mk assumes building for 
a 32-bit LO. 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=e5ad31e2f784856506c28092166dc8704c84e868 
Fix Mac OS X 64 bit SDK settings.mk fixes that on master, but it 
depdns on 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=dea0398ee24eb278cdb14999efbf34d634454eaa 
SDK settings.mk: Don't guess PLATFORM/PROCTYPE, which I would feel 
uneasy to backport to libreoffice-4-2-0 at this stage.  (I haven't even 
been able to test that master change on Windows yet.)


Therefore, I would suggest to stay with a broken Mac OS X 64-bit SDK for 
LO 4.2.0 and only backport those fixes to libreoffice-4-2 towards LO 
4.2.1.  Other opinions?


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


long paras + tools/string.hxx

2014-01-21 Thread Caolán McNamara
The 16 bit length limitation in writer is now removed towards 4.3.

The 16 bit paragraph length limit in editeng still remains, editeng
mixes freely sal_uInt16 with xub_StrLen and has a pile of hard-coded
0x here and there which makes it tricky to see at a glance which
values are string lengths and offsets and which are not. Sigh.

Also, note there is a hidden limit in tools/stream.hxx of 0xFFFE in
things like ReadLine and ReadByteStringLine and their callers.

sc is now the only module that includes tools/string.hxx and uses
STRING_MAXLEN. It would be great if some calc hackers could have a look
there to see if those limits still make sense (e.g.
nArbitraryLineLengthLimit) especially considering the ReadLine, etc.
issue.

C.

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


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

2014-01-21 Thread Caolán McNamara
 dbaccess/source/ui/app/AppDetailPageHelper.cxx |5 ++---
 dbaccess/source/ui/control/tabletree.cxx   |   20 
 dbaccess/source/ui/inc/tabletree.hxx   |   21 -
 3 files changed, 18 insertions(+), 28 deletions(-)

New commits:
commit d2c2ab1cb6e974195ccbd7f96234f8660621b06f
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 12:17:38 2014 +

tidy up OTableTreeListBox to make it easier to .ui-ify

Change-Id: I2463d665c857425bc4cf9df89338a0aab89f0a90

diff --git a/dbaccess/source/ui/app/AppDetailPageHelper.cxx 
b/dbaccess/source/ui/app/AppDetailPageHelper.cxx
index 6c2777e..c8a7dda 100644
--- a/dbaccess/source/ui/app/AppDetailPageHelper.cxx
+++ b/dbaccess/source/ui/app/AppDetailPageHelper.cxx
@@ -589,9 +589,8 @@ void OAppDetailPageHelper::createTablesPage(const 
Reference XConnection _xCon
 
 if ( !m_pLists[E_TABLE] )
 {
-OTableTreeListBox* pTreeView = new OTableTreeListBox(this
-,WB_HASLINES | 
WB_SORT | WB_HASBUTTONS | WB_HSCROLL |WB_HASBUTTONSATROOT | WB_TABSTOP
-,sal_False);
+OTableTreeListBox* pTreeView = new OTableTreeListBox(this,
+WB_HASLINES | WB_SORT | WB_HASBUTTONS | WB_HSCROLL 
|WB_HASBUTTONSATROOT | WB_TABSTOP);
 pTreeView-SetHelpId(HID_APP_TABLE_TREE);
 m_pLists[E_TABLE] = pTreeView;
 
diff --git a/dbaccess/source/ui/control/tabletree.cxx 
b/dbaccess/source/ui/control/tabletree.cxx
index 57aae83..e1f8333 100644
--- a/dbaccess/source/ui/control/tabletree.cxx
+++ b/dbaccess/source/ui/control/tabletree.cxx
@@ -64,10 +64,10 @@ namespace DatabaseObject = 
::com::sun::star::sdb::application::DatabaseObject;
 namespace DatabaseObjectContainer = 
::com::sun::star::sdb::application::DatabaseObjectContainer;
 
 // OTableTreeListBox
-OTableTreeListBox::OTableTreeListBox( Window* pParent, WinBits nWinStyle, 
sal_Bool _bVirtualRoot )
+OTableTreeListBox::OTableTreeListBox(Window* pParent, WinBits nWinStyle)
 :OMarkableTreeListBox(pParent, nWinStyle)
-,m_pImageProvider( new ImageProvider )
-,m_bVirtualRoot(_bVirtualRoot)
+,m_xImageProvider( new ImageProvider )
+,m_bVirtualRoot(false)
 ,m_bNoEmptyFolders( false )
 {
 implSetDefaultImages();
@@ -75,17 +75,13 @@ OTableTreeListBox::OTableTreeListBox( Window* pParent, 
WinBits nWinStyle, sal_Bo
 
 OTableTreeListBox::OTableTreeListBox( Window* pParent, const ResId rResId, 
sal_Bool _bVirtualRoot)
 :OMarkableTreeListBox(pParent, rResId)
-,m_pImageProvider( new ImageProvider )
+,m_xImageProvider( new ImageProvider )
 ,m_bVirtualRoot(_bVirtualRoot)
 ,m_bNoEmptyFolders( false )
 {
 implSetDefaultImages();
 }
 
-OTableTreeListBox::~OTableTreeListBox()
-{
-}
-
 void OTableTreeListBox::implSetDefaultImages()
 {
 ImageProvider aImageProvider;
@@ -122,12 +118,12 @@ void OTableTreeListBox::notifyHiContrastChanged()
 Image aImage;
 if ( isFolderEntry( pEntryLoop ) )
 {
-aImage = m_pImageProvider-getFolderImage( 
DatabaseObject::TABLE );
+aImage = m_xImageProvider-getFolderImage( 
DatabaseObject::TABLE );
 }
 else
 {
 OUString sCompleteName( getQualifiedTableName( pEntryLoop 
) );
-m_pImageProvider-getImages( sCompleteName, 
DatabaseObject::TABLE, aImage );
+m_xImageProvider-getImages( sCompleteName, 
DatabaseObject::TABLE, aImage );
 }
 
 pContextBitmapItem-SetBitmap1( aImage );
@@ -142,7 +138,7 @@ void OTableTreeListBox::notifyHiContrastChanged()
 void OTableTreeListBox::implOnNewConnection( const Reference XConnection  
_rxConnection )
 {
 m_xConnection = _rxConnection;
-m_pImageProvider.reset( new ImageProvider( m_xConnection  ) );
+m_xImageProvider.reset( new ImageProvider( m_xConnection  ) );
 }
 
 void OTableTreeListBox::UpdateTableList( const Reference XConnection  
_rxConnection ) throw(SQLException)
@@ -469,7 +465,7 @@ SvTreeListEntry* OTableTreeListBox::implAddEntry(
 pRet = InsertEntry( sName, pParentEntry, sal_False, LIST_APPEND );
 
 Image aImage;
-m_pImageProvider-getImages( _rTableName, DatabaseObject::TABLE, 
aImage );
+m_xImageProvider-getImages( _rTableName, DatabaseObject::TABLE, 
aImage );
 
 SetExpandedEntryBmp( pRet, aImage );
 SetCollapsedEntryBmp( pRet, aImage );
diff --git a/dbaccess/source/ui/inc/tabletree.hxx 
b/dbaccess/source/ui/inc/tabletree.hxx
index 10f2959..2972867 100644
--- a/dbaccess/source/ui/inc/tabletree.hxx
+++ b/dbaccess/source/ui/inc/tabletree.hxx
@@ -20,6 +20,7 @@
 #ifndef INCLUDED_DBACCESS_SOURCE_UI_INC_TABLETREE_HXX
 #define INCLUDED_DBACCESS_SOURCE_UI_INC_TABLETREE_HXX
 
+#include imageprovider.hxx
 #include marktree.hxx
 
 #include 

Re: long paras + tools/string.hxx

2014-01-21 Thread Olivier Hallot
Hi Caolán

Em 21-01-2014 10:10, Caolán McNamara escreveu:
 Also, note there is a hidden limit in tools/stream.hxx of 0xFFFE in
 things like ReadLine and ReadByteStringLine and their callers.

Is this the reason for the memory leak that I get while saving  a
longpara file? It crashes my Linux.

-- 
Olivier Hallot
Comunidade LibreOffice
http://ask.libreoffice.org/pt-br
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: 5 commits - jvmfwk/plugins jvmfwk/source stoc/source svl/source tools/source unotools/source

2014-01-21 Thread Stephan Bergmann
 jvmfwk/plugins/sunmajor/pluginlib/util.cxx |2 +-
 jvmfwk/source/framework.cxx|4 ++--
 stoc/source/security/permissions.cxx   |2 +-
 svl/source/config/ctloptions.cxx   |2 +-
 tools/source/fsys/urlobj.cxx   |2 +-
 unotools/source/config/cmdoptions.cxx  |2 +-
 unotools/source/config/compatibility.cxx   |2 +-
 unotools/source/config/dynamicmenuoptions.cxx  |2 +-
 unotools/source/config/eventcfg.cxx|2 +-
 unotools/source/config/extendedsecurityoptions.cxx |2 +-
 unotools/source/config/fontoptions.cxx |2 +-
 unotools/source/config/localisationoptions.cxx |2 +-
 unotools/source/config/moduleoptions.cxx   |2 +-
 13 files changed, 14 insertions(+), 14 deletions(-)

New commits:
commit 891ffcb141239cb0c51b37f7cc9be0957915ec31
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 12:07:55 2014 +0100

bool improvements

Change-Id: I2733b729fe516c0e857490369d493bb7d8fb7c77

diff --git a/unotools/source/config/cmdoptions.cxx 
b/unotools/source/config/cmdoptions.cxx
index e2da7df..1bfb2f6 100644
--- a/unotools/source/config/cmdoptions.cxx
+++ b/unotools/source/config/cmdoptions.cxx
@@ -236,7 +236,7 @@ SvtCommandOptions_Impl::SvtCommandOptions_Impl()
 SvtCommandOptions_Impl::~SvtCommandOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/compatibility.cxx 
b/unotools/source/config/compatibility.cxx
index 5d09386..78f280a 100644
--- a/unotools/source/config/compatibility.cxx
+++ b/unotools/source/config/compatibility.cxx
@@ -420,7 +420,7 @@ SvtCompatibilityOptions_Impl::SvtCompatibilityOptions_Impl()
 SvtCompatibilityOptions_Impl::~SvtCompatibilityOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/dynamicmenuoptions.cxx 
b/unotools/source/config/dynamicmenuoptions.cxx
index 990c2b5..964cf50 100644
--- a/unotools/source/config/dynamicmenuoptions.cxx
+++ b/unotools/source/config/dynamicmenuoptions.cxx
@@ -456,7 +456,7 @@ SvtDynamicMenuOptions_Impl::SvtDynamicMenuOptions_Impl()
 SvtDynamicMenuOptions_Impl::~SvtDynamicMenuOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/eventcfg.cxx 
b/unotools/source/config/eventcfg.cxx
index 3fcfc8f..a052613 100644
--- a/unotools/source/config/eventcfg.cxx
+++ b/unotools/source/config/eventcfg.cxx
@@ -126,7 +126,7 @@ GlobalEventConfig_Impl::GlobalEventConfig_Impl()
 GlobalEventConfig_Impl::~GlobalEventConfig_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/extendedsecurityoptions.cxx 
b/unotools/source/config/extendedsecurityoptions.cxx
index ec6f5cc..2b3674e 100644
--- a/unotools/source/config/extendedsecurityoptions.cxx
+++ b/unotools/source/config/extendedsecurityoptions.cxx
@@ -191,7 +191,7 @@ 
SvtExtendedSecurityOptions_Impl::SvtExtendedSecurityOptions_Impl()
 SvtExtendedSecurityOptions_Impl::~SvtExtendedSecurityOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/fontoptions.cxx 
b/unotools/source/config/fontoptions.cxx
index c661fd5..e823087 100644
--- a/unotools/source/config/fontoptions.cxx
+++ b/unotools/source/config/fontoptions.cxx
@@ -184,7 +184,7 @@ SvtFontOptions_Impl::SvtFontOptions_Impl()
 SvtFontOptions_Impl::~SvtFontOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/localisationoptions.cxx 
b/unotools/source/config/localisationoptions.cxx
index 1d67f9a..4fac21c 100644
--- a/unotools/source/config/localisationoptions.cxx
+++ b/unotools/source/config/localisationoptions.cxx
@@ -173,7 +173,7 @@ SvtLocalisationOptions_Impl::SvtLocalisationOptions_Impl()
 SvtLocalisationOptions_Impl::~SvtLocalisationOptions_Impl()
 {
 // We must save our current values .. if user forget it!
-if( IsModified() == sal_True )
+if( IsModified() )
 {
 Commit();
 }
diff --git a/unotools/source/config/moduleoptions.cxx 
b/unotools/source/config/moduleoptions.cxx
index c72fff8..aa3ce5a 100644
--- a/unotools/source/config/moduleoptions.cxx
+++ b/unotools/source/config/moduleoptions.cxx
@@ -403,7 +403,7 @@ 

Re: space between values and units in spinners, fdo#71688

2014-01-21 Thread Stephan Bergmann

On 01/21/2014 12:59 PM, Caolán McNamara wrote:

So to resolve this we now have a function formatPercent which takes the
number and the locale and runs it through icu's PercentInstance to
format it according to that locale's formatting rule. Which gives e.g.
10 % for de and 10% for en. (http://site.icu-project.org/)


But at least my copy of Duden Satz- und Korrekturanweisungen asks for 
a thin instead of a full space in front of percent (and per-mille) signs 
in German.  Could we handle that, too, or is that beyond the state of 
the art in UI typography?


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


Re: New Developer

2014-01-21 Thread Manohar Kh
I tried doing the above step mentioned but it is still not working for me...

manohar@crazyHorse:~/Development$ git clone
https://gerrit.libreoffice.org/p/core.git
Cloning into 'core'...
error: server certificate verification failed. CAfile:
/etc/ssl/certs/ca-certificates.crt CRLfile: none while accessing
https://gerrit.libreoffice.org/p/core.git/info/refs
fatal: HTTP request failed



On Tue, Jan 21, 2014 at 3:41 PM, Miklos Vajna vmik...@collabora.co.ukwrote:

 Hi Manohar,

 On Mon, Jan 20, 2014 at 11:12:17PM +0530, Manohar Kh khmanoh...@gmail.com
 wrote:
  manohar@crazyHorse:~/Development$ git clone git://
  anongit.freedesktop.org/libreoffice/core libreoffice
  Cloning into 'libreoffice'...
  fatal: unable to connect to anongit.freedesktop.org:
  anongit.freedesktop.org[0: 131.252.210.161]: errno=Connection timed out

 In case you're behind a proxy, try https:

 git clone https://gerrit.libreoffice.org/p/core.git

 Miklos

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


Re: space between values and units in spinners, fdo#71688

2014-01-21 Thread Khaled Hosny
On Tue, Jan 21, 2014 at 01:27:37PM +0100, Stephan Bergmann wrote:
 On 01/21/2014 12:59 PM, Caolán McNamara wrote:
 So to resolve this we now have a function formatPercent which takes the
 number and the locale and runs it through icu's PercentInstance to
 format it according to that locale's formatting rule. Which gives e.g.
 10 % for de and 10% for en. (http://site.icu-project.org/)
 
 But at least my copy of Duden Satz- und Korrekturanweisungen asks for a
 thin instead of a full space in front of percent (and per-mille) signs in
 German.  Could we handle that, too, or is that beyond the state of the art
 in UI typography?

Unicode has a thin space character (U+2009), so if translators used it
it should just work. The tricky part would be ICU.

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


Re: New Developer

2014-01-21 Thread Miklos Vajna
On Tue, Jan 21, 2014 at 06:32:09PM +0530, Manohar Kh khmanoh...@gmail.com 
wrote:
 manohar@crazyHorse:~/Development$ git clone
 https://gerrit.libreoffice.org/p/core.git
 Cloning into 'core'...
 error: server certificate verification failed. CAfile:
 /etc/ssl/certs/ca-certificates.crt CRLfile: none while accessing
 https://gerrit.libreoffice.org/p/core.git/info/refs
 fatal: HTTP request failed

Seems you have a problematic /etc/ssl/certs/ca-certificates.crt. Did you
try http insetad of https?


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


Re: long paras + tools/string.hxx

2014-01-21 Thread Caolán McNamara
On Tue, 2014-01-21 at 10:21 -0200, Olivier Hallot wrote:
 Hi Caolán
 
 Em 21-01-2014 10:10, Caolán McNamara escreveu:
  Also, note there is a hidden limit in tools/stream.hxx of 0xFFFE in
  things like ReadLine and ReadByteStringLine and their callers.
 
 Is this the reason for the memory leak that I get while saving  a
 longpara file? It crashes my Linux.

Dunno. Rather doubt it though. How to reproduce ?

C.

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


[Libreoffice-commits] core.git: 2 commits - solenv/bin

2014-01-21 Thread Tor Lillqvist
 solenv/bin/native-code.py |   15 +--
 1 file changed, 9 insertions(+), 6 deletions(-)

New commits:
commit 4917be8a6f0bd650d9259b14af844f9bc081d2fd
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 15:11:24 2014 +0200

Sort the sections of core_constructor_list for clarity

Change-Id: I845c26d0b69c08e07fb7159fb2bd6862ad9e5603

diff --git a/solenv/bin/native-code.py b/solenv/bin/native-code.py
index 7bbee8a..2c91640 100755
--- a/solenv/bin/native-code.py
+++ b/solenv/bin/native-code.py
@@ -51,22 +51,22 @@ core_factory_list = [
 
 core_constructor_list = [
 # framework/util/fwk.component
-com_sun_star_comp_framework_ModuleManager_get_implementation,
-com_sun_star_comp_framework_URLTransformer_get_implementation,
-com_sun_star_comp_framework_Frame_get_implementation,
-com_sun_star_comp_framework_LayoutManager_get_implementation,
 com_sun_star_comp_framework_AddonsToolBarFactory_get_implementation,
 com_sun_star_comp_framework_AutoRecovery_get_implementation,
+com_sun_star_comp_framework_Frame_get_implementation,
 com_sun_star_comp_framework_JobExecutor_get_implementation,
+com_sun_star_comp_framework_LayoutManager_get_implementation,
+com_sun_star_comp_framework_ModuleManager_get_implementation,
 com_sun_star_comp_framework_StatusIndicatorFactory_get_implementation,
+com_sun_star_comp_framework_URLTransformer_get_implementation,
 # sax/source/expatwrap/expwrap.component
-com_sun_star_comp_extensions_xml_sax_ParserExpat_get_implementation,
 com_sun_star_comp_extensions_xml_sax_FastParser_get_implementation,
+com_sun_star_comp_extensions_xml_sax_ParserExpat_get_implementation,
 # sfx2/util/sfx.component
+SfxDocumentMetaData_get_implementation,
+com_sun_star_comp_office_FrameLoader_get_implementation,
 com_sun_star_comp_sfx2_DocumentTemplates_get_implementation,
 com_sun_star_comp_sfx2_GlobalEventBroadcaster_get_implementation,
-com_sun_star_comp_office_FrameLoader_get_implementation,
-SfxDocumentMetaData_get_implementation,
 # svtools/util/svt.component
 com_sun_star_comp_graphic_GraphicProvider_get_implementation,
 # svx/util/svx.component
commit e683bb91915ac52743659bc0434a73ab8eab05b9
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 15:08:26 2014 +0200

Add fwk AutoRecovery, JobExecutor and StatusIndicatorFactory ctors to core

Seem to be needed by TiledLibreOffice at least, so possibly by any
app.

Change-Id: I52d4e6f4ce66e2a79b67e1f17ba926defd4ac8eb

diff --git a/solenv/bin/native-code.py b/solenv/bin/native-code.py
index 9e8f1a2..7bbee8a 100755
--- a/solenv/bin/native-code.py
+++ b/solenv/bin/native-code.py
@@ -56,6 +56,9 @@ core_constructor_list = [
 com_sun_star_comp_framework_Frame_get_implementation,
 com_sun_star_comp_framework_LayoutManager_get_implementation,
 com_sun_star_comp_framework_AddonsToolBarFactory_get_implementation,
+com_sun_star_comp_framework_AutoRecovery_get_implementation,
+com_sun_star_comp_framework_JobExecutor_get_implementation,
+com_sun_star_comp_framework_StatusIndicatorFactory_get_implementation,
 # sax/source/expatwrap/expwrap.component
 com_sun_star_comp_extensions_xml_sax_ParserExpat_get_implementation,
 com_sun_star_comp_extensions_xml_sax_FastParser_get_implementation,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: long paras + tools/string.hxx

2014-01-21 Thread Olivier Hallot
Hi Caolán

Em 21-01-2014 11:12, Caolán McNamara escreveu:
 On Tue, 2014-01-21 at 10:21 -0200, Olivier Hallot wrote:
 Hi Caolán

 Em 21-01-2014 10:10, Caolán McNamara escreveu:
 Also, note there is a hidden limit in tools/stream.hxx of 0xFFFE in
 things like ReadLine and ReadByteStringLine and their callers.

 Is this the reason for the memory leak that I get while saving  a
 longpara file? It crashes my Linux.
 
 Dunno. Rather doubt it though. How to reproduce ?
 

Just save a file with one longpara (master as old as last friday).
-- 
Olivier Hallot
Comunidade LibreOffice
http://ask.libreoffice.org/pt-br
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - solenv/gbuild

2014-01-21 Thread Stephan Bergmann
 solenv/gbuild/UnoApi.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 460f5cab4105451d86451c63be7891be41974911
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 10:56:08 2014 +0100

idl files need to be packaged relative to INSTDIR, not INSTROOT

...so that on OS X they end up in LibreOffice*_SDK.app/idl/, not
LibreOffice.app/Contents/LibreOffice*_SDK.app/idl/.

Change-Id: Id001aed334a0e1d8f835ce797a17f1f67d1a3abb
(cherry picked from commit f4ccd054a94de1290e01bd21d5d45f1720f7cb74)
Reviewed-on: https://gerrit.libreoffice.org/7569
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/solenv/gbuild/UnoApi.mk b/solenv/gbuild/UnoApi.mk
index 21ce4d6..6e39b3d 100644
--- a/solenv/gbuild/UnoApi.mk
+++ b/solenv/gbuild/UnoApi.mk
@@ -20,6 +20,7 @@ $(call gb_UnoApiTarget_UnoApiTarget,$(1),$(1))
 $(call gb_UnoApiHeadersTarget_UnoApiHeadersTarget,$(1))
 ifneq ($(gb_UnoApi_ENABLE_INSTALL),)
 $(call gb_Package_Package_internal,$(1)_idl,$(SRCDIR))
+$(call gb_Package_set_outdir,$(1)_idl,$(INSTDIR))
 endif
 
 $(call gb_UnoApi_get_target,$(1)) :| $(dir $(call 
gb_UnoApi_get_target,$(1))).dir
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: long paras + tools/string.hxx

2014-01-21 Thread Matteo Casalin
Hi all,

On Tue, 21 Jan 2014 12:10:40 +
Caolán McNamara caol...@redhat.com wrote:

 The 16 bit length limitation in writer is now removed towards 4.3.
 
 The 16 bit paragraph length limit in editeng still remains, editeng
 mixes freely sal_uInt16 with xub_StrLen and has a pile of hard-coded
 0x here and there which makes it tricky to see at a glance which
 values are string lengths and offsets and which are not. Sigh.

I'l (very) slowly working on this. Indeed there are some obscure (to me) points 
in which I don't know how to proceed.
I will push to gerrit for review, just to have some cross checks, and the first 
bunch of patches should be ready in the next few days.

BTW: is there any chance that some use of sal_uInt16 is left behind in the 
already converted code? I'm asking just to know what to focus on after this 
task is finally done.

Thanks
Matteo

 Also, note there is a hidden limit in tools/stream.hxx of 0xFFFE in
 things like ReadLine and ReadByteStringLine and their callers.
 
 sc is now the only module that includes tools/string.hxx and uses
 STRING_MAXLEN. It would be great if some calc hackers could have a look
 there to see if those limits still make sense (e.g.
 nArbitraryLineLengthLimit) especially considering the ReadLine, etc.
 issue.
 
 C.
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: New Developer

2014-01-21 Thread Manohar Kh
Yes , but still it is the same problem .


On Tue, Jan 21, 2014 at 6:36 PM, Miklos Vajna vmik...@collabora.co.ukwrote:

 On Tue, Jan 21, 2014 at 06:32:09PM +0530, Manohar Kh khmanoh...@gmail.com
 wrote:
  manohar@crazyHorse:~/Development$ git clone
  https://gerrit.libreoffice.org/p/core.git
  Cloning into 'core'...
  error: server certificate verification failed. CAfile:
  /etc/ssl/certs/ca-certificates.crt CRLfile: none while accessing
  https://gerrit.libreoffice.org/p/core.git/info/refs
  fatal: HTTP request failed

 Seems you have a problematic /etc/ssl/certs/ca-certificates.crt. Did you
 try http insetad of https?

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


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

2014-01-21 Thread Norbert Thiebaud
 vcl/source/gdi/pdfwriter_impl2.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit cdfebf30946307998eb3965ad30bbcd34ce82b02
Author: Norbert Thiebaud nthieb...@gmail.com
Date:   Tue Jan 21 01:48:30 2014 -0600

coverity#707581

Change-Id: I9e9ed9b83d830b1ebe2aae35036f49a649837765

diff --git a/vcl/source/gdi/pdfwriter_impl2.cxx 
b/vcl/source/gdi/pdfwriter_impl2.cxx
index 121f23d..b045659 100644
--- a/vcl/source/gdi/pdfwriter_impl2.cxx
+++ b/vcl/source/gdi/pdfwriter_impl2.cxx
@@ -1370,6 +1370,10 @@ bool PDFWriterImpl::computeEncryptionKey( 
EncHashTransporter* i_pTransporter, vc
 }
 }
 }
+else
+{
+bSuccess = false;
+}
 }
 else
 bSuccess = false;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Miklos Vajna
 oox/source/export/drawingml.cxx |   10 +-
 sw/qa/extras/ooxmlexport/data/drawinglayer-pic-pos.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx|   15 +++
 3 files changed, 24 insertions(+), 1 deletion(-)

New commits:
commit 3aef7b3b3dba2d898f3a278e916166e7c3d87321
Author: Miklos Vajna vmik...@collabora.co.uk
Date:   Tue Jan 21 12:08:00 2014 +0100

drawingML export: fix position of wrapped shapes in DOCX

In case of DOCX, the DOCX wrapper around the drawingML fragment already
contains the real position of the shape (which talks about paragraphs
and other concepts which are unavailable in a spreadsheet /
presentation), and the inner position should be 0, otherwise the shape
is shifted towards the bottom right corner.

Note that this only affects drawinglayer shapes in DOCX, Writer pictures
are handled in sw, and those always had a 0,0 position in the inner
drawingML fragment.

Change-Id: I582b1ae64387b50ffb051e1f5017eab0e5d5ab34

diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index 2cddbf5..f81cd91 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -714,7 +714,15 @@ void DrawingML::WriteTransformation( const Rectangle 
rRect,
   XML_rot, (nRotation % 2160) ? I32S( nRotation ) 
: NULL,
   FSEND );
 
-mpFS-singleElementNS( XML_a, XML_off, XML_x, IS( MM100toEMU( rRect.Left() 
) ), XML_y, IS( MM100toEMU( rRect.Top() ) ), FSEND );
+sal_Int32 nLeft = rRect.Left();
+sal_Int32 nTop = rRect.Top();
+if (GetDocumentType() == DOCUMENT_DOCX  !m_xParent.is())
+{
+nLeft = 0;
+nTop = 0;
+}
+
+mpFS-singleElementNS( XML_a, XML_off, XML_x, IS( MM100toEMU( nLeft ) ), 
XML_y, IS( MM100toEMU( nTop ) ), FSEND );
 mpFS-singleElementNS( XML_a, XML_ext, XML_cx, IS( MM100toEMU( 
rRect.GetWidth() ) ), XML_cy, IS( MM100toEMU( rRect.GetHeight() ) ), FSEND );
 
 mpFS-endElementNS( nXmlNamespace, XML_xfrm );
diff --git a/sw/qa/extras/ooxmlexport/data/drawinglayer-pic-pos.docx 
b/sw/qa/extras/ooxmlexport/data/drawinglayer-pic-pos.docx
new file mode 100644
index 000..bb18ab3
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/drawinglayer-pic-pos.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index ae4ab8c..8bfc251 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -147,6 +147,7 @@ xmlNodeSetPtr Test::getXPathNode(xmlDocPtr pXmlDoc, const 
OString rXPath)
 xmlXPathRegisterNs(pXmlXpathCtx, BAD_CAST(wpg), 
BAD_CAST(http://schemas.microsoft.com/office/word/2010/wordprocessingGroup;));
 xmlXPathRegisterNs(pXmlXpathCtx, BAD_CAST(wp), 
BAD_CAST(http://schemas.openxmlformats.org/drawingml/2006/wordprocessingDrawing;));
 xmlXPathRegisterNs(pXmlXpathCtx, BAD_CAST(a), 
BAD_CAST(http://schemas.openxmlformats.org/drawingml/2006/main;));
+xmlXPathRegisterNs(pXmlXpathCtx, BAD_CAST(pic), 
BAD_CAST(http://schemas.openxmlformats.org/drawingml/2006/picture;));
 xmlXPathObjectPtr pXmlXpathObj = 
xmlXPathEvalExpression(BAD_CAST(rXPath.getStr()), pXmlXpathCtx);
 return pXmlXpathObj-nodesetval;
 }
@@ -2493,6 +2494,20 @@ DECLARE_OOXMLEXPORT_TEST(testDmlTextshape, 
dml-textshape.docx)
 CPPUNIT_ASSERT_EQUAL(sal_Int32(1256), xShape-getPosition().Y);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testDrawinglayerPicPos, drawinglayer-pic-pos.docx)
+{
+// The problem was that the position of the picture was incorrect, it was 
shifted towards the bottom right corner.
+xmlDocPtr pXmlDocument = parseExport(word/document.xml);
+if (!pXmlDocument)
+return;
+
+OString 
aXPath(/w:document/w:body/w:p[1]/w:r[1]/mc:AlternateContent/mc:Choice/w:drawing/wp:anchor/a:graphic/a:graphicData/pic:pic/pic:spPr/a:xfrm/a:off);
+// This was 720.
+assertXPath(pXmlDocument, aXPath, x, 0);
+// This was 1828800.
+assertXPath(pXmlDocument, aXPath, y, 0);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testShapeThemePreservation, 
shape-theme-preservation.docx)
 {
 xmlDocPtr pXmlDocument = parseExport(word/document.xml);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'aoo/trunk' - basctl/source dbaccess/source filter/source sc/source sd/source svtools/source svx/source sw/source vcl/aqua

2014-01-21 Thread Herbert Dürr
 basctl/source/dlged/dlged.cxx  |2 +-
 dbaccess/source/ui/querydesign/JoinTableView.cxx   |2 +-
 filter/source/msfilter/svdfppt.cxx |2 +-
 sc/source/filter/excel/tokstack.cxx|2 +-
 sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx |2 +-
 sc/source/filter/xml/XMLStylesExportHelper.cxx |2 +-
 sd/source/filter/xml/sdxmlwrp.cxx  |2 +-
 svtools/source/edit/syntaxhighlight.cxx|2 +-
 svx/source/table/cell.cxx  |2 +-
 svx/source/unodraw/unoshape.cxx|2 +-
 sw/source/ui/app/appopt.cxx|2 +-
 vcl/aqua/source/app/salinst.cxx|2 +-
 12 files changed, 12 insertions(+), 12 deletions(-)

New commits:
commit 275a1d7a011e79a7f9c44c51e947d2cb50197051
Author: Herbert Dürr h...@apache.org
Date:   Tue Jan 21 13:56:09 2014 +

get rid of an extraneous parentheses in if((A==B))

diff --git a/basctl/source/dlged/dlged.cxx b/basctl/source/dlged/dlged.cxx
index 0623039..296a3f3 100644
--- a/basctl/source/dlged/dlged.cxx
+++ b/basctl/source/dlged/dlged.cxx
@@ -514,7 +514,7 @@ void DlgEditor::MouseButtonUp( const MouseEvent rMEvt )
 {
 sal_Bool bRet = pFunc-MouseButtonUp( rMEvt );
 
-if( (eMode == DLGED_INSERT) )
+if( eMode == DLGED_INSERT )
 bCreateOK = bRet;
 }
 
diff --git a/dbaccess/source/ui/querydesign/JoinTableView.cxx 
b/dbaccess/source/ui/querydesign/JoinTableView.cxx
index 6f57917..27c560e 100644
--- a/dbaccess/source/ui/querydesign/JoinTableView.cxx
+++ b/dbaccess/source/ui/querydesign/JoinTableView.cxx
@@ -1462,7 +1462,7 @@ long OJoinTableView::PreNotify(NotifyEvent rNEvt)
 // no conn for any reason - select the next 
or previous tab win
 if(bForward)
 {
-if ((aIter-second == 
m_aTableMap.rbegin()-second))
+if( aIter-second == 
m_aTableMap.rbegin()-second)
 pNextWin = m_aTableMap.begin()-second;
 else
 {
diff --git a/filter/source/msfilter/svdfppt.cxx 
b/filter/source/msfilter/svdfppt.cxx
index 51af756..692e3d0 100644
--- a/filter/source/msfilter/svdfppt.cxx
+++ b/filter/source/msfilter/svdfppt.cxx
@@ -1089,7 +1089,7 @@ SdrObject* SdrEscherImport::ProcessObj( SvStream rSt, 
DffObjData rObjData, voi
 }
 
 sal_uInt32 nDestinationInstance = aTextObj.GetInstance();
-if ( ( rPersistEntry.ePageKind == PPT_MASTERPAGE ) )
+if ( rPersistEntry.ePageKind == PPT_MASTERPAGE )
 {
 if ( !rPersistEntry.pPresentationObjects )
 {
diff --git a/sc/source/filter/excel/tokstack.cxx 
b/sc/source/filter/excel/tokstack.cxx
index 5701f71..5be7dbe 100644
--- a/sc/source/filter/excel/tokstack.cxx
+++ b/sc/source/filter/excel/tokstack.cxx
@@ -939,7 +939,7 @@ const String* TokenPool::GetExternal( const TokenId rId ) 
const
 if( n  n = nElementAkt )
 {
 n--;
-if( (pType[ n ] == T_Ext) )
+if( pType[ n ] == T_Ext )
 {
 sal_uInt16 nExt = pElement[ n ];
 if ( nExt  nP_Ext  ppP_Ext[ nExt ] )
diff --git a/sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx 
b/sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx
index c4929fd..2173c9d 100644
--- a/sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx
+++ b/sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx
@@ -558,7 +558,7 @@ void 
ScXMLChangeTrackingImportHelper::CreateGeneratedActions(ScMyGeneratedList
 ScMyGeneratedList::iterator aEndItr(rList.end());
 while (aItr != aEndItr)
 {
-if (((*aItr)-nID == 0))
+if( (*aItr)-nID == 0)
 {
 ScBaseCell* pCell = NULL;
 if ((*aItr)-pCellInfo)
diff --git a/sc/source/filter/xml/XMLStylesExportHelper.cxx 
b/sc/source/filter/xml/XMLStylesExportHelper.cxx
index 5faea3e..ad58ffd 100644
--- a/sc/source/filter/xml/XMLStylesExportHelper.cxx
+++ b/sc/source/filter/xml/XMLStylesExportHelper.cxx
@@ -331,7 +331,7 @@ void ScMyValidationsContainer::WriteMessage(ScXMLExport 
rExport,
 sal_Bool bPrevCharWasSpace(sal_True);
 while(i  sText.getLength())
 {
-if ((sText[i] == '\n'))
+if( sText[i] == '\n')
 {
 SvXMLElementExport aElemP(rExport, XML_NAMESPACE_TEXT, XML_P, 
sal_True, sal_False);
 
rExport.GetTextParagraphExport()-exportText(sTemp.makeStringAndClear(), 
bPrevCharWasSpace);
diff --git a/sd/source/filter/xml/sdxmlwrp.cxx 
b/sd/source/filter/xml/sdxmlwrp.cxx
index 99250db..c0e404b 100644
--- a/sd/source/filter/xml/sdxmlwrp.cxx
+++ 

Re: Mac OS X 64-bit, libstdc++ vs. libc++

2014-01-21 Thread Norbert Thiebaud
On Tue, Jan 21, 2014 at 6:10 AM, Stephan Bergmann sberg...@redhat.com wrote:
 On 01/17/2014 10:02 AM, Stephan Bergmann wrote:

 Therefore, I would suggest to stay with a broken Mac OS X 64-bit SDK for LO
 4.2.0 and only backport those fixes to libreoffice-4-2 towards LO 4.2.1.
 Other opinions?

Fine

Norbert

PS: upgrading xcode indeed solved the build problems.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: space between values and units in spinners, fdo#71688

2014-01-21 Thread Caolán McNamara
On Tue, 2014-01-21 at 15:01 +0200, Khaled Hosny wrote:
 On Tue, Jan 21, 2014 at 01:27:37PM +0100, Stephan Bergmann wrote:
  On 01/21/2014 12:59 PM, Caolán McNamara wrote:
  So to resolve this we now have a function formatPercent which takes the
  number and the locale and runs it through icu's PercentInstance to
  format it according to that locale's formatting rule. Which gives e.g.
  10 % for de and 10% for en. (http://site.icu-project.org/)
  
  But at least my copy of Duden Satz- und Korrekturanweisungen asks for a
  thin instead of a full space in front of percent (and per-mille) signs in
  German.  Could we handle that, too, or is that beyond the state of the art
  in UI typography?
 
 Unicode has a thin space character (U+2009), so if translators used it
 it should just work. The tricky part would be ICU.

Yeah it should just work in the UI to use a narrow space. Though because
icu is currently using 0x00A0 no-break space presumably 0x202F narrow
no-break space is desirable given its similar properties. I can easily
special case replacing 0x00A0 with 0x202F from the icu result for de.

FWIW, the percent pattern formats for the various locales can be seen at
http://www.unicode.org/cldr/charts/24/by_type/numbers.number_formatting_patterns.html
under standard-percent

C.

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


Re: long paras + tools/string.hxx

2014-01-21 Thread Caolán McNamara
On Tue, 2014-01-21 at 14:20 +0100, Matteo Casalin wrote:
 I'm (very) slowly working on this. Indeed there are some obscure (to
 me) points in which I don't know how to proceed. I will push to gerrit
 for review, just to have some cross checks, and the first bunch of
 patches should be ready in the next few days.
 
 BTW: is there any chance that some use of sal_uInt16 is left behind in
 the already converted code? I'm asking just to know what to focus on
 after this task is finally done.

There is a chance alright. But in writer at least I made an effort in
the first round to find all the sal_uInt16s which were really
xub_StrLens and rename them as xub_StrLen before going on later to
convert them to sal_Int32. Which doesn't mean I'm sure I got them all of
course.

C.

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


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

2014-01-21 Thread Tor Lillqvist
 svx/source/fmcomp/fmgridcl.cxx |2 +-
 svx/source/fmcomp/gridctrl.cxx |2 +-
 svx/source/svdraw/svdview.cxx  |2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 0a4abbda7c08ecc52712e356371a415137137929
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 16:27:37 2014 +0200

WaE: C4805: unsafe mix of type 'bool' and type 'sal_Bool'

Change-Id: Ia622d25a454eb2fe1928e095c635ad4655032f15

diff --git a/svx/source/fmcomp/gridctrl.cxx b/svx/source/fmcomp/gridctrl.cxx
index 07a41be..69398e8 100644
--- a/svx/source/fmcomp/gridctrl.cxx
+++ b/svx/source/fmcomp/gridctrl.cxx
@@ -2539,7 +2539,7 @@ void DbGridControl::AppendNew()
 
 void DbGridControl::SetDesignMode(bool bMode)
 {
-if (IsDesignMode() != bMode)
+if ((bool) IsDesignMode() != bMode)
 {
 // adjust Enable/Disable for design mode so that the headerbar remains 
configurable
 if (bMode)
commit 2e4157369a7b5127e4a3b312c944a3044b36845a
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 16:26:44 2014 +0200

WaE: C4805: unsafe mix of type 'bool' and type 'sal_Bool'

Change-Id: Ibfd666d4c7b2d92a65a01a84fe83319f056b1e7b

diff --git a/svx/source/fmcomp/fmgridcl.cxx b/svx/source/fmcomp/fmgridcl.cxx
index 27b657c..2a7f49d 100644
--- a/svx/source/fmcomp/fmgridcl.cxx
+++ b/svx/source/fmcomp/fmgridcl.cxx
@@ -1070,7 +1070,7 @@ void FmGridControl::propertyChange(const 
::com::sun::star::beans::PropertyChange
 
 void FmGridControl::SetDesignMode(bool bMode)
 {
-sal_Bool bOldMode = IsDesignMode();
+bool bOldMode = IsDesignMode();
 DbGridControl::SetDesignMode(bMode);
 if (bOldMode != bMode)
 {
commit eb1b3360e641aaf5c9c8e142473c6e88128bc766
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 16:23:52 2014 +0200

WaE: C4805: unsafe mix of type 'bool' and type 'sal_Bool'

Change-Id: I3dfa4495ca7eeba3b3f4f6d81a2ef90fe60ca2d5

diff --git a/svx/source/svdraw/svdview.cxx b/svx/source/svdraw/svdview.cxx
index e8ef1ce..8b5f70a 100644
--- a/svx/source/svdraw/svdview.cxx
+++ b/svx/source/svdraw/svdview.cxx
@@ -1442,7 +1442,7 @@ void SdrView::onAccessibilityOptionsChanged()
 
 void SdrView::SetMasterPagePaintCaching(sal_Bool bOn)
 {
-if(mbMasterPagePaintCaching != bOn)
+if(mbMasterPagePaintCaching != (bool) bOn)
 {
 mbMasterPagePaintCaching = bOn;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Tor Lillqvist
 oox/source/export/drawingml.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 35539318cecddc9ccb2904573a894ea05adc432c
Author: Tor Lillqvist t...@collabora.com
Date:   Tue Jan 21 16:50:58 2014 +0200

WaE: C4701: potentially uninitialized local variable 'nIdx' used

Change-Id: If05dd971ad62b059499c50bfc0b82b28e19fee4a

diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index f81cd91..2fde292 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -1722,7 +1722,7 @@ void DrawingML::WriteShapeStyle( Reference XPropertySet 
 xPropSet )
 if( aFillRefProperties.getLength()  0 )
 {
 OUString sSchemeClr;
-sal_uInt32 nIdx;
+sal_uInt32 nIdx = 0;
 for( sal_Int32 i=0; i  aFillRefProperties.getLength(); ++i)
 if( aFillRefProperties[i].Name == SchemeClr )
 aFillRefProperties[i].Value = sSchemeClr;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 4 commits - dbaccess/inc dbaccess/source dbaccess/uiconfig dbaccess/UIConfig_dbaccess.mk extras/source i18nutil/source include/tools sc/uiconfig

2014-01-21 Thread Caolán McNamara
 dbaccess/UIConfig_dbaccess.mk   |1 
 dbaccess/inc/dbaccess_helpid.hrc|3 
 dbaccess/source/ui/control/tabletree.cxx|   10 +
 dbaccess/source/ui/dlg/dbadmin.hrc  |3 
 dbaccess/source/ui/dlg/dbadmin2.src |   48 
 dbaccess/source/ui/dlg/tablespage.cxx   |  135 
 dbaccess/source/ui/dlg/tablespage.hxx   |7 -
 dbaccess/uiconfig/ui/tablesfilterpage.ui|   74 +
 extras/source/glade/libreoffice-catalog.xml.in  |6 -
 i18nutil/source/utility/unicode.cxx |8 +
 include/tools/string.hxx|3 
 sc/uiconfig/scalc/ui/conditionalformatdialog.ui |2 
 12 files changed, 169 insertions(+), 131 deletions(-)

New commits:
commit 5c72a213c7c4b588d0c1e0884ff2c46c22425faf
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 14:41:16 2014 +

convert table filter page to .ui

Change-Id: I7bfb7698e2a5303b492766fba564c7b9033bcc52

diff --git a/dbaccess/UIConfig_dbaccess.mk b/dbaccess/UIConfig_dbaccess.mk
index 7148268..a493107 100644
--- a/dbaccess/UIConfig_dbaccess.mk
+++ b/dbaccess/UIConfig_dbaccess.mk
@@ -20,6 +20,7 @@ $(eval $(call gb_UIConfig_add_uifiles,dbaccess, \
 dbaccess/uiconfig/ui/rowheightdialog \
 dbaccess/uiconfig/ui/sortdialog \
 dbaccess/uiconfig/ui/tablesfilterdialog \
+dbaccess/uiconfig/ui/tablesfilterpage \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/dbaccess/inc/dbaccess_helpid.hrc b/dbaccess/inc/dbaccess_helpid.hrc
index 2f0e3b4..e1b407f 100644
--- a/dbaccess/inc/dbaccess_helpid.hrc
+++ b/dbaccess/inc/dbaccess_helpid.hrc
@@ -39,7 +39,6 @@
 #define HID_DSADMIN_PAGE_JDBC 
DBACCESS_HID_DSADMIN_PAGE_JDBC
 #define HID_DSADMIN_PAGE_ODBC 
DBACCESS_HID_DSADMIN_PAGE_ODBC
 #define HID_DSADMIN_PAGE_TEXT 
DBACCESS_HID_DSADMIN_PAGE_TEXT
-#define HID_DSADMIN_TABLE_SUBSCRIPTION
DBACCESS_HID_DSADMIN_TABLE_SUBSCRIPTION
 #define HID_DSADMIN_DSACTIONS_NEWDS   
DBACCESS_HID_DSADMIN_DSACTIONS_NEWDS
 #define HID_DSADMIN_DSACTIONS_DELETEDS
DBACCESS_HID_DSADMIN_DSACTIONS_DELETEDS
 #define HID_DSADMIN_DSACTIONS_RESTOREDS   
DBACCESS_HID_DSADMIN_DSACTIONS_RESTOREDS
@@ -63,7 +62,6 @@
 #define HID_DSADMIN_ALL_TABLES
DBACCESS_HID_DSADMIN_ALL_TABLES
 #define HID_DSADMIN_NO_TABLES 
DBACCESS_HID_DSADMIN_NO_TABLES
 #define HID_DSADMIN_SELECT_TABLES 
DBACCESS_HID_DSADMIN_SELECT_TABLES
-#define HID_DSADMIN_TABLE_SELECTOR
DBACCESS_HID_DSADMIN_TABLE_SELECTOR
 #define HID_DSADMIN_SUPPRESS_VERSIONCL
DBACCESS_HID_DSADMIN_SUPPRESS_VERSIONCL
 
 #define HID_GENERIC_SQL_ERROR 
DBACCESS_HID_GENERIC_SQL_ERROR
@@ -178,7 +176,6 @@
 #define HID_BROWSER_VIEW_CREATE_DESIGN
DBACCESS_HID_BROWSER_VIEW_CREATE_DESIGN
 #define HID_DSADMIN_DOCUMENTS 
DBACCESS_HID_DSADMIN_DOCUMENTS
 #define HID_DSADMIN_DOCUMENTLINKS 
DBACCESS_HID_DSADMIN_DOCUMENTLINKS
-#define HID_DSADMIN_FILTER_EXPLANATION
DBACCESS_HID_DSADMIN_FILTER_EXPLANATION
 #define HID_CONFIRM_DROP_BUTTON_ALL   
DBACCESS_HID_CONFIRM_DROP_BUTTON_ALL
 #define HID_DSBROWSER_BOOKMARK_SELECTED   
DBACCESS_HID_DSBROWSER_BOOKMARK_SELECTED
 #define HID_DSADMIN_TABLE_TOOLBOX 
DBACCESS_HID_DSADMIN_TABLE_TOOLBOX
diff --git a/dbaccess/source/ui/control/tabletree.cxx 
b/dbaccess/source/ui/control/tabletree.cxx
index e1f8333..d410b29 100644
--- a/dbaccess/source/ui/control/tabletree.cxx
+++ b/dbaccess/source/ui/control/tabletree.cxx
@@ -21,6 +21,7 @@
 #include imageprovider.hxx
 #include moduledbu.hxx
 #include dbu_control.hrc
+#include vcl/layout.hxx
 #include vcl/menu.hxx
 #include connectivity/dbtools.hxx
 #include comphelper/types.hxx
@@ -73,6 +74,15 @@ OTableTreeListBox::OTableTreeListBox(Window* pParent, 
WinBits nWinStyle)
 implSetDefaultImages();
 }
 
+extern C SAL_DLLPUBLIC_EXPORT Window* SAL_CALL makeOTableTreeListBox(Window 
*pParent, VclBuilder::stringmap rMap)
+{
+WinBits nWinStyle = 0;
+OString sBorder = VclBuilder::extractCustomProperty(rMap);
+if (!sBorder.isEmpty())
+nWinStyle |= WB_BORDER;
+return new OTableTreeListBox(pParent, nWinStyle);
+}
+
 OTableTreeListBox::OTableTreeListBox( Window* pParent, const ResId rResId, 
sal_Bool _bVirtualRoot)
 :OMarkableTreeListBox(pParent, rResId)
 ,m_xImageProvider( new ImageProvider )
diff --git a/dbaccess/source/ui/dlg/dbadmin.hrc 
b/dbaccess/source/ui/dlg/dbadmin.hrc
index 99fe79c..f2f1cbc 100644
--- a/dbaccess/source/ui/dlg/dbadmin.hrc
+++ b/dbaccess/source/ui/dlg/dbadmin.hrc
@@ -31,7 +31,6 @@
 // FREE
 #define FT_JDBCDRIVERCLASS  7
 

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

2014-01-21 Thread Caolán McNamara
 helpers/help_hid.lst   |3 ---
 source/text/shared/explorer/database/menutools.xhp |4 ++--
 2 files changed, 2 insertions(+), 5 deletions(-)

New commits:
commit 23bdde918a84d61f7162980ee9356363c3393202
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 14:54:10 2014 +

update help ids for tables filter dialog .ui conversion

Change-Id: I818dd5af4e86ed9e49901f9b7655dd03109b7201

diff --git a/helpers/help_hid.lst b/helpers/help_hid.lst
index a1168c6..4c24754 100644
--- a/helpers/help_hid.lst
+++ b/helpers/help_hid.lst
@@ -1498,7 +1498,6 @@ HID_DSADMIN_DSTYPE,38807,
 HID_DSADMIN_ENABLEOUTERJOIN,39052,
 HID_DSADMIN_EXTENSION,38822,
 HID_DSADMIN_FIELD_SEPARATOR,38818,
-HID_DSADMIN_FILTER_EXPLANATION,38936,
 HID_DSADMIN_FLAT_PATH,39014,
 HID_DSADMIN_HOSTNAME,39034,
 HID_DSADMIN_IGNORECURRENCY,39151,
@@ -1546,8 +1545,6 @@ HID_DSADMIN_SHOWDELETED,38812,
 HID_DSADMIN_SPECIAL_MESSAGE,38840,
 HID_DSADMIN_SQL92CHECK,38993,
 HID_DSADMIN_SUPPRESS_VERSIONCL,38827,
-HID_DSADMIN_TABLE_SELECTOR,38826,
-HID_DSADMIN_TABLE_SUBSCRIPTION,38802,
 HID_DSADMIN_TABLE_TOOLBOX,38939,
 HID_DSADMIN_TEXT_HEADER,38817,
 HID_DSADMIN_TEXT_SEPARATOR,38819,
diff --git a/source/text/shared/explorer/database/menutools.xhp 
b/source/text/shared/explorer/database/menutools.xhp
index 127cc20..a6cee12 100644
--- a/source/text/shared/explorer/database/menutools.xhp
+++ b/source/text/shared/explorer/database/menutools.xhp
@@ -41,10 +41,10 @@
 bookmark xml-lang=en-US branch=hid/.uno:DBUserAdmin id=bm_id2447279 
localize=false/
 paragraph role=heading id=par_idN1058A xml-lang=en-US level=2 
l10n=NEWUser Administration/paragraph
 paragraph role=paragraph id=par_idN1058E xml-lang=en-US 
l10n=NEWahelp hid=.Opens the User Administration dialog if the database 
supports this feature./ahelpcommentUFI: where is this 
specced?/comment/paragraph
-bookmark xml-lang=en-US branch=hid/DBACCESS_HID_DSADMIN_TABLE_SELECTOR 
id=bm_id3153087 localize=false/
+bookmark xml-lang=en-US 
branch=hid/dbaccess/ui/tablesfilterpage/TablesFilterpage id=bm_id3153087 
localize=false/
 bookmark xml-lang=en-US branch=hid/.uno:DBTableFilter id=bm_id4928739 
localize=false/
 paragraph role=heading id=hd_id3153880 xml-lang=en-US level=2 
l10n=U oldref=13Table Filter/paragraph
-paragraph role=paragraph id=par_id3153252 xml-lang=en-US l10n=U 
oldref=14ahelp hid=HID_DSADMIN_TABLE_SELECTOROpens the Table Filter 
dialog where you can specify which tables of the database to show or to 
hide./ahelpcommentsee text in 
shared\explorer\database\1109.xhp/comment/paragraph
+paragraph role=paragraph id=par_id3153252 xml-lang=en-US l10n=U 
oldref=14ahelp hid=dbaccess/ui/tablesfilterpage/TablesFilterpageOpens 
the Table Filter dialog where you can specify which tables of the database to 
show or to hide./ahelpcommentsee text in 
shared\explorer\database\1109.xhp/comment/paragraph
 paragraph role=paragraph id=par_id3150670 xml-lang=en-US l10n=U 
oldref=20Select the tables that you want to filter in the 
emphFilter/emph list./paragraph
 paragraph role=paragraph id=par_id3150985 xml-lang=en-US l10n=U 
oldref=23If you select the topmost table in a hierarchy, all of the tables 
in the hierarchy are selected./paragraph
 paragraph role=paragraph id=par_id3152349 xml-lang=en-US l10n=U 
oldref=24If you select a table that is at a lower level in the hierarchy, 
the tables that occur above it in the hierarchy are not selected./paragraph
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Jacobo Aragunde Pérez
 sfx2/source/control/thumbnailview.cxx|2 -
 sfx2/source/control/thumbnailviewacc.cxx |   47 +--
 sfx2/source/control/thumbnailviewacc.hxx |6 +++
 3 files changed, 52 insertions(+), 3 deletions(-)

New commits:
commit 01a9f7b2fb6b3445ba9a93a4278ffa3821f7d04b
Author: Jacobo Aragunde Pérez jaragu...@igalia.com
Date:   Mon Jan 20 20:22:32 2014 +0100

fdo#72639: send proper ACTIVE_DESCENDANT_CHANGED events

At ThumbnailView::SelectItem the piece of code in charge of the
accessible events related to selection change was mistakenly using
the accessibility helper class ThumbnailViewAcc instead of
ThumbnailViewItemAcc which is the proper one for managing the events
related to ThumbnailViewItem objects.

The methods FireAccessibleEvent and static getImplementation had to
be added to ThumbnailViewItemAcc to provide the full functionality
required.

Change-Id: Ia5ffb24c920f3c48854021700819be683c979411
Reviewed-on: https://gerrit.libreoffice.org/7551
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 5c8522e..06f5800 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -1139,7 +1139,7 @@ void ThumbnailView::SelectItem( sal_uInt16 nItemId )
 if( ImplHasAccessibleListeners() )
 {
 // focus event (select)
-ThumbnailViewAcc* pItemAcc = ThumbnailViewAcc::getImplementation( 
pItem-GetAccessible( mbIsTransientChildrenDisabled ) );
+ThumbnailViewItemAcc* pItemAcc = 
ThumbnailViewItemAcc::getImplementation( pItem-GetAccessible( 
mbIsTransientChildrenDisabled ) );
 
 if( pItemAcc )
 {
diff --git a/sfx2/source/control/thumbnailviewacc.cxx 
b/sfx2/source/control/thumbnailviewacc.cxx
index 3f61244..cd4a6d2 100644
--- a/sfx2/source/control/thumbnailviewacc.cxx
+++ b/sfx2/source/control/thumbnailviewacc.cxx
@@ -97,7 +97,7 @@ void ThumbnailViewAcc::GetFocus (void)
 {
 mbIsFocused = true;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aNewState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -109,7 +109,7 @@ void ThumbnailViewAcc::LoseFocus (void)
 {
 mbIsFocused = false;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aOldState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -612,6 +612,32 @@ ThumbnailViewItemAcc::~ThumbnailViewItemAcc()
 {
 }
 
+void ThumbnailViewItemAcc::FireAccessibleEvent( short nEventId, const 
uno::Any rOldValue, const uno::Any rNewValue )
+{
+if( nEventId )
+{
+::std::vector uno::Reference accessibility::XAccessibleEventListener 
  aTmpListeners( mxEventListeners );
+accessibility::AccessibleEventObject aEvtObject;
+
+aEvtObject.EventId = nEventId;
+aEvtObject.Source = static_castuno::XWeak*(this);
+aEvtObject.NewValue = rNewValue;
+aEvtObject.OldValue = rOldValue;
+
+for (::std::vector uno::Reference 
accessibility::XAccessibleEventListener  ::const_iterator aIter( 
aTmpListeners.begin() ), aEnd( aTmpListeners.end() );
+aIter != aEnd ; ++aIter)
+{
+try
+{
+(*aIter)-notifyEvent( aEvtObject );
+}
+catch(const uno::Exception)
+{
+}
+}
+}
+}
+
 void ThumbnailViewItemAcc::ParentDestroyed()
 {
 const ::osl::MutexGuard aGuard( maMutex );
@@ -628,6 +654,23 @@ const uno::Sequence sal_Int8  
ThumbnailViewItemAcc::getUnoTunnelId()
 return theValueItemAccUnoTunnelId::get().getSeq();
 }
 
+ThumbnailViewItemAcc* ThumbnailViewItemAcc::getImplementation( const 
uno::Reference uno::XInterface  rxData )
+throw()
+{
+try
+{
+uno::Reference lang::XUnoTunnel  xUnoTunnel( rxData, uno::UNO_QUERY 
);
+return( xUnoTunnel.is() ?
+
reinterpret_castThumbnailViewItemAcc*(sal::static_int_castsal_IntPtr(
+xUnoTunnel-getSomething( 
ThumbnailViewItemAcc::getUnoTunnelId() ))) :
+NULL );
+}
+catch(const ::com::sun::star::uno::Exception)
+{
+return NULL;
+}
+}
+
 uno::Reference accessibility::XAccessibleContext  SAL_CALL 
ThumbnailViewItemAcc::getAccessibleContext()
 throw (uno::RuntimeException)
 {
diff --git a/sfx2/source/control/thumbnailviewacc.hxx 
b/sfx2/source/control/thumbnailviewacc.hxx
index 1e4f582..74f9f14 100644
--- a/sfx2/source/control/thumbnailviewacc.hxx
+++ b/sfx2/source/control/thumbnailviewacc.hxx
@@ -187,10 +187,16 @@ public:
 ThumbnailViewItemAcc( ThumbnailViewItem* pParent, bool 

[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 27/5a1d7a011e79a7f9c44c51e947d2cb50197051

2014-01-21 Thread Caolán McNamara
 27/5a1d7a011e79a7f9c44c51e947d2cb50197051 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 73013204d592a50f5940c952661ce5bacfd1a33f
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 15:09:21 2014 +

Notes added by 'git notes add'

diff --git a/27/5a1d7a011e79a7f9c44c51e947d2cb50197051 
b/27/5a1d7a011e79a7f9c44c51e947d2cb50197051
new file mode 100644
index 000..e7128c3
--- /dev/null
+++ b/27/5a1d7a011e79a7f9c44c51e947d2cb50197051
@@ -0,0 +1 @@
+merged as: 2cb1a05e867c17328ef24bd3489f3c18d80dfcf4
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Herbert Dürr
 dbaccess/uiconfig/ui/tablesfilterpage.ui   |5 -
 sc/source/filter/xml/XMLStylesExportHelper.cxx |2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 2cb1a05e867c17328ef24bd3489f3c18d80dfcf4
Author: Herbert Dürr h...@apache.org
Date:   Tue Jan 21 13:56:09 2014 +

get rid of an extraneous parentheses in if((A==B))

(cherry picked from commit 275a1d7a011e79a7f9c44c51e947d2cb50197051)

Conflicts:
basctl/source/dlged/dlged.cxx
dbaccess/source/ui/querydesign/JoinTableView.cxx
sc/source/filter/xml/XMLChangeTrackingImportHelper.cxx
svtools/source/edit/syntaxhighlight.cxx
svx/source/unodraw/unoshape.cxx
sw/source/ui/app/appopt.cxx
vcl/osx/salinst.cxx

Change-Id: If4ddf4ab23dbcfc4ba280948c73664b9794aecc7

diff --git a/sc/source/filter/xml/XMLStylesExportHelper.cxx 
b/sc/source/filter/xml/XMLStylesExportHelper.cxx
index c3d862d..6a50503 100644
--- a/sc/source/filter/xml/XMLStylesExportHelper.cxx
+++ b/sc/source/filter/xml/XMLStylesExportHelper.cxx
@@ -320,7 +320,7 @@ void ScMyValidationsContainer::WriteMessage(ScXMLExport 
rExport,
 bool bPrevCharWasSpace(true);
 while(i  sText.getLength())
 {
-if ((sText[i] == '\n'))
+if( sText[i] == '\n')
 {
 SvXMLElementExport aElemP(rExport, XML_NAMESPACE_TEXT, XML_P, 
true, false);
 
rExport.GetTextParagraphExport()-exportText(sTemp.makeStringAndClear(), 
bPrevCharWasSpace);
commit 656babddbc862d0d2498c79f616db839577a80ae
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 15:08:36 2014 +

wrap long text in tables filter dialog

Change-Id: I64f513969ff8eae48da4b55449439ea61b828645

diff --git a/dbaccess/uiconfig/ui/tablesfilterpage.ui 
b/dbaccess/uiconfig/ui/tablesfilterpage.ui
index 70ab9ee..cee902e 100644
--- a/dbaccess/uiconfig/ui/tablesfilterpage.ui
+++ b/dbaccess/uiconfig/ui/tablesfilterpage.ui
@@ -1,7 +1,8 @@
 ?xml version=1.0 encoding=UTF-8?
-!-- Generated with glade 3.16.0 on Tue Jan 21 09:48:38 2014 --
+!-- Generated with glade 3.16.0 on Tue Jan 21 15:00:47 2014 --
 interface
   !-- interface-requires gtk+ 3.0 --
+  !-- interface-requires LibreOffice 1.0 --
   object class=GtkFrame id=TablesFilterPage
 property name=visibleTrue/property
 property name=can_focusFalse/property
@@ -31,6 +32,8 @@
 property name=can_focusFalse/property
 property name=xalign0/property
 property name=label translatable=yesMark the tables that 
should be visible for the applications./property
+property name=wrapTrue/property
+property name=max_width_chars56/property
   /object
   packing
 property name=left_attach0/property
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Caolán McNamara
 sd/inc/stlsheet.hxx |2 +-
 svtools/source/control/valueacc.cxx |4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 2e10f314d0d3c3d159f8140fc9073fc86fbcfa17
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 15:15:57 2014 +

boradcast-broadcast

Change-Id: I96720843f815bf6d499eb424c3e4ff579c2bbe23

diff --git a/sd/inc/stlsheet.hxx b/sd/inc/stlsheet.hxx
index 561ce52..53ffab0 100644
--- a/sd/inc/stlsheet.hxx
+++ b/sd/inc/stlsheet.hxx
@@ -135,7 +135,7 @@ protected:
 OUString   msApiName;
 rtl::Reference SfxStyleSheetBasePool  mxPool;
 
-/** boradcast helper for events */
+/** broadcast helper for events */
 ::cppu::OBroadcastHelper mrBHelper;
 
 boost::scoped_ptr ModifyListenerForewarder  mpModifyListenerForewarder;
diff --git a/svtools/source/control/valueacc.cxx 
b/svtools/source/control/valueacc.cxx
index a669d25..1306c96 100644
--- a/svtools/source/control/valueacc.cxx
+++ b/svtools/source/control/valueacc.cxx
@@ -144,7 +144,7 @@ void ValueSetAcc::GetFocus (void)
 {
 mbIsFocused = true;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aNewState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -158,7 +158,7 @@ void ValueSetAcc::LoseFocus (void)
 {
 mbIsFocused = false;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aOldState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Stephan Bergmann
 solenv/gbuild/UnoApi.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit b45983f1bb86b316861b7cbcca500bb1ca894ca5
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jan 21 10:56:08 2014 +0100

idl files need to be packaged relative to INSTDIR, not INSTROOT

...so that on OS X they end up in LibreOffice*_SDK.app/idl/, not
LibreOffice.app/Contents/LibreOffice*_SDK.app/idl/.

Change-Id: Id001aed334a0e1d8f835ce797a17f1f67d1a3abb
(cherry picked from commit f4ccd054a94de1290e01bd21d5d45f1720f7cb74)
Reviewed-on: https://gerrit.libreoffice.org/7570
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com
Reviewed-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com

diff --git a/solenv/gbuild/UnoApi.mk b/solenv/gbuild/UnoApi.mk
index 5f40ee3..69ae8a9 100644
--- a/solenv/gbuild/UnoApi.mk
+++ b/solenv/gbuild/UnoApi.mk
@@ -20,6 +20,7 @@ $(call gb_UnoApiTarget_UnoApiTarget,$(1),$(1))
 $(call gb_UnoApiHeadersTarget_UnoApiHeadersTarget,$(1))
 ifneq ($(gb_UnoApi_ENABLE_INSTALL),)
 $(call gb_Package_Package_internal,$(1)_idl,$(SRCDIR))
+$(call gb_Package_set_outdir,$(1)_idl,$(INSTDIR))
 endif
 
 $(call gb_UnoApi_get_target,$(1)) :| $(dir $(call 
gb_UnoApi_get_target,$(1))).dir
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Zolnai Tamás
 sw/qa/extras/ooxmlimport/data/fdo73389.docx  |binary
 sw/qa/extras/ooxmlimport/ooxmlimport.cxx |7 
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   23 +--
 3 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit f18d3a8f842713de10ecda161b74ef25e21aa14f
Author: Zolnai Tamás tamas.zol...@collabora.com
Date:   Mon Jan 20 13:44:46 2014 +0100

fdo#73389 Writer does not show a docx-document with nested table correctly

The table manager can work with more table simultaneously
and so m_aCellWidths contains more table's properties.Only one
item of it belongs to the current table (getCurrentCellwidths).
Regression from 74c5ed19f430327988194cdcd6bdff09591a93fa

(cherry picked from commit d0c383256ef72d5212d8e2db77582d0ebe417209)

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

Change-Id: I93efac0c004af1b2524c955ffb20c3ecd74a2920
Reviewed-on: https://gerrit.libreoffice.org/7544
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk
Reviewed-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/qa/extras/ooxmlimport/data/fdo73389.docx 
b/sw/qa/extras/ooxmlimport/data/fdo73389.docx
new file mode 100644
index 000..02b55f7
Binary files /dev/null and b/sw/qa/extras/ooxmlimport/data/fdo73389.docx differ
diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
index 6d44d45..e9c5ad0 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport.cxx
@@ -1563,6 +1563,13 @@ DECLARE_OOXMLIMPORT_TEST(testRPrChangeClosed, 
rprchange_closed.docx)
 CPPUNIT_ASSERT_EQUAL(false, hasProperty(getRun(getParagraph(2), 1), 
RedlineType));
 }
 
+DECLARE_OOXMLIMPORT_TEST(testFdo73389,fdo73389.docx)
+{
+uno::Referencetext::XTextTablesSupplier xTablesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Referencecontainer::XIndexAccess 
xTables(xTablesSupplier-getTextTables(), uno::UNO_QUERY);
+// This was 9340, i.e. the width of the inner table was too large.
+CPPUNIT_ASSERT_EQUAL(sal_Int32(2842), 
getPropertysal_Int32(xTables-getByIndex(0), Width));
+}
 #endif
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx 
b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index c4b24a7..cc7d119 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -142,24 +142,17 @@ bool DomainMapperTableManager::sprm(Sprm  rSprm)
 */
 bool bFixed = true;
 sal_Int32 nRowFixedWidth = 0;
-if (!m_aCellWidths.empty())
+IntVectorPtr pCellWidths = getCurrentCellWidths();
+// Step 1. Check whether all cells have fixed 
widths in the given row of table.
+for (std::vectorsal_Int32::const_iterator 
aValIter = pCellWidths-begin(); aValIter != pCellWidths-end(); ++aValIter)
 {
-// Step 1. Check whether all cells have fixed 
widths in the given row of table.
-::std::vector IntVectorPtr ::iterator itr;
-for (itr = m_aCellWidths.begin(); itr != 
m_aCellWidths.end(); ++itr )
+if (*aValIter == -1)
 {
-IntVectorPtr itrVal = (*itr);
-for 
(std::vectorsal_Int32::const_iterator aValIter = itrVal-begin(); aValIter != 
itrVal-end(); ++aValIter)
-{
-if (*aValIter == -1)
-{
-bFixed = false;
-break;
-}
-// Sum the width of cells to find the 
total width of given row
-nRowFixedWidth += (*aValIter);
-}
+bFixed = false;
+break;
 }
+// Sum the width of cells to find the total 
width of given row
+nRowFixedWidth += (*aValIter);
 }
 
 // Check whether the total width of given row is 
compared with the maximum value of rows (m_nMaxFixedWidth).
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Andras Timar
 vcl/source/filter/wmf/winmtf.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ade704ea4ac7317f2ad300c8321c64ba54d0e692
Author: Andras Timar andras.ti...@collabora.com
Date:   Sun Jan 19 19:13:57 2014 +0100

fdo#73764 partially revert 3998d1abb291

Change-Id: Ie9b6d6a318fad256f3f0999fe9c0810a4323f10d
Reviewed-on: https://gerrit.libreoffice.org/7537
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/vcl/source/filter/wmf/winmtf.cxx b/vcl/source/filter/wmf/winmtf.cxx
index 160416b..38e79a8 100644
--- a/vcl/source/filter/wmf/winmtf.cxx
+++ b/vcl/source/filter/wmf/winmtf.cxx
@@ -317,7 +317,7 @@ Point WinMtfOutput::ImplScale( const Point rPt) // Hack to 
set varying defaults
 {
 if (mbIsMapDevSet  mbIsMapWinSet)
 {
-return 
Point((rPt.X())*mnWinExtX/mnDevWidth-mrclFrame.Left(),(rPt.Y())*mnWinExtY/mnDevHeight-mrclFrame.Top());
+return rPt; //fdo#73764
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 65675] LibreOffice 4.2 most annoying bugs

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65675

Bug 65675 depends on bug 72639, which changed state.

Bug 72639 Summary: ACCESSIBILITY: Recent documents in the new start center 
don't send the proper a11y events
https://bugs.freedesktop.org/show_bug.cgi?id=72639

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

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


Re: [Libreoffice-commits] core.git: fwk: Use constructor feature for ModuleUIConfigurationManager

2014-01-21 Thread Stephan Bergmann

On 01/17/2014 12:28 PM, Matúš Kukan wrote:

commit 72b5343bd16deec540d8cd148cd7aebd74e92c16
Author: Matúš Kukan matus.ku...@collabora.com
Date:   Thu Jan 9 20:18:49 2014 +0100

fwk: Use constructor feature for ModuleUIConfigurationManager.

And avoid css::uno::XInitialization protocol.


Removing XInitialization from a service implementation is always a 
delicate move regarding compatibility.  Even though the new-style 
service css.ui.ModuleUIConfigurationManager does not mention 
XInitialization, the old-style variant (prior to 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=9c41a60c0d802c618ac414822615a5c9e9e92f9f 
fdo#46808, Convert ui::ModuleUIConfigurationManager service to new 
style) did, and there is always a chance that (external) code obtains 
the service implementation via a non-argument queryInstance call and 
then explicitly queries for XInitialization.


So although this change is most probably justified, it is nevertheless 
incompatible.  Matúš, can you please manually list your relevant past 
changes at 
https://wiki.documentfoundation.org/ReleaseNotes/4.3#API_changes and 
add [API CHANGE] to the commit messages of relevant future changes (to 
allow for semi-automatic tracking)?


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


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

2014-01-21 Thread Jacobo Aragunde Pérez
 sfx2/source/control/thumbnailview.cxx|2 -
 sfx2/source/control/thumbnailviewacc.cxx |   47 +--
 sfx2/source/control/thumbnailviewacc.hxx |6 +++
 3 files changed, 52 insertions(+), 3 deletions(-)

New commits:
commit cdc6d32d9062b1c143b094910bc913cb8f183afa
Author: Jacobo Aragunde Pérez jaragu...@igalia.com
Date:   Mon Jan 20 20:22:32 2014 +0100

fdo#72639: send proper ACTIVE_DESCENDANT_CHANGED events

At ThumbnailView::SelectItem the piece of code in charge of the
accessible events related to selection change was mistakenly using
the accessibility helper class ThumbnailViewAcc instead of
ThumbnailViewItemAcc which is the proper one for managing the events
related to ThumbnailViewItem objects.

The methods FireAccessibleEvent and static getImplementation had to
be added to ThumbnailViewItemAcc to provide the full functionality
required.

Change-Id: Ia5ffb24c920f3c48854021700819be683c979411
Reviewed-on: https://gerrit.libreoffice.org/7551
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit 01a9f7b2fb6b3445ba9a93a4278ffa3821f7d04b)

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 1fe98f1..35380a8 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -1140,7 +1140,7 @@ void ThumbnailView::SelectItem( sal_uInt16 nItemId )
 if( ImplHasAccessibleListeners() )
 {
 // focus event (select)
-ThumbnailViewAcc* pItemAcc = ThumbnailViewAcc::getImplementation( 
pItem-GetAccessible( mbIsTransientChildrenDisabled ) );
+ThumbnailViewItemAcc* pItemAcc = 
ThumbnailViewItemAcc::getImplementation( pItem-GetAccessible( 
mbIsTransientChildrenDisabled ) );
 
 if( pItemAcc )
 {
diff --git a/sfx2/source/control/thumbnailviewacc.cxx 
b/sfx2/source/control/thumbnailviewacc.cxx
index 3f61244..cd4a6d2 100644
--- a/sfx2/source/control/thumbnailviewacc.cxx
+++ b/sfx2/source/control/thumbnailviewacc.cxx
@@ -97,7 +97,7 @@ void ThumbnailViewAcc::GetFocus (void)
 {
 mbIsFocused = true;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aNewState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -109,7 +109,7 @@ void ThumbnailViewAcc::LoseFocus (void)
 {
 mbIsFocused = false;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aOldState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -612,6 +612,32 @@ ThumbnailViewItemAcc::~ThumbnailViewItemAcc()
 {
 }
 
+void ThumbnailViewItemAcc::FireAccessibleEvent( short nEventId, const 
uno::Any rOldValue, const uno::Any rNewValue )
+{
+if( nEventId )
+{
+::std::vector uno::Reference accessibility::XAccessibleEventListener 
  aTmpListeners( mxEventListeners );
+accessibility::AccessibleEventObject aEvtObject;
+
+aEvtObject.EventId = nEventId;
+aEvtObject.Source = static_castuno::XWeak*(this);
+aEvtObject.NewValue = rNewValue;
+aEvtObject.OldValue = rOldValue;
+
+for (::std::vector uno::Reference 
accessibility::XAccessibleEventListener  ::const_iterator aIter( 
aTmpListeners.begin() ), aEnd( aTmpListeners.end() );
+aIter != aEnd ; ++aIter)
+{
+try
+{
+(*aIter)-notifyEvent( aEvtObject );
+}
+catch(const uno::Exception)
+{
+}
+}
+}
+}
+
 void ThumbnailViewItemAcc::ParentDestroyed()
 {
 const ::osl::MutexGuard aGuard( maMutex );
@@ -628,6 +654,23 @@ const uno::Sequence sal_Int8  
ThumbnailViewItemAcc::getUnoTunnelId()
 return theValueItemAccUnoTunnelId::get().getSeq();
 }
 
+ThumbnailViewItemAcc* ThumbnailViewItemAcc::getImplementation( const 
uno::Reference uno::XInterface  rxData )
+throw()
+{
+try
+{
+uno::Reference lang::XUnoTunnel  xUnoTunnel( rxData, uno::UNO_QUERY 
);
+return( xUnoTunnel.is() ?
+
reinterpret_castThumbnailViewItemAcc*(sal::static_int_castsal_IntPtr(
+xUnoTunnel-getSomething( 
ThumbnailViewItemAcc::getUnoTunnelId() ))) :
+NULL );
+}
+catch(const ::com::sun::star::uno::Exception)
+{
+return NULL;
+}
+}
+
 uno::Reference accessibility::XAccessibleContext  SAL_CALL 
ThumbnailViewItemAcc::getAccessibleContext()
 throw (uno::RuntimeException)
 {
diff --git a/sfx2/source/control/thumbnailviewacc.hxx 
b/sfx2/source/control/thumbnailviewacc.hxx
index 1e4f582..74f9f14 100644
--- a/sfx2/source/control/thumbnailviewacc.hxx
+++ b/sfx2/source/control/thumbnailviewacc.hxx
@@ -187,10 +187,16 @@ 

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

2014-01-21 Thread Chris Sherlock
 include/vcl/svapp.hxx|   21 +++--
 sd/source/ui/slideshow/slideshowimpl.cxx |8 
 vcl/source/app/svapp.cxx |   10 --
 3 files changed, 27 insertions(+), 12 deletions(-)

New commits:
commit 1d621d363cf966648fc5ed551dbc95de3e70e89c
Author: Chris Sherlock chris.sherloc...@gmail.com
Date:   Sun Jan 19 13:13:07 2014 +1100

Change EnableNoYieldMode(bool) to better names

It makes no sense to disable something via an Enable function.
I have split the function into Enable and Disable functions, and
update the code accordingly.

Conflicts:
include/vcl/svapp.hxx

Change-Id: Ic05d69796f43e802a4a2a16aaca44433b6eaa018
Reviewed-on: https://gerrit.libreoffice.org/7525
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/include/vcl/svapp.hxx b/include/vcl/svapp.hxx
index 0432a17..8e7b053 100644
--- a/include/vcl/svapp.hxx
+++ b/include/vcl/svapp.hxx
@@ -469,17 +469,26 @@ public:
  No yield mode prevents @Yield from waiting for events.
 
  @remarks This was originally implemented in OOo bug 98792 to improve
-Impress slideshows. For some reason, to \em disable no yield mode, you
-call on EnableNoYieldMode
+Impress slideshows.
 
- @param i_bNoYield  If set to false, then no yield mode is turned 
off.
-If set to true, then no yield mode is turned on.
+ @see DisableNoYieldMode, Execute, Quit, Reschedule, Yield, EndYield, 
GetSolarMutex,
+  GetMainThreadIdentifier, ReleaseSolarMutex, AcquireSolarMutex,
+  AddPostYieldListener, RemovePostYieldListener
+*/
+static void EnableNoYieldMode();
 
- @see Execute, Quit, Reschedule, Yield, EndYield, GetSolarMutex,
+/** @Brief Disables no yield mode
+
+ No yield mode prevents @Yield from waiting for events.
+
+ @remarks This was originally implemented in OOo bug 98792 to improve
+Impress slideshows.
+
+ @see EnableNoYieldMode, Execute, Quit, Reschedule, Yield, EndYield, 
GetSolarMutex,
   GetMainThreadIdentifier, ReleaseSolarMutex, AcquireSolarMutex,
   AddPostYieldListener, RemovePostYieldListener
 */
-static void EnableNoYieldMode( bool i_bNoYield );
+static void DisableNoYieldMode();
 
 /** Add a listener for yield events
 
diff --git a/sd/source/ui/slideshow/slideshowimpl.cxx 
b/sd/source/ui/slideshow/slideshowimpl.cxx
index 87e6435..2bc4d72 100644
--- a/sd/source/ui/slideshow/slideshowimpl.cxx
+++ b/sd/source/ui/slideshow/slideshowimpl.cxx
@@ -760,7 +760,7 @@ void SAL_CALL SlideshowImpl::disposing()
 
 setActiveXToolbarsVisible( sal_True );
 
-Application::EnableNoYieldMode(false);
+Application::DisableNoYieldMode();
 Application::RemovePostYieldListener(LINK(this, SlideshowImpl, 
PostYieldListener));
 
 mbDisposed = true;
@@ -1855,7 +1855,7 @@ IMPL_LINK_NOARG(SlideshowImpl, PostYieldListener)
 // prevent me from deletion when recursing (App::Reschedule does)
 const rtl::ReferenceSlideshowImpl this_(this);
 
-Application::EnableNoYieldMode(false);
+Application::DisableNoYieldMode();
 Application::RemovePostYieldListener(LINK(this, SlideshowImpl, 
PostYieldListener));
 Application::Reschedule(true); // fix for fdo#32861 - process
// *all* outstanding events after
@@ -1896,7 +1896,7 @@ sal_Int32 SlideshowImpl::updateSlideShow (void)
 if (::basegfx::fTools::equalZero(fUpdate))
 {
 // Use post yield listener for short update intervalls.
-Application::EnableNoYieldMode(true);
+Application::EnableNoYieldMode();
 Application::AddPostYieldListener(LINK(this, SlideshowImpl, 
PostYieldListener));
 }
 else
@@ -1917,7 +1917,7 @@ sal_Int32 SlideshowImpl::updateSlideShow (void)
 // integer may lead to zero value.)
 OSL_ASSERT(static_castsal_uLong(fUpdate * 1000.0)  0);
 
-Application::EnableNoYieldMode(false);
+Application::DisableNoYieldMode();
 Application::RemovePostYieldListener(LINK(this, SlideshowImpl, 
PostYieldListener));
 
 // Use a timer for the asynchronous callback.
diff --git a/vcl/source/app/svapp.cxx b/vcl/source/app/svapp.cxx
index 6d345f3..a754856 100644
--- a/vcl/source/app/svapp.cxx
+++ b/vcl/source/app/svapp.cxx
@@ -986,10 +986,16 @@ void Application::RemoveIdleHdl( const Link rLink )
 pSVData-maAppData.mpIdleMgr-RemoveIdleHdl( rLink );
 }
 
-void Application::EnableNoYieldMode( bool i_bNoYield )
+void Application::EnableNoYieldMode()
 {
 ImplSVData* pSVData = ImplGetSVData();
-pSVData-maAppData.mbNoYield = i_bNoYield;
+pSVData-maAppData.mbNoYield = true;
+}
+
+void 

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

2014-01-21 Thread Caolán McNamara
 sd/uiconfig/simpress/ui/optimpressgeneralpage.ui |  110 +++
 1 file changed, 55 insertions(+), 55 deletions(-)

New commits:
commit a4a6fa38cebf6182769495c74af5f59588056e0b
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Jan 16 15:16:40 2014 +

place some elements side by side to regain a line

Change-Id: I9b838b6c00bc6e456adcf5d63c9936c08bf0d1aa
(cherry picked from commit 5eaa0135c3ffc46ce601ea4cbaaa17eabe44e6ee)
(cherry picked from commit 0d5397234b7589617970bb20d8fb3ad07a4587a1)
Reviewed-on: https://gerrit.libreoffice.org/7475
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Reviewed-by: Eike Rathke er...@redhat.com
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sd/uiconfig/simpress/ui/optimpressgeneralpage.ui 
b/sd/uiconfig/simpress/ui/optimpressgeneralpage.ui
index 750c9f9..595f8f0 100644
--- a/sd/uiconfig/simpress/ui/optimpressgeneralpage.ui
+++ b/sd/uiconfig/simpress/ui/optimpressgeneralpage.ui
@@ -183,60 +183,8 @@
   /object
   packing
 property name=left_attach0/property
-property name=top_attach1/property
-property name=width2/property
-property name=height1/property
-  /packing
-/child
-child
-  object class=GtkCheckButton id=objalwymov
-property name=label translatable=yesObjects always 
moveable/property
-property name=visibleTrue/property
-property name=can_focusTrue/property
-property name=receives_defaultFalse/property
-property name=use_underlineTrue/property
-property name=xalign0/property
-property name=draw_indicatorTrue/property
-  /object
-  packing
-property name=left_attach0/property
-property name=top_attach2/property
-property name=width2/property
-property name=height1/property
-  /packing
-/child
-child
-  object class=GtkCheckButton id=backgroundback
-property name=label translatable=yesUse background 
cache/property
-property name=visibleTrue/property
-property name=can_focusTrue/property
-property name=receives_defaultFalse/property
-property name=use_underlineTrue/property
-property name=xalign0/property
-property name=draw_indicatorTrue/property
-  /object
-  packing
-property name=left_attach0/property
 property name=top_attach0/property
-property name=width2/property
-property name=height1/property
-  /packing
-/child
-child
-  object class=GtkCheckButton id=distrotcb
-property name=label translatable=yesDo not distort 
objects in curve/property
-property name=visibleTrue/property
-property name=can_focusFalse/property
-property name=receives_defaultFalse/property
-property name=double_bufferedFalse/property
-property name=use_underlineTrue/property
-property name=xalign0/property
-property name=draw_indicatorTrue/property
-  /object
-  packing
-property name=left_attach0/property
-property name=top_attach3/property
-property name=width2/property
+property name=width1/property
 property name=height1/property
   /packing
 /child
@@ -277,7 +225,7 @@
   /object
   packing
 property name=left_attach0/property
-property name=top_attach4/property
+property name=top_attach2/property
 property name=width1/property
 property name=height1/property
   /packing
@@ -324,7 +272,59 @@
   /object
   packing
 property name=left_attach1/property
-property name=top_attach4/property
+property name=top_attach2/property
+property name=width1/property
+property name=height1/property
+  /packing
+/child
+child
+  object class=GtkCheckButton id=objalwymov
+property name=label 

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

2014-01-21 Thread Jacobo Aragunde Pérez
 sfx2/source/control/thumbnailview.cxx|2 -
 sfx2/source/control/thumbnailviewacc.cxx |   47 +--
 sfx2/source/control/thumbnailviewacc.hxx |6 +++
 3 files changed, 52 insertions(+), 3 deletions(-)

New commits:
commit eed45184ff542a713d28928b50c5a8ce8efd3a92
Author: Jacobo Aragunde Pérez jaragu...@igalia.com
Date:   Mon Jan 20 20:22:32 2014 +0100

fdo#72639: send proper ACTIVE_DESCENDANT_CHANGED events

At ThumbnailView::SelectItem the piece of code in charge of the
accessible events related to selection change was mistakenly using
the accessibility helper class ThumbnailViewAcc instead of
ThumbnailViewItemAcc which is the proper one for managing the events
related to ThumbnailViewItem objects.

The methods FireAccessibleEvent and static getImplementation had to
be added to ThumbnailViewItemAcc to provide the full functionality
required.

Change-Id: Ia5ffb24c920f3c48854021700819be683c979411
Reviewed-on: https://gerrit.libreoffice.org/7551
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit 01a9f7b2fb6b3445ba9a93a4278ffa3821f7d04b)
(cherry picked from commit cdc6d32d9062b1c143b094910bc913cb8f183afa)
Reviewed-on: https://gerrit.libreoffice.org/7573
Reviewed-by: Michael Stahl mst...@redhat.com
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 1fe98f1..35380a8 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -1140,7 +1140,7 @@ void ThumbnailView::SelectItem( sal_uInt16 nItemId )
 if( ImplHasAccessibleListeners() )
 {
 // focus event (select)
-ThumbnailViewAcc* pItemAcc = ThumbnailViewAcc::getImplementation( 
pItem-GetAccessible( mbIsTransientChildrenDisabled ) );
+ThumbnailViewItemAcc* pItemAcc = 
ThumbnailViewItemAcc::getImplementation( pItem-GetAccessible( 
mbIsTransientChildrenDisabled ) );
 
 if( pItemAcc )
 {
diff --git a/sfx2/source/control/thumbnailviewacc.cxx 
b/sfx2/source/control/thumbnailviewacc.cxx
index 3f61244..cd4a6d2 100644
--- a/sfx2/source/control/thumbnailviewacc.cxx
+++ b/sfx2/source/control/thumbnailviewacc.cxx
@@ -97,7 +97,7 @@ void ThumbnailViewAcc::GetFocus (void)
 {
 mbIsFocused = true;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aNewState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -109,7 +109,7 @@ void ThumbnailViewAcc::LoseFocus (void)
 {
 mbIsFocused = false;
 
-// Boradcast the state change.
+// Broadcast the state change.
 ::com::sun::star::uno::Any aOldState, aNewState;
 aOldState = 
::com::sun::star::accessibility::AccessibleStateType::FOCUSED;
 FireAccessibleEvent(
@@ -612,6 +612,32 @@ ThumbnailViewItemAcc::~ThumbnailViewItemAcc()
 {
 }
 
+void ThumbnailViewItemAcc::FireAccessibleEvent( short nEventId, const 
uno::Any rOldValue, const uno::Any rNewValue )
+{
+if( nEventId )
+{
+::std::vector uno::Reference accessibility::XAccessibleEventListener 
  aTmpListeners( mxEventListeners );
+accessibility::AccessibleEventObject aEvtObject;
+
+aEvtObject.EventId = nEventId;
+aEvtObject.Source = static_castuno::XWeak*(this);
+aEvtObject.NewValue = rNewValue;
+aEvtObject.OldValue = rOldValue;
+
+for (::std::vector uno::Reference 
accessibility::XAccessibleEventListener  ::const_iterator aIter( 
aTmpListeners.begin() ), aEnd( aTmpListeners.end() );
+aIter != aEnd ; ++aIter)
+{
+try
+{
+(*aIter)-notifyEvent( aEvtObject );
+}
+catch(const uno::Exception)
+{
+}
+}
+}
+}
+
 void ThumbnailViewItemAcc::ParentDestroyed()
 {
 const ::osl::MutexGuard aGuard( maMutex );
@@ -628,6 +654,23 @@ const uno::Sequence sal_Int8  
ThumbnailViewItemAcc::getUnoTunnelId()
 return theValueItemAccUnoTunnelId::get().getSeq();
 }
 
+ThumbnailViewItemAcc* ThumbnailViewItemAcc::getImplementation( const 
uno::Reference uno::XInterface  rxData )
+throw()
+{
+try
+{
+uno::Reference lang::XUnoTunnel  xUnoTunnel( rxData, uno::UNO_QUERY 
);
+return( xUnoTunnel.is() ?
+
reinterpret_castThumbnailViewItemAcc*(sal::static_int_castsal_IntPtr(
+xUnoTunnel-getSomething( 
ThumbnailViewItemAcc::getUnoTunnelId() ))) :
+NULL );
+}
+catch(const ::com::sun::star::uno::Exception)
+{
+return NULL;
+}
+}
+
 uno::Reference accessibility::XAccessibleContext  SAL_CALL 
ThumbnailViewItemAcc::getAccessibleContext()
 

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

2014-01-21 Thread Zolnai Tamás
 sfx2/source/control/recentdocsview.cxx |8 
 sfx2/source/dialog/backingcomp.cxx |   14 ++
 sfx2/source/dialog/backingwindow.cxx   |4 +++-
 3 files changed, 25 insertions(+), 1 deletion(-)

New commits:
commit 840e7f004354ebe1239b9271dd758839c477ca7c
Author: Zolnai Tamás tamas.zol...@collabora.com
Date:   Fri Jan 17 20:10:06 2014 +0100

fdo#65826 and fdo#73605: set a minimum size to start center

This minimum size calculated like this:
width: sidebar optimal width + width of text appearing in
the thumbnail view when no recent document is available.
height: menu width + optimal width of GtkBox containing
buttons.

Change-Id: Ic1fd6e33a1ec93e61318af02c09ee88909a76cf4
Reviewed-on: https://gerrit.libreoffice.org/7505
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sfx2/source/control/recentdocsview.cxx 
b/sfx2/source/control/recentdocsview.cxx
index e1b1c0c..69b6059 100644
--- a/sfx2/source/control/recentdocsview.cxx
+++ b/sfx2/source/control/recentdocsview.cxx
@@ -54,6 +54,14 @@ RecentDocsView::RecentDocsView( Window* pParent )
 SetStyle(GetStyle() | WB_VSCROLL);
 setItemMaxTextLength( mnItemMaxTextLength );
 setItemDimensions( mnItemMaxSize, mnItemMaxSize, mnTextHeight, 
mnItemPadding );
+
+// Set prefered width so text lines will not be cut off
+Font aOldFont(GetFont());
+Font aNewFont(aOldFont);
+aNewFont.SetHeight(20);
+SetFont(aNewFont);
+
set_width_request(std::max(GetTextWidth(maWelcomeLine1),GetTextWidth(maWelcomeLine2)));
+SetFont(aOldFont);
 }
 
 extern C SAL_DLLPUBLIC_EXPORT Window* SAL_CALL makeRecentDocsView(Window 
*pParent, VclBuilder::stringmap )
diff --git a/sfx2/source/dialog/backingcomp.cxx 
b/sfx2/source/dialog/backingcomp.cxx
index fa5cfa8..cadf4c3 100644
--- a/sfx2/source/dialog/backingcomp.cxx
+++ b/sfx2/source/dialog/backingcomp.cxx
@@ -448,6 +448,20 @@ void SAL_CALL BackingComp::attachFrame( /*IN*/ const 
css::uno::Reference css::f
 if( pBack )
 pBack-setOwningFrame( m_xFrame );
 
+// Set a minimum size for Start Center
+if( pParent  pBack )
+{
+long nMenuHeight = 0;
+Window* pMenu = pParent-GetWindow(WINDOW_NEXT);
+if( pMenu )
+nMenuHeight = pMenu-GetSizePixel().Height();
+
+pParent-SetMinOutputSizePixel(
+Size(
+pBack-get_width_request(),
+pBack-get_height_request() + nMenuHeight));
+}
+
 /* } SAFE */
 }
 
diff --git a/sfx2/source/dialog/backingwindow.cxx 
b/sfx2/source/dialog/backingwindow.cxx
index 988a4a3..bf128f0 100644
--- a/sfx2/source/dialog/backingwindow.cxx
+++ b/sfx2/source/dialog/backingwindow.cxx
@@ -265,6 +265,9 @@ void BackingWindow::initControls()
 mpThinBox2-SetBackground(aButtonsText);
 
 Resize();
+
+set_width_request(mpAllRecentThumbnails-get_width_request() + 
mpAllButtonsBox-GetOptimalSize().Width());
+set_height_request(mpAllButtonsBox-GetOptimalSize().Height());
 }
 
 void BackingWindow::setupButton( PushButton* pButton )
@@ -555,5 +558,4 @@ void BackingWindow::clearRecentFileList()
 {
 mpAllRecentThumbnails-Clear();
 }
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab:*/
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Julien Nabet
 basic/source/runtime/methods.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit ff48f28f71609b0ab4e6b8bc79818f765e48fddd
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Jan 19 21:56:40 2014 +0100

Resolves: fdo#73771 Non-standard behaviour for Function Mid

Change-Id: I90be516235228574dc219447a0eef3a34f83bee2
Reviewed-on: https://gerrit.libreoffice.org/7539
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 36d7c3c..efa3ea9 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -1302,7 +1302,8 @@ RTLFUNC(Mid)
 {
 nLen = aArgStr.getLength() - nStartPos;
 }
-aResultStr = aArgStr.copy( nStartPos, nLen );
+if (nLen  0)
+aResultStr = aArgStr.copy( nStartPos, nLen );
 }
 rPar.Get(0)-PutString( aResultStr );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Julien Nabet
 basic/source/runtime/methods.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 7713dfc9b50b88cf2548d49e65b099a6a81e9d65
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Jan 19 21:56:40 2014 +0100

Resolves: fdo#73771 Non-standard behaviour for Function Mid

Change-Id: I90be516235228574dc219447a0eef3a34f83bee2
Reviewed-on: https://gerrit.libreoffice.org/7539
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit ff48f28f71609b0ab4e6b8bc79818f765e48fddd)

diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 5351606..ac4b9c4 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -1302,7 +1302,8 @@ RTLFUNC(Mid)
 {
 nLen = aArgStr.getLength() - nStartPos;
 }
-aResultStr = aArgStr.copy( nStartPos, nLen );
+if (nLen  0)
+aResultStr = aArgStr.copy( nStartPos, nLen );
 }
 rPar.Get(0)-PutString( aResultStr );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Michael Stahl
 sw/source/core/text/txtfly.cxx|6 +++---
 sw/source/core/txtnode/fntcap.cxx |4 ++--
 sw/source/core/txtnode/swfont.cxx |8 
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 98176bc2078d4ce18d4eb41b54b4ad1c72a2774d
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 11:16:20 2014 +0100

fdo#68959: sw: fix painting of tab fill characters in small caps text

The member SwDrawTextInfo::pPos was changed from a pointer to some
externally provided Point to a real object; unfortunately there were
some functions which hold backups of the current pPos as a
reference/pointer and then restore it, and the restore is a no-op now.
So use real Points as backups.

(regression from f22006dc6ac34a35a060e15466cf6b2d2058617d)

Change-Id: I8470143ec494cb194a88ddcbd12d96867974278f
(cherry picked from commit c445d40da8daa2f6ec0ae9930a06f50ab80cc85d)
Reviewed-on: https://gerrit.libreoffice.org/7567
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 1e66412..9f7d970 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -568,7 +568,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 }
 
 Point aPos( rInf.GetPos().X(), rInf.GetPos().Y() + rInf.GetAscent() );
-const Point rOld = rInf.GetPos();
+const Point aOldPos(rInf.GetPos());
 rInf.SetPos( aPos );
 
 if( !bOpaque )
@@ -577,7 +577,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawStretchText( rInf );
 else
 rInf.GetFont()-_DrawText( rInf );
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_False;
 }
 else if( !aRegion.empty() )
@@ -595,7 +595,7 @@ sal_Bool SwTxtFly::DrawTextOpaque( SwDrawTextInfo rInf )
 rInf.GetFont()-_DrawText( rInf );
 }
 }
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 return sal_True;
 }
 
diff --git a/sw/source/core/txtnode/fntcap.cxx 
b/sw/source/core/txtnode/fntcap.cxx
index 0c31695..c435eda 100644
--- a/sw/source/core/txtnode/fntcap.cxx
+++ b/sw/source/core/txtnode/fntcap.cxx
@@ -523,9 +523,9 @@ void SwSubFont::DrawStretchCapital( SwDrawTextInfo rInf )
 if( rInf.GetLen() == STRING_LEN )
 rInf.SetLen( rInf.GetText().getLength() );
 
-const Point rOldPos = rInf.GetPos();
+const Point aOldPos = rInf.GetPos();
 const sal_uInt16 nCapWidth = (sal_uInt16)( GetCapitalSize( rInf ).Width() 
);
-rInf.SetPos( rOldPos );
+rInf.SetPos(aOldPos);
 
 rInf.SetDrawSpace( GetUnderline() != UNDERLINE_NONE ||
GetOverline()  != UNDERLINE_NONE ||
diff --git a/sw/source/core/txtnode/swfont.cxx 
b/sw/source/core/txtnode/swfont.cxx
index 2269f24..21c6601 100644
--- a/sw/source/core/txtnode/swfont.cxx
+++ b/sw/source/core/txtnode/swfont.cxx
@@ -1195,12 +1195,12 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 
 SwDigitModeModifier aDigitModeModifier( rInf.GetOut(), 
rInf.GetFont()-GetLanguage() );
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
 
@@ -1300,7 +1300,7 @@ void SwSubFont::_DrawText( SwDrawTextInfo rInf, const 
sal_Bool bGrey )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
@@ -1325,13 +1325,13 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 
 rInf.ApplyAutoColor();
 
+const Point aOldPos(rInf.GetPos());
 Point aPos( rInf.GetPos() );
 
 if( GetEscapement() )
 CalcEsc( rInf, aPos );
 
 rInf.SetKern( CheckKerning() + rInf.GetSperren() / 
SPACING_PRECISION_FACTOR );
-const Point rOld = rInf.GetPos();
 rInf.SetPos( aPos );
 
 if( IsCapital() )
@@ -1382,7 +1382,7 @@ void SwSubFont::_DrawStretchText( SwDrawTextInfo rInf )
 rInf.SetLen( nOldLen );
 }
 
-rInf.SetPos( rOld );
+rInf.SetPos(aOldPos);
 }
 
 xub_StrLen SwSubFont::_GetCrsrOfst( SwDrawTextInfo rInf )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: MABs and priority (was: minutes of ESC call ...)

2014-01-21 Thread Bjoern Michaelsen
Hi,

On Fri, Jan 17, 2014 at 12:01:21PM +0100, Bjoern Michaelsen wrote:
 == short term ==
 
 To keep the highest=MAB equivalence clean, some regular (weekly?) checking for
 bugs that are priority:highest and not a MAB would be needed. Such bugs could
 be seen as proposed MABs and either be:
 - promoted to a MAB with the usual procedure (rationale etc.)
 - or respectfully rejected and bumped to priority:high
 
 Someone volunteering for this task?
 
 If so, it would be interesting:
 - how many such bugs there are each week
 - what is their quality (as in: how many are good MABs? how many are well 
 triaged?)

So to ease this work, I created two stored queries:

 
https://bugs.freedesktop.org/buglist.cgi?cmdtype=runnamednamedcmd=mab-not-highest
 
https://bugs.freedesktop.org/buglist.cgi?cmdtype=runnamednamedcmd=highest-not-mab

QA volunteers concerned with bug priorization are encouraged to check these and
either move them in a consistent highest and MAB or a consistent not-highest
and not-MAB state. Note that usual workflow is still in place, so if you mark
something as a MAB you should provide a good rationale.

Best,

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


[Libreoffice-commits] core.git: Branch 'aoo/trunk' - 2 commits - sd/source slideshow/source solenv/bin sw/source writerfilter/source

2014-01-21 Thread Andre Fischer
 sd/source/ui/slidesorter/view/SlsButtonBar.cxx |2 +-
 slideshow/source/engine/animatedsprite.cxx |2 +-
 slideshow/source/engine/shapes/viewshape.cxx   |2 +-
 slideshow/source/engine/shapesubset.cxx|2 +-
 slideshow/source/engine/slide/slideanimations.cxx  |2 +-
 slideshow/source/inc/shapeattributelayer.hxx   |2 +-
 slideshow/source/inc/shapeattributelayerholder.hxx |2 +-
 solenv/bin/patch_tool.pl   |9 -
 sw/source/core/inc/bookmrk.hxx |2 +-
 writerfilter/source/dmapper/StyleSheetTable.cxx|2 +-
 10 files changed, 17 insertions(+), 10 deletions(-)

New commits:
commit 821f66510a7851e6e6835e0f6c86dc443482caf0
Author: Andre Fischer a...@apache.org
Date:   Tue Jan 21 14:21:45 2014 +

123531: Changed name of MSP to contain product name and source and target 
version.

diff --git a/solenv/bin/patch_tool.pl b/solenv/bin/patch_tool.pl
index f7cc82d..7f2d4de 100644
--- a/solenv/bin/patch_tool.pl
+++ b/solenv/bin/patch_tool.pl
@@ -1549,7 +1549,14 @@ sub CreatePcp ($$%)
 
 # Create filenames.
 my $pcp_filename = File::Spec-catfile($msp_path, openoffice.pcp);
-my $msp_filename = File::Spec-catfile($msp_path, openoffice.msp);
+# Create basename to include product name and source and target version.
+# Hard code platform because that is the only platform supported at the 
moment.
+my $msp_basename = sprintf(%s_%s-%s_Win_x86_patch_%s.msp,
+$context-{'product-name'},
+$source_msi-{'version'},
+$target_msi-{'version'},
+$context-{'language'});
+my $msp_filename = File::Spec-catfile($msp_path, $msp_basename);
 
 # Setup msp path and filename.
 unlink($pcp_filename) if -f $pcp_filename;
commit 1ceda390389a3e7bf4b7bb72b533a355a0fa060d
Author: Herbert Dürr h...@apache.org
Date:   Tue Jan 21 14:09:37 2014 +

#i123817# boost::shared_ptr doesn't have an implicit conversion to bool

Constructs that expect it fail at least in XCode4's clang in C++11 mode.
An implicit conversion from pointer to bool is already suspicious enough
and a shared_ptr-pointer-bool conversion is even worse. Cleaning up
the code fixes the build breaker seen in boost/libc++/clang environments.

diff --git a/sd/source/ui/slidesorter/view/SlsButtonBar.cxx 
b/sd/source/ui/slidesorter/view/SlsButtonBar.cxx
index ff088bf..003e3b6 100644
--- a/sd/source/ui/slidesorter/view/SlsButtonBar.cxx
+++ b/sd/source/ui/slidesorter/view/SlsButtonBar.cxx
@@ -469,7 +469,7 @@ void ButtonBar::Paint (
 
 bool ButtonBar::IsMouseOverButton (void) const
 {
-return mpButtonUnderMouse;
+return (mpButtonUnderMouse.get() != NULL);
 }
 
 
diff --git a/slideshow/source/engine/animatedsprite.cxx 
b/slideshow/source/engine/animatedsprite.cxx
index 22b2cc1..0b6e612 100644
--- a/slideshow/source/engine/animatedsprite.cxx
+++ b/slideshow/source/engine/animatedsprite.cxx
@@ -157,7 +157,7 @@ namespace slideshow
 }
 }
 
-return mpSprite;
+return (mpSprite.get() != NULL);
 }
 
 void AnimatedSprite::setPixelOffset( const ::basegfx::B2DSize 
rPixelOffset )
diff --git a/slideshow/source/engine/shapes/viewshape.cxx 
b/slideshow/source/engine/shapes/viewshape.cxx
index 46f0f2e..0265acb 100644
--- a/slideshow/source/engine/shapes/viewshape.cxx
+++ b/slideshow/source/engine/shapes/viewshape.cxx
@@ -184,7 +184,7 @@ namespace slideshow
 }
 }
 
-return io_rCacheEntry.mpRenderer;
+return (io_rCacheEntry.mpRenderer.get() != NULL);
 }
 
 bool ViewShape::draw( const ::cppcanvas::CanvasSharedPtr   
rDestinationCanvas,
diff --git a/slideshow/source/engine/shapesubset.cxx 
b/slideshow/source/engine/shapesubset.cxx
index 77e76a4..f6783a8 100644
--- a/slideshow/source/engine/shapesubset.cxx
+++ b/slideshow/source/engine/shapesubset.cxx
@@ -110,7 +110,7 @@ namespace slideshow
 maTreeNode );
 }
 
-return mpSubsetShape;
+return (mpSubsetShape.get() != NULL);
 }
 
 void ShapeSubset::disableSubsetShape()
diff --git a/slideshow/source/engine/slide/slideanimations.cxx 
b/slideshow/source/engine/slide/slideanimations.cxx
index 55db7e5..16e16a0 100644
--- a/slideshow/source/engine/slide/slideanimations.cxx
+++ b/slideshow/source/engine/slide/slideanimations.cxx
@@ -80,7 +80,7 @@ namespace slideshow
 
 SHOW_NODE_TREE( mpRootNode );
 
-return mpRootNode;
+return (mpRootNode.get() != NULL);
 }
 
 bool SlideAnimations::isAnimated() const
diff --git a/slideshow/source/inc/shapeattributelayer.hxx 
b/slideshow/source/inc/shapeattributelayer.hxx
index ede31f9..a41ab35 100644
--- a/slideshow/source/inc/shapeattributelayer.hxx
+++ b/slideshow/source/inc/shapeattributelayer.hxx
@@ -471,7 +471,7 @@ namespace slideshow
 // 

Re: A stack trace for BZ bug 73464 - 4.2.0.2 RC2 QA Blocker -- crash when IA2 bridge is active and ENTER key is pressed

2014-01-21 Thread V Stuart Foote
With the pending 4.2.0 RC3 spin, I can't say I'm comfortable with current
state of resolving   fdo#73464
https://bugs.freedesktop.org/show_bug.cgi?id=73464  .

Hey, I'd even be happy if someone else can report reproducing the crash on
the 4.2.0.2 RC2 build,  ;-)

I've posted up a set of the DrMemory logs for the IAccessible2 related
crasher, or can continue with WinDbg based logging.  Have not tested on a
32-bit Windows, but could do that if requested.

Can say with reasonable certainty that the 4.2.0.2 RC2 release build
configuration was trouble, but all TB 42 builds since have been free of the
issue.  Conceivable that it was related to  fdo#72397
https://bugs.freedesktop.org/show_bug.cgi?id=72397   but would hate to run
another release build and have this problem still there.

Stuart









--
View this message in context: 
http://nabble.documentfoundation.org/A-stack-trace-for-BZ-bug-73464-4-2-0-2-RC2-QA-Blocker-crash-when-IA2-bridge-is-active-and-ENTER-key-d-tp4091751p4093314.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2014-01-21 Thread Arnaud Versini
 tools/source/generic/poly.cxx  |   51 -
 tools/source/generic/poly2.cxx |   42 -
 tools/source/stream/stream.cxx |6 
 3 files changed, 99 deletions(-)

New commits:
commit de2d5d9abb7f625269cd4ee54c983c6dfde33767
Author: Arnaud Versini arnaud.vers...@gmail.com
Date:   Sun Jan 19 15:49:55 2014 +0100

TOOLS : Remove usage of DBG_CTOR and DBG_DTOR.

Valgrind is capable of detecting such bugs. No need for extra macros.

Change-Id: Id75ca6991af0bf75e979c7bc2da452f323f389ea
Reviewed-on: https://gerrit.libreoffice.org/7531
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/tools/source/generic/poly.cxx b/tools/source/generic/poly.cxx
index 2e55994..43b19e6 100644
--- a/tools/source/generic/poly.cxx
+++ b/tools/source/generic/poly.cxx
@@ -41,8 +41,6 @@
 #include limits.h
 #include cmath
 
-DBG_NAME( Polygon )
-
 #define EDGE_LEFT   1
 #define EDGE_TOP2
 #define EDGE_RIGHT  4
@@ -300,13 +298,11 @@ inline double ImplGetParameter( const Point rCenter, 
const Point rPt, double f
 
 Polygon::Polygon()
 {
-DBG_CTOR( Polygon, NULL );
 mpImplPolygon = (ImplPolygon*)(aStaticImplPolygon);
 }
 
 Polygon::Polygon( sal_uInt16 nSize )
 {
-DBG_CTOR( Polygon, NULL );
 
 if ( nSize )
 mpImplPolygon = new ImplPolygon( nSize );
@@ -316,7 +312,6 @@ Polygon::Polygon( sal_uInt16 nSize )
 
 Polygon::Polygon( sal_uInt16 nPoints, const Point* pPtAry, const sal_uInt8* 
pFlagAry )
 {
-DBG_CTOR( Polygon, NULL );
 
 if( nPoints )
 mpImplPolygon = new ImplPolygon( nPoints, pPtAry, pFlagAry );
@@ -326,8 +321,6 @@ Polygon::Polygon( sal_uInt16 nPoints, const Point* pPtAry, 
const sal_uInt8* pFla
 
 Polygon::Polygon( const Polygon rPoly )
 {
-DBG_CTOR( Polygon, NULL );
-DBG_CHKOBJ( rPoly, Polygon, NULL );
 DBG_ASSERT( rPoly.mpImplPolygon-mnRefCount  0xFFFE, Polygon: 
RefCount overflow );
 
 mpImplPolygon = rPoly.mpImplPolygon;
@@ -337,7 +330,6 @@ Polygon::Polygon( const Polygon rPoly )
 
 Polygon::Polygon( const Rectangle rRect )
 {
-DBG_CTOR( Polygon, NULL );
 
 if ( rRect.IsEmpty() )
 mpImplPolygon = (ImplPolygon*)(aStaticImplPolygon);
@@ -354,8 +346,6 @@ Polygon::Polygon( const Rectangle rRect )
 
 Polygon::Polygon( const Rectangle rRect, sal_uIntPtr nHorzRound, sal_uIntPtr 
nVertRound )
 {
-DBG_CTOR( Polygon, NULL );
-
 if ( rRect.IsEmpty() )
 mpImplPolygon = (ImplPolygon*)(aStaticImplPolygon);
 else
@@ -409,8 +399,6 @@ Polygon::Polygon( const Rectangle rRect, sal_uIntPtr 
nHorzRound, sal_uIntPtr nV
 
 Polygon::Polygon( const Point rCenter, long nRadX, long nRadY, sal_uInt16 
nPoints )
 {
-DBG_CTOR( Polygon, NULL );
-
 if( nRadX  nRadY )
 {
 // Compute default (depends on size)
@@ -461,8 +449,6 @@ Polygon::Polygon( const Point rCenter, long nRadX, long 
nRadY, sal_uInt16 nPoin
 Polygon::Polygon( const Rectangle rBound, const Point rStart, const Point 
rEnd,
   PolyStyle eStyle, bool bFullCircle )
 {
-DBG_CTOR( Polygon, NULL );
-
 const long  nWidth = rBound.GetWidth();
 const long  nHeight = rBound.GetHeight();
 
@@ -539,8 +525,6 @@ Polygon::Polygon( const Point rBezPt1, const Point 
rCtrlPt1,
   const Point rBezPt2, const Point rCtrlPt2,
   sal_uInt16 nPoints )
 {
-DBG_CTOR( Polygon, NULL );
-
 nPoints = ( 0 == nPoints ) ? 25 : ( ( nPoints  2 ) ? 2 : nPoints );
 
 const doublefInc = 1.0 / ( nPoints - 1 );
@@ -572,7 +556,6 @@ Polygon::Polygon( const Point rBezPt1, const Point 
rCtrlPt1,
 
 Polygon::~Polygon()
 {
-DBG_DTOR( Polygon, NULL );
 
 // Remove if refcount == 0, otherwise decrement refcount
 if ( mpImplPolygon-mnRefCount )
@@ -586,19 +569,16 @@ Polygon::~Polygon()
 
 const Point* Polygon::GetConstPointAry() const
 {
-DBG_CHKTHIS( Polygon, NULL );
 return (Point*)mpImplPolygon-mpPointAry;
 }
 
 const sal_uInt8* Polygon::GetConstFlagAry() const
 {
-DBG_CHKTHIS( Polygon, NULL );
 return mpImplPolygon-mpFlagAry;
 }
 
 void Polygon::SetPoint( const Point rPt, sal_uInt16 nPos )
 {
-DBG_CHKTHIS( Polygon, NULL );
 DBG_ASSERT( nPos  mpImplPolygon-mnPoints,
 Polygon::SetPoint(): nPos = nPoints );
 
@@ -608,7 +588,6 @@ void Polygon::SetPoint( const Point rPt, sal_uInt16 nPos )
 
 void Polygon::SetFlags( sal_uInt16 nPos, PolyFlags eFlags )
 {
-DBG_CHKTHIS( Polygon, NULL );
 DBG_ASSERT( nPos  mpImplPolygon-mnPoints,
 Polygon::SetFlags(): nPos = nPoints );
 
@@ -624,7 +603,6 @@ void Polygon::SetFlags( sal_uInt16 nPos, PolyFlags eFlags )
 
 const Point Polygon::GetPoint( sal_uInt16 nPos ) const
 {
-DBG_CHKTHIS( Polygon, NULL );
 DBG_ASSERT( nPos  mpImplPolygon-mnPoints,
 Polygon::GetPoint(): nPos = nPoints );
 
@@ -633,7 +611,6 @@ const Point Polygon::GetPoint( sal_uInt16 nPos ) 

[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 82/1f66510a7851e6e6835e0f6c86dc443482caf0

2014-01-21 Thread Caolán McNamara
 82/1f66510a7851e6e6835e0f6c86dc443482caf0 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit c3a6ff4164767bc857878d58da87e211d2ecfd68
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 16:11:51 2014 +

Notes added by 'git notes add'

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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-1' - basic/source

2014-01-21 Thread Julien Nabet
 basic/source/runtime/methods.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit b703da40bb052809ff94a714fb8da7aadf2d7d6b
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Jan 19 21:56:40 2014 +0100

Resolves: fdo#73771 Non-standard behaviour for Function Mid

Change-Id: I90be516235228574dc219447a0eef3a34f83bee2
Reviewed-on: https://gerrit.libreoffice.org/7539
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit ff48f28f71609b0ab4e6b8bc79818f765e48fddd)
Signed-off-by: Michael Stahl mst...@redhat.com

diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 200b7b7..31825c8 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -1308,7 +1308,8 @@ RTLFUNC(Mid)
 {
 nLen = aArgStr.getLength() - nStartPos;
 }
-aResultStr = aArgStr.copy( nStartPos, nLen );
+if (nLen  0)
+aResultStr = aArgStr.copy( nStartPos, nLen );
 }
 rPar.Get(0)-PutString( aResultStr );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 1c/eda390389a3e7bf4b7bb72b533a355a0fa060d

2014-01-21 Thread Caolán McNamara
 1c/eda390389a3e7bf4b7bb72b533a355a0fa060d |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 6f27dc99bbb11d4a9348c5781ece52e231e3835d
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 21 16:13:37 2014 +

Notes added by 'git notes add'

diff --git a/1c/eda390389a3e7bf4b7bb72b533a355a0fa060d 
b/1c/eda390389a3e7bf4b7bb72b533a355a0fa060d
new file mode 100644
index 000..aac2217
--- /dev/null
+++ b/1c/eda390389a3e7bf4b7bb72b533a355a0fa060d
@@ -0,0 +1 @@
+prefer: d4bab97023e3569571a92551040574b20aceca7c
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Alexander Wilms
 icon-themes/tango/cmd/sc_colorscaleformatdialog.png
 |binary
 icon-themes/tango/cmd/sc_colorscaleformatdialog.svg
 |  125 ++
 icon-themes/tango/cmd/sc_conddateformatdialog.png  
 |binary
 icon-themes/tango/cmd/sc_conditionalformatdialog.png   
 |binary
 
icon-themes/tango/cmd/sc_conditionalformatmenu-conditionalformatdialog-conddateformatdialog.svg
 |  122 ++
 icon-themes/tango/cmd/sc_conditionalformatmenu.png 
 |binary
 icon-themes/tango/cmd/sc_databarformatdialog.png   
 |binary
 icon-themes/tango/cmd/sc_databarformatdialog.svg   
 |  158 
 icon-themes/tango/cmd/sc_iconsetformatdialog.png   
 |binary
 icon-themes/tango/cmd/sc_iconsetformatdialog.svg   
 |  185 ++
 officecfg/registry/data/org/openoffice/Office/UI/CalcCommands.xcu  
 |   18 
 11 files changed, 608 insertions(+)

New commits:
commit f44116909d356fdc9c9ec8658be6cc465cb46686
Author: Alexander Wilms f.alexander.wi...@gmail.com
Date:   Wed Jan 1 18:59:42 2014 +0100

Added icons for conditional formatting

Change-Id: If2dd67ad08d93e020e0e68043f9270d387bd41d3
Reviewed-on: https://gerrit.libreoffice.org/7246
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/icon-themes/tango/cmd/sc_colorscaleformatdialog.png 
b/icon-themes/tango/cmd/sc_colorscaleformatdialog.png
new file mode 100644
index 000..07554ba
Binary files /dev/null and 
b/icon-themes/tango/cmd/sc_colorscaleformatdialog.png differ
diff --git a/icon-themes/tango/cmd/sc_colorscaleformatdialog.svg 
b/icon-themes/tango/cmd/sc_colorscaleformatdialog.svg
new file mode 100644
index 000..0a9e53c
--- /dev/null
+++ b/icon-themes/tango/cmd/sc_colorscaleformatdialog.svg
@@ -0,0 +1,125 @@
+?xml version=1.0 encoding=UTF-8 standalone=no?
+!-- Created with Inkscape (http://www.inkscape.org/) --
+
+svg
+   xmlns:dc=http://purl.org/dc/elements/1.1/;
+   xmlns:cc=http://creativecommons.org/ns#;
+   xmlns:rdf=http://www.w3.org/1999/02/22-rdf-syntax-ns#;
+   xmlns:svg=http://www.w3.org/2000/svg;
+   xmlns=http://www.w3.org/2000/svg;
+   xmlns:sodipodi=http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd;
+   xmlns:inkscape=http://www.inkscape.org/namespaces/inkscape;
+   id=svg2
+   version=1.1
+   inkscape:version=0.48.4 r9939
+   width=16
+   height=16
+   sodipodi:docname=sc_conditionalformatdialog.svg
+   inkscape:export-filename=/home/alexander/sc_conditionalformatdialog.png
+   inkscape:export-xdpi=90
+   inkscape:export-ydpi=90
+  metadata
+ id=metadata8
+rdf:RDF
+  cc:Work
+ rdf:about=
+dc:formatimage/svg+xml/dc:format
+dc:type
+   rdf:resource=http://purl.org/dc/dcmitype/StillImage; /
+dc:title/dc:title
+  /cc:Work
+/rdf:RDF
+  /metadata
+  defs
+ id=defs6 /
+  sodipodi:namedview
+ pagecolor=#ff
+ bordercolor=#66
+ borderopacity=1
+ objecttolerance=10
+ gridtolerance=10
+ guidetolerance=10
+ inkscape:pageopacity=0
+ inkscape:pageshadow=2
+ inkscape:window-width=1920
+ inkscape:window-height=918
+ id=namedview4
+ showgrid=true
+ inkscape:zoom=45.254834
+ inkscape:cx=8.1355932
+ inkscape:cy=8
+ inkscape:window-x=0
+ inkscape:window-y=27
+ inkscape:window-maximized=1
+ inkscape:current-layer=svg2
+inkscape:grid
+   type=xygrid
+   id=grid3010
+   empspacing=5
+   visible=true
+   enabled=true
+   snapvisiblegridlinesonly=true /
+  /sodipodi:namedview
+  g
+ id=g3825
+ transform=translate(-349.98874,-512.34897)
+rect
+   ry=1
+   rx=1
+   y=512.36218
+   x=350
+   height=15
+   width=16
+   id=rect3805
+   style=fill:#99;fill-opacity:1;stroke:none /
+rect
+   y=513.36218
+   x=351
+   height=13
+   width=14
+   id=rect3807
+   style=fill:#ff;fill-opacity:1;stroke:none /
+rect
+   y=514.36218
+   x=352
+   height=3
+   width=12
+   id=rect3809
+   style=fill:#fab132;fill-opacity:1;stroke:none /
+rect
+   style=fill:#ee3e3e;fill-opacity:1;stroke:none
+   id=rect3811
+   width=12
+   height=3.0625
+   x=352
+   y=518.36218 /
+rect
+   y=522.36218
+   x=352
+   height=3
+   width=12
+   id=rect3813
+   style=fill:#95e24a;fill-opacity:1;stroke:none /
+rect
+   y=523.36218
+   x=353
+   height=1
+   width=10
+   id=rect3819
+   style=opacity:0.3;fill:#ff;fill-opacity:1;stroke:none /
+rect
+   y=519.36218
+   

Re: New Developer

2014-01-21 Thread Miklos Vajna
On Tue, Jan 21, 2014 at 06:53:06PM +0530, Manohar Kh khmanoh...@gmail.com 
wrote:
 Yes , but still it is the same problem .

Ah yes, I see; gerrit redirects http to https. Could you try
http://anongit.freedesktop.org/git/libreoffice/core? (But fixing your
local certificate store is a good idea anyway.)


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


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

2014-01-21 Thread Andras Timar
 vcl/source/filter/wmf/winmtf.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 22818d9fb7551d8f2a7d9389d28dd96b794a1e89
Author: Andras Timar andras.ti...@collabora.com
Date:   Sun Jan 19 19:13:57 2014 +0100

fdo#73764 partially revert 3998d1abb291

Change-Id: Ie9b6d6a318fad256f3f0999fe9c0810a4323f10d
Reviewed-on: https://gerrit.libreoffice.org/7537
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit ade704ea4ac7317f2ad300c8321c64ba54d0e692)
Reviewed-on: https://gerrit.libreoffice.org/7575
Tested-by: Michael Stahl mst...@redhat.com
Reviewed-by: Michael Stahl mst...@redhat.com
Reviewed-by: Jan Holesovsky ke...@collabora.com

diff --git a/vcl/source/filter/wmf/winmtf.cxx b/vcl/source/filter/wmf/winmtf.cxx
index 160416b..38e79a8 100644
--- a/vcl/source/filter/wmf/winmtf.cxx
+++ b/vcl/source/filter/wmf/winmtf.cxx
@@ -317,7 +317,7 @@ Point WinMtfOutput::ImplScale( const Point rPt) // Hack to 
set varying defaults
 {
 if (mbIsMapDevSet  mbIsMapWinSet)
 {
-return 
Point((rPt.X())*mnWinExtX/mnDevWidth-mrclFrame.Left(),(rPt.Y())*mnWinExtY/mnDevHeight-mrclFrame.Top());
+return rPt; //fdo#73764
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Julien Nabet
 basic/source/runtime/methods.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit de19819871a5ce1faabb9dced0e40df97d18e168
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Jan 19 21:56:40 2014 +0100

Resolves: fdo#73771 Non-standard behaviour for Function Mid

Change-Id: I90be516235228574dc219447a0eef3a34f83bee2
Reviewed-on: https://gerrit.libreoffice.org/7539
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit ff48f28f71609b0ab4e6b8bc79818f765e48fddd)
(cherry picked from commit 7713dfc9b50b88cf2548d49e65b099a6a81e9d65)
Reviewed-on: https://gerrit.libreoffice.org/7577
Reviewed-by: Michael Stahl mst...@redhat.com
Reviewed-by: Jan Holesovsky ke...@collabora.com
Reviewed-by: Eike Rathke er...@redhat.com
Tested-by: Eike Rathke er...@redhat.com

diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 5351606..ac4b9c4 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -1302,7 +1302,8 @@ RTLFUNC(Mid)
 {
 nLen = aArgStr.getLength() - nStartPos;
 }
-aResultStr = aArgStr.copy( nStartPos, nLen );
+if (nLen  0)
+aResultStr = aArgStr.copy( nStartPos, nLen );
 }
 rPar.Get(0)-PutString( aResultStr );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [libreoffice-l10n] space between values and units in spinners, fdo#71688

2014-01-21 Thread Adolfo Jayme Barrientos
On Tue, Jan 21, 2014 at 5:59 AM, Caolán McNamara caol...@redhat.com wrote:
 [...]
 c) If you agree with icu's % formatting for your language then update
 the translations for X% to be consistent with that formatting. Otherwise
 we can fix icu or make a blacklist of languages which should be excluded
 from the default icu percent formatting.

Caolán, how can we submit a fix for ICU? For instance, Spanish
formatting should be exactly the same as German, i.e. “#,##0 %”.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-commits] core.git: Get bundled openssl to be used on OS X too, at least on OS X 10.7

2014-01-21 Thread Stephan Bergmann

On 01/20/2014 04:05 PM, Douglas Mencken wrote:

commit 8ae8d5165c29cb899e6a8a57d1021a844ac30b30
Author: Douglas Mencken dougmenc...@gmail.com
Date:   Mon Dec 30 10:55:47 2013 -0500

 Get bundled openssl to be used on OS X too, at least on OS X 10.7


So looking at the context of the below configure.ac change, 
MAC_OS_X_VERSION_MIN_REQUIRED = 1070 (thus incl. the official Mac OS 
X 64-bit builds, for a baseline of OS X 10.8) already used LO's own 
bundled openssl, while other builds for Mac OS X (thus incl. the 
official Mac OS X 32-bit builds, for a baseline of OS X 10.6) used to 
use the OS X openssl.  What is the rationale for switching the latter to 
use LO's own bundled openssl instead?



 Also note that OpenSSL is incompatible with -fvisibility=hidden on OS X


Why?  What exactly did not work?  (For my local --enable-64-bit build I 
see that e.g. libpostgresql-sdbc-impllo.dylib now exports lots of _SSL_* 
symbols that it used to not export, which seemingly used to work fine.)



 Change-Id: I1fc3cf4d515ef210f996cb04ba9c1b3216109137
 Reviewed-on: https://gerrit.libreoffice.org/7233
 Reviewed-by: Caolán McNamara caol...@redhat.com
 Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/configure.ac b/configure.ac
index 9e5f7de..eb024d7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -9375,19 +9375,11 @@ if test $enable_openssl = yes; then
  #
  # 
http://stackoverflow.com/questions/7406946/why-is-apple-deprecating-openssl-in-macos-10-7-lion
  # 
http://stackoverflow.com/questions/7475914/libcrypto-deprecated-on-mac-os-x-10-7-lion
-#
-# They apparently got fed up with the unstable ABI of
-# OpenSSL, or something, and recommend you either move to
-# their Common Crypto Architecture, or, if you have to stay
-# with OpenSSL, just build it yourself statically and bundle
-# with your application.

  with_system_openssl=no
  libo_CHECK_SYSTEM_MODULE([openssl],[OPENSSL],[openssl])
-elif test $_os = Darwin -o $_os = NetBSD -o $_os = OpenBSD -o \
-$_os = DragonFly  test $with_system_openssl != no; then
-# Mac OS builds should get out without extra stuff is the Mac porters'
-# wish. And pkg-config is although Xcode ships a .pc for openssl
+elif test $_os = NetBSD -o $_os = OpenBSD -o $_os = DragonFly \
+ test $with_system_openssl != no; then
  with_system_openssl=yes
  SYSTEM_OPENSSL=YES
  OPENSSL_CFLAGS=
diff --git a/external/openssl/ExternalProject_openssl.mk 
b/external/openssl/ExternalProject_openssl.mk
index 919e354..d04c186 100644
--- a/external/openssl/ExternalProject_openssl.mk
+++ b/external/openssl/ExternalProject_openssl.mk
@@ -83,7 +83,7 @@ $(call gb_ExternalProject_get_state_target,openssl,build):
$(if $(SYSBASE),-I$(SYSBASE)/usr/include 
-L$(SYSBASE)/usr/lib)) \
$(if $(filter 
MACOSX,$(OS)),--prefix=/@.__OOO)
 \
 $(MAKE) build_libs \
-   CC=$(CC) -fPIC $(if $(filter-out WNT,$(OS)),\
+   CC=$(CC) -fPIC $(if $(filter-out WNT MACOSX,$(OS)),\
$(if $(filter TRUE,$(HAVE_GCC_VISIBILITY_FEATURE)),\
-fvisibility=hidden)) \
)


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


Re: New Developer

2014-01-21 Thread Manohar Kh
This is working . Thanks for the help .


On Tue, Jan 21, 2014 at 10:19 PM, Miklos Vajna vmik...@collabora.co.ukwrote:

 On Tue, Jan 21, 2014 at 06:53:06PM +0530, Manohar Kh khmanoh...@gmail.com
 wrote:
  Yes , but still it is the same problem .

 Ah yes, I see; gerrit redirects http to https. Could you try
 http://anongit.freedesktop.org/git/libreoffice/core? (But fixing your
 local certificate store is a good idea anyway.)

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


[Libreoffice-commits] core.git: Branch 'aoo/trunk' - 4 commits - basegfx/inc extras/source framework/inc sw/source ucb/source uui/source

2014-01-21 Thread Tsutomu Uchino
 basegfx/inc/basegfx/tools/b2dclipstate.hxx |2 
 extras/source/autotext/lang/en-US/acor_en-US.dat   |binary
 framework/inc/uielement/genericstatusbarcontroller.hxx |2 
 sw/source/core/access/accpara.hxx  |2 
 ucb/source/ucp/webdav/SerfSession.cxx  |   44 +++--
 ucb/source/ucp/webdav/SerfTypes.hxx|2 
 ucb/source/ucp/webdav/SerfUri.hxx  |2 
 uui/source/iahndl-ssl.cxx  |5 +
 8 files changed, 48 insertions(+), 11 deletions(-)

New commits:
commit 17e2f9c3d2eb0d3b7f559055327d37171db01c2f
Author: Tsutomu Uchino ha...@apache.org
Date:   Tue Jan 21 16:36:08 2014 +

#i124067# remove naive entry from auto correct list for en-US

diff --git a/extras/source/autotext/lang/en-US/acor_en-US.dat 
b/extras/source/autotext/lang/en-US/acor_en-US.dat
index d4e9a3d..681f362 100644
Binary files a/extras/source/autotext/lang/en-US/acor_en-US.dat and 
b/extras/source/autotext/lang/en-US/acor_en-US.dat differ
commit 226085ad2004319c5142b392bb4b48ab79c3b747
Author: Oliver-Rainer Wittmann o...@apache.org
Date:   Tue Jan 21 16:32:58 2014 +

123744: method SerfSession::verifySerfCertificateChain(..) - consider 
certificate's Subject Alternative Name field when searching for matching 
certificate host name

diff --git a/ucb/source/ucp/webdav/SerfSession.cxx 
b/ucb/source/ucp/webdav/SerfSession.cxx
index dc08678..a5354eb 100644
--- a/ucb/source/ucp/webdav/SerfSession.cxx
+++ b/ucb/source/ucp/webdav/SerfSession.cxx
@@ -30,7 +30,7 @@
 #include ucbhelper/simplecertificatevalidationrequest.hxx
 
 #include AprEnv.hxx
-#include apr_strings.h
+#include apr/apr_strings.h
 
 #include DAVAuthListener.hxx
 #include SerfTypes.hxx
@@ -47,6 +47,10 @@
 #include com/sun/star/security/CertificateContainerStatus.hpp
 #include com/sun/star/security/CertificateContainer.hpp
 #include com/sun/star/security/XCertificateContainer.hpp
+#include com/sun/star/security/CertAltNameEntry.hpp
+#include com/sun/star/security/XSanExtension.hpp
+#define OID_SUBJECT_ALTERNATIVE_NAME 2.5.29.17
+
 #include com/sun/star/ucb/Lock.hpp
 #include com/sun/star/xml/crypto/XSEInitializer.hpp
 
@@ -479,7 +483,40 @@ apr_status_t SerfSession::verifySerfCertificateChain (
 
 // When the certificate matches the host name then we can use the
 // result of the verification.
-if (isDomainMatch(sServerCertificateSubject))
+bool bHostnameMatchesCertHostnames = false;
+{
+uno::Sequence uno::Reference security::XCertificateExtension   
extensions = xServerCertificate-getExtensions();
+uno::Sequence security::CertAltNameEntry  altNames;
+for (sal_Int32 i = 0 ; i  extensions.getLength(); ++i)
+{
+uno::Reference security::XCertificateExtension element = 
extensions[i];
+
+const rtl::OString aId ( (const sal_Char 
*)element-getExtensionId().getArray(), element-getExtensionId().getLength());
+if ( aId.equals( OID_SUBJECT_ALTERNATIVE_NAME ) )
+{
+uno::Reference security::XSanExtension  sanExtension ( 
element, uno::UNO_QUERY );
+altNames =  sanExtension-getAlternativeNames();
+break;
+}
+}
+
+uno::Sequence ::rtl::OUString  certHostNames(altNames.getLength() + 
1);
+certHostNames[0] = sServerCertificateSubject;
+for( int n = 0; n  altNames.getLength(); ++n )
+{
+if (altNames[n].Type ==  security::ExtAltNameType_DNS_NAME)
+{
+altNames[n].Value = certHostNames[n+1];
+}
+}
+
+for ( int i = 0; i  certHostNames.getLength()  
!bHostnameMatchesCertHostnames; ++i )
+{
+bHostnameMatchesCertHostnames = isDomainMatch( certHostNames[i] );
+}
+
+}
+if ( bHostnameMatchesCertHostnames )
 {
 
 if (nVerificationResult == 0)
@@ -526,8 +563,7 @@ apr_status_t SerfSession::verifySerfCertificateChain (
 
 if ( xSelection.is() )
 {
-uno::Reference task::XInteractionApprove  xApprove(
-xSelection.get(), uno::UNO_QUERY );
+uno::Reference task::XInteractionApprove  xApprove( 
xSelection.get(), uno::UNO_QUERY );
 if ( xApprove.is() )
 {
 xCertificateContainer-addCertificate( getHostName(), 
sServerCertificateSubject,  sal_True );
diff --git a/ucb/source/ucp/webdav/SerfTypes.hxx 
b/ucb/source/ucp/webdav/SerfTypes.hxx
index d525d6b..23d8bee 100644
--- a/ucb/source/ucp/webdav/SerfTypes.hxx
+++ b/ucb/source/ucp/webdav/SerfTypes.hxx
@@ -24,7 +24,7 @@
 #ifndef INCLUDED_SERFTYPES_HXX
 #define INCLUDED_SERFTYPES_HXX
 
-#include serf.h
+#include serf/serf.h
 
 typedef serf_connection_t SerfConnection;
 
diff --git a/ucb/source/ucp/webdav/SerfUri.hxx 
b/ucb/source/ucp/webdav/SerfUri.hxx
index d6844a1..8bd45de 100644
--- 

Error doing make kdevelop-ide-integration in Windows + Cygwin.

2014-01-21 Thread Anderson Roberto
Hello all!

I'm developing a integration with MSVS, like the one with KDevelop and
XCode.
But, when I try to run a make kdevelop-ide-integration to use the project
files as example and to point some infos, I got this error:
Makefile:29: *** target pattern contains no '%'.  Stop.

Anyone knows what this mean and what I need to do to solve this error?

Thanks for help since now!

Anderson Roberto de Oliveira
Skype: andersonn_roberto
LinkedIn: http://br.linkedin.com/in/andersonroberto/
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 65675] LibreOffice 4.2 most annoying bugs

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65675

Bug 65675 depends on bug 73618, which changed state.

Bug 73618 Summary: Presenter console: Don't see the active slide after launch
https://bugs.freedesktop.org/show_bug.cgi?id=73618

   What|Removed |Added

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

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


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

2014-01-21 Thread Michael Stahl
 ucb/source/ucp/webdav/webdavcontent.cxx |2 +-
 vcl/source/filter/wmf/winmtf.cxx|5 -
 2 files changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 3780738154b8c3b3f9d85c64cccf621d97574886
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 20:14:46 2014 +0100

ucb: fix inverted condition in webdav lcl_sendPartialGETRequest

(regression from a3f32769fc4bb23c64168b412dd10ec769a3854d)

Change-Id: Ic72b0e874ebf19e25e43af1f8750c63c04ed8a8e

diff --git a/ucb/source/ucp/webdav/webdavcontent.cxx 
b/ucb/source/ucp/webdav/webdavcontent.cxx
index aea2a5a..1fce24b 100644
--- a/ucb/source/ucp/webdav/webdavcontent.cxx
+++ b/ucb/source/ucp/webdav/webdavcontent.cxx
@@ -163,7 +163,7 @@ static void lcl_sendPartialGETRequest( bool bError,
 for ( std::vector DAVPropertyValue ::iterator it = 
aResponseProps.begin();
 it != aResponseProps.end(); ++it )
 {
-if ( it-Name != Content-Length )
+if (it-Name == Content-Length)
 {
 it-Value = aSize;
 break;
commit 54a370753cd1affd8c47cf171ef4a0f3bc76ba90
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 16:34:41 2014 +0100

vcl: stylistic cleanup

Change-Id: Iaab13bd37b85128c9cf83b114a155f198b628587

diff --git a/vcl/source/filter/wmf/winmtf.cxx b/vcl/source/filter/wmf/winmtf.cxx
index 20bbbfa..df47935 100644
--- a/vcl/source/filter/wmf/winmtf.cxx
+++ b/vcl/source/filter/wmf/winmtf.cxx
@@ -518,7 +518,10 @@ Polygon WinMtfOutput::ImplScale( Polygon rPolygon )
 PolyPolygon WinMtfOutput::ImplScale( PolyPolygon rPolyPolygon )
 {
 sal_uInt16 nPolys = rPolyPolygon.Count();
-for ( sal_uInt16 i = 0; i  nPolys; ImplScale( rPolyPolygon[ i++ ] ) ) ;
+for (sal_uInt16 i = 0; i  nPolys; ++i)
+{
+ImplScale(rPolyPolygon[i]);
+}
 return rPolyPolygon;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-01-21 Thread Michael Stahl
 ucb/source/ucp/webdav/webdavcontent.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit bd121d45f847ff353e3796510fac3181b88a87b1
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 20:14:46 2014 +0100

ucb: fix inverted condition in webdav lcl_sendPartialGETRequest

(regression from a3f32769fc4bb23c64168b412dd10ec769a3854d)

Change-Id: Ic72b0e874ebf19e25e43af1f8750c63c04ed8a8e
(cherry picked from commit 3780738154b8c3b3f9d85c64cccf621d97574886)
Reviewed-on: https://gerrit.libreoffice.org/7581
Reviewed-by: David Tardon dtar...@redhat.com
Tested-by: David Tardon dtar...@redhat.com

diff --git a/ucb/source/ucp/webdav/webdavcontent.cxx 
b/ucb/source/ucp/webdav/webdavcontent.cxx
index 9bd49f3..d98c39b 100644
--- a/ucb/source/ucp/webdav/webdavcontent.cxx
+++ b/ucb/source/ucp/webdav/webdavcontent.cxx
@@ -163,7 +163,7 @@ static void lcl_sendPartialGETRequest( bool bError,
 for ( std::vector DAVPropertyValue ::iterator it = 
aResponseProps.begin();
 it != aResponseProps.end(); ++it )
 {
-if ( it-Name != Content-Length )
+if (it-Name == Content-Length)
 {
 it-Value = aSize;
 break;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Error doing make kdevelop-ide-integration in Windows + Cygwin.

2014-01-21 Thread Michael Stahl
On 21/01/14 19:51, Anderson Roberto wrote:
 Hello all!
 
 I'm developing a integration with MSVS, like the one with KDevelop and
 XCode.
 But, when I try to run a make kdevelop-ide-integration to use the
 project files as example and to point some infos, I got this error:
 Makefile:29: *** target pattern contains no '%'.  Stop.
 
 Anyone knows what this mean and what I need to do to solve this error?

most likely it means make does not understand Win32 file names with
drive letters.  try /opt/lo/bin/make instead, if you don't have that read:

https://wiki.documentfoundation.org/Development/Windows_Build_Dependencies#Install_make_an_optional_ccache_binaries


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


Re: [libreoffice-l10n] space between values and units in spinners, fdo#71688

2014-01-21 Thread Martin Srebotnjak
Hi,

for Slovenian I already asked to include space in front of % and it was
done. I also changed all translated strings accordingly. It was done for
4.2 or even before, can't remember. Hopefully those settings for Slovenian
were inherited by 4.3.

Lp, m.


2014/1/21 Caolán McNamara caol...@redhat.com

 For 4.3 we are now, in the normal case, putting space between values and
 units in spinners. e.g. now 10 mm instead of 10mm.
 (http://en.wikipedia.org/wiki/ISO_31-0)

 For the degree symbol we are not putting space between value and unit,
 i.e. it remains 10°

 I think those are hopefully uncontroversial.

 The tricky case is the % symbol, because, while ISO-31-0 implies that
 10 % is the preferred scientific format, English style guides prefer
 10% and 10 % looks just plain wrong. It's a locale-dependent
 formatting rule (http://en.wikipedia.org/wiki/Percent_sign)

 So to resolve this we now have a function formatPercent which takes the
 number and the locale and runs it through icu's PercentInstance to
 format it according to that locale's formatting rule. Which gives e.g.
 10 % for de and 10% for en. (http://site.icu-project.org/)

 What this means is that:
 a) In the English strings in .src files the source text should not have
 a space between number and %. I think these are all fixed now.
 b) You should check that the icu percent formatting rule is considered
 correct for your language in the context of a office suite UI e.g. in
 the spinner of view-zoom-zoom-variable
 c) If you agree with icu's % formatting for your language then update
 the translations for X% to be consistent with that formatting. Otherwise
 we can fix icu or make a blacklist of languages which should be excluded
 from the default icu percent formatting.

 C.


 --
 To unsubscribe e-mail to: l10n+unsubscr...@global.libreoffice.org
 Problems?
 http://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
 Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
 List archive: http://listarchives.libreoffice.org/global/l10n/
 All messages sent to this list will be publicly archived and cannot be
 deleted

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


[Bug 66822] refactor HTML export of image bullet list

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66822

Tejas Nikumbh tejasniku...@gmail.com changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |tejasniku...@gmail.com
   |desktop.org |

--- Comment #2 from Tejas Nikumbh tejasniku...@gmail.com ---
Created attachment 92542
  -- https://bugs.freedesktop.org/attachment.cgi?id=92542action=edit
This is a patch for the current bug. #66822 submitted by Tejas Nikumbh

This is one of my first patches for an easy programming hack enlisted on the
LibreOffice Gsoc page. My name is Tejas Nikumbh and I'm a senior undergrad at
IIT Bombay.

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


[Bug 63962] Remove LibreOffice icon themes hardcoding

2014-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=63962

--- Comment #10 from Tobias Lippert d...@fastmail.fm ---
Hello Michael,

In the first version, I will derive the name from the file name of the archive
that contains the icons, i.e. images_crystal.zip -- crystal icon theme.
I think that this is a sensible approach and should suffice for now.

The implementation takes much longer than I anticipated because I have long
feedback cycles. The involved headers are included everywhere, and almost
everything is recompiled if I change something. But I have finished the
implementation and am currently working on tests, i.e. I expect to be finished
in 2-3 weeks.

Tobias

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


Re: Error doing make kdevelop-ide-integration in Windows + Cygwin.

2014-01-21 Thread Anderson Roberto
Thanks for the fast reply Michael!

You're right, I didn't pay attention to this.
But after run: /opt/lo/bin/make kdevelop-ide-integration I got this error:


To build, issue:
/opt/lo/bin/make

For details, run:
/opt/lo/bin/make help

*
* WARNING : If a build created with VS 2012 should run on Windows XP,
* WARNING : use --with-windows-sdk=7.1A (requires VS 2012 Update 1 or newer)
cd G:/sources/libo-core  (make cmd=make -npf Makefile.gbuild all cmd ||
true) | G:/sources/libo-core/bin/gbuild-to-ide --ide kdevelop
Makefile:29: *** target pattern contains no '%'.  Stop.

The complete run output can be viewed here http://pastebin.com/2KUPWuD1.

Anderson Roberto de Oliveira
Skype: andersonn_roberto
LinkedIn: http://br.linkedin.com/in/andersonroberto/


On Tue, Jan 21, 2014 at 5:55 PM, Michael Stahl mst...@redhat.com wrote:

 On 21/01/14 19:51, Anderson Roberto wrote:
  Hello all!
 
  I'm developing a integration with MSVS, like the one with KDevelop and
  XCode.
  But, when I try to run a make kdevelop-ide-integration to use the
  project files as example and to point some infos, I got this error:
  Makefile:29: *** target pattern contains no '%'.  Stop.
 
  Anyone knows what this mean and what I need to do to solve this error?

 most likely it means make does not understand Win32 file names with
 drive letters.  try /opt/lo/bin/make instead, if you don't have that
 read:


 https://wiki.documentfoundation.org/Development/Windows_Build_Dependencies#Install_make_an_optional_ccache_binaries



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


Re: Error doing make kdevelop-ide-integration in Windows + Cygwin.

2014-01-21 Thread Michael Stahl
On 21/01/14 22:24, Anderson Roberto wrote:
 Thanks for the fast reply Michael!
 
 You're right, I didn't pay attention to this.
 But after run: /opt/lo/bin/make kdevelop-ide-integration I got this error:
 
 
 To build, issue:
 /opt/lo/bin/make

 cd G:/sources/libo-core  (make cmd=make -npf Makefile.gbuild all cmd
 || true) | G:/sources/libo-core/bin/gbuild-to-ide --ide kdevelop

this should not run a plain make but /opt/lo/bin/make, not sure why
that happens... ah i see, the developer who added that ide-integration
stuff perhaps wasn't too familiar with our build system and forgot to
use $(GNUMAKE) variable... fixed on master in commit
e20439a1251de36efdd3dfe252b88f33cc94a0b7 (please pull and try again)

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


[Libreoffice-commits] core.git: Makefile.in

2014-01-21 Thread Michael Stahl
 Makefile.in |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e20439a1251de36efdd3dfe252b88f33cc94a0b7
Author: Michael Stahl mst...@redhat.com
Date:   Tue Jan 21 22:31:15 2014 +0100

Makefile.in: hardcoded make - $(GNUMAKE)

Change-Id: I2ec31816562e55f6f95cac38dab4a17a393c0f9a

diff --git a/Makefile.in b/Makefile.in
index 4ac5c46..ec45b51 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -330,7 +330,7 @@ dump-deps-png:
 
 define gb_Top_GbuildToIdeIntegration
 $(1)-ide-integration:
-   cd $(SRCDIR)  (make cmd=make -npf Makefile.gbuild all cmd || true) 
| $(SRCDIR)/bin/gbuild-to-ide --ide $(1)
+   cd $(SRCDIR)  ($(GNUMAKE) cmd=$(GNUMAKE) -npf Makefile.gbuild all 
cmd || true) | $(SRCDIR)/bin/gbuild-to-ide --ide $(1)
 
 endef
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


  1   2   3   >