[Lldb-commits] [PATCH] D23545: Minidump parsing
Dimitar Vlahovski via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 18 11:10:57 PDT 2016
dvlahovski added inline comments.
Comment at: source/Plugins/Process/minidump/MinidumpTypes.h:260
@@ +259,3 @@
+ uint8_t number_of_processors;
+ uint8_t product_type;
> I'll concede that the static_assert is useful. Given that, showing the arithmetic explicitly will be helpful when one of these assertions trips, because you'll be able to see how it's supposed to correspond to the struct.
Yes that was my inital insentive to write explicitly the arithmetic
More information about the lldb-commits