Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

2016-09-16 Thread Kirill Bobyrev via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281713: [clang-tidy] Bugfix for readability-redundant-control-flow check (authored by omtcyfz). Changed prior to commit: https://reviews.llvm.org/D24500?vs=71140=71608#toc Repository: rL LLVM

Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

2016-09-16 Thread Kirill Bobyrev via cfe-commits
omtcyfz added a subscriber: omtcyfz. omtcyfz added a comment. In https://reviews.llvm.org/D24500#542284, @malcolm.parsons wrote: > I didn't report a bug for this issue, and there isn't an existing one. > > I don't have commit access, so please commit it for me. Since Alex is away at the moment

Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

2016-09-14 Thread Malcolm Parsons via cfe-commits
malcolm.parsons added a comment. I didn't report a bug for this issue, and there isn't an existing one. https://reviews.llvm.org/D24500 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

2016-09-14 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. If there's a related buganizer item, please mention it in the commit message. https://reviews.llvm.org/D24500 ___ cfe-commits mailing list

[PATCH] D24500: [clang-tidy] Bugfix for readability-redundant-control-flow check

2016-09-13 Thread Malcolm Parsons via cfe-commits
malcolm.parsons created this revision. malcolm.parsons added reviewers: sbenza, aaron.ballman, alexfh. malcolm.parsons added subscribers: Eugene.Zelenko, cfe-commits, beanz. This check did not create FixItHints when the statement before the redundant control flow was not followed by a semicolon.