[FFmpeg-devel] [PATCH 3/3] videodsp: Fix project name

Stefano Sabatini stefasab at gmail.com
Sat Dec 22 00:53:28 CET 2012


On date Friday 2012-12-21 18:41:06 +0100, Michael Niedermayer encoded:
> These are all part of splited out dsp utils from FFmpeg
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavcodec/arm/videodsp_arm.h          |    8 ++++----
>  libavcodec/arm/videodsp_armv5te.S      |    8 ++++----
>  libavcodec/arm/videodsp_init_arm.c     |    8 ++++----
>  libavcodec/arm/videodsp_init_armv5te.c |    8 ++++----
>  libavcodec/ppc/videodsp_ppc.c          |    8 ++++----
>  libavcodec/videodsp.c                  |    8 ++++----
>  libavcodec/videodsp.h                  |    8 ++++----
>  libavcodec/videodsp_template.c         |    8 ++++----
>  libavcodec/x86/videodsp.asm            |    8 ++++----
>  libavcodec/x86/videodsp_init.c         |    8 ++++----
>  10 files changed, 40 insertions(+), 40 deletions(-)

All patches look OK.

Regarding the Libav/FFmpeg thing, I don't think that writing "This is
part of Foo" means that the code was *created* in the Foo project,
rather means that the code is part of the Foo project, whatever the
original source of the code is. For the same reason we usually write
"This file is part of FFmpeg" and not "This file is part of MPlayer"
when we port a filter to FFmpeg, and discard the original comment (not
that I care a bit and I'm not disturbed by the idea of having some
misnamed headers, so basically do as it is more convenient for you
when merging external repos, this just an opinion of course).
-- 
FFmpeg = Faithful and Forgiving Murdering Programmable Egregious Gangster


More information about the ffmpeg-devel mailing list