[PATCH] D143996: [clang-tidy][doc] Remove unused variable

2023-02-23 Thread Björn Svensson via Phabricator via cfe-commits
bjosv added a comment. @carlosgalvezp Sorry, I should have mentioned that I don't have push rights and that this is my first contribution. My GitHub user is: bjosv and email: bjorn.a.svens...@est.tech Thanks. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143996/new/ https://reviews

[PATCH] D143996: [clang-tidy][doc] Remove unused variable

2023-02-14 Thread Björn Svensson via Phabricator via cfe-commits
bjosv updated this revision to Diff 497344. bjosv added a comment. Updated to fill 80 char as proposed, and changed some wordings. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143996/new/ https://reviews.llvm.org/D143996 Files: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidel

[PATCH] D143996: [clang-tidy][doc] Remove unused variable

2023-02-14 Thread Björn Svensson via Phabricator via cfe-commits
bjosv created this revision. Herald added subscribers: kbarton, xazax.hun, nemanjai. Herald added a reviewer: njames93. Herald added a project: All. bjosv requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Remove mention of a varia