On Wed, Jul 13, 2022 at 9:50 AM Christian Göttsche
<cgzones@xxxxxxxxxxxxxx> wrote:
>
> Since `struct type_set` is forward declared including
> policydb/policydb.h is not necessary and creates a circular include.
>
> Also drop the unnecessary forward declaration of `struct policydb`.
>
> Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
For these three patches:
Acked-by: James Carter <jwcart2@xxxxxxxxx>
> ---
> libsepol/include/sepol/policydb/constraint.h | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/libsepol/include/sepol/policydb/constraint.h b/libsepol/include/sepol/policydb/constraint.h
> index b91fc4e9..82335e21 100644
> --- a/libsepol/include/sepol/policydb/constraint.h
> +++ b/libsepol/include/sepol/policydb/constraint.h
> @@ -18,7 +18,6 @@
> #ifndef _SEPOL_POLICYDB_CONSTRAINT_H_
> #define _SEPOL_POLICYDB_CONSTRAINT_H_
>
> -#include <sepol/policydb/policydb.h>
> #include <sepol/policydb/ebitmap.h>
> #include <sepol/policydb/flask_types.h>
>
> @@ -70,8 +69,6 @@ typedef struct constraint_node {
> struct constraint_node *next; /* next constraint */
> } constraint_node_t;
>
> -struct policydb;
> -
> extern int constraint_expr_init(constraint_expr_t * expr);
> extern void constraint_expr_destroy(constraint_expr_t * expr);
>
> --
> 2.36.1
>
