[PATCH] D23279: clang-reorder-fields

[Alexander Shaposhnikov via cfe-commits]

alexshap updated this revision to Diff 69445.
alexshap added a comment.

Address @djasper 's comments.

1. Try to flash out comments for helper functions / various interesting places in my code
2. More elaborate error-handling
3. Adjust the names of the variables: I -> i, E -> e
4. Minor cleanup & clang-format -i


Repository:
  rL LLVM

https://reviews.llvm.org/D23279

Files:
  CMakeLists.txt
  clang-reorder-fields/CMakeLists.txt
  clang-reorder-fields/ReorderFieldsAction.cpp
  clang-reorder-fields/ReorderFieldsAction.h
  clang-reorder-fields/tool/CMakeLists.txt
  clang-reorder-fields/tool/ClangReorderFields.cpp
  test/CMakeLists.txt
  test/clang-reorder-fields/CStructAmbiguousName.cpp
  test/clang-reorder-fields/CStructFieldsOrder.cpp
  test/clang-reorder-fields/ClassMixedInitialization.cpp
  test/clang-reorder-fields/ClassSimpleCtor.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23279.69445.patch
Type: text/x-patch
Size: 20529 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160826/95e18b36/attachment-0001.bin>