<div dir="ltr"><div dir="ltr">On Thu, Jun 24, 2021 at 11:23 AM John McCall <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 24 Jun 2021, at 13:35, David Blaikie wrote:<br>
> On Wed, Jun 23, 2021 at 2:57 PM John McCall <<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>> wrote:<br>
>> Ah, no, I meant you, but I wrote it out completely wrong: I meant<br>
>> that it feels like the people using NFC this broadly almost mean<br>
>> that a patch *not* being NFC makes it especially questionable<br>
>> and then are looking for nearly any reason to mark it NFC.<br>
>><br>
> That's not so much my feeling, I think - NFC versus non-NFC for me is<br>
> mostly an indicator of how I should review the patch. Usually hinging on<br>
> "does this have tests/is this testable".<br>
<br>
Okay.  Well, we use NFC differently, but because I use it more strictly,<br>
at least I won’t bother you.<br></blockquote><div><br>I mean, a little - If I end up reading the patch, seeing no tests & wondering why it's not tested & have to understand more about the patch to see it's changing something that doesn't need test coverage.<br><br>But not the biggest deal, no.<br><br>- Dave </div></div></div>