From a6f8ad435c7e2fad0aa4d6c313c94f695da7dc89 Mon Sep 17 00:00:00 2001 From: shuaijie Date: Tue, 30 Jul 2024 17:25:19 +0800 Subject: [PATCH] fix: unvail "error" log output due to app_error list Log: modify "error" to unsucess in logdata --- deepin-system-monitor-main/3rdparty/include/closestream.h | 4 ++-- deepin-system-monitor-main/common/common.h | 2 +- deepin-system-monitor-main/dbus/dbusalarmnotify.cpp | 4 ++-- .../gui/dialog/systemprotectionsetting.cpp | 4 ++-- deepin-system-monitor-main/process/process.cpp | 1 + deepin-system-monitor-main/service/service_manager.cpp | 2 +- deepin-system-monitor-main/system/netif_packet_capture.cpp | 2 +- helper.hpp | 5 ++++- 8 files changed, 14 insertions(+), 10 deletions(-) diff --git a/deepin-system-monitor-main/3rdparty/include/closestream.h b/deepin-system-monitor-main/3rdparty/include/closestream.h index a674da1c..0973d740 100644 --- a/deepin-system-monitor-main/3rdparty/include/closestream.h +++ b/deepin-system-monitor-main/3rdparty/include/closestream.h @@ -67,9 +67,9 @@ close_stdout(void) { if (flush_standard_stream(stdout) != 0 && !(errno == EPIPE)) { if (errno) - warn(_("write error")); + warn(_("write unsuccess")); else - warnx(_("write error")); + warnx(_("write unsuccess")); _exit(CLOSE_EXIT_CODE); } diff --git a/deepin-system-monitor-main/common/common.h b/deepin-system-monitor-main/common/common.h index 5701effb..49150760 100644 --- a/deepin-system-monitor-main/common/common.h +++ b/deepin-system-monitor-main/common/common.h @@ -148,7 +148,7 @@ namespace error { inline void print_errno(decltype(errno) e, const QString msg) { - qCWarning(app) << QString("Error: [%1] %2, ").arg(e).arg(strerror(e)) << msg; + qCWarning(app) << QString("unsuccess: [%1] %2, ").arg(e).arg(strerror(e)) << msg; } } // namespace error diff --git a/deepin-system-monitor-main/dbus/dbusalarmnotify.cpp b/deepin-system-monitor-main/dbus/dbusalarmnotify.cpp index 38eba532..e468a892 100644 --- a/deepin-system-monitor-main/dbus/dbusalarmnotify.cpp +++ b/deepin-system-monitor-main/dbus/dbusalarmnotify.cpp @@ -82,8 +82,8 @@ void DBusAlarmNotify::showAlarmNotify(QString topic, QString msg, int timeout) QDBusMessage replyMsg = QDBusConnection::sessionBus().call(ddeNotify); if (replyMsg.type() == QDBusMessage::ErrorMessage) { - qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde notify dbus method call fail , error name :" - << replyMsg.errorName() << " , error msg :" << replyMsg.errorMessage(); + qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde notify dbus method call unsuccess , unsuccess name :" + << replyMsg.errorName() << " , unsuccess msg :" << replyMsg.errorMessage(); QString cmd = QString("gdbus call -e -d org.deepin.SystemMonitorDaemon -o /org/deepin/SystemMonitorDaemon -m org.deepin.SystemMonitorDaemon.setAlaramLastTimeInterval 0"); QProcess::startDetached(cmd); } else { diff --git a/deepin-system-monitor-main/gui/dialog/systemprotectionsetting.cpp b/deepin-system-monitor-main/gui/dialog/systemprotectionsetting.cpp index a6d20bd1..deadc0a1 100644 --- a/deepin-system-monitor-main/gui/dialog/systemprotectionsetting.cpp +++ b/deepin-system-monitor-main/gui/dialog/systemprotectionsetting.cpp @@ -486,7 +486,7 @@ void SystemProtectionSetting::onMessgaeSetting(QVariant value) QDBusMessage replyMsg = QDBusConnection::sessionBus().call(showDDEControlCenterPage); if (replyMsg.type() == QDBusMessage::ErrorMessage) { - qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde control center dbus method call fail , error name :" + qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde control center dbus method call unsuccess , unsuccess name :" << replyMsg.errorName() << " , error msg :" << replyMsg.errorMessage(); } @@ -514,7 +514,7 @@ void SystemProtectionSetting::onMessgaeSetting(QVariant value) QDBusMessage replyMsg = QDBusConnection::sessionBus().call(showDDEControlCenter); if (replyMsg.type() == QDBusMessage::ErrorMessage) { - qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde control center dbus method call fail , error name :" + qCWarning(app) << __FUNCTION__ << __LINE__ << ", dde control center dbus method call unsuccess , unsuccess name :" << replyMsg.errorName() << " , error msg :" << replyMsg.errorMessage(); } } diff --git a/deepin-system-monitor-main/process/process.cpp b/deepin-system-monitor-main/process/process.cpp index b47b204e..0a8ac5f0 100644 --- a/deepin-system-monitor-main/process/process.cpp +++ b/deepin-system-monitor-main/process/process.cpp @@ -597,6 +597,7 @@ void Process::readIO() // scan each line while (fgets(buf, bsiz, fp.get())) { + if (errno != 0) return; if (!strncmp(buf, "read_bytes", 10)) { sscanf(buf + 12, "%llu", &d->read_bytes); } else if (!strncmp(buf, "write_bytes", 11)) { diff --git a/deepin-system-monitor-main/service/service_manager.cpp b/deepin-system-monitor-main/service/service_manager.cpp index 4f05be1e..25caffaa 100644 --- a/deepin-system-monitor-main/service/service_manager.cpp +++ b/deepin-system-monitor-main/service/service_manager.cpp @@ -61,7 +61,7 @@ static bool setServiceEnable(const QString &servieName, bool enable, QString &er } if (!errorString.isEmpty()) { - qCWarning(app) << QString("Set service %1 failed, error %1").arg(enable ? "enable" : "disable").arg(errorString); + qCWarning(app) << QString("Set service %1 failed, unsuccess %1").arg(enable ? "enable" : "disable").arg(errorString); return false; } else { qCDebug(app) << QString("Set service %1 ret: %2").arg(enable ? "enable" : "disable").arg(retMsg.value()); diff --git a/deepin-system-monitor-main/system/netif_packet_capture.cpp b/deepin-system-monitor-main/system/netif_packet_capture.cpp index ef59de45..b0080a39 100644 --- a/deepin-system-monitor-main/system/netif_packet_capture.cpp +++ b/deepin-system-monitor-main/system/netif_packet_capture.cpp @@ -93,7 +93,7 @@ bool NetifPacketCapture::hasDevIP() //获取网络IP(IPv4) if (ioctl(sockfd, SIOCGIFADDR, &ifr) == -1) { close(sockfd); - //qCDebug(app)<<"ioctl error!"; + //qCDebug(app)<<"ioctl unsuccess!"; return false; } diff --git a/helper.hpp b/helper.hpp index 7e853800..2d015223 100644 --- a/helper.hpp +++ b/helper.hpp @@ -4,10 +4,13 @@ // SPDX-License-Identifier: GPL-3.0-or-later #pragma once +#include "ddlog.h" #include #include +using namespace DDLog; + namespace common{ class SystemState { @@ -148,7 +151,7 @@ static inline const SystemState &systemInfo() static SystemState *state = nullptr; if (!state) { const auto version = DTK_CORE_NAMESPACE::DSysInfo::majorVersion(); - qInfo() << "Running desktop environment version is:" << version << ", versionNumber:" << version.toLong(); + qCInfo(app) << "Running desktop environment version is:" << version << ", versionNumber:" << version.toLong(); if (!version.isEmpty() && version.toLong() <= 20) { state = new V20SystemState(); state->updateVersion(true);