[FFmpeg-devel] [PATCH] Add MD5 protocol

Måns Rullgård mans
Sun Jul 18 17:11:38 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?

I went and read RFC1738, and look what I found:

   Scheme names consist of a sequence of characters. The lower case
   letters "a"--"z", digits, and the characters plus ("+"), period
   ("."), and hyphen ("-") are allowed.

So we are actually violating the spec by _not_ allowing numbers.

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



More information about the ffmpeg-devel mailing list