From 4b22dfe5b6b44ecf824d322eff69699203506fc4 Mon Sep 17 00:00:00 2001 From: saturneric Date: Mon, 9 Jun 2025 22:08:51 +0200 Subject: fix: remove redundant flag setting in version check task - remove duplicate setting of current_commit_hash_publish_in_remote flag - flag is already set in the parent function before calling this method --- src/m_ver_check/BKTUSVersionCheckTask.cpp | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/m_ver_check') diff --git a/src/m_ver_check/BKTUSVersionCheckTask.cpp b/src/m_ver_check/BKTUSVersionCheckTask.cpp index dc9f404..fe98c78 100644 --- a/src/m_ver_check/BKTUSVersionCheckTask.cpp +++ b/src/m_ver_check/BKTUSVersionCheckTask.cpp @@ -176,7 +176,6 @@ void BKTUSVersionCheckTask::slot_parse_current_tag_info(QNetworkReply* reply) { if (entry_elem.isNull()) { FLOG_WARN("first xml entry of current version: %1 is null", current_version_); - version_meta_data_.current_commit_hash_publish_in_remote = false; return; } @@ -187,7 +186,6 @@ void BKTUSVersionCheckTask::slot_parse_current_tag_info(QNetworkReply* reply) { if (title_elem.isNull() || id_elem.isNull() || published_elem.isNull()) { FLOG_WARN("illegal xml entry of structure of version: %1", current_version_); - version_meta_data_.current_commit_hash_publish_in_remote = false; return; } -- cgit v1.2.3