<div dir="ltr"><div>Great Kuno and Jeff! Thank you for helping with a solution for the issue with _weakrefset.py (reported on 13th Abril). Looking forward to seeing Kuno's changes in the next IronPython release! </div><div><br></div><div>Jeff, is it there a schedule for next IronPython stable release? (...not pushing, just kindly asking if you have an estimation ;-) )</div><div><br></div><div>Thanks, </div><div>Andres </div><div><br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 5, 2015 at 1:39 PM, Jeff Hardy <span dir="ltr"><<a href="mailto:jdhardy@gmail.com" target="_blank">jdhardy@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Kuno,<br>Thanks for looking into it! The usual workflow is to send a pull<br>request (PR) [1], which creates a single spot for discussion, with<br>nice commenting abilities. If you don't mind doing that, I'll take a<br>look at the changes [2] in the meantime.<br><br>- Jeff<br><br>[1] <a href="https://help.github.com/articles/creating-a-pull-request/" target="_blank">https://help.github.com/articles/creating-a-pull-request/</a><br>[2] <a href="https://github.com/kunom/ipy-weakref/commit/3efacde8b3408fa92ddd4185a4774a0c284c4301" target="_blank">https://github.com/kunom/ipy-weakref/commit/3efacde8b3408fa92ddd4185a4774a0c284c4301</a><br><div class=""><div class="h5"><br>On Mon, May 4, 2015 at 12:13 PM, Kuno Meyer <<a href="mailto:kuno.meyer@gmx.ch">kuno.meyer@gmx.ch</a>> wrote:<br>> Dear all<br>><br>> I'd like to propose <a href="https://github.com/kunom/ipy-weakref" target="_blank">https://github.com/kunom/ipy-weakref</a> (the ipy-27-maint<br>> branch) to be included into the main IronPython code base. It fixes for me<br>> various random SystemError and ValueError exceptions in the _weakref.cs module.<br>><br>> Please note that while my patch changes (and hopefully improves) some<br>> aspects in _weakref.cs, I still do think that _weakref.cs has several<br>> critical issues:<br>><br>> - There is absolutely no locking on the internal data structures, which<br>> makes me think that there could be various runtime issues in a multithreaded<br>> environment. (However, this does not directly apply to the Finalizer thread<br>> since to my knowledge all user threads are suspended during finalization.)<br>><br>> - The weakref collection callback is invoked on the .NET finalizer thread. I<br>> am not sure whether this is what user code expects. At least there is a high<br>> potential of dead-locking the finalizer thread, which is not the nicest<br>> thing to do. (Maybe we should issue a warning on direct calls to<br>> GC.WaitForPendingFinalizers(), similar to what is printed when calling<br>> Thread.Sleep().)<br>><br>> Other remarks:<br>><br>> - The weakref/weakrefset test case brought up in the mailing list on Apr 13<br>> passes with my changes.<br>><br>> - The standard library weakref tests pass with the same number of failures<br>> as before<br>><br>> - Please note that I am unexperienced with git/GitHub as well as with the<br>> IronPython code base, so please show some patience...<br>><br>> Thanks!<br>> Kuno<br>><br>> _______________________________________________<br>> Ironpython-users mailing list<br>> <a href="mailto:Ironpython-users@python.org">Ironpython-users@python.org</a><br>> <a href="https://mail.python.org/mailman/listinfo/ironpython-users" target="_blank">https://mail.python.org/mailman/listinfo/ironpython-users</a></div></div></blockquote></div></div></div></div></div>