mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

If we are going to warn about an "invalid" URL, at least tell what it is

Browse Source 
As such the code handles it perfectly fine, and returns an error from
the function, but.

Change-Id: I356b8140381d3ccd21ff0a7f5c666552571b12f4
This commit is contained in:
Tor Lillqvist
2017-08-02 11:28:39 +03:00
parent 8a24df0bed
commit 13cadf3fe3
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
sal/osl/unx/file_misc.cxx
View File

@@ -347,8 +347,9 @@ oslFileError SAL_CALL osl_getNextDirectoryItem(oslDirectory pDirectory,
oslFileError SAL_CALL osl_getDirectoryItem(rtl_uString* ustrFileURL, oslDirectoryItem* pItem)
{
SAL_WARN_IF((!ustrFileURL) || (ustrFileURL->length == 0), "sal.file", "Invalid file URL");
SAL_WARN_IF(!pItem, "sal.file", "pItem is nullptr");
SAL_WARN_IF((!ustrFileURL) || (ustrFileURL->length == 0), "sal.file",
"Invalid file URL " << (ustrFileURL ? ("'" + OUString(ustrFileURL) + "'") : OUString("(null)")));
SAL_WARN_IF(!pItem, "sal.file", "pItem is null");
rtl_uString* ustrSystemPath = nullptr;
oslFileError osl_error = osl_File_E_INVAL;