[PATCH] clang-modernize: Apply replacements using clang-apply-replacements

Edwin Vane edwin.vane at intel.com
Fri Oct 4 07:56:28 PDT 2013


Hi tareqsiraj, arielbernal, Sarcasm,

The clang-apply-replacements process is now invoked to apply
replacements between applying transforms. This resulted in a massive
simplification of the tool:
- FileOverrides class no longer needed.
- Change tracking and code formatting no longer needed.
- No more dependency on libclangApplyReplacements.
- Final syntax check is easier to do directly now than with a separate
  header/source pair.

Replacement handling stuff abstracted into a new header/source pair to
de-clutter ClangModernize.cpp somewhat.

Tests updated.

http://llvm-reviews.chandlerc.com/D1836

Files:
  clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp
  clang-modernize/AddOverride/AddOverride.cpp
  clang-modernize/AddOverride/AddOverride.h
  clang-modernize/CMakeLists.txt
  clang-modernize/Core/CMakeLists.txt
  clang-modernize/Core/FileOverrides.cpp
  clang-modernize/Core/FileOverrides.h
  clang-modernize/Core/Reformatting.cpp
  clang-modernize/Core/Reformatting.h
  clang-modernize/Core/ReplacementHandling.cpp
  clang-modernize/Core/ReplacementHandling.h
  clang-modernize/Core/SyntaxCheck.cpp
  clang-modernize/Core/SyntaxCheck.h
  clang-modernize/Core/Transform.cpp
  clang-modernize/Core/Transform.h
  clang-modernize/LoopConvert/LoopConvert.cpp
  clang-modernize/LoopConvert/LoopConvert.h
  clang-modernize/PassByValue/PassByValue.cpp
  clang-modernize/PassByValue/PassByValue.h
  clang-modernize/ReplaceAutoPtr/ReplaceAutoPtr.cpp
  clang-modernize/ReplaceAutoPtr/ReplaceAutoPtr.h
  clang-modernize/UseAuto/UseAuto.cpp
  clang-modernize/UseAuto/UseAuto.h
  clang-modernize/UseNullptr/UseNullptr.cpp
  clang-modernize/UseNullptr/UseNullptr.h
  clang-modernize/tool/CMakeLists.txt
  clang-modernize/tool/ClangModernize.cpp
  clang-modernize/tool/Makefile
  test/clang-modernize/Core/Inputs/.clang-format
  test/clang-modernize/Core/NoReformattingNeeded.cpp
  test/clang-modernize/Core/Reformatting.cpp
  test/clang-modernize/HeaderReplacements/main.cpp
  unittests/clang-modernize/CMakeLists.txt
  unittests/clang-modernize/FileOverridesTest.cpp
  unittests/clang-modernize/PerfSupportTest.cpp
  unittests/clang-modernize/ReformattingTest.cpp
  unittests/clang-modernize/TransformTest.cpp
  unittests/clang-modernize/UniqueHeaderNameTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1836.1.patch
Type: text/x-patch
Size: 81901 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131004/922411ea/attachment.bin>


More information about the cfe-commits mailing list