]> git.sesse.net Git - vlc/blobdiff - modules/codec/a52.c
* modules/demux/flac.c: few cosmetic changes.
[vlc] / modules / codec / a52.c
index 24fb533975688e2651a6074082b0a18f1bacb448..e2a8e68eb80463910d7342f52213892b36dfc73e 100644 (file)
@@ -2,7 +2,7 @@
  * a52.c: A/52 basic parser
  *****************************************************************************
  * Copyright (C) 2001-2002 VideoLAN
- * $Id: a52.c,v 1.18 2002/11/14 22:38:47 massiot Exp $
+ * $Id: a52.c,v 1.22 2003/03/31 22:39:27 massiot Exp $
  *
  * Authors: Stéphane Borel <stef@via.ecp.fr>
  *          Christophe Massiot <massiot@via.ecp.fr>
@@ -13,7 +13,7 @@
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
  * (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ -41,7 +41,7 @@
 #endif
 
 /*****************************************************************************
- * dec_thread_t : A52 pass-through thread descriptor
+ * dec_thread_t : decoder thread descriptor
  *****************************************************************************/
 typedef struct dec_thread_t
 {
@@ -86,14 +86,14 @@ vlc_module_end();
 /*****************************************************************************
  * OpenDecoder: probe the decoder and return score
  *****************************************************************************/
-static int OpenDecoder( vlc_object_t *p_this ) 
-{   
+static int OpenDecoder( vlc_object_t *p_this )
+{
     decoder_fifo_t *p_fifo = (decoder_fifo_t*) p_this;
 
     if( p_fifo->i_fourcc != VLC_FOURCC('a','5','2',' ')
          && p_fifo->i_fourcc != VLC_FOURCC('a','5','2','b') )
-    {   
-        return VLC_EGENERIC; 
+    {
+        return VLC_EGENERIC;
     }
 
     p_fifo->pf_run = RunDecoder;
@@ -109,7 +109,7 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
 {
     dec_thread_t * p_dec;
     audio_date_t end_date;
-    
+
     /* Allocate the memory needed to store the thread's structure */
     p_dec = malloc( sizeof(dec_thread_t) );
     if( p_dec == NULL )
@@ -137,21 +137,23 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
         return -1;
     }
 
-    /* decoder thread's main loop */
+    /* Decoder thread's main loop */
     while ( !p_dec->p_fifo->b_die && !p_dec->p_fifo->b_error )
     {
-        int i_frame_size, i_original_channels, i_rate, i_bit_rate;
+        int i_bit_rate;
+        unsigned int i_rate, i_original_channels, i_frame_size;
         mtime_t pts;
         byte_t p_header[7];
         aout_buffer_t * p_buffer;
 
         /* Look for sync word - should be 0x0b77 */
         RealignBits( &p_dec->bit_stream );
-        while ( (ShowBits( &p_dec->bit_stream, 16 ) ) != 0x0b77 && 
-                (!p_dec->p_fifo->b_die) && (!p_dec->p_fifo->b_error))
+        while ( (ShowBits( &p_dec->bit_stream, 16 ) ) != 0x0b77 &&
+                (!p_dec->p_fifo->b_die) && (!p_dec->p_fifo->b_error) )
         {
             RemoveBits( &p_dec->bit_stream, 8 );
         }
+        if ( p_dec->p_fifo->b_die || p_dec->p_fifo->b_error ) break;
 
         /* Set the Presentation Time Stamp */
         NextPTS( &p_dec->bit_stream, &pts, NULL );
@@ -162,7 +164,7 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
 
         /* Get A/52 frame header */
         GetChunk( &p_dec->bit_stream, p_header, 7 );
-        if( p_dec->p_fifo->b_die ) break;
+        if ( p_dec->p_fifo->b_die || p_dec->p_fifo->b_error ) break;
 
         /* Check if frame is valid and get frame info */
         i_frame_size = SyncInfo( p_header, &i_original_channels, &i_rate,
@@ -195,6 +197,7 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
             p_dec->output_format.i_bytes_per_frame = i_frame_size;
             p_dec->output_format.i_frame_length = A52_FRAME_NB;
             aout_DateInit( &end_date, i_rate );
+            aout_DateSet( &end_date, pts );
             p_dec->p_aout_input = aout_DecNew( p_dec->p_fifo,
                                                &p_dec->p_aout,
                                                &p_dec->output_format );
@@ -217,7 +220,11 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
 
         p_buffer = aout_DecNewBuffer( p_dec->p_aout, p_dec->p_aout_input,
                                       A52_FRAME_NB );
-        if ( p_buffer == NULL ) return -1;
+        if ( p_buffer == NULL )
+        {
+            p_dec->p_fifo->b_error = 1;
+            break;
+        }
         p_buffer->start_date = aout_DateGet( &end_date );
         p_buffer->end_date = aout_DateIncrement( &end_date,
                                                  A52_FRAME_NB );
@@ -233,24 +240,22 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
             break;
         }
 
-        /* Send the buffer to the mixer. */
+        /* Send the buffer to the aout core. */
         aout_DecPlay( p_dec->p_aout, p_dec->p_aout_input, p_buffer );
     }
 
-    /* If b_error is set, the spdif thread enters the error loop */
     if( p_dec->p_fifo->b_error )
     {
         DecoderError( p_dec->p_fifo );
     }
 
-    /* End of the spdif decoder thread */
     EndThread( p_dec );
-    
+
     return 0;
 }
 
 /*****************************************************************************
- * EndThread : spdif thread destruction
+ * EndThread : thread destruction
  *****************************************************************************/
 static void EndThread( dec_thread_t * p_dec )
 {
@@ -264,20 +269,21 @@ static void EndThread( dec_thread_t * p_dec )
 }
 
 /*****************************************************************************
- * SyncInfo: parse A52 sync info
+ * SyncInfo: parse A/52 sync info
  *****************************************************************************
  * This code is borrowed from liba52 by Aaron Holtzman & Michel Lespinasse,
  * since we don't want to oblige S/PDIF people to use liba52 just to get
  * their SyncInfo...
  *****************************************************************************/
-int SyncInfo( const byte_t * p_buf, int * pi_channels, int * pi_sample_rate,
-              int * pi_bit_rate)
+static int SyncInfo( const byte_t * p_buf, int * pi_channels,
+                     int * pi_sample_rate, int * pi_bit_rate )
 {
-    static const u8 halfrate[12] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 2, 3};
+    static const uint8_t halfrate[12] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 2, 3 };
     static const int rate[] = { 32,  40,  48,  56,  64,  80,  96, 112,
                                 128, 160, 192, 224, 256, 320, 384, 448,
-                                512, 576, 640};
-    static const u8 lfeon[8] = {0x10, 0x10, 0x04, 0x04, 0x04, 0x01, 0x04, 0x01};
+                                512, 576, 640 };
+    static const uint8_t lfeon[8] = { 0x10, 0x10, 0x04, 0x04,
+                                      0x04, 0x01, 0x04, 0x01 };
     int frmsizecod;
     int bitrate;
     int half;
@@ -339,7 +345,7 @@ int SyncInfo( const byte_t * p_buf, int * pi_channels, int * pi_sample_rate,
     default:
         return 0;
     }
-   
+
     if ( p_buf[6] & lfeon[acmod] ) *pi_channels |= AOUT_CHAN_LFE;
 
     frmsizecod = p_buf[4] & 63;