This is an automated email from the ASF dual-hosted git repository.
gancho pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new 98860a7 coverity 1021924: Missing break in switch
98860a7 is described below
commit 98860a747d1d660c1d996330b8e155a887872517
Author: Gancho Tenev <gan...@apache.com>
AuthorDate: Mon May 8 08:43:34 2017 -0700
coverity 1021924: Missing break in switch
Problem:
CID 1021924 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: The case for value TS_EVENT_VCONN_WRITE_READY is not
terminated by a 'break' statement.
Solution:
It was intended not to have a break (not a bug), so refactored the code
to make coverity happy.
---
example/null_transform/null_transform.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/example/null_transform/null_transform.c
b/example/null_transform/null_transform.c
index 7a55525..cf60424 100644
--- a/example/null_transform/null_transform.c
+++ b/example/null_transform/null_transform.c
@@ -232,14 +232,17 @@ null_transform(TSCont contp, TSEvent event, void *edata
ATS_UNUSED)
*/
TSVConnShutdown(TSTransformOutputVConnGet(contp), 0, 1);
break;
+
+ /* If we get a WRITE_READY event or any other type of
+ * event (sent, perhaps, because we were re-enabled) then
+ * we'll attempt to transform more data.
+ */
case TS_EVENT_VCONN_WRITE_READY:
TSDebug(PLUGIN_NAME, "\tEvent is TS_EVENT_VCONN_WRITE_READY");
+ handle_transform(contp);
+ break;
default:
TSDebug(PLUGIN_NAME, "\t(event is %d)", event);
- /* If we get a WRITE_READY event or any other type of
- * event (sent, perhaps, because we were reenabled) then
- * we'll attempt to transform more data.
- */
handle_transform(contp);
break;
}
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <commits@trafficserver.apache.org>'].
