[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-28 Thread Valeriy Savchenko via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9300ca541164: [doxygen] Fix bad doxygen results for BugReporterVisitors.h (authored by OikawaKirie, committed by vsavchenko). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://rev

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-24 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko accepted this revision. vsavchenko added a comment. This revision is now accepted and ready to land. Awesome! I will submit the patch! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85105/new/ https://reviews.llvm.org/D85105 ___ cfe

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-23 Thread Ella Ma via Phabricator via cfe-commits
OikawaKirie added a comment. @vsavchenko @NoQ The patch has been updated as required. Is there anything I need to do with the patch? Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:56 /// last node should be unique. - /// Use {@code

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-12 Thread Ella Ma via Phabricator via cfe-commits
OikawaKirie updated this revision to Diff 284991. OikawaKirie added a comment. Ok, I got it. I will pay attention to this in the future submits. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85105/new/ https://reviews.llvm.org/D85105 Files: clang/include/clang/StaticAnalyzer/Core/Bug

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-12 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko added a comment. Awesome, thanks! Bt, maybe we can use `\ref` form, it looks like it's a preferred form in the codebase. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85105/new/ https://reviews.llvm.org/D85105 ___ cfe-commits

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-11 Thread Ella Ma via Phabricator via cfe-commits
OikawaKirie updated this revision to Diff 284952. OikawaKirie marked an inline comment as done. OikawaKirie added a comment. Now it has a link to the `getEndPath`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85105/new/ https://reviews.llvm.org/D85105 Files: clang/include/clang/Stat

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-05 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko added a comment. Hey, thanks again for cleaning up the analyzer's docs 😄 Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:56 /// last node should be unique. - /// Use {@code getEndPath} to customize the note associated with t

[PATCH] D85105: [doxygen] Fix bad doxygen results for BugReporterVisitors.h

2020-08-02 Thread Ella Ma via Phabricator via cfe-commits
OikawaKirie created this revision. OikawaKirie added reviewers: NoQ, vsavchenko. OikawaKirie added a project: clang. Herald added subscribers: cfe-commits, Charusso. OikawaKirie requested review of this revision. Because of `{@code x}` will trigger a Doxygen bug. And as far as I am thinking,