Fix : coherency in const parameters
authorJulien Desfossez <julien.desfossez@efficios.com>
Tue, 21 Feb 2012 15:45:14 +0000 (10:45 -0500)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tue, 21 Feb 2012 15:45:14 +0000 (10:45 -0500)
This patchs adds the const attribute to the field access functions
(get_int64 and others) in order to keep the coherency with the get_field
and get_field_list function which return a const pointer.

fixes #84

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
formats/ctf/events.c
include/babeltrace/ctf/events.h
include/babeltrace/types.h
types/array.c
types/integer.c
types/string.c

index 8ecb7c487f74384847b6f02aced03035c7ce3ee1..22623aa52455a425427f61360508c34daaf64ebd 100644 (file)
@@ -224,7 +224,7 @@ const char *bt_ctf_field_name(const struct definition *def)
        return NULL;
 }
 
-enum ctf_type_id bt_ctf_field_type(struct definition *def)
+enum ctf_type_id bt_ctf_field_type(const struct definition *def)
 {
        if (def)
                return def->declaration->id;
@@ -339,7 +339,7 @@ int bt_ctf_field_get_error(void)
        return ret;
 }
 
-uint64_t bt_ctf_get_uint64(struct definition *field)
+uint64_t bt_ctf_get_uint64(const struct definition *field)
 {
        unsigned int ret = 0;
 
@@ -351,7 +351,7 @@ uint64_t bt_ctf_get_uint64(struct definition *field)
        return ret;
 }
 
-int64_t bt_ctf_get_int64(struct definition *field)
+int64_t bt_ctf_get_int64(const struct definition *field)
 {
        int ret = 0;
 
@@ -364,7 +364,7 @@ int64_t bt_ctf_get_int64(struct definition *field)
 
 }
 
-char *bt_ctf_get_char_array(struct definition *field)
+char *bt_ctf_get_char_array(const struct definition *field)
 {
        char *ret = NULL;
 
@@ -376,7 +376,7 @@ char *bt_ctf_get_char_array(struct definition *field)
        return ret;
 }
 
-char *bt_ctf_get_string(struct definition *field)
+char *bt_ctf_get_string(const struct definition *field)
 {
        char *ret = NULL;
 
index 61e3f6a31a948d060088b66a165c3db2526dedbd..c08314f8775bbb6d0418b9f7011ccc0c30fb04f1 100644 (file)
@@ -160,7 +160,7 @@ const char *bt_ctf_field_name(const struct definition *def);
 /*
  * bt_ctf_field_type: returns the type of a field or -1 if unknown
  */
-enum ctf_type_id bt_ctf_field_type(struct definition *def);
+enum ctf_type_id bt_ctf_field_type(const struct definition *def);
 
 /*
  * Field access functions
@@ -172,10 +172,10 @@ enum ctf_type_id bt_ctf_field_type(struct definition *def);
  * returned is undefined. To check if an error occured, use the
  * bt_ctf_field_error() function after accessing a field.
  */
-uint64_t bt_ctf_get_uint64(struct definition *field);
-int64_t bt_ctf_get_int64(struct definition *field);
-char *bt_ctf_get_char_array(struct definition *field);
-char *bt_ctf_get_string(struct definition *field);
+uint64_t bt_ctf_get_uint64(const struct definition *field);
+int64_t bt_ctf_get_int64(const struct definition *field);
+char *bt_ctf_get_char_array(const struct definition *field);
+char *bt_ctf_get_string(const struct definition *field);
 
 /*
  * bt_ctf_field_error: returns the last error code encountered while
index 0cedb337b03d389dfdda9199c8a2477986c33c46..ad2b42d0a92d7d020ff8ab17f36a88b605122ff7 100644 (file)
@@ -373,8 +373,8 @@ struct declaration_integer *integer_declaration_new(size_t len, int byte_order,
                                  int signedness, size_t alignment,
                                  int base, enum ctf_string_encoding encoding,
                                  struct ctf_clock *clock);
-uint64_t get_unsigned_int(struct definition *field);
-int64_t get_signed_int(struct definition *field);
+uint64_t get_unsigned_int(const struct definition *field);
+int64_t get_signed_int(const struct definition *field);
 
 /*
  * mantissa_len is the length of the number of bytes represented by the mantissa
@@ -421,7 +421,7 @@ struct declaration_enum *
 
 struct declaration_string *
        string_declaration_new(enum ctf_string_encoding encoding);
-char *get_string(struct definition *field);
+char *get_string(const struct definition *field);
 
 struct declaration_struct *
        struct_declaration_new(struct declaration_scope *parent_scope,
@@ -486,7 +486,7 @@ struct declaration_array *
 uint64_t array_len(struct definition_array *array);
 struct definition *array_index(struct definition_array *array, uint64_t i);
 int array_rw(struct stream_pos *pos, struct definition *definition);
-GString *get_char_array(struct definition *field);
+GString *get_char_array(const struct definition *field);
 
 /*
  * int_declaration and elem_declaration passed as parameter now belong
index f2c10adc683171fb63dc73f460b311a505c1429e..dbdcb4be8efd2161e8df6bcc42a3db2711db489c 100644 (file)
@@ -207,7 +207,7 @@ struct definition *array_index(struct definition_array *array, uint64_t i)
        return g_ptr_array_index(array->elems, i);
 }
 
-GString *get_char_array(struct definition *field)
+GString *get_char_array(const struct definition *field)
 {
        struct definition_array *array_definition;
        struct declaration_array *array_declaration;
index 59a6c1bb792c5a4aa1474cc4f0764c6e533320fb..6e1243059b51ce533d598d1b8ed5652598c22571 100644 (file)
@@ -107,7 +107,7 @@ void _integer_definition_free(struct definition *definition)
        g_free(integer);
 }
 
-uint64_t get_unsigned_int(struct definition *field)
+uint64_t get_unsigned_int(const struct definition *field)
 {
        struct definition_integer *integer_definition;
        const struct declaration_integer *integer_declaration;
@@ -122,7 +122,7 @@ uint64_t get_unsigned_int(struct definition *field)
        return (uint64_t)integer_definition->value._signed;
 }
 
-int64_t get_signed_int(struct definition *field)
+int64_t get_signed_int(const struct definition *field)
 {
        struct definition_integer *integer_definition;
        const struct declaration_integer *integer_declaration;
index 0fdd81e33266b52b9b80a91c3dba72f41223edb0..dadcd080e71fc69cd7ae5b704b13d45014121940 100644 (file)
@@ -101,7 +101,7 @@ void _string_definition_free(struct definition *definition)
        g_free(string);
 }
 
-char *get_string(struct definition *field)
+char *get_string(const struct definition *field)
 {
        struct definition_string *string_definition =
                container_of(field, struct definition_string, p);
This page took 0.040481 seconds and 4 git commands to generate.