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 616921 - [PLUGIN-MOVE] video parsers should be moved to -good
[PLUGIN-MOVE] video parsers should be moved to -good
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 774538 (view as bug list)
Depends on: 631200 631389 631814
Blocks:
 
 
Reported: 2010-04-27 08:07 UTC by Stefan Sauer (gstreamer, gtkdoc dev)
Modified: 2018-11-03 13:05 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stefan Sauer (gstreamer, gtkdoc dev) 2010-04-27 08:07:31 UTC
Plus:
- the element uses GST_BOILERPLATE
- the element has docs

Minus:
- there are no tests
- the naming is weired (gst_mp3parse_*, but GstMPEGAudioParse and GstMPEGAudioParseClass) - should the GstMPEGAudioParse be renamed to GstMP3Parse or the other way around. I vote for GstMP3Parse to keep the element name

Also there are no leagal issues around the parser, so it should leave -ugly.
Comment 1 Tim-Philipp Müller 2010-04-27 09:06:32 UTC
I agree that we should move this into good (same for the other parsers). The main reason it went into ugly back in the days was that it wasn't considered useful without decoders from ugly like 'mad', but that's not the case any longer now that GStreamer is used more for transmuxing, streaming, etc. 

However, I would like to see mp3parse use the parser baseclass that the other audio parsers from -bad use before moving it to -good.
Comment 2 Tim-Philipp Müller 2010-04-27 09:48:48 UTC
> - the naming is weird (gst_mp3parse_*, but GstMPEGAudioParse and
> GstMPEGAudioParseClass) - should the GstMPEGAudioParse be renamed to
> GstMP3Parse or the other way around. I vote for GstMP3Parse to keep the element
> name.

I think it should be GstMpegAudioParse and "mpegaudioparse". We can then also register "mp3parse" for backwards compatibility reasons, but I don't think we should let past mistakes (?) influence our future naming schemes. Or maybe GstMP123Parse? ;-)
Comment 3 Stefan Sauer (gstreamer, gtkdoc dev) 2010-04-27 11:02:14 UTC
Agree about the naming. How could we do the logistics for the baseparse porting?

1.) move baseparse from bad to good, then port mpegaudioparse and integrate to good.
2.) port mpegaudioparse and integrate to bad, then move everything to good?

I'll try to write a test in the meantime.
Comment 4 Tim-Philipp Müller 2010-04-27 11:14:07 UTC
I would add a new mpegaudioparse to -bad/gst/audioparsers/, and when we think it's good enough we can move that to -good and remove the mp3parse in -ugly. But putting a private baseparse.[ch] copy next to the -ugly mp3parse would work too, I just think the former may be easier.
Comment 5 Stefan Sauer (gstreamer, gtkdoc dev) 2010-04-27 11:26:40 UTC
Doh, right. When we have moved all the parsers to good, we can kill the one in ugly and add the mp3parse alias in good. Another point for calling the new one "mpegaudioparse" :)
Comment 6 Tim-Philipp Müller 2010-08-31 13:48:06 UTC
I think Jan has played around with baseparse + mp3 a bit and one of the issues that came up is that baseparse currently doesn't support reverse playback, but mp3parse does.
Comment 7 Jan Schmidt 2010-08-31 13:59:07 UTC
mp3parse only supports reverse playback, poorly, with the hacky patch in bug #619204

baseparse has no support for reverse playback at the moment at all, so it would be a regression to port it as that stands.

Implementing reverse playback support in baseparse would be a good thing to do, but needs someone with the time to do it.
Comment 8 Mark Nauwelaerts 2010-10-04 08:32:46 UTC
Note that mpegaudioparse (based on baseparse) has now arrived in -bad (outranking mp3parse.)  It only remains to test and/or review ...

Baseparse also has reverse playback support now.
Comment 9 Tim-Philipp Müller 2010-10-05 09:24:45 UTC
Re-purposing this bug as a general 'move audioparsers from -bad to -good' bug.
Comment 10 Mark Nauwelaerts 2011-01-28 11:56:18 UTC
Also including videoparsers in this (see also bug #622276, comment #20).
Comment 11 Tim-Philipp Müller 2011-04-14 13:04:11 UTC
audio parsers have been moved to -good now.
Comment 12 Sebastian Dröge (slomo) 2017-11-20 19:03:27 UTC
*** Bug 774538 has been marked as a duplicate of this bug. ***
Comment 13 GStreamer system administrator 2018-11-03 13:05:45 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/19.