aaron.ballman added a comment.

In https://reviews.llvm.org/D43581#1014903, @stephanemoore wrote:

> In https://reviews.llvm.org/D43581#1014664, @aaron.ballman wrote:
>
> > LGTM with a few additional test cases.
> >
> > It'd be nice if the style guide linked actually described what a "good" 
> > prefix is rather than make the reader guess.
>
>
> I will work on updating the Google Objective-C style guide to provide more 
> clarity regarding what constitutes an appropriate prefix.


Fantastic, thank you! This patch LGTM.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43581



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

Reply via email to