[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-04-12 Thread Nathan James 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 rG27dfcd978edc: [clang-tidy] Add include to misc-uniqueptr-reset-release (authored by njames93). Repository: rG LLVM Github Monorepo CHAN

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-04-12 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 336982. njames93 added a comment. Add store options method. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97683/new/ https://reviews.llvm.org/D97683 Files: clang-tools-extra/clang-tidy/misc/UniqueptrResetRe

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-04-12 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97683/new/ https://reviews.llvm.org/D97683 __

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-04-10 Thread Nathan James via Phabricator via cfe-commits
njames93 added a comment. Herald added a project: clang-tools-extra. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97683/new/ https://reviews.llvm.org/D97683 ___ cfe-commits mailing list cfe-commits

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-03-01 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 327185. njames93 added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97683/new/ https://reviews.llvm.org/D97683 Files: clang-tools-extra/clang-tidy/misc/UniqueptrResetReleaseCheck.cpp c

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-03-01 Thread Nathan James via Phabricator via cfe-commits
njames93 added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/UniqueptrResetReleaseCheck.cpp:22 +: ClangTidyCheck(Name, Context), + Inserter(Options.getLocalOrGlobal("IncludeStyle", +utils::IncludeSorter::IS_LLVM))

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-03-01 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/misc/UniqueptrResetReleaseCheck.cpp:22 +: ClangTidyCheck(Name, Context), + Inserter(Options.getLocalOrGlobal("IncludeStyle", +utils::IncludeSorter::IS_L

[PATCH] D97683: [clang-tidy] Add include to misc-uniqueptr-reset-release

2021-03-01 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added reviewers: alexfh, aaron.ballman. Herald added a subscriber: xazax.hun. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This is the only remaining check that creates `std::move` includ