<div dir="ltr">For the record I like this as a first start. Thanks for the work Duncan.<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 9, 2014 at 3:04 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch addresses the first stage of PR17891 by folding constant<br>
arguments together into a single MDString.  Integers are stringified and<br>
a `\0` character is used as a separator.<br>
<br>
----<br>
<br>
Known defects:<br>
<br>
  - I've only barely started to update testcases.<br>
  - There are bound to be bugs ~~~~~~~~~~~~^<br>
  - I deleted dead code (I'll separate that out before committing).<br>
<br>
I'd really appreciate some early feedback on this nevertheless.  I'm<br>
finding the testcases painful enough to update that I don't want to do it<br>
twice.<br>
<br>
 1. I think `\0` is cleaner than `,`.  Bike shed?<br>
<br>
 2. I'm using hex for the tags.  Good idea?<br>
<br>
 3. I didn't add a prefix, which David suggested just now in PR17891.<br>
    This would be the time to do it, but is the hex tag sufficient?<br>
<br>
 4. Anything (other than testcases) I've missed updating?<br>
<br>
 5. Do you have specific advice on automating the testcase update flow?<br>
    My next step is building tool support, so suggestions (and starter<br>
    scripts) are welcome.<br>
<br>
</blockquote></div><br></div>