diff mbox series

[userspace,3/4] checkpolicy: remove unused te_assertions

Message ID 20200122154655.257233-4-omosnace@redhat.com (mailing list archive)
State Superseded
Headers show
Series Fix build with -fno-common | expand

Commit Message

Ondrej Mosnacek Jan. 22, 2020, 3:46 p.m. UTC 
This variable is declared in a header file, but never defined or used.
Remove it.

Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
---
 checkpolicy/checkpolicy.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Ondrej Mosnacek Jan. 22, 2020, 4:10 p.m. UTC | #1 
On Wed, Jan 22, 2020 at 4:46 PM Ondrej Mosnacek <omosnace@redhat.com> wrote:
> This variable is declared in a header file, but never defined or used.
> Remove it.
>
> Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
> ---
>  checkpolicy/checkpolicy.h | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/checkpolicy/checkpolicy.h b/checkpolicy/checkpolicy.h
> index 3868f1fa..8fbc1b7d 100644
> --- a/checkpolicy/checkpolicy.h
> +++ b/checkpolicy/checkpolicy.h
> @@ -13,8 +13,6 @@ typedef struct te_assert {
>         struct te_assert *next;
>  } te_assert_t;

Actually, it looks like this whole struct definition can go as well.
I'll post a v2 later.

>
> -te_assert_t *te_assertions;
> -
>  extern unsigned int policyvers;
>
>  #endif
> --
> 2.24.1
>
diff mbox series

Patch

diff --git a/checkpolicy/checkpolicy.h b/checkpolicy/checkpolicy.h
index 3868f1fa..8fbc1b7d 100644
--- a/checkpolicy/checkpolicy.h
+++ b/checkpolicy/checkpolicy.h
@@ -13,8 +13,6 @@  typedef struct te_assert {
 	struct te_assert *next;
 } te_assert_t;
 
-te_assert_t *te_assertions;
-
 extern unsigned int policyvers;
 
 #endif