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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 19 01:46:45 PST 2022


MyDeveloperDay added a comment.

ok, this is better but not quite right still.. I though I mentioned this before (please add as a unit test)

  HRESULT
  Foo::bar() {}
  
  HRESULT
  Foo::bar() {}

formats as

  HRESULT
  
  Foo::bar() {}
  
  HRESULT
  
  Foo::bar() {}

If you are windows developer using clang-format  (HRESULT,LRESULT,LPSTR,LPCSTR,DWORD_PTR etc.. ) its an issue.

Sorry I so want this to be right, its singularly one of the things I've wanted for some time (you just got to it before me ;-) ) so I'm super excited by it.

>From the other aspects we are getting there.


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

https://reviews.llvm.org/D117520



More information about the cfe-commits mailing list