Page MenuHomePhabricator

Fixed global constant/variable naming check on C++ class for ObjC++ files.
ClosedPublic

Authored by Wizard on Tue, Mar 12, 7:22 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

Wizard created this revision.Tue, Mar 12, 7:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Mar 12, 7:22 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Wizard edited the summary of this revision. (Show Details)Tue, Mar 12, 7:23 PM
Wizard added reviewers: benhamilton, hokein.
hokein added inline comments.Wed, Mar 13, 4:26 AM
clang-tidy/google/GlobalVariableDeclarationCheck.cpp
82 ↗(On Diff #190371)

nit: removing the {}, the same below.

test/clang-tidy/google-objc-global-variable-declaration.mm
38 ↗(On Diff #190371)

instead of moving .m to .mm, I think we can create a new ".mm" file for testing this case.

Wizard updated this revision to Diff 190461.Wed, Mar 13, 11:42 AM

Resolve comments

Wizard marked 2 inline comments as done.Wed, Mar 13, 11:43 AM
Wizard updated this revision to Diff 190462.Wed, Mar 13, 11:46 AM

fix ObjC++ test

hokein accepted this revision.Thu, Mar 14, 1:44 AM
hokein added inline comments.
test/clang-tidy/google-objc-global-variable-declaration.mm
11 ↗(On Diff #190462)

nit: I think we should have a newline at end of file.

This revision is now accepted and ready to land.Thu, Mar 14, 1:44 AM
Wizard updated this revision to Diff 190750.Thu, Mar 14, 4:19 PM

add new line

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptThu, Mar 14, 5:16 PM