<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">2018-05-15 15:56 GMT+03:00 Pavel Krivanek <span dir="ltr"><<a href="mailto:pavel.krivanek@gmail.com" target="_blank">pavel.krivanek@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br><div dir="ltr">I'm not sure if it a cause of all leaks but I realized that the method  ClyDataSourceUpdateScheduler>><wbr>#processQueue that looks like this:<div><br></div><div><div>processQueue</div><div><br></div><div><span style="white-space:pre-wrap">        </span>| available next |</div><div><span style="white-space:pre-wrap">       </span>[</div><div><span style="white-space:pre-wrap">                </span>available := OrderedCollection with: updateQueue next.</div><div><span style="white-space:pre-wrap">           </span>[ next := updateQueue nextOrNil ] whileNotNil: [ available add: next ].</div><div><span style="white-space:pre-wrap">          </span>self runUpdateFor: available.<span style="white-space:pre-wrap">   </span></div><div><span style="white-space:pre-wrap"> </span>] repeat</div></div><div><br></div><div>keeps always an indirect pointer to the full Calypso browser window because of the escaping variable (next) of the closure. Moving the temporaries definition to the inner closure fixes the leak.</div></div></blockquote><div><br></div><div>Interesting.<br></div><div>But I only see that available variable will keep instances from previous iteration, up to next update event.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>-- Pavel</div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-05-09 21:44 GMT+02:00 Denis Kudriashov <span dir="ltr"><<a href="mailto:dionisiydk@gmail.com" target="_blank">dionisiydk@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2018-05-09 22:18 GMT+03:00 Stephan Eggermont <span dir="ltr"><<a href="mailto:stephan@stack.nl" target="_blank">stephan@stack.nl</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br>
Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>> wrote:<br>
<br>
> Can you tell me whether this is an application bug or possibly a bug with<br>
> the Spur VMs weak collections?  I'd hate for this to be an undiagnosed and<br>
> unfixed VM bug.<br>
<br>
Much more likely an application bug. Copying Morphs that contain Rubric<br>
editors is non-trivial. <br></blockquote><div><br></div><div>Yes. I am sure it can't be VM issue. </div><div>My candidate for leak is spotter windows history. I remember I detect some leaks when it was integrated in Pharo7. But I was not sure, it was very fast test. And it can be wrong.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<span class="gmail-m_-9016431158907465416m_1092993276203220523HOEnZb"><font color="#888888"><br>
Stephan<br>
<br>
</font></span></blockquote></div><br></div></div>
<br></blockquote></div><br></div>
<br></blockquote></div><br></div></div>