[Libreoffice-commits] core.git: comphelper/source cui/inc cui/source dbaccess/source desktop/source filter/qa hwpfilter/qa include/comphelper include/sfx2 include/unotest lotuswordpro/qa package/qa sc

2015-03-19 Thread Noel Grandin
 comphelper/source/misc/mimeconfighelper.cxx|   36 ++---
 cui/inc/pch/precompiled_cui.hxx|1 
 cui/source/customize/macropg.cxx   |1 
 cui/source/dialogs/about.cxx   |1 
 cui/source/options/optsave.cxx |8 +-
 dbaccess/source/ui/dlg/ConnectionHelper.cxx|2 
 desktop/source/app/dispatchwatcher.cxx |8 +-
 filter/qa/cppunit/filters-dxf-test.cxx |4 -
 filter/qa/cppunit/filters-eps-test.cxx |4 -
 filter/qa/cppunit/filters-met-test.cxx |4 -
 filter/qa/cppunit/filters-pcd-test.cxx |4 -
 filter/qa/cppunit/filters-pcx-test.cxx |4 -
 filter/qa/cppunit/filters-pict-test.cxx|4 -
 filter/qa/cppunit/filters-ppm-test.cxx |4 -
 filter/qa/cppunit/filters-psd-test.cxx |4 -
 filter/qa/cppunit/filters-ras-test.cxx |4 -
 filter/qa/cppunit/filters-tga-test.cxx |4 -
 filter/qa/cppunit/filters-tiff-test.cxx|4 -
 hwpfilter/qa/cppunit/test_hwpfilter.cxx|4 -
 include/comphelper/documentconstants.hxx   |   57 +++-
 include/comphelper/mimeconfighelper.hxx|6 +-
 include/sfx2/docfac.hxx|1 
 include/sfx2/docfilt.hxx   |   15 ++---
 include/sfx2/fcontnr.hxx   |   33 +--
 include/sfx2/filedlghelper.hxx |5 -
 include/sfx2/sfxdefs.hxx   |   29 --
 include/unotest/filters-test.hxx   |8 +-
 lotuswordpro/qa/cppunit/test_lotuswordpro.cxx  |4 -
 package/qa/cppunit/test_package.cxx|4 -
 sc/inc/scdll.hxx   |1 
 sc/qa/unit/filters-test.cxx|4 -
 sc/qa/unit/helper/qahelper.cxx |   14 ++---
 sc/qa/unit/helper/qahelper.hxx |   26 -
 sc/qa/unit/opencl-test.cxx |4 -
 sc/qa/unit/subsequent_export-test.cxx  |4 -
 sc/qa/unit/subsequent_filters-test.cxx |   12 ++--
 sc/source/ui/docshell/docsh.cxx|2 
 sd/inc/sddll.hxx   |1 
 sd/qa/unit/filters-test.cxx|6 +-
 sd/qa/unit/sdmodeltestbase.hxx |   20 +++
 sd/source/ui/func/fuinsfil.cxx |   26 -
 sd/source/ui/view/sdview4.cxx  |2 
 sfx2/source/appl/appopen.cxx   |4 -
 sfx2/source/appl/shutdownicon.cxx  |2 
 sfx2/source/bastyp/fltfnc.cxx  |   30 +-
 sfx2/source/dialog/filedlghelper.cxx   |   18 +++---
 sfx2/source/doc/docfile.cxx|5 -
 sfx2/source/doc/docfilt.cxx|   10 +--
 sfx2/source/doc/guisaveas.cxx  |   58 ++---
 sfx2/source/doc/objserv.cxx|2 
 sfx2/source/doc/objstor.cxx|   26 -
 sfx2/source/view/frmload.cxx   |4 -
 sfx2/source/view/viewfrm.cxx   |2 
 sot/qa/cppunit/test_sot.cxx|4 -
 starmath/qa/extras/mmlimport-test.cxx  |2 
 svx/source/unodraw/unoshap2.cxx|2 
 sw/inc/swdll.hxx   |1 
 sw/qa/core/filters-test.cxx|   30 +-
 sw/qa/core/uwriter.cxx |2 
 sw/source/core/doc/docglbl.cxx |2 
 sw/source/core/unocore/unocrsrhelper.cxx   |2 
 sw/source/ui/dbui/mmdocselectpage.cxx  |2 
 sw/source/ui/dbui/mmoutputpage.cxx |6 +-
 sw/source/ui/envelp/mailmrge.cxx   |4 -
 sw/source/uibase/dbui/dbmgr.cxx|2 
 sw/source/uibase/dochdl/gloshdl.cxx|2 
 sw/source/uibase/uiview/srcview.cxx|2 
 sw/source/uibase/uiview/view2.cxx  |4 -
 unotest/source/cpp/filters-test.cxx|4 -
 vcl/qa/cppunit/graphicfilter/filters-test.cxx  |4 -
 writerfilter/qa/cppunittests/rtftok/testrtftok.cxx |4 -
 writerperfect/qa/unit/WpftImportTestBase.cxx   |2 
 writerperfect/qa/unit/WpftImportTestBase.hxx   |2 
 73 files changed, 302 insertions(+), 331 deletions(-)

New commits:
commit 48bb2913a7459edb17a5f62c2f6e6fbc4f1051e2
Author: Noel Grandin 
Date:   Wed Mar 18 13:03:28 2015 +0200

convert SFX_FILTER_ constants to enum class

Change-Id: I7e53cfc90cefd9da7d6ecd795b09214bd44b1613

diff --git a/comphelper/source/misc/mimeconfighelper.cxx 
b/comphelper/source/misc/mimeconfighelper.cxx
index ee9b6e5..806fc64 100644
--- 

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

2015-03-19 Thread Tor Lillqvist
 include/sfx2/docfilt.hxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 366af3559aa8e5767486f76e4a281f703a77ccea
Author: Tor Lillqvist 
Date:   Thu Mar 19 09:34:16 2015 +0200

SFX_FILTER_STARTPRESENTATION is unused now

Change-Id: I0b82e6d582ad7fa7e860a6b00d48444c65439671

diff --git a/include/sfx2/docfilt.hxx b/include/sfx2/docfilt.hxx
index dad0da5..976ffd6 100644
--- a/include/sfx2/docfilt.hxx
+++ b/include/sfx2/docfilt.hxx
@@ -32,8 +32,6 @@
 #include 
 #include 
 
-#define SFX_FILTER_STARTPRESENTATION 0x2000L
-
 class SfxFilterContainer;
 class SotStorage;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Noel Grandin
 comphelper/source/misc/mimeconfighelper.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 4d2de5816378d119d78e81f60075c88baec8fef2
Author: Noel Grandin 
Date:   Thu Mar 19 09:55:37 2015 +0200

fix windows build

after my commit 48bb2913a7459edb17a5f62c2f6e6fbc4f1051e2
"convert SFX_FILTER_ constants to enum class"

Change-Id: Ieaf524e076f5ecfe16b312a310cd6f5cb8bd637d

diff --git a/comphelper/source/misc/mimeconfighelper.cxx 
b/comphelper/source/misc/mimeconfighelper.cxx
index 806fc64..17269a3 100644
--- a/comphelper/source/misc/mimeconfighelper.cxx
+++ b/comphelper/source/misc/mimeconfighelper.cxx
@@ -671,9 +671,9 @@ OUString 
MimeConfigurationHelper::UpdateMediaDescriptorWithFilterName(
 
 #ifdef WNT
 
-sal_Int32 MimeConfigurationHelper::GetFilterFlags( const OUString& aFilterName 
)
+SfxFilterFlags MimeConfigurationHelper::GetFilterFlags( const OUString& 
aFilterName )
 {
-sal_Int32 nFlags = 0;
+SfxFilterFlags nFlags = SfxFilterFlags::NONE;
 try
 {
 if ( !aFilterName.isEmpty() )
@@ -687,7 +687,7 @@ sal_Int32 MimeConfigurationHelper::GetFilterFlags( const 
OUString& aFilterName )
 if ( aFilterAny >>= aData )
 {
 SequenceAsHashMap aFilterHM( aData );
-nFlags = aFilterHM.getUnpackedValueOrDefault( "Flags", 
(sal_Int32)0 );
+nFlags = 
static_cast(aFilterHM.getUnpackedValueOrDefault( "Flags", 
(sal_Int32)0 ));
 }
 }
 } catch( uno::Exception& )
@@ -704,7 +704,7 @@ sal_Bool MimeConfigurationHelper::AddFilterNameCheckOwnFile(
 OUString aFilterName = UpdateMediaDescriptorWithFilterName( aMediaDescr, 
sal_False );
 if ( !aFilterName.isEmpty() )
 {
-sal_Int32 nFlags = GetFilterFlags( aFilterName );
+SfxFilterFlags nFlags = GetFilterFlags( aFilterName );
 // check the OWN flag
 bResult = ( nFlags & SfxFilterFlags::OWN );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Miklos Vajna
 sw/source/filter/ww8/docxsdrexport.cxx |8 +++-
 sw/source/filter/ww8/docxsdrexport.hxx |2 +-
 2 files changed, 4 insertions(+), 6 deletions(-)

New commits:
commit 5b55db59f93d526e7c2e192194aa562c640cce16
Author: Miklos Vajna 
Date:   Thu Mar 19 09:16:16 2015 +0100

DocxSdrExport::Impl: use std::unique_ptr for m_pFlyFillAttrList

Change-Id: I134f375f3785beb05f320be67990d3b2d572d46f

diff --git a/sw/source/filter/ww8/docxsdrexport.cxx 
b/sw/source/filter/ww8/docxsdrexport.cxx
index a642b17..0f0bc3a 100644
--- a/sw/source/filter/ww8/docxsdrexport.cxx
+++ b/sw/source/filter/ww8/docxsdrexport.cxx
@@ -140,7 +140,7 @@ struct DocxSdrExport::Impl
 bool m_bParagraphSdtOpen;
 bool m_bParagraphHasDrawing; ///Flag for checking drawing in a paragraph.
 bool m_bFlyFrameGraphic;
-sax_fastparser::FastAttributeList* m_pFlyFillAttrList;
+std::unique_ptr m_pFlyFillAttrList;
 sax_fastparser::FastAttributeList* m_pFlyWrapAttrList;
 sax_fastparser::FastAttributeList* m_pBodyPrAttrList;
 sax_fastparser::FastAttributeList* m_pDashLineStyleAttr;
@@ -166,7 +166,6 @@ struct DocxSdrExport::Impl
   m_bParagraphSdtOpen(false),
   m_bParagraphHasDrawing(false),
   m_bFlyFrameGraphic(false),
-  m_pFlyFillAttrList(0),
   m_pFlyWrapAttrList(0),
   m_pBodyPrAttrList(0),
   m_pDashLineStyleAttr(0),
@@ -263,7 +262,7 @@ void DocxSdrExport::setParagraphHasDrawing(bool 
bParagraphHasDrawing)
 m_pImpl->m_bParagraphHasDrawing = bParagraphHasDrawing;
 }
 
-sax_fastparser::FastAttributeList*& DocxSdrExport::getFlyFillAttrList()
+std::unique_ptr& 
DocxSdrExport::getFlyFillAttrList()
 {
 return m_pImpl->m_pFlyFillAttrList;
 }
@@ -1618,8 +1617,7 @@ void DocxSdrExport::writeVMLTextFrame(sw::Frame* 
pParentFrame, bool bTextBoxOnly
 m_pImpl->textFrameShadow(rFrmFmt);
 if (m_pImpl->m_pFlyFillAttrList)
 {
-sax_fastparser::XFastAttributeListRef 
xFlyFillAttrList(m_pImpl->m_pFlyFillAttrList);
-m_pImpl->m_pFlyFillAttrList = NULL;
+sax_fastparser::XFastAttributeListRef 
xFlyFillAttrList(m_pImpl->m_pFlyFillAttrList.release());
 pFS->singleElementNS(XML_v, XML_fill, xFlyFillAttrList);
 }
 if (m_pImpl->m_pDashLineStyleAttr)
diff --git a/sw/source/filter/ww8/docxsdrexport.hxx 
b/sw/source/filter/ww8/docxsdrexport.hxx
index da2d975..a086b2a 100644
--- a/sw/source/filter/ww8/docxsdrexport.hxx
+++ b/sw/source/filter/ww8/docxsdrexport.hxx
@@ -76,7 +76,7 @@ public:
 bool IsDMLAndVMLDrawingOpen();
 bool IsParagraphHasDrawing();
 void setParagraphHasDrawing(bool bParagraphHasDrawing);
-sax_fastparser::FastAttributeList*& getFlyFillAttrList();
+std::unique_ptr& getFlyFillAttrList();
 sax_fastparser::FastAttributeList* getFlyWrapAttrList();
 void setFlyWrapAttrList(sax_fastparser::FastAttributeList* pAttrList);
 /// Attributes of , used during DML export of text frames.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Noel Grandin
 include/comphelper/mimeconfighelper.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit adef956348b9ad0420a482367f34ca1df81ab159
Author: Noel Grandin 
Date:   Thu Mar 19 10:34:15 2015 +0200

fix windows build(2)

after my commit 48bb2913a7459edb17a5f62c2f6e6fbc4f1051e2
"convert SFX_FILTER_ constants to enum class"

Change-Id: I4e8e04bb5b9f2493281826fb156fee51cf40

diff --git a/include/comphelper/mimeconfighelper.hxx 
b/include/comphelper/mimeconfighelper.hxx
index 9f9d9fd..5ba0eb9 100644
--- a/include/comphelper/mimeconfighelper.hxx
+++ b/include/comphelper/mimeconfighelper.hxx
@@ -113,7 +113,7 @@ public:
 ::com::sun::star::uno::Sequence< 
::com::sun::star::beans::PropertyValue >& aMediaDescr,
 ::com::sun::star::uno::Sequence< 
::com::sun::star::beans::NamedValue >& aObject );
 #ifdef WNT
-sal_Int32 GetFilterFlags( const OUString& aFilterName );
+SfxFilterFlags GetFilterFlags( const OUString& aFilterName );
 
 sal_Bool AddFilterNameCheckOwnFile(
 ::com::sun::star::uno::Sequence< 
::com::sun::star::beans::PropertyValue >& aMediaDescr );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: cli_ure/source include/o3tl include/sal pyuno/source solenv/gcc-wrappers

2015-03-19 Thread Tor Lillqvist
 cli_ure/source/climaker/climaker_app.cxx  |8 
 cli_ure/source/climaker/climaker_emit.cxx |5 -
 include/o3tl/typed_flags_set.hxx  |   12 +++-
 include/sal/log.hxx   |   12 
 pyuno/source/module/pyuno_impl.hxx|2 +-
 solenv/gcc-wrappers/wrapper.cxx   |2 --
 6 files changed, 4 insertions(+), 37 deletions(-)

New commits:
commit a251fe4d48237a4d9c9530dafc7bbdce6028e9cf
Author: Tor Lillqvist 
Date:   Thu Mar 19 10:57:17 2015 +0200

We require MSVC2013 (_MSC_VER == 1800) now

Change-Id: Ice8504041f22e00f2e5010813d9dff1d2987c8d6

diff --git a/cli_ure/source/climaker/climaker_app.cxx 
b/cli_ure/source/climaker/climaker_app.cxx
index b8e3f0f..42381ad 100644
--- a/cli_ure/source/climaker/climaker_app.cxx
+++ b/cli_ure/source/climaker/climaker_app.cxx
@@ -630,20 +630,12 @@ SAL_IMPLEMENT_MAIN()
 
 if (g_verbose)
 {
-#if _MSC_VER < 1700
-// Bogus: warning C4564: method 'CheckInvalidPathChars' of class 
'System::IO::Path' defines unsupported default parameter 'checkAdditional'
-#pragma warning (push)
-#pragma warning (disable: 4564)
-#endif
 ::System::Console::Write(
 "> saving assembly {0}{1}{2}...",
 output_dir,
 gcnew ::System::String(
 ::System::IO::Path::DirectorySeparatorChar, 1 ),
 output_file );
-#if _MSC_VER < 1700
-#pragma warning (pop)
-#endif
 }
 assembly_builder->Save( output_file );
 if (g_verbose)
diff --git a/cli_ure/source/climaker/climaker_emit.cxx 
b/cli_ure/source/climaker/climaker_emit.cxx
index 6f0de95..8706c3c 100644
--- a/cli_ure/source/climaker/climaker_emit.cxx
+++ b/cli_ure/source/climaker/climaker_emit.cxx
@@ -876,11 +876,6 @@ Assembly ^ TypeEmitter::type_resolve(
  MethodAttributes::Virtual |
  MethodAttributes::NewSlot |
  MethodAttributes::HideBySig);
-//#if defined(_MSC_VER) && (_MSC_VER < 1400)
-// MethodAttributes::Instance);
-//#else
-//   Instance);
-//#endif
 
 if (TypeClass_INTERFACE_METHOD == xMember->getTypeClass())
 {
diff --git a/include/o3tl/typed_flags_set.hxx b/include/o3tl/typed_flags_set.hxx
index 4b6e4f5..3e96003 100644
--- a/include/o3tl/typed_flags_set.hxx
+++ b/include/o3tl/typed_flags_set.hxx
@@ -58,15 +58,9 @@ struct is_typed_flags {
 
 operator E() { return static_cast(value_); }
 
-#if !defined _MSC_VER || _MSC_VER > 1700
-explicit
-#endif
-operator typename underlying_type::type() { return value_; }
-
-#if !defined _MSC_VER || _MSC_VER > 1700
-explicit
-#endif
-operator bool() { return value_ != 0; }
+explicit operator typename underlying_type::type() { return value_; 
}
+
+explicit operator bool() { return value_ != 0; }
 
 private:
 typename underlying_type::type value_;
diff --git a/include/sal/log.hxx b/include/sal/log.hxx
index 563ed57..62cdb77 100644
--- a/include/sal/log.hxx
+++ b/include/sal/log.hxx
@@ -74,27 +74,18 @@ template< typename T > inline StreamIgnore operator <<(
 SAL_UNUSED_PARAMETER StreamStart const &, SAL_UNUSED_PARAMETER T const &)
 {
 std::abort();
-#if defined _MSC_VER && _MSC_VER < 1700
-return StreamIgnore();
-#endif
 }
 
 template< typename T > inline StreamIgnore operator <<(
 SAL_UNUSED_PARAMETER StreamString const &, SAL_UNUSED_PARAMETER T const &)
 {
 std::abort();
-#if defined _MSC_VER && _MSC_VER < 1700
-return StreamIgnore();
-#endif
 }
 
 template< typename T > inline StreamIgnore operator <<(
 SAL_UNUSED_PARAMETER StreamIgnore const &, SAL_UNUSED_PARAMETER T const &)
 {
 std::abort();
-#if defined _MSC_VER && _MSC_VER < 1700
-return StreamIgnore();
-#endif
 }
 
 template< typename T > typename T::Result getResult(T const &);
@@ -103,9 +94,6 @@ inline char const * unwrapStream(StreamString const & s) { 
return s.string; }
 
 inline char const * unwrapStream(SAL_UNUSED_PARAMETER StreamIgnore const &) {
 std::abort();
-#if defined _MSC_VER && _MSC_VER < 1700
-return 0;
-#endif
 }
 
 } }
diff --git a/pyuno/source/module/pyuno_impl.hxx 
b/pyuno/source/module/pyuno_impl.hxx
index 83fbda5..e066b73 100644
--- a/pyuno/source/module/pyuno_impl.hxx
+++ b/pyuno/source/module/pyuno_impl.hxx
@@ -19,7 +19,7 @@
 #ifndef INCLUDED_PYUNO_SOURCE_MODULE_PYUNO_IMPL_HXX
 #define INCLUDED_PYUNO_SOURCE_MODULE_PYUNO_IMPL_HXX
 
-#if defined(_MSC_VER) && (_MSC_VER >= 1600)
+#if defined(_MSC_VER)
 // Workaround for some horrible hypot() mess
 #include 
 #endif
diff --git a/solenv/gcc-wrappers/wrapper.cxx b/solenv/gcc-wrappers/wrapper.cxx
index 918a9b5..9dacb26 100644
--- a/solenv/gcc-wrappers/wrapper.cxx
+++ b/solenv/gcc-wrappers/wrapper.cxx
@@ -118,9 +118,7 @@ string processccargs(vector rawargs) {
 }
 else if(*i == "-g") {
 args.append("-Zi");
-#if _MSC_VER >= 1800
 args.append(" -FS");
-#endif

Re: Summer of Code (March 17,2015)

2015-03-19 Thread Caolán McNamara
On Wed, 2015-03-18 at 21:04 +0200, Efe Gürkan YALAMAN wrote:
> 
> Somewhere in the bugzilla I think Caolan mentioned it can be
> implemented as a tree-view because the options are in a tree structure
> and it would fit better then a simple table. Also it solves the
> dialog's usability issues too. I hope that will solve the
> accessibility problems too.

Yeah, I think that's the best approach, to try and use a tree in there.
(a SvTreeList I suppose) because the amount of elements in the list is
just crazy, 30,000+ IIRC. Ideally on-demand loaded, e.g. tree is
collapsed and just shows +com.sun.star, when you expand that, then only
then is the subtree inserted, and so on. So the full 30,000+ list isn't
actually in the widget, only the expanded portion.

C.


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


Re: Crash test update

2015-03-19 Thread Caolán McNamara
On Tue, 2015-03-17 at 10:40 +, Crashtest VM wrote:
> New crashtest update available at 
> http://dev-builds.libreoffice.org/crashtest/ea7f16bf0b5e195d65e51c624b0fb37980bf2b95/

The hundreds of presentation import failures should now be fixed with
http://cgit.freedesktop.org/libreoffice/core/commit/?id=8a5069cbaacf8a78ee66d6d8f2a46daefe1bec56

C.

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


[Libreoffice-commits] core.git: Branch 'feature/tiled-editing' - configure.ac

2015-03-19 Thread Miklos Vajna
 configure.ac |   12 
 1 file changed, 12 insertions(+)

New commits:
commit be1e711aab64ec56f07fda40c4670c5415832a1c
Author: Miklos Vajna 
Date:   Thu Mar 19 10:08:26 2015 +0100

android: add a check for the installed memory in the machine

... and just error out with < 16G and all symbols. Should help newcomers
who have e.g. 8GB of memory, build with dbgutil, but then unable to
complete the build as the linker never finishes.

Change-Id: I9f3ad5c6d76a18d6aa5b0b2d61aa115ed441b8e3

diff --git a/configure.ac b/configure.ac
index e8d4fe8..57fcdb0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3858,6 +3858,18 @@ else
 fi
 AC_SUBST(ENABLE_SYMBOLS)
 
+if test -n "$with_android_ndk" -a \( -n "$ENABLE_SYMBOLS" -o -n 
"$ENABLE_DEBUG" -o -n "$ENABLE_DBGUTIL" \) -a "$ENABLE_DEBUGINFO_FOR" = "all"; 
then
+# Building on Android with full symbols: without enough memory the linker 
never finishes currently.
+AC_MSG_CHECKING([whether enough memory is available for linking])
+mem_size=$(grep -o 'MemTotal: *.\+ kB' /proc/meminfo | sed 's/MemTotal: 
*\(.\+\) kB/\1/')
+# Check for 15GB, as Linux reports a bit less than the physical memory 
size.
+if test -n "$mem_size" -a $mem_size -lt 15728640; then
+AC_MSG_ERROR([building with full symbols and less than 16GB of memory 
is not supported])
+else
+AC_MSG_RESULT([yes])
+fi
+fi
+
 # Debug information format for iOS. Running dsymutil takes a long time... you 
really need a separate
 # .dSYM only if running Instruments, I think. (Not for normal debugging in 
Xcode.) To enable a
 # separate .dSYM, either use --enable-release-build or change manually to 
"DWARF with DSYM" in Xcode.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/tiled-editing' - libreofficekit/qa

2015-03-19 Thread Miklos Vajna
 libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx |8 
 1 file changed, 8 deletions(-)

New commits:
commit 060c443e2b0b400a376f9e6ae48f178c6c8ec173
Author: Miklos Vajna 
Date:   Thu Mar 19 10:43:46 2015 +0100

gtktiledviewer: remove unimplemented gtk-zoom-fit button

Change-Id: Iaecedd01d7e7bba90e49ddaa5f36859d9218242d

diff --git a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx 
b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
index 5b7fa65..755b3b2 100644
--- a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
+++ b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
@@ -73,10 +73,6 @@ void changeZoom( GtkWidget* pButton, gpointer /* pItem */ )
 {
 fZoom = 1;
 }
-else if ( strcmp(sName, "gtk-zoom-fit") == 0)
-{
-// TODO -- will need access to lokdocview internals?
-}
 else if ( strcmp(sName, "gtk-zoom-out") == 0)
 {
 for ( unsigned int i = 0; i < sizeof( fZooms ) / sizeof( fZooms[0] ); 
i++ )
@@ -287,10 +283,6 @@ int main( int argc, char* argv[] )
 gtk_toolbar_insert( GTK_TOOLBAR(pToolbar), pZoom1, -1);
 g_signal_connect( G_OBJECT(pZoom1), "clicked", G_CALLBACK(changeZoom), 
NULL );
 
-GtkToolItem* pZoomFit = gtk_tool_button_new_from_stock( GTK_STOCK_ZOOM_FIT 
);
-gtk_toolbar_insert( GTK_TOOLBAR(pToolbar), pZoomFit, -1);
-g_signal_connect( G_OBJECT(pZoomFit), "clicked", G_CALLBACK(changeZoom), 
NULL );
-
 GtkToolItem* pZoomOut = gtk_tool_button_new_from_stock( GTK_STOCK_ZOOM_OUT 
);
 gtk_toolbar_insert( GTK_TOOLBAR(pToolbar), pZoomOut, -1);
 g_signal_connect( G_OBJECT(pZoomOut), "clicked", G_CALLBACK(changeZoom), 
NULL );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Caolán McNamara
 comphelper/source/misc/mimeconfighelper.cxx |   10 --
 include/comphelper/mimeconfighelper.hxx |2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)

New commits:
commit 944d1fd5ca97969bc2cf45376737813c588f6637
Author: Caolán McNamara 
Date:   Thu Mar 19 10:21:05 2015 +

fix windows build and sal_Bool->bool

Change-Id: If88a5b1171ef0b586861dd071820bc993c4cf06e

diff --git a/comphelper/source/misc/mimeconfighelper.cxx 
b/comphelper/source/misc/mimeconfighelper.cxx
index 17269a3..361cc00 100644
--- a/comphelper/source/misc/mimeconfighelper.cxx
+++ b/comphelper/source/misc/mimeconfighelper.cxx
@@ -696,23 +696,21 @@ SfxFilterFlags MimeConfigurationHelper::GetFilterFlags( 
const OUString& aFilterN
 return nFlags;
 }
 
-sal_Bool MimeConfigurationHelper::AddFilterNameCheckOwnFile(
+bool MimeConfigurationHelper::AddFilterNameCheckOwnFile(
 uno::Sequence< beans::PropertyValue >& aMediaDescr )
 {
-sal_Bool bResult = sal_False;
-
 OUString aFilterName = UpdateMediaDescriptorWithFilterName( aMediaDescr, 
sal_False );
 if ( !aFilterName.isEmpty() )
 {
 SfxFilterFlags nFlags = GetFilterFlags( aFilterName );
 // check the OWN flag
-bResult = ( nFlags & SfxFilterFlags::OWN );
+return bool(nFlags & SfxFilterFlags::OWN);
 }
 
-return bResult;
+return false;
 }
-#endif
 
+#endif
 
 OUString MimeConfigurationHelper::GetDefaultFilterFromServiceName( const 
OUString& aServiceName, sal_Int32 nVersion )
 {
diff --git a/include/comphelper/mimeconfighelper.hxx 
b/include/comphelper/mimeconfighelper.hxx
index 5ba0eb9..4791109 100644
--- a/include/comphelper/mimeconfighelper.hxx
+++ b/include/comphelper/mimeconfighelper.hxx
@@ -115,7 +115,7 @@ public:
 #ifdef WNT
 SfxFilterFlags GetFilterFlags( const OUString& aFilterName );
 
-sal_Bool AddFilterNameCheckOwnFile(
+bool AddFilterNameCheckOwnFile(
 ::com::sun::star::uno::Sequence< 
::com::sun::star::beans::PropertyValue >& aMediaDescr );
 #endif
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Caolán McNamara
 sfx2/source/bastyp/fltfnc.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0597e4be3eb1be2987db28507055126bc5a26421
Author: Caolán McNamara 
Date:   Thu Mar 19 10:25:39 2015 +

WaE: -Werror=maybe-uninitialized

Change-Id: I310099e8e12fb6440e98fbca97af512a4f872ac0

diff --git a/sfx2/source/bastyp/fltfnc.cxx b/sfx2/source/bastyp/fltfnc.cxx
index 78d1897..e759875e 100644
--- a/sfx2/source/bastyp/fltfnc.cxx
+++ b/sfx2/source/bastyp/fltfnc.cxx
@@ -998,7 +998,7 @@ void SfxFilterContainer::ReadSingleFilter_Impl(
 }
 else if ( lFilterProperties[nFilterProperty].Name == "Flags" )
 {
-sal_Int32 nTmp;
+sal_Int32 nTmp(0);
 lFilterProperties[nFilterProperty].Value >>= nTmp;
 nFlags = static_cast(nTmp);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Noel Grandin
 desktop/source/app/dispatchwatcher.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b8d6062a120a70f155e296c72a856cde2c5d142c
Author: Noel Grandin 
Date:   Thu Mar 19 12:29:25 2015 +0200

fix android build

workaround template issue

Change-Id: Ifeb828685b3f63159249211aa32a90dcba2e5570

diff --git a/desktop/source/app/dispatchwatcher.cxx 
b/desktop/source/app/dispatchwatcher.cxx
index c61aef3..c243766 100644
--- a/desktop/source/app/dispatchwatcher.cxx
+++ b/desktop/source/app/dispatchwatcher.cxx
@@ -97,7 +97,7 @@ const SfxFilter* impl_lookupExportFilterForUrl( const 
rtl::OUString& rUrl, const
 sQuery.append(":iflags=");
 
sQuery.append(OUString::number(static_cast(SfxFilterFlags::EXPORT)));
 sQuery.append(":eflags=");
-
sQuery.append(OUString::number(static_cast(SFX_FILTER_NOTINSTALLED)));
+sQuery.append(OUString::number(static_cast(SFX_FILTER_NOTINSTALLED)));
 
 const Reference< XComponentContext > xContext( 
comphelper::getProcessComponentContext() );
 const Reference< XContainerQuery > xFilterFactory(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sal/android

2015-03-19 Thread Michaël Lefèvre
 sal/android/uthash.h |   16 
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit b18bd0b774436ad2a85b95491a3efd02c249
Author: Michaël Lefèvre 
Date:   Fri Mar 13 15:46:44 2015 +0100

CppCheck cleaning : scope reduction and printf format

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

diff --git a/sal/android/uthash.h b/sal/android/uthash.h
index 0760469..fe9d13d 100644
--- a/sal/android/uthash.h
+++ b/sal/android/uthash.h
@@ -81,9 +81,9 @@ typedef unsigned int uint32_t;
 
 #define HASH_FIND(hh,head,keyptr,keylen,out)   
  \
 do {   
  \
-  unsigned _hf_bkt,_hf_hashv;  
  \
   out=NULL;
  \
   if (head) {  
  \
+ unsigned _hf_bkt,_hf_hashv;   
  \
  HASH_FCN(keyptr,keylen, (head)->hh.tbl->num_buckets, _hf_hashv, _hf_bkt); 
  \
  if (HASH_BLOOM_TEST((head)->hh.tbl, _hf_hashv)) { 
  \
HASH_FIND_IN_BKT((head)->hh.tbl, hh, (head)->hh.tbl->buckets[ _hf_bkt 
],  \
@@ -247,11 +247,11 @@ do {
 #define HASH_OOPS(...) do { fprintf(stderr,__VA_ARGS__); exit(-1); } while (0)
 #define HASH_FSCK(hh,head) 
  \
 do {   
  \
-unsigned _bkt_i;   
  \
-unsigned _count, _bkt_count;   
  \
-char *_prev;   
  \
-struct UT_hash_handle *_thh;   
  \
 if (head) {
  \
+unsigned _bkt_i;   
  \
+unsigned _count, _bkt_count;   
  \
+char *_prev;   
  \
+struct UT_hash_handle *_thh;   
  \
 _count = 0;
  \
 for( _bkt_i = 0; _bkt_i < (head)->hh.tbl->num_buckets; _bkt_i++) { 
  \
 _bkt_count = 0;
  \
@@ -268,12 +268,12 @@ do {
 }  
  \
 _count += _bkt_count;  
  \
 if ((head)->hh.tbl->buckets[_bkt_i].count !=  _bkt_count) {
  \
-   HASH_OOPS("invalid bucket count %d, actual %d\n",   
  \
+   HASH_OOPS("invalid bucket count %d, actual %u\n",   
  \
 (head)->hh.tbl->buckets[_bkt_i].count, _bkt_count);
  \
 }  
  \
 }  
  \
 if (_count != (head)->hh.tbl->num_items) { 
  \
-HASH_OOPS("invalid hh item count %d, actual %d\n", 
  \
+HASH_OOPS("invalid hh item count %d, actual %u\n", 
  \
 (head)->hh.tbl->num_items, _count );   
  \
 }  
  \
 /* traverse hh in app order; check next/prev integrity, count */   
  \
@@ -291,7 +291,7 @@ do {
   (head)->hh.tbl->hho) : NULL );   
  \
 }  
  \
 if (_count != (head)->hh.tbl->num_items) { 
  \
-HASH_OOPS("invalid app item count %d, actual %d\n",
  \
+HASH_OOPS("invalid app item count %d, actual %u\n",
  \
 (head)->hh.tbl->num_items, _count );   
  \
 }  
  \
 }  
  \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Rishabh Kumar
 cui/source/tabpages/page.cxx |   16 
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit 0b76f7ee3f7ac7c37c62b4d91307a20d9faca01a
Author: Rishabh Kumar 
Date:   Tue Mar 10 08:33:53 2015 +0530

tdf#86787: Page style margin limited to 99.99mm

Normailizes MarginEdit before assigning values to SetMax and SetLast

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

diff --git a/cui/source/tabpages/page.cxx b/cui/source/tabpages/page.cxx
index 80880c6..d321103 100644
--- a/cui/source/tabpages/page.cxx
+++ b/cui/source/tabpages/page.cxx
@@ -354,14 +354,14 @@ SvxPageDescPage::SvxPageDescPage( vcl::Window* pParent, 
const SfxItemSet& rAttr
 
m_pPaperHeightEdit->SetLast(m_pPaperHeightEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperHeight()),
 FUNIT_CM);
 
 // #i4219# also for margins (1/100th cm). Was: , keeping.
-m_pLeftMarginEdit->SetMax(aDrawinglayerOpt.GetMaximumPaperLeftMargin());
-m_pLeftMarginEdit->SetLast(aDrawinglayerOpt.GetMaximumPaperLeftMargin());
-m_pRightMarginEdit->SetMax(aDrawinglayerOpt.GetMaximumPaperRightMargin());
-m_pRightMarginEdit->SetLast(aDrawinglayerOpt.GetMaximumPaperRightMargin());
-m_pTopMarginEdit->SetMax(aDrawinglayerOpt.GetMaximumPaperTopMargin());
-m_pTopMarginEdit->SetLast(aDrawinglayerOpt.GetMaximumPaperTopMargin());
-
m_pBottomMarginEdit->SetMax(aDrawinglayerOpt.GetMaximumPaperBottomMargin());
-
m_pBottomMarginEdit->SetLast(aDrawinglayerOpt.GetMaximumPaperBottomMargin());
+
m_pLeftMarginEdit->SetMax(m_pLeftMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperLeftMargin()),
 FUNIT_MM);
+
m_pLeftMarginEdit->SetLast(m_pLeftMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperLeftMargin()),
 FUNIT_MM);
+
m_pRightMarginEdit->SetMax(m_pRightMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperRightMargin()),
 FUNIT_MM);
+
m_pRightMarginEdit->SetLast(m_pRightMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperRightMargin()),
 FUNIT_MM);
+
m_pTopMarginEdit->SetMax(m_pTopMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperTopMargin()),
 FUNIT_MM);
+
m_pTopMarginEdit->SetLast(m_pTopMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperTopMargin()),
 FUNIT_MM);
+
m_pBottomMarginEdit->SetMax(m_pBottomMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperBottomMargin()),
 FUNIT_MM);
+
m_pBottomMarginEdit->SetLast(m_pBottomMarginEdit->Normalize(aDrawinglayerOpt.GetMaximumPaperBottomMargin()),
 FUNIT_MM);
 
 m_pPortraitBtn->SetAccessibleRelationMemberOf(m_pOrientationFT);
 m_pLandscapeBtn->SetAccessibleRelationMemberOf(m_pOrientationFT);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Noel Grandin
 vcl/generic/glyphs/scrptrun.cxx |  155 +++-
 vcl/generic/glyphs/scrptrun.h   |   10 --
 2 files changed, 75 insertions(+), 90 deletions(-)

New commits:
commit b9880e49e5fbae377eca82e60b88cbbdca8bd5d2
Author: Noel Grandin 
Date:   Wed Mar 11 13:55:51 2015 +0200

Speed up ScriptRun::getPairIndex

Noted as a hot spot in the callgrind of
test-files/loperf/091_fdo#39372.odt

The idea for the lookup tables is credited to mjayfrancis.

This takes it from 1.8 billion instructions to 1.1 billion instructions.

This change has been submitted upstream (along with other changes
to these source files that have not yet been pushed there)
in ICU ticket:
http://bugs.icu-project.org/trac/ticket/11577

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

diff --git a/vcl/generic/glyphs/scrptrun.cxx b/vcl/generic/glyphs/scrptrun.cxx
index cd7ce41..d6557d7 100644
--- a/vcl/generic/glyphs/scrptrun.cxx
+++ b/vcl/generic/glyphs/scrptrun.cxx
@@ -33,103 +33,98 @@
  *   created on: 10/17/2001
  *   created by: Eric R. Mader
  */
-
+/**
+  * This file is largely copied from the ICU project,
+  * under folder source/extra/scrptrun/scrptrun.cpp
+  */
 #include "unicode/utypes.h"
 #include "unicode/uscript.h"
 
 #include "scrptrun.h"
+#include 
 
-#define ARRAY_SIZE(array) (sizeof array  / sizeof array[0])
-
-namespace vcl {
-
-const char ScriptRun::fgClassID=0;
-
-UChar32 ScriptRun::pairedChars[] = {
-0x0028, 0x0029, // ascii paired punctuation
-0x003c, 0x003e,
-0x005b, 0x005d,
-0x007b, 0x007d,
-0x00ab, 0x00bb, // guillemets
-0x2018, 0x2019, // general punctuation
-0x201c, 0x201d,
-0x2039, 0x203a,
-0x3008, 0x3009, // chinese paired punctuation
-0x300a, 0x300b,
-0x300c, 0x300d,
-0x300e, 0x300f,
-0x3010, 0x3011,
-0x3014, 0x3015,
-0x3016, 0x3017,
-0x3018, 0x3019,
-0x301a, 0x301b
-};
-
-const int32_t ScriptRun::pairedCharCount = ARRAY_SIZE(pairedChars);
-const int32_t ScriptRun::pairedCharPower = 1 << highBit(pairedCharCount);
-const int32_t ScriptRun::pairedCharExtra = pairedCharCount - pairedCharPower;
+namespace {
 
-int8_t ScriptRun::highBit(int32_t value)
+struct PairIndices
 {
-if (value <= 0) {
-return -32;
-}
-
-int8_t bit = 0;
-
-if (value >= 1 << 16) {
-value >>= 16;
-bit += 16;
+int8_t ma00[0xff];
+int8_t ma20[0x7f];
+int8_t ma30[0x7f];
+
+PairIndices()
+{
+std::fill_n(ma00, 0xff, -1);
+std::fill_n(ma20, 0x7f, -1);
+std::fill_n(ma30, 0x7f, -1);
+
+// characters in the range 0x - 0x007e (inclusive)
+// ascii paired punctuation
+ma00[0x28] =  0;
+ma00[0x29] =  1;
+ma00[0x3c] =  2;
+ma00[0x3e] =  3;
+ma00[0x5b] =  4;
+ma00[0x5d] =  5;
+ma00[0x7b] =  6;
+ma00[0x7d] =  7;
+// guillemets
+ma00[0xab] =  8;
+ma00[0xbb] =  9;
+
+// characters in the range 0x2000 - 0x207e (inclusive)
+// general punctuation
+ma20[0x18] = 10;
+ma20[0x19] = 11;
+ma20[0x1c] = 12;
+ma20[0x1d] = 13;
+ma20[0x39] = 14;
+ma20[0x3a] = 15;
+
+// characters in the range 0x3000 - 0x307e (inclusive)
+// chinese paired punctuation
+ma30[0x08] = 16;
+ma30[0x09] = 17;
+ma30[0x0a] = 18;
+ma30[0x0b] = 19;
+ma30[0x0c] = 20;
+ma30[0x0d] = 21;
+ma30[0x0e] = 22;
+ma30[0x0f] = 23;
+ma30[0x10] = 24;
+ma30[0x11] = 25;
+ma30[0x14] = 26;
+ma30[0x15] = 27;
+ma30[0x16] = 28;
+ma30[0x17] = 29;
+ma30[0x18] = 30;
+ma30[0x19] = 31;
+ma30[0x1a] = 32;
+ma30[0x1b] = 33;
 }
 
-if (value >= 1 << 8) {
-value >>= 8;
-bit += 8;
+inline int32_t getPairIndex(UChar32 ch) const
+{
+if (ch < 0xff)
+return ma00[ch];
+if (ch >= 0x2000 && ch < 0x207f)
+return ma20[ch - 0x2000];
+if (ch >= 0x3000 && ch < 0x307f)
+return ma30[ch - 0x3000];
+return -1;
 }
 
-if (value >= 1 << 4) {
-value >>= 4;
-bit += 4;
-}
-
-if (value >= 1 << 2) {
-value >>= 2;
-bit += 2;
-}
-
-if (value >= 1 << 1) {
-value >>= 1;
-bit += 1;
-}
+};
 
-return bit;
 }
 
-int32_t ScriptRun::getPairIndex(UChar32 ch)
-{
-int32_t probe = pairedCharPower;
-int32_t index = 0;
+static const PairIndices gPairIndices;
 
-if (ch >= pairedChars[pairedCharExtra]) {
-index = pairedCharExtra;
-}
-
-while (probe > (1 << 0)) {
-probe >>= 1;
 
-if (ch >= pairedChars[index + probe]) {
-index += probe;
-  

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

2015-03-19 Thread Michaël Lefèvre
 filter/source/config/cache/typedetection.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit c83476ab28544c25ab72e7e51131ab481bcbcc5d
Author: Michaël Lefèvre 
Date:   Fri Mar 13 16:34:28 2015 +0100

CppCheck cleaning : unnecessary assignment

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

diff --git a/filter/source/config/cache/typedetection.cxx 
b/filter/source/config/cache/typedetection.cxx
index 15ba4d0..a9d1ebd 100644
--- a/filter/source/config/cache/typedetection.cxx
+++ b/filter/source/config/cache/typedetection.cxx
@@ -705,7 +705,6 @@ bool TypeDetection::impl_getPreselectionForType(
 WildCard aCheck(*pIt);
 if (aCheck.Matches(aParsedURL.Main))
 {
-bBreakDetection= true;
 bMatchByPattern= true;
 break;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Michael Stahl
 stoc/source/inspect/introspection.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c44a8dee64021e15fcd9ff2294176d27d665a5f4
Author: Michael Stahl 
Date:   Fri Mar 13 18:43:21 2015 +0100

stoc: must check *both* bDisposed and bInDispose

Implementation::inspect() was crashing with a null reflection_ in
CppunitTest_libreofficekit_tiledrendering because another thread
was disposing it.

Actually, why are there 2 bools for this and not one enum with 3 values?

Change-Id: Icd27145d3203e1f48a9841ee6251a50cb73f7988
(cherry picked from commit d19249c66d13c779a4a46f34e1d8094c74bbb20f)
Reviewed-on: https://gerrit.libreoffice.org/14860
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/stoc/source/inspect/introspection.cxx 
b/stoc/source/inspect/introspection.cxx
index bd76197..4805aa5 100644
--- a/stoc/source/inspect/introspection.cxx
+++ b/stoc/source/inspect/introspection.cxx
@@ -1599,7 +1599,7 @@ css::uno::Reference 
Implementation::inspect(
 throw (css::uno::RuntimeException, std::exception)
 {
 osl::MutexGuard g(m_aMutex);
-if (rBHelper.bDisposed) {
+if (rBHelper.bDisposed || rBHelper.bInDispose) {
 throw css::lang::DisposedException(
 getImplementationName(), static_cast(this));
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Justin Luth
 sw/source/filter/ww8/ww8par5.cxx |   11 ++-
 1 file changed, 10 insertions(+), 1 deletion(-)

New commits:
commit d66458347f552cbd436d82700626d45eeb1f80fb
Author: Justin Luth 
Date:   Tue Mar 10 13:12:30 2015 +0300

tdf#68963 avoid __refheading__ regression reported in tdf#89482

bugfix tdf#68963 (scheduled for 4.3.7) marked all __refheading__ bookmarks
at TOC, but those crossreferences were deleted if they are not recorded in
aReferencedTOCBookmarks.

I decided to NOT introduce the full fix (which allows LO to author
a .doc with working cross-references) into the stable 4.3 branch to ensure
there are no further regressions.  This patch both avoids a regression and
allows sharing of crossref documents authored in LO 4.4.2+.

Change-Id: I5d8b979bf1b884cbf74b31d875dc7c80a6c4b903
Reviewed-on: https://gerrit.libreoffice.org/14865
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/source/filter/ww8/ww8par5.cxx b/sw/source/filter/ww8/ww8par5.cxx
index e4f9a6f..4b35205 100644
--- a/sw/source/filter/ww8/ww8par5.cxx
+++ b/sw/source/filter/ww8/ww8par5.cxx
@@ -100,7 +100,7 @@ namespace
 // #120879# - helper method to identify a bookmark name to match the 
internal TOC bookmark naming convention
 bool IsTOCBookmarkName( const ::rtl::OUString& rName )
 {
-return rName.startsWith("_Toc") || 
rName.startsWith(IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix());
+return rName.startsWith("_Toc") || 
rName.startsWith(IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix()+"_Toc");
 }
 
 ::rtl::OUString EnsureTOCBookmarkName( const ::rtl::OUString& rName )
@@ -1960,6 +1960,15 @@ eF_ResT SwWW8ImplReader::Read_F_Ref( WW8FieldDesc*, 
OUString& rStr )
 
 OUString sBkmName(GetMappedBookmark(sOrigBkmName));
 
+// #i120879# add cross reference bookmark name prefix, if it
+// matches internal TOC bookmark naming convention
+if ( IsTOCBookmarkName( sBkmName ) )
+{
+sBkmName = EnsureTOCBookmarkName(sBkmName);
+// track  as referenced TOC bookmark.
+pReffedStck->aReferencedTOCBookmarks.insert( sBkmName );
+}
+
 SwGetRefField aFld(
 (SwGetRefFieldType*)rDoc.GetSysFldType( RES_GETREFFLD ),
 sBkmName,REF_BOOKMARK,0,eFormat);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Justin Luth
 dev/null |binary
 sw/source/core/crsr/crossrefbookmark.cxx |2 +-
 sw/source/filter/ww8/ww8par5.cxx |   11 ++-
 3 files changed, 11 insertions(+), 2 deletions(-)

New commits:
commit 92a43d05d14b1ab7b8bfb4052874d0f0e6c62ef9
Author: Justin Luth 
Date:   Mon Mar 9 15:26:14 2015 +0300

tdf#89482 fix __refheading__ regression, set only CrossRefs as TOC.

bugfix tdf#68963 marked all __refheading__ bookmarks at TOC, but
those crossreferences were deleted if they are not recorded in
aReferencedTOCBookmarks.  Redesigned the fix to include _Toc in the
crossreference name that is auto-generated, which is how MSWord
differentiates the markers.

Change-Id: Ic5bdc4b475f2eb7963cbcf3bf0ec74337daef5be
Reviewed-on: https://gerrit.libreoffice.org/14864
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/qa/extras/ww8import/data/fdo68963.doc 
b/sw/qa/extras/ww8import/data/fdo68963.doc
deleted file mode 100644
index 4b331f0..000
Binary files a/sw/qa/extras/ww8import/data/fdo68963.doc and /dev/null differ
diff --git a/sw/source/core/crsr/crossrefbookmark.cxx 
b/sw/source/core/crsr/crossrefbookmark.cxx
index 700a3f2..67a25d0 100644
--- a/sw/source/core/crsr/crossrefbookmark.cxx
+++ b/sw/source/core/crsr/crossrefbookmark.cxx
@@ -72,7 +72,7 @@ namespace sw { namespace mark
 const vcl::KeyCode& rCode,
 const OUString& rName,
 const OUString& rShortName)
-: CrossRefBookmark(rPaM, rCode, rName, rShortName, 
IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix())
+: CrossRefBookmark(rPaM, rCode, rName, rShortName, 
IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix()+"_Toc")
 { }
 
 bool CrossRefHeadingBookmark::IsLegalName(const OUString& rName)
diff --git a/sw/source/filter/ww8/ww8par5.cxx b/sw/source/filter/ww8/ww8par5.cxx
index 246d3f6..bbde7d7 100644
--- a/sw/source/filter/ww8/ww8par5.cxx
+++ b/sw/source/filter/ww8/ww8par5.cxx
@@ -102,7 +102,7 @@ namespace
 // #120879# - helper method to identify a bookmark name to match the 
internal TOC bookmark naming convention
 bool IsTOCBookmarkName( const ::rtl::OUString& rName )
 {
-return rName.startsWith("_Toc") || 
rName.startsWith(IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix());
+return rName.startsWith("_Toc") || 
rName.startsWith(IDocumentMarkAccess::GetCrossRefHeadingBookmarkNamePrefix()+"_Toc");
 }
 
 ::rtl::OUString EnsureTOCBookmarkName( const ::rtl::OUString& rName )
@@ -1970,6 +1970,15 @@ eF_ResT SwWW8ImplReader::Read_F_Ref( WW8FieldDesc*, 
OUString& rStr )
 
 OUString sBkmName(GetMappedBookmark(sOrigBkmName));
 
+// #i120879# add cross reference bookmark name prefix, if it
+// matches internal TOC bookmark naming convention
+if ( IsTOCBookmarkName( sBkmName ) )
+{
+sBkmName = EnsureTOCBookmarkName(sBkmName);
+// track  as referenced TOC bookmark.
+pReffedStck->aReferencedTOCBookmarks.insert( sBkmName );
+}
+
 SwGetRefField aFld(
 (SwGetRefFieldType*)rDoc.getIDocumentFieldsAccess().GetSysFldType( 
RES_GETREFFLD ),
 sBkmName,REF_BOOKMARK,0,eFormat);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Michael Stahl
 editeng/source/misc/acorrcfg.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 32a466e6a95723f26d20a20e74d1f0e563d5dbca
Author: Michael Stahl 
Date:   Tue Mar 10 23:56:55 2015 +0100

tdf#89867: editeng: fix crash on shutdown after changing AutoCorrect options

Since commit 5bff4b016c4b44f4123e0e6a4fd4c0c4dc0cfa2d the
SvxAutoCorrCfg::pAutoCorrect is cleared by one terminate() listener but
then another terminate() listener calls Commit() on all modified
utl::ConfigItem and these two have a Commit() that does not clear the
modified flag so they are always modified.

Sadly there's no non-virtual Commit() wrapper that
calls ClearModified() on the base class...

Change-Id: I9ae220d78bb109c7bf0fdc544754a0686b357115
(cherry picked from commit d1698027e9f1c4a88b17da7357f257be3cfb7c1a)
Reviewed-on: https://gerrit.libreoffice.org/14828
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/editeng/source/misc/acorrcfg.cxx b/editeng/source/misc/acorrcfg.cxx
index dce7df4..9ca989c 100644
--- a/editeng/source/misc/acorrcfg.cxx
+++ b/editeng/source/misc/acorrcfg.cxx
@@ -321,6 +321,7 @@ void SvxBaseAutoCorrCfg::Commit()
 }
 }
 PutProperties(aNames, aValues);
+ClearModified();
 }
 
 void SvxBaseAutoCorrCfg::Notify( const Sequence& /* aPropertyNames 
*/)
@@ -640,6 +641,7 @@ void SvxSwAutoCorrCfg::Commit()
 }
 }
 PutProperties(aNames, aValues);
+ClearModified();
 }
 
 void SvxSwAutoCorrCfg::Notify( const Sequence& /* aPropertyNames */ )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Michael Stahl
 i18npool/source/search/textsearch.cxx |   25 +
 1 file changed, 21 insertions(+), 4 deletions(-)

New commits:
commit afae8d1e9eb37265bd356773caa77d6d8ac481bc
Author: Michael Stahl 
Date:   Tue Mar 10 23:19:18 2015 +0100

i18npool: fix spurious regex ^ matching in TextSearch::searchForward()

Thanks to Eike for finding this:

The anchors ^ and $ now anchor at the selection boundary because the
only text the regex matcher gets passed is the selected text. This in
two paragraphs

 aaa bbb aaa bbb
 aaa bbb aaa bbb

when the selection spans from the second aaa to the third bbb, for
"^aaa" finds the second aaa, where previously it found the third aaa at
the real paragraph start.

This may not be expected by the user, because the behavior of ^ is
described as "Match at the beginning of a line" (or paragraph in our
case), which the previous implementation did.

(regression from 806ced87cfe3da72df0d8e4faf5b82535fc7d1b7)

Unfortunately it's not obvious how to implement the same in
searchBackward().

Change-Id: I07f7a8476b672d9511fa74ca473c32eea427698f
(cherry picked from commit 9aae521b451269007f03527c83645b8b935eb419)
Reviewed-on: https://gerrit.libreoffice.org/14829
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/i18npool/source/search/textsearch.cxx 
b/i18npool/source/search/textsearch.cxx
index 959227df..094b554 100644
--- a/i18npool/source/search/textsearch.cxx
+++ b/i18npool/source/search/textsearch.cxx
@@ -242,13 +242,26 @@ SearchResult TextSearch::searchForward( const OUString& 
searchStr, sal_Int32 sta
 in_str = xTranslit->transliterate( searchStr, startPos, endPos - 
startPos, offset );
 
 // JP 20.6.2001: also the start and end positions must be corrected!
-sal_Int32 const newStartPos =
+sal_Int32 newStartPos =
 (startPos == 0) ? 0 : FindPosInSeq_Impl( offset, startPos );
 
-sal_Int32 const newEndPos = (endPos < searchStr.getLength())
+sal_Int32 newEndPos = (endPos < searchStr.getLength())
 ? FindPosInSeq_Impl( offset, endPos )
 : in_str.getLength();
 
+sal_Int32 nExtraOffset = 0;
+if (pRegexMatcher && startPos > 0)
+{
+// avoid matching ^ here - in_str omits a prefix of the searchStr
+// this is a really lame way to do it, but ICU only offers
+// useAnchoringBounds() to disable *both* bounds but what is needed
+// here is to disable only one bound and respect the other
+in_str = "X" + in_str;
+nExtraOffset = 1;
+newStartPos += nExtraOffset;
+newEndPos += nExtraOffset;
+}
+
 sres = (this->*fnForward)( in_str, newStartPos, newEndPos );
 
 // Map offsets back to untransliterated string.
@@ -260,14 +273,14 @@ SearchResult TextSearch::searchForward( const OUString& 
searchStr, sal_Int32 sta
 const sal_Int32 nGroups = sres.startOffset.getLength();
 for ( sal_Int32 k = 0; k < nGroups; k++ )
 {
-const sal_Int32 nStart = sres.startOffset[k];
+const sal_Int32 nStart = sres.startOffset[k] - nExtraOffset;
 if (startPos > 0 || nStart > 0)
 sres.startOffset[k] = (nStart < nOffsets ? offset[nStart] 
: (offset[nOffsets - 1] + 1));
 // JP 20.6.2001: end is ever exclusive and then don't return
 //   the position of the next character - return 
the
 //   next position behind the last found character!
 //   "a b c" find "b" must return 2,3 and not 
2,4!!!
-const sal_Int32 nStop = sres.endOffset[k];
+const sal_Int32 nStop = sres.endOffset[k] - nExtraOffset;
 if (startPos > 0 || nStop > 0)
 sres.endOffset[k] = offset[(nStop <= nOffsets ? nStop : 
nOffsets) - 1] + 1;
 }
@@ -345,6 +358,10 @@ SearchResult TextSearch::searchBackward( const OUString& 
searchStr, sal_Int32 st
 sal_Int32 const newEndPos =
 (endPos == 0) ? 0 : FindPosInSeq_Impl( offset, endPos );
 
+// TODO: this would need nExtraOffset handling to avoid $ matching
+// if (pRegexMatcher && startPos < searchStr.getLength())
+// but that appears to be impossible with ICU regex
+
 sres = (this->*fnBackward)( in_str, newStartPos, newEndPos );
 
 // Map offsets back to untransliterated string.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Laszlo Kis-Adam
 vcl/source/control/field.cxx |   18 +++---
 1 file changed, 15 insertions(+), 3 deletions(-)

New commits:
commit 591e34f868cc3a3f3ce7927480d224f137bfd83c
Author: Laszlo Kis-Adam 
Date:   Thu Mar 12 17:44:01 2015 +0100

Support "a/b" style of fractions, not just "a b/c" style. Also fixes a 
crash.

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

diff --git a/vcl/source/control/field.cxx b/vcl/source/control/field.cxx
index fc83af0..cab79f5 100644
--- a/vcl/source/control/field.cxx
+++ b/vcl/source/control/field.cxx
@@ -115,9 +115,21 @@ static bool ImplNumericGetValue( const OUString& rStr, 
sal_Int64& rValue,
 {
 bFrac = true;
 nFracNumPos = aStr.lastIndexOf(' ', nFracDivPos);
-aStr1.append(aStr.getStr(), nFracNumPos);
-aStrNum.append(aStr.getStr()+nFracNumPos+1, nFracDivPos-nFracNumPos-1);
-aStrDenom.append(aStr.getStr()+nFracDivPos+1);
+
+// If in "a b/c" format.
+if(nFracNumPos != -1 )
+{
+aStr1.append(aStr.getStr(), nFracNumPos);
+aStrNum.append(aStr.getStr()+nFracNumPos+1, 
nFracDivPos-nFracNumPos-1);
+aStrDenom.append(aStr.getStr()+nFracDivPos+1);
+}
+// "a/b" format, or not a fraction at all
+else
+{
+aStrNum.append(aStr.getStr(), nFracDivPos);
+aStrDenom.append(aStr.getStr()+nFracDivPos+1);
+}
+
 }
 // parse decimal strings
 else if ( nDecPos >= 0)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Adolfo Jayme Barrientos
 svx/source/stbctrls/stbctrls.src |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit d6e839e9e4ce5470c304dbe2b87dd8b0bb07db59
Author: Adolfo Jayme Barrientos 
Date:   Fri Mar 6 05:05:43 2015 -0600

Percentages should also be translatable…

as they are not written the same way in all languages.

Change-Id: Ibc3a5963bf8bd737516a003618befd1de5f3f5ca
Reviewed-on: https://gerrit.libreoffice.org/14767
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/svx/source/stbctrls/stbctrls.src b/svx/source/stbctrls/stbctrls.src
index 3ad69cf..ac26b26 100644
--- a/svx/source/stbctrls/stbctrls.src
+++ b/svx/source/stbctrls/stbctrls.src
@@ -168,31 +168,31 @@ Menu RID_SVXMNU_ZOOM
 {
 Identifier = ZOOM_50 ;
 HelpId = HID_MNU_ZOOM_50 ;
-Text = "50%" ;
+Text [ en-US ] = "50%" ;
 };
 MenuItem
 {
 Identifier = ZOOM_75 ;
 HelpId = HID_MNU_ZOOM_75 ;
-text = "75%" ;
+Text [ en-US ] = "75%" ;
 };
 MenuItem
 {
 Identifier = ZOOM_100 ;
 HelpId = HID_MNU_ZOOM_100 ;
-Text = "100%" ;
+Text [ en-US ] = "100%" ;
 };
 MenuItem
 {
 Identifier = ZOOM_150 ;
 HelpId = HID_MNU_ZOOM_150 ;
-Text = "150%" ;
+Text [ en-US ] = "150%" ;
 };
 MenuItem
 {
 Identifier = ZOOM_200 ;
 HelpId = HID_MNU_ZOOM_200 ;
-Text = "200%" ;
+Text [ en-US ] = "200%" ;
 };
 };
 };
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Caolán McNamara
 cui/source/options/optsave.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 4582eabb00494b0e29a899e78084ebac895c64a8
Author: Caolán McNamara 
Date:   Thu Mar 19 11:13:44 2015 +

push build along a bit further

Change-Id: I6b662e8588153a9437615de0652e3169fc867b82

diff --git a/cui/source/options/optsave.cxx b/cui/source/options/optsave.cxx
index 312aa12..2767ef9 100644
--- a/cui/source/options/optsave.cxx
+++ b/cui/source/options/optsave.cxx
@@ -403,9 +403,9 @@ void SfxSaveTabPage::Reset( const SfxItemSet* )
 sal_IntPtr nData = 
reinterpret_cast(aDocTypeLB->GetEntryData(n));
 OUString sCommand;
 sCommand = "matchByDocumentService=%1:iflags=" +
-   
OUString::number(static_cast(SfxFilterFlags::IMPORT|SfxFilterFlags::EXPORT))
 +
+   
OUString::number(static_cast(SfxFilterFlags::IMPORT|SfxFilterFlags::EXPORT))
 +
":eflags=" +
-   
OUString::number(static_cast(SfxFilterFlags::NOTINFILEDLG)) +
+   
OUString::number(static_cast(SfxFilterFlags::NOTINFILEDLG)) +
":default_first";
 OUString sReplace;
 switch(nData)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Payment from FS for fixing tdf#58723

2015-03-19 Thread Zeki Bildirici
Hi,

I've sponsored an issue on freedomsponsors.org for fixing
https://bugs.documentfoundation.org/show_bug.cgi?id=58723 . (see
https://freedomsponsors.org/issue/429/editing-partial-acceptance-or-rejection-of-the-changes)

Now i see tht the bug is fixed but there is no notification on the
report about the comit and i don't know who fixed it.

Not much, it is only USD.20 but i want to pay it -just consider having
some beers or coffee from me for this great enchancemet :)

If you know who fixed it, or you are the one please contact with me,
otherwise i'll donate the reward to TDF.

Thanks again.

Best regards,
Zeki
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: Branch 'feature/vclref' - chart2/source editeng/source include/sfx2 include/svx sc/inc sc/source sd/inc sd/source starmath/inc starmath/source sw/source

2015-03-19 Thread Michael Meeks
 chart2/source/inc/chartview/DrawModelWrapper.hxx |2 
 chart2/source/view/main/DrawModelWrapper.cxx |4 
 editeng/source/editeng/impedit.hxx   |   14 --
 editeng/source/editeng/impedit2.cxx  |6 -
 include/sfx2/objsh.hxx   |2 
 include/svx/svdmodel.hxx |4 
 sc/inc/document.hxx  |4 
 sc/source/core/data/documen2.cxx |4 
 sc/source/core/data/documen8.cxx |6 -
 sc/source/ui/dbgui/csvgrid.cxx   |  136 +++
 sc/source/ui/docshell/sizedev.cxx|2 
 sc/source/ui/inc/csvgrid.hxx |4 
 sc/source/ui/inc/output.hxx  |6 -
 sc/source/ui/inc/sizedev.hxx |   12 +-
 sd/inc/sdmod.hxx |2 
 sd/source/ui/app/sdmod.cxx   |2 
 sd/source/ui/docshell/docshel4.cxx   |8 -
 sd/source/ui/docshell/docshell.cxx   |2 
 sd/source/ui/inc/DrawDocShell.hxx|2 
 starmath/inc/document.hxx|   12 +-
 starmath/source/document.cxx |6 -
 sw/source/core/doc/DocumentDeviceManager.cxx |   22 +--
 sw/source/core/inc/DocumentDeviceManager.hxx |6 -
 23 files changed, 134 insertions(+), 134 deletions(-)

New commits:
commit 3eb193aed1e3fcf3ab7cc0c687960aca7a26a50f
Author: Michael Meeks 
Date:   Wed Mar 18 20:52:22 2015 +

start of unwinding reference OutputDevice & Printer lifecycle.

Change-Id: I066337f46a3c39d1eaba5e7a1f4e1278816a0b27

diff --git a/chart2/source/inc/chartview/DrawModelWrapper.hxx 
b/chart2/source/inc/chartview/DrawModelWrapper.hxx
index 986a6b9..0e590a6 100644
--- a/chart2/source/inc/chartview/DrawModelWrapper.hxx
+++ b/chart2/source/inc/chartview/DrawModelWrapper.hxx
@@ -41,7 +41,7 @@ private:
 ::com::sun::star::uno::Reference< ::com::sun::star::drawing::XDrawPage > 
m_xMainDrawPage;
 ::com::sun::star::uno::Reference< ::com::sun::star::drawing::XDrawPage > 
m_xHiddenDrawPage;
 
-boost::scoped_ptr m_pRefDevice;
+VclPtr m_pRefDevice;
 
 //no default constructor
 DrawModelWrapper();
diff --git a/chart2/source/view/main/DrawModelWrapper.cxx 
b/chart2/source/view/main/DrawModelWrapper.cxx
index d917ee1..c65af68 100644
--- a/chart2/source/view/main/DrawModelWrapper.cxx
+++ b/chart2/source/view/main/DrawModelWrapper.cxx
@@ -150,7 +150,8 @@ DrawModelWrapper::DrawModelWrapper( const 
uno::Reference
 OutputDevice* pDefaultDevice = rOutliner.GetRefDevice();
 if( !pDefaultDevice )
 pDefaultDevice = Application::GetDefaultDevice();
-m_pRefDevice.reset(new VirtualDevice(*pDefaultDevice));
+m_pRefDevice.disposeAndClear();
+m_pRefDevice = new VirtualDevice(*pDefaultDevice);
 MapMode aMapMode = m_pRefDevice->GetMapMode();
 aMapMode.SetMapUnit(MAP_100TH_MM);
 m_pRefDevice->SetMapMode(aMapMode);
@@ -176,6 +177,7 @@ DrawModelWrapper::~DrawModelWrapper()
 }
 SfxItemPool::Free(m_pChartItemPool);
 }
+m_pRefDevice.disposeAndClear();
 }
 
 uno::Reference< uno::XInterface > DrawModelWrapper
diff --git a/editeng/source/editeng/impedit.hxx 
b/editeng/source/editeng/impedit.hxx
index 9d644cf..e193dbd 100644
--- a/editeng/source/editeng/impedit.hxx
+++ b/editeng/source/editeng/impedit.hxx
@@ -408,8 +408,8 @@ private:
 SfxStyleSheetPool*  pStylePool;
 SfxItemPool*pTextObjectPool;
 
-VirtualDevice*  pVirtDev;
-OutputDevice*   pRefDev;
+VclPtr< VirtualDevice> pVirtDev;
+VclPtr< OutputDevice > pRefDev;
 
 svtools::ColorConfig*   pColorConfig;
 mutable SvtCTLOptions*  pCTLOptions;
@@ -651,7 +651,7 @@ private:
 boolIsForceAutoColor() const { return bForceAutoColor; }
 
 inline VirtualDevice*   GetVirtualDevice( const MapMode& rMapMode, 
sal_uLong nDrawMode );
-inline void EraseVirtualDevice();
+inline void EraseVirtualDevice() { pVirtDev.disposeAndClear(); 
}
 
 DECL_LINK(StatusTimerHdl, void *);
 DECL_LINK(IdleFormatHdl, void *);
@@ -862,7 +862,7 @@ public:
 voidUpdateParagraphsWithStyleSheet( SfxStyleSheet* pStyle 
);
 voidRemoveStyleFromParagraphs( SfxStyleSheet* pStyle );
 
-OutputDevice*   GetRefDevice() const { return pRefDev; }
+OutputDevice*   GetRefDevice() const { return pRefDev.get(); }
 voidSetRefDevice( OutputDevice* pRefDef );
 
 const MapMode&  GetRefMapMode() { return pRefDev->GetMapMode(); }
@@ -1072,12 +1072,6 @@ inline VirtualDevice* ImpEditEngine::GetVirtualDevice( 
const MapMode& rMapMode,
 return pVirtDev;
 }
 
-inline void ImpEditEngine::EraseVirtualDevice()
-{
-delete pVirtDev;
-pVirtDev = 0;
-}
-
 inline void ImpEditEngine::IdleFormatAndUpdate( EditView* pCurView )
 {
 aIdleFormatter.DoI

[Libreoffice-commits] core.git: Branch 'feature/vclref' - 3 commits - canvas/source compilerplugins/clang cppcanvas/source desktop/source drawinglayer/source editeng/source include/editeng include/svt

2015-03-19 Thread Noel Grandin
 canvas/source/vcl/bitmapbackbuffer.hxx   |2 
 canvas/source/vcl/impltools.hxx  |2 
 compilerplugins/clang/vclwidgets.cxx |  112 +++---
 cppcanvas/source/mtfrenderer/implrenderer.cxx|   12 
 cppcanvas/source/mtfrenderer/transparencygroupaction.cxx |   14 
 desktop/source/splash/splash.cxx |4 
 drawinglayer/source/primitive2d/controlprimitive2d.cxx   |8 
 drawinglayer/source/primitive2d/graphicprimitivehelper2d.cxx |   48 +-
 drawinglayer/source/primitive2d/textlayoutdevice.cxx |4 
 drawinglayer/source/processor2d/vclhelperbufferdevice.cxx|8 
 drawinglayer/source/processor2d/vclhelperbufferdevice.hxx|6 
 drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx   |   40 +-
 drawinglayer/source/processor2d/vclprocessor2d.hxx   |3 
 drawinglayer/source/tools/converters.cxx |   24 -
 editeng/source/editeng/editeng.cxx   |4 
 editeng/source/editeng/impedit.hxx   |2 
 editeng/source/items/svxfont.cxx |2 
 include/editeng/outliner.hxx |3 
 include/svtools/ctrlbox.hxx  |2 
 include/svtools/ctrltool.hxx |4 
 include/svtools/prnsetup.hxx |6 
 include/svtools/ruler.hxx|2 
 include/svtools/valueset.hxx |2 
 include/toolkit/awt/vclxgraphics.hxx |3 
 include/vcl/animate.hxx  |9 
 include/vcl/decoview.hxx |3 
 include/vcl/event.hxx|4 
 include/vcl/gdimtf.hxx   |3 
 include/vcl/outdev.hxx   |6 
 include/vcl/print.hxx|9 
 include/vcl/texteng.hxx  |3 
 include/vcl/vclptr.hxx   |   48 ++
 include/vcl/virdev.hxx   |4 
 include/vcl/window.hxx   |2 
 svtools/source/contnr/imivctl.hxx|8 
 svtools/source/contnr/imivctl1.cxx   |   16 
 svtools/source/control/ctrlbox.cxx   |   29 -
 svtools/source/control/ctrltool.cxx  |2 
 svtools/source/control/ruler.cxx |   98 ++---
 svtools/source/control/valueset.cxx  |   74 ++--
 svtools/source/dialogs/prnsetup.cxx  |9 
 svtools/source/graphic/grfmgr2.cxx   |   16 
 svtools/source/graphic/provider.cxx  |   16 
 svtools/source/inc/renderer.hxx  |2 
 svtools/source/misc/sampletext.cxx   |   14 
 svtools/source/table/gridtablerenderer.cxx   |8 
 svtools/source/toolpanel/paneltabbar.cxx |   24 -
 svtools/source/toolpanel/toolpaneldrawer.hxx |2 
 vcl/inc/brdwin.hxx   |8 
 vcl/inc/controldata.hxx  |2 
 vcl/inc/outdev.h |3 
 vcl/inc/printdlg.hxx |2 
 vcl/inc/svdata.hxx   |   20 -
 vcl/inc/window.h |2 
 vcl/source/app/svdata.cxx|2 
 vcl/source/edit/texteng.cxx  |2 
 vcl/source/edit/textview.cxx |7 
 vcl/source/filter/graphicfilter.cxx  |   16 
 vcl/source/filter/sgfbram.cxx|   14 
 vcl/source/filter/sgvmain.cxx|4 
 vcl/source/filter/sgvtext.cxx|   16 
 vcl/source/filter/wmf/emfwr.cxx  |  114 +++---
 vcl/source/filter/wmf/emfwr.hxx  |2 
 vcl/source/filter/wmf/wmfwr.cxx  |2 
 vcl/source/filter/wmf/wmfwr.hxx  |2 
 vcl/source/gdi/animate.cxx   |6 
 vcl/source/gdi/bitmapex.cxx  |   14 
 vcl/source/gdi/cvtsvm.cxx|   26 -
 vcl/source/gdi/gdimetafiletools.cxx  |   20 -
 vcl/source/gdi/gdimtf.cxx|  180 +--
 vcl/source/gdi/impanmvw.cxx   

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

2015-03-19 Thread Tor Lillqvist
 include/LibreOfficeKit/LibreOfficeKit.h |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 2a9bd88e427c0646566ca988bb23e7dab25744c1
Author: Tor Lillqvist 
Date:   Thu Mar 19 13:58:12 2015 +0200

Indent parameter lists consistently.

Other inconsistencies in spacing and alignment remain, though.

Change-Id: I2b4f9e34781288c5146518be853d3831ec32e968

diff --git a/include/LibreOfficeKit/LibreOfficeKit.h 
b/include/LibreOfficeKit/LibreOfficeKit.h
index f10e5e3..85f760c 100644
--- a/include/LibreOfficeKit/LibreOfficeKit.h
+++ b/include/LibreOfficeKit/LibreOfficeKit.h
@@ -136,14 +136,14 @@ struct _LibreOfficeKitDocumentClass
 
   /// @see lok::Document::setTextSelection
   void (*setTextSelection)(LibreOfficeKitDocument* pThis,
- int nType,
- int nX,
- int nY);
+   int nType,
+   int nX,
+   int nY);
   /// @see lok::Document::setGraphicSelection
   void (*setGraphicSelection)(LibreOfficeKitDocument* pThis,
- int nType,
- int nX,
- int nY);
+  int nType,
+  int nX,
+  int nY);
   /// @see lok::Document::resetSelection
   void (*resetSelection)(LibreOfficeKitDocument* pThis);
 #endif // LOK_USE_UNSTABLE_API
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - sd/source

2015-03-19 Thread Michael Meeks
 sd/source/ui/docshell/docshel4.cxx |6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit 94ba923808ff8ed6fe893b0a85d27b41eccec3f8
Author: Michael Meeks 
Date:   Thu Mar 19 12:12:46 2015 +

compile fixes.

Change-Id: I210c71c1a7236d286256a498f3f4f28f4185d052

diff --git a/sd/source/ui/docshell/docshel4.cxx 
b/sd/source/ui/docshell/docshel4.cxx
index 951b8d5..58c2bda 100644
--- a/sd/source/ui/docshell/docshel4.cxx
+++ b/sd/source/ui/docshell/docshel4.cxx
@@ -145,10 +145,8 @@ void DrawDocShell::SetPrinter(SfxPrinter *pNewPrinter)
 pView->SdrEndTextEdit();
 }
 
-if ( mpPrinter && mbOwnPrinter && (mpPrinter != pNewPrinter) )
-{
+if ( mpPrinter && mbOwnPrinter && (mpPrinter.get() != pNewPrinter) )
 mpPrinter.disposeAndClear();
-}
 
 mpPrinter = pNewPrinter;
 mbOwnPrinter = true;
@@ -222,7 +220,7 @@ void DrawDocShell::UpdateRefDevice()
 // as a fall-back.
 DBG_ASSERT(false, "DrawDocShell::UpdateRefDevice(): Unexpected 
printer layout mode");
 
-pRefDevice = mpPrinter;
+pRefDevice = mpPrinter.get();
 break;
 }
 mpDoc->SetRefDevice( pRefDevice.get() );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Noel Grandin
 sc/qa/unit/helper/qahelper.hxx |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 98c532b38589945717ff17ed344a358479074e5b
Author: Noel Grandin 
Date:   Thu Mar 19 14:34:57 2015 +0200

convert opaque numeric value to underlying symbolic constants

Change-Id: Id2f74c5104873e78b188c9a2b8c779c8ada3bacc

diff --git a/sc/qa/unit/helper/qahelper.hxx b/sc/qa/unit/helper/qahelper.hxx
index 3d3e0da..e89745e 100644
--- a/sc/qa/unit/helper/qahelper.hxx
+++ b/sc/qa/unit/helper/qahelper.hxx
@@ -40,15 +40,15 @@
 #define SCQAHELPER_DLLPUBLIC  SAL_DLLPUBLIC_IMPORT
 #endif
 
-#define ODS_FORMAT_TYPE  static_cast(50331943)
-#define XLS_FORMAT_TYPE  static_cast(318767171)
-#define XLSX_FORMAT_TYPE static_cast(268959811)
-#define LOTUS123_FORMAT_TYPE static_cast(268435649)
+#define ODS_FORMAT_TYPE  (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::TEMPLATE | SfxFilterFlags::OWN | SfxFilterFlags::DEFAULT | 
SfxFilterFlags::ENCRYPTION | SfxFilterFlags::PASSWORDTOMODIFY)
+#define XLS_FORMAT_TYPE  (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN | SfxFilterFlags::ENCRYPTION | 
SfxFilterFlags::PASSWORDTOMODIFY | SfxFilterFlags::PREFERED)
+#define XLSX_FORMAT_TYPE (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN | SfxFilterFlags::STARONEFILTER | 
SfxFilterFlags::PREFERED)
+#define LOTUS123_FORMAT_TYPE (SfxFilterFlags::IMPORT | 
 SfxFilterFlags::ALIEN | SfxFilterFlags::USESOPTIONS | SfxFilterFlags::PREFERED)
 #define CSV_FORMAT_TYPE  (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN | SfxFilterFlags::USESOPTIONS)
 #define HTML_FORMAT_TYPE (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN | SfxFilterFlags::USESOPTIONS)
 #define DIF_FORMAT_TYPE  (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN | SfxFilterFlags::USESOPTIONS)
 #define XLS_XML_FORMAT_TYPE  (SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT 
| SfxFilterFlags::ALIEN)
-#define XLSB_XML_FORMAT_TYPE static_cast(268959809)
+#define XLSB_XML_FORMAT_TYPE (SfxFilterFlags::IMPORT | 
 SfxFilterFlags::ALIEN | SfxFilterFlags::STARONEFILTER | 
SfxFilterFlags::PREFERED)
 
 #define ODS  0
 #define XLS  1
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Julien Nabet
 xmloff/source/style/undlihdl.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 61a3ace9f42a37640b1ec18fad776ba7c1f61714
Author: Julien Nabet 
Date:   Sun Mar 8 21:32:39 2015 +0100

Related tdf#50133: Wrong underlining in Reports

By running some tests, I noticed double wave didn't appear when opening 
report in non edit mode.
First you must know that to indicate "double wave", xml contains these 2 
information:
style:text-underline-style="wave" style:text-underline-type="double"

The problem is the reading order of the parser doesn't seem the same each 
time.
So if it reads style then type, it's ok thanks to this part:
132 case awt::FontUnderline::DOUBLE:
133 // A double line style has priority over a bold 
line style,
134 // but not over the line style itself.
135 switch( eUnderline )
136 {
137 case awt::FontUnderline::SINGLE:
138 case awt::FontUnderline::BOLD:
139 break;
140 case awt::FontUnderline::WAVE:
141 case awt::FontUnderline::BOLDWAVE:
142 eNewUnderline = awt::FontUnderline::DOUBLEWAVE;
143 break;

but if the parser read type in the first place then style, the case 
FontUnderline "DOUBLEWAVE" wasn't managed

Change-Id: Ie090507898e537827eebaae157dfac248baa43f1
Reviewed-on: https://gerrit.libreoffice.org/14801
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/xmloff/source/style/undlihdl.cxx b/xmloff/source/style/undlihdl.cxx
index 268d042..2832d11 100644
--- a/xmloff/source/style/undlihdl.cxx
+++ b/xmloff/source/style/undlihdl.cxx
@@ -233,7 +233,9 @@ bool XMLUnderlineStylePropHdl::importXML( const OUString& 
rStrImpValue, uno::Any
 eNewUnderline = awt::FontUnderline::BOLDDASHDOTDOT;
 break;
 case awt::FontUnderline::WAVE:
-if( awt::FontUnderline::BOLD == eUnderline )
+if( awt::FontUnderline::DOUBLE == eUnderline )
+eNewUnderline = awt::FontUnderline::DOUBLEWAVE;
+else if( awt::FontUnderline::BOLD == eUnderline )
 eNewUnderline = awt::FontUnderline::BOLDWAVE;
 break;
 case awt::FontUnderline::SMALLWAVE:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - vcl/source

2015-03-19 Thread Michael Meeks
 vcl/source/window/window.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 1f0745af1be1edd771b2582b40127a8f150ff82d
Author: Michael Meeks 
Date:   Thu Mar 19 13:12:53 2015 +

remove erroneous new disposeAndClear.

Change-Id: Ib593f05447a8a657649ef83febb4ef0e5b6476dd

diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
index 48414cb..d1d2acb 100644
--- a/vcl/source/window/window.cxx
+++ b/vcl/source/window/window.cxx
@@ -497,7 +497,6 @@ void Window::dispose()
 // release SalGraphics
 OutputDevice *pOutDev = GetOutDev();
 pOutDev->ReleaseGraphics();
-mpOutputDevice.disposeAndClear();
 
 // notify ImplDelData subscribers of this window about the window deletion
 ImplDelData* pDelData = mpWindowImpl->mpFirstDel;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Yousuf Philips
 officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu |   13 
+++--
 sd/uiconfig/sdraw/toolbar/drawingobjectbar.xml   |   24 
+-
 sd/uiconfig/sdraw/toolbar/graphicobjectbar.xml   |   18 
+++
 sd/uiconfig/sdraw/toolbar/standardbar.xml|   22 
+++--
 sd/uiconfig/sdraw/toolbar/toolbar.xml|   18 
---
 5 files changed, 57 insertions(+), 38 deletions(-)

New commits:
commit c5e012fc65403e960d7f7ab5a3da50f6a2d1c53a
Author: Yousuf Philips 
Date:   Thu Mar 19 16:08:48 2015 +0400

tdf#90090 - rearrange Draw's UI by moving around its toolbars

Change-Id: I5f76e976ea457cceaaf22c8e2328c944a87d03b5
Reviewed-on: https://gerrit.libreoffice.org/14904
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
index 39d82c9..3b43ac2 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
@@ -45,7 +45,7 @@
   
   
 
-  0,1
+  1,0
 
 
   true
@@ -122,7 +122,7 @@
   
   
 
-  0,1
+  1,0
 
 
   true
@@ -145,11 +145,14 @@
   true
 
 
-  1
+  2
 
 
   true
 
+
+  true
+
 
   Drawing
 
@@ -497,13 +500,13 @@
   
   
 
-  0,1
+  1,0
 
 
   true
 
 
-  true
+  false
 
 
   Text Formatting
diff --git a/sd/uiconfig/sdraw/toolbar/drawingobjectbar.xml 
b/sd/uiconfig/sdraw/toolbar/drawingobjectbar.xml
index ea0b79c..1d6b675 100644
--- a/sd/uiconfig/sdraw/toolbar/drawingobjectbar.xml
+++ b/sd/uiconfig/sdraw/toolbar/drawingobjectbar.xml
@@ -20,26 +20,26 @@
 http://openoffice.org/2001/toolbar"; 
xmlns:xlink="http://www.w3.org/1999/xlink";>
  
  
- 
- 
+ 
+ 
  
- 
+ 
  
- 
+ 
  
  
  
- 
+ 
  
  
  
- 
- 
+ 
+ 
  
- 
+ 
  
- 
- 
- 
- 
+ 
+ 
+ 
+ 
 
diff --git a/sd/uiconfig/sdraw/toolbar/graphicobjectbar.xml 
b/sd/uiconfig/sdraw/toolbar/graphicobjectbar.xml
index 90abc87..3af5bc9 100644
--- a/sd/uiconfig/sdraw/toolbar/graphicobjectbar.xml
+++ b/sd/uiconfig/sdraw/toolbar/graphicobjectbar.xml
@@ -20,21 +20,21 @@
 http://openoffice.org/2001/toolbar"; 
xmlns:xlink="http://www.w3.org/1999/xlink"; toolbar:id="toolbar">
  
  
- 
+ 
  
- 
+ 
  
- 
+ 
  
- 
- 
- 
+ 
+ 
+ 
  
  
  
- 
+ 
  
  
- 
- 
+ 
+ 
 
diff --git a/sd/uiconfig/sdraw/toolbar/standardbar.xml 
b/sd/uiconfig/sdraw/toolbar/standardbar.xml
index 601c7f9..ef32191 100644
--- a/sd/uiconfig/sdraw/toolbar/standardbar.xml
+++ b/sd/uiconfig/sdraw/toolbar/standardbar.xml
@@ -43,18 +43,30 @@
  
  
  
- 
+ 
  
  
- 
- 
- 
- 
  
  
  
+ 
+ 
+ 
+ 
+ 
+ 
  
  
  
  
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
+ 
 
diff --git a/sd/uiconfig/sdraw/toolbar/toolbar.xml 
b/sd/uiconfig/sdraw/toolbar/toolbar.xml
index 8340dcc..4f2207e 100644
--- a/sd/uiconfig/sdraw/toolbar/toolbar.xml
+++ b/sd/uiconfig/sdraw/toolbar/toolbar.xml
@@ -19,18 +19,24 @@
 -->
 http://openoffice.org/2001/toolbar"; 
xmlns:xlink="http://www.w3.org/1999/xlink"; toolbar:id="toolbar">
  
+ 
+ 
+ 
+ 
  
  
- 
+ 
  
  
- 
- 
  
- 
- 
+ 
+ 
+ 
  
+ 
+ 
  
+ 
  
  
  
@@ -45,8 +51,6 @@
  
  
  
- 
- 
  
  
  
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - 2 commits - canvas/source vcl/source

2015-03-19 Thread Michael Meeks
 canvas/source/vcl/backbuffer.cxx |   10 --
 canvas/source/vcl/backbuffer.hxx |3 ++-
 canvas/source/vcl/spritecanvashelper.cxx |6 ++
 canvas/source/vcl/spritecanvashelper.hxx |4 +++-
 vcl/source/outdev/outdevstate.cxx|1 +
 vcl/source/outdev/text.cxx   |8 
 6 files changed, 24 insertions(+), 8 deletions(-)

New commits:
commit e821e10c7c605845968604831eb0ffbc30c5682c
Author: Michael Meeks 
Date:   Thu Mar 19 13:14:32 2015 +

remove un-necessary type punning.

Change-Id: I4f05929daa8b78b309d8a0498a2bb3246af9e18a

diff --git a/vcl/source/outdev/outdevstate.cxx 
b/vcl/source/outdev/outdevstate.cxx
index a71c7d6..bc6bd4a 100644
--- a/vcl/source/outdev/outdevstate.cxx
+++ b/vcl/source/outdev/outdevstate.cxx
@@ -641,6 +641,7 @@ void OutputDevice::InitFillColor()
 void OutputDevice::ImplReleaseFonts()
 {
 mpGraphics->ReleaseFonts();
+
 mbNewFont = true;
 mbInitFont = true;
 
diff --git a/vcl/source/outdev/text.cxx b/vcl/source/outdev/text.cxx
index 7a2973c..1409a5a 100644
--- a/vcl/source/outdev/text.cxx
+++ b/vcl/source/outdev/text.cxx
@@ -2706,8 +2706,8 @@ bool OutputDevice::GetTextOutlines( 
::basegfx::B2DPolyPolygonVector& rVector,
 if (pSalLayout == 0)
 return false;
 long nWidth = pSalLayout->GetTextWidth();
-long nHeight = ((OutputDevice*)&aVDev)->mpFontEntry->mnLineHeight + 
((OutputDevice*)&aVDev)->mnEmphasisAscent
-+ ((OutputDevice*)&aVDev)->mnEmphasisDescent;
+long nHeight = aVDev->mpFontEntry->mnLineHeight + aVDev->mnEmphasisAscent +
+   aVDev->mnEmphasisDescent;
 pSalLayout->Release();
 
 if( !nWidth || !nHeight )
@@ -2756,8 +2756,8 @@ bool OutputDevice::GetTextOutlines( 
::basegfx::B2DPolyPolygonVector& rVector,
 // draw glyph into virtual device
 aVDev->Erase();
 pSalLayout->DrawBase() += aOffset;
-pSalLayout->DrawBase() += Point( 
((OutputDevice*)&aVDev)->mnTextOffX, ((OutputDevice*)&aVDev)->mnTextOffY );
-pSalLayout->DrawText( *((OutputDevice*)&aVDev)->mpGraphics );
+pSalLayout->DrawBase() += Point( aVDev->mnTextOffX, 
aVDev->mnTextOffY );
+pSalLayout->DrawText( *aVDev->mpGraphics );
 pSalLayout->Release();
 
 // convert character image into outline
commit bc7ea2108ec1cdde1b02d7cd972d7127658d5a24
Author: Michael Meeks 
Date:   Thu Mar 19 13:13:47 2015 +

Unwind VCLObject bits for OutputDevice derivatives.

Change-Id: I1abdf0a6d43029fb29e5cbc6c3f788aa98f55a1f

diff --git a/canvas/source/vcl/backbuffer.cxx b/canvas/source/vcl/backbuffer.cxx
index 73d6320..216b5eb 100644
--- a/canvas/source/vcl/backbuffer.cxx
+++ b/canvas/source/vcl/backbuffer.cxx
@@ -43,14 +43,20 @@ namespace vclcanvas
 }
 }
 
+BackBuffer::~BackBuffer()
+{
+SolarMutexGuard aGuard;
+maVDev.disposeAndClear();
+}
+
 OutputDevice& BackBuffer::getOutDev()
 {
-return maVDev.get();
+return *maVDev.get();
 }
 
 const OutputDevice& BackBuffer::getOutDev() const
 {
-return maVDev.get();
+return *maVDev.get();
 }
 
 void BackBuffer::setSize( const ::Size& rNewSize )
diff --git a/canvas/source/vcl/backbuffer.hxx b/canvas/source/vcl/backbuffer.hxx
index 0fea559..479f4dd 100644
--- a/canvas/source/vcl/backbuffer.hxx
+++ b/canvas/source/vcl/backbuffer.hxx
@@ -43,6 +43,7 @@ namespace vclcanvas
  */
 BackBuffer( const OutputDevice& rRefDevice,
 boolbMonochromeBuffer=false );
+~BackBuffer();
 
 virtual OutputDevice&   getOutDev() SAL_OVERRIDE;
 virtual const OutputDevice& getOutDev() const SAL_OVERRIDE;
@@ -50,7 +51,7 @@ namespace vclcanvas
 void setSize( const ::Size& rNewSize );
 
 private:
-::canvas::vcltools::VCLObjectmaVDev;
+VclPtr< VirtualDevice > maVDev;
 };
 
 typedef ::boost::shared_ptr< BackBuffer > BackBufferSharedPtr;
diff --git a/canvas/source/vcl/spritecanvashelper.cxx 
b/canvas/source/vcl/spritecanvashelper.cxx
index 037bc48..9164d86 100644
--- a/canvas/source/vcl/spritecanvashelper.cxx
+++ b/canvas/source/vcl/spritecanvashelper.cxx
@@ -189,6 +189,12 @@ namespace vclcanvas
 #endif
 }
 
+SpriteCanvasHelper::~SpriteCanvasHelper()
+{
+SolarMutexGuard aGuard;
+maVDev.disposeAndClear();
+}
+
 void SpriteCanvasHelper::init( const OutDevProviderSharedPtr& rOutDev,
SpriteCanvas&  
rOwningSpriteCanvas,
::canvas::SpriteRedrawManager& rManager,
diff --git a/canvas/source/vcl/spritecanvashelper.hxx 
b/canvas/source/vcl/spritecanvashelper.hxx
index dc165d4..0396217 100644
--- a/canvas/source/vcl/spritecanvashelper.hxx
+++ b/canvas/source/vcl/spritecanvashelper.hxx
@@ -23,6 +23,7 @@
 #include 
 #include 
 
+#include 
 #include 
 
 #

[Libreoffice-commits] core.git: canvas/source include/vcl vcl/generic vcl/headless vcl/inc vcl/quartz vcl/source vcl/unx vcl/win

2015-03-19 Thread Caolán McNamara
 canvas/source/cairo/cairo_cairo.hxx|5 +++--
 canvas/source/cairo/cairo_devicehelper.cxx |   15 +++
 canvas/source/cairo/cairo_quartz_cairo.cxx |   15 ---
 canvas/source/cairo/cairo_quartz_cairo.hxx |2 --
 canvas/source/cairo/cairo_win32_cairo.cxx  |   14 --
 canvas/source/cairo/cairo_win32_cairo.hxx  |2 --
 canvas/source/cairo/cairo_xlib_cairo.cxx   |5 +++--
 canvas/source/cairo/cairo_xlib_cairo.hxx   |2 +-
 include/vcl/outdev.hxx |1 -
 vcl/generic/print/genpspgraphics.cxx   |5 -
 vcl/headless/svpgdi.cxx|5 -
 vcl/inc/generic/genpspgraphics.h   |1 -
 vcl/inc/headless/svpgdi.hxx|1 -
 vcl/inc/quartz/salgdi.h|1 -
 vcl/inc/salgdi.hxx |1 -
 vcl/inc/unx/salgdi.h   |1 -
 vcl/inc/win/salgdi.h   |1 -
 vcl/quartz/salgdicommon.cxx|5 -
 vcl/source/outdev/outdev.cxx   |   11 ---
 vcl/unx/generic/gdi/salgdi.cxx |5 -
 vcl/win/source/gdi/salgdi.cxx  |5 -
 21 files changed, 14 insertions(+), 89 deletions(-)

New commits:
commit 6c806749421caf9267325cd86372c488f585ab5a
Author: Caolán McNamara 
Date:   Thu Mar 19 10:13:57 2015 +

reorganize resizing surfaces logic

the only case where we don't just create a new surface is the X11 case

Change-Id: I98a3c8f227e208ac9db969ed7711e293fb237c2e
Reviewed-on: https://gerrit.libreoffice.org/14902
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/canvas/source/cairo/cairo_cairo.hxx 
b/canvas/source/cairo/cairo_cairo.hxx
index ab73a56..e256610 100644
--- a/canvas/source/cairo/cairo_cairo.hxx
+++ b/canvas/source/cairo/cairo_cairo.hxx
@@ -63,8 +63,9 @@ namespace cairo {
 /// factory for VirDev on this surface
 virtual boost::shared_ptr createVirtualDevice() const = 
0;
 
-/// Resize the surface (possibly destroying content)
-virtual void Resize( int width, int height ) = 0;
+/// Resize the surface (possibly destroying content), only possible 
for X11 typically
+/// so on failure create a new surface instead
+virtual bool Resize( int /*width*/, int /*height*/ ) { return false; }
 
 /// Flush all pending output to surface
 virtual void flush() const = 0;
diff --git a/canvas/source/cairo/cairo_devicehelper.cxx 
b/canvas/source/cairo/cairo_devicehelper.cxx
index 920481e..c5296b7 100644
--- a/canvas/source/cairo/cairo_devicehelper.cxx
+++ b/canvas/source/cairo/cairo_devicehelper.cxx
@@ -90,15 +90,14 @@ namespace cairocanvas
 if( !mpRefDevice )
 return; // disposed
 
-OutputDevice* pOutDev=getOutputDevice();
+OutputDevice* pOutDev = getOutputDevice();
 
-if (mpSurface && pOutDev->CanResizeCairoSurface())
-{
-// X11 only
-mpSurface->Resize( rSize.getX() + pOutDev->GetOutOffXPixel(),
-   rSize.getY() + pOutDev->GetOutOffYPixel() );
-}
-else
+// X11 only
+bool bReuseSurface = mpSurface &&
+ mpSurface->Resize(rSize.getX() + 
pOutDev->GetOutOffXPixel(),
+   rSize.getY() + 
pOutDev->GetOutOffYPixel());
+
+if (!bReuseSurface)
 {
 mpSurface = cairo::createSurface(
 *pOutDev,
diff --git a/canvas/source/cairo/cairo_quartz_cairo.cxx 
b/canvas/source/cairo/cairo_quartz_cairo.cxx
index f4456aa..8da24f3 100644
--- a/canvas/source/cairo/cairo_quartz_cairo.cxx
+++ b/canvas/source/cairo/cairo_quartz_cairo.cxx
@@ -163,21 +163,6 @@ namespace cairo
 }
 
 /**
- * QuartzSurface::Resize:  Resizes the Canvas surface.
- * @param width new width of the surface
- * @param height new height of the surface
- *
- * Only used on X11.
- *
- * @return The new surface or NULL
- **/
-void QuartzSurface::Resize( int /* width */, int /* height */ )
-{
-OSL_FAIL("not supposed to be called!");
-}
-
-
-/**
  * QuartzSurface::flush:  Draw the data to screen
  **/
 void QuartzSurface::flush() const
diff --git a/canvas/source/cairo/cairo_quartz_cairo.hxx 
b/canvas/source/cairo/cairo_quartz_cairo.hxx
index cbcbf27..a3f94d2 100644
--- a/canvas/source/cairo/cairo_quartz_cairo.hxx
+++ b/canvas/source/cairo/cairo_quartz_cairo.hxx
@@ -57,8 +57,6 @@ namespace cairo {
 
 virtual boost::shared_ptr createVirtualDevice() const;
 
-virtual void Resize( int width, int height );
-
 virtual void flush() const;
 
 int getDepth() const;
diff --git a/canvas/source/cairo/cairo_win32_cairo.cxx 
b/canvas/source/cairo/cairo_win32_cairo.cxx
index 20efd33..a54c7e1 100644
--- a/canvas/source/cairo/cairo_win32_cairo.cxx
+++ b/canvas/source/c

4.4.3.0.0+ build fails (unit tests)

2015-03-19 Thread jbfaure

Hi,
it seems that the following commit breaks the build of LibreOffice 
4.4.3.0.0+ :


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

I get the following error message in unit tests:

File tested,Execution Time (ms)
fdo68963.doc,unknown:0:testfdo68963::Import
An uncaught exception of type com.sun.star.lang.IllegalArgumentException
- Unsupported URL 
: "type 
detection failed"

[...]
##Failure Location unknown## : Error
Test name: testfdo68963::Import
An uncaught exception of type com.sun.star.lang.IllegalArgumentException
- Unsupported URL 
: "type 
detection failed"


Best regards.
JBF


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


Re: 4.4.3.0.0+ build fails (unit tests)

2015-03-19 Thread Justin Luth
I thought that fdo68963.doc only existed in the 4.5 tree.   That file
did not need to be deleted.

P.S.  I have not been able to figure out how to remove a file from a
commit if I add it accidentally.  If I do a "git commit -a" and get an
unintended file, how do I amend the commit to "ignore" the file?  (That
is what happened here.)

Thanks,
Justin


On 19/03/15 16:25, jbfa...@libreoffice.org wrote:
> Hi,
> it seems that the following commit breaks the build of LibreOffice
> 4.4.3.0.0+ :
>
> https://gerrit.libreoffice.org/gitweb?p=core.git;a=commit;h=92a43d05d14b1ab7b8bfb4052874d0f0e6c62ef9
>
>
> I get the following error message in unit tests:
>
> File tested,Execution Time (ms)
> fdo68963.doc,unknown:0:testfdo68963::Import
> An uncaught exception of type com.sun.star.lang.IllegalArgumentException
> - Unsupported URL
> : "type
> detection failed"
> [...]
> ##Failure Location unknown## : Error
> Test name: testfdo68963::Import
> An uncaught exception of type com.sun.star.lang.IllegalArgumentException
> - Unsupported URL
> : "type
> detection failed"
>
> Best regards.
> JBF
>
>

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


[Libreoffice-commits] core.git: Branch 'feature/tiled-editing' - svx/source

2015-03-19 Thread Miklos Vajna
 svx/source/svdraw/svdmrkv.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 97abf479e785d1a6098f972f5215271960d0f79f
Author: Miklos Vajna 
Date:   Thu Mar 19 14:26:36 2015 +0100

SdrMarkView tiled rendering: suppress handles during text edit

Without this, graphic selection was still around during text editing, so
long push on an editeng word (in a Writer shape) triggered the shape
movement action, not a word selection in editeng.

Change-Id: I2ec2f1f9a417bacd08d8b357b4203d48ebaf15c7

diff --git a/svx/source/svdraw/svdmrkv.cxx b/svx/source/svdraw/svdmrkv.cxx
index cc8badb..ac5d4ff 100644
--- a/svx/source/svdraw/svdmrkv.cxx
+++ b/svx/source/svdraw/svdmrkv.cxx
@@ -686,6 +686,9 @@ void SdrMarkView::SetMarkHandles()
 
 if(pSdrTextObj && pSdrTextObj->IsInEditMode())
 {
+if (GetModel()->isTiledRendering())
+// Suppress handles -> empty graphic selection.
+
GetModel()->libreOfficeKitCallback(LOK_CALLBACK_GRAPHIC_SELECTION, "EMPTY");
 return;
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - 2 commits - vcl/inc vcl/qa vcl/source

2015-03-19 Thread Michael Meeks
 vcl/inc/svdata.hxx|2 ++
 vcl/qa/cppunit/outdev.cxx |   26 +-
 vcl/source/app/svdata.cxx |8 
 3 files changed, 23 insertions(+), 13 deletions(-)

New commits:
commit 3af9a3aba0d2148e1e0ad93c16e198af05730edc
Author: Michael Meeks 
Date:   Thu Mar 19 13:56:13 2015 +

ImplSVGDIData didn't dispose its contents previously; clobber it for now.

Change-Id: I71c83cbdabe10f617d1d3d7141ebd6d93923

diff --git a/vcl/inc/svdata.hxx b/vcl/inc/svdata.hxx
index a8bbd58..cdba5cd 100644
--- a/vcl/inc/svdata.hxx
+++ b/vcl/inc/svdata.hxx
@@ -152,6 +152,8 @@ struct ImplSVAppData
 
 struct ImplSVGDIData
 {
+~ImplSVGDIData();
+
 VclPtrmpFirstWinGraphics; // First 
OutputDevice with a Frame Graphics
 VclPtrmpLastWinGraphics;  // Last 
OutputDevice with a Frame Graphics
 VclPtrmpFirstVirGraphics; // First 
OutputDevice with a VirtualDevice Graphics
diff --git a/vcl/source/app/svdata.cxx b/vcl/source/app/svdata.cxx
index 4459fad..213f13d 100644
--- a/vcl/source/app/svdata.cxx
+++ b/vcl/source/app/svdata.cxx
@@ -88,6 +88,14 @@ ImplSVData::ImplSVData()
 maNWFData.maMenuBarHighlightTextColor = Color( COL_TRANSPARENT );
 }
 
+ImplSVGDIData::~ImplSVGDIData()
+{
+// FIXME: deliberately leak any remaining OutputDevice
+// until we have their pGraphics reference counted, doing
+// any disposes so late in shutdown is rather unsafe.
+memset( this, 0, sizeof( ImplSVGDIData ) );
+}
+
 void ImplDeInitSVData()
 {
 ImplSVData* pSVData = ImplGetSVData();
commit 01cd8cbebeba7221123ad01b29bc7b3d3a378c59
Author: Michael Meeks 
Date:   Thu Mar 19 13:56:03 2015 +

fix VirtualDevice unit test.

Change-Id: I7a1f95ae1d8577114634c62aa89d3c7e1fde62c1

diff --git a/vcl/qa/cppunit/outdev.cxx b/vcl/qa/cppunit/outdev.cxx
index 63d88b8..8377a19 100644
--- a/vcl/qa/cppunit/outdev.cxx
+++ b/vcl/qa/cppunit/outdev.cxx
@@ -32,17 +32,17 @@ public:
 
 void VclOutdevTest::testVirtualDevice()
 {
-VirtualDevice aVDev;
-aVDev.SetOutputSizePixel(Size(32,32));
-aVDev.SetBackground(Wallpaper(COL_WHITE));
-aVDev.Erase();
-aVDev.DrawPixel(Point(1,2),COL_BLUE);
-aVDev.DrawPixel(Point(31,30),COL_RED);
-
-Size aSize = aVDev.GetOutputSizePixel();
+ScopedVclPtr pVDev( new VirtualDevice() );
+pVDev->SetOutputSizePixel(Size(32,32));
+pVDev->SetBackground(Wallpaper(COL_WHITE));
+pVDev->Erase();
+pVDev->DrawPixel(Point(1,2),COL_BLUE);
+pVDev->DrawPixel(Point(31,30),COL_RED);
+
+Size aSize = pVDev->GetOutputSizePixel();
 CPPUNIT_ASSERT(aSize == Size(32,32));
 
-Bitmap aBmp = aVDev.GetBitmap(Point(),aSize);
+Bitmap aBmp = pVDev->GetBitmap(Point(),aSize);
 
 #if 0
 OUString rFileName("/tmp/foo-unx.png");
@@ -56,12 +56,12 @@ void VclOutdevTest::testVirtualDevice()
 }
 #endif
 
-CPPUNIT_ASSERT_EQUAL(COL_WHITE, aVDev.GetPixel(Point(0,0)).GetColor());
+CPPUNIT_ASSERT_EQUAL(COL_WHITE, pVDev->GetPixel(Point(0,0)).GetColor());
 #if defined LINUX //TODO: various failures on Mac and Windows tinderboxes
-CPPUNIT_ASSERT_EQUAL(COL_BLUE, aVDev.GetPixel(Point(1,2)).GetColor());
-CPPUNIT_ASSERT_EQUAL(COL_RED, aVDev.GetPixel(Point(31,30)).GetColor());
+CPPUNIT_ASSERT_EQUAL(COL_BLUE, pVDev->GetPixel(Point(1,2)).GetColor());
+CPPUNIT_ASSERT_EQUAL(COL_RED, pVDev->GetPixel(Point(31,30)).GetColor());
 #endif
-CPPUNIT_ASSERT_EQUAL(COL_WHITE, aVDev.GetPixel(Point(30,31)).GetColor());
+CPPUNIT_ASSERT_EQUAL(COL_WHITE, pVDev->GetPixel(Point(30,31)).GetColor());
 
 // Gotcha: y and x swap for BitmapReadAccess: deep joy.
 Bitmap::ScopedReadAccess pAcc(aBmp);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: 4.4.3.0.0+ build fails (unit tests)

2015-03-19 Thread Miklos Vajna
Hi Justin,

On Thu, Mar 19, 2015 at 04:32:35PM +0300, Justin Luth  
wrote:
> P.S.  I have not been able to figure out how to remove a file from a
> commit if I add it accidentally.  If I do a "git commit -a" and get an
> unintended file, how do I amend the commit to "ignore" the file?  (That
> is what happened here.)

rm , then git commit -a --amend

Regards,

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 'feature/vclref' - canvas/source include/svx svx/source sw/inc sw/qa sw/source vcl/workben

2015-03-19 Thread Michael Meeks
 canvas/source/vcl/spritecanvashelper.cxx  |4 +-
 canvas/source/vcl/spritecanvashelper.hxx  |2 -
 include/svx/sdrpaintwindow.hxx|6 +--
 include/svx/svdobj.hxx|3 +
 include/svx/svdpntv.hxx   |4 +-
 svx/source/svdraw/sdrpaintwindow.cxx  |   26 +---
 svx/source/svdraw/svdedxv.cxx |8 ++---
 svx/source/svdraw/svdpntv.cxx |8 ++---
 svx/source/svdraw/svdview.cxx |   14 -
 svx/source/unodraw/UnoGraphicExporter.cxx |   46 +++---
 svx/source/unodraw/unoshape.cxx   |6 +--
 svx/source/xoutdev/_xoutbmp.cxx   |   38 
 svx/source/xoutdev/xattrbmp.cxx   |   24 +++
 svx/source/xoutdev/xtabdash.cxx   |   16 +-
 svx/source/xoutdev/xtabgrdt.cxx   |   10 +++---
 svx/source/xoutdev/xtabhtch.cxx   |   16 +-
 svx/source/xoutdev/xtablend.cxx   |   16 +-
 sw/inc/viewsh.hxx |   12 +++
 sw/qa/tiledrendering/tiledrendering.cxx   |6 +--
 sw/source/core/frmedt/fecopy.cxx  |   18 +--
 sw/source/core/layout/paintfrm.cxx|   14 -
 sw/source/core/view/vnew.cxx  |2 -
 sw/source/filter/ww8/wrtww8gr.cxx |   12 +++
 vcl/workben/vcldemo.cxx   |5 +--
 24 files changed, 160 insertions(+), 156 deletions(-)

New commits:
commit 32f1a7dbca9827642653d314bd96560c093a06ea
Author: Michael Meeks 
Date:   Thu Mar 19 14:38:23 2015 +

Fix OutputDevice members / stack allocation.

Change-Id: Ie57434607b61085a882af40b63d6a4b7aac0d4d3

diff --git a/canvas/source/vcl/spritecanvashelper.cxx 
b/canvas/source/vcl/spritecanvashelper.cxx
index 9164d86..51d01f4 100644
--- a/canvas/source/vcl/spritecanvashelper.cxx
+++ b/canvas/source/vcl/spritecanvashelper.cxx
@@ -324,7 +324,7 @@ namespace vclcanvas
 mpRedrawManager->forEachSprite(
 ::boost::bind(
 &spriteRedraw,
-::boost::ref( maVDev.get() ),
+::boost::ref( *maVDev.get() ),
 _1 ) );
 
 // flush to screen
@@ -582,7 +582,7 @@ namespace vclcanvas
 ::std::for_each( rSortedUpdateSprites.begin(),
  rSortedUpdateSprites.end(),
  ::boost::bind( &spriteRedrawStub2,
-::boost::ref( maVDev.get() ),
+::boost::ref( *maVDev.get() ),
 ::vcl::unotools::b2DPointFromPoint(
 aOutputPosition),
 _1 ) );
diff --git a/canvas/source/vcl/spritecanvashelper.hxx 
b/canvas/source/vcl/spritecanvashelper.hxx
index 0396217..a6b3c03 100644
--- a/canvas/source/vcl/spritecanvashelper.hxx
+++ b/canvas/source/vcl/spritecanvashelper.hxx
@@ -41,7 +41,7 @@ namespace vclcanvas
 {
 public:
 SpriteCanvasHelper();
-~SpriteCanvasHelper()
+~SpriteCanvasHelper();
 
 void init( const OutDevProviderSharedPtr& rOutDev,
SpriteCanvas&  rOwningSpriteCanvas,
diff --git a/include/svx/sdrpaintwindow.hxx b/include/svx/sdrpaintwindow.hxx
index 3a1f1ef..d8f0918 100644
--- a/include/svx/sdrpaintwindow.hxx
+++ b/include/svx/sdrpaintwindow.hxx
@@ -50,10 +50,10 @@ PaintTransparentChildren(vcl::Window & rWindow, Rectangle 
const& rPixelRect);
 class SdrPreRenderDevice
 {
 // The original OutputDevice
-OutputDevice&   mrOutputDevice;
+OutputDevice&  mrOutputDevice;
 
 // The VirtualDevice for PreRendering
-VirtualDevice   maPreRenderDevice;
+VclPtr  mpPreRenderDevice;
 
 public:
 explicit SdrPreRenderDevice(OutputDevice& rOriginal);
@@ -63,7 +63,7 @@ public:
 void OutputPreRenderDevice(const vcl::Region& rExpandedRegion);
 
 OutputDevice& GetOriginalOutputDevice() const { return mrOutputDevice; }
-OutputDevice& GetPreRenderDevice() { return maPreRenderDevice; }
+OutputDevice& GetPreRenderDevice() { return *mpPreRenderDevice.get(); }
 };
 
 
diff --git a/include/svx/svdobj.hxx b/include/svx/svdobj.hxx
index fd1f56f..4ecf505 100644
--- a/include/svx/svdobj.hxx
+++ b/include/svx/svdobj.hxx
@@ -24,6 +24,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -172,7 +173,7 @@ class SVX_DLLPUBLIC SdrObjMacroHitRec
 public:
 Point   aPos;
 Point   aDownPos;
-OutputDevice*   pOut;
+VclPtrpOut;
 const SetOfByte*pVisiLayer;
 const SdrPageView*  pPageView;
 sal_uInt16  nTol;
diff --git a/include/svx/svdpntv.hxx b/include/svx/svdpntv.hxx
index 110bb84..347de0b 100644
--- a/i

[Libreoffice-commits] core.git: Branch 'feature/vclref' - canvas/source chart2/source cui/source

2015-03-19 Thread Michael Meeks
 canvas/source/cairo/cairo_textlayout.cxx  |   10 +--
 canvas/source/opengl/ogl_canvashelper.cxx |   12 +--
 canvas/source/vcl/canvasfont.cxx  |6 -
 canvas/source/vcl/canvashelper_texturefill.cxx|   14 ++--
 canvas/source/vcl/textlayout.cxx  |   26 

 chart2/source/controller/drawinglayer/ViewElementListProvider.cxx |6 -
 chart2/source/view/main/3DChartObjects.cxx|   16 ++---
 chart2/source/view/main/DummyXShape.cxx   |   16 ++---
 chart2/source/view/main/OpenGLRender.cxx  |   16 ++---
 cui/source/customize/cfg.cxx  |   32 
+-
 cui/source/tabpages/tpline.cxx|6 -
 11 files changed, 80 insertions(+), 80 deletions(-)

New commits:
commit 0998ad0fec9150a491b4ccdcf7490715bcdc2eeb
Author: Michael Meeks 
Date:   Thu Mar 19 14:45:20 2015 +

Fix OutputDevice members / stack allocation: chart, canvas, cui.

Change-Id: Ieb6f58222ffc3e03c1d599752a6f02600e1bee1f

diff --git a/canvas/source/cairo/cairo_textlayout.cxx 
b/canvas/source/cairo/cairo_textlayout.cxx
index 892830e..bde8c48 100644
--- a/canvas/source/cairo/cairo_textlayout.cxx
+++ b/canvas/source/cairo/cairo_textlayout.cxx
@@ -180,14 +180,14 @@ namespace cairocanvas
 if( !pOutDev )
 return geometry::RealRectangle2D();
 
-VirtualDevice aVDev( *pOutDev );
-aVDev.SetFont( mpFont->getVCLFont() );
+ScopedVclPtr pVDev( new VirtualDevice( *pOutDev ) );
+pVDev->SetFont( mpFont->getVCLFont() );
 
 // need metrics for Y offset, the XCanvas always renders
 // relative to baseline
-const ::FontMetric& aMetric( aVDev.GetFontMetric() );
+const ::FontMetric& aMetric( pVDev->GetFontMetric() );
 
-setupLayoutMode( aVDev, mnTextDirection );
+setupLayoutMode( *pVDev.get(), mnTextDirection );
 
 const sal_Int32 nAboveBaseline( -aMetric.GetIntLeading() - 
aMetric.GetAscent() );
 const sal_Int32 nBelowBaseline( aMetric.GetDescent() );
@@ -201,7 +201,7 @@ namespace cairocanvas
 else
 {
 return geometry::RealRectangle2D( 0, nAboveBaseline,
-  aVDev.GetTextWidth(
+  pVDev->GetTextWidth(
   maText.Text,
   
::canvas::tools::numeric_cast(maText.StartPosition),
   
::canvas::tools::numeric_cast(maText.Length) ),
diff --git a/canvas/source/opengl/ogl_canvashelper.cxx 
b/canvas/source/opengl/ogl_canvashelper.cxx
index 8c4969b..d3a897d 100644
--- a/canvas/source/opengl/ogl_canvashelper.cxx
+++ b/canvas/source/opengl/ogl_canvashelper.cxx
@@ -725,8 +725,8 @@ namespace oglcanvas
 
 if( mpDevice )
 {
-VirtualDevice aVDev;
-aVDev.EnableOutput(false);
+ScopedVclPtr pVDev(new VirtualDevice());
+pVDev->EnableOutput(false);
 
 CanvasFont* 
pFont=dynamic_cast(xLayoutetText->getFont().get());
 const rendering::StringContext& rTxt=xLayoutetText->getText();
@@ -749,7 +749,7 @@ namespace oglcanvas
 // adjust to stretched font
 if(!::rtl::math::approxEqual(rFontMatrix.m00, rFontMatrix.m11))
 {
-const Size aSize = aVDev.GetFontMetric( aFont ).GetSize();
+const Size aSize = pVDev->GetFontMetric( aFont ).GetSize();
 const double fDividend( rFontMatrix.m10 + rFontMatrix.m11 
);
 double fStretch = (rFontMatrix.m00 + rFontMatrix.m01);
 
@@ -762,7 +762,7 @@ namespace oglcanvas
 }
 
 // set font
-aVDev.SetFont(aFont);
+pVDev->SetFont(aFont);
 
 mpRecordedActions->push_back( Action() );
 Action& rAct=mpRecordedActions->back();
@@ -780,7 +780,7 @@ namespace oglcanvas
 pDXArray[i] = basegfx::fround( aLogicalAdvancements[i] 
);
 
 // get the glyphs
-aVDev.GetTextOutlines(rAct.maPolyPolys,
+pVDev->GetTextOutlines(rAct.maPolyPolys,
   rTxt.Text,
   0,
   rTxt.StartPosition,
@@ -792,7 +792,7 @@ namespace oglcanvas
 else
 {
 // get the glyphs
-aVDev.GetTextOutlines(rAct.maPolyPolys,
+pVDev->GetTextOutlines(rAct.maPolyPolys,
   rTxt.Text,
   0,
 

[Libreoffice-commits] core.git: Branch 'feature/vclref' - cui/source desktop/source lotuswordpro/source sfx2/source

2015-03-19 Thread Michael Meeks
 cui/source/tabpages/tpline.cxx   |6 +++---
 desktop/source/lib/init.cxx  |8 
 desktop/source/splash/splash.cxx |   26 +-
 lotuswordpro/source/filter/lwppagelayout.cxx |8 
 sfx2/source/dialog/backingwindow.cxx |   10 +-
 sfx2/source/doc/objcont.cxx  |   12 ++--
 6 files changed, 35 insertions(+), 35 deletions(-)

New commits:
commit 14c931f8354f5ca6e127fa49b06264002ede1491
Author: Michael Meeks 
Date:   Thu Mar 19 14:58:31 2015 +

Fix OutputDevice members / stack allocation: desktop, sfx2, lwp.

Change-Id: I5563cfa35142f088533ae998d087dec669a13287

diff --git a/cui/source/tabpages/tpline.cxx b/cui/source/tabpages/tpline.cxx
index 9fbaa2e..f64eddb 100644
--- a/cui/source/tabpages/tpline.cxx
+++ b/cui/source/tabpages/tpline.cxx
@@ -360,8 +360,8 @@ void SvxLineTabPage::InitSymbols(MenuButton* pButton)
 
 if(!pButton->GetPopupMenu()->GetPopupMenu( MN_SYMBOLS ) && pSymbolList)
 {
-VirtualDevice aVDev;
-aVDev.SetMapMode(MapMode(MAP_100TH_MM));
+ScopedVclPtr pVDev( new VirtualDevice() );
+pVDev->SetMapMode(MapMode(MAP_100TH_MM));
 boost::scoped_ptr pModel(new SdrModel);
 pModel->GetItemPool().FreezeIdRanges();
 // Page
@@ -370,7 +370,7 @@ void SvxLineTabPage::InitSymbols(MenuButton* pButton)
 pModel->InsertPage( pPage, 0 );
 {
 // 3D View
-boost::scoped_ptr pView(new SdrView( pModel.get(), &aVDev ));
+boost::scoped_ptr pView(new SdrView( pModel.get(), pVDev ));
 pView->hideMarkHandles();
 pView->ShowSdrPage(pPage);
 
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index b3593d0..9d1ae9d 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -566,16 +566,16 @@ void doc_paintTile (LibreOfficeKitDocument* pThis,
 SvpSalInstance* pSalInstance = static_cast< SvpSalInstance* 
>(pSVData->mpDefInst);
 pSalInstance->setBitCountFormatMapping( 32, 
::basebmp::FORMAT_THIRTYTWO_BIT_TC_MASK_RGBA );
 
-VirtualDevice aDevice(0, Size(1, 1), (sal_uInt16)32);
+ScopedVclPtr pDevice( new VirtualDevice(0, Size(1, 1), 
(sal_uInt16)32)) ;
 boost::shared_array< sal_uInt8 > aBuffer( pBuffer, NoDelete< sal_uInt8 >() 
);
-aDevice.SetOutputSizePixelScaleOffsetAndBuffer(
+pDevice->SetOutputSizePixelScaleOffsetAndBuffer(
 Size(nCanvasWidth, nCanvasHeight), Fraction(1.0), Point(),
 aBuffer, true );
 
-pDoc->paintTile(aDevice, nCanvasWidth, nCanvasHeight,
+pDoc->paintTile(*pDevice.get(), nCanvasWidth, nCanvasHeight,
 nTilePosX, nTilePosY, nTileWidth, nTileHeight);
 
-SvpSalVirtualDevice* pSalDev = static_cast< SvpSalVirtualDevice* 
>(aDevice.getSalVirtualDevice());
+SvpSalVirtualDevice* pSalDev = static_cast< SvpSalVirtualDevice* 
>(pDevice->getSalVirtualDevice());
 basebmp::BitmapDeviceSharedPtr pBmpDev = pSalDev->getBitmapDevice();
 
 *pRowStride = pBmpDev->getScanlineStride();
diff --git a/desktop/source/splash/splash.cxx b/desktop/source/splash/splash.cxx
index e3be3e0..1fdc406 100644
--- a/desktop/source/splash/splash.cxx
+++ b/desktop/source/splash/splash.cxx
@@ -149,7 +149,7 @@ SplashScreen::SplashScreen()
 , _yoffset(18)
 {
 loadConfig();
-_vdev.EnableRTL(IsRTLEnabled());
+_vdev->EnableRTL(IsRTLEnabled());
 }
 
 SplashScreen::~SplashScreen()
@@ -258,7 +258,7 @@ SplashScreen::initialize( const 
::com::sun::star::uno::Sequence< ::com::sun::sta
 SetScreenBitmap (_aIntroBmp);
 Size aSize = _aIntroBmp.GetSizePixel();
 SetOutputSizePixel( aSize );
-_vdev.SetOutputSizePixel( aSize );
+_vdev->SetOutputSizePixel( aSize );
 _height = aSize.Height();
 _width = aSize.Width();
 if (_width > 500)
@@ -615,7 +615,7 @@ void SplashScreen::Paint( const Rectangle&)
 //non native drawing
 // draw bitmap
 if (_bPaintBitmap)
-_vdev.DrawBitmapEx( Point(), _aIntroBmp );
+_vdev->DrawBitmapEx( Point(), _aIntroBmp );
 
 if (_bPaintProgress) {
 // draw progress...
@@ -623,20 +623,20 @@ void SplashScreen::Paint( const Rectangle&)
 if (length < 0) length = 0;
 
 // border
-_vdev.SetFillColor();
-_vdev.SetLineColor( _cProgressFrameColor );
-_vdev.DrawRect(Rectangle(_tlx, _tly, _tlx+_barwidth, _tly+_barheight));
-_vdev.SetFillColor( _cProgressBarColor );
-_vdev.SetLineColor();
-_vdev.DrawRect(Rectangle(_tlx+_barspace, _tly+_barspace, 
_tlx+_barspace+length, _tly+_barheight-_barspace));
+_vdev->SetFillColor();
+_vdev->SetLineColor( _cProgressFrameColor );
+_vdev->DrawRect(Rectangle(_tlx, _tly, _tlx+_barwidth, 
_tly+_barheight));
+_vdev->SetFillColor( _cProgressBarColor );
+_vdev->SetLineColor();
+_vdev->DrawRect(Rectangle(_tlx+_b

minutes of ESC call ...

2015-03-19 Thread Michael Meeks
* Present: 
+ Muthu, Sophie, Andras, Miklos, Caolan, David, Michael M, Robinson,
  Norbert, Jan-Marek, Kendy, Lionel
 
* Completed Action Items:
 
* Pending Action Items:
+ please file an Easy-Hack this week: just a bug with whiteboard: (All)
"EasyHack DifficultyBeginner SkillCpp TopicCleanup"
+ thanks to: 
+ http://bit.ly/1KR4HWX
+ make sure to create a new openid if you are using the Google openID (all)
[ still need to get this done ! ]
+ poke Mathiew wrt. Windows / locking bug & revert if no response (Michael 
S)
https://bugs.documentfoundation.org/show_bug.cgi?id=72337
 
* GSoC (?)
+ student application period: March 16 to March 27th
+ Wiki page for tasks:
https://wiki.documentfoundation.org/Development/Gsoc/Ideas
 
* Release Engineering update (Robinson)
+ Cloph on vacation.
+ 4.4.2 RC2 after the hack-fest
+ 4.3.7 RC1 April 7th or so
+ 4.5.0 / 5.0.0? - 1st Alpha release - April 13th
+ Android & iOS Remote
 
* Crashtest update (Caolan)
+ couple of hundred crashers; all the same problem in the
  new service naming scheme, next report should be small again.
   + more service bits in gerrit (Miklos)
 
* Hackfests (Bjoern)
+ Las Palmas de Gran Canaria, Spain
+ March 19th, 20th: Thur / Fri.
  https://wiki.documentfoundation.org/Hackfest/GranCanaria2015
+ Cambridge, UK - May 2015 - contact (Michael)
+ 21st -> 23rd May confirmed. Thur / Fri / Sat.
+ Cambridge Beer Festival overlap / Punting Excursion planned etc.
  https://wiki.documentfoundation.org/Hackfest/Cambridge2015
 
* UX Update (Kendy)
+ not so much this week; finalized & blogged about CMIS
  updated workflow
  + not huge changes; a new toolbar button in toolbar & start-center
  + replace the internal file dialog with an on-line CMIS browser
  + makes no sense not to use the system dialog handler.
+ breeze icons now in the repository & compressed by Andrew
 
* l10n Update (Sophie)
+ still working on the pootle upgrade
 
* Developer Certification (Stephan/Bjoern/Kendy)
+ still pending script foo.
 
* Jenkins update (Norbert)
+ jenkins: http://ci.libreoffice.org/view/Dashboard/
+ Windows system is more resilient
+ this morning a job got stuck & timed out / killed after a while.
+ can build a feature branch if we want to
+ need to push a patch through gerrit; at the end -
  put one of the last patches
+ git push origin HEAD:refs/for/feature/branch
 + and bingo.
+ lcov: http://lcov.libreoffice.org/
+ failing around coinmp - Makefile doesn't like it.
+ still pending.
 
* Tinderbox functionality stats (Norbert)
+ what %age of time is green for each platform ?
+ Linux:   90% green
+ Mac: 75% green
+ Windows: 66% green
+ is it related to build-times ? (Michael)
+ Linux & Mac - are incremental
+ mostly tied to people fixing on that platform (Kendy)
+ waits for Windows users to fix things.
+ Linux started a lot later as a TB (Norbert)
 
* VclPtr (Michael)
+ Noel and I working hard on it
+ tentacles of lifecycle badness everywhere
 
* Document Liberation (David)
+ some good candidates for GSOC
+ lots of followers on twitter.
 
* QA (Robinson)
+ UNCONFIRMED at 319
+ needAdvice at 11
+ running out of 4.4 regressions to bisect (Matthew Francis)
+ goal is to be able to build bibisect in real-time
+ compressing the pack takes over a day (Norbert)
+ would like to do it all on one machine (Robinson)
+ vmiklos' dbgutil / master bibisect repo on dev-builds
+ useful for chasing master.
 
* QA stats:
  + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
+123-79(+44 overall)
many thanks to the top bug squashers:
Jean-Baptiste Faure   12
raal  10
Serg Bormant   7
V Stuart Foote 6
Michael Stahl  5
Cor Nouws  3
Beluga 2
Julien Nabet   2
David Tardon   2
Adolfo Jayme   2
Matthew Francis2
Laurent BP 2
Kohei Yoshida  1
Joni_M 1
Markus Mohrhard1
 
* Open 4.5 MAB
  + 1/3 1/3 0/2 0/2 1/2 0/1 0/1 0/1 0/1
  + 
https://bugs.documentfoundation.org/showdependencytree.cgi?id=86696&hide_resolved=1
 
* Open 4.4 MAB
  + 13/49 15/49 14/48 13/47 12/46 16/44 16/43 16/40 14/38 11/34 6/26 6/24 7/24
 26%   30%   29%   27%   26%   36%   37%   40%   36%   32%  23%  25%  29%
  + 
https://bugs.documentfoundation.org/showdependencytree.cgi?id=79641&hide_resolved=1
 
* Open 4.3 MAB
  + 78/160 79/160 77/158 77/158 76/158 77/156 77/155 78/155 86

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

2015-03-19 Thread Jan-Marek Glogowski
 sw/source/uibase/dbui/dbmgr.cxx |5 -
 1 file changed, 5 deletions(-)

New commits:
commit adb98d94ca1059b56d6b6726ca9aea96ecae8a96
Author: Jan-Marek Glogowski 
Date:   Thu Mar 19 15:48:37 2015 +0100

tdf#70346 MM: don't restore section visibility

Mail merging the current document is expected to change the
source document. So there is no need to try to restore the pre-MM
document, even the visibility. In the end any effort is futile,
as field changes cannot be undone.

Even worse: it introduced a regression using the "data to fields"
function from the data source manager (F4), which would never
set the correct section visibility.

We need to copy the document to prevent changes to the source
document and skip the quite probably unexpected save in MM.

Change-Id: I814905f1566a63b2ee14cb5f6d08da34c05942aa

diff --git a/sw/source/uibase/dbui/dbmgr.cxx b/sw/source/uibase/dbui/dbmgr.cxx
index 1b3e2c5..fa635d7 100644
--- a/sw/source/uibase/dbui/dbmgr.cxx
+++ b/sw/source/uibase/dbui/dbmgr.cxx
@@ -453,11 +453,6 @@ bool SwDBManager::MergeNew( const SwMergeDescriptor& 
rMergeDesc )
 
 DELETEZ( pImpl->pMergeData );
 
-// Recalculate original section visibility states, as field changes aren't
-// tracked (not undo-able).  Has to be done, after pImpl->pMergeData is
-//  gone, otherwise merge data is used for calculation!
-rMergeDesc.rSh.SwViewShell::UpdateFlds();
-
 bInMerge = false;
 
 return bRet;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/tiled-editing' - android/Bootstrap android/experimental README.cross

2015-03-19 Thread Miklos Vajna
 README.cross |   27 +++
 android/Bootstrap/Makefile.shared|2 +-
 android/experimental/LOAndroid3/Makefile |5 -
 3 files changed, 8 insertions(+), 26 deletions(-)

New commits:
commit f3d4d3c6e5916b5f6ba88060cd07dbd28660dd9b
Author: Miklos Vajna 
Date:   Thu Mar 19 16:16:55 2015 +0100

android: add debugrun Makefile target

Also, if we are at it:

- clean up 'run' as well: since the doc browser is the default activity,
  no need to pass the test doc path anymore
- make 'install' not depend on build: a full build would need a toplevel
  'make' anyway

Change-Id: Ia55d52f767ab3e0be02a753a95b2aac02f8491cc

diff --git a/README.cross b/README.cross
index 109588e..efaa8c2 100644
--- a/README.cross
+++ b/README.cross
@@ -339,13 +339,8 @@ course need an appropriate system image for that.
 Here is an autogen.input for Android on ARM when cross-compiling
 from Linux:
 
---build=x86_64-unknown-linux-gnu
 --enable-dbgutil
---enable-debug
 --enable-werror
---with-android-ndk=/home/tml/android-ndk-r9c
---with-android-ndk-toolchain-version=4.8
---with-android-sdk=/home/tml/adt-bundle-linux/sdk
 --with-distro=LibreOfficeAndroid
 
 And here is an (quite old) autogen.input for Android on X86:
@@ -368,25 +363,9 @@ Note that none of these apps in any way are claimed to be 
ready for
 end-users. No "beta testing" offers needed, it is painfully obvious
 what problems they have.
 
-To run some of the apps, do "make install" followed by either "make
-run" or starting it from Android itself. You most likely want to have
-an "adb logcat" running in another window.
-
-To debug, do manually what "make run" would do and when the app has
-started, run ndk-gdb.
-
-You can also do that manually; to install:
-
-  /platform-tools/adb install -r 
android/experimental/LOAndroid3/bin/LibreOfficeViewer-debug.apk
-
-To see the log:
-
-  /platform-tools/adb logcat
-
-To debug, install the .apk, run it, and then:
-
-  cd android/experimental/LOAndroid3
-  /ndk-gdb --adb=/platform-tools/adb
+To run some of the apps, do "make install" followed by either "make run" or
+starting it from Android itself. You most likely want to have an "adb logcat"
+running in another window. To debug, run "make debugrun".
 
 NB: If you happen to upgrade to Android SDK Tools 23, and the build (using
 'make verbose=t android') fails for you with:
diff --git a/android/Bootstrap/Makefile.shared 
b/android/Bootstrap/Makefile.shared
index 2a8b545..5020773 100644
--- a/android/Bootstrap/Makefile.shared
+++ b/android/Bootstrap/Makefile.shared
@@ -64,7 +64,7 @@ properties:
echo "APP_ABI := $(ANDROID_APP_ABI)" > jni/Application.mk
echo "APP_PLATFORM := android-15" >> jni/Application.mk
 
-install: build-ant
+install:
unset JAVA_HOME && $(ANT) $(if $(VERBOSE)$(verbose),,-quiet) debug 
install
@echo
@echo 'Run it with "make run"'
diff --git a/android/experimental/LOAndroid3/Makefile 
b/android/experimental/LOAndroid3/Makefile
index ecefd10..fbcec0c 100644
--- a/android/experimental/LOAndroid3/Makefile
+++ b/android/experimental/LOAndroid3/Makefile
@@ -35,4 +35,7 @@ build-ant: android_version_setup copy-stuff prepare-appcompat 
link-so properties
unset JAVA_HOME && $(ANT) $(if $(VERBOSE)$(verbose),,-quiet) $(if 
$(ENABLE_RELEASE_BUILD),release,debug)
 
 run:
-   adb shell am start -n $(APP_PACKAGE)/.ui.LibreOfficeUIActivity -e input 
/assets/test1.odt
+   adb shell am start -n $(APP_PACKAGE)/.ui.LibreOfficeUIActivity
+
+debugrun:
+   $(SYSBASE)/../../../ndk-gdb --start
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - sc/source sd/source slideshow/source starmath/qa

2015-03-19 Thread Michael Meeks
 sc/source/core/data/global.cxx |   10 +-
 sc/source/filter/rtf/eeimpars.cxx  |4 
 sc/source/ui/app/transobj.cxx  |   20 ++--
 sc/source/ui/docshell/docfunc.cxx  |1 
 sc/source/ui/docshell/docsh.cxx|4 
 sc/source/ui/docshell/docsh3.cxx   |   10 +-
 sc/source/ui/undo/undobase.cxx |8 -
 sc/source/ui/undo/undoblk.cxx  |4 
 sc/source/ui/undo/undoblk3.cxx |6 -
 sc/source/ui/undo/undostyl.cxx |6 -
 sc/source/ui/unoobj/styleuno.cxx   |   24 ++---
 sc/source/ui/view/viewfunc.cxx |   12 +-
 sd/source/filter/html/buttonset.cxx|   12 +-
 sd/source/ui/annotations/annotationtag.cxx |   20 ++--
 sd/source/ui/dlg/animobjs.cxx  |   18 ++--
 sd/source/ui/dlg/docprev.cxx   |   18 ++--
 sd/source/ui/docshell/docshel2.cxx |   20 ++--
 sd/source/ui/slideshow/showwin.cxx |   16 +--
 sd/source/ui/slidesorter/view/SlsInsertionIndicatorOverlay.cxx |   16 +--
 slideshow/source/engine/rehearsetimingsactivity.cxx|   40 -
 slideshow/source/engine/shapes/gdimtftools.cxx |   42 
+-
 starmath/qa/cppunit/test_nodetotextvisitors.cxx|   32 +++
 22 files changed, 171 insertions(+), 172 deletions(-)

New commits:
commit bc536dd1ab7ca452f63460755a1ead89b73d256e
Author: Michael Meeks 
Date:   Thu Mar 19 15:39:43 2015 +

Fix OutputDevice members / stack allocation: slideshow, starmath, sd, sc.

Change-Id: I6d6d41b8a6501ce7d35c78e5b730ea45143d6b73

diff --git a/sc/source/core/data/global.cxx b/sc/source/core/data/global.cxx
index 9189ba8..d4aa8f8 100644
--- a/sc/source/core/data/global.cxx
+++ b/sc/source/core/data/global.cxx
@@ -575,13 +575,13 @@ void ScGlobal::InitTextHeight(SfxItemPool* pPool)
 }
 
 OutputDevice* pDefaultDev = Application::GetDefaultDevice();
-VirtualDevice aVirtWindow( *pDefaultDev );
-aVirtWindow.SetMapMode(MAP_PIXEL);
+ScopedVclPtr pVirtWindow( new VirtualDevice( *pDefaultDev ) 
);
+pVirtWindow->SetMapMode(MAP_PIXEL);
 vcl::Font aDefFont;
-pPattern->GetFont(aDefFont, SC_AUTOCOL_BLACK, &aVirtWindow); // Font color 
doesn't matter here
-aVirtWindow.SetFont(aDefFont);
+pPattern->GetFont(aDefFont, SC_AUTOCOL_BLACK, pVirtWindow); // Font color 
doesn't matter here
+pVirtWindow->SetFont(aDefFont);
 sal_uInt16 nTest = static_cast(
-aVirtWindow.PixelToLogic(Size(0, aVirtWindow.GetTextHeight()), 
MAP_TWIP).Height());
+pVirtWindow->PixelToLogic(Size(0, pVirtWindow->GetTextHeight()), 
MAP_TWIP).Height());
 
 if (nTest > nDefFontHeight)
 nDefFontHeight = nTest;
diff --git a/sc/source/filter/rtf/eeimpars.cxx 
b/sc/source/filter/rtf/eeimpars.cxx
index 1133568..a8ac3d5 100644
--- a/sc/source/filter/rtf/eeimpars.cxx
+++ b/sc/source/filter/rtf/eeimpars.cxx
@@ -433,8 +433,8 @@ void ScEEImport::WriteToDocument( bool bSizeColsRows, 
double nOutputFactor, SvNu
 // Factor is printer to display ratio
 double nPPTX = ScGlobal::nScreenPPTX * (double) aZoom / nOutputFactor;
 double nPPTY = ScGlobal::nScreenPPTY * (double) aZoom;
-VirtualDevice aVirtDev;
-sc::RowHeightContext aCxt(nPPTX, nPPTY, aZoom, aZoom, &aVirtDev);
+ScopedVclPtr pVirtDev( new VirtualDevice() );
+sc::RowHeightContext aCxt(nPPTX, nPPTY, aZoom, aZoom, pVirtDev);
 aCxt.setExtraHeight(ScGlobal::nLastRowHeightExtra);
 mpDoc->SetOptimalHeight(aCxt, 0, nEndRow, 0);
 
diff --git a/sc/source/ui/app/transobj.cxx b/sc/source/ui/app/transobj.cxx
index cb822e2..abf034a 100644
--- a/sc/source/ui/app/transobj.cxx
+++ b/sc/source/ui/app/transobj.cxx
@@ -336,13 +336,13 @@ bool ScTransferObj::GetData( const 
datatransfer::DataFlavor& rFlavor, const OUSt
 Rectangle aMMRect = pDoc->GetMMRect( aBlock.aStart.Col(), 
aBlock.aStart.Row(),
  aBlock.aEnd.Col(), 
aBlock.aEnd.Row(),
  aBlock.aStart.Tab() );
-VirtualDevice aVirtDev;
-aVirtDev.SetOutputSizePixel( aVirtDev.LogicToPixel( 
aMMRect.GetSize(), MAP_100TH_MM ) );
+ScopedVclPtr pVirtDev( new VirtualDevice() );
+pVirtDev->SetOutputSizePixel( pVirtDev->LogicToPixel( 
aMMRect.GetSize(), MAP_100TH_MM ) );
 
-PaintToDev( &aVirtDev, pDoc, 1.0, aBlock, false );
+PaintToDev( pVirtDev, pDoc, 1.0, aBlock, false );
 
-aVirtDev.SetMapMode( MapMode( MAP_PIXEL ) );
-Bitmap aBmp = aVirtDe

New Defects reported by Coverity Scan for LibreOffice

2015-03-19 Thread scan-admin

Hi,

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

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

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


** CID 1288898:  Uninitialized members  (UNINIT_CTOR)
/vcl/unx/generic/printer/cupsmgr.cxx: 189 in psp::CUPSManager::CUPSManager()()



*** CID 1288898:  Uninitialized members  (UNINIT_CTOR)
/vcl/unx/generic/printer/cupsmgr.cxx: 189 in psp::CUPSManager::CUPSManager()()
183 m_nDests( 0 ),
184 m_pDests( NULL ),
185 m_bNewDests( false ),
186 m_bPPDThreadRunning( false )
187 {
188 m_aDestThread = osl_createThread( run_dest_thread_stub, this );
>>> CID 1288898:  Uninitialized members  (UNINIT_CTOR)
>>> Non-static class member "batchMode" is not initialized in this 
>>> constructor nor in any functions that it calls.
189 }
190 
191 CUPSManager::~CUPSManager()
192 {
193 if( m_aDestThread )
194 {



To view the defects in Coverity Scan visit, 
https://scan.coverity.com/projects/211?tab=overview

To manage Coverity Scan email notifications for 
"libreoffice@lists.freedesktop.org", click 
https://scan.coverity.com/subscriptions/edit?email=libreoffice%40lists.freedesktop.org&token=d6481d718a775246b2340f282ebe5939
 .

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


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

2015-03-19 Thread Miklos Vajna
 vcl/unx/generic/printer/cupsmgr.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 2250caf30d5e9fb297e97952a2bc061856701563
Author: Miklos Vajna 
Date:   Thu Mar 19 17:06:59 2015 +0100

CID 1288898: Uninitialized member

Change-Id: I1523ba22d419a12ae60de4f7afd293fe6cbfb6b5

diff --git a/vcl/unx/generic/printer/cupsmgr.cxx 
b/vcl/unx/generic/printer/cupsmgr.cxx
index beb2e1d..c0ece24 100644
--- a/vcl/unx/generic/printer/cupsmgr.cxx
+++ b/vcl/unx/generic/printer/cupsmgr.cxx
@@ -183,7 +183,8 @@ CUPSManager::CUPSManager() :
 m_nDests( 0 ),
 m_pDests( NULL ),
 m_bNewDests( false ),
-m_bPPDThreadRunning( false )
+m_bPPDThreadRunning( false ),
+batchMode( false )
 {
 m_aDestThread = osl_createThread( run_dest_thread_stub, this );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - svx/source

2015-03-19 Thread Michael Meeks
 svx/source/customshapes/EnhancedCustomShapeFontWork.cxx |   30 ++--
 svx/source/dialog/_bmpmask.cxx  |   12 ++--
 svx/source/dialog/_contdlg.cxx  |   26 +-
 svx/source/dialog/dlgctrl.cxx   |   40 
 svx/source/dialog/svxruler.cxx  |   14 ++---
 svx/source/gallery2/galobj.cxx  |4 -
 svx/source/gallery2/galtheme.cxx|6 +-
 svx/source/svdraw/svdfmtf.cxx   |8 +--
 svx/source/svdraw/svdograf.cxx  |   10 ++--
 svx/source/svdraw/svdotext.cxx  |8 +--
 svx/source/svdraw/svdtrans.cxx  |   16 +++---
 svx/source/svdraw/svdxcgv.cxx   |   22 
 svx/source/tbxctrls/colrctrl.cxx|   14 ++---
 svx/source/tbxctrls/fontworkgallery.cxx |   14 ++---
 svx/source/tbxctrls/linectrl.cxx|   16 +++---
 15 files changed, 120 insertions(+), 120 deletions(-)

New commits:
commit a8f6f46f5dbbc9d1fc74c1ded97bbdda9b1690b1
Author: Michael Meeks 
Date:   Thu Mar 19 16:19:50 2015 +

Fix OutputDevice members / stack allocation: svx.

Change-Id: Id28aeb44b4c48fec944e21fed7a9acf9f7f8b68a

diff --git a/svx/source/customshapes/EnhancedCustomShapeFontWork.cxx 
b/svx/source/customshapes/EnhancedCustomShapeFontWork.cxx
index 7332449..1b69bc7 100644
--- a/svx/source/customshapes/EnhancedCustomShapeFontWork.cxx
+++ b/svx/source/customshapes/EnhancedCustomShapeFontWork.cxx
@@ -165,9 +165,9 @@ void CalculateHorizontalScalingFactor( const SdrObject* 
pCustomShape,
 aFont.SetOrientation( 0 );
 // initializing virtual device
 
-VirtualDevice aVirDev( 1 );
-aVirDev.SetMapMode( MAP_100TH_MM );
-aVirDev.SetFont( aFont );
+ScopedVclPtr pVirDev( new VirtualDevice( 1 ) );
+pVirDev->SetMapMode( MAP_100TH_MM );
+pVirDev->SetFont( aFont );
 
 if ( nOutlinesCount2d & 1 )
 bSingleLineMode = true;
@@ -187,7 +187,7 @@ void CalculateHorizontalScalingFactor( const SdrObject* 
pCustomShape,
 std::vector< FWParagraphData >::const_iterator aParagraphIEnd( 
aTextAreaIter->vParagraphs.end() );
 while( aParagraphIter != aParagraphIEnd )
 {
-double fTextWidth = aVirDev.GetTextWidth( aParagraphIter->aString 
);
+double fTextWidth = pVirDev->GetTextWidth( aParagraphIter->aString 
);
 if ( fTextWidth > 0.0 )
 {
 double fScale = fWidth / fTextWidth;
@@ -259,12 +259,12 @@ void GetTextAreaOutline( const FWData& rFWData, const 
SdrObject* pCustomShape, F
 aFont.SetWeight( rWeightItem.GetWeight() );
 
 // initializing virtual device
-VirtualDevice aVirDev( 1 );
-aVirDev.SetMapMode( MAP_100TH_MM );
-aVirDev.SetFont( aFont );
-aVirDev.EnableRTL( true );
+ScopedVclPtr pVirDev( new VirtualDevice( 1 ) );
+pVirDev->SetMapMode( MAP_100TH_MM );
+pVirDev->SetFont( aFont );
+pVirDev->EnableRTL( true );
 if ( aParagraphIter->nFrameDirection == FRMDIR_HORI_RIGHT_TOP )
-aVirDev.SetLayoutMode( TEXT_LAYOUT_BIDI_RTL );
+pVirDev->SetLayoutMode( TEXT_LAYOUT_BIDI_RTL );
 
 const SvxCharScaleWidthItem& rCharScaleWidthItem = 
static_cast(pCustomShape->GetMergedItem( 
EE_CHAR_FONTWIDTH ));
 sal_uInt16 nCharScaleWidth = rCharScaleWidthItem.GetValue();
@@ -282,9 +282,9 @@ void GetTextAreaOutline( const FWData& rFWData, const 
SdrObject* pCustomShape, F
 {
 FWCharacterData aCharacterData;
 OUString aCharText( (sal_Unicode)rText[ i ] );
-if ( aVirDev.GetTextOutlines( aCharacterData.vOutlines, 
aCharText, 0, 0, -1, true, nWidth, pDXArry ) )
+if ( pVirDev->GetTextOutlines( aCharacterData.vOutlines, 
aCharText, 0, 0, -1, true, nWidth, pDXArry ) )
 {
-sal_Int32 nTextWidth = aVirDev.GetTextWidth( 
aCharText);
+sal_Int32 nTextWidth = pVirDev->GetTextWidth( 
aCharText);
 std::vector< tools::PolyPolygon >::iterator 
aOutlineIter = aCharacterData.vOutlines.begin();
 std::vector< tools::PolyPolygon >::iterator 
aOutlineIEnd = aCharacterData.vOutlines.end();
 if ( aOutlineIter == aOutlineIEnd )
@@ -334,13 +334,13 @@ void GetTextAreaOutline( const FWData& rFWData, const 
SdrObject* pCustomShape, F
 if ( ( nCharScaleWidth != 100 ) && nCharScaleWidth )
 {   // applying character spacing
 pDXArry = new long[ rText.getLength() ];
-aVirDev.GetTextArray( rText, pDXArry);
-FontMetric aFontMetric( aVirDev.

[Libreoffice-commits] core.git: helpcontent2

2015-03-19 Thread Andras Timar
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f7e67e253c4524f382d988ec058b31b91baea3c6
Author: Andras Timar 
Date:   Thu Mar 19 17:28:28 2015 +0100

Updated core
Project: help  325b71813e476e25acbcecbb34f57ec2609ee98b

diff --git a/helpcontent2 b/helpcontent2
index 2bd951e..325b718 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 2bd951ef710b4a93937519270b19592c8762a626
+Subproject commit 325b71813e476e25acbcecbb34f57ec2609ee98b
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Andras Timar
 source/text/scalc/01/04060181.xhp |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 325b71813e476e25acbcecbb34f57ec2609ee98b
Author: Andras Timar 
Date:   Thu Mar 19 17:28:28 2015 +0100

fix copypaste: a required parameter cannot be omitted

Change-Id: I09ec850aea96962555c526efa0b4c1355236fb2f

diff --git a/source/text/scalc/01/04060181.xhp 
b/source/text/scalc/01/04060181.xhp
index 7c1ac57..ade5291 100644
--- a/source/text/scalc/01/04060181.xhp
+++ b/source/text/scalc/01/04060181.xhp
@@ -268,7 +268,7 @@ oldref="547">COUNTIF
  Number (required) is the value between 
Start and End at which to evaluate the 
function.
  Alpha (required) is a parameter to the 
distribution.
  Beta (required) is a parameter to the 
distribution.
- Cumulative (required) can be 0 or 
False to calculate the probability density function. It can be any other value 
or True or omitted to calculate the cumulative distribution 
function.
+ Cumulative (required) can be 0 or 
False to calculate the probability density function. It can be any other value 
or True to calculate the cumulative distribution function.
  Start (optional) is the lower bound for 
Number.
  End (optional) is the upper bound for 
Number.
  
commit c8cbe130afa50dacc47c2d1d30e1774d44786a0c
Author: Andras Timar 
Date:   Thu Mar 19 17:26:07 2015 +0100

typos: CHI.TEST -> CHISQ.TEST, CHI.DIST -> CHISQ.DIST

Change-Id: I2057217b5027283525f570b065f10a04c7165cb2

diff --git a/source/text/scalc/01/04060181.xhp 
b/source/text/scalc/01/04060181.xhp
index a27a4c3..7c1ac57 100644
--- a/source/text/scalc/01/04060181.xhp
+++ b/source/text/scalc/01/04060181.xhp
@@ -1,6 +1,6 @@
 
 
-   
+
 
- 
-   
+
+
 
   
  Statistical Functions Part 
One
@@ -530,8 +530,8 @@ oldref="547">COUNTIF
 
 
 CHISQ.TEST
-Returns the probability of a 
deviance from a random distribution of two test series based on the chi-squared 
test for independence. CHI.TEST returns the chi-squared distribution of 
the data.
-The probability determined by CHI.TEST can also be determined with 
CHI.DIST, in which case the Chi square of the random sample must then be passed 
as a parameter instead of the data row.
+Returns the probability of a 
deviance from a random distribution of two test series based on the chi-squared 
test for independence. CHISQ.TEST returns the chi-squared distribution 
of the data.
+The probability determined by CHISQ.TEST can also be determined 
with CHISQ.DIST, in which case the Chi square of the random sample must then be 
passed as a parameter instead of the data row.
 Syntax
 CHISQ.TEST(DataB; DataE)
 DataB is the array of the observations.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Jan-Marek Glogowski
 sw/source/uibase/dbui/dbmgr.cxx |5 -
 1 file changed, 5 deletions(-)

New commits:
commit ea1caeed5cc70f41957d56af0ccb4bfdbd01cb87
Author: Jan-Marek Glogowski 
Date:   Thu Mar 19 15:48:37 2015 +0100

tdf#70346 MM: don't restore section visibility

Mail merging the current document is expected to change the
source document. So there is no need to try to restore the pre-MM
document, even the visibility. In the end any effort is futile,
as field changes cannot be undone.

Even worse: it introduced a regression using the "data to fields"
function from the data source manager (F4), which would never
set the correct section visibility.

We need to copy the document to prevent changes to the source
document and skip the quite probably unexpected save in MM.

Change-Id: I814905f1566a63b2ee14cb5f6d08da34c05942aa
(cherry picked from commit adb98d94ca1059b56d6b6726ca9aea96ecae8a96)
Reviewed-on: https://gerrit.libreoffice.org/14906
Tested-by: Caolán McNamara 
Reviewed-by: Caolán McNamara 

diff --git a/sw/source/uibase/dbui/dbmgr.cxx b/sw/source/uibase/dbui/dbmgr.cxx
index 584efa3..00f4d29 100644
--- a/sw/source/uibase/dbui/dbmgr.cxx
+++ b/sw/source/uibase/dbui/dbmgr.cxx
@@ -441,11 +441,6 @@ bool SwDBManager::MergeNew( const SwMergeDescriptor& 
rMergeDesc )
 
 DELETEZ( pImpl->pMergeData );
 
-// Recalculate original section visibility states, as field changes aren't
-// tracked (not undo-able).  Has to be done, after pImpl->pMergeData is
-//  gone, otherwise merge data is used for calculation!
-rMergeDesc.rSh.SwViewShell::UpdateFlds();
-
 bInMerge = false;
 
 return bRet;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - filter/source

2015-03-19 Thread Michael Meeks
 filter/source/graphicfilter/eos2met/eos2met.cxx |   37 
 filter/source/graphicfilter/epict/epict.cxx |   29 +-
 filter/source/graphicfilter/eps/eps.cxx |   30 +--
 filter/source/graphicfilter/ieps/ieps.cxx   |   36 +++
 filter/source/msfilter/escherex.cxx |   18 +--
 filter/source/msfilter/msdffimp.cxx |6 +--
 filter/source/svg/svgfontexport.cxx |   24 +++
 7 files changed, 90 insertions(+), 90 deletions(-)

New commits:
commit feb50ac675d194b84743736f571b652221c5e869
Author: Michael Meeks 
Date:   Thu Mar 19 16:56:13 2015 +

Fix OutputDevice members / stack allocation: filter.

Change-Id: Ib67fd510626809baab774382d37b20462253fc31

diff --git a/filter/source/graphicfilter/eos2met/eos2met.cxx 
b/filter/source/graphicfilter/eos2met/eos2met.cxx
index 36ffc74..0fd0114 100644
--- a/filter/source/graphicfilter/eos2met/eos2met.cxx
+++ b/filter/source/graphicfilter/eos2met/eos2met.cxx
@@ -156,7 +156,7 @@ private:
 sal_uInt32   nWrittenBitmaps;  // number of already written 
Bitmaps
 sal_uInt32   nActBitmapPercent; // percentage of the next 
bitmap that's already written
 
-::std::unique_ptr< VirtualDevice >  apDummyVDev;
+ScopedVclPtr  apDummyVDev;
 OutputDevice*   pCompDev;
 
 com::sun::star::uno::Reference< com::sun::star::task::XStatusIndicator > 
xStatusIndicator;
@@ -261,6 +261,7 @@ public:
 pCompDev = reinterpret_cast< OutputDevice* >( 
Application::GetAppWindow() );
 if( !pCompDev )
 {
+apDummyVDev.disposeAndClear();
 apDummyVDev.reset( new VirtualDevice );
 pCompDev = apDummyVDev.get();
 }
@@ -1854,12 +1855,12 @@ void METWriter::WriteOrders( const GDIMetaFile* pMTF )
 
 if( aGDIFont.GetAlign() != ALIGN_BASELINE)
 {
-VirtualDevice aVDev;
+ScopedVclPtr pVDev( new VirtualDevice() );
 
 if( aGDIFont.GetAlign()==ALIGN_TOP )
-aPt.Y()+=(long)aVDev.GetFontMetric( aGDIFont 
).GetAscent();
+aPt.Y()+=(long)pVDev->GetFontMetric( aGDIFont 
).GetAscent();
 else
-aPt.Y()-=(long)aVDev.GetFontMetric( aGDIFont 
).GetDescent();
+aPt.Y()-=(long)pVDev->GetFontMetric( aGDIFont 
).GetDescent();
 }
 
 METSetMix(eGDIRasterOp);
@@ -1883,11 +1884,11 @@ void METWriter::WriteOrders( const GDIMetaFile* pMTF )
 
 if( aGDIFont.GetAlign() != ALIGN_BASELINE )
 {
-VirtualDevice aVDev;
+ScopedVclPtr pVDev( new VirtualDevice() );
 if( aGDIFont.GetAlign() == ALIGN_TOP )
-
aPt.Y()+=(long)aVDev.GetFontMetric(aGDIFont).GetAscent();
+
aPt.Y()+=(long)pVDev->GetFontMetric(aGDIFont).GetAscent();
 else
-
aPt.Y()-=(long)aVDev.GetFontMetric(aGDIFont).GetDescent();
+
aPt.Y()-=(long)pVDev->GetFontMetric(aGDIFont).GetDescent();
 }
 
 METSetMix(eGDIRasterOp);
@@ -1926,7 +1927,7 @@ void METWriter::WriteOrders( const GDIMetaFile* pMTF )
 case META_STRETCHTEXT_ACTION:
 {
 const MetaStretchTextAction*pA = static_cast(pMA);
-VirtualDevice   aVDev;
+ScopedVclPtr pVDev( new VirtualDevice() );
 sal_uInt16  i;
 sal_Int32   nNormSize;
 OUStringaStr;
@@ -1935,14 +1936,14 @@ void METWriter::WriteOrders( const GDIMetaFile* pMTF )
 Point   aPt( pA->GetPoint() );
 Point   aPt2;
 
-aVDev.SetFont( aGDIFont );
+pVDev->SetFont( aGDIFont );
 
 if( aGDIFont.GetAlign() != ALIGN_BASELINE)
 {
 if( aGDIFont.GetAlign() == ALIGN_TOP )
-aPt.Y()+=(long)aVDev.GetFontMetric().GetAscent();
+aPt.Y()+=(long)pVDev->GetFontMetric().GetAscent();
 else
-aPt.Y()-=(long)aVDev.GetFontMetric().GetDescent();
+aPt.Y()-=(long)pVDev->GetFontMetric().GetDescent();
 }
 
 METSetMix(eGDIRasterOp);
@@ -1953,7 +1954,7 @@ void METWriter::WriteOrders( const GDIMetaFile* pMTF )
 METSetChrSet(FindChrSet(aGDIFont));
 aStr = pA->GetText().copy(pA->GetIndex(),pA->GetLen());
 boost::scoped_array pDXAry(new long[aStr.getLength()]);
-nNormSize = aVDev.GetTextArray( aStr, pDXA

[Libreoffice-commits] core.git: Branch 'feature/tiled-editing' - android/experimental

2015-03-19 Thread Miklos Vajna
 
android/experimental/LOAndroid3/src/java/org/libreoffice/InvalidationHandler.java
 |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 2dad7eb36753f9c84bfcc20c456e33104be06482
Author: Miklos Vajna 
Date:   Thu Mar 19 17:53:38 2015 +0100

android: when in selection, avoid showing the middle handle

editeng sends LOK_CALLBACK_CURSOR_VISIBLE after the text selection is
already created. The effect of this is that after a long push on a word
you get not only the selection overlay + the start/end handles, but also
the middle one, which is not expected. Just check if we're in a
selection already, and if so, don't react to
LOK_CALLBACK_CURSOR_VISIBLE=true.

Change-Id: I939c957feb7889dfb1997fac1051cb4eaa77004c

diff --git 
a/android/experimental/LOAndroid3/src/java/org/libreoffice/InvalidationHandler.java
 
b/android/experimental/LOAndroid3/src/java/org/libreoffice/InvalidationHandler.java
index e9110f6..805ed9c 100644
--- 
a/android/experimental/LOAndroid3/src/java/org/libreoffice/InvalidationHandler.java
+++ 
b/android/experimental/LOAndroid3/src/java/org/libreoffice/InvalidationHandler.java
@@ -3,6 +3,7 @@ package org.libreoffice;
 import android.content.Intent;
 import android.graphics.RectF;
 import android.net.Uri;
+import android.util.Log;
 
 import org.libreoffice.kit.Document;
 import org.mozilla.gecko.TextSelection;
@@ -211,7 +212,9 @@ public class InvalidationHandler implements 
Document.MessageCallback {
 private synchronized void cursorVisibility(String payload) {
 if (payload.equals("true")) {
 mTextCursorLayer.showCursor();
-mTextSelection.showHandle(TextSelectionHandle.HandleType.MIDDLE);
+if (mState != OverlayState.SELECTION) {
+
mTextSelection.showHandle(TextSelectionHandle.HandleType.MIDDLE);
+}
 } else if (payload.equals("false")) {
 mTextCursorLayer.hideCursor();
 mTextSelection.hideHandle(TextSelectionHandle.HandleType.MIDDLE);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: New Defects reported by Coverity Scan for LibreOffice

2015-03-19 Thread Caolán McNamara
On Thu, 2015-03-19 at 08:58 -0700, scan-ad...@coverity.com wrote:
> Hi,
> 
> Please find the latest report on new defect(s) introduced to LibreOffice 
> found with Coverity Scan.

caolanm->Tamás: cid#1288897
after f7424ed710e54bb2437a28380b03ed7c26290edc
SdTabTemplateDlg::GetRefreshedSet->SdTabTemplateDlg::RefreshInputSet has
a leaky "pRet = new SfxItemSet" in it now, what's the right fix there ?

caolanm->Bjoern: cid#1288896
after b0dae8c4c57aa50bbff7b3fd4d84476636bf1677
SwModify::ModifyBroadcast now has...
 while(aIter.m_pPosition && !aIter.m_pPosition->IsA( nType ) )
   ...
 aIter.m_pCurrent = aIter.m_pPosition;
 aIter->Modify( pOldValue, pNewValue );
which suggests that m_pCurrent can be NULL because of the m_pPosition
check, and so the deref of aIter derefs the NULL m_pCurrent ?

C.




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


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

2015-03-19 Thread Andras Timar
 source/text/scalc/01/04060181.xhp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit be139ba7c2e4eef2c529c8506820641185867ae0
Author: Andras Timar 
Date:   Thu Mar 19 18:10:21 2015 +0100

fix copypaste: a required parameter cannot be omitted

Change-Id: Ibfa020c85212d4df6d35925e3f89bab044c996a7

diff --git a/source/text/scalc/01/04060181.xhp 
b/source/text/scalc/01/04060181.xhp
index ade5291..76c0847 100644
--- a/source/text/scalc/01/04060181.xhp
+++ b/source/text/scalc/01/04060181.xhp
@@ -649,7 +649,7 @@ oldref="547">COUNTIF
 CHISQ.DIST(Number; DegreesFreedom; Cumulative)
 Number is the chi-square value of the random sample 
used to determine the error probability.
 DegreesFreedom are the degrees of freedom of the 
experiment.
-Cumulative can be 0 or False to calculate the 
probability density function. It can be any other value or True or omitted to 
calculate the cumulative distribution function.
+Cumulative can be 0 or False to calculate the 
probability density function. It can be any other value or True to calculate 
the cumulative distribution function.
 Example
 =CHISQ.DIST(3; 2; 0)  equals 
0.1115650801, the probability density function with 2 degrees of freedom, at x 
= 3.
 =CHISQ.DIST(3; 2; 1)  equals 
0.7768698399, the cumulative chi-square distribution with 2 degrees of freedom, 
at the value x = 3
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2015-03-19 Thread Andras Timar
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 45ad2b6f0fe4528995ca1eca7539857a95e4e848
Author: Andras Timar 
Date:   Thu Mar 19 18:10:21 2015 +0100

Updated core
Project: help  be139ba7c2e4eef2c529c8506820641185867ae0

diff --git a/helpcontent2 b/helpcontent2
index 325b718..be139ba 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 325b71813e476e25acbcecbb34f57ec2609ee98b
+Subproject commit be139ba7c2e4eef2c529c8506820641185867ae0
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - filter/source reportdesign/source uui/source

2015-03-19 Thread Michael Meeks
 filter/source/flash/swfwriter.cxx   |2 +-
 filter/source/flash/swfwriter.hxx   |2 +-
 filter/source/graphicfilter/eos2met/eos2met.cxx |2 +-
 filter/source/graphicfilter/eps/eps.cxx |   10 +-
 filter/source/graphicfilter/icgm/cgm.cxx|2 +-
 filter/source/graphicfilter/icgm/cgm.hxx|2 +-
 filter/source/graphicfilter/idxf/dxf2mtf.cxx|2 +-
 filter/source/graphicfilter/idxf/dxf2mtf.hxx|2 +-
 filter/source/graphicfilter/ios2met/ios2met.cxx |4 ++--
 filter/source/graphicfilter/ipict/ipict.cxx |4 ++--
 filter/source/svg/svgwriter.cxx |2 +-
 filter/source/svg/svgwriter.hxx |4 ++--
 reportdesign/source/ui/inc/ColorChanger.hxx |2 +-
 uui/source/fltdlg.cxx   |4 ++--
 14 files changed, 22 insertions(+), 22 deletions(-)

New commits:
commit 6a1532d89a2e50bc7775f5b21e0881925bce66d4
Author: Michael Meeks 
Date:   Thu Mar 19 17:16:37 2015 +

Move OutputDevice members to VclPtr: filter, reportdesign, uui.

Change-Id: Iafac8812714a0be68f5608f657c76c159f2912a0

diff --git a/filter/source/flash/swfwriter.cxx 
b/filter/source/flash/swfwriter.cxx
index ded932b..4d1ad68 100644
--- a/filter/source/flash/swfwriter.cxx
+++ b/filter/source/flash/swfwriter.cxx
@@ -100,7 +100,7 @@ Writer::Writer( sal_Int32 nTWIPWidthOutput, sal_Int32 
nTWIPHeightOutput, sal_Int
 
 Writer::~Writer()
 {
-delete mpVDev;
+mpVDev.disposeAndClear();
 delete mpSprite;
 delete mpTag;
 }
diff --git a/filter/source/flash/swfwriter.hxx 
b/filter/source/flash/swfwriter.hxx
index efa2247..8355bc4 100644
--- a/filter/source/flash/swfwriter.hxx
+++ b/filter/source/flash/swfwriter.hxx
@@ -395,7 +395,7 @@ private:
 sal_uInt16 mnWhiteBackgroundShapeId;
 sal_uInt16 mnPageButtonId;
 
-VirtualDevice*  mpVDev;
+VclPtr mpVDev;
 
 const tools::PolyPolygon* mpClipPolyPolygon;
 
diff --git a/filter/source/graphicfilter/eos2met/eos2met.cxx 
b/filter/source/graphicfilter/eos2met/eos2met.cxx
index 0fd0114..66a2773 100644
--- a/filter/source/graphicfilter/eos2met/eos2met.cxx
+++ b/filter/source/graphicfilter/eos2met/eos2met.cxx
@@ -157,7 +157,7 @@ private:
 sal_uInt32   nActBitmapPercent; // percentage of the next 
bitmap that's already written
 
 ScopedVclPtr  apDummyVDev;
-OutputDevice*   pCompDev;
+VclPtr pCompDev;
 
 com::sun::star::uno::Reference< com::sun::star::task::XStatusIndicator > 
xStatusIndicator;
 
diff --git a/filter/source/graphicfilter/eps/eps.cxx 
b/filter/source/graphicfilter/eps/eps.cxx
index 06acb9a..8832ca6 100644
--- a/filter/source/graphicfilter/eps/eps.cxx
+++ b/filter/source/graphicfilter/eps/eps.cxx
@@ -257,7 +257,7 @@ PSWriter::PSWriter()
 , mpPS(NULL)
 , pMTF(NULL)
 , pAMTF(NULL)
-, aVDev()
+, pVDev()
 , nBoundingX1(0)
 , nBoundingY1(0)
 , nBoundingX2(0)
@@ -419,13 +419,13 @@ bool PSWriter::WritePS( const Graphic& rGraphic, 
SvStream& rTargetStream, Filter
 Bitmap aBmp( rGraphic.GetBitmap() );
 pAMTF = new GDIMetaFile();
 ScopedVclPtr pTmpVDev(new VirtualDevice());
-pAMTF->Record( &aTmpVDev );
+pAMTF->Record( pTmpVDev );
 pTmpVDev->DrawBitmap( Point(), aBmp );
 pAMTF->Stop();
 pAMTF->SetPrefSize( aBmp.GetSizePixel() );
 pMTF = pAMTF;
 }
-aVDev.SetMapMode( pMTF->GetPrefMapMode() );
+pVDev->SetMapMode( pMTF->GetPrefMapMode() );
 nBoundingX1 = nBoundingY1 = 0;
 nBoundingX2 = pMTF->GetPrefSize().Width();
 nBoundingY2 = pMTF->GetPrefSize().Height();
@@ -454,7 +454,7 @@ bool PSWriter::WritePS( const Graphic& rGraphic, SvStream& 
rTargetStream, Filter
 {
 ImplWriteProlog( ( mnPreview & EPS_PREVIEW_EPSI ) ? &rGraphic : NULL );
 mnCursorPos = 0;
-ImplWriteActions( *pMTF, aVDev );
+ImplWriteActions( *pMTF, *pVDev.get() );
 ImplWriteEpilog();
 if ( mnPreview & EPS_PREVIEW_TIFF )
 {
@@ -1608,7 +1608,7 @@ void PSWriter::ImplIntersect( const tools::PolyPolygon& 
rPolyPoly )
 
 void PSWriter::ImplWriteGradient( const tools::PolyPolygon& rPolyPoly, const 
Gradient& rGradient, VirtualDevice& rVDev )
 {
-ScopedVclPtr l_pVirDev( new VirtualDevice() );
+ScopedVclPtr l_pVDev( new VirtualDevice() );
 GDIMetaFile aTmpMtf;
 l_pVDev->SetMapMode( rVDev.GetMapMode() );
 l_pVDev->AddGradientActions( rPolyPoly.GetBoundRect(), rGradient, aTmpMtf 
);
diff --git a/filter/source/graphicfilter/icgm/cgm.cxx 
b/filter/source/graphicfilter/icgm/cgm.cxx
index fdf2582..7abfe96 100644
--- a/filter/source/graphicfilter/icgm/cgm.cxx
+++ b/filter/source/graphicfilter/icgm/cgm.cxx
@@ -80,7 +80,7 @@ CGM::~CGM()
 mpGDIMetaFile->Stop();
 mpGDIMetaFile->SetPrefMapMode( MapMode() );
 mpGDIMetaFile->SetPrefSize( Size( static_cast< long >( mnOutdx ), 
static_cast< long >( mnOutdy ) ) )

Re: Summer of Code (March 17,2015)

2015-03-19 Thread Jan-Marek Glogowski
Am 19.03.2015 um 10:20 schrieb Caolán McNamara:
> On Wed, 2015-03-18 at 21:04 +0200, Efe Gürkan YALAMAN wrote:
>>
>> Somewhere in the bugzilla I think Caolan mentioned it can be
>> implemented as a tree-view because the options are in a tree structure
>> and it would fit better then a simple table. Also it solves the
>> dialog's usability issues too. I hope that will solve the
>> accessibility problems too.
> 
> Yeah, I think that's the best approach, to try and use a tree in there.
> (a SvTreeList I suppose) because the amount of elements in the list is
> just crazy, 30,000+ IIRC. Ideally on-demand loaded, e.g. tree is
> collapsed and just shows +com.sun.star, when you expand that, then only
> then is the subtree inserted, and so on. So the full 30,000+ list isn't
> actually in the widget, only the expanded portion.

Not sure if it would be better - performance wise. I guess most people
actually want to search and I'm not sure if it's better to rebuild the
tree with partial data based on the search string.

Is the tree view able to filter the content, like the Gtk+ one allows
via GtkTreeModelFilter?

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


[Libreoffice-commits] core.git: 4 commits - include/vcl solenv/bin vcl/opengl vcl/source

2015-03-19 Thread Jan Holesovsky
 include/vcl/salbtype.hxx|2 +-
 solenv/bin/unittest-failed.sh   |   31 ---
 vcl/opengl/gdiimpl.cxx  |2 +-
 vcl/source/opengl/OpenGLContext.cxx |2 +-
 4 files changed, 19 insertions(+), 18 deletions(-)

New commits:
commit f3c0085d966d4a8e2223326c52c045b7529ef320
Author: Jan Holesovsky 
Date:   Thu Mar 19 18:06:27 2015 +0100

opengl: Off-by-one in getPixel.

Change-Id: I971d94c353adcdf70e0f251d6103b1e19042687c

diff --git a/vcl/opengl/gdiimpl.cxx b/vcl/opengl/gdiimpl.cxx
index baf04ff..5ed9db6 100644
--- a/vcl/opengl/gdiimpl.cxx
+++ b/vcl/opengl/gdiimpl.cxx
@@ -1529,7 +1529,7 @@ SalColor OpenGLSalGraphicsImpl::getPixel( long nX, long 
nY )
 char pixel[3] = { 0, 0, 0 };
 
 PreDraw();
-nY = GetHeight() - nY;
+nY = GetHeight() - nY - 1;
 glReadPixels( nX, nY, 1, 1, GL_RGB, GL_UNSIGNED_BYTE, pixel);
 PostDraw();
 
commit 3637dc8a573b025106b4a652d837458c2e427d14
Author: Jan Holesovsky 
Date:   Thu Mar 19 15:56:12 2015 +0100

opengl: We need to return the GLWindow.

Fixes an unitialized read in wglDeleteContext, thanks Dr.Memory!

Change-Id: I3706b18526c48277f9b4a6801c95c26a10a86965

diff --git a/vcl/source/opengl/OpenGLContext.cxx 
b/vcl/source/opengl/OpenGLContext.cxx
index 352b630..09b0257 100644
--- a/vcl/source/opengl/OpenGLContext.cxx
+++ b/vcl/source/opengl/OpenGLContext.cxx
@@ -180,7 +180,7 @@ static LRESULT CALLBACK WndProc(HWND hwnd, UINT message, 
WPARAM wParam, LPARAM l
 }
 }
 
-int InitTempWindow(HWND *hwnd, int width, int height, const 
PIXELFORMATDESCRIPTOR& inPfd, GLWindow glWin)
+int InitTempWindow(HWND *hwnd, int width, int height, const 
PIXELFORMATDESCRIPTOR& inPfd, GLWindow& glWin)
 {
 PIXELFORMATDESCRIPTOR  pfd = inPfd;
 int  pfmt;
commit 8e0d758f852b0b4e4242b7fb2ebee8cbd17fc2f4
Author: Jan Holesovsky 
Date:   Thu Mar 19 15:35:04 2015 +0100

Hint how to use Dr.Memory for unit tests / memory checking.

Change-Id: I7df8716e90e9097336919d2a9208a549b8f7d3d0

diff --git a/solenv/bin/unittest-failed.sh b/solenv/bin/unittest-failed.sh
index 6bbd458..b1e93d5 100755
--- a/solenv/bin/unittest-failed.sh
+++ b/solenv/bin/unittest-failed.sh
@@ -17,27 +17,28 @@
 #   except in compliance with the License. You may obtain a copy of
 #   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 
-cat << EOF
-
-Error: a unit test failed, please do one of:
+ECHECKING='export DEBUGCPPUNIT=TRUE# for exception catching'
+DEBUGGING=
+MCHECKING='export VALGRIND=memcheck# for memory checking'
 
-export DEBUGCPPUNIT=TRUE# for exception catching
-EOF
 if [ "$3" = "WNT" ]; then
-cat << EOF
-export CPPUNITTRACE="\"[full path to devenv.exe]\" /debugexe" # for 
interactive debugging in Visual Studio
-EOF
+ECHECKING='export DEBUGCPPUNIT=TRUE  # 
for exception catching'
+DEBUGGING='export CPPUNITTRACE="\"[full path to devenv.exe]\" /debugexe" # 
for interactive debugging in Visual Studio'
+MCHECKING='export CPPUNITTRACE="drmemory -free_max_frames 20"# 
for memory checking (install Dr.Memory first)'
 elif [ "$3" = "MACOSX" ]; then
-cat << EOF
-export CPPUNITTRACE="lldb --" # for interactive debugging on OS X
-EOF
+DEBUGGING='export CPPUNITTRACE="lldb --"   # for interactive debugging 
on OS X'
 else
-cat << EOF
-export CPPUNITTRACE="gdb --args"# for interactive debugging on Linux
-EOF
+DEBUGGING='export CPPUNITTRACE="gdb --args"# for interactive debugging 
on Linux'
 fi
+
+# print the actual error message
 cat << EOF
-export VALGRIND=memcheck# for memory checking
+
+Error: a unit test failed, please do one of:
+
+$ECHECKING
+$DEBUGGING
+$MCHECKING
 
 and retry using: make $1Test_$2
 
commit 684aec5496f7394969743614cfda12b11c0466af
Author: Jan Holesovsky 
Date:   Tue Mar 17 12:44:36 2015 +0100

Dr.Memory complains about this...

Change-Id: I00ad970ab7cc97cfa37d6785540d6cfa825976c8

diff --git a/include/vcl/salbtype.hxx b/include/vcl/salbtype.hxx
index 40dfd8f..1dd62c2 100644
--- a/include/vcl/salbtype.hxx
+++ b/include/vcl/salbtype.hxx
@@ -406,7 +406,7 @@ inline BitmapColor& BitmapColor::Invert()
 inline sal_uInt8 BitmapColor::GetLuminance() const
 {
 DBG_ASSERT( !mbIndex, "Pixel represents index into colortable!" );
-return( (sal_uInt8) ( ( mcBlueOrIndex * 28UL + mcGreen * 151UL + mcRed * 
77UL ) >> 8UL ) );
+return (unsigned long(mcBlueOrIndex) * 28UL + unsigned long(mcGreen) * 
151UL + unsigned long(mcRed) * 77UL) >> 8;
 }
 
 inline BitmapColor& BitmapColor::IncreaseLuminance( sal_uInt8 cGreyInc )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/vclref' - 2 commits - basctl/source canvas/source chart2/source cui/source dbaccess/source sfx2/source

2015-03-19 Thread Michael Meeks
 basctl/source/basicide/basdoc.cxx|3 +++
 basctl/source/basicide/basdoc.hxx|2 +-
 canvas/source/cairo/cairo_cairo.hxx  |3 ++-
 canvas/source/cairo/cairo_canvasfont.cxx |2 +-
 canvas/source/cairo/cairo_canvashelper.cxx   |4 ++--
 canvas/source/cairo/cairo_canvashelper.hxx   |4 +++-
 canvas/source/cairo/cairo_canvashelper_text.cxx  |4 ++--
 canvas/source/cairo/cairo_devicehelper.cxx   |2 +-
 canvas/source/cairo/cairo_devicehelper.hxx   |2 +-
 canvas/source/cairo/cairo_quartz_cairo.cxx   |4 ++--
 canvas/source/cairo/cairo_quartz_cairo.hxx   |2 +-
 canvas/source/cairo/cairo_win32_cairo.cxx|4 ++--
 canvas/source/cairo/cairo_win32_cairo.hxx|2 +-
 canvas/source/cairo/cairo_xlib_cairo.cxx |4 ++--
 canvas/source/cairo/cairo_xlib_cairo.hxx |2 +-
 chart2/source/controller/dialogs/DataBrowser.cxx |4 ++--
 cui/source/inc/page.hxx  |2 +-
 cui/source/tabpages/page.cxx |3 +--
 dbaccess/source/ui/browser/dataview.cxx  |2 +-
 dbaccess/source/ui/inc/ConnectionLine.hxx|2 +-
 sfx2/source/doc/printhelper.cxx  |4 ++--
 21 files changed, 33 insertions(+), 28 deletions(-)

New commits:
commit 84648f990ff80946423ea3e48e6dfa85563e33e4
Author: Michael Meeks 
Date:   Thu Mar 19 17:21:42 2015 +

Move OutputDevice members to VclPtr: dbaccess, canvas.

Change-Id: Iae01ddcb6a0cde3be1bd79200e9177983be2f924

diff --git a/basctl/source/basicide/basdoc.cxx 
b/basctl/source/basicide/basdoc.cxx
index d11bf03..2f00c5c 100644
--- a/basctl/source/basicide/basdoc.cxx
+++ b/basctl/source/basicide/basdoc.cxx
@@ -66,7 +66,10 @@ SfxPrinter* DocShell::GetPrinter( bool bCreate )
 void DocShell::SetPrinter( SfxPrinter* pPr )
 {
 if (pPr != pPrinter.get())
+{
+pPrinter.disposeAndClear();
 pPrinter.reset(pPr);
+}
 }
 
 void DocShell::FillClass( SvGlobalName*, sal_uInt32*, OUString*, OUString*, 
OUString*, sal_Int32, bool bTemplate) const
diff --git a/basctl/source/basicide/basdoc.hxx 
b/basctl/source/basicide/basdoc.hxx
index f86bff4..f8eaf6a 100644
--- a/basctl/source/basicide/basdoc.hxx
+++ b/basctl/source/basicide/basdoc.hxx
@@ -33,7 +33,7 @@ namespace basctl
 
 class DocShell: public SfxObjectShell
 {
-boost::scoped_ptr pPrinter;
+ScopedVclPtr pPrinter;
 
 protected:
 virtual voidDraw( OutputDevice *, const JobSetup & rSetup,
diff --git a/canvas/source/cairo/cairo_cairo.hxx 
b/canvas/source/cairo/cairo_cairo.hxx
index 10c9203..c03c09b 100644
--- a/canvas/source/cairo/cairo_cairo.hxx
+++ b/canvas/source/cairo/cairo_cairo.hxx
@@ -22,6 +22,7 @@
 
 #include 
 #include 
+#include 
 
 struct SystemEnvData;
 struct BitmapSystemData;
@@ -61,7 +62,7 @@ namespace cairo {
 virtual boost::shared_ptr getSimilar( Content aContent, int 
width, int height ) const = 0;
 
 /// factory for VirDev on this surface
-virtual boost::shared_ptr createVirtualDevice() const = 
0;
+virtual VclPtr createVirtualDevice() const = 0;
 
 /// Resize the surface (possibly destroying content)
 virtual void Resize( int width, int height ) = 0;
diff --git a/canvas/source/cairo/cairo_canvasfont.cxx 
b/canvas/source/cairo/cairo_canvasfont.cxx
index 62bf3e2..be07a54 100644
--- a/canvas/source/cairo/cairo_canvasfont.cxx
+++ b/canvas/source/cairo/cairo_canvasfont.cxx
@@ -63,7 +63,7 @@ namespace cairocanvas
 // adjust to stretched/shrunk font
 if( !::rtl::math::approxEqual( rFontMatrix.m00, rFontMatrix.m11) )
 {
-OutputDevice* pOutDev( mpRefDevice->getOutputDevice() );
+VclPtr pOutDev( mpRefDevice->getOutputDevice() );
 
 if( pOutDev )
 {
diff --git a/canvas/source/cairo/cairo_canvashelper.cxx 
b/canvas/source/cairo/cairo_canvashelper.cxx
index 68e05fe..ecdfea3 100644
--- a/canvas/source/cairo/cairo_canvashelper.cxx
+++ b/canvas/source/cairo/cairo_canvashelper.cxx
@@ -83,7 +83,7 @@ namespace cairocanvas
 {
 mpSurface.reset();
 mpCairo.reset();
-mpVirtualDevice.reset();
+mpVirtualDevice.disposeAndClear();
 mpDevice = NULL;
 mpSurfaceProvider = NULL;
 }
@@ -105,7 +105,7 @@ namespace cairocanvas
 void CanvasHelper::setSurface( const SurfaceSharedPtr& pSurface, bool 
bHasAlpha )
 {
 mbHaveAlpha = bHasAlpha;
-mpVirtualDevice.reset();
+mpVirtualDevice.disposeAndClear();
 mpSurface = pSurface;
 mpCairo = pSurface->getCairo();
 }
diff --git a/canvas/source/cairo/cairo_canvashelper.hxx 
b/canvas/source/cairo/cairo_canvashelper.hxx
index f6f0099..9efbb80 100644
--- a/canvas/source/cairo/cairo_canvashelper.hxx
+++ b/canvas/source/cairo/cairo_canvashelper.hxx
@@ -28,6 +28,8 @@
 #include 
 #include 
 
+#include 
+#include 
 #include 
 
 #include "cairo_cairo.hxx"
@

[Libreoffice-commits] core.git: Branch 'feature/vclref' - include/svx starmath/inc starmath/source svx/inc svx/source

2015-03-19 Thread Michael Meeks
 include/svx/dialcontrol.hxx  |6 
 include/svx/dlgctrl.hxx  |4 
 include/svx/float3d.hxx  |2 
 include/svx/numvset.hxx  |2 
 starmath/inc/dialog.hxx  |4 
 starmath/inc/smmod.hxx   |2 
 starmath/inc/visitors.hxx|2 
 starmath/source/smmod.cxx|2 
 svx/inc/sdr/overlay/overlaymanagerbuffered.hxx   |4 
 svx/source/dialog/dlgctrl.cxx|2 
 svx/source/dialog/fntctrl.cxx|6 
 svx/source/dialog/frmsel.cxx |   31 ++--
 svx/source/dialog/svxbmpnumvalueset.cxx  |2 
 svx/source/engine3d/float3d.cxx  |2 
 svx/source/form/fmobj.cxx|4 
 svx/source/inc/fmobj.hxx |2 
 svx/source/inc/frmselimpl.hxx|2 
 svx/source/sdr/contact/viewobjectcontactofunocontrol.cxx |6 
 svx/source/sdr/overlay/overlaymanagerbuffered.cxx|   98 +++
 svx/source/sidebar/line/LineWidthValueSet.cxx|5 
 svx/source/sidebar/line/LineWidthValueSet.hxx|2 
 svx/source/svdraw/svdfmtf.cxx|   46 +++
 svx/source/svdraw/svdfmtf.hxx|   36 ++---
 23 files changed, 136 insertions(+), 136 deletions(-)

New commits:
commit 4f56dc7e40baa53f31a454e28a1eae5a7ab3dde2
Author: Michael Meeks 
Date:   Thu Mar 19 17:47:31 2015 +

Move OutputDevice members to VclPtr: starmath, svx.

Change-Id: I1a5174334f41bc319e0806b9d324be64ba6d0135

diff --git a/include/svx/dialcontrol.hxx b/include/svx/dialcontrol.hxx
index 7519a2c..e8d8a88 100644
--- a/include/svx/dialcontrol.hxx
+++ b/include/svx/dialcontrol.hxx
@@ -126,9 +126,9 @@ public:
 protected:
 struct DialControl_Impl
 {
-::boost::scoped_ptr  mpBmpEnabled;
-::boost::scoped_ptr  mpBmpDisabled;
-::boost::scoped_ptr  mpBmpBuffered;
+ScopedVclPtr mpBmpEnabled;
+ScopedVclPtr mpBmpDisabled;
+ScopedVclPtr mpBmpBuffered;
 LinkmaModifyHdl;
 VclPtr   mpLinkField;
 sal_Int32   mnLinkedFieldValueMultiplyer;
diff --git a/include/svx/dlgctrl.hxx b/include/svx/dlgctrl.hxx
index e18d4d6..a3f237f 100644
--- a/include/svx/dlgctrl.hxx
+++ b/include/svx/dlgctrl.hxx
@@ -358,8 +358,8 @@ class SdrModel;
 class SvxPreviewBase : public Control
 {
 private:
-SdrModel*   mpModel;
-VirtualDevice*  mpBufferDevice;
+SdrModel* mpModel;
+VclPtr mpBufferDevice;
 
 protected:
 void InitSettings(bool bForeground, bool bBackground);
diff --git a/include/svx/float3d.hxx b/include/svx/float3d.hxx
index f7912c7..5b47add 100644
--- a/include/svx/float3d.hxx
+++ b/include/svx/float3d.hxx
@@ -191,7 +191,7 @@ private:
 
 // Model, Page, View etc. for favourites
 FmFormModel*pModel;
-VirtualDevice*  pVDev;
+VclPtr pVDev;
 E3dView*p3DView;
 
 SfxBindings*pBindings;
diff --git a/include/svx/numvset.hxx b/include/svx/numvset.hxx
index 15161dc..7bd523e 100644
--- a/include/svx/numvset.hxx
+++ b/include/svx/numvset.hxx
@@ -52,7 +52,7 @@ class SVX_DLLPUBLIC SvxNumValueSet : public ValueSet
 sal_uInt16  nPageType;
 boolbHTMLMode;
 Rectangle   aOrgRect;
-VirtualDevice*  pVDev;
+VclPtr pVDev;
 
 com::sun::star::uno::Reference 
xFormatter;
 com::sun::star::lang::Locale aLocale;
diff --git a/starmath/inc/dialog.hxx b/starmath/inc/dialog.hxx
index 776000a..3865c5b 100644
--- a/starmath/inc/dialog.hxx
+++ b/starmath/inc/dialog.hxx
@@ -153,7 +153,7 @@ class SmFontTypeDialog : public ModalDialog
 VclPtr m_pMenuButton;
 VclPtr m_pDefaultButton;
 
-OutputDevice   *pFontListDev;
+VclPtr pFontListDev;
 
 DECL_LINK(MenuSelectHdl, Menu *);
 DECL_LINK(DefaultButtonClickHdl, Button *);
@@ -358,7 +358,7 @@ class SmSymbolDialog : public ModalDialog
 OUString   aSymbolSetName;
 SymbolPtrVec_t  aSymbolSet;
 
-OutputDevice   *pFontListDev;
+VclPtr pFontListDev;
 
 DECL_LINK(SymbolSetChangeHdl, void*);
 DECL_LINK(SymbolChangeHdl, void*);
diff --git a/starmath/inc/smmod.hxx b/starmath/inc/smmod.hxx
index a49d8ca..a2b57ec 100644
--- a/starmath/inc/smmod.hxx
+++ b/starmath/inc/smmod.hxx
@@ -90,7 +90,7 @@ class SmModule : public SfxModule, utl::ConfigurationListener
 SmConfig*pConfig;
 SmLocalizedSymbolData   *pLocSymbolData;
 SvtSysLocale*pSysLocale;
-VirtualDevice   *pVirtualDev;
+VclPtrpVirtualDev;
 
   

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

2015-03-19 Thread Tor Lillqvist
 include/vcl/salbtype.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 33d2475d9f2f01308ba4675ac99aaabb43870e0f
Author: Tor Lillqvist 
Date:   Thu Mar 19 20:13:01 2015 +0200

Revert "Dr.Memory complains about this..."

Obviously some thinko, does not compile.

This reverts commit 684aec5496f7394969743614cfda12b11c0466af.

diff --git a/include/vcl/salbtype.hxx b/include/vcl/salbtype.hxx
index 1dd62c2..40dfd8f 100644
--- a/include/vcl/salbtype.hxx
+++ b/include/vcl/salbtype.hxx
@@ -406,7 +406,7 @@ inline BitmapColor& BitmapColor::Invert()
 inline sal_uInt8 BitmapColor::GetLuminance() const
 {
 DBG_ASSERT( !mbIndex, "Pixel represents index into colortable!" );
-return (unsigned long(mcBlueOrIndex) * 28UL + unsigned long(mcGreen) * 
151UL + unsigned long(mcRed) * 77UL) >> 8;
+return( (sal_uInt8) ( ( mcBlueOrIndex * 28UL + mcGreen * 151UL + mcRed * 
77UL ) >> 8UL ) );
 }
 
 inline BitmapColor& BitmapColor::IncreaseLuminance( sal_uInt8 cGreyInc )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Tor Lillqvist
 include/vcl/salbtype.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a25fa1cb0266bb8461323b5d856c3c979d538114
Author: Tor Lillqvist 
Date:   Thu Mar 19 20:24:39 2015 +0200

MSVC and gcc are wrong, Clang was right, apparently

The 'unsigned int(expression)' syntax is *not* correct, says
http://en.cppreference.com/w/cpp/language/explicit_cast, but
apparently MSVC and gcc don't mind. Clang doesn't like it, though.
Re-introduce kendy's change but use static_cast instead of functional
cast.

This reverts commit 33d2475d9f2f01308ba4675ac99aaabb43870e0f

Change-Id: I799cce13e9bebf1a375243c294c4fa8788502ec6

diff --git a/include/vcl/salbtype.hxx b/include/vcl/salbtype.hxx
index 40dfd8f..d5bbc73 100644
--- a/include/vcl/salbtype.hxx
+++ b/include/vcl/salbtype.hxx
@@ -406,7 +406,7 @@ inline BitmapColor& BitmapColor::Invert()
 inline sal_uInt8 BitmapColor::GetLuminance() const
 {
 DBG_ASSERT( !mbIndex, "Pixel represents index into colortable!" );
-return( (sal_uInt8) ( ( mcBlueOrIndex * 28UL + mcGreen * 151UL + mcRed * 
77UL ) >> 8UL ) );
+return (static_cast(mcBlueOrIndex) * 28UL + 
static_cast(mcGreen) * 151UL + static_cast(mcRed) 
* 77UL) >> 8;
 }
 
 inline BitmapColor& BitmapColor::IncreaseLuminance( sal_uInt8 cGreyInc )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Enes Ateş - licence statement

2015-03-19 Thread Enes Ateş
All of my past & future contributions to LibreOffice may be
   licensed under the MPLv2/LGPLv3+ dual license.



-- 

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


[Libreoffice-commits] core.git: Branch 'feature/vclref' - filter/source sw/qa sw/source vcl/workben

2015-03-19 Thread Michael Meeks
 filter/source/graphicfilter/icgm/cgm.hxx |1 +
 sw/qa/tiledrendering/tiledrendering.cxx  |6 +++---
 sw/source/core/frmedt/fecopy.cxx |2 +-
 sw/source/core/layout/paintfrm.cxx   |8 
 vcl/workben/vcldemo.cxx  |   30 +++---
 5 files changed, 24 insertions(+), 23 deletions(-)

New commits:
commit fe7412b0560366f7e0f3927ba9966fbc99e98cf6
Author: Michael Meeks 
Date:   Thu Mar 19 19:07:14 2015 +

vclptr: misc. bug fixing.

Change-Id: Id56188c0f72b74bc9ce6eed558a8339d4175d628

diff --git a/filter/source/graphicfilter/icgm/cgm.hxx 
b/filter/source/graphicfilter/icgm/cgm.hxx
index c6c0fa6..e851819 100644
--- a/filter/source/graphicfilter/icgm/cgm.hxx
+++ b/filter/source/graphicfilter/icgm/cgm.hxx
@@ -28,6 +28,7 @@
 
 #include 
 #include 
+#include 
 #include "cgmtypes.hxx"
 
 class   Graphic;
diff --git a/sw/qa/tiledrendering/tiledrendering.cxx 
b/sw/qa/tiledrendering/tiledrendering.cxx
index 73845bd..82ffe0c 100644
--- a/sw/qa/tiledrendering/tiledrendering.cxx
+++ b/sw/qa/tiledrendering/tiledrendering.cxx
@@ -130,14 +130,14 @@ IMPL_LINK ( TiledRenderingDialog, RenderHdl, Button *, 
EMPTYARG )
 //
 // SystemGraphicsData aData;
 // [setup the aData]
-// VirtualDevice aDevice(&aData, [color depth]);
+// VirtualDevice pDevice(&aData, [color depth]);
 ScopedVclPtr< VirtualDevice > pDevice(new VirtualDevice());
 
 // paint to it
 pViewShell->PaintTile(*pDevice.get(), contextWidth, contextHeight, 
tilePosX, tilePosY, tileWidth, tileHeight);
 
-// copy the aDevice content to mpImage
-Bitmap aBitmap(pDevice->GetBitmap(aDevice->PixelToLogic(Point(0,0)), 
pDevice->PixelToLogic(Size(contextWidth, contextHeight;
+// copy the pDevice content to mpImage
+Bitmap aBitmap(pDevice->GetBitmap(pDevice->PixelToLogic(Point(0,0)), 
pDevice->PixelToLogic(Size(contextWidth, contextHeight;
 mpImage->SetImage(Image(aBitmap));
 
 // update the dialog size
diff --git a/sw/source/core/frmedt/fecopy.cxx b/sw/source/core/frmedt/fecopy.cxx
index 69dc7e8..6274e97 100644
--- a/sw/source/core/frmedt/fecopy.cxx
+++ b/sw/source/core/frmedt/fecopy.cxx
@@ -1241,7 +1241,7 @@ bool SwFEShell::GetDrawObjGraphic( sal_uLong nFmt, 
Graphic& rGrf ) const
 
 GDIMetaFile aMtf;
 aMtf.Record( pVirtDev.get() );
-aGrf.Draw( &aVirtDev, aPt, aSz );
+aGrf.Draw( pVirtDev, aPt, aSz );
 aMtf.Stop();
 aMtf.SetPrefMapMode( aTmp );
 aMtf.SetPrefSize( aSz );
diff --git a/sw/source/core/layout/paintfrm.cxx 
b/sw/source/core/layout/paintfrm.cxx
index b408f36..ecdc339 100644
--- a/sw/source/core/layout/paintfrm.cxx
+++ b/sw/source/core/layout/paintfrm.cxx
@@ -7675,7 +7675,7 @@ Graphic SwFlyFrmFmt::MakeGraphic( ImageMap* pMap )
 
 vcl::Window *pWin = pSh->GetWin();
 sal_uInt16 nZoom = pSh->GetViewOptions()->GetZoom();
-::SetOutDevAndWin( pSh, &aDev, 0, 100 );
+::SetOutDevAndWin( pSh, pDev, 0, 100 );
 gProp.bSFlyMetafile = true;
 gProp.pSFlyMetafileOut = pWin;
 
@@ -7693,15 +7693,15 @@ Graphic SwFlyFrmFmt::MakeGraphic( ImageMap* pMap )
 pImp->PaintLayer( pIDDMA->GetHellId(), 0, aOut, &aPageBackgrdColor,
   pFlyPage->IsRightToLeft(),
   &aSwRedirector );
-gProp.pSLines->PaintLines( &aDev, gProp );
+gProp.pSLines->PaintLines( pDev, gProp );
 if ( pFly->IsFlyInCntFrm() )
 pFly->Paint( aOut );
-gProp.pSLines->PaintLines( &aDev, gProp );
+gProp.pSLines->PaintLines( pDev, gProp );
 // OD 30.08.2002 #102450# - add 3rd parameter
 pImp->PaintLayer( pIDDMA->GetHeavenId(), 0, aOut, &aPageBackgrdColor,
   pFlyPage->IsRightToLeft(),
   &aSwRedirector );
-gProp.pSLines->PaintLines( &aDev, gProp );
+gProp.pSLines->PaintLines( pDev, gProp );
 DELETEZ( gProp.pSLines );
 gProp.pSFlyOnlyDraw = 0;
 
diff --git a/vcl/workben/vcldemo.cxx b/vcl/workben/vcldemo.cxx
index 8936553..0cd32fe 100644
--- a/vcl/workben/vcldemo.cxx
+++ b/vcl/workben/vcldemo.cxx
@@ -807,18 +807,18 @@ public:
 // Compositing onto 2x colors beyond our control
 ScopedVclPtr< VirtualDevice > aWhite(new VirtualDevice());
 ScopedVclPtr< VirtualDevice > aBlack(new VirtualDevice());
-aWhite.SetOutputSizePixel(aSrc.GetSizePixel());
-aWhite.SetBackground(Wallpaper(COL_WHITE));
-aWhite.Erase();
-aBlack.SetOutputSizePixel(aSrc.GetSizePixel());
-aBlack.SetBackground(Wallpaper(COL_BLACK));
-aBlack.Erase();
-aWhite.DrawBitmapEx(Point(), aSrc);
-aBlack.DrawBitmapEx(Point

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

2015-03-19 Thread Jan Holesovsky
 vcl/inc/win/salgdi.h |3 +++
 vcl/win/source/gdi/salgdi.cxx|3 ++-
 vcl/win/source/gdi/salnativewidgets-luna.cxx |   10 ++
 3 files changed, 11 insertions(+), 5 deletions(-)

New commits:
commit 5b0f3cf836054da94968eddffc2647b50236739e
Author: Jan Holesovsky 
Date:   Thu Mar 19 20:24:07 2015 +0100

opengl dr.memory: DC that contains selected object being deleted.

Change-Id: I3cfb09eacfa129a92dace47fd26ccf6e5d1a6756

diff --git a/vcl/inc/win/salgdi.h b/vcl/inc/win/salgdi.h
index 287daf4..e956e0f 100644
--- a/vcl/inc/win/salgdi.h
+++ b/vcl/inc/win/salgdi.h
@@ -155,6 +155,9 @@ private:
 /// DIBSection that we use for the GDI drawing, and later obtain.
 HBITMAP mhBitmap;
 
+/// Return the previous bitmap to undo the SelectObject.
+HBITMAP mhOrigBitmap;
+
 /// DIBSection data.
 sal_uInt32 *mpData;
 
diff --git a/vcl/win/source/gdi/salgdi.cxx b/vcl/win/source/gdi/salgdi.cxx
index fc74d05..ff74b0f 100644
--- a/vcl/win/source/gdi/salgdi.cxx
+++ b/vcl/win/source/gdi/salgdi.cxx
@@ -582,13 +582,14 @@ OpenGLCompatibleDC::OpenGLCompatibleDC(SalGraphics 
&rGraphics, int x, int y, int
 
 mhBitmap = WinSalVirtualDevice::ImplCreateVirDevBitmap(mhCompatibleDC, 
width, height, 32, reinterpret_cast(&mpData));
 
-SelectObject(mhCompatibleDC, mhBitmap);
+mhOrigBitmap = (HBITMAP) SelectObject(mhCompatibleDC, mhBitmap);
 }
 
 OpenGLCompatibleDC::~OpenGLCompatibleDC()
 {
 if (mpImpl)
 {
+SelectObject(mhCompatibleDC, mhOrigBitmap);
 DeleteObject(mhBitmap);
 DeleteDC(mhCompatibleDC);
 }
commit a175977e1eb7fe63b2314dcfb3944e87889af455
Author: Jan Holesovsky 
Date:   Thu Mar 19 20:11:31 2015 +0100

dr.memory: Deleting a drawing object that is selected into DC.

Change-Id: I2b04152896cdd231cc9be4ab008bc57a65c840da

diff --git a/vcl/win/source/gdi/salnativewidgets-luna.cxx 
b/vcl/win/source/gdi/salnativewidgets-luna.cxx
index 9b690db..aedbf89 100644
--- a/vcl/win/source/gdi/salnativewidgets-luna.cxx
+++ b/vcl/win/source/gdi/salnativewidgets-luna.cxx
@@ -475,12 +475,13 @@ static void impl_drawAeroToolbar( HDC hDC, RECT rc, bool 
bHorizontal )
 
 // and a darker horizontal line under that
 HPEN hpen = CreatePen( PS_SOLID, 1, RGB( 0xb0, 0xb0, 0xb0 ) );
-SelectObject( hDC, hpen );
+HPEN hOrigPen = (HPEN) SelectObject(hDC, hpen);
 
 MoveToEx( hDC, rc.left, gradient_bottom, NULL );
 LineTo( hDC, rc.right, gradient_bottom );
 
-DeleteObject( hpen );
+SelectObject(hDC, hOrigPen);
+DeleteObject(hpen);
 }
 else
 {
@@ -500,12 +501,13 @@ static void impl_drawAeroToolbar( HDC hDC, RECT rc, bool 
bHorizontal )
 from_y = to_y = rc.top;
 
 HPEN hpen = CreatePen( PS_SOLID, 1, RGB( 0xb0, 0xb0, 0xb0 ) );
-SelectObject( hDC, hpen );
+HPEN hOrigPen = (HPEN) SelectObject(hDC, hpen);
 
 MoveToEx( hDC, from_x, from_y, NULL );
 LineTo( hDC, to_x, to_y );
 
-DeleteObject( hpen );
+SelectObject(hDC, hOrigPen);
+DeleteObject(hpen);
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: svx/Executable_pixelctl.mk vcl/Executable_mtfdemo.mk vcl/Executable_outdevgrind.mk vcl/Executable_svdemo.mk vcl/Executable_svpclient.mk vcl/Executable_svptest.mk vcl/Ex

2015-03-19 Thread Tor Lillqvist
 svx/Executable_pixelctl.mk  |1 -
 vcl/Executable_mtfdemo.mk   |1 -
 vcl/Executable_outdevgrind.mk   |1 -
 vcl/Executable_svdemo.mk|1 -
 vcl/Executable_svpclient.mk |1 -
 vcl/Executable_svptest.mk   |1 -
 vcl/Executable_tdefilepicker.mk |1 -
 vcl/Executable_vcldemo.mk   |1 -
 vcl/Library_vclplug_tde.mk  |1 -
 9 files changed, 9 deletions(-)

New commits:
commit 2847c520c3449d60dcd3f8b525fba006bc1d9e01
Author: Tor Lillqvist 
Date:   Thu Mar 19 21:41:54 2015 +0200

There are no include files in solenv/inc

Change-Id: I69f20c3632c0bee7a3189aaa5a96bd9fff2da222

diff --git a/svx/Executable_pixelctl.mk b/svx/Executable_pixelctl.mk
index d019d0c..3140af4 100644
--- a/svx/Executable_pixelctl.mk
+++ b/svx/Executable_pixelctl.mk
@@ -21,7 +21,6 @@ $(eval $(call gb_Executable_set_include,pixelctl,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
 -I$(SRCDIR)/svx/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,pixelctl,\
diff --git a/vcl/Executable_mtfdemo.mk b/vcl/Executable_mtfdemo.mk
index c52980b..52af4a3 100644
--- a/vcl/Executable_mtfdemo.mk
+++ b/vcl/Executable_mtfdemo.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,mtfdemo,boost_headers))
 $(eval $(call gb_Executable_set_include,mtfdemo,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,mtfdemo,\
diff --git a/vcl/Executable_outdevgrind.mk b/vcl/Executable_outdevgrind.mk
index f1a822f..7b60d5e 100644
--- a/vcl/Executable_outdevgrind.mk
+++ b/vcl/Executable_outdevgrind.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,outdevgrind,boost_headers))
 $(eval $(call gb_Executable_set_include,outdevgrind,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,outdevgrind,\
diff --git a/vcl/Executable_svdemo.mk b/vcl/Executable_svdemo.mk
index 8937129..0a958a2 100644
--- a/vcl/Executable_svdemo.mk
+++ b/vcl/Executable_svdemo.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,svdemo,boost_headers))
 $(eval $(call gb_Executable_set_include,svdemo,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,svdemo,\
diff --git a/vcl/Executable_svpclient.mk b/vcl/Executable_svpclient.mk
index 5f3ba57..9f92ec5 100644
--- a/vcl/Executable_svpclient.mk
+++ b/vcl/Executable_svpclient.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,svpclient,boost_headers))
 $(eval $(call gb_Executable_set_include,svpclient,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,svpclient,\
diff --git a/vcl/Executable_svptest.mk b/vcl/Executable_svptest.mk
index 35c9806..c373aa6 100644
--- a/vcl/Executable_svptest.mk
+++ b/vcl/Executable_svptest.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,svptest,boost_headers))
 $(eval $(call gb_Executable_set_include,svptest,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,svptest,\
diff --git a/vcl/Executable_tdefilepicker.mk b/vcl/Executable_tdefilepicker.mk
index 9811e6d..fcbf55f 100644
--- a/vcl/Executable_tdefilepicker.mk
+++ b/vcl/Executable_tdefilepicker.mk
@@ -18,7 +18,6 @@ $(eval $(call gb_Executable_add_defs,tdefilepicker,\
 $(eval $(call gb_Executable_set_include,tdefilepicker,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 -I$(SRCDIR)/vcl/inc/unx/kde \
 ))
 
diff --git a/vcl/Executable_vcldemo.mk b/vcl/Executable_vcldemo.mk
index da9d9e6..5fa49e6 100644
--- a/vcl/Executable_vcldemo.mk
+++ b/vcl/Executable_vcldemo.mk
@@ -20,7 +20,6 @@ $(eval $(call 
gb_Executable_use_external,vcldemo,boost_headers))
 $(eval $(call gb_Executable_set_include,vcldemo,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 ))
 
 $(eval $(call gb_Executable_use_libraries,vcldemo,\
diff --git a/vcl/Library_vclplug_tde.mk b/vcl/Library_vclplug_tde.mk
index d05780b..af5da99d 100644
--- a/vcl/Library_vclplug_tde.mk
+++ b/vcl/Library_vclplug_tde.mk
@@ -14,7 +14,6 @@ $(eval $(call gb_Library_Library,vclplug_tde))
 $(eval $(call gb_Library_set_include,vclplug_tde,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
--I$(SRCDIR)/solenv/inc \
 -I$(SRCDIR)/vcl/inc/unx/kde \
 -I$(SRCDIR)/include/shell \
 ))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: connectivity/Library_tdeab1.mk connectivity/Library_tdeabdrv1.mk shell/Library_tdebe.mk vcl/Library_vclplug_tde.mk

2015-03-19 Thread Tor Lillqvist
 connectivity/Library_tdeab1.mk|1 -
 connectivity/Library_tdeabdrv1.mk |1 -
 shell/Library_tdebe.mk|5 -
 vcl/Library_vclplug_tde.mk|1 -
 4 files changed, 8 deletions(-)

New commits:
commit 3a699403f1c6d979fc61f4bc14e5c1f740c8aab0
Author: Tor Lillqvist 
Date:   Thu Mar 19 21:54:24 2015 +0200

No need to use -I$(SRCDIR)/include/shell

We always use -I$(SRCDIR)/include, and the #include statements use
.

Change-Id: I62d32b3a1bd0840f3a4bb149c5552c2aed98cb56

diff --git a/connectivity/Library_tdeab1.mk b/connectivity/Library_tdeab1.mk
index 8243280..298ed4c 100644
--- a/connectivity/Library_tdeab1.mk
+++ b/connectivity/Library_tdeab1.mk
@@ -18,7 +18,6 @@ $(eval $(call 
gb_Library_set_componentfile,tdeab1,connectivity/source/drivers/ka
 
 $(eval $(call gb_Library_set_include,tdeab1,\
-I$(SRCDIR)/connectivity/source/inc \
--I$(SRCDIR)/include/shell \
$$(INCLUDE) \
 ))
 
diff --git a/connectivity/Library_tdeabdrv1.mk 
b/connectivity/Library_tdeabdrv1.mk
index 1398d9f..8301e5f 100644
--- a/connectivity/Library_tdeabdrv1.mk
+++ b/connectivity/Library_tdeabdrv1.mk
@@ -12,7 +12,6 @@ $(eval $(call gb_Library_Library,tdeabdrv1))
 
 $(eval $(call gb_Library_set_include,tdeabdrv1,\
-I$(SRCDIR)/connectivity/source/inc \
-   -I$(SRCDIR)/include/shell \
$$(INCLUDE) \
-I$(WORKDIR)/YaccTarget/connectivity/source/parse \
 ))
diff --git a/shell/Library_tdebe.mk b/shell/Library_tdebe.mk
index 3b516d8..dc5b735 100644
--- a/shell/Library_tdebe.mk
+++ b/shell/Library_tdebe.mk
@@ -9,11 +9,6 @@
 
 $(eval $(call gb_Library_Library,tdebe1))
 
-$(eval $(call gb_Library_set_include,tdebe1,\
-$$(INCLUDE) \
--I$(SRCDIR)/include/shell \
-))
-
 $(eval $(call gb_Library_use_sdk_api,tdebe1))
 
 $(eval $(call gb_Library_use_externals,tdebe1,\
diff --git a/vcl/Library_vclplug_tde.mk b/vcl/Library_vclplug_tde.mk
index af5da99d..ad1ed40 100644
--- a/vcl/Library_vclplug_tde.mk
+++ b/vcl/Library_vclplug_tde.mk
@@ -15,7 +15,6 @@ $(eval $(call gb_Library_set_include,vclplug_tde,\
 $$(INCLUDE) \
 -I$(SRCDIR)/vcl/inc \
 -I$(SRCDIR)/vcl/inc/unx/kde \
--I$(SRCDIR)/include/shell \
 ))
 
 $(eval $(call gb_Library_set_include,vclplug_tde,\
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: New Defects reported by Coverity Scan for LibreOffice

2015-03-19 Thread Bjoern Michaelsen
Hi,

On Thu, Mar 19, 2015 at 05:09:54PM +, Caolán McNamara wrote:
> caolanm->Bjoern: cid#1288896
> after b0dae8c4c57aa50bbff7b3fd4d84476636bf1677

Yes, thats me. Im refactoring the whole SwClient/SwModify mess and will take
care of those cids. I have some more commits that I hope I can push tonight, 
with
that it should be out of the roughest part.

Best,

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


Re: Summer of Code (March 17,2015)

2015-03-19 Thread Caolán McNamara
On Thu, 2015-03-19 at 18:12 +0100, Jan-Marek Glogowski wrote:
> Am 19.03.2015 um 10:20 schrieb Caolán McNamara:
> > On Wed, 2015-03-18 at 21:04 +0200, Efe Gürkan YALAMAN wrote:
> >>
> >> Somewhere in the bugzilla I think Caolan mentioned it can be
> >> implemented as a tree-view because the options are in a tree structure
> >> and it would fit better then a simple table. Also it solves the
> >> dialog's usability issues too. I hope that will solve the
> >> accessibility problems too.
> > 
> > Yeah, I think that's the best approach, to try and use a tree in there.
> > (a SvTreeList I suppose) because the amount of elements in the list is
> > just crazy, 30,000+ IIRC. Ideally on-demand loaded, e.g. tree is
> > collapsed and just shows +com.sun.star, when you expand that, then only
> > then is the subtree inserted, and so on. So the full 30,000+ list isn't
> > actually in the widget, only the expanded portion.
> 
> Not sure if it would be better - performance wise. I guess most people
> actually want to search and I'm not sure if it's better to rebuild the
> tree with partial data based on the search string.

FWIW, the basctl TreeListBox is an example of what I had in mind, see
the RequestingChildren and so forth there for something similar.

The last time I had a look at the a11y performance there is an insanely
slow a11y-caused iterate-over-the-whole-view to get to the right index
loop triggered by every insert into the model. I attach the experimental
patch of
http://lists.freedesktop.org/archives/libreoffice/2014-January/059081.htmlhttp://lists.freedesktop.org/archives/libreoffice/2014-January/059081.html
that highlights that hot-spot.

> Is the tree view able to filter the content, like the Gtk+ one allows
> via GtkTreeModelFilter?

I don't think so, well not on a quick look anyway, mostly seems that
SvTreeList is the GtkTreeModel-alike model and SvTreeListBox is the
GtkTreeView-alike view.

C.
diff --git a/svtools/source/contnr/svtabbx.cxx b/svtools/source/contnr/svtabbx.cxx
index adb759c..f01ff64 100644
--- a/svtools/source/contnr/svtabbx.cxx
+++ b/svtools/source/contnr/svtabbx.cxx
@@ -436,9 +436,20 @@ OUString SvTabListBox::GetTabEntryText( sal_uLong nPos, sal_uInt16 nCol ) const
 
 SvTreeListEntry* SvTabListBox::GetEntryOnPos( sal_uLong _nEntryPos ) const
 {
+static sal_uLong nCachedEntryPos;
+static sal_uLong nKnownStartIndex;
+static sal_uLong nKnownStartPos;
+
 SvTreeListEntry* pEntry = NULL;
-sal_uLong i, nPos = 0, nCount = GetLevelChildCount( NULL );
-for ( i = 0; i < nCount; ++i )
+sal_uLong i = 0, nPos = 0, nCount = GetLevelChildCount( NULL );
+
+if (_nEntryPos >= nCachedEntryPos)
+{
+i = nKnownStartIndex;
+nPos = nKnownStartPos;
+}
+
+while (i < nCount)
 {
 SvTreeListEntry* pParent = GetEntry(i);
 if ( nPos == _nEntryPos )
@@ -453,8 +464,12 @@ SvTreeListEntry* SvTabListBox::GetEntryOnPos( sal_uLong _nEntryPos ) const
 if ( pEntry )
 break;
 }
+++i;
 }
 
+nCachedEntryPos = _nEntryPos;
+nKnownStartIndex = i;
+nKnownStartPos = nPos;
 return pEntry;
 }
 
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2015-03-19 Thread Julien Nabet
 xmloff/source/style/undlihdl.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 17f741678ea58841ba463465f22eeb6b83c40bc3
Author: Julien Nabet 
Date:   Sun Mar 8 21:32:39 2015 +0100

Related tdf#50133: Wrong underlining in Reports

By running some tests, I noticed double wave didn't appear when opening 
report in non edit mode.
First you must know that to indicate "double wave", xml contains these 2 
information:
style:text-underline-style="wave" style:text-underline-type="double"

The problem is the reading order of the parser doesn't seem the same each 
time.
So if it reads style then type, it's ok thanks to this part:
132 case awt::FontUnderline::DOUBLE:
133 // A double line style has priority over a bold 
line style,
134 // but not over the line style itself.
135 switch( eUnderline )
136 {
137 case awt::FontUnderline::SINGLE:
138 case awt::FontUnderline::BOLD:
139 break;
140 case awt::FontUnderline::WAVE:
141 case awt::FontUnderline::BOLDWAVE:
142 eNewUnderline = awt::FontUnderline::DOUBLEWAVE;
143 break;

but if the parser read type in the first place then style, the case 
FontUnderline "DOUBLEWAVE" wasn't managed

Change-Id: Ie090507898e537827eebaae157dfac248baa43f1
Reviewed-on: https://gerrit.libreoffice.org/14801
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 
(cherry picked from commit 61a3ace9f42a37640b1ec18fad776ba7c1f61714)
Reviewed-on: https://gerrit.libreoffice.org/14914

diff --git a/xmloff/source/style/undlihdl.cxx b/xmloff/source/style/undlihdl.cxx
index 268d042..2832d11 100644
--- a/xmloff/source/style/undlihdl.cxx
+++ b/xmloff/source/style/undlihdl.cxx
@@ -233,7 +233,9 @@ bool XMLUnderlineStylePropHdl::importXML( const OUString& 
rStrImpValue, uno::Any
 eNewUnderline = awt::FontUnderline::BOLDDASHDOTDOT;
 break;
 case awt::FontUnderline::WAVE:
-if( awt::FontUnderline::BOLD == eUnderline )
+if( awt::FontUnderline::DOUBLE == eUnderline )
+eNewUnderline = awt::FontUnderline::DOUBLEWAVE;
+else if( awt::FontUnderline::BOLD == eUnderline )
 eNewUnderline = awt::FontUnderline::BOLDWAVE;
 break;
 case awt::FontUnderline::SMALLWAVE:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Caolán McNamara
 sc/qa/unit/ucalc_pivottable.cxx  |3 ++-
 sfx2/source/dialog/tabdlg.cxx|2 +-
 sw/source/core/txtnode/ndtxt.cxx |3 ++-
 3 files changed, 5 insertions(+), 3 deletions(-)

New commits:
commit 4dce503b2777e589d713f0fca2604a9a7916813b
Author: Caolán McNamara 
Date:   Thu Mar 19 16:56:23 2015 +

coverity#1288895 Explicit null dereferenced

Change-Id: I3b8fe2e1aec18b41b1d4553bca0fa6c52d5cfde6

diff --git a/sfx2/source/dialog/tabdlg.cxx b/sfx2/source/dialog/tabdlg.cxx
index cdb6a1b..4cea451 100644
--- a/sfx2/source/dialog/tabdlg.cxx
+++ b/sfx2/source/dialog/tabdlg.cxx
@@ -1355,7 +1355,7 @@ void SfxTabDialog::SetInputSet( const SfxItemSet* pInSet )
 delete pSet;
 pSet = pInSet ? new SfxItemSet(*pInSet) : 0;
 
-if ( !bSet && !pExampleSet && !pOutSet )
+if (!bSet && !pExampleSet && !pOutSet && pSet)
 {
 pExampleSet = new SfxItemSet( *pSet );
 pOutSet = new SfxItemSet( *pSet->GetPool(), pSet->GetRanges() );
commit 3f6589a649cf6d411391409c3ed637e6b641ba80
Author: Caolán McNamara 
Date:   Thu Mar 19 16:53:22 2015 +

coverity#1288894 Dereference after null check, golden

regression from

commit 553a058ec877e5ea858dbc47eec456033a6323ab
Date:   Sat Mar 14 20:43:43 2015 +0100

core: dont use 'shortcut' that is longer than the original

Change-Id: I0154e484a4f4d6ce6672ddf35d1869157a61f71e

Change-Id: I4159a5caf492900bcfb863bc0bc5cf59a47a311c

diff --git a/sw/source/core/txtnode/ndtxt.cxx b/sw/source/core/txtnode/ndtxt.cxx
index 924c117..c630288 100644
--- a/sw/source/core/txtnode/ndtxt.cxx
+++ b/sw/source/core/txtnode/ndtxt.cxx
@@ -312,7 +312,8 @@ static void lcl_ChangeFtnRef( SwTxtNode &rNode )
 {
 if( !pFrm )
 {
-if(!SwIterator( rNode ).First())
+pFrm = SwIterator(rNode).First();
+if (!pFrm)
 return;
 }
 SwTxtFtn *pAttr = static_cast(pHt);
commit 19fd963db6684a69ac13101c056a135d1b96687b
Author: Caolán McNamara 
Date:   Thu Mar 19 16:49:48 2015 +

coverity#1209001 Unchecked return value

Change-Id: I1d2615ed478417a965ebddb37539ce4697b8e92f

diff --git a/sc/qa/unit/ucalc_pivottable.cxx b/sc/qa/unit/ucalc_pivottable.cxx
index 07b04ca..932efd6 100644
--- a/sc/qa/unit/ucalc_pivottable.cxx
+++ b/sc/qa/unit/ucalc_pivottable.cxx
@@ -2166,7 +2166,8 @@ void Test::testFuncGETPIVOTDATA()
 pDPObj = createDPFromRange(m_pDoc, aDataRange, aFields, 
SAL_N_ELEMENTS(aFields), false);
 }
 
-pDPs->InsertNewTable(pDPObj);
+bSuccess = pDPs->InsertNewTable(pDPObj);
+CPPUNIT_ASSERT_MESSAGE("InsertNewTable failed", bSuccess);
 aOutRange = refresh(pDPObj);
 
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'aoo/trunk' - vcl/Module_vcl.mk

2015-03-19 Thread Kay Schenk
 vcl/Module_vcl.mk |8 +---
 1 file changed, 5 insertions(+), 3 deletions(-)

New commits:
commit 1b4025b58e01bcc3151d2b32ed4a3a99c7329245
Author: Kay Schenk 
Date:   Thu Mar 19 21:37:55 2015 +

#126118# Fix malformed if statements in makefile.

diff --git a/vcl/Module_vcl.mk b/vcl/Module_vcl.mk
index 3779f68..6272197 100644
--- a/vcl/Module_vcl.mk
+++ b/vcl/Module_vcl.mk
@@ -38,17 +38,19 @@ $(eval $(call gb_Module_add_targets,vcl,\
 Library_desktop_detector \
 ))
 
-ifneq ($(ENABLE_GTK),)
+
+ifeq ($(ENABLE_GTK),TRUE)
 $(eval $(call gb_Module_add_targets,vcl,\
 Library_vclplug_gtk \
 ))
 endif
-ifneq ($(ENABLE_KDE),)
+
+ifeq ($(ENABLE_KDE),TRUE)
 $(eval $(call gb_Module_add_targets,vcl,\
 Library_vclplug_kde \
 ))
 endif
-ifneq ($(ENABLE_KDE4),)
+ifeq ($(ENABLE_KDE4),TRUE)
 $(eval $(call gb_Module_add_targets,vcl,\
 Library_vclplug_kde4 \
 ))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: translations

2015-03-19 Thread Andras Timar
 translations |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f54bc1365de88efc82ddfee86f4a24bc132d8e9c
Author: Andras Timar 
Date:   Thu Mar 19 23:08:51 2015 +0100

Updated core
Project: translations  6a11450be8a19648f3f76d7b9a098a9ef7c4d76e

diff --git a/translations b/translations
index add7a7e..6a11450 16
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit add7a7e0967a3fc04666ba061d81d1ba9b326554
+Subproject commit 6a11450be8a19648f3f76d7b9a098a9ef7c4d76e
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] translations.git: source/sl

2015-03-19 Thread Andras Timar
 source/sl/basctl/uiconfig/basicide/ui.po   |   40 
 source/sl/cui/source/dialogs.po|   24 
 source/sl/cui/source/options.po|   45 
 source/sl/cui/uiconfig/ui.po   |  516 +-
 source/sl/dbaccess/source/ui/dlg.po|   10 
 source/sl/dbaccess/uiconfig/ui.po  |  359 -
 source/sl/desktop/uiconfig/ui.po   |4 
 source/sl/extensions/source/bibliography.po|   10 
 source/sl/extensions/source/scanner.po |  254 -
 source/sl/extensions/uiconfig.po   |  142 
 source/sl/filter/source/config/fragments/filters.po|  554 +-
 source/sl/filter/source/config/fragments/types.po  |   29 
 source/sl/filter/source/pdf.po |   20 
 source/sl/filter/uiconfig/ui.po|   27 
 source/sl/formula/source/core/resource.po  |  149 
 source/sl/fpicker/uiconfig/ui.po   |   13 
 source/sl/framework/source/classes.po  |4 
 source/sl/helpcontent2/source/text/sbasic/guide.po |   12 
 source/sl/helpcontent2/source/text/sbasic/shared/01.po |   20 
 source/sl/helpcontent2/source/text/scalc.po|   22 
 source/sl/helpcontent2/source/text/scalc/01.po | 2114 
++
 source/sl/helpcontent2/source/text/scalc/04.po |   16 
 source/sl/helpcontent2/source/text/scalc/05.po |  254 +
 source/sl/helpcontent2/source/text/scalc/guide.po  |   24 
 source/sl/helpcontent2/source/text/sdraw/guide.po  |8 
 source/sl/helpcontent2/source/text/shared/00.po|   36 
 source/sl/helpcontent2/source/text/shared/01.po|  324 -
 source/sl/helpcontent2/source/text/shared/02.po|   36 
 source/sl/helpcontent2/source/text/shared/explorer/database.po |   14 
 source/sl/helpcontent2/source/text/shared/guide.po |   50 
 source/sl/helpcontent2/source/text/shared/optionen.po  |   56 
 source/sl/helpcontent2/source/text/simpress/01.po  |   72 
 source/sl/helpcontent2/source/text/simpress/04.po  |   16 
 source/sl/helpcontent2/source/text/swriter.po  |   20 
 source/sl/helpcontent2/source/text/swriter/01.po   |  100 
 source/sl/helpcontent2/source/text/swriter/02.po   |   48 
 source/sl/helpcontent2/source/text/swriter/04.po   |   14 
 source/sl/helpcontent2/source/text/swriter/guide.po|   36 
 source/sl/helpcontent2/source/text/swriter/librelogo.po|   12 
 source/sl/officecfg/registry/data/org/openoffice/Office.po |   54 
 source/sl/officecfg/registry/data/org/openoffice/Office/UI.po  |  455 +-
 source/sl/sc/source/ui/StatisticsDialogs.po|4 
 source/sl/sc/source/ui/drawfunc.po |   28 
 source/sl/sc/source/ui/src.po  |  861 +---
 source/sl/sc/uiconfig/scalc/ui.po  |  449 --
 source/sl/scp2/source/accessories.po   |   16 
 source/sl/scp2/source/impress.po   |   22 
 source/sl/scp2/source/ooo.po   |   44 
 source/sl/sd/source/core.po|  100 
 source/sl/sd/source/filter/html.po |  557 --
 source/sl/sd/source/ui/app.po  |   91 
 source/sl/sd/source/ui/dlg.po  |  460 --
 source/sl/sd/uiconfig/simpress/ui.po   |  485 ++
 source/sl/sfx2/source/dialog.po|   12 
 source/sl/sfx2/source/doc.po   |   12 
 source/sl/sfx2/source/sidebar.po   |   15 
 source/sl/sfx2/uiconfig/ui.po  |   32 
 source/sl/starmath/source.po   |   56 
 source/sl/starmath/uiconfig/smath/ui.po|   13 
 source/sl/svtools/uiconfig/ui.po   |  115 
 source/sl/svx/source/dialog.po |  590 +-
 source/sl/svx/source/engine3d.po   |  785 ---
 source/sl/svx/source/form.po   |   12 
 source/sl/svx/source/gallery2.po   |   12 
 source/sl/svx/source/sidebar/paragraph.po  |  512 --
 source/sl/svx/source/stbctrls.po   |4 
 source/sl/svx/source/tbxctrls.po   |  214 -
 source/sl/svx/uiconfig/ui.po   | 1535 ++-
 source/sl/sw/source/ui/app.po 

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

2015-03-19 Thread Bjoern Michaelsen
 sw/inc/calbck.hxx|  191 ++-
 sw/inc/switerator.hxx|   68 
 sw/source/core/access/accmap.cxx |2 
 sw/source/core/attr/calbck.cxx   |   26 +--
 sw/source/core/attr/cellatr.cxx  |2 
 sw/source/core/crsr/annotationmark.cxx   |2 
 sw/source/core/crsr/crstrvl.cxx  |2 
 sw/source/core/doc/DocumentFieldsManager.cxx |2 
 sw/source/core/doc/DocumentStatisticsManager.cxx |2 
 sw/source/core/doc/doc.cxx   |2 
 sw/source/core/doc/docdraw.cxx   |2 
 sw/source/core/doc/docfld.cxx|2 
 sw/source/core/doc/docfly.cxx|2 
 sw/source/core/doc/docglbl.cxx   |2 
 sw/source/core/doc/docnum.cxx|2 
 sw/source/core/doc/doctxm.cxx|2 
 sw/source/core/doc/fmtcol.cxx|2 
 sw/source/core/doc/htmltbl.cxx   |2 
 sw/source/core/doc/tblrwcl.cxx   |2 
 sw/source/core/docnode/ndtbl.cxx |2 
 sw/source/core/docnode/ndtbl1.cxx|2 
 sw/source/core/docnode/node.cxx  |2 
 sw/source/core/docnode/node2lay.cxx  |2 
 sw/source/core/docnode/section.cxx   |2 
 sw/source/core/docnode/swbaslnk.cxx  |2 
 sw/source/core/draw/dcontact.cxx |2 
 sw/source/core/edit/edfld.cxx|2 
 sw/source/core/edit/edfldexp.cxx |2 
 sw/source/core/edit/editsh.cxx   |2 
 sw/source/core/fields/authfld.cxx|2 
 sw/source/core/fields/dbfld.cxx  |2 
 sw/source/core/fields/ddefld.cxx |2 
 sw/source/core/fields/docufld.cxx|2 
 sw/source/core/fields/expfld.cxx |2 
 sw/source/core/fields/fldbas.cxx |2 
 sw/source/core/fields/fldlst.cxx |2 
 sw/source/core/fields/postithelper.cxx   |2 
 sw/source/core/fields/reffld.cxx |2 
 sw/source/core/fields/tblcalc.cxx|2 
 sw/source/core/frmedt/feshview.cxx   |2 
 sw/source/core/frmedt/tblsel.cxx |2 
 sw/source/core/layout/atrfrm.cxx |2 
 sw/source/core/layout/colfrm.cxx |2 
 sw/source/core/layout/findfrm.cxx|2 
 sw/source/core/layout/flowfrm.cxx|2 
 sw/source/core/layout/fly.cxx|2 
 sw/source/core/layout/flypos.cxx |2 
 sw/source/core/layout/frmtool.cxx|2 
 sw/source/core/layout/ftnfrm.cxx |2 
 sw/source/core/layout/movedfwdfrmsbyobjpos.cxx   |2 
 sw/source/core/layout/pagechg.cxx|2 
 sw/source/core/layout/pagedesc.cxx   |2 
 sw/source/core/layout/paintfrm.cxx   |2 
 sw/source/core/layout/softpagebreak.cxx  |2 
 sw/source/core/layout/tabfrm.cxx |2 
 sw/source/core/table/swnewtable.cxx  |2 
 sw/source/core/table/swtable.cxx |2 
 sw/source/core/text/EnhancedPDFExportHelper.cxx  |2 
 sw/source/core/text/itratr.cxx   |2 
 sw/source/core/text/porlay.cxx   |2 
 sw/source/core/text/txtdrop.cxx  |2 
 sw/source/core/text/txtfrm.cxx   |2 
 sw/source/core/tox/tox.cxx   |2 
 sw/source/core/txtnode/atrflyin.cxx  |2 
 sw/source/core/txtnode/atrftn.cxx|2 
 sw/source/core/txtnode/ndtxt.cxx |2 
 sw/source/core/undo/unattr.cxx   |2 
 sw/source/core/undo/untbl.cxx|2 
 sw/source/core/unocore/unocoll.cxx   |2 
 sw/source/core/unocore/unodraw.cxx   |2 
 sw/source/core/unocore/unofield.cxx  |2 
 sw/source/core/unocore/unoframe.cxx  |2 
 sw/source/core/unocore/unoobj2.cxx   |2 
 sw/source/core/unocore/unoportenum.cxx   |2 
 sw/source/core/unocore/unoredlines.cxx   |2 
 sw/source/core/unocore/unotbl.cxx|2 
 sw/source/core/view/vnew.cxx |2 
 sw/source/filter/ww8/ww8atr.cxx  |2 
 sw/source/filter/xml/xmltexti.cxx|2 
 sw/source/ui/fldui/fldpage.cxx   |2 
 sw/source/uibase/app/docsh.cxx   |2 
 sw/source/uibase/dochdl/swdtflvr.cxx |2 
 sw/source/uibase/docvw/PostItMgr.cxx |2 
 sw/source/uibase/shells/textfld.cxx  |2 
 sw/source/uibase/uno/unotxvw.cxx |2 
 85 files changed, 214 insertions(+

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

2015-03-19 Thread Caolán McNamara
 canvas/source/cairo/cairo_textlayout.cxx  |   43 +-
 canvas/source/cairo/cairo_win32_cairo.cxx |   38 --
 canvas/source/cairo/cairo_win32_cairo.hxx |2 -
 3 files changed, 42 insertions(+), 41 deletions(-)

New commits:
commit b517c251a733692f39d02f986e5010f214aea697
Author: Caolán McNamara 
Date:   Thu Mar 19 15:59:56 2015 +

move ucs4toindex

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

diff --git a/canvas/source/cairo/cairo_textlayout.cxx 
b/canvas/source/cairo/cairo_textlayout.cxx
index 9b1fa47..936593a 100644
--- a/canvas/source/cairo/cairo_textlayout.cxx
+++ b/canvas/source/cairo/cairo_textlayout.cxx
@@ -325,6 +325,47 @@ namespace cairocanvas
 return true;
 }
 
+#ifdef CAIRO_HAS_WIN32_SURFACE
+namespace
+{
+/**
+ * cairo::ucs4toindex: Convert ucs4 char to glyph index
+ * @param ucs4 an ucs4 char
+ * @param hfont current font
+ *
+ * @return true if successful
+ **/
+unsigned long ucs4toindex(unsigned int ucs4, HFONT hfont)
+{
+wchar_t unicode[2];
+WORD glyph_index;
+HDC hdc = NULL;
+
+hdc = CreateCompatibleDC (NULL);
+
+if (!hdc) return 0;
+if (!SetGraphicsMode (hdc, GM_ADVANCED))
+{
+DeleteDC (hdc);
+return 0;
+}
+
+SelectObject (hdc, hfont);
+SetMapMode (hdc, MM_TEXT);
+
+unicode[0] = ucs4;
+unicode[1] = 0;
+if (GetGlyphIndicesW (hdc, unicode, 1, &glyph_index, 0) == 
GDI_ERROR)
+{
+glyph_index = 0;
+}
+
+DeleteDC (hdc);
+return glyph_index;
+}
+}
+#endif
+
   /**
* TextLayout::draw
*
@@ -446,7 +487,7 @@ namespace cairocanvas
 #ifdef CAIRO_HAS_WIN32_SURFACE
 // Cairo requires standard glyph indexes (ETO_GLYPH_INDEX), 
while vcl/win/* uses ucs4 chars.
 // Convert to standard indexes
-aGlyph.index = cairo::ucs4toindex((unsigned int) aGlyph.index, 
rSysFontData.hFont);
+aGlyph.index = ucs4toindex((unsigned int) aGlyph.index, 
rSysFontData.hFont);
 #endif
 aGlyph.x = systemGlyph.x;
 aGlyph.y = systemGlyph.y;
diff --git a/canvas/source/cairo/cairo_win32_cairo.cxx 
b/canvas/source/cairo/cairo_win32_cairo.cxx
index a54c7e1..d6a4aba 100644
--- a/canvas/source/cairo/cairo_win32_cairo.cxx
+++ b/canvas/source/cairo/cairo_win32_cairo.cxx
@@ -249,44 +249,6 @@ namespace cairo
 return SurfaceSharedPtr();
 }
 
-
-/**
- * cairo::ucs4toindex: Convert ucs4 char to glyph index
- * @param ucs4 an ucs4 char
- * @param hfont current font
- *
- * @return true if successful
- **/
-unsigned long ucs4toindex(unsigned int ucs4, HFONT hfont)
-{
-wchar_t unicode[2];
-WORD glyph_index;
-HDC hdc = NULL;
-
-hdc = CreateCompatibleDC (NULL);
-
-if (!hdc) return 0;
-if (!SetGraphicsMode (hdc, GM_ADVANCED))
-{
-DeleteDC (hdc);
-return 0;
-}
-
-SelectObject (hdc, hfont);
-SetMapMode (hdc, MM_TEXT);
-
-unicode[0] = ucs4;
-unicode[1] = 0;
-if (GetGlyphIndicesW (hdc, unicode, 1, &glyph_index, 0) == GDI_ERROR)
-{
-glyph_index = 0;
-}
-
-DeleteDC (hdc);
-return glyph_index;
-}
-
-
 }  // namespace cairo
 
 #endif   // CAIRO_HAS_WIN32_SURFACE
diff --git a/canvas/source/cairo/cairo_win32_cairo.hxx 
b/canvas/source/cairo/cairo_win32_cairo.hxx
index 50bd137..6df30f5 100644
--- a/canvas/source/cairo/cairo_win32_cairo.hxx
+++ b/canvas/source/cairo/cairo_win32_cairo.hxx
@@ -49,8 +49,6 @@ namespace cairo {
 
 int getDepth() const;
 };
-
-unsigned long ucs4toindex(unsigned int ucs4, HFONT hfont);
 }
 
 #endif
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2015-03-19 Thread Niklas Johansson
 editeng/source/misc/svxacorr.cxx |  108 +--
 1 file changed, 59 insertions(+), 49 deletions(-)

New commits:
commit 307c25fe460857c52f0b9c2078e83f6d12f8bdc9
Author: Niklas Johansson 
Date:   Thu Mar 19 20:30:41 2015 +0100

tdf#89437  Ordinal suffix should never be superscript in some languages

Printing ordinal suffixes as superscript is just not done in Swedish and
likely a few other languages but this change only cares for Swedish at
the moment.

Change-Id: Ib7600ceb0534793d900f13b2740e63c1f7f34ba9
Reviewed-on: https://gerrit.libreoffice.org/14913
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 79868dd..e74ad5d 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -441,69 +441,79 @@ bool SvxAutoCorrect::FnCptlSttWrd( SvxAutoCorrDoc& rDoc, 
const OUString& rTxt,
 
 
 bool SvxAutoCorrect::FnChgOrdinalNumber(
-SvxAutoCorrDoc& rDoc, const OUString& rTxt,
-sal_Int32 nSttPos, sal_Int32 nEndPos,
-LanguageType eLang )
+SvxAutoCorrDoc& rDoc, const OUString& rTxt,
+sal_Int32 nSttPos, sal_Int32 nEndPos,
+LanguageType eLang)
 {
-// 1st, 2nd, 3rd, 4 - 0th
-// 201th or 201st
-// 12th or 12nd
-CharClass& rCC = GetCharClass( eLang );
+// 1st, 2nd, 3rd, 4 - 0th
+// 201th or 201st
+// 12th or 12nd
 bool bChg = false;
 
-for( ; nSttPos < nEndPos; ++nSttPos )
-if( !lcl_IsInAsciiArr( sImplSttSkipChars, rTxt[ nSttPos ] ))
-break;
-for( ; nSttPos < nEndPos; --nEndPos )
-if( !lcl_IsInAsciiArr( sImplEndSkipChars, rTxt[ nEndPos - 1 ] ))
-break;
+// In some languages ordinal suffixes should never be
+// changed to superscript. Let's break for those languages.
+switch (eLang)
+{
+case LANGUAGE_SWEDISH:
+case LANGUAGE_SWEDISH_FINLAND:
+break;
+default:
+CharClass& rCC = GetCharClass(eLang);
 
+for (; nSttPos < nEndPos; ++nSttPos)
+if (!lcl_IsInAsciiArr(sImplSttSkipChars, rTxt[nSttPos]))
+break;
+for (; nSttPos < nEndPos; --nEndPos)
+if (!lcl_IsInAsciiArr(sImplEndSkipChars, rTxt[nEndPos - 1]))
+break;
 
-// Get the last number in the string to check
-sal_Int32 nNumEnd = nEndPos;
-bool foundEnd = false;
-bool validNumber = true;
-sal_Int32 i = nEndPos;
 
-while ( i > nSttPos )
-{
-i--;
-bool isDigit = rCC.isDigit( rTxt, i );
-if ( foundEnd )
-validNumber |= isDigit;
+// Get the last number in the string to check
+sal_Int32 nNumEnd = nEndPos;
+bool foundEnd = false;
+bool validNumber = true;
+sal_Int32 i = nEndPos;
 
-if ( isDigit && !foundEnd )
+while (i > nSttPos)
 {
-foundEnd = true;
-nNumEnd = i;
-}
-}
+i--;
+bool isDigit = rCC.isDigit(rTxt, i);
+if (foundEnd)
+validNumber |= isDigit;
 
-if ( foundEnd && validNumber ) {
-sal_Int32 nNum = rTxt.copy( nSttPos, nNumEnd - nSttPos + 1 ).toInt32( 
);
+if (isDigit && !foundEnd)
+{
+foundEnd = true;
+nNumEnd = i;
+}
+}
 
-// Check if the characters after that number correspond to the ordinal 
suffix
-uno::Reference< i18n::XOrdinalSuffix > xOrdSuffix
-= i18n::OrdinalSuffix::create( 
comphelper::getProcessComponentContext() );
+if (foundEnd && validNumber) {
+sal_Int32 nNum = rTxt.copy(nSttPos, nNumEnd - nSttPos + 
1).toInt32();
 
-uno::Sequence< OUString > aSuffixes = xOrdSuffix->getOrdinalSuffix( 
nNum, rCC.getLanguageTag().getLocale( ) );
-for ( sal_Int32 nSuff = 0; nSuff < aSuffixes.getLength(); nSuff++ )
-{
-OUString sSuffix( aSuffixes[ nSuff ] );
-OUString sEnd = rTxt.copy( nNumEnd + 1, nEndPos - nNumEnd - 1 );
+// Check if the characters after that number correspond to the 
ordinal suffix
+uno::Reference< i18n::XOrdinalSuffix > xOrdSuffix
+= 
i18n::OrdinalSuffix::create(comphelper::getProcessComponentContext());
 
-if ( sSuffix == sEnd )
+uno::Sequence< OUString > aSuffixes = 
xOrdSuffix->getOrdinalSuffix(nNum, rCC.getLanguageTag().getLocale());
+for (sal_Int32 nSuff = 0; nSuff < aSuffixes.getLength(); nSuff++)
 {
-// Check if the ordinal suffix has to be set as super script
-if ( rCC.isLetter( sSuffix ) )
+OUString sSuffix(aSuffixes[nSuff]);
+OUString sEnd = rTxt.copy(nNumEnd + 1, nEndPos - nNumEnd - 1);
+
+if (sSuffix == sEnd)
  

Re: 4.4.3.0.0+ build fails (unit tests)

2015-03-19 Thread Norbert Thiebaud
On Thu, Mar 19, 2015 at 9:01 AM, Miklos Vajna  wrote:
> Hi Justin,
>
> On Thu, Mar 19, 2015 at 04:32:35PM +0300, Justin Luth  
> wrote:
>> P.S.  I have not been able to figure out how to remove a file from a
>> commit if I add it accidentally.  If I do a "git commit -a" and get an
>> unintended file, how do I amend the commit to "ignore" the file?  (That
>> is what happened here.)
>
> rm , then git commit -a --amend

or

git rm 
git commit --amend

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


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

2015-03-19 Thread Bjoern Michaelsen
 sw/inc/calbck.hxx  |  166 +++--
 sw/qa/core/uwriter.cxx |   95 
 2 files changed, 176 insertions(+), 85 deletions(-)

New commits:
commit 85e969fc230a49ea40fd3a799bc5828d3e7c2e95
Author: Bjoern Michaelsen 
Date:   Fri Mar 20 02:07:00 2015 +0100

use IsChanged() some more

Change-Id: Ic32f27fe5aa6be115af0d3f28252b98c7b1788a3

diff --git a/sw/inc/calbck.hxx b/sw/inc/calbck.hxx
index 062ebdf..2af23b7 100644
--- a/sw/inc/calbck.hxx
+++ b/sw/inc/calbck.hxx
@@ -317,7 +317,7 @@ public:
 }
 TElementType* Next()
 {
-if( m_pPosition == m_pCurrent )
+if(!IsChanged())
 m_pPosition = GetRightOfPos();
 while(m_pPosition && !m_pPosition->IsA( TYPE(TElementType) ) )
 m_pPosition = GetRightOfPos();
@@ -352,7 +352,7 @@ public:
 }
 SwClient* Next()
 {
-if( m_pPosition == m_pCurrent )
+if(!IsChanged())
 m_pPosition = GetRightOfPos();
 return Sync();
 }
commit 87bc662386ffe53b8b4b1fa07deb0bb665fe39c9
Author: Bjoern Michaelsen 
Date:   Thu Mar 19 03:23:24 2015 +0100

add unittest

Change-Id: Iba119145155ee13067565b1f314c85b0e6a8de85

diff --git a/sw/qa/core/uwriter.cxx b/sw/qa/core/uwriter.cxx
index e0860c8f..edfa241 100644
--- a/sw/qa/core/uwriter.cxx
+++ b/sw/qa/core/uwriter.cxx
@@ -60,6 +60,7 @@
 #include "scriptinfo.hxx"
 #include "IMark.hxx"
 #include "ring.hxx"
+#include "calbck.hxx"
 
 typedef tools::SvRef SwDocShellRef;
 
@@ -105,8 +106,10 @@ public:
 void testTransliterate();
 void testMarkMove();
 void testIntrusiveRing();
+void testClientModify();
 
 CPPUNIT_TEST_SUITE(SwDocTest);
+
 CPPUNIT_TEST(testTransliterate);
 CPPUNIT_TEST(randomTest);
 CPPUNIT_TEST(testPageDescName);
@@ -134,6 +137,7 @@ public:
 CPPUNIT_TEST(testGraphicAnchorDeletion);
 CPPUNIT_TEST(testMarkMove);
 CPPUNIT_TEST(testIntrusiveRing);
+CPPUNIT_TEST(testClientModify);
 CPPUNIT_TEST_SUITE_END();
 
 private:
@@ -1384,6 +1388,97 @@ void SwDocTest::testIntrusiveRing()
 CPPUNIT_ASSERT_EQUAL(&foo, foo.GetPrev());
 }
 
+namespace
+{
+struct TestModify : SwModify
+{
+TYPEINFO();
+};
+TYPEINIT1( TestModify, SwModify );
+struct TestClient : SwClient
+{
+TYPEINFO();
+int m_nModifyCount;
+TestClient() : m_nModifyCount(0) {};
+virtual void Modify( const SfxPoolItem*, const SfxPoolItem*)
+{
+ShowReg();
+++m_nModifyCount;
+}
+
+void ShowReg()
+{
+if(GetRegisteredIn())
+{
+std::cout << "TestClient " << this << " registered  in " << 
GetRegisteredIn() << std::endl;
+}
+else
+std::cout << "TestClient " << this << " not registered " << 
std::endl;
+}
+};
+TYPEINIT1( TestClient, SwClient );
+struct OtherTestClient : SwClient
+{ TYPEINFO(); };
+TYPEINIT1( OtherTestClient, SwClient );
+}
+void SwDocTest::testClientModify()
+{
+TestModify aMod;
+TestClient aClient1, aClient2;
+aMod.Add(&aClient1);
+aMod.Add(&aClient2);
+
CPPUNIT_ASSERT_EQUAL(aClient1.GetRegisteredIn(),static_cast(&aMod));
+
CPPUNIT_ASSERT_EQUAL(aClient2.GetRegisteredIn(),static_cast(&aMod));
+aMod.ModifyBroadcast(nullptr, nullptr);
+CPPUNIT_ASSERT_EQUAL(aClient1.m_nModifyCount,1);
+CPPUNIT_ASSERT_EQUAL(aClient2.m_nModifyCount,1);
+aMod.ModifyBroadcast(nullptr, nullptr);
+CPPUNIT_ASSERT_EQUAL(aClient1.m_nModifyCount,2);
+CPPUNIT_ASSERT_EQUAL(aClient2.m_nModifyCount,2);
+CPPUNIT_ASSERT(!aClient1.IsA(TYPE(OtherTestClient)));
+{
+SwIterator aIter(aMod);
+for(OtherTestClient* pClient = aIter.First(); pClient ; pClient = 
aIter.Next())
+CPPUNIT_ASSERT(false);
+}
+{
+int nCount = 0;
+SwIterator aIter(aMod);
+for(TestClient* pClient = aIter.First(); pClient ; pClient = 
aIter.Next())
+{
+CPPUNIT_ASSERT_EQUAL(pClient->m_nModifyCount,2);
+++nCount;
+}
+CPPUNIT_ASSERT_EQUAL(nCount,2);
+}
+
CPPUNIT_ASSERT_EQUAL(aClient1.GetRegisteredIn(),static_cast(&aMod));
+
CPPUNIT_ASSERT_EQUAL(aClient2.GetRegisteredIn(),static_cast(&aMod));
+{
+int nCount = 0;
+SwIterator aIter(aMod);
+for(TestClient* pClient = aIter.First(); pClient ; pClient = 
aIter.Next())
+{
+aMod.Remove(pClient);
+++nCount;
+}
+CPPUNIT_ASSERT_EQUAL(nCount,2);
+}
+CPPUNIT_ASSERT_EQUAL(aClient1.GetRegisteredIn(), 
static_cast(nullptr));
+CPPUNIT_ASSERT_EQUAL(aClient2.GetRegisteredIn(), 
static_cast(nullptr));
+{
+int nCount = 0;
+SwIterator aIter(aMod);
+for(TestClient* pClient = aIter.First(); pClient ; pClient = 
aIter.Next())
+{
+CPPUNIT_ASSERT(false);
+   

Laszlo Kis-Adam license statement

2015-03-19 Thread Laszlo Kis-Adam

Hi there!
According to the LibreOffice rules,
I hereby state that all of my past & future contributions to LibreOffice 
may be licensed under the MPLv2/LGPLv3+ dual license.


Cheers,
Laszlo Kis-Adam

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


Re: [Libreoffice-qa] minutes of ESC call ...

2015-03-19 Thread Norbert Thiebaud
On Thu, Mar 19, 2015 at 10:30 AM, Michael Meeks
 wrote:
>
> * Tinderbox functionality stats (Norbert)
> + what %age of time is green for each platform ?

Here are the precise stats for the past 2 weeks:

Job name#-Successful#-Failed#-Total Builds-Failure Rate
lo_gerrit_master-101-140-241-58.09%
lo_gerrit_master/Gerrit=Gerrit,Platform=Linux-180-61-241-25.31%
lo_gerrit_master/Gerrit=Gerrit,Platform=MacOSX-182-59-241-24.48%
lo_gerrit_master/Gerrit=Gerrit,Platform=Windows-138-103-241-42.74%
lo_tb_libreoffice-4-4_mac-23-6-29-20.69%
lo_tb_master_linux-191-21-214-10.75%
lo_tb_master_mac-169-35  204-17.16%
lo_tb_master_win-114-47-161-29.19%

Total: 1098-472-1572-30.15%
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Crash test update

2015-03-19 Thread Crashtest VM
New crashtest update available at 
http://dev-builds.libreoffice.org/crashtest/c83476ab28544c25ab72e7e51131ab481bcbcc5d/


exportCrashes.csv
Description: Binary data


importCrash.csv
Description: Binary data


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


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

2015-03-19 Thread David Tardon
 desktop/source/lib/init.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit a8c96af2bca17fcd031a245c52da314ab4902013
Author: David Tardon 
Date:   Fri Mar 20 06:51:48 2015 +0100

drop unneeded include

Change-Id: I566678d55fa5d052a94a9b69732e2be4c0c39280

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index eff82b6..df4593f 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -51,7 +51,6 @@
 #include 
 
 #include 
-#include 
 
 #if defined(UNX) && !defined(MACOSX) && !defined(ENABLE_HEADLESS)
 // Let's grab the SvpSalInstance and SvpSalVirtualDevice
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


LibreOffice Gerrit News for core on 2015-03-20

2015-03-19 Thread gerrit
Moin!

* Open changes on master for project core changed in the last 25 hours:

 First time contributors doing great things! 
+ tdf#42897 Warn the user if string without quote is entered as condition v
  in https://gerrit.libreoffice.org/14916 from Laszlo Kis-Adam
  about module sc
+ tdf#64575 Photo Album: Better Image Layout
  in https://gerrit.libreoffice.org/14912 from Enes Ateş
  about module sd
+ tdf#57174 Opening Any Dialog the First Value in the Dialog Should be Sele
  in https://gerrit.libreoffice.org/14881 from Qiwen Gao
  about module include, sc, vcl
+ tdf#89756 Switched postfix to prefix operator++/--
  in https://gerrit.libreoffice.org/14903 from Gülşah Köse
  about module include, sc, sd, svtools, sw, vcl, xmloff
+ tdf#89405 DOC export: fix corrupted comment order
  in https://gerrit.libreoffice.org/14841 from Piet van Oostrum
  about module sw
 End of freshness 

+ move cairo helpers to vcl and make per-plug
  in https://gerrit.libreoffice.org/14907 from Caolán McNamara
  about module canvas, include, vcl
+ Adjustable hidden content removal in e-mails
  in https://gerrit.libreoffice.org/14909 from Stefan Heinemann
  about module cui, officecfg, sw
+ tdf#89666: vcl: speed up HbLayoutEngine with cache in SwTxtFormatInfo
  in https://gerrit.libreoffice.org/14732 from Michael Stahl
  about module include, sw, vcl
+ tdf#30716 Engineering notation
  in https://gerrit.libreoffice.org/14886 from Laurent BP
  about module basic, svl
+ tdf#30716 start Engineering notation: implement exponent-interval
  in https://gerrit.libreoffice.org/14875 from Laurent BP
  about module include, xmloff
+ tdf#60060: DOCX export of section protection
  in https://gerrit.libreoffice.org/14895 from Eilidh McAdam
  about module sw


* Merged changes on master for project core changed in the last 25 hours:

+ tdf#89437  Ordinal suffix should never be superscript in some languages
  in https://gerrit.libreoffice.org/14913 from Niklas Johansson
+ move ucs4toindex
  in https://gerrit.libreoffice.org/14908 from Caolán McNamara
+ reorganize resizing surfaces logic
  in https://gerrit.libreoffice.org/14902 from Caolán McNamara
+ tdf#90090 - rearrange Draw's UI by moving around its toolbars
  in https://gerrit.libreoffice.org/14904 from Yousuf Philips
+ loplugin:constantfunction: unotools
  in https://gerrit.libreoffice.org/14590 from Noel Grandin
+ loplugin:constantfunction: vcl
  in https://gerrit.libreoffice.org/14712 from Noel Grandin
+ loplugin:constantfunction: sfx2
  in https://gerrit.libreoffice.org/14713 from Noel Grandin
+ loplugin:constantfunction: svtools
  in https://gerrit.libreoffice.org/14727 from Noel Grandin
+ loplugin:constantfunction: sot
  in https://gerrit.libreoffice.org/14728 from Noel Grandin
+ loplugin:constantfunction: tools
  in https://gerrit.libreoffice.org/14729 from Noel Grandin
+ loplugin:constantfunction: registry
  in https://gerrit.libreoffice.org/14730 from Noel Grandin
+ Related tdf#50133: Wrong underlining in Reports
  in https://gerrit.libreoffice.org/14801 from Julien Nabet
+ loplugin:constantfunction: svl
  in https://gerrit.libreoffice.org/14731 from Noel Grandin
+ loplugin:constantfunction: svx
  in https://gerrit.libreoffice.org/14759 from Noel Grandin
+ Optimize 'Breeze' icon theme
  in https://gerrit.libreoffice.org/14900 from Andrew Dent
+ loplugin:constantfunction: linguistic
  in https://gerrit.libreoffice.org/14760 from Noel Grandin
+ Percentages should also be translatable…
  in https://gerrit.libreoffice.org/14767 from Adolfo Jayme Barrientos
+ Support "a/b" style of fractions, not just "a b/c" style. Also fixes a cr
  in https://gerrit.libreoffice.org/14842 from Laszlo Kis-Adam
+ CppCheck cleaning : unnecessary assignment
  in https://gerrit.libreoffice.org/14856 from Michaël Lefèvre
+ Speed up ScriptRun::getPairIndex
  in https://gerrit.libreoffice.org/14834 from Noel Grandin
+ tdf#86787: Page style margin limited to 99.99mm
  in https://gerrit.libreoffice.org/14821 from Rishabh Kumar
+ CppCheck cleaning : scope reduction and printf format
  in https://gerrit.libreoffice.org/14854 from Michaël Lefèvre
+ loplugin:constantfunction: editeng
  in https://gerrit.libreoffice.org/14763 from Noel Grandin
+ loplugin:constantfunction: canvas
  in https://gerrit.libreoffice.org/14762 from Noel Grandin
+ loplugin:constantfunction: oox
  in https://gerrit.libreoffice.org/14761 from Noel Grandin


* Abandoned changes on master for project core changed in the last 25 hours:

+ Fix build...
  in https://gerrit.libreoffice.org/14917 from Laszlo Kis-Adam
+ Adjustable hidden content removal in e-mails
  in https://gerrit.libreoffice.org/14889 from Stefan Heinemann


* Open changes needing tweaks, but being untouched for more than a week:

+ tdf#88290: bring numbering back to 8 entries, ZH_CN locale, tdf#87626 rel
  in https://gerrit.libreoffice.org/14836 from Jingtao Yan
+ xmloff: convert xof.component to use constructor syntax
  in https://gerrit.libreoffice.org/14820 from Chr

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

2015-03-19 Thread Zolnai Tamás
 sd/source/ui/dlg/tabtempl.cxx |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 3640717c8153e8b8e212e3aaa505007f7b8b0430
Author: Zolnai Tamás 
Date:   Fri Mar 20 07:21:27 2015 +0100

coverity#1288897 Resource leak

Here pRet was used to change the input set, so
use SetInputSet to get the same result.
Plus rename pRet since it's not a return value
any more.

Change-Id: I9e75abe2a86b372beb6fe5bd211dcf4fbbefb3da

diff --git a/sd/source/ui/dlg/tabtempl.cxx b/sd/source/ui/dlg/tabtempl.cxx
index a213c38..e2e7613 100644
--- a/sd/source/ui/dlg/tabtempl.cxx
+++ b/sd/source/ui/dlg/tabtempl.cxx
@@ -174,15 +174,15 @@ void SdTabTemplateDlg::PageCreated( sal_uInt16 nId, 
SfxTabPage &rPage )
 
 void SdTabTemplateDlg::RefreshInputSet()
 {
-SfxItemSet* pRet = GetInputSetImpl();
+SfxItemSet* pInputSet = GetInputSetImpl();
 
-if( pRet )
+if( pInputSet )
 {
-pRet->ClearItem();
-pRet->SetParent( GetStyleSheet().GetItemSet().GetParent() );
+pInputSet->ClearItem();
+pInputSet->SetParent( GetStyleSheet().GetItemSet().GetParent() );
 }
 else
-pRet = new SfxItemSet( GetStyleSheet().GetItemSet() );
+SetInputSet(&GetStyleSheet().GetItemSet());
 }
 
 /* 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: sc/source

2015-03-19 Thread Julien Nabet
 sc/source/filter/excel/xeroot.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 2d279bc7d96b26c0721605cc98bd8e07bd03b707
Author: Julien Nabet 
Date:   Fri Mar 20 07:35:42 2015 +0100

Typo: intead->instead

Change-Id: I6aff104fa5749217d5c9628b5dfbfd3ac79bdc43

diff --git a/sc/source/filter/excel/xeroot.cxx 
b/sc/source/filter/excel/xeroot.cxx
index 96f5085..876ed65 100644
--- a/sc/source/filter/excel/xeroot.cxx
+++ b/sc/source/filter/excel/xeroot.cxx
@@ -216,7 +216,7 @@ void XclExpRoot::InitializeGlobals()
 ScDocument& rDoc = GetDoc();
 // Pass the model factory to OpCodeProvider, not the process
 // service factory, otherwise a FormulaOpCodeMapperObj would be
-// instantiated intead of a ScFormulaOpCodeMapperObj and the
+// instantiated instead of a ScFormulaOpCodeMapperObj and the
 // ScCompiler virtuals not be called! Which would be the case with
 // the current (2013-01-24) rDoc.GetServiceManager()
 const SfxObjectShell* pShell = rDoc.GetDocumentShell();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: New Defects reported by Coverity Scan for LibreOffice

2015-03-19 Thread Zolnai Tamás
> caolanm->Tamás: cid#1288897
> after f7424ed710e54bb2437a28380b03ed7c26290edc
> SdTabTemplateDlg::GetRefreshedSet->SdTabTemplateDlg::RefreshInputSet has
> a leaky "pRet = new SfxItemSet" in it now, what's the right fix there ?

Thanks for the notification! It's fixed now.

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


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

2015-03-19 Thread Julien Nabet
 include/svx/svdmodel.hxx |2 -
 include/svx/unopool.hxx  |2 -
 include/svx/unoshape.hxx |   28 +--
 opencl/source/opencl_device.cxx  |2 -
 svtools/source/uno/unoimap.cxx   |4 +-
 svx/source/unodraw/shapeimpl.hxx |8 ++---
 svx/source/unodraw/unoshape.cxx  |2 -
 winaccessibility/source/UAccCOM/AccAction.cxx|2 -
 winaccessibility/source/UAccCOM/AccAction.h  |4 +-
 winaccessibility/source/UAccCOM/AccActionBase.cxx|2 -
 winaccessibility/source/UAccCOM/AccActionBase.h  |2 -
 winaccessibility/source/UAccCOM/AccComponent.cxx |2 -
 winaccessibility/source/UAccCOM/AccComponent.h   |2 -
 winaccessibility/source/UAccCOM/AccComponentBase.cxx |2 -
 winaccessibility/source/UAccCOM/AccComponentBase.h   |2 -
 winaccessibility/source/UAccCOM/AccEditableText.cxx  |2 -
 winaccessibility/source/UAccCOM/AccEditableText.h|2 -
 winaccessibility/source/UAccCOM/AccHyperLink.h   |4 +-
 winaccessibility/source/UAccCOM/AccHypertext.h   |2 -
 winaccessibility/source/UAccCOM/AccImage.h   |2 -
 winaccessibility/source/UAccCOM/AccRelation.h|2 -
 winaccessibility/source/UAccCOM/AccTable.cxx |2 -
 winaccessibility/source/UAccCOM/AccTable.h   |2 -
 winaccessibility/source/UAccCOM/AccText.h|2 -
 winaccessibility/source/UAccCOM/AccTextBase.h|2 -
 winaccessibility/source/UAccCOM/AccValue.h   |2 -
 26 files changed, 45 insertions(+), 45 deletions(-)

New commits:
commit 62820766ef00c93192e0e8a7ce7c8f77f84a710e
Author: Julien Nabet 
Date:   Fri Mar 20 07:57:02 2015 +0100

Typo: (O/o)veride(n)->(O/o)verride(n)

Change-Id: Ied81ede9aaf045866596adc40d357cbbddb3d704

diff --git a/include/svx/svdmodel.hxx b/include/svx/svdmodel.hxx
index ddd7330..388e82e 100644
--- a/include/svx/svdmodel.hxx
+++ b/include/svx/svdmodel.hxx
@@ -579,7 +579,7 @@ public:
 void SetSdrUndoManager( SfxUndoManager* pUndoManager );
 SfxUndoManager* GetSdrUndoManager() const;
 
-/** applications can set their own undo factory to overide creation of
+/** applications can set their own undo factory to override creation of
 undo actions. The SdrModel will become owner of the given 
SdrUndoFactory
 and delete it upon its destruction. */
 void SetSdrUndoFactory( SdrUndoFactory* pUndoFactory );
diff --git a/include/svx/unopool.hxx b/include/svx/unopool.hxx
index e7eaab1..1339199 100644
--- a/include/svx/unopool.hxx
+++ b/include/svx/unopool.hxx
@@ -51,7 +51,7 @@ public:
 */
 virtual SfxItemPool* getModelPool( bool bReadOnly ) throw();
 
-// overiden helpers from comphelper::PropertySetHelper
+// overriden helpers from comphelper::PropertySetHelper
 virtual void _setPropertyValues( const comphelper::PropertyMapEntry** 
ppEntries, const ::com::sun::star::uno::Any* pValues ) 
throw(::com::sun::star::beans::UnknownPropertyException, 
::com::sun::star::beans::PropertyVetoException, 
::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::lang::WrappedTargetException ) SAL_OVERRIDE;
 virtual void _getPropertyValues( const comphelper::PropertyMapEntry** 
ppEntries, ::com::sun::star::uno::Any* pValue ) 
throw(::com::sun::star::beans::UnknownPropertyException, 
::com::sun::star::lang::WrappedTargetException ) SAL_OVERRIDE;
 
diff --git a/include/svx/unoshape.hxx b/include/svx/unoshape.hxx
index de130b5..20f2e22 100644
--- a/include/svx/unoshape.hxx
+++ b/include/svx/unoshape.hxx
@@ -159,7 +159,7 @@ protected:
 void updateShapeKind();
 void endSetPropertyValues();
 
-// overide these for special property handling in subcasses. Return true 
if property is handled
+// override these for special property handling in subcasses. Return true 
if property is handled
 virtual bool setPropertyValueImpl( const OUString& rName, const 
SfxItemPropertySimpleEntry* pProperty, const ::com::sun::star::uno::Any& rValue 
) throw(::com::sun::star::beans::UnknownPropertyException, 
::com::sun::star::beans::PropertyVetoException, 
::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::lang::WrappedTargetException, 
::com::sun::star::uno::RuntimeException, std::exception);
 virtual bool getPropertyValueImpl( const OUString& rName, const 
SfxItemPropertySimpleEntry* pProperty, ::com::sun::star::uno::Any& rValue ) 
throw(::com::sun::star::beans::UnknownPropertyException, 
::com::sun::star::lang::WrappedTargetException, 
::com::sun::star::uno::RuntimeException, std::exception);
 virtual bool getPropertyStateImpl( const SfxItemPropertySimpleEntry* 
pProperty, ::com::sun::star::beans::PropertyState& rState ) 
throw(::com::sun::star::beans::UnknownPropertyException, 
::com::sun::star::uno::RuntimeException);
@@ -327,7 +3