<div dir="ltr"><p dir="ltr" class="gmail_msg">It's by design. Do we want to change this? I always had the impression turning on the state changes for the list is a bit noisy, but I'm happy if that's not the general sentiment.</p>
<br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Mon, Jan 16, 2017, 11:07 PM Benjamin Kramer <<a href="mailto:benny.kra@gmail.com" class="gmail_msg" target="_blank">benny.kra@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I got an email where it says that I accepted the revision. Looks like<br class="gmail_msg">
phab didn't add cfe-commits to the list of recipients though :(<br class="gmail_msg">
<br class="gmail_msg">
On Mon, Jan 16, 2017 at 6:43 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="gmail_msg" target="_blank">dblaikie@gmail.com</a>> wrote:<br class="gmail_msg">
> Looks like Ben signed off on this on Phab - but the email didn't go to the<br class="gmail_msg">
> list (making this look like code was sent for review, then committed,<br class="gmail_msg">
> without review/approval happening)<br class="gmail_msg">
><br class="gmail_msg">
> Ben: I think Phab doesn't send mail for an approval with no text, so at<br class="gmail_msg">
> least as a workaround you can write something in the comments section when<br class="gmail_msg">
> approving (people often write "LGTM" or similar) to ensure the approval is<br class="gmail_msg">
> reflected on the mailing list.<br class="gmail_msg">
><br class="gmail_msg">
> On Wed, Jan 11, 2017 at 1:59 AM Manuel Klimek via Phabricator via<br class="gmail_msg">
> cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br class="gmail_msg">
>><br class="gmail_msg">
>> klimek created this revision.<br class="gmail_msg">
>> klimek added a reviewer: bkramer.<br class="gmail_msg">
>> klimek added a subscriber: cfe-commits.<br class="gmail_msg">
>><br class="gmail_msg">
>> Instead of just using popularity, we also take into account how similar<br class="gmail_msg">
>> the<br class="gmail_msg">
>> path of the current file is to the path of the header.<br class="gmail_msg">
>> Our first approach is to get popularity into a reasonably small scale by<br class="gmail_msg">
>> taking<br class="gmail_msg">
>> log2 (which is roughly intuitive to how humans would bucket popularity),<br class="gmail_msg">
>> and<br class="gmail_msg">
>> multiply that with the number of matching prefix path fragments of the<br class="gmail_msg">
>> included<br class="gmail_msg">
>> header with the current file.<br class="gmail_msg">
>> Note that currently we do not take special care for unclean paths<br class="gmail_msg">
>> containing<br class="gmail_msg">
>> "../" or "./".<br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> <a href="https://reviews.llvm.org/D28548" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D28548</a><br class="gmail_msg">
>><br class="gmail_msg">
>> Files:<br class="gmail_msg">
>>   include-fixer/IncludeFixer.cpp<br class="gmail_msg">
>>   include-fixer/SymbolIndexManager.cpp<br class="gmail_msg">
>>   include-fixer/SymbolIndexManager.h<br class="gmail_msg">
>>   include-fixer/tool/ClangIncludeFixer.cpp<br class="gmail_msg">
>>   test/include-fixer/Inputs/fake_yaml_db.yaml<br class="gmail_msg">
>>   test/include-fixer/ranking.cpp<br class="gmail_msg">
>><br class="gmail_msg">
>> _______________________________________________<br class="gmail_msg">
>> cfe-commits mailing list<br class="gmail_msg">
>> <a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a><br class="gmail_msg">
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="gmail_msg">
</blockquote></div></div>