klimek added inline comments.
================ Comment at: docs/RefactoringActionTutorial.rst:7 + + This tutorial talks about a work-in-progress library in Clang. + Some of the described features might not be available yet in trunk, but should ---------------- hokein wrote: > I'm a bit concerned about this. If we check in this doc before all features > are implemented, it probably confuses users. > I'm not too concerned about this. The code is already there, so I believe more docs are better :) ================ Comment at: docs/RefactoringActionTutorial.rst:41-42 + int computeArrayOffset(int x, int y) { + if (x >= 0) { + if (y >= 0) { + return x * 4 + y; ---------------- This looks like an example that's a better fit for clang-tidy? Perhaps a good example would be replacing all int x, int y parameters to an const Point& parameter or something like that? Repository: rL LLVM https://reviews.llvm.org/D39027 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits