[PATCH] D139937: [clang-format] fix typo in doc for SLS_Inline

2022-12-13 Thread Zhikai Zeng via Phabricator via cfe-commits
Backl1ght added a comment. Seems like I have misunderstood it, after some test I'm sure that it means merge if its an argument of a function. I think there are something todo to make the doc more clearly. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D139937: [clang-format] fix typo in doc for SLS_Inline

2022-12-13 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. I'm not sure if that is correct, as that doesn't fit the example (unless the example is wrong) It feels like its saying merge if its an argument of a function (sort i

[PATCH] D139937: [clang-format] fix typo in doc for SLS_Inline

2022-12-13 Thread Zhikai Zeng via Phabricator via cfe-commits
Backl1ght added a comment. OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139937/new/ https://reviews.llvm.org/D139937 _

[PATCH] D139937: [clang-format] fix typo in doc for SLS_Inline

2022-12-13 Thread Zhikai Zeng via Phabricator via cfe-commits
Backl1ght created this revision. Backl1ght added reviewers: HazardyKnusperkeks, MyDeveloperDay. Backl1ght added a project: clang-format. Herald added a project: All. Backl1ght requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG