已合并
soundpool读取wav格式文件逻辑优化 #3513
AtomGit-Bot创建于 2024年6月21日
soundpool读取wav格式文件逻辑优化 #3513
已合并
AtomGit-Bot创建于 2024年6月21日
refs/pull/3513/head合入到master
AtomGit-Bot
2024年6月21日

soundpool读取wav格式文件逻辑优化

likedislike
Pull Request已成功合入, 合并人@
(感谢 AtomGit-Bot 的贡献)
AAtomGit-Bot
2024年6月21日 指派了 袁景华 参与评审
AAtomGit-Bot
2024年6月21日 指派了 liuyuehua 参与评审
AAtomGit-Bot
2024年6月21日 指派了测试人openharmony_ci
AAtomGit-Bot
2024年6月21日 关联了 openharmony/multimedia_player_framework Issue IA52XS
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   waiting_on_author 标签
openharmony_ci
openharmony_ci成员
2024年6月21日 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   dco检查成功 标签
openharmony_ci
openharmony_ci成员
2024年6月21日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【dayu200_xts_precise编译, dayu200_tdd编译, dayu200测试, 格式化检查, dayu200_xts编译, dayu200编译】,预计在60分钟内完成,门禁结果会同步发送到码云注册邮箱。您可以通过如下链接跟踪门禁进展:http://ci.openharmony.cn/workbench/cicd/detail/6674ee6d64650f998be16fd1/runlist

likedislike
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   格式化检查成功 标签
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   编译成功 标签
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   静态检查成功 标签
openharmony_ciopenharmony_ci成员
2024年6月21日 测试通过
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   冒烟测试成功 标签
openharmony_ci
openharmony_ci成员
2024年6月21日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://ci.openharmony.cn/workbench/cicd/detail/6674ee6d64650f998be16fd1/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 dayu200 success success >>>
2 dayu200_xts success NA >>>
3 dayu200_tdd success NA >>>
4 format_check success NA >>>
5 dayu200_xts_precise success(IGNORE) NA >>>

likedislike
openharmony_ciopenharmony_ci成员
2024年6月21日 移除了   waiting_on_author 标签
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   waiting_for_review 标签
openharmony_ci
openharmony_ci成员
2024年6月21日 评论:

您好,Committer @liuyuehua1 @acefighteryuan ,请分配检视人员检视该PR,可以通过命令"assign [@someone_gitee_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @liuyuehua1 @acefighteryuan . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_gitee_id]", or you can comment "assign" to review the PR by yourself.

likedislike
zhangever
zhangever2024年6月21日进行代码检视1
test/unittest/soundpool_test/src/soundpool_unit_test.cpp
@@ -22,3 +22,3 @@
2222using namespace std;
2323 
24-static const std::string g_fileName[6] = {
24+static const std::string g_fileName[7] = {
zhangever
zhangever2024年6月21日评论:

这个大小可以不要吧

likedislike
zhangever
zhangever2024年6月21日进行代码检视1
frameworks/native/soundpool/sound_parser.h
@@ -191,3 +198,3 @@
191198 static constexpr int32_t AUDIO_SOURCE_TRACK_COUNT = 1;
192199 static constexpr int32_t AUDIO_SOURCE_TRACK_INDEX = 0;
193- static constexpr int64_t MIN_FD = 0;
200+ static constexpr int64_t minFd = 3;
zhangever
zhangever2024年6月21日评论:

常量命名用大写

likedislike
AtomGit-Bot2024年6月21日进行代码检视1
frameworks/native/soundpool/sound_parser.h
@@ -191,3 +198,3 @@
191198 static constexpr int32_t AUDIO_SOURCE_TRACK_COUNT = 1;
192199 static constexpr int32_t AUDIO_SOURCE_TRACK_INDEX = 0;
193- static constexpr int64_t MIN_FD = 0;
200+ static constexpr int64_t minFd = 3;
AtomGit-Bot2024年6月21日评论:

常量命名用大写

如上图,原来的命名方式codeCheck过不了,门禁给出的建议是使用驼峰的命名方式

likedislike
liuyuehua1liuyuehua1成员
2024年6月21日 审查通过
openharmony_ciopenharmony_ci成员
2024年6月21日 合并了Pull Request(通过 API 合并)
openharmony_ciopenharmony_ci成员
2024年6月21日 移除了   waiting_for_review 标签
openharmony_ciopenharmony_ci成员
2024年6月21日 添加了   merged 标签