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

Clément Bœsch ubitux at gmail.com
Wed Apr 10 10:03:12 CEST 2013


On Wed, Apr 10, 2013 at 04:40:18AM -0300, James Almer wrote:
> 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.
> 

Thanks, applied.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130410/fce5b64c/attachment.asc>


More information about the ffmpeg-devel mailing list