Skip to content

fix(ffmpeg): 恢复 #171 原生会话串行化供重新审阅 - #179

Closed
3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-171-ffmpeg-sessions
Closed

3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-171-ffmpeg-sessions

Conversation

@3316891527

@3316891527 3316891527 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

变更说明 / Description

FFmpeg 执行与 FFprobe 共用原生会话锁,避免并发访问 FFmpegKit 会话状态;媒体转换复用首次探测结果,减少重复探测。

背景与动机 / Context and motivation

维护者要求撤销 #170–#176 并恢复 open 审阅。已合并的原 PR 无法重开,本 PR 对应原 #171。未经维护者明确允许不得合并或启用自动合并。

改动范围 / Changes

完整恢复 #171 的 3 文件补丁:FFmpegUtil、StandardFFmpegTool 和对应回归测试,保留当前包结构适配。

兼容性与风险 / Compatibility and risks

原生会话将串行执行,并发请求可能等待同一把锁。当前暂以撤销 PR #177 的分支为目标,仅展示本项差异;待 #177 获准进入 dev 后应将目标改为 dev 并重新检查,不能合入临时基准分支。

关联 Issue / Related issue

原 #171;撤销与恢复审阅要求见 #177。

验证方式 / Verification

初次重建时已逐项核对原完整合并补丁。本轮在清理历史空白并补齐 CI 的共同回滚基准上重放该项,保持独立功能差异。当前 PR 启用 ci:full-android,要求固定合并候选实际运行 assembleDebug 与 :app:testDebugUnitTest,Candidate checks 必须检查两项真实成功。新的结果以本 PR Checks 为准;历史 CI 不代替本轮构建和测试。

证据 / Evidence

原实现及历史检查见 #171;当前回滚基准见 #177。

检查清单 / Checklist

  • 已记录静态核验及未运行项原因。
  • 保留原修复、回归、分支与备份。
  • 待 revert: 撤销 PR #170–#176,恢复合入前代码 #177 获准合入后将目标改为 dev。
  • 本轮固定候选 assembleDebug、testDebugUnitTest 与 Candidate checks 实际成功。
  • 等待维护者明确批准合并。

本项已随 集成 PR #191 合入 dev,并经 发布同步 PR #193 同步到 main。原来源提交 4d7b99e82649aa5ae434096784103313a9765360 已核对保留。

本 PR 原目标为 fix/revert-pr-170-176;由于 dev 已包含全部来源提交,GitHub 拒绝将目标改到 dev。关闭此重复待合并条目,代码与原始审阅记录均保留。

@3316891527 3316891527 added the ci:full-android Require assembleDebug and Android JVM tests for this PR candidate label Nov 9, 2026
Reapply the complete original merge diff from PR #171 on the reviewed rollback base.
Original merge: 7f83633

Keep this work open for maintainer review. No merge or auto-merge without explicit permission.
@3316891527
3316891527 force-pushed the fix/reopen-pr-171-ffmpeg-sessions branch from c4752b0 to 4d7b99e Compare November 9, 2026 03:16
@3316891527 3316891527 closed this Nov 10, 2026
@3316891527
3316891527 deleted the fix/reopen-pr-171-ffmpeg-sessions branch November 10, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full-android Require assembleDebug and Android JVM tests for this PR candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant