[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-4.4' - 5 commits - editeng/source external/python3 solenv/bin sw/source xmloff/source

Markus Mohrhard markus.mohrhard at googlemail.com
Sun May 10 21:56:57 PDT 2015


 editeng/source/editeng/impedit3.cxx           |    3 
 external/python3/python-3.3.3-py17797.patch.1 |   87 ++++++++++----------------
 solenv/bin/modules/installer/simplepackage.pm |   12 ++-
 sw/source/core/doc/doclay.cxx                 |    3 
 xmloff/source/chart/SchXMLLegendContext.cxx   |    6 -
 5 files changed, 49 insertions(+), 62 deletions(-)

New commits:
commit e3d0ca440cfe5573efbdb658f659e1a7bef19e0a
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Sat May 2 20:36:30 2015 +0200

    it works if we first set the size and then the position, tdf#86624
    
    Change-Id: I8ed98981ef3041839ab73b749a04febe369f66fe
    Reviewed-on: https://gerrit.libreoffice.org/15600
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/xmloff/source/chart/SchXMLLegendContext.cxx b/xmloff/source/chart/SchXMLLegendContext.cxx
index 7bb30bf..80e829d 100644
--- a/xmloff/source/chart/SchXMLLegendContext.cxx
+++ b/xmloff/source/chart/SchXMLLegendContext.cxx
@@ -188,14 +188,14 @@ void SchXMLLegendContext::StartElement( const uno::Reference< xml::sax::XAttribu
         }
     }
 
-    if( bHasXPosition && bHasYPosition )
-        xLegendShape->setPosition( aLegendPos );
-
     if( bHasExpansion && nLegendExpansion!= chart::ChartLegendExpansion_CUSTOM )
         xLegendProps->setPropertyValue("Expansion", uno::makeAny(nLegendExpansion) );
     else if( bHasHeight && bHasWidth )
         xLegendShape->setSize( aLegendSize );
 
+    if( bHasXPosition && bHasYPosition )
+        xLegendShape->setPosition( aLegendPos );
+
     // the fill style has the default "none" in XML, but "solid" in the model.
     xLegendProps->setPropertyValue("FillStyle", uno::makeAny( drawing::FillStyle_NONE ));
 
commit f523f77c97e77b32ce689e394b925fe5845893c4
Author: Michael Stahl <mstahl at redhat.com>
Date:   Wed May 6 22:28:05 2015 +0200

    tdf#82968: python3: fix stdio detection on WNT harder
    
    Upgrade to the latest patch from http://bugs.python.org/issue17797
    which appears to work even if you invoke from cmd.exe
    
    Change-Id: I85f1cc5ad7d8c059d972ae2a6fd2be1bb5604c2c
    (cherry picked from commit be3e1d65f50fe8b4ce5e4a87a82ff231c00aae79)
    Reviewed-on: https://gerrit.libreoffice.org/15688
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/external/python3/python-3.3.3-py17797.patch.1 b/external/python3/python-3.3.3-py17797.patch.1
index d4f7ab8..8fcb703 100644
--- a/external/python3/python-3.3.3-py17797.patch.1
+++ b/external/python3/python-3.3.3-py17797.patch.1
@@ -4,59 +4,42 @@ http://connect.microsoft.com/VisualStudio/feedback/details/785119/
 Visual Studio 2012 changed return value for fileno function that breaks
 when python tries to check/setup stdin/out/err
 GetStdHandle on Windows XP behaves contrary to the documentation...
-diff -ur python3.org/Python/pythonrun.c python3/Python/pythonrun.c
---- python3.org/Python/pythonrun.c	2014-05-24 16:36:20.361672900 +0200
-+++ python3/Python/pythonrun.c	2014-05-24 16:37:38.424159100 +0200
-@@ -1036,7 +1036,15 @@
-     int status = 0, fd;
-     PyObject * encoding_attr;
-     char *encoding = NULL, *errors;
--
-+#ifdef MS_WINDOWS
-+    OSVERSIONINFOEX osvi;
-+    BOOL bIsWindowsXP;
+
+diff --git a/Python/pythonrun.c b/Python/pythonrun.c
+index 91d56b7..d28ffc7 100644
+--- a/Python/pythonrun.c
++++ b/Python/pythonrun.c
+@@ -1015,13 +1015,28 @@ error:
+ static int
+ is_valid_fd(int fd)
+ {
+-    int dummy_fd;
+     if (fd < 0 || !_PyVerify_fd(fd))
+         return 0;
+-    dummy_fd = dup(fd);
+-    if (dummy_fd < 0)
+-        return 0;
+-    close(dummy_fd);
 +
-+    ZeroMemory(&osvi, sizeof(osvi));
-+    osvi.dwOSVersionInfoSize = sizeof(osvi);
-+    GetVersionEx(&osvi);
-+    bIsWindowsXP = (osvi.dwMajorVersion < 6);
-+#endif
-     /* Hack to avoid a nasty recursion issue when Python is invoked
-        in verbose mode: pre-import the Latin-1 and UTF-8 codecs */
-     if ((m = PyImport_ImportModule("encodings.utf_8")) == NULL) {
-@@ -1084,7 +1092,11 @@
-      * and fileno() may point to an invalid file descriptor. For example
-      * GUI apps don't have valid standard streams by default.
-      */
-+#ifdef MS_WINDOWS
-+    if (!is_valid_fd(fd) || GetStdHandle(STD_INPUT_HANDLE) == NULL || bIsWindowsXP) {
++#if defined(MS_WINDOWS) && defined(HAVE_FSTAT)
++    /* dup (DuplicateHandle) doesn't say fd is a valid *file* handle.
++     * It could be a current thread pseudo-handle.
++     */
++    {
++        struct stat buf;
++        if (fstat(fd, &buf) < 0 && (errno == EBADF || errno == ENOENT))
++            return 0;
++    }
 +#else
-     if (!is_valid_fd(fd)) {
++    {
++        int dummy_fd;
++        dummy_fd = dup(fd);
++        if (dummy_fd < 0)
++            return 0;
++        close(dummy_fd);
++    }
 +#endif
-         std = Py_None;
-         Py_INCREF(std);
-     }
-@@ -1099,7 +1111,11 @@
++
+     return 1;
+ }
  
-     /* Set sys.stdout */
-     fd = fileno(stdout);
-+#ifdef MS_WINDOWS
-+    if (!is_valid_fd(fd) || GetStdHandle(STD_OUTPUT_HANDLE) == NULL || bIsWindowsXP) {
-+#else
-     if (!is_valid_fd(fd)) {
-+#endif
-         std = Py_None;
-         Py_INCREF(std);
-     }
-@@ -1115,7 +1131,11 @@
- #if 1 /* Disable this if you have trouble debugging bootstrap stuff */
-     /* Set sys.stderr, replaces the preliminary stderr */
-     fd = fileno(stderr);
-+#ifdef MS_WINDOWS
-+    if (!is_valid_fd(fd) || GetStdHandle(STD_ERROR_HANDLE) == NULL || bIsWindowsXP) {
-+#else
-     if (!is_valid_fd(fd)) {
-+#endif
-         std = Py_None;
-         Py_INCREF(std);
-     }
commit fe64ece0ceda50c08a8fae02175801d0aeb7f9f3
Author: Michael Stahl <mstahl at redhat.com>
Date:   Sat May 9 00:30:45 2015 +0200

    sw: remove bogus assert in SwDoc::IsInHeaderFooter()
    
    If the for loop has a break after the first frame anchored on that node,
    it makes no sense to assert that all frames have been visited.
    
    (regression from bb95f7e6f7c9b1281875e6d729b66b6018794ee0)
    
    Change-Id: Ibd8d65f286b441127be7735c63f4b84ba94dcb6f
    (cherry picked from commit fb0553d6b4905fe2c923a2b5591c001d0be52398)
    Reviewed-on: https://gerrit.libreoffice.org/15687
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/doc/doclay.cxx b/sw/source/core/doc/doclay.cxx
index b2f57f6..6e35170 100644
--- a/sw/source/core/doc/doclay.cxx
+++ b/sw/source/core/doc/doclay.cxx
@@ -1565,9 +1565,6 @@ bool SwDoc::IsInHeaderFooter( const SwNodeIndex& rIdx ) const
             OSL_ENSURE( mbInReading, "Found a FlySection but not a Format!" );
             return false;
         }
-#if OSL_DEBUG_LEVEL > 0
-        assert( checkFmts.empty());
-#endif
     }
 
     return 0 != pNd->FindHeaderStartNode() ||
commit 4b2db8beda661a761d6f6014081c86b58ad6f79f
Author: David Tardon <dtardon at redhat.com>
Date:   Mon Apr 27 16:38:23 2015 +0200

    rhbz#1215443 avoid null pointer dereference
    
    Change-Id: I0a7986703a6997c756d583d0e46907691f807b16
    Signed-off-by: Michael Meeks <michael.meeks at collabora.com>

diff --git a/editeng/source/editeng/impedit3.cxx b/editeng/source/editeng/impedit3.cxx
index c68520d..c8c50a5 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -3473,7 +3473,8 @@ void ImpEditEngine::Paint( OutputDevice* pOutDev, Rectangle aClipRect, Point aSt
                                         }
                                     }
 
-                                    if ( GetStatus().DoOnlineSpelling() && !pPortion->GetNode()->GetWrongList()->empty() && pTextPortion->GetLen() )
+                                    const WrongList* const pWrongList = pPortion->GetNode()->GetWrongList();
+                                    if ( GetStatus().DoOnlineSpelling() && pWrongList && !pWrongList->empty() && pTextPortion->GetLen() )
                                     {
                                         {//#105750# adjust LinePos for superscript or subscript text
                                             short _nEsc = aTmpFont.GetEscapement();
commit 65eec2d5fe33ff593a3bb4c8466b096ff98b4063
Author: Andras Timar <andras.timar at collabora.com>
Date:   Fri May 8 11:38:56 2015 +0200

    tdf#78834 enable localized system dialogs on OS X
    
    Change-Id: I37f5b183b36f2a3077f591162402ddd83e4a46d4
    Reviewed-on: https://gerrit.libreoffice.org/15673
    Reviewed-by: Andras Timar <andras.timar at collabora.com>
    Tested-by: Andras Timar <andras.timar at collabora.com>
    (cherry picked from commit 2df138b483f60ffdcd3b62c8c2884d2f5cc076af)
    Reviewed-on: https://gerrit.libreoffice.org/15674
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/solenv/bin/modules/installer/simplepackage.pm b/solenv/bin/modules/installer/simplepackage.pm
index ae91d0c..d70394e 100644
--- a/solenv/bin/modules/installer/simplepackage.pm
+++ b/solenv/bin/modules/installer/simplepackage.pm
@@ -393,10 +393,16 @@ sub create_package
 
             chdir $localfrom;
         }
-        else
+        elsif ($volume_name_classic_app eq 'LibreOffice' || $volume_name_classic_app eq 'LibreOfficeDev')
         {
-            if (($volume_name_classic_app eq 'LibreOffice' || $volume_name_classic_app eq 'LibreOfficeDev') &&
-                defined($ENV{'MACOSX_CODESIGNING_IDENTITY'}) && $ENV{'MACOSX_CODESIGNING_IDENTITY'} ne "" )
+            my $subdir = "$tempdir/$packagename/$volume_name_classic_app.app/Contents/Resources";
+            if ( ! -d $subdir ) { installer::systemactions::create_directory($subdir); }
+            # iterate over OS X localizations
+            foreach $lang ("ca", "cs", "da", "de", "el", "en", "es", "fi", "fr", "hr", "hu", "id", "it", "ja", "ko", "ms", "nl", "no", "pl", "pt", "pt_PT", "ro", "ru", "sk", "sv", "th", "tr", "uk", "vi", "zh_CN", "zh_TW")
+            {
+                installer::systemactions::create_directory($subdir . "/" . $lang . ".lproj");
+            }
+            if ( defined($ENV{'MACOSX_CODESIGNING_IDENTITY'}) && $ENV{'MACOSX_CODESIGNING_IDENTITY'} ne "" )
             {
                 $systemcall = "$ENV{'SRCDIR'}/solenv/bin/macosx-codesign-app-bundle $localtempdir/$folder/$volume_name_classic_app.app";
                 print "... $systemcall ...\n";


More information about the Libreoffice-commits mailing list