================
@@ -89,8 +89,11 @@ Status 
MinidumpFileBuilder::AddHeaderAndCalculateDirectories() {
         "Failed to fill in header and directory "
         "sections. Written / Expected (%" PRIx64 " / %" PRIx64 ")",
         new_offset, m_saved_data_size);
-
   return error;
+  if (error.Fail())
+    return error;
----------------
Jlalond wrote:

@labath I agree, I think it would be ideal to agree with Breakpad/Crashpad on a 
section about what program generated it, including version of that software. 
From Memory I know there is some brakepad relevant data, but a unified section 
would be great.

https://github.com/llvm/llvm-project/pull/120166
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to