<br><br><div class="gmail_quote">On Mon, Jan 14, 2013 at 8:24 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_default">On Thu, Jan 10, 2013 at 10:34 PM, Philip Craig <span dir="ltr"><<a href="mailto:philipjcraig@gmail.com" target="_blank">philipjcraig@gmail.com</a>></span> wrote:<br></div>
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
  Are the parentheses still useful after doing this? They might be useful in some editors for quick navigation, but otherwise I think they make it a bit cluttered.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D281" target="_blank">http://llvm-reviews.chandlerc.com/D281</a><br>
</blockquote></div><br></div><div class="gmail_extra">So you would prefer it like this?</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_extra">TranslationUnitDecl 0x4bd5660 <<invalid sloc>></div>

<div class="gmail_extra">|-TypedefDecl 0x4bd5ba0 <<invalid sloc>> __int128_t '__int128'</div><div class="gmail_extra">|-TypedefDecl 0x4bd5c00 <<invalid sloc>> __uint128_t 'unsigned __int128'</div>

<div class="gmail_extra">|-TypedefDecl 0x4bd5fc0 <<invalid sloc>> __builtin_va_list '__va_list_tag [1]'</div><div class="gmail_extra">`-FunctionDecl 0x4bd61c0 <simple2.cc:1:1, line:5:1> main 'int (int, char **)'</div>

<div class="gmail_extra">  |-ParmVarDecl 0x4bd6020 <line:1:10, col:14> argc 'int'</div><div class="gmail_extra">  |-ParmVarDecl 0x4bd60f0 <col:20, col:27> argv 'char **'</div><div class="gmail_extra">

  `-CompoundStmt 0x4c028c0 <col:33, line:5:1></div><div class="gmail_extra">    |-DeclStmt 0x4bd62f8 <line:2:3, col:12></div><div class="gmail_extra">    | `-VarDecl 0x4bd6280 <col:3, col:11> x 'int'</div>

<div class="gmail_extra">    |   `-IntegerLiteral 0x4bd62d8 <col:11> 'int' 5</div><div class="gmail_extra">    |-DeclStmt 0x4c027c0 <line:3:3, col:12></div><div class="gmail_extra">    | `-VarDecl 0x4bd6320 <col:3, col:11> y 'int'</div>

<div class="gmail_extra">    |   `-IntegerLiteral 0x4c027a0 <col:11> 'int' 2</div><div class="gmail_extra">    `-ReturnStmt 0x4c028a0 <line:4:3, col:16></div><div class="gmail_extra">      `-ParenExpr 0x4c02880 <col:10, col:16> 'int'</div>

<div class="gmail_extra">        `-BinaryOperator 0x4c02858 <col:11, col:15> 'int' '*'</div><div class="gmail_extra">          |-ImplicitCastExpr 0x4c02828 <col:11> 'int' <LValueToRValue></div>

<div class="gmail_extra">          | `-DeclRefExpr 0x4c027d8 <col:11> 'int' lvalue Var 0x4bd6280 'x' 'int'</div><div class="gmail_extra">          `-ImplicitCastExpr 0x4c02840 <col:15> 'int' <LValueToRValue></div>

<div class="gmail_extra">            `-DeclRefExpr 0x4c02800 <col:15> 'int' lvalue Var 0x4bd6320 'y' 'int'</div></div></div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div>I would!<br><br>Though an extraneous blank between the dash and the name of the node might help readability too.<br><br>-- Matthieu<br>