[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