I would prefer to use llvm types to do the parsing wherever possible.  Why do we need DataExtractor?  If the only reason is to force little endian, just use the types in llvm/Endian.h<br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 16, 2016 at 8:13 AM Dimitar Vlahovski via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dvlahovski updated this revision to Diff 68180.<br>
dvlahovski added a comment.<br>
<br>
Fixing a little bug - should get the byteorder<br>
<br>
after calling SignatureMatchAndSetByteOrder<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23545" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23545</a><br>
<br>
Files:<br>
  cmake/LLDBDependencies.cmake<br>
  source/Plugins/Process/CMakeLists.txt<br>
  source/Plugins/Process/minidump/CMakeLists.txt<br>
  source/Plugins/Process/minidump/MinidumpParser.cpp<br>
  source/Plugins/Process/minidump/MinidumpParser.h<br>
  source/Plugins/Process/minidump/MinidumpTypes.cpp<br>
  source/Plugins/Process/minidump/MinidumpTypes.h<br>
  unittests/Process/CMakeLists.txt<br>
  unittests/Process/minidump/CMakeLists.txt<br>
  unittests/Process/minidump/Inputs/linux-x86_64.dmp<br>
  unittests/Process/minidump/MinidumpParserTest.cpp<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>