[PATCH] D22208: clang-tidy] Fixes to modernize-use-emplace

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 10 22:42:00 PDT 2016


Prazek created this revision.
Prazek added reviewers: sbenza, alexfh, aaron.ballman.
Prazek added subscribers: cfe-commits, hokein, sbarzowski, mnbvmar, staronj, sbenza.
Prazek set the repository for this revision to rL LLVM.
Prazek added a project: clang-extra.

Not everything is valid (as you see in FIXME), but it should works for 99.8% cases.

I gonna fix it in next week, but I would like to land this bug fixes right now.

This is mainly fixes of the review that @sbenza gave me last time.

Repository:
  rL LLVM

http://reviews.llvm.org/D22208

Files:
  clang-tidy/modernize/UseEmplaceCheck.cpp
  clang-tidy/modernize/UseEmplaceCheck.h
  clang-tidy/utils/Matchers.h
  docs/clang-tidy/checks/modernize-use-emplace.rst
  test/clang-tidy/modernize-use-emplace.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22208.63461.patch
Type: text/x-patch
Size: 13686 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160711/2436829c/attachment.bin>


More information about the cfe-commits mailing list