Remove empty remark.

In addition, my compiler complains about some non-sensical byte
overflow checks, which break my build (with -Werror enabled):
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

It seems that there was a bug introduced in renaming message
dvbpsi_message in dvbpsi_private.h. This is masked (in my environment
at least) by the configure script concluding that variadic macros are
not supported (i.e. ac_cv_cpp_variadic_macros=no) if -Werror is
included in the C flags, as the fragment in configure.ac used to
determine this generates a warning and thus failure.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Some distros (Ubuntu lucid) have slightly different glibc headers and require sys/types.h to be
included explicitly.

This struct was defined in two descriptors.

Found better solution.

This reverts commit 3b68bd17.

Make sure these definitions do not interact with each other.

When p->i_number == p_section->i_number AND p_prev == NULL, then the function
will crash with a segmentation fault. In this case the first element in the linked
list is going to be replaced. The pointer p_prev does not point to a valid sections,
since p is the first element in the list. To solve this case it needs to be treated
seperate.

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

Added dr_4f.{c,h} to src/descriptors/dr.h and src/Makefile.am.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

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

Various fixes:
- add boundary check for p_decoded->i_references
- fix comments

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

Duplicating p_descriptor->p_decoded which has just been newly create is
wrong. The pointer is empty. What should have been done is duplicating
the pointer p_decoded which is passed to the generator function for the
descriptor.