[Lldb-commits] [lldb] [llvm] Modify the localCache API to require an explicit commit on CachedFile… (PR #115331)

[Teresa Johnson via lldb-commits]

================
@@ -30,6 +30,7 @@ class CachedFileStream {
   CachedFileStream(std::unique_ptr<raw_pwrite_stream> OS,
                    std::string OSPath = "")
       : OS(std::move(OS)), ObjectPathName(OSPath) {}
+  virtual Error commit() { return Error::success(); }
----------------
teresajohnson wrote:

If the expectation is that commit() should now always be called, should there be some error detection in the base class?

https://github.com/llvm/llvm-project/pull/115331