[Ffmpeg-cvslog] r6161 - in trunk: libavcodec/dv.c libavcodec/dvdata.h libavformat/avformat.h libavformat/dv.c tests/ffmpeg.regression.ref tests/libav.regression.ref tests/rotozoom.regression.ref

[Michael Niedermayer]

Hi

On Mon, Sep 04, 2006 at 01:21:38PM +0200, Guillaume POIRIER wrote:
> Hi,
> 
> On 9/4/06, Michael Niedermayer <michaelni at gmx.at> wrote:
> >Hi
> >
> >On Mon, Sep 04, 2006 at 05:33:12AM +0200, romansh wrote:
> >> Author: romansh
> >> Date: Mon Sep  4 05:33:11 2006
> >> New Revision: 6161
> >>
> >> Modified:
> >>    trunk/libavcodec/dv.c
> >>    trunk/libavcodec/dvdata.h
> >>    trunk/libavformat/avformat.h
> >>    trunk/libavformat/dv.c
> >>    trunk/tests/ffmpeg.regression.ref
> >>    trunk/tests/libav.regression.ref
> >>    trunk/tests/rotozoom.regression.ref
> >>
> >> Log:
> >>
> >>   * Restructuring the division of labor between DV codec and DV format
> >>     [ Based on a patch by Brian Brice (bbrice at newtek dot com) ]
> >
> >[...]
> >> +enum dv_pack_type {
> >> +     dv_header525     = 0x3f, /* see dv_write_pack for important 
> >details on */
> >> +     dv_header625     = 0xbf, /* these two packs */
> >
> >not doxygen compatible comment
> 
> FWIW, it should be:
> 
> dv_header525     = 0x3f, ///< see dv_write_pack for important details on ...
> 
> I don't know how you can refer to the 2 packs as a whole without
> either copying/pasting the comment 2 times

i think 

//@{
and
//@}

will do that ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is