[thrift] branch master updated: CPP generator test, catching exceptions by ref instead.

[jk...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jking pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new 29ecfab  CPP generator test, catching exceptions by ref instead.
29ecfab is described below

commit 29ecfaba536984e0a41fb1f1bcb1520640dd5471
Author: David Carlier <dc...@afilias.info>
AuthorDate: Fri Jun 14 13:01:31 2019 +0000

    CPP generator test, catching exceptions by ref instead.
---
 lib/cpp/test/processor/EventLog.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/cpp/test/processor/EventLog.cpp b/lib/cpp/test/processor/EventLog.cpp
index c8e0d9b..c75955d 100644
--- a/lib/cpp/test/processor/EventLog.cpp
+++ b/lib/cpp/test/processor/EventLog.cpp
@@ -98,7 +98,7 @@ Event EventLog::waitForEvent(int64_t timeout) {
     while (events_.empty()) {
       monitor_.wait(timeout);
     }
-  } catch (TimedOutException ex) {
+  } catch (const TimedOutException &) {
     return Event(ET_LOG_END, 0, 0, "");
   }
 
@@ -119,7 +119,7 @@ Event EventLog::waitForConnEvent(uint32_t connId, int64_t timeout) {
       while (it == events_.end()) {
         monitor_.wait(timeout);
       }
-    } catch (TimedOutException ex) {
+    } catch (const TimedOutException &) {
       return Event(ET_LOG_END, 0, 0, "");
     }