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
Development
Submit: code patches (MediaPortal/TV-Server/etc.)
MP1-4502 - TVServer ChannelState.DoSetChannelStates: An unknown error occured setting channel states
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="Sebastiii" data-source="post: 1084693" data-attributes="member: 60104"><p>Hi, </p><p></p><p>[USER=98245]@SiggiK[/USER] :</p><p>There is a comment about the patch from others dev: </p><p></p><p>It would be better to either create one thread that idles all the time or use thread pool <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite7" alt=":p" title="Stick Out Tongue :p" loading="lazy" data-shortname=":p" /></p><p>Because creating constantly new threads is not an ideal way to handle resources (although it might be even that the current code is using that create & destroy a thread approach instead of using "thread pool, single thread or .NET task" approach</p><p>Not sure about (wakeupEvent.WaitOne) it can be nasty </p><p>So can you try to make another patch with only one worker thread that waits for two things:</p><p>1) Event that some new work has been added to the queue</p><p>2) Command to kill the thread</p><p></p><p>When i have copy your patch to the branch, it was needed to change/correct tabs <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite7" alt=":p" title="Stick Out Tongue :p" loading="lazy" data-shortname=":p" />, MP use space and 2 instead of 4 <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile :)" loading="lazy" data-shortname=":)" /> but if you can provide another version of your patch it could be nice/great.</p><p>Thanks <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile :)" loading="lazy" data-shortname=":)" /></p></blockquote><p></p>
[QUOTE="Sebastiii, post: 1084693, member: 60104"] Hi, [USER=98245]@SiggiK[/USER] : There is a comment about the patch from others dev: It would be better to either create one thread that idles all the time or use thread pool :P Because creating constantly new threads is not an ideal way to handle resources (although it might be even that the current code is using that create & destroy a thread approach instead of using "thread pool, single thread or .NET task" approach Not sure about (wakeupEvent.WaitOne) it can be nasty So can you try to make another patch with only one worker thread that waits for two things: 1) Event that some new work has been added to the queue 2) Command to kill the thread When i have copy your patch to the branch, it was needed to change/correct tabs :P, MP use space and 2 instead of 4 :) but if you can provide another version of your patch it could be nice/great. Thanks :) [/QUOTE]
Insert quotes…
Verification
Post reply
Forums
MediaPortal 1
Development
Submit: code patches (MediaPortal/TV-Server/etc.)
MP1-4502 - TVServer ChannelState.DoSetChannelStates: An unknown error occured setting channel states
Contact us
RSS
Top
Bottom