Yeah, I was thinking that there seemed to be very little validation of -sseof combinations. That's why I initially didn't add any of those for -toeof.
You are correct, that is missing, I'll add that once your patch is done then. Also, was this the correct way to send in a revised patch? I'm completely new to this. Regards, Morten 2018-06-11 22:23 GMT+02:00 Gyan Doshi <gyando...@gmail.com>: > > > On 12-06-2018 12:48 AM, Morten With wrote: > > commit message to "stop reading"), here is the revised patch attached. >> > > Maybe I missed it, but in the new patch, you don't vet ss against toeof > i.e. for a 10-second file, -ss 7 -toeof -4 should be declared invalid. > > Also, this isn't an issue with your patch but ss / sseof dual setting > isn't checked nor is the validity of the sseof value. > > Let me get a patch in first to rectify that. > > Regards, > Gyan > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel