[PATCH 2/2] Handle streaming fail of the source object during condfetch

[Poul-Henning Kamp]

--------
In message <1418750241-13168-2-git-send-email-martin at varnish-software.com>, Mar
tin Blix Grydeland writes:

>Fixes: #1648

>+		if (ois == OIS_ERROR) {
>+			VFP_Error(bo->vfc, "Conditional fetch object failed");
>+		}

unnecessary {...}


>-	if (bo->vfc->failed)
>+	if (bo->ims_oc->flags & OC_F_FAILED)
>+		VFP_Error(bo->vfc, "Conditional fetch object failed");

That text confused me for a moment and I suspect it will have that
effect on other people as well.

Maybe we should call the object we IMS from "the template object" or
some such ?  ("model" ?  "stencil" ? ...)


Otherwise it looks fine.


-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.