This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix syntax error in modernize-use-emplace.rst
AbandonedPublic

Authored by mboehme on Jul 12 2016, 1:59 AM.

Details

Reviewers
alexfh
Summary

The missing newline after the ".. code:: c++" was causing a compile error when producing the documentation

Diff Detail

Event Timeline

mboehme updated this revision to Diff 63656.Jul 12 2016, 1:59 AM
mboehme retitled this revision from to [clang-tidy] Fix syntax error in modernize-use-emplace.rst.
mboehme updated this object.
mboehme added a reviewer: alexfh.
mboehme added a subscriber: cfe-commits.

List of Clang-tidy checks was not updated for long time (I still don't see modernize-use-emplace). May be documentation contains other mistakes?

hokein added a subscriber: hokein.Jul 13 2016, 1:23 AM

Someone fixed this error r275254 several hours ago.

List of Clang-tidy checks was not updated for long time (I still don't see modernize-use-emplace). May be documentation contains other mistakes?

Are the clang-tools-extra documents updated automatically or manually? I see the documents get updated now (modernize-use-emplace check is in there).

mboehme abandoned this revision.Jul 13 2016, 1:32 AM

Good to hear -- abandoning this patch then.