fix: the temporal performance of recording gif

linuxdeepin / deepin-screen-recorder

Screen recorder for DDE

GNU General Public License v3.0

104 stars 69 forks source link

fix: the temporal performance of recording gif #502

Closed add-uos closed 4 months ago

add-uos commented 4 months ago

the temporal performance of recording gif

Log: the temporal performance of recording gif Bug: https://pms.uniontech.com/bug-view-264145.html

deepin-ci-robot commented 4 months ago

deepin pr auto review

关键摘要:

移除了 "-vf" 选项可能会影响转码的质量，需要确认是否有替代方案或者转码需求是否变更。
使用了 path.replace("mp4", "gif")，需要确保路径和文件扩展名的处理是正确的，避免潜在的文件名错误。

是否建议立即修改:

是，需要确保转码质量不受影响，并且文件处理逻辑是正确的。

deepin-ci-robot commented 4 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, max-lvs

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/deepin-screen-recorder/blob/release/eagle/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment