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

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 2 11:12:12 PDT 2016


Prazek added a comment.

In https://reviews.llvm.org/D22208#503194, @alexfh wrote:

> Please add revision number (this can be automated, if include differential revision URL in your commit message as described in http://llvm.org/docs/Phabricator.html#committing-a-change).


I normally use arc, but I was working in half from my laptop and in half from workstation, and I am not sure if arc would handle this.

I added the url in the patch, but I didn't know that "Differential Revision:" is required. I will keep it in mind next time.


https://reviews.llvm.org/D22208





More information about the cfe-commits mailing list