[icecast-dev] Patches - Was: Stream metadata settings
msmith at xiph.org
Mon Jan 12 17:32:48 PST 2004
On Tuesday 02 December 2003 20:43, Melanie wrote:
> I've split it into three patches now and also made it fit to current CVS,
> with, I believe, Karl's changes for the double free bug.
> Order for the patches is:
> As for docs: How? What format? Any specific layout?
Since we've now (finally!) got 2.0 out the door, I'm now coming back to these
patches - hoping to actually get the functionality incorporated.
I've got a few questions, though:
1) Documentation: did you ever manage to write any? The options you added are
fairly straightforward for the most part, so this shouldn't be too hard.
2) The 'no-mount' option: why? Does this have any effect at all that isn't
handled by setting max-listeners to zero?
3) In the multilevel fallbacks patch, you've added a comment saying
"Likely safe to do since there can only be one thread, ever", along with some
static local variables, in the source_find_mount() function. I'm not sure why
you said this, it's clearly incorrect: the source_* functions are called from
many different threads, and this is clearly not safe. This is easily fixed,
though, by passing the neccesary variables recursively, and splitting the
function into two.
I'll probably have more questions once I read the other two patches in detail.
Thanks again for your contributions,
<p>--- >8 ----
List archives: http://www.xiph.org/archives/
icecast project homepage: http://www.icecast.org/
To unsubscribe from this list, send a message to 'icecast-dev-request at xiph.org'
containing only the word 'unsubscribe' in the body. No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.
More information about the Icecast-dev