- add boundary checking
- add doxygen comment
- fix some typos.
- fix L1L2MERGE macro

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

The API in version 1.0.0 has changed and the macros dvbpsi_New*/dvbpsi_Delete* have been
deprecated (and removed).

Remove unnecessary #ifndef BSD. netinet/ip.h is a standard header on *BSD. Also, most BSDs don't define "BSD".
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

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

The transport stream loop started 2 bytes too far and the while() condition was
pointing to 2 bytes too soon. The code has been corrected to first add 2 bytes
to the current position, thereby skipping the length fields (which have already
been read). Now the loop starts at the correct position.

The member i_crc is not used anymore now the CRC32 can be checked and
generated by the dvbpsi_BuildPSISection() and dvbpsi_ValidPSISection()
functions.

From dvbpsi_packet_push() we can learn that for the TOT table,
with syntax_indicator '0' the p_section->p_payload_start pointer
points to UTC Time field (3 bytes after PSI section start).

To get the byte position of the descriptor loop one needs to add
2 bytes instead of 5, as was previously done.

If one needs to calculate the CRC32 outside of libdvbpsi, then they
should use the new dvbpsi_CalculateCRC32() function instead.

Rename AM_CONFIG_HEADER to AC_CONFIG_HEADERS

In the past the b_syntax_indicator field could be used to distinguish between
tables with a CRC32 and those without. However these days this assumption is no
longer valid and the check should be modified to include tables that have a CRC32
even when syntax_indcator is '0'.

CAT: The p_section->p_payload_start pointer already points to the fist descriptor for this PSI section.

According to ITU-T Rec H.222.0 and the code in dvbpsi_packet_push() the
p_section->p_payload_start pointer already points to the first descriptor for
this PSI table. Adding 5 bytes is plain wrong.

- document first argument of dvbpsi_bat_ts_descriptor_add()

The function dvbpsi_decoder_psi_sections_completed() is meant to detect if all
sections of a multi section psi table have arrived. After the rewrite in
commit-id 57c6cd63 it ignored gaps in the arrived sections. This means that
the function now expects that individual sections of multi section psi tables
arrive in order. However the ISO/IEC 13818-1 standard explains in Annex C, C 2ii
that it is allowed to transmit sections out of order.

 "The section_number field allows the sections of a particular table to be reassembled
  in their original order by the decoder. There is no obligation within this Recommendation
  | International Standard that sections must be transmitted in numerical order, but this
  is recommended, unless it is desired to transmit some sections of the table more frequently
  than others, e.g. due to random access considerations."

The p_decoder->p_sections linked list is sequentially ordered and is now changed to detect
a gap in the ordering. This is possible because the first section in a multi-section psi table
is numbered 0.

change 0x8C & 0x9C to 0xC8 & 0xC9
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

The SDT field known as i_ts_id is named i_extension in the struct dvbpsi_psi_section_t.