Re: [FFmpeg-devel] [PATCH] lavu/timecode: clarify error msg for timecode_rate

2017-11-09 Thread Michael Niedermayer
On Sun, Nov 05, 2017 at 09:40:41PM +0530, Gyan Doshi wrote: > Prompted by an issue reported at StackExchange, patch changes error > message for timecode_rate values below 1. > > The user-supplied value for timecode_rate in drawtext is rounded > to nearest integer. So, a supplied value of 0.49 or

Re: [FFmpeg-devel] [PATCH] lavu/timecode: clarify error msg for timecode_rate

2017-11-09 Thread Gyan Doshi
Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] lavu/timecode: clarify error msg for timecode_rate

2017-11-05 Thread Gyan Doshi
Prompted by an issue reported at StackExchange, patch changes error message for timecode_rate values below 1. The user-supplied value for timecode_rate in drawtext is rounded to nearest integer. So, a supplied value of 0.49 or lower is rounded to 0. This throws a misleading error message which