aaron.ballman added inline comments.
================ Comment at: clang-tidy/fuchsia/MultipleInheritanceCheck.cpp:35 + StringRef Name = Node->getIdentifier()->getName(); + auto Pair = InterfaceMap.find(Name); + if (Pair == InterfaceMap.end()) ---------------- Eugene.Zelenko wrote: > aaron.ballman wrote: > > Don't use `auto` as the type is not spelled out explicitly in the > > initialization. > But such cases are included in modernize-use-auto. Hmm, I suppose you could maybe make a case that `StringMap<bool>::iterator` is "complex enough" and that the returned type is sufficiently clear to warrant using `auto`... https://reviews.llvm.org/D40580 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
