[MPlayer-cvslog] r23530 - in trunk: Changelog DOCS/man/en/mplayer.1 DOCS/tech/MAINTAINERS DOCS/tech/slave.txt cfg-common.h command.c configure input/input.c input/input.h libvo/sub.c libvo/sub.h mpcommon.c mplayer.c spudec.c spudec.h stream/Makef

[Vladimir Voroshilov]

2007/6/16, Michael Niedermayer <michaelni at gmx.at>:
> Hi
>
> On Fri, Jun 15, 2007 at 11:36:57PM +0700, Vladimir Voroshilov wrote:
> > I've tried your code. Unfortunately i could'nt get it working
> > (something wrong at sync stage or i didn't undestand correclty how
> > your code is working), thus i have got that part of code from MythTV
> > project (it does not use sin() and cos(), and works only with help of
> > fixed point operations).
>
> its a little hard to comment on code i dont see ...
Sorry, i forget to attach sample. Attached file is current version
with 'min/max' and clock run-in sequence based sync routine.


> so ill explain how my sync code works
> the sync burst (or whatever it was called) is correlated (dot product to be
> precisse) against a sine and cosine (these can be replaced by 2 tiny integer
> tables to avoid FPU use)
> if you now look at the 2 values you got from the 2 dot products against
> sin and cos and think of them as x and y coordinates in the plane then the
> angle they have from (0,0) is the phase of the signal
>
> this is optimal, other variants like searching for the sample with largest
> amplitude to find the phase are not
What means more optimal? Better precise, speed?

> > XawTV has even non-standard latin1->koi8-r
> > renderer.
>
> text should _always_ be utf8
This will require just latin1->utf8 conversion table for each language
(and some option to force language, because standard does not contain
id e.g. for russian language).

> > Now i have working version of teletext grabber (for packets 0,1-23
> > only, no graphics or color).
> > Unfortunately, i don't know how to implement renderer, even text-only
> > render is non-trivial.
>
> wheres the problem?
Specs. I didn't find any open documentation about teletext
pseudographics coding scheme.
I event don't know how to drop out this non-text part from page.


> > > also looking at zvbis upcomming features
> > > * Multi-station Teletext page cache
> > > that one my code supported 8 years ago already ...
> >
> > I can't get it caching all teletext pages, so i've made my own trivial
> > array-based cache.
>
> elaborate please
Even with 0x999 cache size zvbi drops page out of cache after some
time (i have to wait untill page is loaded through vbi after page
jump, perhaps, my testing was not enough clear, though)

> > > * Teletext and CC caption/subtitle recording in formats like MPSub,
> > >   QuickTime Text, SAMI or SubViewer, with color, styles and position
> > >   preserved as far as possible
> > > that one qualifies the lib as something i will never ever install on
> > > any system i own
> > This library also has nice debug fprintf(stderr,...) for about four
> > years long ;)
> >
> > > PS: and yes i do vote for reverting the patch and i vote against any
> > > patch using an external lib for teletext decoding theres no need for
> > > this, teletext is very simple
> > If i undestand you right, your main objections is
> > unnessary extra functionality of zvbi
>
> yes
>
>
> > and unrealistic future plans?
>
> not unrealistic but unneeded and inappropriate, its like putting a
> web browser in libc
>
> [...]
> > What about keeping zvbi as-is untill native grabber/renderer will be
> > implemented (i'm afraid this will take a looong time)?
>
> i dont think it will take a long time
>
I want to beleive too :)

-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mp_msg.h
Type: application/octet-stream
Size: 347 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20070616/a254cee8/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 4.txt.c
Type: application/octet-stream
Size: 18814 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20070616/a254cee8/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Makefile
Type: application/octet-stream
Size: 70 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20070616/a254cee8/attachment-0002.obj>