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

Noel Grandin (via logerrit) logerrit at kemper.freedesktop.org
Mon Jun 7 20:06:00 UTC 2021


 binaryurp/source/reader.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 039180f8255b6356cb89b1777e4e53323541898d
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Mon Jun 7 20:55:51 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Mon Jun 7 22:05:23 2021 +0200

    Revert "improve logging message"
    
    This reverts commit 9f0484c2bca8e7ec5e0d83289fae9b474b8598e0.
    
    Reason for revert: this patch was unnecessary, the operator<< for css::uno::Exception alreadys logs the message, I got confused
    
    Change-Id: I45932be060059eac92d460a70c944e1b19b5c375
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116706
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/binaryurp/source/reader.cxx b/binaryurp/source/reader.cxx
index 902225e1bd54..b11b77ffe658 100644
--- a/binaryurp/source/reader.cxx
+++ b/binaryurp/source/reader.cxx
@@ -120,9 +120,9 @@ void Reader::execute() {
             block.done();
         }
     } catch (const css::uno::Exception & e) {
-        SAL_WARN("binaryurp", "caught UNO exception, " << e << " " << e.Message);
+        SAL_WARN("binaryurp", "caught UNO exception '" << e << '\'');
     } catch (const std::exception & e) {
-        SAL_WARN("binaryurp", "caught C++ exception, " << e.what() );
+        SAL_WARN("binaryurp", "caught C++ exception '" << e.what() << '\'');
     }
     bridge_->terminate(false);
     bridge_.clear();


More information about the Libreoffice-commits mailing list