r750 - trunk/varnish-cache/lib/libvcl
phk at projects.linpro.no
phk at projects.linpro.no
Mon Aug 7 22:47:17 CEST 2006
Author: phk
Date: 2006-08-07 22:47:16 +0200 (Mon, 07 Aug 2006)
New Revision: 750
Modified:
trunk/varnish-cache/lib/libvcl/vcc_acl.c
trunk/varnish-cache/lib/libvcl/vcc_compile.c
trunk/varnish-cache/lib/libvcl/vcc_priv.h
trunk/varnish-cache/lib/libvcl/vcc_token.c
Log:
Eliminate use of extensible printf
Modified: trunk/varnish-cache/lib/libvcl/vcc_acl.c
===================================================================
--- trunk/varnish-cache/lib/libvcl/vcc_acl.c 2006-08-07 20:24:47 UTC (rev 749)
+++ trunk/varnish-cache/lib/libvcl/vcc_acl.c 2006-08-07 20:47:16 UTC (rev 750)
@@ -34,7 +34,8 @@
vcc_NextToken(tl);
ExpectErr(tl, ID);
AddRef(tl, tl->t, R_ACL);
- Fc(tl, 1, "VRT_acl_match(sp, \"%T\", acl_%T)\n", tl->t, tl->t);
+ Fc(tl, 1, "VRT_acl_match(sp, \"%.*s\", acl_%.*s)\n",
+ PF(tl->t), PF(tl->t));
vcc_NextToken(tl);
break;
default:
@@ -61,8 +62,8 @@
vcc_NextToken(tl);
AddDef(tl, an, R_ACL);
- Fh(tl, 0, "static struct vrt_acl acl_%T[];\n", an);
- Fc(tl, 1, "static struct vrt_acl acl_%T[] = {\n", an);
+ Fh(tl, 0, "static struct vrt_acl acl_%.*s[];\n", PF(an));
+ Fc(tl, 1, "static struct vrt_acl acl_%.*s[] = {\n", PF(an));
tl->indent += INDENT;
@@ -92,7 +93,7 @@
ExpectErr(tl, CNUM);
mask = UintVal(tl);
}
- Fc(tl, 1, "{ %u, %u, %u, %T, \"", not, mask, para, t);
+ Fc(tl, 1, "{ %u, %u, %u, %.*s, \"", not, mask, para, PF(t));
if (para)
Fc(tl, 0, "(");
if (not)
@@ -120,6 +121,6 @@
ExpectErr(tl, '}');
vcc_NextToken(tl);
- Fi(tl, 1, "\tVRT_acl_init(acl_%T);\n", an);
- Ff(tl, 1, "\tVRT_acl_fini(acl_%T);\n", an);
+ Fi(tl, 1, "\tVRT_acl_init(acl_%.*s);\n", PF(an));
+ Ff(tl, 1, "\tVRT_acl_fini(acl_%.*s);\n", PF(an));
}
Modified: trunk/varnish-cache/lib/libvcl/vcc_compile.c
===================================================================
--- trunk/varnish-cache/lib/libvcl/vcc_compile.c 2006-08-07 20:24:47 UTC (rev 749)
+++ trunk/varnish-cache/lib/libvcl/vcc_compile.c 2006-08-07 20:47:16 UTC (rev 750)
@@ -43,7 +43,6 @@
#include <fcntl.h>
#include <netdb.h>
#include <stdio.h>
-#include <printf.h>
#include <stdarg.h>
#include <stdlib.h>
#include <string.h>
@@ -505,7 +504,7 @@
Fc(tl, 1, "VRT_re_match(%s, %s)\n", str, buf);
Fh(tl, 0, "void *%s;\n", buf);
- Fi(tl, 0, "\tVRT_re_init(&%s, %T);\n", buf, re);
+ Fi(tl, 0, "\tVRT_re_init(&%s, %.*s);\n", buf, PF(re));
Ff(tl, 0, "\tVRT_re_fini(%s);\n", buf);
}
@@ -529,7 +528,7 @@
tl->t->tok == T_EQ ? "!" : "", vp->rname);
vcc_NextToken(tl);
ExpectErr(tl, CSTR);
- Fc(tl, 0, "%T)\n", tl->t);
+ Fc(tl, 0, "%.*s)\n", PF(tl->t));
vcc_NextToken(tl);
break;
default:
@@ -550,7 +549,7 @@
case T_GEQ:
case '>':
case '<':
- Fc(tl, 0, "%T ", tl->t);
+ Fc(tl, 0, "%.*s ", PF(tl->t));
vcc_NextToken(tl);
switch(vp->fmt) {
case TIME:
@@ -558,7 +557,7 @@
break;
case INT:
ExpectErr(tl, CNUM);
- Fc(tl, 0, "%T ", tl->t);
+ Fc(tl, 0, "%.*s ", PF(tl->t));
vcc_NextToken(tl);
break;
case SIZE:
@@ -755,7 +754,7 @@
a = 0;
Fc(tl, 1, "VRT_error(sp, %u, ", a);
if (tl->t->tok == CSTR) {
- Fc(tl, 0, "%T);\n", tl->t);
+ Fc(tl, 0, "%.*s);\n", PF(tl->t));
vcc_NextToken(tl);
} else
Fc(tl, 0, "(const char *)0);\n");
@@ -763,23 +762,23 @@
return;
case T_SWITCH_CONFIG:
ExpectErr(tl, ID);
- Fc(tl, 1, "VCL_switch_config(\"%T\");\n", tl->t);
+ Fc(tl, 1, "VCL_switch_config(\"%.*s\");\n", PF(tl->t));
vcc_NextToken(tl);
return;
case T_CALL:
ExpectErr(tl, ID);
AddCall(tl, tl->t);
AddRef(tl, tl->t, R_FUNC);
- Fc(tl, 1, "if (VGC_function_%T(sp))\n", tl->t);
+ Fc(tl, 1, "if (VGC_function_%.*s(sp))\n", PF(tl->t));
Fc(tl, 1, "\treturn (1);\n");
vcc_NextToken(tl);
return;
case T_REWRITE:
ExpectErr(tl, CSTR);
- Fc(tl, 1, "VCL_rewrite(%T", tl->t);
+ Fc(tl, 1, "VCL_rewrite(%.*s", PF(tl->t));
vcc_NextToken(tl);
ExpectErr(tl, CSTR);
- Fc(tl, 0, ", %T);\n", tl->t);
+ Fc(tl, 0, ", %.*s);\n", PF(tl->t));
vcc_NextToken(tl);
return;
case T_SET:
@@ -835,7 +834,7 @@
if (tl->t->tok == '=') {
vcc_NextToken(tl);
AddRef(tl, tl->t, R_BACKEND);
- Fc(tl, 0, "= &VGC_backend_%T;\n", tl->t);
+ Fc(tl, 0, "= &VGC_backend_%.*s;\n", PF(tl->t));
vcc_NextToken(tl);
break;
}
@@ -934,15 +933,15 @@
AddDef(tl, tl->t, R_BACKEND);
if (tl->nbackend == 0)
AddRef(tl, tl->t, R_BACKEND);
- Fh(tl, 1, "#define VGC_backend_%T (VCL_conf.backend[%d])\n",
- tl->t, tl->nbackend);
+ Fh(tl, 1, "#define VGC_backend_%.*s (VCL_conf.backend[%d])\n",
+ PF(tl->t), tl->nbackend);
Fc(tl, 0, "\n");
Fc(tl, 0, "static void\n");
- Fc(tl, 1, "VGC_init_backend_%T (void)\n", tl->t);
+ Fc(tl, 1, "VGC_init_backend_%.*s (void)\n", PF(tl->t));
Fc(tl, 1, "{\n");
- Fc(tl, 1, "\tstruct backend *backend = VGC_backend_%T;\n", tl->t);
+ Fc(tl, 1, "\tstruct backend *backend = VGC_backend_%.*s;\n", PF(tl->t));
Fc(tl, 1, "\n");
- Fc(tl, 1, "\tVRT_set_backend_name(backend, \"%T\");\n", tl->t);
+ Fc(tl, 1, "\tVRT_set_backend_name(backend, \"%.*s\");\n", PF(tl->t));
vcc_NextToken(tl);
ExpectErr(tl, '{');
vcc_NextToken(tl);
@@ -962,13 +961,13 @@
case HOSTNAME:
ExpectErr(tl, CSTR);
t_host = tl->t;
- Fc(tl, 1, "\t%s %T);\n", vp->lname, tl->t);
+ Fc(tl, 1, "\t%s %.*s);\n", vp->lname, PF(tl->t));
vcc_NextToken(tl);
break;
case PORTNAME:
ExpectErr(tl, CSTR);
t_port = tl->t;
- Fc(tl, 1, "\t%s %T);\n", vp->lname, tl->t);
+ Fc(tl, 1, "\t%s %.*s);\n", vp->lname, PF(tl->t));
vcc_NextToken(tl);
break;
default:
@@ -982,14 +981,15 @@
}
ExpectErr(tl, '}');
if (t_host == NULL) {
- vsb_printf(tl->sb, "Backend '%T' has no hostname\n", t_be);
+ vsb_printf(tl->sb, "Backend '%.*s' has no hostname\n",
+ PF(t_be));
vcc_ErrWhere(tl, tl->t);
return;
}
host = EncString(t_host);
ep = CheckHostPort(host, "80");
if (ep != NULL) {
- vsb_printf(tl->sb, "Backend '%T': %s\n", t_be, ep);
+ vsb_printf(tl->sb, "Backend '%.*s': %s\n", PF(t_be), ep);
vcc_ErrWhere(tl, t_host);
return;
}
@@ -997,7 +997,7 @@
port = EncString(t_port);
ep = CheckHostPort(host, port);
if (ep != NULL) {
- vsb_printf(tl->sb, "Backend '%T': %s\n", t_be, ep);
+ vsb_printf(tl->sb, "Backend '%.*s': %s\n", PF(t_be), ep);
vcc_ErrWhere(tl, t_port);
return;
}
@@ -1006,8 +1006,8 @@
vcc_NextToken(tl);
Fc(tl, 1, "}\n");
Fc(tl, 0, "\n");
- Fi(tl, 0, "\tVGC_init_backend_%T();\n", t_be);
- Ff(tl, 0, "\tVRT_fini_backend(VGC_backend_%T);\n", t_be);
+ Fi(tl, 0, "\tVGC_init_backend_%.*s();\n", PF(t_be));
+ Ff(tl, 0, "\tVRT_fini_backend(VGC_backend_%.*s);\n", PF(t_be));
tl->nbackend++;
}
@@ -1024,15 +1024,16 @@
tl->curproc->exists++;
tn = tl->t;
AddDef(tl, tl->t, R_FUNC);
- Fh(tl, 0, "static int VGC_function_%T (struct sess *sp);\n", tl->t);
+ Fh(tl, 0, "static int VGC_function_%.*s (struct sess *sp);\n",
+ PF(tl->t));
Fc(tl, 1, "static int\n");
- Fc(tl, 1, "VGC_function_%T (struct sess *sp)\n", tl->t);
+ Fc(tl, 1, "VGC_function_%.*s (struct sess *sp)\n", PF(tl->t));
vcc_NextToken(tl);
tl->indent += INDENT;
Fc(tl, 1, "{\n");
L(tl, Compound(tl));
if (IsMethod(tn) == 1) {
- Fc(tl, 1, "VGC_function_default_%T(sp);\n", tn);
+ Fc(tl, 1, "VGC_function_default_%.*s(sp);\n", PF(tn));
}
Fc(tl, 1, "}\n");
tl->indent -= INDENT;
@@ -1124,7 +1125,8 @@
struct proccall *pc;
if (!p->exists) {
- vsb_printf(tl->sb, "Function %T does not exist\n", p->name);
+ vsb_printf(tl->sb, "Function %.*s does not exist\n",
+ PF(p->name));
return (1);
}
if (p->active) {
@@ -1225,19 +1227,20 @@
}
if (r->defcnt == 0 && r->name->tok == METHOD) {
vsb_printf(tl->sb,
- "No definition for method %T\n", r->name);
+ "No definition for method %.*s\n", PF(r->name));
continue;
}
if (r->defcnt == 0) {
vsb_printf(tl->sb,
- "Undefined %s %T, first reference:\n",
- type, r->name);
+ "Undefined %s %.*s, first reference:\n",
+ type, PF(r->name));
vcc_ErrWhere(tl, r->name);
continue;
}
- vsb_printf(tl->sb, "Unused %s %T, defined:\n", type, r->name);
+ vsb_printf(tl->sb, "Unused %s %.*s, defined:\n",
+ type, PF(r->name));
vcc_ErrWhere(tl, r->name);
}
return (nerr);
@@ -1279,8 +1282,8 @@
pos++;
}
- Fc(tl, 0, " [%3u] = { %d, %4u, %3u, 0, \"%T\" },\n",
- t->cnt, fil, lin, pos + 1, t);
+ Fc(tl, 0, " [%3u] = { %d, %4u, %3u, 0, \"%.*s\" },\n",
+ t->cnt, fil, lin, pos + 1, PF(t));
}
Fc(tl, 0, "};\n");
}
@@ -1477,31 +1480,6 @@
/*--------------------------------------------------------------------*/
-static int
-VCC_T_render(FILE *f, const struct printf_info *info, const void *const *args)
-{
- const struct token *t;
-
- (void)info;
-
- t = *((const struct token * const*) (args[0]));
- return (fprintf(f, "%*.*s",
- t->e - t->b, t->e - t->b, t->b));
-}
-
-static int
-VCC_T_arginfo(const struct printf_info *info, size_t n, int *argtypes)
-{
-
- (void)info;
-
- if (n > 0)
- argtypes[0] = PA_POINTER;
- return 1;
-}
-
-/*--------------------------------------------------------------------*/
-
void
VCC_InitCompile(const char *default_vcl)
{
@@ -1511,7 +1489,6 @@
vcc_default_vcl_e = strchr(default_vcl, '\0');
assert(vcc_default_vcl_e != NULL);
- register_printf_function ('T', VCC_T_render, VCC_T_arginfo);
vcl_init_tnames();
for (v = vcc_vars; v->name != NULL; v++)
v->len = strlen(v->name);
Modified: trunk/varnish-cache/lib/libvcl/vcc_priv.h
===================================================================
--- trunk/varnish-cache/lib/libvcl/vcc_priv.h 2006-08-07 20:24:47 UTC (rev 749)
+++ trunk/varnish-cache/lib/libvcl/vcc_priv.h 2006-08-07 20:47:16 UTC (rev 750)
@@ -11,3 +11,5 @@
extern const char *vcl_tnames[256];
void vcl_init_tnames(void);
void vcl_output_lang_h(FILE *f);
+
+#define PF(t) ((t)->e - (t)->b), (t)->b
Modified: trunk/varnish-cache/lib/libvcl/vcc_token.c
===================================================================
--- trunk/varnish-cache/lib/libvcl/vcc_token.c 2006-08-07 20:24:47 UTC (rev 749)
+++ trunk/varnish-cache/lib/libvcl/vcc_token.c 2006-08-07 20:47:16 UTC (rev 750)
@@ -26,7 +26,7 @@
if (t->tok == EOI)
vsb_printf(tl->sb, "end of input");
else
- vsb_printf(tl->sb, "'%T'", t);
+ vsb_printf(tl->sb, "'%.*s'", PF(t));
}
void
More information about the varnish-commit
mailing list