Retire fetch return option from vcl_hit{}

This has been superseded by return (miss).

bin/varnishd/cache/cache_req_fsm.c

@@ -439,10 +439,6 @@ cnt_lookup(struct worker *wrk, struct req *req)
 		req->is_hit = 1;
 		req->req_step = R_STP_DELIVER;
 		return (REQ_FSM_MORE);
-	case VCL_RET_FETCH:
-		VSLb(req->vsl, SLT_VCL_Error,
-		    "change return(fetch) to return(miss) in vcl_hit{}");
-		/* FALL-THROUGH */
 	case VCL_RET_MISS:
 		if (busy != NULL) {
 			req->objcore = busy;

bin/varnishtest/tests/r01335.vtc

@@ -10,7 +10,7 @@ server s1 {
 varnish v1 -vcl+backend {
 	sub vcl_hit {
 		if (req.http.two == "2") {
-			return (fetch);		// also #1603
+			return (miss);		// also #1603
 		}
 	}
 } -start

lib/libvcc/generate.py

@@ -109,7 +109,7 @@ returns =(
 	),
 	('hit',
 		"C",
-		('synth', 'restart', 'pass', 'fetch', 'miss', 'deliver',)
+		('synth', 'restart', 'pass', 'miss', 'deliver',)
 	),
 	('deliver',
 		"C",