<div dir="ltr"><div>Hi all,</div><div><br></div><div>I think Tim is correct and this is an old artifact from the previous and semaphore-based event mechanism.</div><div><br></div><div>Does anyone object to merging this into trunk (with the class comment fixed of course)? It's a simple cleanup, not a new feature.</div><div><br></div><div>Fabio</div><div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 30, 2020 at 2:31 AM tim Rowledge <<a href="mailto:tim@rowledge.org">tim@rowledge.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">As best I can recall we used to signal the input sem from the vm when a key event came in. We also used to signal a specific interrupt key semaphore, which had the great virtue of really making sure the image got the message that you wanted it to stop chewing gum and pay attention *right now*young fella.<br>
<br>
> On 2020-01-29, at 5:00 PM, David T. Lewis <<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>> wrote:<br>
> <br>
> On Wed, Jan 29, 2020 at 05:47:15PM +0000, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
>> A new version of Kernel was added to project The Inbox:<br>
>> <a href="http://source.squeak.org/inbox/Kernel-fn.1294.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox/Kernel-fn.1294.mcz</a><br>
>> <br>
>> ==================== Summary ====================<br>
>> <br>
>> Name: Kernel-fn.1294<br>
>> Author: fn<br>
>> Time: 29 January 2020, 6:47:12.028071 pm<br>
>> UUID: 0b61cdfe-41e2-4aee-a5f0-80d8e7d45127<br>
>> Ancestors: Kernel-tonyg.1293<br>
>> <br>
>> Remove inputSemaphore and hasInputSemaphore from EventSensor (both are no longer in use). Also drop EventSensor>>primSetInputSemaphore:.<br>
>> <br>
> <br>
> Is it actually the case that inputSemaphore and hasInputSemaphore are<br>
> no longer in use? They appear to be used now in exactly the same way<br>
> they were used in Squeak 3.8. But in Squeak 3.6, the inputSemaphore was<br>
> waited on by an ioProcess, so it looks like it served a useful purpose<br>
> back then.<br>
> <br>
> So yes, this actually does appear to be dead code that has been waiting<br>
> all these years for cleanup :-)<br>
> <br>
> Dave<br>
> <br>
> <br>
> <br>
<br>
<br>
tim<br>
--<br>
tim Rowledge; <a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
"Bother," said Pooh, reading his bank statement from Barings.<br>
<br>
<br>
<br>
</blockquote></div></div>