CLEANUP: include: move sample_data out of sample-t.h
The struct sample_data is used by pattern, map and vars, and currently
requires to include sample-t which comes with many other dependencies.
Let's move sample_data into its own file to shorten the dependency tree.
This revealed a number of issues in adjacent files which were hidden by
the fact that sample-t.h brought everything that was missing.
diff --git a/include/haproxy/acl-t.h b/include/haproxy/acl-t.h
index 8418ca9..a96aa42 100644
--- a/include/haproxy/acl-t.h
+++ b/include/haproxy/acl-t.h
@@ -22,14 +22,9 @@
#ifndef _HAPROXY_ACL_T_H
#define _HAPROXY_ACL_T_H
-#include <import/ebmbtree.h>
-
-#include <haproxy/arg-t.h>
#include <haproxy/list-t.h>
#include <haproxy/pattern-t.h>
-#include <haproxy/server-t.h>
-#include <haproxy/api-t.h>
-
+#include <haproxy/sample-t.h>
/* ACL test result.
*
diff --git a/include/haproxy/pattern-t.h b/include/haproxy/pattern-t.h
index a977527..592de81 100644
--- a/include/haproxy/pattern-t.h
+++ b/include/haproxy/pattern-t.h
@@ -29,7 +29,7 @@
#include <haproxy/thread-t.h>
#include <haproxy/api-t.h>
-#include <haproxy/sample-t.h>
+#include <haproxy/sample_data-t.h>
/* Pattern matching function result.
@@ -206,7 +206,9 @@
struct pattern_expr *expr; /* The used expr. */
};
-/* This struct contain a list of pattern expr */
+
+/* This struct contains a list of pattern expr */
+struct sample;
struct pattern_head {
int (*parse)(const char *text, struct pattern *pattern, int flags, char **err);
int (*parse_smp)(const char *text, struct sample_data *data);
diff --git a/include/haproxy/pattern.h b/include/haproxy/pattern.h
index 6be2474..e1c246c 100644
--- a/include/haproxy/pattern.h
+++ b/include/haproxy/pattern.h
@@ -26,6 +26,7 @@
#include <haproxy/api.h>
#include <haproxy/pattern-t.h>
+#include <haproxy/sample-t.h>
/* pattern management function arrays */
extern char *pat_match_names[PAT_MATCH_NUM];
diff --git a/include/haproxy/sample-t.h b/include/haproxy/sample-t.h
index 9403d7b..0f38f18 100644
--- a/include/haproxy/sample-t.h
+++ b/include/haproxy/sample-t.h
@@ -23,13 +23,9 @@
#ifndef _HAPROXY_SAMPLE_T_H
#define _HAPROXY_SAMPLE_T_H
-#include <sys/socket.h>
-#include <netinet/in.h>
-
-#include <haproxy/api-t.h>
-#include <haproxy/buf-t.h>
-#include <haproxy/http-t.h>
#include <haproxy/list-t.h>
+#include <haproxy/api-t.h>
+#include <haproxy/sample_data-t.h>
/* input and output sample types */
enum {
@@ -225,26 +221,6 @@
void *a[8]; /* any array of up to 8 pointers */
};
-/* Note: the strings below make use of chunks. Chunks may carry an allocated
- * size in addition to the length. The size counts from the beginning (str)
- * to the end. If the size is unknown, it MUST be zero, in which case the
- * sample will automatically be duplicated when a change larger than <len> has
- * to be performed. Thus it is safe to always set size to zero.
- */
-union sample_value {
- long long int sint; /* used for signed 64bits integers */
- struct in_addr ipv4; /* used for ipv4 addresses */
- struct in6_addr ipv6; /* used for ipv6 addresses */
- struct buffer str; /* used for char strings or buffers */
- struct http_meth meth; /* used for http method */
-};
-
-/* Used to store sample constant */
-struct sample_data {
- int type; /* SMP_T_* */
- union sample_value u; /* sample data */
-};
-
/* a sample is a typed data extracted from a stream. It has a type, contents,
* validity constraints, a context for use in iterative calls.
*/
diff --git a/include/haproxy/sample_data-t.h b/include/haproxy/sample_data-t.h
new file mode 100644
index 0000000..2546028
--- /dev/null
+++ b/include/haproxy/sample_data-t.h
@@ -0,0 +1,51 @@
+/*
+ * include/haproxy/sample_data-t.h
+ * Definitions of sample data
+ *
+ * Copyright (C) 2009-2010 EXCELIANCE, Emeric Brun <ebrun@exceliance.fr>
+ * Copyright (C) 2020 Willy Tarreau <w@1wt.eu>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation, version 2.1
+ * exclusively.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#ifndef _HAPROXY_SAMPLE_DATA_T_H
+#define _HAPROXY_SAMPLE_DATA_T_H
+
+#include <sys/socket.h>
+#include <netinet/in.h>
+#include <haproxy/buf-t.h>
+#include <haproxy/http-t.h>
+
+/* Note: the strings below make use of chunks. Chunks may carry an allocated
+ * size in addition to the length. The size counts from the beginning (str)
+ * to the end. If the size is unknown, it MUST be zero, in which case the
+ * sample will automatically be duplicated when a change larger than <len> has
+ * to be performed. Thus it is safe to always set size to zero.
+ */
+union sample_value {
+ long long int sint; /* used for signed 64bits integers */
+ struct in_addr ipv4; /* used for ipv4 addresses */
+ struct in6_addr ipv6; /* used for ipv6 addresses */
+ struct buffer str; /* used for char strings or buffers */
+ struct http_meth meth; /* used for http method */
+};
+
+/* Used to store sample constant */
+struct sample_data {
+ int type; /* SMP_T_* */
+ union sample_value u; /* sample data */
+};
+
+#endif /* _HAPROXY_SAMPLE_DATA_T_H */
diff --git a/include/haproxy/vars-t.h b/include/haproxy/vars-t.h
index 0d8f337..388d208 100644
--- a/include/haproxy/vars-t.h
+++ b/include/haproxy/vars-t.h
@@ -23,7 +23,7 @@
#define _HAPROXY_VARS_T_H
#include <haproxy/list-t.h>
-#include <haproxy/sample-t.h>
+#include <haproxy/sample_data-t.h>
#include <haproxy/thread-t.h>
enum vars_scope {