On Wed, Jun 30, 2010 at 2:08 AM, Ian Monroe <<a href="mailto:ian@monroe.nu">ian@monroe.nu</a>> wrote:<br>> Good point I completely forgot that we already have a subtitle API.<br><br>Probably better to rework/clarify current API before to add new functions that will complexify phonon :)<br>
<br>> Which is funny since I think I wrote it. So I'm concerned with how<br>> these two APIs will interact. Does the subtitle get added to the<br>> available subtitles?<br><br>I didn't get your question.<br>
<br>To sum up here is a diagram of current Phonon subtitle API and the way it is used by phonon-mplayer:<br><a href="http://gitorious.org/phonon/phonon-mplayer/blobs/raw/master/mplayer/doc/phonon-subtitles.png">http://gitorious.org/phonon/phonon-mplayer/blobs/raw/master/mplayer/doc/phonon-subtitles.png</a><br>
<br>About setSubtitleEncoding, I checked MPlayer documentation and there is a parameter to specify this. Btw the specification seems to be unclear cf <a href="http://lists.mplayerhq.hu/pipermail/mplayer-docs/2004-May/001541.html">http://lists.mplayerhq.hu/pipermail/mplayer-docs/2004-May/001541.html</a><br>
<br>If we want to be able to change subtitle encoding, I think a separated function setSubtitleEncoding() is probably a bad idea. It should be a parameter to setSubtitleFile() instead otherwise this won't work:<br>[...]<br>
setSubtitleFIle()<br>setSubtitleEncoding()<br>[...]<br>Or you need to re-read the subtitle inside the function setSubtitleEncoding().<br><br>On the other hand a function like setSubtitleFont() can be separated since it won't require to re-read the subtitle.<br>
<br>-- <br>Tanguy Krotoff <<a href="mailto:tkrotoff@gmail.com">tkrotoff@gmail.com</a>><br>+33 6 68 42 70 24