ChinaFFmpeg

 找回密码
 立即注册

QQ登录

只需一步,快速开始

查看: 8182|回复: 6

ffplay的代码写的真的好烂好烂

[复制链接]
发表于 2020-4-17 11:52:22 | 显示全部楼层 |阅读模式
简直是烂到家了,全局变量满天飞,锁满天飞。你以为把全局变量封装到一个结构体就是一个变量了?
这种软件如何维护,为什么不能用C++,你确定你写的packet quene和frame quene一定就比std::quene好?
另外ffmpeg那些函数,接口改来改去的,1 2 3 4命名的函数到处都是。
很难想象,这种软件怎么做到行业第一的
回复

使用道具 举报

发表于 2020-4-17 15:43:27 | 显示全部楼层
ffplay并不是行业第一,ffplay ffmpeg ffprobe 你可以理解为是FFmpeg的使用示例,FFmpeg其实也不是最好的音视频处理框架,看您也是一名高手,祝早日写出一个超越FFmpeg的软件
回复 支持 反对

使用道具 举报

 楼主| 发表于 2020-4-17 16:40:29 | 显示全部楼层
孙悟空 发表于 2020-4-17 15:43
ffplay并不是行业第一,ffplay ffmpeg ffprobe 你可以理解为是FFmpeg的使用示例,FFmpeg其实也不是最好的音 ...

ffmpeg行业第一,这并没错啊。
我没说ffplay第一,我首先是吐槽这个例子写得烂,应该没有说错吧。
ffmpeg我确实是写不出来,但是写个类ffplay还是不难,参照ffplay的线程模式,写好了。
ffmpeg的框架,我不评论,因为我还没有仔细研究里面的代码,但是动不动就换接口确实不好。

这东西,早就应该用C++改造了,我不认为他写的这些智能指针能比c++11的unique_ptr和shared_ptr好。weak_ptr和shared_ptr这种组合不正好符合这种软件的要求吗?死抱着C语言不放,真不知道ffmpeg项目组怎么想的。
回复 支持 反对

使用道具 举报

 楼主| 发表于 2020-4-17 16:42:25 | 显示全部楼层
孙悟空 发表于 2020-4-17 15:43
ffplay并不是行业第一,ffplay ffmpeg ffprobe 你可以理解为是FFmpeg的使用示例,FFmpeg其实也不是最好的音 ...

ffmpeg行业第一,这并没错啊。
我没说ffplay第一,我首先是吐槽这个例子写得烂,应该没有说错吧。
ffmpeg我确实是写不出来,但是写个类ffplay还是不难,参照ffplay的线程模式,写好了。
ffmpeg的框架,我不评论,因为我还没有仔细研究里面的代码,但是动不动就换接口确实不好。

这东西,早就应该用C++改造了,我不认为他写的这些智能指针能比c++11的unique_ptr和shared_ptr好。weak_ptr和shared_ptr这种组合不正好符合这种软件的要求吗?死抱着C语言不放,真不知道ffmpeg项目组怎么想的。
回复 支持 反对

使用道具 举报

发表于 2020-4-17 17:47:06 | 显示全部楼层
abcdabcd 发表于 2020-4-17 16:42
ffmpeg行业第一,这并没错啊。
我没说ffplay第一,我首先是吐槽这个例子写得烂,应该没有说错吧。
ffmpeg ...

实不相瞒,我不喜欢C++,这对我来说是最好的反馈

如果你觉得自己C++好的话,用C++单独撸一个比FFmpeg优秀的就可以了嘛,在这里吐槽那么多没有啥意义,ffmpeg的开发者们不会汉语,您可以考虑发邮件到ffmpeg-devel@ffmpeg.org去吐槽,可能那里会有很多人给您一个合理的反馈
回复 支持 反对

使用道具 举报

 楼主| 发表于 2020-4-17 18:16:34 | 显示全部楼层
孙悟空 发表于 2020-4-17 17:47
实不相瞒,我不喜欢C++,这对我来说是最好的反馈

如果你觉得自己C++好的话,用C++单独撸一个比FFmpeg优 ...

我本想到ffmpeg的官方论坛去吐槽,无奈注册不了
回复 支持 反对

使用道具 举报

发表于 2020-4-17 18:22:43 | 显示全部楼层
abcdabcd 发表于 2020-4-17 18:16
我本想到ffmpeg的官方论坛去吐槽,无奈注册不了

那挺遗憾的,哈哈
回复 支持 反对

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

手机版|Archiver|ChinaFFmpeg

GMT+8, 2024-4-27 01:21 , Processed in 0.054005 second(s), 14 queries .

Powered by Discuz! X3.4

Copyright © 2001-2021, Tencent Cloud.

快速回复 返回顶部 返回列表