[FFmpeg-devel] [PATCH] fftools/ffmpeg_opt: Add -toeof option to stop writing relative to EOF

[Morten With]

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 <gyandoshi at 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 at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>