Page MenuHomePhabricator

[clang-tidy] Use StringMap for ClangTidyOptions::OptionsMap
ClosedPublic

Authored by njames93 on Wed, Jul 29, 9:23 AM.

Details

Summary

Ordering of options isn't important so an llvm::StringMap is a much better container for this purpose.

Diff Detail

Event Timeline

njames93 created this revision.Wed, Jul 29, 9:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jul 29, 9:23 AM
njames93 requested review of this revision.Wed, Jul 29, 9:23 AM
njames93 updated this revision to Diff 281639.Wed, Jul 29, 9:27 AM

Missing call to getValue()

gribozavr2 accepted this revision.Wed, Jul 29, 10:40 AM
This revision is now accepted and ready to land.Wed, Jul 29, 10:40 AM