[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