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

Michael Niedermayer michaelni at gmx.at
Sun Jan 6 13:40:06 CET 2013

On Sat, Jan 05, 2013 at 12:54:37PM +0100, Nicolas George wrote:
> Le quintidi 15 nivôse, an CCXXI, Clement Boesch a écrit :
> > I fail to see how it is more elegant; the codec properties sounds like the
> > best place to declare such generalities.
> It is hard tu put elegance considerations into words. Looking at the various
> existing CODEC_CAP, I find they are usually more universal and/or more
> relevant to the API user, although I realize there are already exceptions.
> >					   Using the context structure is
> > only a necessity if we need on the fly changes, which don't sound common
> > at all. And if we find such insanity, I'd suggest to fix that mess in the
> > decoder or the demuxer itself.
> What about this, that I thought of this morning:
> Sometimes, the recoding will be perforce be done by the demuxer. At other
> times, it will be done by lavc. In any case, the original encoding should be
> exposed to the API caller, so that this:
> ffmpeg -ss 5 -i file.fmt [ -sub_charenc copy ] shifted_file.fmt
> can work. And for convenience and compatibility reasons, it is probably be
> best if the original encoding is exported in the same field.
> Thus my proposal with sub_charenc_mode and the first component that decides
> it can do the work sets it. That would work like that:
> 1. If the demuxer knows the character encoding, it sets sub_charenc.
> 2. If the demuxer does the recoding, then it sets sub_charenc_mode to DONE,
>    otherwise it leaves it to its default 0.
> 3. If mode is still 0, the codec init function sets it to either PRE, POST
>    or INTERNAL depending on its need.
> 4. If mode is still 0 after codec init and a character encoding is set, lavc
>    reports an error.

All this pre, post, ... stuff sounds rather messy.
Where is the problem of simply having every public function
communicate with the "outside" through UTF-8 unicode ?


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- 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/20130106/a310c4b4/attachment.asc>

More information about the ffmpeg-devel mailing list