The dvbpsi_bat_sections_decode method parsing index was wrongly incremented
dvbpsi_AddSectionBAT method did not correctly check for memory allocation.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

The function arguments and its order have changed during cleaning up.
This breaks the old API and a rename of functions was needed to properly
mark the older API as deprecated.

- rename p_dvbpsi->p_private to p_dvbpsi->p_decoder
- use dvbpsi_NewDecoder() where possible

The function reimplemented the CRC check function which is available
in src/psi.c

Adding a descriptor to the end of the descriptor list has now been
moved to a utility function implemented in src/descriptor.c

Add a PSI section to dvbpsi_decoder_t::ap_sections[] array. The function returns
true when it overwrites an earlier entry and false otherwise.

The function dvbpsi_ChainSectionsDecoder() turns the dvbpsi_decoder_t::ap_sections[] array
into a linked list.

The table decoder check for all sections received in the same manner, this
is now abstracted away in the dvbpsi_SectionsCompleteDecoder() function.

Add a function to reinit a decoder as is needed by PSI decoders.

The struct members ::p_current_valid, ::i_last_section_number and
::ap_sections[] array are used by all PSI section decoders. They are
moved to DVBPSI_DECODER_COMMON.

Implemented a function fifo_size() that returns the current size of the fifo.
The dvbinfo example program uses fifo_size() to detect if the total size of
buffers inside the fifo exceeds a certain threshold. The threshold is currently
defined at compile time.

Refactor out common code for checking PSI sections for PSI tables.

All PSI table check the same common things before accepting a section as
being a valid section. Centralize these checks into a single function.

Refactor dvbpsi_GatherPATSections().

The callback member of dvbpsi_decoder_t is used by dvbpsi_GatherXXSections()
functions. This patch makes the usage visisble.

Update doxygen information and fix linkage of howto-new-api.doxygen

The functions dvbpsi_GatherXSections() and dvbpsi_NewDemuxSubDecoder() take a
private decoder as argument. Change this argument from 'void *' to 'dvbpsi_decoder_t *'.
This makes its use more clear.

Use new utility functions to replace common code blocks. Moving
code to demux.c instead of repeating them in the subtable functions.
This reduces the amount of code in subtable files and reduces the risk
of introducing errors in future for this code.

Refactor common demux code structures into utility functions.
See the complete list below:
- rearrange arguments of dvbpsi_demux_new_cb_t callback
- define dvbpsi_demux_gather_cb_t callback type
- define dvbpsi_demux_detach_cb_t callback type
- new utility function dvbpsi_NewDemuxSubDecoder()
- new utility function dvbpsi_DeleteDemuxSubDecoder()
- new utility funciton dvbpsi_AttachDemuxSubDecoder()
- new utility function dvbpsi_DetachDemuxSubDecoder()