diff options
author | Michael Niedermayer <michaelni@gmx.at> | 2004-07-14 13:07:23 +0000 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2004-07-14 13:07:23 +0000 |
commit | 80d1c272be337ac679a32e6e1d6981e781767da0 (patch) | |
tree | 37e27b520c1570f589df6d83e21d6e76325e934d /doc | |
parent | bb0b93ba8114ed792163de956ccc1bd8f954e609 (diff) | |
download | ffmpeg-80d1c272be337ac679a32e6e1d6981e781767da0.tar.gz |
all non trivial functions, structurs and their member variables should be documented
Originally committed as revision 3313 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'doc')
-rw-r--r-- | doc/ffmpeg-doc.texi | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/doc/ffmpeg-doc.texi b/doc/ffmpeg-doc.texi index f62a0eabf9..4d21dd84f8 100644 --- a/doc/ffmpeg-doc.texi +++ b/doc/ffmpeg-doc.texi @@ -944,6 +944,7 @@ You can use libavcodec or libavformat in your commercial program, but @emph{any patch you make must be published}. The best way to proceed is to send your patches to the ffmpeg mailing list. +@node Coding Rules @section Coding Rules ffmpeg is programmed in the ISO C90 language with a few additional @@ -985,15 +986,20 @@ The presentation is the one specified by 'indent -i4 -kr'. Main priority in ffmpeg is simplicity and small code size (=less bugs). -Comments: for functions visible from other modules, use the JavaDoc -format (see examples in @file{libav/utils.c}) so that a documentation -can be generated automatically. +Comments: use the JavaDoc/Doxygen +format (see examples in @file{libavformat/utils.c}) so that a documentation +can be generated automatically. All non trivial functions should have a comment +above it explaining what the function does, even if its just one sentance. +All Structures and their member variables should be documented too. + fprintf and printf are forbidden in libavformat and libavcodec, please use av_log() instead. @section Submitting patches +First, (@pxref{Coding Rules}) above if you didn't yet. + When you submit your patch, try to send a unified diff (diff '-up' option). I cannot read other diffs :-) |