[PATCH] D48395: Added PublicOnly flag

2018-06-20 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk created this revision. anniecherk added reviewers: juliehockett, jakehehrlich, mcgrathr, phosek. Herald added a subscriber: cfe-commits. Added a flag which, when enabled, documents only those methods and fields which have a Public attribute. Repository: rCTE Clang Tools Extra http

[PATCH] D48395: Added PublicOnly flag

2018-06-29 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk updated this revision to Diff 153597. anniecherk added a comment. Herald added a subscriber: eraman. - addressed Julie's comments - updated logic to consider both access specifier & linkage - added tests for each type of linkage https://reviews.llvm.org/D48395 Files: clang-tools-ex

[PATCH] D48395: Added PublicOnly flag

2018-07-02 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk updated this revision to Diff 153790. anniecherk added a comment. Updated the tests to more accurately check that the files that we expect to not be generated by clang-doc with the public flag are in fact not being created. The tests now run clang-doc over the test files with and with

[PATCH] D48395: Added PublicOnly flag

2018-07-02 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk updated this revision to Diff 153825. anniecherk marked 4 inline comments as done. anniecherk added a comment. Addressed Julie's comments https://reviews.llvm.org/D48395 Files: clang-tools-extra/clang-doc/ClangDoc.cpp clang-tools-extra/clang-doc/ClangDoc.h clang-tools-extra/cla

[PATCH] D48395: Added PublicOnly flag

2018-07-03 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk marked 11 inline comments as done. anniecherk added inline comments. Comment at: clang-tools-extra/clang-doc/Serialize.cpp:322-324 + if(PublicOnly && ! isPublic(D->getAccess(), D->getLinkageInternal())){ +return ""; + } juliehockett wrote: > Sinc

[PATCH] D48395: Added PublicOnly flag

2018-07-06 Thread Annie Cherkaev via Phabricator via cfe-commits
anniecherk updated this revision to Diff 154470. anniecherk added a comment. Updated according to Julie's suggestions, mostly adjusted formatting. https://reviews.llvm.org/D48395 Files: clang-tools-extra/clang-doc/ClangDoc.cpp clang-tools-extra/clang-doc/ClangDoc.h clang-tools-extra/clang