commit d789550da4bf666ca11487ddcb5dd6b9b13ec3d7
author Cary Clark <caryclark@skia.org> Wed Jul 18 15:10:08 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Jul 18 19:42:52 2018 +0000
tree bf58fd88ff28d0ecb52a4b34d2e1a8739f403f59
parent 5ed4df358318161edf7e50b7f04ed972651af913

add more fiddle hash checks

Convert some of bookmaker to use real json instead
of rolling its own. Also check to see if all
hashes are read.

TBR=jcgregario@google.com

Docs-Preview: https://skia.org/?cl=142166
Bug: skia:8151
Change-Id: Ib35ecd69648faec3522903e0b552d37b04b73f8b
Reviewed-on: https://skia-review.googlesource.com/142166
Commit-Queue: Cary Clark <caryclark@skia.org>
Auto-Submit: Cary Clark <caryclark@skia.org>
Reviewed-by: Cary Clark <caryclark@skia.org>

tools/bookmaker/includeParser.cpp

diff --git a/tools/bookmaker/includeParser.cpp b/tools/bookmaker/includeParser.cpp
index fc378dd..d6594ea 100644
--- a/tools/bookmaker/includeParser.cpp
+++ b/tools/bookmaker/includeParser.cpp
@@ -2464,6 +2464,9 @@
             if (!this->checkForWord()) {
                 return false;
             }
+            if (!fParent->fTokens.size()) {
+                break;
+            }
             {
                 const Definition& lastToken = fParent->fTokens.back();
                 if (lastToken.fType != Definition::Type::kWord) {