[FFmpeg-devel] [PATCH] av_find_best_stream

[Nicolas George]

Le duodi 2 niv?se, an CCXIX, Stefano Sabatini a ?crit?:
> user is a confusing name, I'd prefer something like user_stream_nb or
> wanted_stream_nb.

Ok.

> not NULL -> non-NULL

Ok.

> a bit confusing, I suggest:
> returns the decoder for the selected stream

Ok.

> *codec -> *decoder (or ther other way change the parameter name as it
> fits you best)

Fixed.

> my suggestion:
> decoder_ptr
> decoder
> best_decoder

I went for decoder_ret, decoder and best_decoder.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-findstream-20101222-1843-01-findstream.diff
Type: text/x-diff
Size: 6528 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101222/5770ce1f/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101222/5770ce1f/attachment.pgp>