hanchenye-llvm-project/clang-tools-extra
Angel Garcia Gomez 432ff5e205 Handle correctly containers that are data members in modernize-loop-convert.
Summary:
I recently found that the variable naming wasn't working as expected with containers that are data members. The new index always received the name "Elem" (or equivalent) regardless of the container's name.
The check was assuming that the container's declaration was a VarDecl, which cannot be converted to a FieldDecl (a data member), and then it could never retrieve its name.

This also fixes some cases where the check failed to find the container at all (so it didn't do any fix) because of the same reason.

Reviewers: klimek

Subscribers: cfe-commits, alexfh

Differential Revision: http://reviews.llvm.org/D14289

llvm-svn: 251943
2015-11-03 16:38:31 +00:00
..
clang-apply-replacements Replace double negation of !FileID.isInvalid() with FileID.isValid(). 2015-10-03 10:46:20 +00:00
clang-modernize Revert "Apply modernize-use-default to clang-tools-extra." 2015-10-20 21:45:52 +00:00
clang-query Revert "Apply modernize-use-default to clang-tools-extra." 2015-10-20 21:45:52 +00:00
clang-rename Revert r241330. It compiled with Visual C++ 2013 and gcc 4.9.1 (mingw) but now fails the bots. 2015-07-03 09:30:33 +00:00
clang-tidy Handle correctly containers that are data members in modernize-loop-convert. 2015-11-03 16:38:31 +00:00
docs [clang-tidy] Add new check cppcoreguidelines-pro-bounds-array-to-pointer-decay 2015-10-26 21:56:02 +00:00
modularize Revert "Apply modernize-use-default to clang-tools-extra." 2015-10-20 21:45:52 +00:00
pp-trace Revert "Apply modernize-use-default to clang-tools-extra." 2015-10-20 21:45:52 +00:00
test Handle correctly containers that are data members in modernize-loop-convert. 2015-11-03 16:38:31 +00:00
tool-template Revert "Adapt clang-tools-extra to clang module format changes." 2015-02-25 02:46:37 +00:00
unittests Revert "Apply modernize-use-default to clang-tools-extra." 2015-10-20 21:45:52 +00:00
.arcconfig Update arc config to new domain. 2014-04-07 10:23:19 +00:00
.gitignore
CMakeLists.txt Fix the *unchecked* commit. 2015-10-01 23:50:06 +00:00
CODE_OWNERS.TXT Updating the code owners list. 2015-09-02 20:00:41 +00:00
LICENSE.TXT
Makefile Fix ./configure build after r232338. 2015-03-16 00:43:57 +00:00
README.txt Update references to new lists.llvm.org mailing lists. 2015-08-05 03:56:44 +00:00

README.txt

//===----------------------------------------------------------------------===//
// Clang Tools repository
//===----------------------------------------------------------------------===//

Welcome to the repository of extra Clang Tools.  This repository holds tools
that are developed as part of the LLVM compiler infrastructure project and the
Clang frontend.  These tools are kept in a separate "extra" repository to
allow lighter weight checkouts of the core Clang codebase.

This repository is only intended to be checked out inside of a full LLVM+Clang
tree, and in the 'tools/extra' subdirectory of the Clang checkout.

All discussion regarding Clang, Clang-based tools, and code in this repository
should be held using the standard Clang mailing lists:
  http://lists.llvm.org/mailman/listinfo/cfe-dev

Code review for this tree should take place on the standard Clang patch and
commit lists:
  http://lists.llvm.org/mailman/listinfo/cfe-commits

If you find a bug in these tools, please file it in the LLVM bug tracker:
  http://llvm.org/bugs/