[PATCH] D113826: [clang-format][c++2b] support removal of the space between auto and {} in P0849R8

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 13 11:23:40 PST 2021


HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.

Looks good. :)

In D113826#3129229 <https://reviews.llvm.org/D113826#3129229>, @Quuxplusone wrote:

> Personally I'd remove the tests that are invalid expressions; but I don't know what's the usual practice. Ship it AFAIC!

It's nice if even invalid code gets formatted, because some of us have formatting on typing, and the then invalid code may become valid very quickly and it would be nice if the code does not jump around. :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113826/new/

https://reviews.llvm.org/D113826



More information about the cfe-commits mailing list