<div dir="ltr">For the yaml2elf stuff, can you fix it by passing an ArrayRef? (it has an implicit conversion from std::vector) I think that would be cleaner. (also update vectorDataSize to use ArrayRef and rename the two functions to writeArrayData and arrayDataSize).<div>
<br></div><div>-- Sean Silva</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 27, 2013 at 5:00 PM, Will Dietz <span dir="ltr"><<a href="mailto:wdietz2@uiuc.edu" target="_blank">wdietz2@uiuc.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The ambiguity I'm encountering is between:<br>
<br>
std::string::append(size_t, char);<br>
template <class InputIterator> std::string::append(InputIterator<br>
first, InputIterator last) [with InputIterator=int]<br>
<br>
This patch makes it clear the '0' is a char and therefore should be<br>
using the first version.<br>
<br>
Does that make sense/sound good? :)<br>
<span class="HOEnZb"><font color="#888888"><br>
~Will<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Fri, Sep 27, 2013 at 2:56 PM, Rafael Espíndola<br>
<<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> LGTM.<br>
><br>
> Just out of curiosity, what is the ambiguity that you are getting with int,int?<br>
><br>
> On 26 September 2013 18:01, Will Dietz <<a href="mailto:wdietz2@uiuc.edu">wdietz2@uiuc.edu</a>> wrote:<br>
>> See attached.<br>
>><br>
>> Thanks!<br>
>><br>
>> ~Will<br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>