[FFmpeg-devel] [PATCH 1/3] avfilter/vf_telecine: use the name 's' for the pointer to the private context

Stefano Sabatini stefasab at gmail.com
Sun Oct 13 16:18:45 CEST 2013


On date Friday 2013-10-11 11:40:41 +0000, Paul B Mahol encoded:
> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> ---
>  libavfilter/vf_telecine.c | 110 +++++++++++++++++++++++-----------------------
>  1 file changed, 55 insertions(+), 55 deletions(-)

I'm mildly against renames to a meaningless single char, especially
given that the context is private and can't be copy pasted to other
part of the code, and makes it harder to read the code out-of-context
(e.g. in a patch). This also adds unnecessary code churnup for the
sake of one character to type.

That said, I'm no maintainer of the code in question.
-- 
FFmpeg = Frenzy Funny Mastodontic Problematic Elected Ghost


More information about the ffmpeg-devel mailing list