diff mbox series

elf.h: Remove duplicate definition of VER_FLG_WEAK

Message ID 87r10vr794.fsf@oldenburg.str.redhat.com
State New
Headers show
Series elf.h: Remove duplicate definition of VER_FLG_WEAK | expand

Commit Message

Florian Weimer Sept. 1, 2022, 5:30 a.m. UTC
This did not cause a warning before because the token sequence for
the two definitions was identical.

---
 elf/elf.h | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)


base-commit: 4d3b77f020c2accf72af7a54f207d225c61db9bf

Comments

Adhemerval Zanella Sept. 1, 2022, 1:09 p.m. UTC | #1
On 01/09/22 02:30, Florian Weimer via Libc-alpha wrote:
> This did not cause a warning before because the token sequence for
> the two definitions was identical.

LGTM, thanks.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

> 
> ---
>  elf/elf.h | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/elf/elf.h b/elf/elf.h
> index 065deca83c..273305a7cd 100644
> --- a/elf/elf.h
> +++ b/elf/elf.h
> @@ -1064,7 +1064,8 @@ typedef struct
>  
>  /* Legal values for vd_flags (version information flags).  */
>  #define VER_FLG_BASE	0x1		/* Version definition of file itself */
> -#define VER_FLG_WEAK	0x2		/* Weak version identifier */
> +#define VER_FLG_WEAK	0x2		/* Weak version identifier.  Also
> +					   used by vna_flags below.  */
>  
>  /* Versym symbol index values.  */
>  #define	VER_NDX_LOCAL		0	/* Symbol is local.  */
> @@ -1142,10 +1143,6 @@ typedef struct
>  } Elf64_Vernaux;
>  
>  
> -/* Legal values for vna_flags.  */
> -#define VER_FLG_WEAK	0x2		/* Weak version identifier */
> -
> -
>  /* Auxiliary vector.  */
>  
>  /* This vector is normally only used by the program interpreter.  The
> 
> base-commit: 4d3b77f020c2accf72af7a54f207d225c61db9bf
>
diff mbox series

Patch

diff --git a/elf/elf.h b/elf/elf.h
index 065deca83c..273305a7cd 100644
--- a/elf/elf.h
+++ b/elf/elf.h
@@ -1064,7 +1064,8 @@  typedef struct
 
 /* Legal values for vd_flags (version information flags).  */
 #define VER_FLG_BASE	0x1		/* Version definition of file itself */
-#define VER_FLG_WEAK	0x2		/* Weak version identifier */
+#define VER_FLG_WEAK	0x2		/* Weak version identifier.  Also
+					   used by vna_flags below.  */
 
 /* Versym symbol index values.  */
 #define	VER_NDX_LOCAL		0	/* Symbol is local.  */
@@ -1142,10 +1143,6 @@  typedef struct
 } Elf64_Vernaux;
 
 
-/* Legal values for vna_flags.  */
-#define VER_FLG_WEAK	0x2		/* Weak version identifier */
-
-
 /* Auxiliary vector.  */
 
 /* This vector is normally only used by the program interpreter.  The