[Lldb-commits] [PATCH] D23545: Minidump parsing
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 18 10:25:25 PDT 2016
zturner added inline comments.
Comment at: source/Plugins/Process/minidump/MinidumpTypes.h:259
@@ +258,3 @@
+static_assert(sizeof(MinidumpSystemInfo) == MINIDUMP_SYSTEM_INFO_SIZE, "sizeof MinidumpSystemInfo is not correct!");
I think the `static_assert` is a good idea. Since this corresponds to an on disk format where the size of the structure is fixed and known, the static assert is a good idea. But the calculation is unnecessary I think. It's more readable if someone can just look at it and say "ok it has to be 32 bytes" or whatever, rather than doing this math in their head.
More information about the lldb-commits