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

[Julien Nabet]

 avmedia/source/gstreamer/gstplayer.cxx |    2 +-
 basic/source/runtime/methods.cxx       |    8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

New commits:
commit 41392516ca0fc099c383c4319923b895ecc16be6
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Sat Apr 18 23:13:51 2015 +0200

    tdf#55502: Shell BASIC function with empty third param hangs
    
    bt:
    2  0x00002aaaab2b5226 in __assert_fail_base (fmt=0x2aaaab3ebce8 "%s%s%s:%u: %s%sAssertion `%s' failed.
    %n",
        assertion=assertion at entry=0x2aaaae2ecf10 "index >= 0 && static_cast<sal_uInt32>(index) < static_cast<sal_uInt32>(getLength())",
        file=file at entry=0x2aaaae2ecec8 "/home/julien/compile-libreoffice/libreoffice/include/rtl/ustring.hxx", line=line at entry=500,
        function=function at entry=0x2aaaae2ed740 <rtl::OUString::operator[](int) const::__PRETTY_FUNCTION__> "sal_Unicode rtl::OUString::operator[](sal_Int32) const") at assert.c:92
    3  0x00002aaaab2b52d2 in __GI___assert_fail (assertion=0x2aaaae2ecf10 "index >= 0 && static_cast<sal_uInt32>(index) < static_cast<sal_uInt32>(getLength())",
        file=0x2aaaae2ecec8 "/home/julien/compile-libreoffice/libreoffice/include/rtl/ustring.hxx", line=500,
        function=0x2aaaae2ed740 <rtl::OUString::operator[](int) const::__PRETTY_FUNCTION__> "sal_Unicode rtl::OUString::operator[](sal_Int32) const") at assert.c:101
    4  0x00002aaaae1f44f8 in rtl::OUString::operator[] (this=0x7fffffff36b0, index=13) at /home/julien/compile-libreoffice/libreoffice/include/rtl/ustring.hxx:500
    5  0x00002aaaae274036 in SbRtl_Shell (pBasic=0x3531fc0, rPar=..., bWrite=false) at /home/julien/compile-libreoffice/libreoffice/basic/source/runtime/methods.cxx:3613
    Check if third param isn't empty, if not, aCmdLine finishes with a space character
    
    Change-Id: I095b6dc7c7d2d4c826fb47aa1414c558bbc27c04

diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 17e2c5f..3b84e4b 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -3590,8 +3590,12 @@ RTLFUNC(Shell)
         // attach additional parameters - everything must be parsed anyway
         if( nArgCount >= 4 )
         {
-            aCmdLine += " ";
-            aCmdLine += rPar.Get(3)->GetOUString();
+            OUString tmp = rPar.Get(3)->GetOUString();
+            if (!tmp.isEmpty())
+            {
+                aCmdLine += " ";
+                aCmdLine += tmp;
+            }
         }
         else if( aCmdLine.isEmpty() )
         {
commit d1aae56014a3e8fa2552ce178a2374c31cf5de12
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Sat Apr 18 23:12:30 2015 +0200

    Typo: thow->throw
    
    Change-Id: I3db188dbd7db19d69dc2683fac3e912d1aef8847

diff --git a/avmedia/source/gstreamer/gstplayer.cxx b/avmedia/source/gstreamer/gstplayer.cxx
index 88e62cf..111e7ae 100644
--- a/avmedia/source/gstreamer/gstplayer.cxx
+++ b/avmedia/source/gstreamer/gstplayer.cxx
@@ -91,7 +91,7 @@ Player::Player( const uno::Reference< lang::XMultiServiceFactory >& rxMgr ) :
 
     if (pError != NULL)
     {
-        // TODO: thow an exception?
+        // TODO: throw an exception?
         DBG( "%p Player::Player error '%s'", this, pError->message );
         g_error_free (pError);
     }