[FFmpeg-devel] [PATCH 1/2] lavu/hash: Add support for SHA-2 512
michaelni at gmx.at
Sat Jun 15 23:38:58 CEST 2013
On Sat, Jun 15, 2013 at 04:10:44PM -0300, James Almer wrote:
> On 15/06/13 6:08 AM, Michael Niedermayer wrote:
> > On Sun, Jun 02, 2013 at 05:58:15PM -0300, James Almer wrote:
> >> Signed-off-by: James Almer <jamrial at gmail.com>
> > [...]
> >> diff --git a/libavutil/hash.h b/libavutil/hash.h
> >> index 5dd8712..9bf715e 100644
> >> --- a/libavutil/hash.h
> >> +++ b/libavutil/hash.h
> >> @@ -58,7 +58,7 @@ const char *av_hash_get_name(const struct AVHashContext *ctx);
> >> * with larger sizes will not be considered an ABI change and should not cause
> >> * your code to overflow a buffer.
> >> */
> >> -#define AV_HASH_MAX_SIZE 32
> >> +#define AV_HASH_MAX_SIZE 64
> > this needs a minor version bump
> This is not public API, though. The header is not being installed yet.
> Should i bump it anyway?
without a bump an application that needs "support for SHA-2 512"
has no means to declare a dependancy on a libavutil that supportd
And even worse libavformat itself couldnt declare a dependancy so
applications using libavformat couldnt be sure that the latest
libavformat supports SHA-2 512 because it could be linked to a
older libavutil (with same version but no SHA-2 512 in hash.*)
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel