Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cxx_verifier): ensure primitive members are always initialized #5640

Merged
merged 1 commit into from
May 19, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 4 additions & 4 deletions kythe/cxx/verifier/assertions.h
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ class AssertionParser {
Verifier& verifier_;

/// The arena from the verifier; needed by the parser implementation.
Arena* arena_;
Arena* arena_ = nullptr;

std::vector<GoalGroup> groups_;
bool inside_goal_group_ = false;
Expand Down Expand Up @@ -313,17 +313,17 @@ class AssertionParser {
bool had_errors_ = false;
/// Save the end-of-file location from the lexer.
yy::location last_eof_;
size_t last_eof_ofs_;
size_t last_eof_ofs_ = 0;
/// Inspections to be performed after the verifier stops.
std::vector<Inspection> inspections_;
/// Context mapping symbols to AST nodes.
std::unordered_map<Symbol, Identifier*> identifier_context_;
std::unordered_map<Symbol, EVar*> evar_context_;
std::unordered_map<EVar*, Symbol> singleton_evars_;
/// Are we dumping lexer trace information?
bool trace_lex_;
bool trace_lex_ = false;
/// Are we dumping parser trace information?
bool trace_parse_;
bool trace_parse_ = false;
/// Should we inspect every user-provided EVar?
bool default_inspect_ = false;
/// The current file's path.
Expand Down