cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] librtmp support

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Wed, 12 May 2010 16:17:47 +0200 (CEST)

On Wed, 12 May 2010, Howard Chu wrote:

> OK. That one-line-per-ifdef style seems a bit excessive, but here you go.

Right, if there's nothing that can separate them then there's no point in
splitting them up. But further, perhaps we could just view them all as 'rtmp'
to make things easier? The supported protocols are just info, they don't have
to exactly list all the byte sequences that can start a URL that we support.
Or should we have that? I can't but to help to think that the rtmp* list
becomes a bit excessive...

It would also make sense to add librtmp/[version] to curl_version() like the
other libs are. And if you don't have a way to (dynamically) extract librtmp's
version, then I think you should really should consider adding one!

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2010-05-12