From 33a3326991983b297026dcefc6d2226ed5efb415 Mon Sep 17 00:00:00 2001 From: shahramn Date: Wed, 6 Mar 2024 18:31:35 +0000 Subject: [PATCH] Dead code removal --- src/eccodes_prototypes.h | 6 -- src/grib_accessor.cc | 130 +++++++++++++++++++-------------------- 2 files changed, 63 insertions(+), 73 deletions(-) diff --git a/src/eccodes_prototypes.h b/src/eccodes_prototypes.h index 2ac2f9e13..abd512a9f 100644 --- a/src/eccodes_prototypes.h +++ b/src/eccodes_prototypes.h @@ -108,7 +108,6 @@ grib_action* grib_action_create_transient_darray(grib_context* context, const ch /* grib_accessor.cc*/ void grib_accessor_dump(grib_accessor* a, grib_dumper* f); int grib_pack_missing(grib_accessor* a); -int grib_pack_zero(grib_accessor* a); int grib_is_missing_internal(grib_accessor* a); int grib_pack_double(grib_accessor* a, const double* v, size_t* len); int grib_pack_float(grib_accessor* a, const float* v, size_t* len); @@ -150,17 +149,12 @@ grib_accessor* grib_next_accessor(grib_accessor* a); void grib_resize(grib_accessor* a, size_t new_size); int grib_compare_accessors(grib_accessor* a1, grib_accessor* a2, int compare_flags); int grib_accessor_add_attribute(grib_accessor* a, grib_accessor* attr, int nest_if_clash); -int grib_accessor_replace_attribute(grib_accessor* a, grib_accessor* attr); -int grib_accessor_delete_attribute(grib_accessor* a, const char* name); -grib_accessor* grib_accessor_get_attribute_by_index(grib_accessor* a, int index); -const char* grib_accessor_get_name(grib_accessor* a); grib_accessor* grib_accessor_get_attribute_index(grib_accessor* a, const char* name, int* index); int grib_accessor_has_attributes(grib_accessor* a); grib_accessor* grib_accessor_get_attribute(grib_accessor* a, const char* name); grib_accessors_list* grib_accessors_list_create(grib_context* c); void grib_accessors_list_push(grib_accessors_list* al, grib_accessor* a, int rank); grib_accessors_list* grib_accessors_list_last(grib_accessors_list* al); -grib_accessors_list* grib_accessors_list_find(grib_accessors_list* al, const grib_accessor* a); void grib_accessors_list_delete(grib_context* c, grib_accessors_list* al); /* grib_concept.cc*/ diff --git a/src/grib_accessor.cc b/src/grib_accessor.cc index 0b4b7ed99..a08f5348e 100644 --- a/src/grib_accessor.cc +++ b/src/grib_accessor.cc @@ -54,18 +54,18 @@ int grib_pack_missing(grib_accessor* a) return 0; } -int grib_pack_zero(grib_accessor* a) -{ - grib_accessor_class* c = a->cclass; - while (c) { - if (c->clear) { - return c->clear(a); - } - c = c->super ? *(c->super) : NULL; - } - DEBUG_ASSERT(0); - return 0; -} +// int grib_pack_zero(grib_accessor* a) +// { +// grib_accessor_class* c = a->cclass; +// while (c) { +// if (c->clear) { +// return c->clear(a); +// } +// c = c->super ? *(c->super) : NULL; +// } +// DEBUG_ASSERT(0); +// return 0; +// } int grib_is_missing_internal(grib_accessor* a) { @@ -526,7 +526,7 @@ int grib_accessor_notify_change(grib_accessor* a, grib_accessor* changed) c = c->super ? *(c->super) : NULL; } if (a && a->cclass) - printf("notify_change not implemented for %s %s\n", a->cclass->name, a->name); + fprintf(stderr, "Error: notify_change not implemented for %s %s\n", a->cclass->name, a->name); DEBUG_ASSERT(0); return 0; } @@ -726,48 +726,47 @@ int grib_accessor_add_attribute(grib_accessor* a, grib_accessor* attr, int nest_ return GRIB_TOO_MANY_ATTRIBUTES; } -int grib_accessor_replace_attribute(grib_accessor* a, grib_accessor* attr) -{ - int id = 0; - int idx = 0; - if (grib_accessor_get_attribute_index(a, attr->name, &id) != NULL) { - grib_accessor_delete(a->context, a->attributes[id]); - a->attributes[id] = attr; - attr->parent_as_attribute = a; - if (a->same) - attr->same = grib_accessor_get_attribute_index(a->same, attr->name, &idx); - } - else { - grib_accessor_add_attribute(a, attr, 0); - } - return GRIB_SUCCESS; -} +// int grib_accessor_replace_attribute(grib_accessor* a, grib_accessor* attr) +// { +// int id = 0; +// int idx = 0; +// if (grib_accessor_get_attribute_index(a, attr->name, &id) != NULL) { +// grib_accessor_delete(a->context, a->attributes[id]); +// a->attributes[id] = attr; +// attr->parent_as_attribute = a; +// if (a->same) +// attr->same = grib_accessor_get_attribute_index(a->same, attr->name, &idx); +// } +// else { +// grib_accessor_add_attribute(a, attr, 0); +// } +// return GRIB_SUCCESS; +// } -int grib_accessor_delete_attribute(grib_accessor* a, const char* name) -{ - int id = 0; - if (grib_accessor_get_attribute_index(a, name, &id) != NULL) { - grib_accessor_delete(a->context, a->attributes[id]); - a->attributes[id] = NULL; - return GRIB_SUCCESS; - } - else { - return GRIB_NOT_FOUND; - } -} +// int grib_accessor_delete_attribute(grib_accessor* a, const char* name) +// { +// int id = 0; +// if (grib_accessor_get_attribute_index(a, name, &id) != NULL) { +// grib_accessor_delete(a->context, a->attributes[id]); +// a->attributes[id] = NULL; +// return GRIB_SUCCESS; +// } +// else { +// return GRIB_NOT_FOUND; +// } +// } -grib_accessor* grib_accessor_get_attribute_by_index(grib_accessor* a, int index) -{ - if (index < MAX_ACCESSOR_ATTRIBUTES) - return a->attributes[index]; +// grib_accessor* grib_accessor_get_attribute_by_index(grib_accessor* a, int index) +// { +// if (index < MAX_ACCESSOR_ATTRIBUTES) +// return a->attributes[index]; +// return NULL; +// } - return NULL; -} - -const char* grib_accessor_get_name(grib_accessor* a) -{ - return a->name; -} +// const char* grib_accessor_get_name(grib_accessor* a) +// { +// return a->name; +// } grib_accessor* grib_accessor_get_attribute_index(grib_accessor* a, const char* name, int* index) { @@ -847,24 +846,21 @@ grib_accessors_list* grib_accessors_list_last(grib_accessors_list* al) // last=next; // next=last->next; // } - return al->last; } -grib_accessors_list* grib_accessors_list_find(grib_accessors_list* al, const grib_accessor* a) -{ - grib_accessors_list* last = al; - grib_accessors_list* next = al->next; - - while (next) { - if (last->accessor == a) - return last; - last = next; - next = last->next; - } - - return NULL; -} +// grib_accessors_list* grib_accessors_list_find(grib_accessors_list* al, const grib_accessor* a) +// { +// grib_accessors_list* last = al; +// grib_accessors_list* next = al->next; +// while (next) { +// if (last->accessor == a) +// return last; +// last = next; +// next = last->next; +// } +// return NULL; +// } void grib_accessors_list_delete(grib_context* c, grib_accessors_list* al) {