Single-variant enums were still getting placed on a single line even
when AllowShortEnumsOnASingleLine was false. This fixes that by checking
that setting when looking to merge lines.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
clang/lib/Format/UnwrappedLineFormatter.cpp | ||
---|---|---|
398–403 | Nits:
| |
406 |
Comment Actions
Addressed reviewer comments
- Removed unnecessary null checks
- Added tests for typedefs preceding short enums
Comment Actions
It had looked like arcanist was retaining this information, but I now realize what it was doing.
At any rate, I would appreciate it if someone could merge this for me as I don't have commit access.
Gabriel Smith <ga29smith@gmail.com>
Nits: