<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi Oleg,</p>
<p>could you please send that patch to `reviews.llvm.org`. That
makes it easier to review and discuss on the code.</p>
<p>Best, Jonas<br>
</p>
<br>
<div class="moz-cite-prefix">Am 10.09.2018 um 22:47 schrieb Oleg
Smolsky via cfe-dev:<br>
</div>
<blockquote type="cite"
cite="mid:3bd0edcc-acef-6aa2-21a4-c3a808e6ef71@cohesity.com">OK,
the functionality is a special case introduced in 2014 here:
8228889b01404d7e59270b1f97a83977531a7748.
<br>
<br>
The minimal hack is to check for the preceding comma... but that
breaks some "literal" cases... So, I need to make these selections
even more particular (to either exclude literals or only include
lambdas). Does this sound right?
<br>
<br>
On 2018-09-10 12:47, Oleg Smolsky wrote:
<br>
<blockquote type="cite">Hi, I've just boiled down an interesting
C++ lambda formatting trait and would like to clarify the tool's
behavior. Consider the following snippet (please view with a
fixed-width font):
<br>
<br>
void f() {
<br>
something->One(
<br>
[this] {
<br>
Do1();
<br>
Do2();
<br>
},
<br>
1);
<br>
something->Two(1,
<br>
[this] {
<br>
Do1();
<br>
Do2();
<br>
},
<br>
1);
<br>
}
<br>
<br>
There is an inconsistency in the way lambda args are formatted,
depending on whether it is first (the "One()" call above) or not
(the "Two()" call above). Is there some internal guide that the
tool uses to decide between the two layouts? Or is it just an
artifact of the implementation?
<br>
<br>
More generally, would you entertain a patch that forces the
format one way or another? (Perhaps even with a user-defined
setting?)
<br>
<br>
Thanks,
<br>
Oleg.
<br>
<br>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
cfe-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>
<a class="moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>