Re: [PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

2015-08-24 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thank you! http://reviews.llvm.org/D12281 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

Re: [PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

2015-08-24 Thread Aaron Ballman via cfe-commits
LGTM! ~Aaron On Mon, Aug 24, 2015 at 9:45 AM, Angel Garcia via cfe-commits wrote: > angelgarcia created this revision. > angelgarcia added a reviewer: alexfh. > angelgarcia added subscribers: cfe-commits, klimek. > > Update python script, so that it doesn't print that line in new tests. > > http

[PATCH] D12281: Tests no longer need the 'REQUIRES: SHELL' line.

2015-08-24 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: alexfh. angelgarcia added subscribers: cfe-commits, klimek. Update python script, so that it doesn't print that line in new tests. http://reviews.llvm.org/D12281 Files: clang-tidy/add_new_check.py Index: clang-tidy/add_new_check