Repository navigation
Conversation
Convert plain qDebug/qInfo/qWarning to categorized qCDebug/qCInfo/qCWarning with org.deepin.camera category, register log_rules DConfig listening before application creation, and add dconfig meta org.deepin.camera.json. 将普通日志宏改为带 org.deepin.camera 分类的 qCDebug/qCInfo/qCWarning, 在应用创建前注册 log_rules DConfig 监听,并新增 dconfig 元数据文件。 Log: 适配deepin-debug-config调试模式,支持动态调整日志级别 Influence: 可通过deepin-debug-config动态切换deepin-camera的日志输出级别
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: add-uos The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Sorry @add-uos, your pull request is larger than the review limit of 150,000 diff characters |
deepin pr auto review🤖 AI 代码审查报告📊 总体评价
🔍 详细分析1. 语法逻辑 ✅评价: 语法正确,逻辑清晰 ✅ 通过 潜在问题: 建议: [] 2. 代码质量 ✅评价: 代码结构清晰,注释完整 ✅ 通过 潜在问题: 建议: [] 3. 代码性能 ✅评价: 性能良好,资源使用合理 ✅ 通过 潜在问题: 建议: [] 4. 代码安全 🔒评价: 存在0个安全漏洞 ✅ 通过
安全漏洞详情: 建议: [] 💡 改进建议代码示例本报告由 AI 代码审查工具自动生成 |
|
本 PR(25 个文件合为 1 个 commit)已按要求拆分为 3 个独立 PR,每个 PR 一个 commit:
三个 PR 文件集无交集、均无合并冲突(#539 已 rebase 最新上游 master 解决 a11y 改动冲突),三 PR 依次合并后的最终状态已实测 Qt6 Release 构建通过。请转为评审 #538/#539/#540,本 PR 关闭。 |
概要 / Summary
适配 deepin-debug-config 调试模式,支持通过 DConfig 动态调整 deepin-camera 日志级别。
qDebug()/qInfo()/qWarning()调用(21 个源文件、743 处)转换为qCDebug/qCInfo/qCWarning(appLog),统一使用org.deepin.camera分类(ddlog.h),并补充对应头文件引入;注释掉的代码保持原样未改动logconfigread.cpp在应用实例创建前注册 DConfig(org.deepin.camera/org.deepin.camera,键log_rules)并监听valueChanged,运行中可实时切换日志级别;DConfig 不可用时回退环境变量src/assets/org.deepin.camera.json(键log_rules,默认*.debug=false;*.info=false;org.deepin.camera*.warning=true,public/readwrite),并从org.deepin.camera.encode.json移除旧的rules键src/CMakeLists.txt同步安装两个元数据文件(dtk_add_config_meta_files / dsg/configs 目录)验证 / Verification
org.deepin.camera,无 Qt 框架日志刷屏 ✓配套 debug-config 侧提交(deepin-camera.json + deepin-camera_debug.sh)见 deepin-debug-config 仓库 develop/snipe 分支。