[vlc-devel] [PATCH 1/2] [demux/ogg] Fix off-by-one default framerate table indexing

David Flynn davidf+nntp at woaf.net
Tue Jul 29 19:13:24 CEST 2008


From: David Flynn <davidf at rd.bbc.co.uk>

Values obtained from the specification/stream are one-indexed.  Add
fake zero index to compensate.

Signed-off-by: David Flynn <davidf at rd.bbc.co.uk>
---
 modules/demux/ogg.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/modules/demux/ogg.c b/modules/demux/ogg.c
index e3440d5..9b86a25 100644
--- a/modules/demux/ogg.c
+++ b/modules/demux/ogg.c
@@ -1628,6 +1628,7 @@ static void Ogg_ReadDiracHeader( logical_stream_t *p_stream,
     static const struct {
         uint32_t u_n /* numerator */, u_d /* denominator */;
     } dirac_frate_tbl[] = { /* table 10.3 */
+        {1,1}, /* this first value is never used */
         {24000,1001}, {24,1}, {25,1}, {30000,1001}, {30,1},
         {50,1}, {60000,1001}, {60,1}, {15000,1001}, {25,2},
     };
-- 
1.5.5.4





More information about the vlc-devel mailing list