[MPlayer-dev-eng] [PATCH] dvb input update

Arpi arpi at thot.banki.hu
Wed Aug 6 00:45:49 CEST 2003


Hi,

> here's the dvb patch updated with documentation in HTML.
> Diego, can you please take a look at and correct  the docs?.
> 
> I also fixed a small bug in the dvb channel parsing code.
> 
> I hope to see this patch applied :)

Maybe, after some changes.

Please, split it to at least 3 parts:
- DOCS/
- menu/input changes (i dont know that code, so i wont apply it, maybe Albeu
  will)
- demuxer changes (i'll check and commit)

About the demuxer part of your patch:
- too much cosmetics
- i see some progress in removing namespace mess (lots of global vars)
  but:
- ther are new, even worse 'namespace polluting' entries, like the very very
  confusing DVB-specific demux_start(), open_fe(), get_tuner_type()...
  please prefix all exported (non-static) variabne and function name with
  dvb_ or dvbin_. but it's better make them static when possible, i doubt
  you really need so much global funcs and vars. think of g2, where globals
  are not allowed at all!

if you fix these, i'll commit the patch (the demuxer part).


A'rpi / Astral & ESP-team

--
Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu



More information about the MPlayer-dev-eng mailing list