<div dir="ltr">On Tue, Aug 13, 2013 at 1:48 AM, Klemens Baum <span dir="ltr"><<a href="mailto:klemensbaum@gmail.com" target="_blank">klemensbaum@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  For the Phrabricator issue: I tried uploading a full-context diff [here](<a href="http://llvm-reviews.chandlerc.com/differential/diff/3416/" target="_blank">http://llvm-reviews.chandlerc.com/differential/diff/3416/</a>). If you expand FormatTest.cpp, you'll see that it gets truncated at line 5782.<br>
</blockquote><div><br></div><div>Can you give me the command line you used to generate the patch, then I'll file an upstream bug. Thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
<br>
================<br>
Comment at: ../tools/clang/lib/Format/Format.cpp:656<br>
@@ +655,3 @@<br>
+      } else if (Current.Type == TT_CtorInitializerComma) {<br>
+        assert(Style.BreakConstructorInitializersBeforeComma);<br>
+        State.Column = State.Stack.back().Indent;<br>
----------------<br>
</div><div class="im">Daniel Jasper wrote:<br>
> Klemens Baum wrote:<br>
> > Daniel Jasper wrote:<br>
> > > Do not assert!!! This is easy to force by user input, e.g. by:<br>
> > ><br>
> > >   Constructor::Constructor() : a(a) // a<br>
> > >   , b(b) {}<br>
> > ><br>
> > > It's fine not to format perfectly in that case, but we shouldn't crash.<br>
> > Since this whole block is within the `if (Newline)`, `Current` refers to the first token to emit on that line. Unless we are using the BreakConstructorInitializersBeforeComma style, it cannot be an initializer list comma token. So I don't think this can be triggered by user input.<br>

> Well, I have given you the user input that triggers it.<br>
</div>Oh, right. I didn't realize that comment was there.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1360" target="_blank">http://llvm-reviews.chandlerc.com/D1360</a><br>
</blockquote></div><br></div></div>