Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018 Google Inc. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license that can be |
| 5 | * found in the LICENSE file. |
| 6 | */ |
| 7 | |
| 8 | #include "bookmaker.h" |
| 9 | |
Cary Clark | 4855f78 | 2018-02-06 09:41:53 -0500 | [diff] [blame^] | 10 | #ifdef SK_BUILD_FOR_WIN |
| 11 | #include <windows.h> |
| 12 | #endif |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 13 | |
| 14 | // Check that mutiple like-named methods are under one Subtopic |
| 15 | |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 16 | // Check that SeeAlso reference each other |
| 17 | |
| 18 | // Would be nice to check if other classes have 'create' methods that are included |
| 19 | // SkSurface::makeImageSnapShot should be referenced under SkImage 'creators' |
| 20 | |
| 21 | class SelfChecker { |
| 22 | public: |
| 23 | SelfChecker(const BmhParser& bmh) |
| 24 | : fBmhParser(bmh) |
| 25 | {} |
| 26 | |
| 27 | bool check() { |
| 28 | for (const auto& topic : fBmhParser.fTopicMap) { |
| 29 | Definition* topicDef = topic.second; |
| 30 | if (topicDef->fParent) { |
| 31 | continue; |
| 32 | } |
| 33 | if (!topicDef->isRoot()) { |
| 34 | return fBmhParser.reportError<bool>("expected root topic"); |
| 35 | } |
| 36 | fRoot = topicDef->asRoot(); |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 37 | if (!this->checkSeeAlso()) { |
| 38 | return false; |
| 39 | } |
Cary Clark | 4855f78 | 2018-02-06 09:41:53 -0500 | [diff] [blame^] | 40 | // report functions that are not covered by related hierarchy |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 41 | if (!this->checkRelatedFunctions()) { |
| 42 | return false; |
| 43 | } |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 44 | } |
| 45 | return true; |
| 46 | } |
| 47 | |
| 48 | protected: |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 49 | |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 50 | bool checkRelatedFunctions() { |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 51 | const Definition* cs = this->classOrStruct(); |
| 52 | vector<string> methodNames; |
| 53 | if (cs) { |
| 54 | string prefix = cs->fName + "::"; |
| 55 | for (auto& csChild : cs->fChildren) { |
| 56 | if (MarkType::kMethod != csChild->fMarkType) { |
| 57 | // only check methods for now |
| 58 | continue; |
| 59 | } |
| 60 | if (Definition::MethodType::kConstructor == csChild->fMethodType) { |
| 61 | continue; |
| 62 | } |
| 63 | if (Definition::MethodType::kDestructor == csChild->fMethodType) { |
| 64 | continue; |
| 65 | } |
| 66 | if (Definition::MethodType::kOperator == csChild->fMethodType) { |
| 67 | continue; |
| 68 | } |
| 69 | if (csChild->fClone) { |
| 70 | // FIXME: check to see if all cloned methods are in table |
| 71 | // since format of clones is in flux, defer this check for now |
| 72 | continue; |
| 73 | } |
Cary Clark | 4855f78 | 2018-02-06 09:41:53 -0500 | [diff] [blame^] | 74 | bool containsMarkTypeIn = csChild->fDeprecated; // no markup for deprecated |
| 75 | for (auto child : csChild->fChildren) { |
| 76 | if (MarkType::kIn == child->fMarkType) { |
| 77 | containsMarkTypeIn = true; |
| 78 | break; |
| 79 | } |
| 80 | } |
| 81 | if (!containsMarkTypeIn) { |
| 82 | #ifdef SK_BUILD_FOR_WIN |
| 83 | /* SkDebugf works in both visual studio and git shell, but |
| 84 | in git shell output is not piped to grep. |
| 85 | printf does not generate output in visual studio, but |
| 86 | does in git shell and can be piped. |
| 87 | */ |
| 88 | if (IsDebuggerPresent()) { |
| 89 | SkDebugf("No #In: %s\n", csChild->fName.c_str()); |
| 90 | } else |
| 91 | #endif |
| 92 | printf("No #In: %s\n", csChild->fName.c_str()); |
| 93 | } |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 94 | } |
| 95 | } |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 96 | return true; |
| 97 | } |
| 98 | |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 99 | bool checkSeeAlso() { |
| 100 | return true; |
| 101 | } |
| 102 | |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 103 | const Definition* classOrStruct() { |
| 104 | for (auto& rootChild : fRoot->fChildren) { |
Cary Clark | 08895c4 | 2018-02-01 09:37:32 -0500 | [diff] [blame] | 105 | if (rootChild->isStructOrClass()) { |
Cary Clark | ab2621d | 2018-01-30 10:08:57 -0500 | [diff] [blame] | 106 | return rootChild; |
| 107 | } |
| 108 | } |
| 109 | return nullptr; |
| 110 | } |
| 111 | |
Cary Clark | 2dc84ad | 2018-01-26 12:56:22 -0500 | [diff] [blame] | 112 | enum class Optional { |
| 113 | kNo, |
| 114 | kYes, |
| 115 | }; |
| 116 | |
Cary Clark | ac47b88 | 2018-01-11 10:35:44 -0500 | [diff] [blame] | 117 | private: |
| 118 | const BmhParser& fBmhParser; |
| 119 | RootDefinition* fRoot; |
| 120 | }; |
| 121 | |
| 122 | bool SelfCheck(const BmhParser& bmh) { |
| 123 | SelfChecker checker(bmh); |
| 124 | return checker.check(); |
| 125 | } |