diff options
author | Tanu Kaskinen <tanuk@iki.fi> | 2011-03-27 21:35:03 +0300 |
---|---|---|
committer | Colin Guthrie <colin@mageia.org> | 2011-03-28 09:52:55 +0100 |
commit | 77da2c4bcf843980fe27d4878a468741bdad3a38 (patch) | |
tree | 773697292a73733510c325dd571bcd7a4f355509 /src/pulsecore/core-util.h | |
parent | 60f191941b628947398207a26a2f6a3c463db7bb (diff) |
alsa-mixer: Get rid of a compiler warning.
On 64-bit systems LONG_MAX is greater than the largest possible value of a
uint32_t variable, which caused the compiler to warn about a comparison that is
always false. On 32-bit systems pa_atou() can return a value that will overflow
when assigned to e->volume_limit, which has type long, so the comparison was
necessary.
This dilemma is resolved by using pa_atol() instead of pa_atou().
Diffstat (limited to 'src/pulsecore/core-util.h')
-rw-r--r-- | src/pulsecore/core-util.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/pulsecore/core-util.h b/src/pulsecore/core-util.h index 32641a3f..0b34a183 100644 --- a/src/pulsecore/core-util.h +++ b/src/pulsecore/core-util.h @@ -135,6 +135,7 @@ char *pa_state_path(const char *fn, pa_bool_t prepend_machine_id); int pa_atoi(const char *s, int32_t *ret_i); int pa_atou(const char *s, uint32_t *ret_u); +int pa_atol(const char *s, long *ret_l); int pa_atod(const char *s, double *ret_d); size_t pa_snprintf(char *str, size_t size, const char *format, ...); |