On Fri, Jan 29, 2016 at 10:10:48AM +0100, Clément Bœsch wrote: > On Thu, Jan 28, 2016 at 06:33:56PM -0800, Timothy Gu wrote: > > --- > > tests/fate/source-check.sh | 11 +++++++++++ > > tests/ref/fate/source | 12 ++++++++++++ > > 2 files changed, 23 insertions(+) > > > > diff --git a/tests/fate/source-check.sh b/tests/fate/source-check.sh > > index fb7af98..5524236 100755 > > --- a/tests/fate/source-check.sh > > +++ b/tests/fate/source-check.sh > > @@ -16,5 +16,16 @@ git grep -L -E "This file is part of FFmpeg|This file is > > part of libswresample|" > > "This program is free software; you can redistribute it and/or modify|"\ > > "This file is placed in the public domain" | grep -E > > '\.c$|\.h$|\.S$|\.asm$' > > > > +echo Headers without standard inclusion guards: > > +for f in `git ls-files | grep '\.h$'` ; do > > + macro="`echo $f | sed \ > > + -e '/\/\|^ff/!{s/\(.*\)/ffmpeg\/\1/}' \ > > + -e 's/^lib//' \ > > + -e 's/[^A-Za-z0-9]\{1\,\}/_/g' \ > > + -e 's/_\(a\|v\|av\)f_/_/' \ > > > + | tr '[:lower:]' '[:upper:]'`" > > Not sure about the portability of these arguments. I think it might be > safer to copy/use toupper() from the configure.
Changed locally. Timothy _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel