[FFmpeg-devel] [PATCH 2/2] lavc: support subtitles charset conversion.

Nicolas George nicolas.george at normalesup.org
Sun Feb 17 12:46:21 CET 2013


Le nonidi 29 pluviôse, an CCXXI, Clement Boesch a écrit :
> On top of the structure:
> 
>  "Please use AVOptions (av_opt* / av_set/get*()) to access these fields
>  from user applications."

Yes, but a lot of examples and code access the usual fields directly, and
the rule about not accessing directly the fields is written for each
publicly useful ffmpeg-only field (see codec_descriptor).

> Removed the comment. Also added options to access the field.

Thanks.

> Was left for later, removed.

I wondered why it was disabled, in fact. It does not matter much, though.

> I'm glad to hear this :)
> 
> I'll push this version soon, along with the other patches (filter changes).

LGTM at a quick glance.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130217/d0e8f999/attachment.asc>


More information about the ffmpeg-devel mailing list