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

Giuseppe Castagno giuseppe.castagno at acca-esse.eu
Tue Dec 15 00:12:56 PST 2015


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

New commits:
commit 9b9329e479615194dda0288d699263412e5c4f1d
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
    
    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>

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