[FFmpeg-devel] [PATCH] lavf: Replace all occurrences of PRI in sscanf() calls with SCN

James Almer jamrial at gmail.com
Wed Apr 10 09:40:18 CEST 2013


On 10/04/13 4:19 AM, Clément Bœsch wrote:
> On Tue, Apr 09, 2013 at 11:12:53PM -0300, James Almer wrote:
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavformat/aqtitledec.c | 2 +-
>>  libavformat/mpeg.c       | 2 +-
>>  libavformat/mpl2dec.c    | 8 ++++----
>>  libavformat/pjsdec.c     | 4 ++--
>>  libavformat/rpl.c        | 2 +-
>>  5 files changed, 9 insertions(+), 9 deletions(-)
>>
> 
> There are some in libswscale/swscale-test.c and tools/seek_print.c too
> (remember to remove the lavf in the commit message).
> 
> I wasn't aware of such macro; while it's indeed more correct, is it fixing
> a bug in practice?
> 
> Patch looks OK otherwise, thanks.

No bug. It's more correct as you said, so i thought it was worth changing.
I'll send a new patch covering swscale-test.c and seek_print.c in a moment.

Regards.


More information about the ffmpeg-devel mailing list