[PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-10-11 Thread Malcolm Parsons via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL283869: [clang-tidy] modernize-use-default default constructor bugfix (authored by malcolm.parsons). Changed prior to commit: https://reviews.llvm.org/D2?vs=70966&id=74232#toc Repository: rL LLVM

Re: [PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-09-26 Thread Malcolm Parsons via cfe-commits
malcolm.parsons added a comment. I don't have commit access, so please commit this and https://reviews.llvm.org/D24652 for me. Thanks. https://reviews.llvm.org/D2 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

Re: [PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-09-26 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a reviewer: alexfh. alexfh added a comment. This revision is now accepted and ready to land. LG. Thank you! https://reviews.llvm.org/D2 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://li

Re: [PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-09-26 Thread Malcolm Parsons via cfe-commits
malcolm.parsons added a comment. ping https://reviews.llvm.org/D2 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-09-11 Thread Malcolm Parsons via cfe-commits
malcolm.parsons created this revision. malcolm.parsons added a reviewer: klimek. malcolm.parsons added subscribers: cfe-commits, klimek, alexfh. Only member initializers that are written should prevent using '= default' on a default constructor. https://reviews.llvm.org/D2 Files: clang-tid