[Lldb-commits] [PATCH] D48479: Represent invalid UUIDs as UUIDs with length zero
Greg Clayton via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jun 25 09:26:20 PDT 2018
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
Patch is good. Feel free to remove the DataExtractor::DumpUUID() in a separate NFC commit or just remove it in this patch
================
Comment at: source/Utility/DataExtractor.cpp:1101
//----------------------------------------------------------------------
void DataExtractor::DumpUUID(Stream *s, offset_t offset) const {
if (s) {
----------------
labath wrote:
> clayborg wrote:
> > This should take a UUID byte size as a second parameter and callers should be required to specify it explicitly
> Hmm.. this is actually unused. Can I just remove it?
Yes, please remove.
https://reviews.llvm.org/D48479
More information about the lldb-commits
mailing list