diff options
author | Kevin Lubick <kjlubick@google.com> | 2018-01-05 10:11:11 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-05 18:03:25 +0000 |
commit | 4284613cfe211bfdcf3506f363d382c044ae8b51 (patch) | |
tree | 1004283289d9e70c786dd01702c8a8b144aa5932 /tools/bookmaker/includeWriter.cpp | |
parent | 4c79ecf2f493c0eaa4f58cf0bb086d61ea2aebcb (diff) |
Enable conditional-uninitialized flag
Bug: skia:7462
Cq-Include-Trybots: skia.primary:Test-Debian9-Clang-GCE-CPU-AVX2-x86_64-Release-All-SKNX_NO_SIMD
Change-Id: I1c0a09984bf28a5c620a89af56040f018bae6310
Reviewed-on: https://skia-review.googlesource.com/90941
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'tools/bookmaker/includeWriter.cpp')
-rw-r--r-- | tools/bookmaker/includeWriter.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/bookmaker/includeWriter.cpp b/tools/bookmaker/includeWriter.cpp index 463ac92c91..aeb1200a49 100644 --- a/tools/bookmaker/includeWriter.cpp +++ b/tools/bookmaker/includeWriter.cpp @@ -685,7 +685,7 @@ Definition* IncludeWriter::structMemberOut(const Definition* memberStart, const const char* commentStart = nullptr; ptrdiff_t commentLen = 0; string name(child.fContentStart, (int) (child.fContentEnd - child.fContentStart)); - bool isShort; + bool isShort = false; Definition* commentBlock = nullptr; for (auto memberDef : fBmhStructDef->fChildren) { if (memberDef->fName.length() - name.length() == memberDef->fName.find(name)) { @@ -902,7 +902,7 @@ bool IncludeWriter::populate(Definition* def, ParentPair* prevPair, RootDefiniti // skip include comment // if there is a series of same named methods, write one set of comments, then write all methods string methodName; - const Definition* method; + const Definition* method = nullptr; const Definition* clonedMethod = nullptr; const Definition* memberStart = nullptr; const Definition* memberEnd = nullptr; @@ -1721,7 +1721,7 @@ IncludeWriter::Wrote IncludeWriter::rewriteBlock(int size, const char* data, Phr int lastPrintable = 0; int lastSpace = -1; char c = 0; - char last; + char last = 0; bool embeddedIndirection = false; bool embeddedSymbol = false; bool hasLower = false; |