benhamilton marked 3 inline comments as done.
benhamilton added inline comments.
================
Comment at: unittests/Format/FormatTestObjC.cpp:527
+ // Make sure selectors with 0, 1, or more arguments are not indented
+ // when IndentWrappedFunctionNames is false.
+ verifyFormat("- (aaaaaaaaaaaaaaaaaaaaaaaaaaaaa)\n"
----------------
jolesiak wrote:
> jolesiak wrote:
> > jolesiak wrote:
> > > I know that `Style.IndentWrappedFunctionNames` is false by default, but
> > > how about adding
> > > ```
> > > Style.IndentWrappedFunctionNames = false;
> > > ```
> > > before these `verifyFormat` calls? I feel like that makes it more
> > > readable and makes test independent on this `Style` constant.
> > This comment is kind of irrelevant as this change is included in D45005.
> My bad, this change is NOT included in D45005. (I don't know how to remove
> inline comment)
Sure, done.
Repository:
rC Clang
https://reviews.llvm.org/D44996
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
