Its the same logic <div class="gmail_quote">On Sun, Jun 5, 2011 at 6:44 AM, Chr. Rossmanith <<a href="mailto:ChrRossmanith@gmx.de">ChrRossmanith@gmx.de</a>> wrote: <blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> Hi, could someone please double check that the new if statement is equivalent to the old one... sal/osl/unx/file_url.cxx | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sal/osl/unx/file_url.cxx b/sal/osl/unx/file_url.cxx index a005cc8..858a7d7 100644 --- a/sal/osl/unx/file_url.cxx +++ b/sal/osl/unx/file_url.cxx @@ -762,7 +762,7 @@ oslFileError osl_searchFileURL(rtl_uString* ustrFilePath, rtl_uString* ustrSearc // try to interpret search path as file url else assume it's a system path list rc = FileBase::getSystemPathFromFileURL(rtl::OUString(ustrFilePath), file_path); - if ((FileBase::E_None != rc) && (FileBase::E_INVAL == rc)) + if (FileBase::E_INVAL == rc) file_path = ustrFilePath; else if (FileBase::E_None != rc) return oslFileError(rc); I'll push it if it's ok. Christina _______________________________________________ LibreOffice mailing list <a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a> </blockquote></div>