[PATCH] Improve formatting of lambda functions.

Daniel Jasper djasper at google.com
Fri Jun 12 00:43:04 PDT 2015


I am not actually sure here. The pro is that it doesn't really belong with the "{". The con is that it looks a lot more like a multiplication. Not so important for "int", but as soon as you have something where you aren't exactly sure that it is a type, this can get confusing.

Also, there is the consistency with "int *f() { return nullptr; }" which we should think about.

All in all, I don't see compelling reasons to do this change yet (but I can be convinced).

All of this just shows that the pointer belongs with the type ;-).


http://reviews.llvm.org/D10410

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list