[FFmpeg-devel] [PATCH] rtmpdh: Initialize gcrypt before using it
michael at niedermayer.cc
Sun Jan 10 00:32:15 CET 2016
On Mon, Jan 04, 2016 at 02:39:41AM +0000, Ricardo Constantino wrote:
> On 3 January 2016 at 23:13, Jan Ekstrom <jeebjp at gmail.com> wrote:
> > Hi,
> > In general looks good, although it might look a bit weird for someone
> > as usually libraries have initialization functions called like that.
> > That said, this is what
> > https://gnupg.org/documentation/manuals/gcrypt/Initializing-the-library.html
> > recommends.
> > My only comment would be that we might want to set the parameter to
> > GCRYPT_VERSION instead of NULL, as we most probably care if the
> > library loaded matches our version (unless these versions change even
> > if API doesn't change).
> This is where it shows my ignorance but I don't know which of these is the
> best way of dealing with that.
> I tested all three and they all work with the linked sample.
we should pass the version number we need i think
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel