[vlc-commits] packetizer: flac: add some comments

[Francois Cartegnie]

vlc | branch: master | Francois Cartegnie <fcvlcdev at free.fr> | Fri Jan 10 15:20:21 2014 +0100| [f39052fe93cce2bb0e0d897b317cf426b8514233] | committer: Francois Cartegnie

packetizer: flac: add some comments

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=f39052fe93cce2bb0e0d897b317cf426b8514233
---

 modules/packetizer/flac.c |    8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/modules/packetizer/flac.c b/modules/packetizer/flac.c
index 8040f4e..ee858fe 100644
--- a/modules/packetizer/flac.c
+++ b/modules/packetizer/flac.c
@@ -554,6 +554,9 @@ static block_t *Packetize(decoder_t *p_dec, block_t **pp_block)
         }
 
     case STATE_SYNC:
+        /* Sync state is unverified until we have read frame header and checked CRC
+           Once validated, we'll send data from NEXT_SYNC state where we'll
+           compute frame size */
         p_sys->i_state = STATE_HEADER;
 
     case STATE_HEADER:
@@ -581,6 +584,11 @@ static block_t *Packetize(decoder_t *p_dec, block_t **pp_block)
         p_sys->i_frame_size = p_sys->b_stream_info && p_sys->stream_info.min_framesize > 0 ?
                                                         p_sys->stream_info.min_framesize : 1;
 
+        /* We have to read until next frame sync code to compute current frame size
+         * from that boundary.
+         * The confusing part below is that sync code needs to be verified in case
+         * it would appear in data, so we also need to check next frame header CRC
+         */
     case STATE_NEXT_SYNC:
         /* TODO: If pp_block == NULL, flush the buffer without checking the
          * next sync word */