On Tue, Jun 30, 2020 at 08:31:39AM +0530, Gautam Ramakrishnan wrote: > On Tue, Jun 30, 2020 at 5:49 AM Michael Niedermayer > <mich...@niedermayer.cc> wrote: [...] > > > > > > > + if (temp > INT_MAX) > > > + return -1; > > > + ret = temp; > > > + } > > > + > > > + if (ret >= 0) > > > + return ret; > > > + return -1; > > > +} > > > + > > > > > +static int ff_pgx_decode_header(AVCodecContext *avctx, PGXContext * s, > > > > why does this have a ff_ prefix ? > I'll remove it. I was following pnmdec.c while writing this. Does the ff > prefix > indicate that it is a non-static function?
non statics need a ff, av, avpriv, ... prefix each has a slightly different case to be used in, see https://ffmpeg.org/developer.html#Naming-conventions thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB "I am not trying to be anyone's saviour, I'm trying to think about the future and not be sad" - Elon Musk
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".