<div dir="ltr">PR 1391 cannot be accepted for multiple reasons:<div><br></div><div>-The commit message is empty and has no licensing;</div><div>-It changes MonoInternalThread layout for no good reason, the introduced field is not used anywhere;</div><div>-It's adding code to the io-layer for which we don't want to continue to expand;</div><div><br></div><div>How to do it in way that's mergeable?</div><div><br></div><div>Add a pair of functions to utils/mono-threads.h that get/set the priority.</div><div>Implement that functions on all backends or add stubs when appropriate.</div><div>Use this new API from metadata/threads.h to implement the managed functionality;</div><div>Use C89.</div><div>License the code under the MIT license.</div><div><br></div><div>--</div><div>Rodrigo</div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 6, 2014 at 9:16 AM, Bent Bisballe Nyeng <span dir="ltr"><<a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I have made pull request now: <a href="https://github.com/mono/mono/pull/1391" target="_blank">https://github.com/mono/mono/<u></u>pull/1391</a><br>
<br>
I have updated the patch to work with HEAD and tested it. Everything seems to work as expected.<br>
<br>
I'm a bit new to the whole github concept, so forgive me if I have not done everything by the book ;-)<span class="im HOEnZb"><br>
<br>
Kind regards<br>
Bent Bisballe Nyeng<br>
<br></span><div class="HOEnZb"><div class="h5">
On 11/06/14 12:37, Alexander Köplinger wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There is a PR that also claims to implement SetThreadPriority<br>
(<a href="https://github.com/mono/mono/pull/1272" target="_blank">https://github.com/mono/mono/<u></u>pull/1272</a>), but it has many other<br>
unrelated changes, so not in a state to be merged.<br>
 From a quick look, your patch seems to be much more focused and thus<br>
more likely to get merged. Can you open a pull request on GitHub?<br>
<br>
-- Alex<br>
<br>
<br>
 > Date: Thu, 6 Nov 2014 09:12:02 +0100<br>
 > From: <a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a><br>
 > To: <a href="mailto:mono-devel-list@lists.ximian.com" target="_blank">mono-devel-list@lists.ximian.<u></u>com</a><br>
 > Subject: [Mono-dev] SetThreadPriority patch for mono-3.2.8<br>
 ><br>
 > Hi mono devs<br>
 ><br>
 > I created a patch for SetThreadPriority support in mono-3.2.8 and would<br>
 > very much like som feedback on it.<br>
 > It can be found here:<br>
 > <a href="https://gist.github.com/aasimon/c8ae6fc3cf5d9b82b6ca" target="_blank">https://gist.github.com/<u></u>aasimon/c8ae6fc3cf5d9b82b6ca</a><br>
 > Comments are welcome both here on the list as well as on the actual gist<br>
 > paste.<br>
 ><br>
 > Kind regards<br>
 > Bent Bisballe Nyeng<br>
 > ______________________________<u></u>_________________<br>
 > Mono-devel-list mailing list<br>
 > <a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.<u></u>com</a><br>
 > <a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/<u></u>mailman/listinfo/mono-devel-<u></u>list</a><br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.<u></u>com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/<u></u>mailman/listinfo/mono-devel-<u></u>list</a><br>
</div></div></blockquote></div><br></div>