[FFmpeg-devel] [VOTE] Multiple inclusion guards in headers

Diego Biurrun diego
Mon Aug 18 08:59:50 CEST 2008


On Mon, Aug 18, 2008 at 08:49:31AM +0200, Reimar Doeffinger wrote:
> On Sun, Aug 17, 2008 at 07:17:22PM +0200, Michael Niedermayer wrote:
> > This is the second issue id like to see clarified
> > 
> > Do we require headers that do not need multiple inclusion guards out of
> > technical reasons to have multiple inclusion guards?
> > technical here is speed, compiler warnings or errors, or spec compliance
> > 
> > 
> > My vote is of course, no, as it makes the headers bigger and thus means
> > more to read aka worse readability.
> 
> I do not think the overhead of these is significant and avoids the annoyance
> of possibly having to add them later.
> I am against making a big deal about it though, this is nothing critical and
> while everyone should make an effort to remember adding them, forgetting
> it should not lead to flaming - IMO it might be a good idea to here "discard"
> the normal commit rules and allow everyone to add them without sending a patch first...

So you want to start handing out licenses for sloppyness?  That's one
hell of a slippery slope...

Diego




More information about the ffmpeg-devel mailing list