Closed hualet closed 3 weeks ago
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅
I have read the CLA Document and I hereby sign the CLA.
/check clacheck/CLAssistant
Deepin Obs Bot Starting find obs webhook event and retrigger!
https://github.com/linuxdeepin/deepin-terminal/blob/7a69abbf9cfbaf5be52531aae967d95a72fe3225/src/views/termwidget.cpp#L126 这里有设置最小的大小限制,小窗口下显示分屏多的话,下面的分屏终端显示会被遮盖。
recheck
I have read the CLA Document and I hereby sign the CLA.
@rb-union @ArchieMeng done!
关键摘要:
TermWidget::canSplit
函数中,使用了qDebug
宏,建议使用更合适的日志记录方法。TermWidgetPage::setSplitStyle
函数中,对QSplitterHandle
的设置重复,可以考虑重构以减少重复代码。TermWidgetPage::split
函数中,处理splitter
为nullptr
的情况时,没有检查term
是否为nullptr
,可能会导致空指针解引用。TermWidgetPage::closeSplit
函数中,对upSplit
的类型转换为QSplitter
后没有检查转换结果,可能会导致运行时错误。TermWidgetPage::closeSplit
函数中,对brother
的类型转换为TermWidget
后没有检查转换结果,可能会导致运行时错误。TermWidgetPage::closeSplit
函数中,对nextTerm
的类型转换为TermWidget
后没有检查转换结果,可能会导致运行时错误。TermWidgetPage::setCurrentTerminal
函数中,直接调用setFocus
可能不会按预期工作,因为setFocus
可能需要在合适的上下文中调用。TermWidgetPage::setCurrentTerminal
函数中,没有处理term
为nullptr
的情况,可能会导致未定义行为。是否建议立即修改: 是
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ArchieMeng, hualet
The full list of commands accepted by this bot can be found here.
a feature request from here: https://bbs.deepin.org/post/277237