PRs are like branches, you can keep changing/adding things until it is merged or dropped.<br><br><div>:)</div><br><div class="gmail_quote">On Fri Nov 07 2014 at 07:11:03 Bent Bisballe Nyeng <<a href="mailto:deva@aasimon.org">deva@aasimon.org</a>> 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<br>
once I get it done or should I open up a new one?<br>
<br>
Kind regards<br>
Bent Bisballe Nyeng<br>
<br>
On 11/06/14 17:35, Rodrigo Kumpera wrote:<br>
> 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>
> <mailto:<a href="mailto:deva@aasimon.org" target="_blank">deva@aasimon.org</a>>> wrote:<br>
><br>
>     I have made pull request now:<br>
>     <a href="https://github.com/mono/mono/__pull/1391" target="_blank">https://github.com/mono/mono/_<u></u>_pull/1391</a><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>
>         (<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<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>
>           > 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>><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>
>           > <a href="https://gist.github.com/__aasimon/c8ae6fc3cf5d9b82b6ca" target="_blank">https://gist.github.com/__<u></u>aasimon/c8ae6fc3cf5d9b82b6ca</a><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>
>           > ______________________________<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>
<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></div>