[PATCH] D117520: [clang-format] Fix SeparateDefinitionBlocks issues

[ksyx via Phabricator via cfe-commits]

ksyx created this revision.
ksyx added reviewers: MyDeveloperDay, curdeius, HazardyKnusperkeks.
ksyx requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

- Fixes https://github.com/llvm/llvm-project/issues/53227 that wrongly indents multiline comments
- Fixes wrong detection of single-line opening bracket when used along with those only opening scopes:

  	void foo()
  	{
  	  {
  		int x;
  	  }
  	}

- Fixes wrong recognition of first line of definition when the line starts with block comment:

          /*
             Some descriptions about function
          */
  	/*inline*/ void bar() {
  	}

- Fixes wrong recognition of enum when used as a type name rather than starting definition block:

  	void foobar(const enum EnumType e) {
  	}


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D117520

Files:
  clang/lib/Format/DefinitionBlockSeparator.cpp
  clang/unittests/Format/DefinitionBlockSeparatorTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117520.400662.patch
Type: text/x-patch
Size: 11947 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220118/57687387/attachment.bin>