[FFmpeg-devel] [PATCH] add multiple inclusion guard note to policy

Michael Niedermayer michaelni
Mon Oct 15 11:44:22 CEST 2007


On Mon, Oct 15, 2007 at 10:56:41AM +0200, Diego Biurrun wrote:
> On Mon, Oct 15, 2007 at 10:45:36AM +0200, Diego Biurrun wrote:
> > All header files should come with multiple inclusion guards.  Here is a
> > patch for the policy.
> .. I guess I have become the undisputed king of forgotten attachments,
> what a crown to wear ..
> Diego AKA scatterbrain #1

iam fine with adding this but i do not think that such rather trivial 
rules belong into the policy

rules like, use/keep alphabetical ordering where possible, add multiple
inclusion guards on headers, add proper license headers, update the changelog
and docs for new codecs would fit much better into the coding rules
section where also doxygen comments are mentioned or another section ...

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071015/3488a738/attachment.pgp>

More information about the ffmpeg-devel mailing list