After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 536067 - [matroska] Add support for real audio/video
[matroska] Add support for real audio/video
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal enhancement
: 0.10.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-06-01 15:01 UTC by Edward Hervey
Modified: 2008-08-26 22:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add realaudio/realvideo support (8.23 KB, patch)
2008-06-01 15:01 UTC, Edward Hervey
committed Details | Review

Description Edward Hervey 2008-06-01 15:01:04 UTC
Matroska can contain realaudio and realvideo stream, the following patch adds some support for it.
Comment 1 Edward Hervey 2008-06-01 15:01:55 UTC
Created attachment 111898 [details] [review]
Add realaudio/realvideo support
Comment 2 Sebastian Dröge (slomo) 2008-06-02 07:44:56 UTC
Looks good to me, you even care about the codec data :)
Comment 3 Edward Hervey 2008-06-03 09:56:19 UTC
Oh, forgot to mention something. That mdpr_data is the whole chunk of extra_data contained in a realmedia stream (which needs to be extracted by rmdemux).

The reason why I did that is because not all information contained in a mdpr is not parsed/extracted by rmdemux into individual caps information.

Extraction of it in rmdemux allows you to be able to *transmux* from a rm container to a mkv container, but wouldn't allow you to encode realaudio/realvideo streams that come from a non-rm container.... but i think the first case is the main reason you'd want to use real-in-matroska.
Comment 4 Edward Hervey 2008-08-25 14:17:38 UTC
2008-08-25  Edward Hervey  <edward.hervey@collabora.co.uk>

	* gst/matroska/matroska-demux.c: (gst_matroska_demux_send_event),
	(gst_matroska_demux_video_caps), (gst_matroska_demux_audio_caps):
	* gst/matroska/matroska-mux.c:
	(gst_matroska_mux_video_pad_setcaps),
	(gst_matroska_mux_audio_pad_setcaps), (gst_matroska_mux_finish):
	Add Real[Audio|Video] support to Matroska containers.
	It works fine for:
	* decoding real audio/video streams contained in mkv
	* 'transmuxing' real (.rm) files into .mkv files
	It will not work though for encoding real[audio/video] streams that
	don't contain the 'mdpr_data' extra data on the caps.
	The reason why this will not work is because I never intended to
	duplicate virtually all the 'mdpr' block creation into mkvmux.
	Fixes #536067