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