[FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

Michael Niedermayer michael at niedermayer.cc
Mon Jan 18 01:40:00 CET 2016


On Sun, Jan 17, 2016 at 11:17:36PM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michael <at> niedermayer.cc> writes:
> 
> > > > > > this should check that teres no "tail" after the integer
> > > > >
> > > > > For which command line will this make a difference?
> > > > > I didn't find one...
> 
> > indeed, i guess x264 checks this internally before 
> > our code gets a chance to fail
> 
> So is the patch ok?

id use strtol instead of strtod to parse a value that is expeted to
be in integer, or is it not expected to be an integer ?

either way, it should be ok

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160118/b24400ec/attachment.sig>


More information about the ffmpeg-devel mailing list