[FFmpeg-devel] [PATCH 4/4] avfilter/f_metadata: whitespace clean-up

Tobias Rapp t.rapp at noa-archive.com
Thu Feb 11 14:31:43 CET 2016


On 11.02.2016 14:12, Paul B Mahol wrote:
> On 2/11/16, Tobias Rapp <t.rapp at noa-archive.com> wrote:
>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
>> ---
>>   libavfilter/f_metadata.c | 16 ++++++++--------
>>   1 file changed, 8 insertions(+), 8 deletions(-)
>>
>
> Patches LGTM.
>
> Removed tabs from patches, gonna apply it soon.

Sorry for the tabs, I forgot to add "set expandtab" in .vimrc on my machine.

I guess after patch #2 the "string" function can be reduced to check for 
full string match only and doesn't need "length" anymore? If you agree 
what about renaming "string" to "same_str"?

Regards,
Tobias



More information about the ffmpeg-devel mailing list