Skip to content

Commit c5d6a3b

Browse files
committed
refactor(sdk): 重构部分调用CloseAllStreams()函数的逻辑部分, 不再进行二次调用。
commit #88#90#91
1 parent addd713 commit c5d6a3b

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

sdk/server/server.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -798,9 +798,9 @@ func keytonePkgRouters(r *gin.Engine) {
798798

799799
// 在正式删除音频源文件之前, 需要先释放所有流的文件句柄, 因为在Win系统中, 不释放的话是没办法成功关闭的。
800800
keySound.CloseAllStreams()
801-
time.Sleep(10 * time.Millisecond)
802-
// 需要调用两次的原因是 -> 前端在单击ui中的删除按钮时的行为本身, 会增加一个额外的正在播放的声音流, 而由于sync.map天然的锁机制, 它并不会包含在上述的关闭流程中。
803-
keySound.CloseAllStreams()
801+
// time.Sleep(10 * time.Millisecond)
802+
// // 需要调用两次的原因是 -> 前端在单击ui中的删除按钮时的行为本身, 会增加一个额外的正在播放的声音流, 而由于sync.map天然的锁机制, 它并不会包含在上述的关闭流程中。
803+
// keySound.CloseAllStreams() // 由于CloseAllStreams()函数内部已通过升级变得足够可靠, 因此无需再进行二次调用。
804804

805805
// 删除音频源文件
806806
err := os.Remove(filepath.Join(audioPackageConfig.AudioPackagePath, audioPkgUUID, "audioFiles", arg.Sha256+arg.Type))
@@ -1492,9 +1492,10 @@ func keytonePkgRouters(r *gin.Engine) {
14921492
// 覆盖模式:删除现有目录
14931493
// * 在正式删除音频源文件之前, 需要先释放所有流的文件句柄, 因为在Win系统中, 不释放的话是没办法成功关闭的。
14941494
keySound.CloseAllStreams()
1495-
time.Sleep(10 * time.Millisecond)
1496-
// * 需要调用两次的原因是 -> 前端在单击ui中的删除按钮时的行为本身, 会增加一个额外的正在播放的声音流, 而由于sync.map天然的锁机制, 它并不会包含在上述的关闭流程中。
1497-
keySound.CloseAllStreams()
1495+
// time.Sleep(10 * time.Millisecond)
1496+
// // * 需要调用两次的原因是 -> 前端在单击ui中的删除按钮时的行为本身, 会增加一个额外的正在播放的声音流, 而由于sync.map天然的锁机制, 它并不会包含在上述的关闭流程中。
1497+
// keySound.CloseAllStreams() // 由于CloseAllStreams()函数内部已通过升级变得足够可靠, 因此无需再进行二次调用。
1498+
14981499
// * 正式删除现有目录
14991500
if err := os.RemoveAll(targetPath); err != nil {
15001501
ctx.JSON(http.StatusInternalServerError, gin.H{

0 commit comments

Comments
 (0)