[Lldb-commits] [lldb] [lldb] Skip remote PutFile when MD5 hashes equal (PR #88812)

[David Spickett via lldb-commits]

================
@@ -1197,6 +1197,34 @@ Status Platform::PutFile(const FileSpec &source, const FileSpec &destination,
   if (!source_file)
     return Status(source_file.takeError());
   Status error;
+
+  bool requires_upload = true;
+  {
----------------
DavidSpickett wrote:

If you really need a new scope, a small static function in the same file is the usual way. In this case it's fine to have them in the scope of the main function I think, I don't think they overlap with any existing var.

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