dexonsmith added a comment.

In https://reviews.llvm.org/D35159#809724, @erik.pilkington wrote:

> Rebase. I don't think the issue of purging underscores from this file/libcxx 
> should block this, if we want to discuss that cfe-dev would probably be the 
> place. I agree that it would be nice to clang-format this file, would anyone 
> have any problems with me committing that after this? I found the git-blame 
> of this file to be pretty useless, almost every line just points to r184097. 
>  Are there any other thoughts on this change?


I suppose you may as well pick a new name for "string_ref" that doesn't have 
underscores, since it's a new type, if someone is going to follow up to drop 
underscores.  Perhaps "StringView"?


https://reviews.llvm.org/D35159



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to