I assume it's optional and ReadAST does not have to set the
counter on success.
Without the patch msan complains that we pass
uninitialized value into noundef parameters of setCounterValue.
Differential D150492
[AST] Initialize local counter vitalybuka on May 12 2023, 3:21 PM. Authored by
Details
I assume it's optional and ReadAST does not have to set the Without the patch msan complains that we pass
Diff Detail
Event TimelineComment Actions Not my area, but the change is trivial. Nit: could move the variable closer to its use.
|
Looks like we only write the counter value when it's non-zero. So the reader would never set it. Thanks for fixing!