[FFmpeg-devel] [PATCH] Add MD5 protocol

Måns Rullgård mans
Sun Jul 18 17:07:52 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Sun, Jul 18, 2010 at 03:40:57PM +0100, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> 
>> > On Sun, Jul 18, 2010 at 12:31:10AM +0100, Mans Rullgard wrote:
>> >> This is a write-only protocol which computes the md5sum of data written,
>> >> and on close writes this to the designated output or stdout if none
>> >> is specified.  It can be used to test muxers without writing an actual
>> >> file.
>> >> ---
>> >>  libavformat/Makefile     |    1 +
>> >>  libavformat/allformats.c |    1 +
>> >>  libavformat/md5proto.c   |   92 ++++++++++++++++++++++++++++++++++++++++++++++
>> >>  3 files changed, 94 insertions(+), 0 deletions(-)
>> >>  create mode 100644 libavformat/md5proto.c
>> >
>> > ok
>> 
>> What about allowing numbers in protocol names?  
>
> you want to violate the specs?

What specs?  I see no reason why we must allow only rfc-conforming
names.  As long as all proper URLs are parsed correctly, there's no
harm in allowing a few extensions when convenient.  In fact, we
already do.

>> I think md5: is nicer
>> than mdfive:, and certainly sha256 is better than shatwofivesix...
>
> than maybe shadowfivesix ;)

Shadowfax...

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list