This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy][NFC] Fix namespace comments in AvoidThrowingObjCExceptionCheck.cpp 🧹
AbandonedPublic

Authored by stephanemoore on Nov 9 2022, 1:45 PM.

Details

Summary

This commit fixes namespace comments in AvoidThrowingObjCExceptionCheck.cpp as suggested by the linter.

Diff Detail

Event Timeline

stephanemoore created this revision.Nov 9 2022, 1:45 PM
Herald added a project: Restricted Project. · View Herald Transcript
stephanemoore published this revision for review.Nov 9 2022, 1:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 9 2022, 1:50 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
stephanemoore edited the summary of this revision. (Show Details)Nov 9 2022, 1:50 PM
Eugene.Zelenko retitled this revision from [clang-tidy] Fix namespace comments in AvoidThrowingObjCExceptionCheck.cpp 🧹 to [clang-tidy][NFC] Fix namespace comments in AvoidThrowingObjCExceptionCheck.cpp 🧹.Nov 9 2022, 2:43 PM

Will be good idea to combine this patch with D137738.

Will be good idea to combine this patch with D137738.

My intent was to land the simpler linter fixes first while iterating on the fixes in D137738 as necessary but I can combine the two patches if that is preferred.

Will be good idea to combine this patch with D137738.

My intent was to land the simpler linter fixes first while iterating on the fixes in D137738 as necessary but I can combine the two patches if that is preferred.

I don't think that such patches deserve dedicated commit unless you fix them on entire Clang-tidy code.

LegalizeAdulthood resigned from this revision.Mar 29 2023, 8:11 AM
stephanemoore abandoned this revision.Mar 30 2023, 8:04 AM

Abandoned this revision and folded the changes into another revision.