core
/
eccodes
mirror of https://github.com/ecmwf/eccodes.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Dead code removal

This commit is contained in:
shahramn 2024-03-06 18:31:35 +00:00
parent fbd27b037e
commit 33a3326991
2 changed files with 63 additions and 73 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/eccodes_prototypes.h
View File

@ -108,7 +108,6 @@ grib_action* grib_action_create_transient_darray(grib_context* context, const ch
/* grib_accessor.cc*/ /* grib_accessor.cc*/
void grib_accessor_dump(grib_accessor* a, grib_dumper* f); void grib_accessor_dump(grib_accessor* a, grib_dumper* f);
int grib_pack_missing(grib_accessor* a); int grib_pack_missing(grib_accessor* a);
int grib_pack_zero(grib_accessor* a);
int grib_is_missing_internal(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_double(grib_accessor* a, const double* v, size_t* len);
int grib_pack_float(grib_accessor* a, const float* 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); 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_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_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); grib_accessor* grib_accessor_get_attribute_index(grib_accessor* a, const char* name, int* index);
int grib_accessor_has_attributes(grib_accessor* a); int grib_accessor_has_attributes(grib_accessor* a);
grib_accessor* grib_accessor_get_attribute(grib_accessor* a, const char* name); grib_accessor* grib_accessor_get_attribute(grib_accessor* a, const char* name);
grib_accessors_list* grib_accessors_list_create(grib_context* c); grib_accessors_list* grib_accessors_list_create(grib_context* c);
void grib_accessors_list_push(grib_accessors_list* al, grib_accessor* a, int rank); 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_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); void grib_accessors_list_delete(grib_context* c, grib_accessors_list* al);
/* grib_concept.cc*/ /* grib_concept.cc*/

130
src/grib_accessor.cc
View File

@ -54,18 +54,18 @@ int grib_pack_missing(grib_accessor* a)
return 0; return 0;
} }
int grib_pack_zero(grib_accessor* a) // int grib_pack_zero(grib_accessor* a)
{ // {
grib_accessor_class* c = a->cclass; // grib_accessor_class* c = a->cclass;
while (c) { // while (c) {
if (c->clear) { // if (c->clear) {
return c->clear(a); // return c->clear(a);
} // }
c = c->super ? *(c->super) : NULL; // c = c->super ? *(c->super) : NULL;
} // }
DEBUG_ASSERT(0); // DEBUG_ASSERT(0);
return 0; // return 0;
} // }
int grib_is_missing_internal(grib_accessor* a) 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; c = c->super ? *(c->super) : NULL;
} }
if (a && a->cclass) 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); DEBUG_ASSERT(0);
return 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; return GRIB_TOO_MANY_ATTRIBUTES;
} }
int grib_accessor_replace_attribute(grib_accessor* a, grib_accessor* attr) // int grib_accessor_replace_attribute(grib_accessor* a, grib_accessor* attr)
{ // {
int id = 0; // int id = 0;
int idx = 0; // int idx = 0;
if (grib_accessor_get_attribute_index(a, attr->name, &id) != NULL) { // if (grib_accessor_get_attribute_index(a, attr->name, &id) != NULL) {
grib_accessor_delete(a->context, a->attributes[id]); // grib_accessor_delete(a->context, a->attributes[id]);
a->attributes[id] = attr; // a->attributes[id] = attr;
attr->parent_as_attribute = a; // attr->parent_as_attribute = a;
if (a->same) // if (a->same)
attr->same = grib_accessor_get_attribute_index(a->same, attr->name, &idx); // attr->same = grib_accessor_get_attribute_index(a->same, attr->name, &idx);
} // }
else { // else {
grib_accessor_add_attribute(a, attr, 0); // grib_accessor_add_attribute(a, attr, 0);
} // }
return GRIB_SUCCESS; // return GRIB_SUCCESS;
} // }
int grib_accessor_delete_attribute(grib_accessor* a, const char* name) // int grib_accessor_delete_attribute(grib_accessor* a, const char* name)
{ // {
int id = 0; // int id = 0;
if (grib_accessor_get_attribute_index(a, name, &id) != NULL) { // if (grib_accessor_get_attribute_index(a, name, &id) != NULL) {
grib_accessor_delete(a->context, a->attributes[id]); // grib_accessor_delete(a->context, a->attributes[id]);
a->attributes[id] = NULL; // a->attributes[id] = NULL;
return GRIB_SUCCESS; // return GRIB_SUCCESS;
} // }
else { // else {
return GRIB_NOT_FOUND; // return GRIB_NOT_FOUND;
} // }
} // }
grib_accessor* grib_accessor_get_attribute_by_index(grib_accessor* a, int index) // grib_accessor* grib_accessor_get_attribute_by_index(grib_accessor* a, int index)
{ // {
if (index < MAX_ACCESSOR_ATTRIBUTES) // if (index < MAX_ACCESSOR_ATTRIBUTES)
return a->attributes[index]; // 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) 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; // last=next;
// next=last->next; // next=last->next;
// } // }
return al->last; return al->last;
} }
grib_accessors_list* grib_accessors_list_find(grib_accessors_list* al, const grib_accessor* a) // grib_accessors_list* grib_accessors_list_find(grib_accessors_list* al, const grib_accessor* a)
{ // {
grib_accessors_list* last = al; // grib_accessors_list* last = al;
grib_accessors_list* next = al->next; // grib_accessors_list* next = al->next;
// while (next) {
while (next) { // if (last->accessor == a)
if (last->accessor == a) // return last;
return last; // last = next;
last = next; // next = last->next;
next = last->next; // }
} // return NULL;
// }
return NULL;
}
void grib_accessors_list_delete(grib_context* c, grib_accessors_list* al) void grib_accessors_list_delete(grib_context* c, grib_accessors_list* al)
{ {