changeset 3538:e50efb387048 libavformat

spelling cosmetics
author diego
date Fri, 04 Jul 2008 13:05:06 +0000
parents f02643ff8be2
children 4dee13c8073a
files avidec.c
diffstat 1 files changed, 25 insertions(+), 23 deletions(-) [+]
line wrap: on
line diff
--- a/avidec.c	Thu Jul 03 12:12:03 2008 +0000
+++ b/avidec.c	Fri Jul 04 13:05:06 2008 +0000
@@ -1,6 +1,6 @@
 /*
  * AVI demuxer
- * Copyright (c) 2001 Fabrice Bellard.
+ * Copyright (c) 2001 Fabrice Bellard
  *
  * This file is part of FFmpeg.
  *
@@ -101,7 +101,7 @@
         return -1;
 
     if(header[7] == 0x19)
-        av_log(NULL, AV_LOG_INFO, "file has been generated with a totally broken muxer\n");
+        av_log(NULL, AV_LOG_INFO, "This file has been generated by a totally broken muxer.\n");
 
     return 0;
 }
@@ -262,7 +262,7 @@
 
         switch(tag) {
         case MKTAG('L', 'I', 'S', 'T'):
-            /* ignored, except when start of video packets */
+            /* Ignored, except at start of video packets. */
             tag1 = get_le32(pb);
 #ifdef DEBUG
             print_tag("list", tag1, 0);
@@ -284,7 +284,7 @@
         case MKTAG('a', 'm', 'v', 'h'):
             amv_file_format=1;
         case MKTAG('a', 'v', 'i', 'h'):
-            /* avi header */
+            /* AVI header */
             /* using frame_period is bad idea */
             frame_period = get_le32(pb);
             bit_rate = get_le32(pb) * 8;
@@ -330,7 +330,7 @@
 
                 /*
                  * After some consideration -- I don't think we
-                 * have to support anything but DV in a type1 AVIs.
+                 * have to support anything but DV in type1 AVIs.
                  */
                 if (s->nb_streams != 1)
                     goto fail;
@@ -362,7 +362,7 @@
                 }
                 /*
                  * else, leave duration alone; timing estimation in utils.c
-                 *      will make a guess based on bit rate.
+                 *      will make a guess based on bitrate.
                  */
 
                 stream_index = s->nb_streams - 1;
@@ -380,7 +380,7 @@
             ast->scale = get_le32(pb);
             ast->rate = get_le32(pb);
             if(!(ast->scale && ast->rate)){
-                av_log(s, AV_LOG_WARNING, "Scale/Rate is %u/%u which is invalid. (This file has been generated by broken software)\n", ast->scale, ast->rate);
+                av_log(s, AV_LOG_WARNING, "scale/rate is %u/%u which is invalid. (This file has been generated by broken software.)\n", ast->scale, ast->rate);
                 if(frame_period){
                     ast->rate = 1000000;
                     ast->scale = frame_period;
@@ -469,9 +469,9 @@
                     if(st->codec->extradata_size & 1) //FIXME check if the encoder really did this correctly
                         get_byte(pb);
 
-                    /* Extract palette from extradata if bpp <= 8 */
-                    /* This code assumes that extradata contains only palette */
-                    /* This is true for all paletted codecs implemented in ffmpeg */
+                    /* Extract palette from extradata if bpp <= 8. */
+                    /* This code assumes that extradata contains only palette. */
+                    /* This is true for all paletted codecs implemented in FFmpeg. */
                     if (st->codec->extradata_size && (st->codec->bits_per_sample <= 8)) {
                         st->codec->palctrl = av_mallocz(sizeof(AVPaletteControl));
 #ifdef WORDS_BIGENDIAN
@@ -490,7 +490,7 @@
                     st->codec->codec_type = CODEC_TYPE_VIDEO;
                     st->codec->codec_tag = tag1;
                     st->codec->codec_id = codec_get_id(codec_bmp_tags, tag1);
-                    st->need_parsing = AVSTREAM_PARSE_HEADERS; // this is needed to get the pict type which is needed for generating correct pts
+                    st->need_parsing = AVSTREAM_PARSE_HEADERS; // This is needed to get the pict type which is necessary for generating correct pts.
 //                    url_fskip(pb, size - 5 * 4);
                     break;
                 case CODEC_TYPE_AUDIO:
@@ -502,9 +502,11 @@
                     if (size%2) /* 2-aligned (fix for Stargate SG-1 - 3x18 - Shades of Grey.avi) */
                         url_fskip(pb, 1);
                     /* Force parsing as several audio frames can be in
-                     * one packet and timestamps refer to packet start*/
+                     * one packet and timestamps refer to packet start. */
                     st->need_parsing = AVSTREAM_PARSE_TIMESTAMPS;
-                    /* ADTS header is in extradata, AAC without header must be stored as exact frames, parser not needed and it will fail */
+                    /* ADTS header is in extradata, AAC without header must be
+                     * stored as exact frames. Parser not needed and it will
+                     * fail. */
                     if (st->codec->codec_id == CODEC_ID_AAC && st->codec->extradata_size)
                         st->need_parsing = AVSTREAM_PARSE_NONE;
                     /* AVI files with Xan DPCM audio (wrongly) declare PCM
@@ -581,8 +583,8 @@
             break;
         default:
             if(size > 1000000){
-                av_log(s, AV_LOG_ERROR, "well something went wrong during header parsing, "
-                                        "ill ignore it and try to continue anyway\n");
+                av_log(s, AV_LOG_ERROR, "Something went wrong during header parsing, "
+                                        "I will ignore it and try to continue anyway.\n");
                 avi->movi_list = url_ftell(pb) - 4;
                 avi->movi_end  = url_fsize(pb);
                 goto end_of_header;
@@ -605,7 +607,7 @@
     avi->index_loaded = 1;
     avi->non_interleaved |= guess_ni_flag(s);
     if(avi->non_interleaved) {
-        av_log(s, AV_LOG_INFO, "Non interleaved AVI\n");
+        av_log(s, AV_LOG_INFO, "non-interleaved AVI\n");
         clean_index(s);
     }
 
@@ -707,7 +709,7 @@
             pkt->destruct = dstr;
             pkt->flags |= PKT_FLAG_KEY;
         } else {
-            /* XXX: how to handle B frames in avi ? */
+            /* XXX: How to handle B-frames in AVI? */
             pkt->dts = ast->frame_offset;
 //                pkt->dts += ast->start;
             if(ast->sample_size)
@@ -778,7 +780,7 @@
             && d[1] >= '0' && d[1] <= '9'){
             n= (d[0] - '0') * 10 + (d[1] - '0');
         }else{
-            n= 100; //invalid stream id
+            n= 100; //invalid stream ID
         }
 
         //parse ##dc/##wb
@@ -802,7 +804,7 @@
                     n=1;
                     st = st1;
                     ast = ast1;
-                    av_log(s, AV_LOG_WARNING, "Invalid stream+prefix combination, assuming audio\n");
+                    av_log(s, AV_LOG_WARNING, "Invalid stream + prefix combination, assuming audio.\n");
                 }
             }
 
@@ -857,8 +859,8 @@
     return -1;
 }
 
-/* XXX: we make the implicit supposition that the position are sorted
-   for each stream */
+/* XXX: We make the implicit supposition that the positions are sorted
+   for each stream. */
 static int avi_read_idx1(AVFormatContext *s, int size)
 {
     AVIContext *avi = s->priv_data;
@@ -873,7 +875,7 @@
     if (nb_index_entries <= 0)
         return -1;
 
-    /* read the entries and sort them in each stream component */
+    /* Read the entries and sort them in each stream component. */
     for(i = 0; i < nb_index_entries; i++) {
         tag = get_le32(pb);
         flags = get_le32(pb);
@@ -1005,7 +1007,7 @@
         assert(stream_index == 0);
 
         /* Feed the DV video stream version of the timestamp to the */
-        /* DV demux so it can synth correct timestamps              */
+        /* DV demux so it can synthesize correct timestamps.        */
         dv_offset_reset(avi->dv_demux, timestamp);
 
         url_fseek(s->pb, pos, SEEK_SET);