From 6d2cd169b0a73915964076a9dc653011e07875d3 Mon Sep 17 00:00:00 2001 From: Jonathan Beck Date: Thu, 29 Jan 2009 18:54:18 +0100 Subject: Make pointer argument in API const when possible. --- include/plist/plist.h | 14 +++++++------- src/plist.c | 16 ++++++++-------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/include/plist/plist.h b/include/plist/plist.h index b666d2f..c5af845 100644 --- a/include/plist/plist.h +++ b/include/plist/plist.h @@ -50,17 +50,17 @@ typedef enum { plist_t plist_new_dict(); plist_t plist_new_array(); //Plist edition, create a new root if node is NULL -plist_t plist_add_sub_element( plist_t node, plist_type type, void* value, uint64_t length); +plist_t plist_add_sub_element( plist_t node, plist_type type, const void* value, uint64_t length); //Plist edition, only work for dict and array node void plist_add_sub_node(plist_t node, plist_t subnode); -void plist_add_sub_key_el(plist_t node, char* val); -void plist_add_sub_string_el(plist_t node, char* val); +void plist_add_sub_key_el(plist_t node, const char* val); +void plist_add_sub_string_el(plist_t node, const char* val); void plist_add_sub_bool_el(plist_t node, uint8_t val); void plist_add_sub_uint_el(plist_t node, uint64_t val); void plist_add_sub_real_el(plist_t node, double val); -void plist_add_sub_data_el(plist_t node, char* val, uint64_t length); +void plist_add_sub_data_el(plist_t node, const char* val, uint64_t length); //plist free @@ -72,9 +72,9 @@ plist_t plist_get_next_sibling(plist_t node); plist_t plist_get_prev_sibling(plist_t node); //utili function to find first (and only the first encountred) corresponding node -plist_t plist_find_node(plist_t plist, plist_type type, void *value, uint64_t length); -plist_t plist_find_node_by_key(plist_t plist, char *value); -plist_t plist_find_node_by_string(plist_t plist, char *value); +plist_t plist_find_node(plist_t plist, plist_type type, const void *value, uint64_t length); +plist_t plist_find_node_by_key(plist_t plist, const char *value); +plist_t plist_find_node_by_string(plist_t plist, const char *value); void plist_get_type_and_value(plist_t node, plist_type * type, void *value, uint64_t * length); diff --git a/src/plist.c b/src/plist.c index 74868cc..709f12d 100644 --- a/src/plist.c +++ b/src/plist.c @@ -71,7 +71,7 @@ plist_t plist_new_array() return plist_new_node(data); } -plist_t plist_add_sub_element(plist_t node, plist_type type, void *value, uint64_t length) +plist_t plist_add_sub_element(plist_t node, plist_type type, const void *value, uint64_t length) { //only structured types can have children plist_type node_type = plist_get_node_type(node); @@ -140,7 +140,7 @@ plist_t plist_get_prev_sibling(plist_t node) return (plist_t) g_node_prev_sibling((GNode *) node); } -static char compare_node_value(plist_type type, plist_data_t data, void *value, uint64_t length) +static char compare_node_value(plist_type type, plist_data_t data, const void *value, uint64_t length) { char res = FALSE; switch (type) { @@ -172,7 +172,7 @@ static char compare_node_value(plist_type type, plist_data_t data, void *value, return res; } -plist_t plist_find_node(plist_t plist, plist_type type, void *value, uint64_t length) +plist_t plist_find_node(plist_t plist, plist_type type, const void *value, uint64_t length) { if (!plist) return NULL; @@ -194,12 +194,12 @@ plist_t plist_find_node(plist_t plist, plist_type type, void *value, uint64_t le return NULL; } -plist_t plist_find_node_by_key(plist_t plist, char *value) +plist_t plist_find_node_by_key(plist_t plist, const char *value) { return plist_find_node(plist, PLIST_KEY, value, strlen(value)); } -plist_t plist_find_node_by_string(plist_t plist, char *value) +plist_t plist_find_node_by_string(plist_t plist, const char *value) { return plist_find_node(plist, PLIST_STRING, value, strlen(value)); } @@ -268,12 +268,12 @@ void plist_add_sub_node(plist_t node, plist_t subnode) } } -void plist_add_sub_key_el(plist_t node, char *val) +void plist_add_sub_key_el(plist_t node, const char *val) { plist_add_sub_element(node, PLIST_KEY, val, strlen(val)); } -void plist_add_sub_string_el(plist_t node, char *val) +void plist_add_sub_string_el(plist_t node, const char *val) { plist_add_sub_element(node, PLIST_STRING, val, strlen(val)); } @@ -293,7 +293,7 @@ void plist_add_sub_real_el(plist_t node, double val) plist_add_sub_element(node, PLIST_REAL, &val, sizeof(double)); } -void plist_add_sub_data_el(plist_t node, char *val, uint64_t length) +void plist_add_sub_data_el(plist_t node, const char *val, uint64_t length) { plist_add_sub_element(node, PLIST_DATA, val, length); } -- cgit v1.1-32-gdbae