[FFmpeg-devel] [PATCH] lavu: introduce av_parse_ratio() and use it in ffmpeg and lavfi/aspect

Stefano Sabatini stefasab at gmail.com
Tue Jan 24 23:46:57 CET 2012


On date Saturday 2012-01-21 15:24:23 +0100, Reimar Döffinger encoded:
> On Sat, Jan 21, 2012 at 02:59:59PM +0100, Stefano Sabatini wrote:
> > av_parse_ratio_l(AVRational *q, const char *str, int max, void *log_ctx, int log_level_off)
> 
> As long as you write out "log". Saving 2 characters is hardly worth the
> obfuscation.

The problem is that then it sounds "av_parse_ratio_log()" which reads
as "parse the ratio log" why "_l" just denotes a variant of the
function (like _e, _d, _s, used in other APIs).

Updated.
-- 
FFmpeg = Fast and Frightening Meaningless Portable Evil Ghost
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavu-introduce-av_parse_ratio-and-use-it-in-ffmpeg-a.patch
Type: text/x-diff
Size: 7433 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120124/97da8a58/attachment.bin>


More information about the ffmpeg-devel mailing list