home
products
contribute
download
documentation
forum
Home
Forums
New posts
Search forums
What's new
New posts
All posts
Latest activity
Members
Registered members
Current visitors
Donate
Log in
Register
What's new
Search
Search
Search titles only
By:
New posts
Search forums
Search titles only
By:
Menu
Log in
Register
Navigation
Install the app
Install
More options
Contact us
Close Menu
Forums
MediaPortal 1
Quality Assurance
Bugtracker Feed
0003660: TsReader problems with PCR rollover
Contact us
RSS
JavaScript is disabled. For a better experience, please enable JavaScript in your browser before proceeding.
You are using an out of date browser. It may not display this or other websites correctly.
You should upgrade or use an
alternative browser
.
Reply to thread
Message
<blockquote data-quote="MediaPortal-Bot" data-source="post: 770888" data-attributes="member: 48617"><p>The handling of PCR rollover in TsReader is incorrect in some cases. Quote from Asselin (who raised the issue originally):</p><p></p><p>"TsReader has 2 bugs in it when dealing with TS files that have PCR rollover:</p><p>1. When you start playback, MediaPortal freezes, HDD light goes solid, and after a while (could be minutes), the file starts playing back.</p><p>2. While playing, if you try to skip forward or backward, the HDD light goes solid again, and then after a while, the recording restarts from the beginning.</p><p></p><p>Problem <a href="http://mantis.team-mediaportal.com/view.php?id=1" target="_blank">0000001</a> is a bug in TsDuration. It calculates the duration of the TS file by looking for the max PCR in the TS file before the rollover happens, and does this by a sequential (!) backwards (!!) file search. All that is unnecessary-- the code should just use a constant value (the MAX possible PCR of all binary 1s) as the value.</p><p></p><p>Problem <a href="http://mantis.team-mediaportal.com/view.php?id=2" target="_blank">0000002</a> is a bug in TsFileSeek. Here, there's a section of code specifically for scanning a TS file with PCR rollover in it, and there's an if() statement where the logic is backwards. Fixing the if() fixes the problem, but beyond that, there's a whole other algorithm in it to deal with skipping around in TS files with PCR rollover that is unnecessary. The standard binary search algorithm works just fine, and is in fact faster."</p><p></p><p><a href="http://mantis.team-mediaportal.com/view.php?id=3660" target="_blank">http://mantis.team-mediaportal.com/view.php?id=3660</a></p><p></p><p><a href="http://mantis.team-mediaportal.com/view.php?id=3660" target="_blank">Open the issue in Mantis...</a></p></blockquote><p></p>
[QUOTE="MediaPortal-Bot, post: 770888, member: 48617"] The handling of PCR rollover in TsReader is incorrect in some cases. Quote from Asselin (who raised the issue originally): "TsReader has 2 bugs in it when dealing with TS files that have PCR rollover: 1. When you start playback, MediaPortal freezes, HDD light goes solid, and after a while (could be minutes), the file starts playing back. 2. While playing, if you try to skip forward or backward, the HDD light goes solid again, and then after a while, the recording restarts from the beginning. Problem [URL="http://mantis.team-mediaportal.com/view.php?id=1"]0000001[/URL] is a bug in TsDuration. It calculates the duration of the TS file by looking for the max PCR in the TS file before the rollover happens, and does this by a sequential (!) backwards (!!) file search. All that is unnecessary-- the code should just use a constant value (the MAX possible PCR of all binary 1s) as the value. Problem [URL="http://mantis.team-mediaportal.com/view.php?id=2"]0000002[/URL] is a bug in TsFileSeek. Here, there's a section of code specifically for scanning a TS file with PCR rollover in it, and there's an if() statement where the logic is backwards. Fixing the if() fixes the problem, but beyond that, there's a whole other algorithm in it to deal with skipping around in TS files with PCR rollover that is unnecessary. The standard binary search algorithm works just fine, and is in fact faster." [url]http://mantis.team-mediaportal.com/view.php?id=3660[/url] [url=http://mantis.team-mediaportal.com/view.php?id=3660]Open the issue in Mantis...[/url] [/QUOTE]
Insert quotes…
Verification
Post reply
Forums
MediaPortal 1
Quality Assurance
Bugtracker Feed
0003660: TsReader problems with PCR rollover
Contact us
RSS
Top
Bottom