Closed add-uos closed 3 months ago
关键摘要:
RecordTimePlugin::clear()
函数中,直接修改了m_timeWidget->setting()->setValue()
,可能会导致与预期不符的行为,建议通过方法来修改配置。TimeWidget::~TimeWidget()
中,对m_setting
的访问应该在m_setting
不为nullptr
的前提下进行,以避免潜在的空指针解引用错误。TimeWidget::setting()
函数中,返回的m_setting
指针应该检查是否为nullptr
,以避免在调用deleteLater()
时出现未定义行为。TimeWidget::paintEvent()
中,qInfo()
的调用应该在调试模式下使用,以避免生产环境中输出日志。TimeWidget::start()
函数中,m_baseTime
的初始化应该在m_setting->value(RECORDER_TIME_STARTCONFIG).toTime()
不为QTime(0,0,0)
时进行,以避免未定义行为。TimeWidget::stop()
函数中,m_timerCount
的初始化应该在m_setting->value(RECORDER_TIME_STARTCONFIG).toTime()
不为QTime(0,0,0)
时进行,以避免未定义行为。是否建议立即修改: 是
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: add-uos, lzwind
The full list of commands accepted by this bot can be found here.
/merge
fix the status bar display error time
Log: fix the status bar display error time Bug: https://pms.uniontech.com/bug-view-262791.html