[prev in list] [next in list] [prev in thread] [next in thread] 

List:       varnish-commit
Subject:    [master] 57cbd8450 varnishtest: Remove redundant WRONGs
From:       Dridi Boukelmoune <dridi.boukelmoune () gmail ! com>
Date:       2021-05-28 9:28:05
Message-ID: 20210528092805.58756116ADF () lists ! varnish-cache ! org
[Download RAW message or body]


commit 57cbd8450682336196ccc3c5433e0b4829474d47
Author: Dridi Boukelmoune <dridi.boukelmoune@gmail.com>
Date:   Fri May 28 11:26:07 2021 +0200

    varnishtest: Remove redundant WRONGs
    
    A wrong WRONG doesn't make it right for SunCC.

diff --git a/bin/varnishtest/vtc_tunnel.c b/bin/varnishtest/vtc_tunnel.c
index 4f6f24c7b..231fcc919 100644
--- a/bin/varnishtest/vtc_tunnel.c
+++ b/bin/varnishtest/vtc_tunnel.c
@@ -192,7 +192,6 @@ cmd_tunnel_pause(CMD_ARGS)
 	if (t->state == TUNNEL_PAUSED) {
 		AZ(pthread_mutex_unlock(&t->mtx));
 		vtc_fatal(vl, "Tunnel already paused");
-		WRONG("unreachable");
 	}
 	assert(t->state == TUNNEL_RUNNING);
 	t->state = TUNNEL_PAUSED;
@@ -230,7 +229,6 @@ cmd_tunnel_send(CMD_ARGS)
 	if (t->state == TUNNEL_RUNNING) {
 		AZ(pthread_mutex_unlock(&t->mtx));
 		vtc_fatal(vl, "Tunnel still running");
-		WRONG("unreachable");
 	}
 	assert(t->state == TUNNEL_PAUSED);
 	AZ(t->send_lane->wrk_len);
@@ -285,7 +283,6 @@ cmd_tunnel_resume(CMD_ARGS)
 	if (t->state == TUNNEL_RUNNING) {
 		AZ(pthread_mutex_unlock(&t->mtx));
 		vtc_fatal(vl, "Tunnel already running");
-		WRONG("unreachable");
 	}
 	assert(t->state == TUNNEL_PAUSED);
 	t->state = TUNNEL_RUNNING;
_______________________________________________
varnish-commit mailing list
varnish-commit@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit
[prev in list] [next in list] [prev in thread] [next in thread]