[Lldb-commits] [lldb] r162795 - /lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp
Johnny Chen
johnny.chen at apple.com
Tue Aug 28 14:05:21 PDT 2012
Author: johnny
Date: Tue Aug 28 16:05:21 2012
New Revision: 162795
URL: http://llvm.org/viewvc/llvm-project?rev=162795&view=rev
Log:
Fix wrong logic with respect to warning messages.
Modified:
lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp
Modified: lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp?rev=162795&r1=162794&r2=162795&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp (original)
+++ lldb/trunk/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.cpp Tue Aug 28 16:05:21 2012
@@ -50,8 +50,17 @@
{
// Make sure the UUIDs match
lldb_private::UUID dsym_uuid;
- if (ofile->GetUUID(&dsym_uuid))
- return dsym_uuid == module->GetUUID();
+
+ if (!ofile->GetUUID(&dsym_uuid))
+ {
+ Host::SystemLog (Host::eSystemLogWarning,
+ "warning: failed to get the uuid for object file: '%s'\n",
+ ofile->GetFileSpec().GetFilename().GetCString());
+ return false;
+ }
+
+ if (dsym_uuid == module->GetUUID())
+ return true;
// Emit some warning messages since the UUIDs do not match!
const FileSpec &m_file_spec = module->GetFileSpec();
More information about the lldb-commits
mailing list