[MPlayer-dev-eng] [PATCH] new libvo driver: vo_ivtv [ask for review]

Benjamin Zores ben at geexbox.org
Tue Jul 11 10:45:14 CEST 2006


>> +static char ivtvbuf[262144]; /* 256 kB */
>
> IMHO write it as 256 * 1024 and get rid of the comment.

ok, fine by me.

>> +  if (arg)
>> +  {
>> +    if (!strncmp (arg, VO_IVTV_SUBOPTION_HELP,
>> +                  strlen (VO_IVTV_SUBOPTION_HELP)))
>> +    {
>
> I'd prefer if you could use the suboption parser as e.g. vo_gl does.

wasn't aware of that and it'll for sure be better
(and it would have saved me time for writing this small parser).

>> +  printf ("Out: %d\n", out);
>
> hmmm left-over debugging code?

exactly.

> Hmmm... I always supposed the data from the draw frame must stay valid
> until after the flip. In that case you could save the memcpy and write
> the data from the buffer directly. Did you try that?

ok, i'll have a try, we'll see.

> Ok, end of very quick surfacial review.

thx for it (there's no tricky parts in this vo anyhow).

Ben




More information about the MPlayer-dev-eng mailing list