<div dir="auto">That was left on purpose, we can discuss that tomorrow. There's a pull request to remove things that overlap with vmod vtc after the 5.2 release, phk wanted to keep them in despite vmod debug not being installed.<div dir="auto"><br></div><div dir="auto">Dridi</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sep 10, 2017 22:48, "Federico G. Schwindt" <<a href="mailto:fgsch@lodoss.net">fgsch@lodoss.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
commit ddf99e4e8202c7b4831048ed0bfbed<wbr>1844d46806<br>
Author: Federico G. Schwindt <<a href="mailto:fgsch@lodoss.net">fgsch@lodoss.net</a>><br>
Date:   Sun Sep 10 19:02:24 2017 +0200<br>
<br>
    Remove blob operations from vmod-debug<br>
<br>
    Missed in 32877991.<br>
<br>
diff --git a/lib/libvmod_debug/vmod.vcc b/lib/libvmod_debug/vmod.vcc<br>
index 5a3ce7e..324d389 100644<br>
--- a/lib/libvmod_debug/vmod.vcc<br>
+++ b/lib/libvmod_debug/vmod.vcc<br>
@@ -64,14 +64,6 @@ $Function STRING test_priv_top(PRIV_TOP, STRING)<br>
<br>
 Test function for TOP private pointers<br>
<br>
-$Function BLOB str2blob(STRING src="foo")<br>
-<br>
-Turn a string into a blob<br>
-<br>
-$Function STRING blob2hex(BLOB src)<br>
-<br>
-Hexdump a blob<br>
-<br>
 $Function BACKEND no_backend()<br>
<br>
 Fails at backend selection<br>
diff --git a/lib/libvmod_debug/vmod_<wbr>debug.c b/lib/libvmod_debug/vmod_<wbr>debug.c<br>
index f4fac46..09312cf 100644<br>
--- a/lib/libvmod_debug/vmod_<wbr>debug.c<br>
+++ b/lib/libvmod_debug/vmod_<wbr>debug.c<br>
@@ -151,42 +151,6 @@ vmod_test_priv_vcl(VRT_CTX, struct vmod_priv *priv)<br>
        assert(!strcmp(priv_vcl->foo, "FOO"));<br>
 }<br>
<br>
-VCL_BLOB<br>
-vmod_str2blob(VRT_CTX, VCL_STRING s)<br>
-{<br>
-       struct vmod_priv *p;<br>
-<br>
-       CHECK_OBJ_NOTNULL(ctx, VRT_CTX_MAGIC);<br>
-       WARN_RETIRED();<br>
-       p = (void*)WS_Alloc(ctx->ws, sizeof *p);<br>
-       AN(p);<br>
-       memset(p, 0, sizeof *p);<br>
-       p->len = strlen(s);<br>
-       p->priv = WS_Copy(ctx->ws, s, -1);<br>
-       return (p);<br>
-}<br>
-<br>
-VCL_STRING<br>
-vmod_blob2hex(VRT_CTX, VCL_BLOB b)<br>
-{<br>
-       char *s, *p;<br>
-       uint8_t *q;<br>
-       int i;<br>
-<br>
-       WARN_RETIRED();<br>
-       s = WS_Alloc(ctx->ws, b->len * 2 + 2);<br>
-       AN(s);<br>
-       p = s;<br>
-       q = b->priv;<br>
-       for (i = 0; i < b->len; i++) {<br>
-               assert(snprintf(p, 3, "%02x", *q) == 2);<br>
-               p += 2;<br>
-               q += 1;<br>
-       }<br>
-       VRT_priv_fini(b);<br>
-       return (s);<br>
-}<br>
-<br>
 VCL_BACKEND<br>
 vmod_no_backend(VRT_CTX)<br>
 {<br>
______________________________<wbr>_________________<br>
varnish-commit mailing list<br>
<a href="mailto:varnish-commit@varnish-cache.org">varnish-commit@varnish-cache.<wbr>org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit" rel="noreferrer" target="_blank">https://www.varnish-cache.org/<wbr>lists/mailman/listinfo/<wbr>varnish-commit</a><br>
</blockquote></div></div>