<div dir="ltr">Up to you to use a new PR or update & reopen the existing one.<div><br></div><div>Just remember to ping me either here on or github to keep me honest and the review cycle short.</div><div><br></div><div>Thanks a lot for looking into this.</div><div><br></div><div>--</div><div>Rodrigo</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 7, 2014 at 4:10 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'll look into the issues you describe. Should I have the PR reopened once I get it done or should I open up a new one?<span class=""><br>
<br>
Kind regards<br>
Bent Bisballe Nyeng<br>
<br></span><span class="">
On 11/06/14 17:35, Rodrigo Kumpera wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
PR 1391 cannot be accepted for multiple reasons:<br>
<br>
-The commit message is empty and has no licensing;<br>
-It changes MonoInternalThread layout for no good reason, the introduced<br>
field is not used anywhere;<br>
-It's adding code to the io-layer for which we don't want to continue to<br>
expand;<br>
<br>
How to do it in way that's mergeable?<br>
<br>
Add a pair of functions to utils/mono-threads.h that get/set the priority.<br>
Implement that functions on all backends or add stubs when appropriate.<br>
Use this new API from metadata/threads.h to implement the managed<br>
functionality;<br>
Use C89.<br>
License the code under the MIT license.<br>
<br>
--<br>
Rodrigo<br>
<br>
<br>
<br>
<br>
<br>
On Thu, Nov 6, 2014 at 9:16 AM, Bent Bisballe Nyeng <<a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a><br></span><span class="">
<mailto:<a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a>>> wrote:<br>
<br>
    I have made pull request now:<br></span>
    <a href="https://github.com/mono/mono/__pull/1391" target="_blank">https://github.com/mono/mono/_<u></u>_pull/1391</a><span class=""><br>
    <<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<br>
    seems to work as expected.<br>
<br>
    I'm a bit new to the whole github concept, so forgive me if I have<br>
    not done everything by the book ;-)<br>
<br>
    Kind regards<br>
    Bent Bisballe Nyeng<br>
<br>
    On 11/06/14 12:37, Alexander Köplinger wrote:<br>
<br>
        There is a PR that also claims to implement SetThreadPriority<br></span>
        (<a href="https://github.com/mono/mono/__pull/1272" target="_blank">https://github.com/mono/mono/<u></u>__pull/1272</a><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<span class=""><br>
        unrelated changes, so not in a state to be merged.<br>
          From a quick look, your patch seems to be much more focused<br>
        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></span>
          > From: <a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a> <mailto:<a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a>><br>
          > To: mono-devel-list@lists.ximian._<u></u>_com<br>
        <mailto:<a href="mailto:mono-devel-list@lists.ximian.com" target="_blank">mono-devel-list@lists.<u></u>ximian.com</a>><span class=""><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<br>
        mono-3.2.8 and would<br>
          > very much like som feedback on it.<br>
          > It can be found here:<br></span>
          > <a href="https://gist.github.com/__aasimon/c8ae6fc3cf5d9b82b6ca" target="_blank">https://gist.github.com/__<u></u>aasimon/c8ae6fc3cf5d9b82b6ca</a><span class=""><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<br>
        actual gist<br>
          > paste.<br>
          ><br>
          > Kind regards<br>
          > Bent Bisballe Nyeng<br></span>
          > ______________________________<u></u>___________________<br>
          > Mono-devel-list mailing list<br>
          > Mono-devel-list@lists.ximian._<u></u>_com<br>
        <mailto:<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.<u></u>ximian.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>
        <<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>
<br>
<br>
    ______________________________<u></u>___________________<br>
    Mono-devel-list mailing list<br>
    Mono-devel-list@lists.ximian._<u></u>_com<br>
    <mailto:<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.<u></u>ximian.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>
    <<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>
<br>
<br>
</blockquote><div class="HOEnZb"><div class="h5">
<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>