Reply to thread

Message

<blockquote data-quote="iosub" data-source="post: 268595" data-attributes="member: 20095">Yes I have seen that running mp in debug, it does a lot of &quot;unecesary&quot; things I think.For example on this post <a href="https://forum.team-mediaportal.com/1-0-rc1-svn-builds-270/unsuported-fix-18882-fixed-mantis-change-ts-detects-if-open-myrecordings-39372/#post263826" target="_blank">https://forum.team-mediaportal.com/1-0-rc1-svn-builds-270/unsuported-fix-18882-fixed-mantis-change-ts-detects-if-open-myrecordings-39372/#post263826</a>I have seen that they check if a file is stream or video instead of using the properties that the class has already to know what type of file is going to play.. check the post and I have patch it using &quot;if (this._mediaType == g_Player.MediaType.Video)&quot;, that is my opinion should be the way to do.I think MP has a good base arquitecture, the only problem is that sometimes the devs, don't use all the potencial of MP arquitecture and they do a lot of extra work, instead of using what already has mp .So, MP can be more responsible and faster, and that's the way to do I think.4 Years in MS teach me that <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" /> don't ask me in what proyect I can't say I have a (NDA) So is nice that the fix is working.. Now I will try to do more clean ups a send to touretes or gemx.. lets see if the decide to introduce the changes.IosuBTW I just add couple application.doevents.. so I let time to other process so vista thinks that MP has not stop.. Vista is so &quot;stupid&quot; thinking that if a process does not response&nbsp; for more that a few seconds it has stoped.. you can see the same with many other process like &quot;outlook&quot; iexplorer etc... I know that MS is going to change that, internaly it has already changed.. &quot;Hope I dont break a NDA with that&quot; <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" />&nbsp; but&nbsp; the fix is not going to be on vista...</blockquote>

[QUOTE="iosub, post: 268595, member: 20095"] Yes I have seen that running mp in debug, it does a lot of "unecesary" things I think. For example on this post [url]https://forum.team-mediaportal.com/1-0-rc1-svn-builds-270/unsuported-fix-18882-fixed-mantis-change-ts-detects-if-open-myrecordings-39372/#post263826[/url] I have seen that they check if a file is stream or video instead of using the properties that the class has already to know what type of file is going to play.. check the post and I have patch it using "if (this._mediaType == g_Player.MediaType.Video)", that is my opinion should be the way to do. I think MP has a good base arquitecture, the only problem is that sometimes the devs, don't use all the potencial of MP arquitecture and they do a lot of extra work, instead of using what already has mp . So, MP can be more responsible and faster, and that's the way to do I think. 4 Years in MS teach me that :-) don't ask me in what proyect I can't say I have a (NDA) So is nice that the fix is working.. Now I will try to do more clean ups a send to touretes or gemx.. lets see if the decide to introduce the changes. Iosu BTW I just add couple application.doevents.. so I let time to other process so vista thinks that MP has not stop.. Vista is so "stupid" thinking that if a process does not response for more that a few seconds it has stoped.. you can see the same with many other process like "outlook" iexplorer etc... I know that MS is going to change that, internaly it has already changed.. "Hope I dont break a NDA with that" :-) but the fix is not going to be on vista... [/QUOTE]

Verification