Call for tester for 1.4.0 features and fixes (2 Viewers)

[FreakyJ]

If you are interested:
Commit https://github.com/MediaPortal/MediaPortal-1/commit/c109142235201b02e0e20801509f24c92dbf838c
was the bad guy
I have to say it again: thanks for your great work so far! You bring MP on a complete new level

 

[Scythe42]

New build attached to first post.

• TS playback should be fine now again
• Added a grace period for the refresh rate changer of 250ms to avoid a minimizing to tray.

Please test if these issues are fixed and what other you might encounter.

If you are interested:
Commit https://github.com/MediaPortal/MediaPortal-1/commit/c109142235201b02e0e20801509f24c92dbf838c
was the bad guy

Sure it was me. No doubt about that one... As I don't watch TV and therefore don't play any TS files I never noticed it got broken for quite some time now.

Off to break some more stuff . Maybe just Radio this time. Don't listen to it either. Only Pandora. Anything with commercials is a no go for me.

 

[chrissnow]

Great news
Will test it later, is it in GIT? I only ask as without merging DJBlu's EPG mods I wont be allowed to test it on the TV

 

[Scythe42]

I need to sort out my current local commit history (lot of temporary stuff) before I push it to github. Branch is: EXP-Windows-7-Half-Full-Screen

 

[tourettes]

I spotted on change that will cause regression:

-	Thread.CurrentThread.Name = "MPMain"; 
+	Thread.CurrentThread.Name = "MediaPortal";

It will break the BaseTSReaderPlayer.cs in

if (_ireader != null && Thread.CurrentThread.Name == "MPMain")
{		
  _ireader.SetMediaPosition((long)(_streamPos * 10000000d));
}

 

[offbyone]

It will also break other Plugins that rely on the Name of that thread like OnlineVideos.

 

[tourettes]

It will also break other Plugins that rely on the Name of that thread like OnlineVideos.

In generally it would be good to use the thread id instead of the thread name which is usually used for the debugging aid. But to not to make regression then it might be safer to rename the thead back as we don't know what additional plugins are going to break.

 

[Scythe42]

Already done guys in the latest uploaded build. Name is back to what it was.

 

[infinite.loop]

With the new build I can play *.ts files again.

Still have the "blackscreen" issue after the fullscreensplashscreen is gone (did you have time for that additional logging? ), and MP still miminizes when I play a video that triggers a RR change.
Attached logs contain both.

 

[Scythe42]

@infinite.loop: I'll need to add some more debug code to find out where the delay is coming from and why the RR change still steals focus. Do you use stock RR changer or do you call any external programs in addition?