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

Expression packing: Use the type of the expression and not the accessor

This commit is contained in:
Shahram Najm 2024-01-25 17:31:44 +00:00
parent 6570759cec
commit 22becf5269
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/grib_accessor_class_gen.cc
View File

@ -404,7 +404,8 @@ static int pack_expression(grib_accessor* a, grib_expression* e)
int ret = 0;
grib_handle* hand = grib_handle_of_accessor(a);
switch (grib_accessor_get_native_type(a)) {
// Use the native type of the expression not the accessor
switch (grib_expression_native_type(hand, e)) {
case GRIB_TYPE_LONG: {
len = 1;
ret = grib_expression_evaluate_long(hand, e, &lval);
@ -542,7 +543,7 @@ static int pack_string(grib_accessor* a, const char* v, size_t* len)
double val = strtod(v, &endPtr);
if (*endPtr) {
grib_context_log(a->context, GRIB_LOG_ERROR,
"%s: Invalid value (%s) for %s. String cannot be converted to a double",
"%s: Invalid value (%s) for key '%s'. String cannot be converted to a double",
__func__, v, a->name);
return GRIB_WRONG_TYPE;
}