[MPlayer-dev-eng] [PATCH] demux_ts fixes

Arpi arpi at thot.banki.hu
Fri Jun 6 21:31:08 CEST 2003


Hi,

> I fixed the following in demux_ts:

I'm about commiting this, but i've find 2 serious problem:
- you seem to use tabwidth=4, with tabwidth=8 it has unreadable random
indentation. it wouldn't be problem fi you use tabs all time, but 4 spaces
and tab is mixed resulting stuff like:

        ES_stream_t *tss;
    uint8_t done = 0;
    int buf_size, is_start;
    int len, pid, last_pid, cc, cc_ok, afc, _read;
        ts_priv_t * priv = (ts_priv_t*) demuxer->priv;
        stream_t *stream = demuxer->stream;
        char *p, tmp[TS_FEC_PACKET_SIZE];
        demux_stream_t *ds;
        demux_packet_t *dp;


    while(! done)       //while pid=last_pid add_to_buffer
    {
          if(! ts_sync(stream))
          {
            mp_msg(MSGT_DEMUX, MSGL_V, "TS_PARSE: COULDN'T SYNC\n");
            return 0;
      }
 
i don't really care, but as this patch is mostly cosmetics anyway,
and the original source was much better indented, i vote for fixing this
(either convert tabs to spaces or 4tabs to 8tabs or sth) before commiting.

- second problem: there are lots of namespace pollution: ie. private
functions declared as global. please make all static except the really
exported functions
(demux_seek_ts,demux_close_ts,demux_open_t,sts_check_file,demux_ts_fill_buffer)
exporting functions like ts_parse adn ts_sync ha sno sence and may result in
symbol clashes with external libraries.

note that dvbin.c and friends ahs a lot more exported private variables with
very short, basic names, for example:

dvb_channels_list  *list_ptr = NULL;
int card=0;



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