Kostya Shishkov <[email protected]> writes: > 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
The file name and contents are fairly self-descriptive. Either way, repeating the filename here is clearly pointless and the patch thus OK. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
