From 9ae04b75c6a9abb562be62741ce377e98654f04c Mon Sep 17 00:00:00 2001 From: "Scott E. Graves" Date: Tue, 24 Dec 2024 14:24:11 -0600 Subject: [PATCH] refactor --- .../src/file_manager/open_file.cpp | 35 ++++++++++--------- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/repertory/librepertory/src/file_manager/open_file.cpp b/repertory/librepertory/src/file_manager/open_file.cpp index 9c28cf03..a1b605ce 100644 --- a/repertory/librepertory/src/file_manager/open_file.cpp +++ b/repertory/librepertory/src/file_manager/open_file.cpp @@ -431,26 +431,27 @@ auto open_file::native_operation( : new_file_size % chunk_size_); } - if (original_file_size != new_file_size) { - set_modified(); + if (original_file_size == new_file_size) { + return res; + } + set_modified(); - fsi_.size = new_file_size; - auto now = std::to_string(utils::time::get_time_now()); - res = provider_.set_item_meta( - fsi_.api_path, { - {META_CHANGED, now}, - {META_MODIFIED, now}, - {META_SIZE, std::to_string(new_file_size)}, - {META_WRITTEN, now}, - }); - if (res != api_error::success) { - utils::error::raise_api_path_error(function_name, get_api_path(), res, - "failed to set file meta"); - return set_api_error(res); - } + fsi_.size = new_file_size; + auto now = std::to_string(utils::time::get_time_now()); + res = provider_.set_item_meta(fsi_.api_path, + { + {META_CHANGED, now}, + {META_MODIFIED, now}, + {META_SIZE, std::to_string(new_file_size)}, + {META_WRITTEN, now}, + }); + if (res == api_error::success) { + return res; } - return res; + utils::error::raise_api_path_error(function_name, get_api_path(), res, + "failed to set file meta"); + return set_api_error(res); } auto open_file::read(std::size_t read_size, std::uint64_t read_offset,