[libdvbpsi-devel] [Git][videolan/libdvbpsi][master] 4 commits: really set last section number

[Jean-Paul Saman]

Jean-Paul Saman pushed to branch master at VideoLAN / libdvbpsi


Commits:
65295c7b by Francois Cartegnie at 2019-07-18T17:22:14Z
really set last section number

fixes regressions by commit
7e9f4ed39444c84679b1135684ec2aa4f39e476c
(early multi sections table end, segmented EIT failure)

- - - - -
b130588d by Francois Cartegnie at 2019-07-18T17:26:16Z
output table number in debug

otherwise that's pretty useless in SI flow

- - - - -
90b14176 by Francois Cartegnie at 2019-07-18T17:27:03Z
debug EIT last section values

- - - - -
36d88fa6 by Francois Cartegnie at 2019-07-18T17:27:15Z
eit: remove double last_number setting

- - - - -


3 changed files:

- src/dvbpsi.c
- src/psi.c
- src/tables/eit.c


Changes:

=====================================
src/dvbpsi.c
=====================================
@@ -222,7 +222,7 @@ bool dvbpsi_decoder_psi_section_add(dvbpsi_decoder_t *p_decoder, dvbpsi_psi_sect
     /* Add to end of list */
     if (p_prev->i_number < p_section->i_number)
     {
-        p_decoder->i_last_section_number = p_section->i_number;
+        p_decoder->i_last_section_number = p_section->i_last_number;
         p_prev->p_next = p_section;
         p_section->p_next = NULL;
     }


=====================================
src/psi.c
=====================================
@@ -215,8 +215,9 @@ bool dvbpsi_CheckPSISection(dvbpsi_t *p_dvbpsi, dvbpsi_psi_section_t *p_section,
     }
 
     dvbpsi_debug(p_dvbpsi, psz_table_name,
-                   "Table version %2d, " "i_extension %5d, "
+                   "Table %3d version %2d, " "i_extension %5d, "
                    "section %3d up to %3d, " "current %1d",
+                   p_section->i_table_id,
                    p_section->i_version, p_section->i_extension,
                    p_section->i_number, p_section->i_last_number,
                    p_section->b_current_next);


=====================================
src/tables/eit.c
=====================================
@@ -316,8 +316,9 @@ static bool dvbpsi_CheckEIT(dvbpsi_t *p_dvbpsi, dvbpsi_eit_decoder_t *p_eit_deco
     {
         /* last_section_number */
         dvbpsi_error(p_dvbpsi, "EIT decoder",
-                     "'last_section_number' differs"
-                     " whereas no discontinuity has occurred");
+                     "'last_section_number' differs %2d / %2d "
+                     " whereas no discontinuity has occurred",
+                     p_eit_decoder->i_last_section_number, p_section->i_last_number);
         b_reinit = true;
     }
 
@@ -403,7 +404,6 @@ static bool dvbpsi_AddSectionEIT(dvbpsi_t *p_dvbpsi, dvbpsi_eit_decoder_t *p_eit
 
         if (p_eit_decoder->p_building_eit == NULL)
             return false;
-        p_eit_decoder->i_last_section_number = p_section->i_last_number;
     }
 
     /* Add to linked list of sections */



View it on GitLab: https://code.videolan.org/videolan/libdvbpsi/compare/be3ae3225f5270d259d4e434898946cb95ca3217...36d88fa6abe60d3acfd1b038ecf89fcbc102fed0

-- 
View it on GitLab: https://code.videolan.org/videolan/libdvbpsi/compare/be3ae3225f5270d259d4e434898946cb95ca3217...36d88fa6abe60d3acfd1b038ecf89fcbc102fed0
You're receiving this email because of your account on code.videolan.org.