r4904 - in trunk/varnish-cache/bin: . varnishd varnishncsa varnishtest
phk at varnish-cache.org
phk at varnish-cache.org
Sat Jun 5 15:30:56 CEST 2010
Author: phk
Date: 2010-06-05 15:30:55 +0200 (Sat, 05 Jun 2010)
New Revision: 4904
Modified:
trunk/varnish-cache/bin/flint.lnt
trunk/varnish-cache/bin/varnishd/flint.lnt
trunk/varnish-cache/bin/varnishncsa/flint.lnt
trunk/varnish-cache/bin/varnishtest/flint.lnt
Log:
More .lnt centralizing
Modified: trunk/varnish-cache/bin/flint.lnt
===================================================================
--- trunk/varnish-cache/bin/flint.lnt 2010-06-05 13:21:13 UTC (rev 4903)
+++ trunk/varnish-cache/bin/flint.lnt 2010-06-05 13:30:55 UTC (rev 4904)
@@ -19,6 +19,43 @@
-sem(vas_fail, r_no)
///////////////////////////////////////////////////////////////////////
+// <vqueue.h>
+
+-esym(755, VLIST_*) // Global macro not ref.
+-esym(755, VSLIST_*)
+-esym(755, VSTAILQ_*)
+-esym(755, VTAILQ_*)
+
+-emacro((826), VTAILQ_LAST) // Suspicious pointer-to-pointer conversion (area too small)
+-emacro((826), VTAILQ_PREV) // Suspicious pointer-to-pointer conversion (area too small)
+-emacro(506, VTAILQ_FOREACH_SAFE) // constant value boolean
+-emacro(506, VSTAILQ_FOREACH_SAFE) // constant value boolean
+-emacro(740, VTAILQ_LAST) // Unusual pointer cast (incompatible indirect types)
+-emacro(740, VTAILQ_PREV) // Unusual pointer cast (incompatible indirect types)
+-esym(755, VTAILQ_*)
+
+///////////////////////////////////////////////////////////////////////
+// miniobj
+
+-emacro(755, CAST_OBJ)
+
+///////////////////////////////////////////////////////////////////////
+// VSB
+
+-esym(759, vsb_*) // header decl could be moved
+-esym(765, vsb_*) // exten could be made static
+-esym(714, vsb_*) // symb not ref
+-sem(vsb_new, @p == (1p ? 1p : malloc(1)))
+-sem(vsb_delete, custodial(1))
+
+-esym(534, vsb_cat) // Ignore retval
+-esym(534, vsb_bcat)
+-esym(534, vsb_putc)
+-esym(534, vsb_printf)
+-esym(534, vsb_vprintf)
+-esym(534, vsb_putc)
+
+///////////////////////////////////////////////////////////////////////
// Ignore return values OK for these functions
-esym(534, printf)
@@ -32,10 +69,3 @@
-esym(534, strcat)
-esym(534, strcpy)
-esym(534, strlcpy)
-
--esym(534, vsb_cat)
--esym(534, vsb_bcat)
--esym(534, vsb_putc)
--esym(534, vsb_printf)
--esym(534, vsb_vprintf)
--esym(534, vsb_putc)
Modified: trunk/varnish-cache/bin/varnishd/flint.lnt
===================================================================
--- trunk/varnish-cache/bin/varnishd/flint.lnt 2010-06-05 13:21:13 UTC (rev 4903)
+++ trunk/varnish-cache/bin/varnishd/flint.lnt 2010-06-05 13:30:55 UTC (rev 4904)
@@ -12,9 +12,6 @@
-esym(759, vev_*)
-esym(765, vev_*)
-esym(714, vev_*)
--esym(759, vsb_*)
--esym(765, vsb_*)
--esym(714, vsb_*)
-esym(759, vpf_*)
-esym(765, vpf_*)
-esym(714, vpf_*)
@@ -33,11 +30,6 @@
-esym(759, CLI_*)
-esym(755, CLI_*)
--esym(755, VLIST_*)
--esym(755, VSLIST_*)
--esym(755, VSTAILQ_*)
--esym(755, VTAILQ_*)
-
// XXX: I think this is a flexelint bug:
-esym(522, vbit_clr)
@@ -126,8 +118,6 @@
// Fix strchr() semtics, it can only return NULL if arg2 != 0
-sem(strchr, 1p, type(1), 2n == 0 ? (@p < 1p) : (@p < 1p || @p == 0 ))
--sem(vsb_new, @p == (1p ? 1p : malloc(1)))
--sem(vsb_delete, custodial(1))
-sem(pthread_create, custodial(4))
-sem(WS_Init, custodial(2))
-sem(http_Setup, custodial(2))
@@ -141,13 +131,6 @@
-e785 // Too few initializers for aggregate
-e786 // String concatenation within initializer
--emacro(740, VTAILQ_PREV) // Unusual pointer cast (incompatible indirect types)
--emacro(740, VTAILQ_LAST) // Unusual pointer cast (incompatible indirect types)
--emacro((826), VTAILQ_PREV) // Suspicious pointer-to-pointer conversion (area too small)
--emacro((826), VTAILQ_LAST) // Suspicious pointer-to-pointer conversion (area too small)
--emacro(506, VTAILQ_FOREACH_SAFE) // constant value boolean
--emacro(506, VSTAILQ_FOREACH_SAFE) // constant value boolean
-
-esym(765, vcc_ProcAction) // could be made static
-esym(759, vcc_ProcAction) // could be moved to module
-esym(714, vcc_ProcAction) // not ref.
Modified: trunk/varnish-cache/bin/varnishncsa/flint.lnt
===================================================================
--- trunk/varnish-cache/bin/varnishncsa/flint.lnt 2010-06-05 13:21:13 UTC (rev 4903)
+++ trunk/varnish-cache/bin/varnishncsa/flint.lnt 2010-06-05 13:30:55 UTC (rev 4904)
@@ -11,11 +11,6 @@
-e785 // Too few initializers for aggregate
-e786 // String concatenation within initializer
--emacro(740, TAILQ_PREV) // Unusual pointer cast (incompatible indirect types)
--emacro((826), TAILQ_PREV) // Suspicious pointer-to-pointer conversion (area too small)
-
-
-
// cache.h
-emacro(506, INCOMPL) // Constant value Boolean
Modified: trunk/varnish-cache/bin/varnishtest/flint.lnt
===================================================================
--- trunk/varnish-cache/bin/varnishtest/flint.lnt 2010-06-05 13:21:13 UTC (rev 4903)
+++ trunk/varnish-cache/bin/varnishtest/flint.lnt 2010-06-05 13:30:55 UTC (rev 4904)
@@ -33,11 +33,6 @@
// -e785 // Too few initializers for aggregate
// -e786 // String concatenation within initializer
//
-// -emacro(740, VTAILQ_PREV) // Unusual pointer cast (incompatible indirect types)
-// -emacro(740, VTAILQ_LAST) // Unusual pointer cast (incompatible indirect types)
-// -emacro((826), VTAILQ_PREV) // Suspicious pointer-to-pointer conversion (area too small)
-// -emacro((826), VTAILQ_LAST) // Suspicious pointer-to-pointer conversion (area too small)
--emacro(506, VTAILQ_FOREACH_SAFE) // constant value boolean
-emacro(779, REPLACE) // string constant !=
-emacro(774, REPLACE) // if(bool) always true
-emacro(506, REPLACE) // const bool
More information about the varnish-commit
mailing list