Closed Whale107 closed 3 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Whale107
The full list of commands accepted by this bot can be found here.
关键摘要:
DLogManagerPrivate
类的构造函数中使用了new
关键字,但没有看到相应的delete
操作,可能会导致内存泄漏。DLogManagerPrivate
类的成员变量使用了Q_DECLARE_PUBLIC
进行保护,但类中没有对应的Q_DECLARE_PRIVATE
,这可能是一个遗漏。DLogManagerPrivate
类的initLoggingRules
方法中使用了Q_D(DLogManager)
进行对象引用,但在类中没有看到DLogManager
的构造函数,这可能是一个逻辑错误。DLogManagerPrivate
类的isDefaultValue
方法被重载了,但参数类型为const QString &key
,建议改为QString key
以符合regex的用法。DLogManagerPrivate
类的updateLoggingRules
方法中使用了Q_D(DLogManager)
进行对象引用,但在类中没有看到DLogManager
的构造函数,这可能是一个逻辑错误。DLogManagerPrivate
类的initLoggingRules
方法中调用了updateLoggingRules
,但没有看到initConsoleAppender
和initRollingFileAppender
的类似调用,可能存在功能不完整。DLogManagerPrivate
类的m_dsgConfig
和m_fallbackConfig
使用了QScopedPointer
进行内存管理,但没有看到相应的reset
方法,可能会导致悬挂指针。DLogManagerPrivate
类的appendOrganizationAndApp
方法使用了Q_D(DLogManager)
进行对象引用,但在类中没有看到DLogManager
的构造函数,这可能是一个逻辑错误。DLogManagerPrivate
类的isDefaultValue
方法被重载了,但参数类型为const QString &key
,建议改为QString key
以符合regex的用法。DLogManagerPrivate
类的updateLoggingRules
方法中使用了Q_D(DLogManager)
进行对象引用,但在类中没有看到DLogManager
的构造函数,这可能是一个逻辑错误。DLogManagerPrivate
类的initLoggingRules
方法中调用了updateLoggingRules
,但没有看到initConsoleAppender
和initRollingFileAppender
的类似调用,可能存在功能不完整。DLogManagerPrivate
类的m_dsgConfig
和m_fallbackConfig
使用了QScopedPointer
进行内存管理,但没有看到相应的reset
方法,可能会导致悬挂指针。DLogManagerPrivate
类的appendOrganizationAndApp
方法使用了Q_D(DLogManager)
进行对象引用,但在类中没有看到DLogManager
的构造函数,这可能是一个逻辑错误。是否建议立即修改: 是
@18202781743 砍下dconfig 有没有遗漏的
@18202781743 砍下dconfig 有没有遗漏的
dde-app-services项目跟这个isDefaultValue 有关联, 是一起升么, DConfigFile里添加了对appid,subpath,filename的检查,