On Sun, Oct 30, 2011 at 10:08:47PM +0100, Diego Biurrun wrote:
> Filenames are brittle across renames and add no useful information.
> ---
>  libavcodec/arm/simple_idct_arm.S |    1 -
>  libavcodec/jfdctfst.c            |    2 --
>  libavcodec/jfdctint_template.c   |    2 --
>  libavcodec/jrevdct.c             |    2 --
>  libavcodec/libdirac.h            |    2 +-
>  libavcodec/libschroedinger.c     |    2 +-
>  libavcodec/libschroedinger.h     |    2 +-
>  libavcodec/x86/idct_mmx.c        |    1 -
>  8 files changed, 3 insertions(+), 11 deletions(-)
> 
> diff --git a/libavcodec/arm/simple_idct_arm.S 
> b/libavcodec/arm/simple_idct_arm.S
> index 717b12c..fce40b7 100644
> --- a/libavcodec/arm/simple_idct_arm.S
> +++ b/libavcodec/arm/simple_idct_arm.S
> @@ -1,5 +1,4 @@
>  /*
> - * simple_idct_arm.S

'twould be nice to have a description here

>   * Copyright (C) 2002 Frederic 'dilb' Boulay
>   *
>   * Author: Frederic Boulay <[email protected]>
> diff --git a/libavcodec/jfdctfst.c b/libavcodec/jfdctfst.c
> index 8d3448a..b8dc407 100644
> --- a/libavcodec/jfdctfst.c
> +++ b/libavcodec/jfdctfst.c
> @@ -1,6 +1,4 @@
>  /*
> - * jfdctfst.c
> - *
>   * This file is part of the Independent JPEG Group's software.
>   *
>   * The authors make NO WARRANTY or representation, either express or implied,
> diff --git a/libavcodec/jfdctint_template.c b/libavcodec/jfdctint_template.c
> index e60e72a..02b6f09 100644
> --- a/libavcodec/jfdctint_template.c
> +++ b/libavcodec/jfdctint_template.c
> @@ -1,6 +1,4 @@
>  /*
> - * jfdctint.c
> - *

and here

>   * This file is part of the Independent JPEG Group's software.
>   *
>   * The authors make NO WARRANTY or representation, either express or implied,
> diff --git a/libavcodec/jrevdct.c b/libavcodec/jrevdct.c
> index 9e28dae..5ed96da 100644
> --- a/libavcodec/jrevdct.c
> +++ b/libavcodec/jrevdct.c
> @@ -1,6 +1,4 @@
>  /*
> - * jrevdct.c
> - *

and here, or maybe not touch those two files at all - they are not ours

>   * This file is part of the Independent JPEG Group's software.
>   *
>   * The authors make NO WARRANTY or representation, either express or implied,
> diff --git a/libavcodec/libdirac.h b/libavcodec/libdirac.h
> index e054f61..4403fb2 100644
> --- a/libavcodec/libdirac.h
> +++ b/libavcodec/libdirac.h
> @@ -20,7 +20,7 @@
>  
>  /**
>  * @file
> -* data structures common to libdiracenc.c and libdiracdec.c
> +* data structures common to libdirac encoder and decoder
>  */
>  
>  #ifndef AVCODEC_LIBDIRAC_H
> diff --git a/libavcodec/libschroedinger.c b/libavcodec/libschroedinger.c
> index 1d31712..527c492 100644
> --- a/libavcodec/libschroedinger.c
> +++ b/libavcodec/libschroedinger.c
> @@ -20,7 +20,7 @@
>  
>  /**
>  * @file
> -* function definitions common to libschroedingerdec.c and 
> libschroedingerenc.c
> +* function definitions common to libschroedinger decoder and encoder
>  */
>  
>  #include "libdirac_libschro.h"
> diff --git a/libavcodec/libschroedinger.h b/libavcodec/libschroedinger.h
> index c6ae5ea..8147821 100644
> --- a/libavcodec/libschroedinger.h
> +++ b/libavcodec/libschroedinger.h
> @@ -20,7 +20,7 @@
>  
>  /**
>  * @file
> -* data structures common to libschroedingerdec.c and libschroedingerenc.c
> +* data structures common to libschroedinger decoder and encoder
>  */
>  
>  #ifndef AVCODEC_LIBSCHROEDINGER_H
> diff --git a/libavcodec/x86/idct_mmx.c b/libavcodec/x86/idct_mmx.c
> index 57fa818..ed2d321 100644
> --- a/libavcodec/x86/idct_mmx.c
> +++ b/libavcodec/x86/idct_mmx.c
> @@ -1,5 +1,4 @@
>  /*
> - * idct_mmx.c
>   * Copyright (C) 1999-2001 Aaron Holtzman <[email protected]>
>   *
>   * This file is part of mpeg2dec, a free MPEG-2 video stream decoder.

I wouldn't touch this file too, the rest look fine though.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to