Fix the privious assertion().
authorAndre Noll <maan@systemlinux.org>
Sat, 15 Sep 2007 07:50:14 +0000 (09:50 +0200)
committerAndre Noll <maan@systemlinux.org>
Sat, 15 Sep 2007 07:50:14 +0000 (09:50 +0200)
It's OK to ask for the name of an audio format id < 0, as id -1 is used for
"currently no audio file is open". So allow this case, and fix the upper bound
which was off by one while we're at it.

vss.c

diff --git a/vss.c b/vss.c
index 98ad11f..3ba5994 100644 (file)
--- a/vss.c
+++ b/vss.c
@@ -153,8 +153,9 @@ unsigned int vss_paused(void)
  */
 const char *audio_format_name(int i)
 {
-       assert(i >= 0 && i < ARRAY_SIZE(afl));
-       return afl[i].name;
+       //PARA_NOTICE_LOG("array size: %u¸ requested: %d\n", ARRAY_SIZE(afl), i);
+       assert(i < 0 || i < ARRAY_SIZE(afl) - 1);
+       return i >= 0?  afl[i].name : "(none)";
 }
 
 /**