hokein marked an inline comment as done.
hokein added a comment.
Now this patch only contains the removal part of the API (I have cleaned all
usages of `writeFileAtomically` API, except a remaining one in lldb
https://reviews.llvm.org/D154329).
================
Comment at: lldb/tools/lldb-server/lldb-platform.cpp:107
Status status;
- if (auto Err =
- handleErrors(llvm::writeFileAtomically(
- temp_file_path, file_spec.GetPath(), socket_id),
- [&status, &file_spec](const AtomicFileWriteError &E) {
- std::string ErrorMsgBuffer;
- llvm::raw_string_ostream S(ErrorMsgBuffer);
- E.log(S);
-
- switch (E.Error) {
- case atomic_write_error::failed_to_create_uniq_file:
- status = Status("Failed to create temp file: %s",
- ErrorMsgBuffer.c_str());
- break;
- case atomic_write_error::output_stream_error:
- status = Status("Failed to write to port file.");
- break;
- case atomic_write_error::failed_to_rename_temp_file:
- status = Status("Failed to rename file %s to %s:
%s",
- ErrorMsgBuffer.c_str(),
- file_spec.GetPath().c_str(),
- ErrorMsgBuffer.c_str());
- break;
- }
- })) {
+ if (auto Err = handleErrors(file_spec.GetPath(),
+ [&socket_id](llvm::raw_ostream &OS) {
----------------
avl wrote:
> the call to llvm::writeToOutput is probably missed here.
oops, good catch, the `handleError` should be `writeToOutput`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153740/new/
https://reviews.llvm.org/D153740
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
