diff options
author | Maarten Bosmans <mkbosmans@gmail.com> | 2011-03-19 13:59:16 +0100 |
---|---|---|
committer | Colin Guthrie <cguthrie@mandriva.org> | 2011-03-20 11:29:25 +0000 |
commit | 7c011e7688f1eddeee7636f1bbec11769315135e (patch) | |
tree | 72145bd4aa51bb5fbfd29c4a2ac2378864b5dbe5 /src/modules/alsa/alsa-mixer.c | |
parent | 33a88fbfdee773b1473cb5339540d79809363bdc (diff) |
Get rid of some warnings: -Wunsafe-loop-optimizations
pulsecore/core-util.c: In function ‘pa_hexstr’:
pulsecore/core-util.c:1858: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations]
modules/alsa/alsa-mixer.c: In function ‘pa_alsa_decibel_fix_dump’:
modules/alsa/alsa-mixer.c:3678: warning: cannot optimize possibly infinite loops [-Wunsafe-loop-optimizations]
modules/alsa/alsa-mixer.c: In function ‘pa_alsa_path_set_new’:
modules/alsa/alsa-mixer.c:2640: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations]
Diffstat (limited to 'src/modules/alsa/alsa-mixer.c')
-rw-r--r-- | src/modules/alsa/alsa-mixer.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/modules/alsa/alsa-mixer.c b/src/modules/alsa/alsa-mixer.c index 3eef5f9c..a0fa865f 100644 --- a/src/modules/alsa/alsa-mixer.c +++ b/src/modules/alsa/alsa-mixer.c @@ -2637,7 +2637,7 @@ pa_alsa_path_set *pa_alsa_path_set_new(pa_alsa_mapping *m, pa_alsa_direction_t d pa_bool_t duplicate = FALSE; char **kn, *fn; - for (kn = pn; kn != in; kn++) + for (kn = pn; kn < in; kn++) if (pa_streq(*kn, *in)) { duplicate = TRUE; break; @@ -3669,14 +3669,14 @@ void pa_alsa_decibel_fix_dump(pa_alsa_decibel_fix *db_fix) { if (db_fix->db_values) { pa_strbuf *buf; - long i; - long max_i = db_fix->max_step - db_fix->min_step; + unsigned long i, nsteps; - buf = pa_strbuf_new(); - pa_strbuf_printf(buf, "[%li]:%0.2f", db_fix->min_step, db_fix->db_values[0] / 100.0); + pa_assert(db_fix->min_step <= db_fix->max_step); + nsteps = db_fix->max_step - db_fix->min_step + 1; - for (i = 1; i <= max_i; ++i) - pa_strbuf_printf(buf, " [%li]:%0.2f", i + db_fix->min_step, db_fix->db_values[i] / 100.0); + buf = pa_strbuf_new(); + for (i = 0; i < nsteps; ++i) + pa_strbuf_printf(buf, "[%li]:%0.2f ", i + db_fix->min_step, db_fix->db_values[i] / 100.0); db_values = pa_strbuf_tostring_free(buf); } |