Talk:Small FFmpeg Tasks

From MultimediaWiki
Jump to navigation Jump to search
The printable version is no longer supported and may have rendering errors. Please update your browser bookmarks and please use the default browser print function instead.

Patch Rules

should we link or quote some ffmpeg patch rules? e.g. tabs are forbidden, use diff -u or svn diff or (insert git command). it seems like this is a common mistake for SoC students. -- Compn 10:53, 29 March 2008 (EDT)

Well the mailing list page has a link to the MPlayer patch guidelines, but it's missing a few things such as:
  • No trailing whitespace
  • Must not break the regression tests
  • If it does change the regression tests, you must update regression.ref
  • Must check bounds on all input and must not allow overflow or other out-of-bounds condition
  • Use meaningful variable names
  • Globals/externs should have ff_ prefix to avoid naming conflicts
  • Doxygen comments
  • No breaking C standards (eg using underscore prefixes)
  • Keep { } tidy
  • No code duplication
  • Workarounds for compiler bugs (eg MSVC) must not break other platforms
Anything else? -- Mkhodor 13:18, 31 March 2008 (EDT)
er, the whole rule list is availible here http://ffmpeg.mplayerhq.hu/general.html#SEC20 , feel free to add it-- Compn 14:09, 31 March 2008 (EDT)

Delphine CIN?

But we already have it in FFmpeg - look at libavcodec/dsicinav.c and libavformat/dsicin.c --Kostya 01:43, 31 March 2008 (EDT)

Awesome. See, that's what this collaborative Wiki model is good for-- keeping me honest. :-) I'll get a test in FATE soon. --Multimedia Mike 02:05, 31 March 2008 (EDT)


new ffmpeg demuxers

from mplayer wishlist:

Port libmpdemux demuxers to libavformat or write your own from scratch.
  libmpdemux is considered deprecated and should eventually be removed.
  As of 2008-01-28, the following demuxers are missing from libavformat:
  1. TiVo (ty streams, not TiVo To Go)
  2. VIVO
  3. VQF
  4. XMMS
  5. libnemesi
  6. SL support for MPEG-TS
Vivo is already on the list. Feel free to add more if you can clearly outline the end goal and the steps toward achieving that goal. --Multimedia Mike 14:50, 4 April 2008 (EDT)