Jackarain / avplayer

一个基于FFmpeg、libtorrent的P2P播放器实现
421 stars 265 forks source link

video renderer的一点建议 #18

Closed wang-bin closed 11 years ago

wang-bin commented 11 years ago

我觉得现在的render_one_frame里做的事情太多了,很多代码重复的,比如计算rect什么的,而这个应该和特定的渲染方式无关,可以移到基类里,render_one_frame只要读取算好的rect就行。还有aspect_ratio这个函数没必要纯虚的,也只要基类里实现就行。

其他部分还没怎么看。

我也在写播放器相关的,有几个月了。最近加了几个renderer,OpenGL因为不懂的参考了你的代码,满不错的。也许以后可以参与你的项目

我的项目:https://github.com/wang-bin/QtAV

Jackarain commented 11 years ago

Good job!

嗯嗯, 你说的有道理, 计算宽高和aspect_ratio确实可以移出来做. 目前我的精力主要放在其它模块, 可能需要些时间才会去做这方面的调整. 当然更欢迎你能提交patch, 或直接参与.