aboutsummaryrefslogtreecommitdiffstats
path: root/src/replay_gain_info.h
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2010-02-17 07:07:00 +0100
committerMax Kellermann <max@duempel.org>2010-02-17 07:07:00 +0100
commitc05e6a1275621421eb0a7c3112b0401fa458841e (patch)
treef4fc40e299335245977a9ee4b82f269bbb96b8b8 /src/replay_gain_info.h
parentb21e4d9a589c02bffed9a1793d9fc2dc52044c12 (diff)
downloadmpd-c05e6a1275621421eb0a7c3112b0401fa458841e.tar.gz
mpd-c05e6a1275621421eb0a7c3112b0401fa458841e.tar.xz
mpd-c05e6a1275621421eb0a7c3112b0401fa458841e.zip
replay_gain_info: use INFINITY to mark undefined values
The previous patch not only moved code, it also changed the check. Negative gain values seem to be valid after all, there just was the "magic" value 0.0 which means "not available". This patch changes the "magic" value to "INFINITY", and uses the C99 function isinf() to check. It might have been a better idea to use "NAN", but the "NAN" macro is a GNU extension.
Diffstat (limited to '')
-rw-r--r--src/replay_gain_info.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/replay_gain_info.h b/src/replay_gain_info.h
index 9fcae22ab..2465a250e 100644
--- a/src/replay_gain_info.h
+++ b/src/replay_gain_info.h
@@ -23,6 +23,7 @@
#include "check.h"
#include <stdbool.h>
+#include <math.h>
enum replay_gain_mode {
REPLAY_GAIN_OFF = -1,
@@ -54,7 +55,7 @@ replay_gain_info_free(struct replay_gain_info *info);
static inline bool
replay_gain_tuple_defined(const struct replay_gain_tuple *tuple)
{
- return tuple->gain > 0.0;
+ return !isinf(tuple->gain);
}
float