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

Giuseppe Castagno giuseppe.castagno at acca-esse.eu
Tue Dec 15 07:07:26 PST 2015


 ucb/source/ucp/webdav-neon/NeonPropFindRequest.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit edc742e4975024dd28d3b8e0c900bbe1b466d08e
Author: Giuseppe Castagno <giuseppe.castagno at acca-esse.eu>
Date:   Thu Dec 3 15:48:40 2015 +0100

    Some WebDAV server answer without 'DAV:' XML namespace.
    
    The call to assert() is not appropriate, better use a SAL_WARN_IF
    instead.
    
    The call to DAVProperties::createUCBPropName() method immediately
    after will take care of the missing namespace.
    
    The change to assert() was introduced in commit
    dd9c15b36f69bec4d4fc9b0049f7e496e5eb9e9d
    
    In libreoffice-5-1 branch it was introduced in commit
    0ee79446c8c6daa8426eb2948b6e607b14e50059
    
    Change-Id: Ib9b85426fc84907fb12f70759a189cedcf117b3a
    Reviewed-on: https://gerrit.libreoffice.org/20380
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>
    (cherry picked from commit 9b9329e479615194dda0288d699263412e5c4f1d)
    Reviewed-on: https://gerrit.libreoffice.org/20711
    Reviewed-by: Giuseppe Castagno <giuseppe.castagno at acca-esse.eu>
    Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice at googlemail.com>
    Tested-by: Christian Lohmaier <lohmaier+LibreOffice at googlemail.com>

diff --git a/ucb/source/ucp/webdav-neon/NeonPropFindRequest.cxx b/ucb/source/ucp/webdav-neon/NeonPropFindRequest.cxx
index 33ff51a..ca73cd2 100644
--- a/ucb/source/ucp/webdav-neon/NeonPropFindRequest.cxx
+++ b/ucb/source/ucp/webdav-neon/NeonPropFindRequest.cxx
@@ -109,7 +109,7 @@ extern "C" int NPFR_propfind_iter( void* userdata,
     DAVPropertyValue thePropertyValue;
     thePropertyValue.IsCaseSensitive = true;
 
-    assert( pname->nspace && "NPFR_propfind_iter - No namespace!" );
+    SAL_WARN_IF( !pname->nspace, "ucb.ucp.webdav", "NPFR_propfind_iter - No XML namespace!" );
 
     DAVProperties::createUCBPropName( pname->nspace,
                                       pname->name,


More information about the Libreoffice-commits mailing list