[Lldb-commits] [PATCH] D132191: Treat a UUID of all zeros consistently to mean "Invalid UUID"

[Jim Ingham via Phabricator via lldb-commits]

jingham updated this revision to Diff 456473.
jingham added a comment.
Herald added subscribers: arphaman, MaskRay, emaste.
Herald added a reviewer: ributzka.

Remove the static factories and replace them with constructors.

This was mostly mechanical, but a biggish patch against the previous version, so I thought I'd put this up for one more look in case folks were interested.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132191/new/

https://reviews.llvm.org/D132191

Files:
  lldb/include/lldb/Utility/UUID.h
  lldb/source/API/SBModuleSpec.cpp
  lldb/source/Core/DataFileCache.cpp
  lldb/source/Host/macosx/objcxx/HostInfoMacOSX.mm
  lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp
  lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp
  lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp
  lldb/source/Plugins/ObjectFile/Breakpad/BreakpadRecords.cpp
  lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
  lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
  lldb/source/Plugins/ObjectFile/PDB/ObjectFilePDB.cpp
  lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
  lldb/source/Plugins/Process/minidump/MinidumpParser.cpp
  lldb/source/Utility/UUID.cpp
  lldb/unittests/ObjectFile/Breakpad/BreakpadRecordsTest.cpp
  lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
  lldb/unittests/SymbolFile/DWARF/DWARFIndexCachingTest.cpp
  lldb/unittests/Utility/UUIDTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132191.456473.patch
Type: text/x-patch
Size: 24590 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20220829/7edf071d/attachment-0001.bin>