Add application/ogg and audio/ogg as MIME types for Vorbis streams

[bugs@…]

Currently stream/network.c only recognizes "application/x-ogg" as a valid MIME type for Vorbis streams.

It should also include "application/ogg" (as for RFC 3534 and www.vorbis.com/faq, currently the most used MIME type for Vorbis streams as it's the one used by Icecast) and "audio/ogg" (as for newer RFC 5334, used for example in Music Player Daemon)

[compn]

is there a stream or file that does not play correctly due to this ?

[bugs@…]

Yes, all those streams that send tag updates between songs. For example:

​http://ice.sess.nu:8000/classic.ogg

When a song change arrives, the stream is reset in order to send the title change, but mplayer doesn't play the next substream, instead writing "Cache not filling!".

This only happens with the LAVF demuxer (i.e, with "application/ogg" or "audio/ogg" MIME types). When the internal mplayer OGG demuxer is in use (only with "application/x-ogg" streams), there's no such a problem and the next substream is correctly played on song changes.

Perhaps this should be actually addressed in the LAVF demuxer, but forcing the internal demuxer doesn't look like a bad solution.

[reimar]

I am quite sure the lavf demuxer is long since fixed, and indeed that stream plays just fine across song switches.
While those other mime types should probably be added, they rather should be added to default to the lavf demuxer now I think.

[compn]

lavf (and all of ffmpeg) doesnt keep track of mime types. so since the sample works, lets just close this bug. i dont think we even need those mime types in our code either? i wonder when they were used? hmm.

if you have a sample stream that still shows a problem, please report and reopen bug!