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 110832 - [0.6.2] make avimux caps nego slightly better
[0.6.2] make avimux caps nego slightly better
Status: RESOLVED DUPLICATE of bug 116387
Product: GStreamer
Classification: Platform
Component: gst-plugins
git master
Other other
: Normal normal
: 0.7.x
Assigned To: Ronald Bultje
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-04-15 10:17 UTC by Joshua N Pritikin
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (797 bytes, patch)
2003-04-15 10:19 UTC, Joshua N Pritikin
none Details | Review

Description Joshua N Pritikin 2003-04-15 10:17:14 UTC
If the layer (2 or 3) is not indicated for an audio/x-mp3 stream the avimux
prints some cryptic warnings.  This patch makes it easier to understand
what is going wrong.
Comment 1 Joshua N Pritikin 2003-04-15 10:19:34 UTC
Created attachment 15731 [details] [review]
proposed patch
Comment 2 Ronald Bultje 2003-04-28 09:08:12 UTC
Minor nitpicking: I think we want to assume mp3 by default. Reason is
that most mp3 decoders can decode mp2 just fine, while the reverse is
less often true.

Apart from this - well noticed, thanks!
Comment 3 Christian Fredrik Kalager Schaller 2003-05-12 13:26:54 UTC
Assigning to Ronald for review/small change/commit
Comment 4 Ronald Bultje 2003-05-14 22:08:54 UTC
I've applied the patch to HEAD CVS (well, that was actually an
accident, but ohwell).

http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins/gst/avi/gstavimux.c.diff?r1=1.32&r2=1.33

We probably want this in 0.6.2 too, so marking for that.
Comment 5 Ronald Bultje 2003-05-18 16:41:13 UTC
this will take too long. We'll see about this later. Testing first.
Comment 6 Ronald Bultje 2003-05-30 23:21:37 UTC
HEAD now has a slightly improved AVI muxer. It's not 100% yet, but it
should do common things well now. We might want to backport it to
0.6.x fully, although it uses some features not present there (the
GstBuffer duration field, for example).
Comment 7 Ronald Bultje 2003-06-30 22:12:58 UTC
I've got the rest of this in my caps remake.

*** This bug has been marked as a duplicate of 116387 ***