[PATCH] D85480: [NFC] Use value initializer for OVERLAPPED
Adrian McCarthy via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 7 16:48:42 PDT 2020
amccarth added a comment.
Huh, I thought aggregate initialization was effectively zero-initialization for the remaining fields, so I'm surprised this changes anything. I would have expected the original line, which is idiomatic in Win32 code, to have zero-initialized all of the fields.
But it looks like the field in question is part of a union, so maybe the rules there are different. I'll read up on it, but I don't see any harm here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85480/new/
https://reviews.llvm.org/D85480
More information about the llvm-commits
mailing list