CCExtractor / ccextractor

CCExtractor - Official version maintained by the core team
https://www.ccextractor.org
GNU General Public License v2.0
721 stars 426 forks source link

[FIX] Compatibility of Arguments in C #1564

Closed prateekmedia closed 10 months ago

prateekmedia commented 1 year ago

In raising this pull request, I confirm the following (please check boxes):

My familiarity with the project is as follows (check one):


Breaking change, will help in compatibility between rust and c arguments

ccextractor-bot commented 10 months ago

CCExtractor CI platform finished running the test files on windows. Below is a summary of the test results, when compared to test for commit 63a259a...:

Report Name Tests Passed
Broken 12/13
CEA-708 14/14
DVB 4/7
DVD 3/3
DVR-MS 2/2
General 16/27
Hauppage 1/3
MP4 3/3
NoCC 10/10
Options 82/86
Teletext 21/21
WTV 13/13
XDS 27/34

All tests passing on the master branch were passed completely.

NOTE: The following tests have been failing on the master branch as well as the PR:

Congratulations: Merging this PR would fix the following tests:


Check the result page for more info.

prateekmedia commented 10 months ago

cc @cfsmp3 @canihavesomecoffee, The tests that are not passing seems to be regressions as they are not working with master branch either. This PR is ready for review from my side.

ccextractor-bot commented 10 months ago

CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results, when compared to test for commit 63a259a...:

Report Name Tests Passed
Broken 12/13
CEA-708 14/14
DVB 4/7
DVD 3/3
DVR-MS 2/2
General 16/27
Hauppage 1/3
MP4 2/3
NoCC 10/10
Options 82/86
Teletext 20/21
WTV 0/13
XDS 27/34

All tests passing on the master branch were passed completely.

NOTE: The following tests have been failing on the master branch as well as the PR:

Congratulations: Merging this PR would fix the following tests:


Check the result page for more info.