<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">ср, 25 янв. 2023 г., 11:56 Stefan de Konink <<a href="mailto:stefan@konink.de">stefan@konink.de</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wednesday, January 25, 2023 9:51:46 AM CET, Andrew Randrianasulu wrote:<br>
> It does not fix any bug I know of, so probably safest way to <br>
> leave code as it was before me (because threads!) and probably <br>
> revisit if some tool show issue or user report it.<br>
<br>
Developing software is not equal to copy-pasting. I would like to see an <br>
explanation why someone would not want to unlock, while that happens in any <br>
other case.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Just for making things more clear: *I* added this unlock, not because I knew what I was doing but simply because yeah, it looked like this place might need it. </div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">In theory oicks protects shared resources, so one thread may leave object/variable around without worry another thread scheduled by kernel will overwrite it.</div><div dir="auto"><br></div><div dir="auto">But I do not know enough of details for carelessly altering locking like I did. So, Phyllis removed this line on my request, because, I reiterate, I put it there literally on a whim.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
Stefan<br>
</blockquote></div></div></div>