<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif"><span style="font-family:arial,sans-serif;font-size:12.8px">UnixSignals::NeedsUpdating() method suggests that there is always at most one observer of changes in UnixSignals, the guys who clear the flag. Even though it might be the case now, it still feels like a bomb waiting to explode when another observer starts calling </span><span style="font-size:12.8px;font-family:arial,sans-serif">UnixSignals::</span><span style="font-family:arial,sans-serif;font-size:12.8px">ResetNeedsUpdating(). I wouldn't want to introduce this limitation unless I absolutely have to.</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 6, 2017 at 5:09 PM, Jim Ingham <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I like it in the base class and Greg was okay with that too.  So let's leave that where it is.<br>
<br>
The only bit of this behavior that could be moved into UnixSignals as it seemed to me was the handling of "needs updating".  I was mostly proposing that because then you can already pass the UnixSignals into the communication client to test that the signals get suppressed, so if the "needs updating" was in UnixSignals you could test this part in the same way.<br>
<br>
So I'm not pushing hard on this change.<br>
<span class="HOEnZb"><font color="#888888"><br>
Jim<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
> On Mar 6, 2017, at 5:09 PM, Eugene Zemtsov via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br>
><br>
> eugene added a comment.<br>
><br>
> UnixSignals is a nice self contained class that already does 99% of the work (see UnixSignals::<wbr>GetFilteredSignals). I don't think we should have it call anybody.<br>
> Only process knows when it is the right time to send actual QPassSignals packet, there is not need to somehow push this knowledge into UnixSignals.<br>
><br>
> Let's just decide if UpdateAutomaticSignalFiltering<wbr>() is specific enough to live in GDBProcess or it's generic enough to live in the base process. I'm fine either way.<br>
><br>
><br>
> <a href="https://reviews.llvm.org/D30520" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D30520</a><br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><font face="verdana, sans-serif">Thanks,</font><div><font face="verdana, sans-serif">Eugene Zemtsov.</font></div></div></div>
</div>