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

Nico nsabbi at libero.it
Thu Aug 7 23:50:35 CEST 2003


>
>
>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).
>
>  
>

Hi,
following your suggestion I made static all non necessary functions, 
renamed the others to dvb_* and  removed all  globals
variables but one: dvb_list_ptr.
Actually it's global because I need to access it from the menu code, but 
I'd like to remove even that.
Is it possible to access the current stream->priv from the menu?

Thanks,
    Nico

>A'rpi / Astral & ESP-team
>
>--
>Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu
>
>_______________________________________________
>MPlayer-dev-eng mailing list
>MPlayer-dev-eng at mplayerhq.hu
>http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>  
>




More information about the MPlayer-dev-eng mailing list