[binutils-gdb] libctf: fix the type of ctf_enum.cte_value

Jose E.Marchesi jemarch@sourceware.org
Tue Jun 4 16:12:00 GMT 2019


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a610aa4f9cf61d38b8e0fe60dfaac078d636089f

commit a610aa4f9cf61d38b8e0fe60dfaac078d636089f
Author: Nick Alcock <nick.alcock@oracle.com>
Date:   Mon Jun 3 11:38:08 2019 +0100

    libctf: fix the type of ctf_enum.cte_value
    
    This stops the file format from depending on the size of the host int.
    (It does mean that we cannot encode enums with a value > 2^32 on
    platforms with an int > 2^32: this will be fixed in the next format
    revision.)
    
    include/
    	* ctf.h (ctf_enum.cte_value): Fix type to int32_t.

Diff:
---
 include/ChangeLog | 4 ++++
 include/ctf.h     | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/include/ChangeLog b/include/ChangeLog
index 1f216b3..f84498b 100644
--- a/include/ChangeLog
+++ b/include/ChangeLog
@@ -1,3 +1,7 @@
+2019-06-03  Nick Alcock  <nick.alcock@oracle.com>
+
+	* ctf.h (ctf_enum.cte_value): Fix type to int32_t.
+
 2019-05-29  Nick Alcock  <nick.alcock@oracle.com>
 
 	* ctf-api.h (ctf_sect_t): Drop cts_type, cts_flags, and cts_offset.
diff --git a/include/ctf.h b/include/ctf.h
index 2c3384f..e99a673 100644
--- a/include/ctf.h
+++ b/include/ctf.h
@@ -507,7 +507,7 @@ typedef struct ctf_lmember_v2
 typedef struct ctf_enum
 {
   uint32_t cte_name;		/* Reference to name in string table.  */
-  int cte_value;		/* Value associated with this name.  */
+  int32_t cte_value;		/* Value associated with this name.  */
 } ctf_enum_t;
 
 /* The ctf_archive is a collection of ctf_file_t's stored together. The format



More information about the Binutils-cvs mailing list