Howard Chu wrote:
> Daniel Stenberg wrote:
>> 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...
>
> Yeah. OK, I'm fine with that, as long as users don't whine and ask "how come
> you don't support rtmpe?"
>
>> 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!
>
> <librtmp/rtmp.h> defines RTMP_LIB_VERSION. Since we currently don't support
> building as a shared library, I don't see any need (yet) for a runtime version
> query. I'll add this to the patch...
>
OK, try this instead.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html