[PATCH] D63911: [clang-doc] Serialize child namespaces and records

Diego Astiazarán via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 12:14:13 PDT 2019


DiegoAstiazaran updated this revision to Diff 207126.
DiegoAstiazaran marked 6 inline comments as done.
DiegoAstiazaran added a comment.
Herald added subscribers: kadircet, arphaman.

Add comments.
Extract repeated logic into mapDecl function in SerializeTest.cpp
Use EmptySID instead of specific USR in tests.
Use {} instead of {nullptr, nullptr}.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63911/new/

https://reviews.llvm.org/D63911

Files:
  clang-tools-extra/clang-doc/Mapper.cpp
  clang-tools-extra/clang-doc/Serialize.cpp
  clang-tools-extra/clang-doc/Serialize.h
  clang-tools-extra/unittests/clang-doc/ClangDocTest.cpp
  clang-tools-extra/unittests/clang-doc/SerializeTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63911.207126.patch
Type: text/x-patch
Size: 20780 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190628/1b016a8a/attachment-0001.bin>


More information about the cfe-commits mailing list