[FFmpeg-soc] [soc]: r312 - in dvbmuxer: mpeg_pes.h mpeg_pes_enc.c pes.diff

[Diego Biurrun]

On Thu, Jul 05, 2007 at 08:27:47AM +0200, realsun wrote:
> 
> Log:
> Fixed some spelling mistakes.

Some more nitpicks, might be duplicated from what I wrote on
ffmpeg-devel...

> --- dvbmuxer/mpeg_pes.h	(original)
> +++ dvbmuxer/mpeg_pes.h	Thu Jul  5 08:27:47 2007
> @@ -20,7 +20,7 @@
>  
>  /**
>   * @file mpeg_pes.h
> - * PES packetizer api header.
> + * MPEG PES packetizer api header.

API, stray period

> @@ -86,21 +86,21 @@ typedef struct {
>  
> - * Initialization of PES mux.
> + * Initialization of PES muxer.

I think it's better English to write "PES muxer initialization".

>  /**
>   * Finalization of PES mux.

PES muxer finalization

> @@ -108,17 +108,17 @@ void ff_pes_mux_end(AVFormatContext *ctx
>  
>  /**
> - * Find the most fit stream to be muxed.
> + * Find which stream is most appropriate be muxed.

Not a very good description, but I'm not very happy with my alternative
either:

  Find the stream which is most appropriate for muxing.

> @@ -148,8 +148,8 @@ int ff_pes_mux_write(AVFormatContext *ct
>  /**
>   * Remove decoded packets of each stream.

I think you mean "from each stream" if the end result is the stream
minus the decoded packets.

Diego