[FFmpeg-devel] [PATCH] DES en-/decryption
Diego Biurrun
diego
Sun Oct 14 11:29:21 CEST 2007
On Sun, Oct 14, 2007 at 09:05:03AM +0200, Reimar D?ffinger wrote:
> On Sun, Oct 14, 2007 at 02:05:36AM +0200, Aurelien Jacobs wrote:
> > On Sat, 13 Oct 2007 22:15:42 +0200
> > Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> > > --- libavutil/des.h (revision 0)
> > > +++ libavutil/des.h (revision 0)
> > > @@ -0,0 +1 @@
> > > +uint64_t ff_des_encdec(uint64_t in, uint64_t key, int decrypt);
> >
> > I think you should add multiple inclusion protection.
>
> Why? As long as there are only function prototypes there should not be
> any reason to use them.
There are only function prototypes in the file *now*. But in the future
when something else gets added the probability of the multiple inclusion
guards being forgotten is quite high. The solution is to waste a few
bytes now and never worry about possible future bugs.
Diego
More information about the ffmpeg-devel
mailing list