[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-04-04 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Herald added a project: All. This seems to have caused https://github.com/llvm/llvm-project/issues/54703 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120374/new/ https://reviews.llvm.org/D120374

[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-02-24 Thread Marek Kurdej via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGdbc4d281bd69: [clang-format] Do not insert space after new/delete keywords in C function… (authored by penagos, committed by curdeius). Repository:

[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-02-23 Thread Luis Penagos via Phabricator via cfe-commits
penagos marked an inline comment as done. penagos added a comment. In D120374#3339554 , @curdeius wrote: > In D120374#3339337 , @penagos wrote: > >> Though it's still unclear to me whether or not it'd make more

[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-02-23 Thread Luis Penagos via Phabricator via cfe-commits
penagos updated this revision to Diff 410788. penagos added a comment. Add missing space in unittest. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120374/new/ https://reviews.llvm.org/D120374 Files: clang/lib/Format/TokenAnnotator.cpp

[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-02-22 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. In D120374#3339337 , @penagos wrote: > Though it's still unclear to me whether or not it'd make more sense to > continue to pursue a change to

[PATCH] D120374: [clang-format] Do not insert space after new/delete keywords in C function declarations

2022-02-22 Thread Luis Penagos via Phabricator via cfe-commits
penagos created this revision. penagos added reviewers: curdeius, MyDeveloperDay, HazardyKnusperkeks. penagos updated this revision to Diff 410700. penagos added a comment. penagos published this revision for review. penagos added a comment. Herald added a project: clang. Herald added a