<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, May 19, 2016 at 1:11 PM Justin Bogner via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">James Y Knight via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> writes:<br>
> GAH! Sigh.... :(<br>
><br>
> That is indeed what it does. See stripQuotedText<br>
> in src/applications/metamta/parser/PhabricatorMetaMTAEmailBodyParser.php if<br>
> anyone's interested in hacking on php code, it looks like it might be<br>
> fairly straightforward to have it strip only the lines starting with ">"<br>
> after the "On ... wrote:" line.<br>
<br>
Wouldn't that be confusing? Such comments tend to refer to the quoted<br>
text just before the comment. We'd probably want to keep some arbitrary<br>
number of the quoted lines before each comment or something.<br>
<br>
Note also that if we fix that we should probably also teach phab not to<br>
re-send comments to the list when the list and phab were both on cc.<br>
AFAICT any top posted comments on phab reviews show up on the list twice<br>
currently, once from the author and once from phab.<br></blockquote><div><br></div><div>Maybe file both of these in the Phab bug component?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
> On Thu, May 19, 2016 at 3:21 PM, Tim Northover <<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>><br>
> wrote:<br>
><br>
>> On 19 May 2016 at 11:59, James Y Knight via llvm-dev<br>
>> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>> > Does anyone know why phab sometimes misses replies sent by email? Usually<br>
>> > they make it through, but sometimes not.<br>
>><br>
>> I think I've noticed it dropping e-mails that start with quotes<br>
>> before, and certainly ignoring everything below the first quote line.<br>
>> It seems to assume everyone will be top-posting.<br>
>><br>
>> Tim.<br>
>><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>