[FFmpeg-devel] [Ffmpeg-devel] [PATCH] DV timecode

[Baptiste Coudurier]

Hi guys,

I'd like to resurrect this thread, I hope this won't cause flames this
time, and I'll address all comments.

Baptiste Coudurier wrote:
> Hi
> 
> 3 patches:
> 
> - add timecode.h to avoid code duplication for drop frame timecode
> adjustment

I moved code to utils.c in ff_timecode_drop_frame_adjust function, added
doxygen.

> - use new lavc timecode api to set timecode in DV. Regression tests
> change because now default behaviour is non drop frame timecode, you can
> still activate it by using -flags2 +drop_frame_timecode

This still apply, however I notice that I can move the code without
using the new adjust function, and will commit that first.

> - timecode helper function for ffmpeg to set timecode.

Removed the helper for now, I still use it in my own tree though,
because it is still very useful to me at least.

[...]

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dv_move_timecode_codec.patch
Type: text/x-diff
Size: 5765 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081009/e2696614/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: factorize_timecode_adjust.patch
Type: text/x-diff
Size: 3144 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081009/e2696614/attachment-0001.patch>