I agree with that, but did not make the change because not sure why there are two such types.<span></span><br><br>On Wednesday, May 1, 2013, Aaron Ballman  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I noticed that this interface uses size_t and size_type both; should<br>
we make it more consistent by using size_type in place of size_t?<br>
<br>
~Aaron<br>
<br>
On Wed, May 1, 2013 at 10:18 AM, Peng Cheng <<a href="javascript:;" onclick="_e(event, 'cvml', 'gm4cheng@gmail.com')">gm4cheng@gmail.com</a>> wrote:<br>
> Author: chp<br>
> Date: Wed May  1 09:18:06 2013<br>
> New Revision: 180847<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=180847&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=180847&view=rev</a><br>
> Log:<br>
> fix windows warning.<br>
><br>
> warning C4244: 'argument' : conversion from 'uint64_t' to 'const unsigned int', possible loss of data<br>
><br>
> Modified:<br>
>     llvm/trunk/include/llvm/ADT/StringRef.h<br>
><br>
> Modified: llvm/trunk/include/llvm/ADT/StringRef.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/StringRef.h?rev=180847&r1=180846&r2=180847&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/StringRef.h?rev=180847&r1=180846&r2=180847&view=diff</a><br>

> ==============================================================================<br>
> --- llvm/trunk/include/llvm/ADT/StringRef.h (original)<br>
> +++ llvm/trunk/include/llvm/ADT/StringRef.h Wed May  1 09:18:06 2013<br>
> @@ -390,14 +390,14 @@ namespace llvm {<br>
><br>
>      /// Return a StringRef equal to 'this' but with the first \p N elements<br>
>      /// dropped.<br>
> -    StringRef drop_front(unsigned N = 1) const {<br>
> +    StringRef drop_front(size_t N = 1) const {<br>
>        assert(size() >= N && "Dropping more elements than exist");<br>
>        return substr(N);<br>
>      }<br>
><br>
>      /// Return a StringRef equal to 'this' but with the last \p N elements<br>
>      /// dropped.<br>
> -    StringRef drop_back(unsigned N = 1) const {<br>
> +    StringRef drop_back(size_t N = 1) const {<br>
>        assert(size() >= N && "Dropping more elements than exist");<br>
>        return substr(0, size()-N);<br>
>      }<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="javascript:;" onclick="_e(event, 'cvml', '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>
</blockquote>