clayborg updated this revision to Diff 156544.
clayborg added a comment.
Herald added subscribers: mgorny, srhines.

Improve comments to indicate why padding might be there, fix a test case to use 
the right file, and add all new .dmp files to the CMakeLists.txt


https://reviews.llvm.org/D49579

Files:
  lldb.xcodeproj/project.pbxproj
  source/Plugins/Process/minidump/MinidumpTypes.cpp
  unittests/Process/minidump/CMakeLists.txt
  unittests/Process/minidump/Inputs/memory-list-not-padded.dmp
  unittests/Process/minidump/Inputs/memory-list-padded.dmp
  unittests/Process/minidump/Inputs/module-list-not-padded.dmp
  unittests/Process/minidump/Inputs/module-list-padded.dmp
  unittests/Process/minidump/Inputs/thread-list-not-padded.dmp
  unittests/Process/minidump/Inputs/thread-list-padded.dmp
  unittests/Process/minidump/MinidumpParserTest.cpp



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to