Fix a problem where a ValueObject could fail to update itself, but since it was previously valid, we'd have an old checksum to compare aginst no new checksum (because failure to update), and assert() and die. Fix the problem by only caring about this assertion logic if updates succeed

llvm-svn: 225418

lldb/source/Core/ValueObject.cpp

@@ -250,7 +250,7 @@ ValueObject::UpdateValueIfNeeded (bool update_format)
                 m_value_checksum.clear();
             }
             
-            assert (old_checksum.empty() == !need_compare_checksums);
+            assert (success && (old_checksum.empty() == !need_compare_checksums));
             
             if (first_update)
                 SetValueDidChange (false);