Fix double promotion in format specifier %f by casting floats to double. (#608)

This commit is contained in:
Steve Karg
2024-03-26 15:23:49 -05:00
committed by GitHub
parent f6c6fe87c5
commit 4e0a37fd75
+6 -6
View File
@@ -497,7 +497,7 @@ int lighting_command_to_ascii(
priority = value->priority; priority = value->priority;
} }
len = snprintf(buf, buf_size, "%u,%f,%lu,%u", value->operation, len = snprintf(buf, buf_size, "%u,%f,%lu,%u", value->operation,
(float)target_level, (unsigned long)fade_time, (double)target_level, (unsigned long)fade_time,
(unsigned)priority); (unsigned)priority);
break; break;
case BACNET_LIGHTS_RAMP_TO: case BACNET_LIGHTS_RAMP_TO:
@@ -511,8 +511,8 @@ int lighting_command_to_ascii(
priority = value->priority; priority = value->priority;
} }
len = snprintf(buf, buf_size, "%u,%f,%f,%u", len = snprintf(buf, buf_size, "%u,%f,%f,%u",
(unsigned)value->operation, (float)target_level, (unsigned)value->operation, (double)target_level,
(float)ramp_rate, (unsigned)priority); (double)ramp_rate, (unsigned)priority);
break; break;
case BACNET_LIGHTS_STEP_UP: case BACNET_LIGHTS_STEP_UP:
case BACNET_LIGHTS_STEP_DOWN: case BACNET_LIGHTS_STEP_DOWN:
@@ -525,7 +525,7 @@ int lighting_command_to_ascii(
priority = value->priority; priority = value->priority;
} }
len = snprintf(buf, buf_size, "%u,%f,%u", len = snprintf(buf, buf_size, "%u,%f,%u",
(unsigned)value->operation, (float)step_increment, (unsigned)value->operation, (double)step_increment,
(unsigned)priority); (unsigned)priority);
break; break;
case BACNET_LIGHTS_WARN: case BACNET_LIGHTS_WARN:
@@ -945,8 +945,8 @@ bool xy_color_same(BACNET_XY_COLOR *value1, BACNET_XY_COLOR *value2)
int xy_color_to_ascii(const BACNET_XY_COLOR *value, char *buf, size_t buf_size) int xy_color_to_ascii(const BACNET_XY_COLOR *value, char *buf, size_t buf_size)
{ {
return snprintf( return snprintf(
buf, buf_size, "(%f,%f)", (float)value->x_coordinate, buf, buf_size, "(%f,%f)", (double)value->x_coordinate,
(float)value->x_coordinate); (double)value->x_coordinate);
} }
/** /**