[gstreamer-bugs] [Bug 349180] gst_parse_launch is not reentrant (or recursively callable)

[GStreamer (bugzilla.gnome.org)]

Do NOT reply to this via email (we are currently unable to handle email
responses and they WILL be discarded).

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=349180.
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

  GStreamer | gstreamer (core) | Ver: HEAD CVS


Sebastian Dröge changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #84824|0                           |1
        is obsolete|                            |




------- Comment #17 from Sebastian Dröge  2007-03-25 18:16 UTC -------
Created an attachment (id=85273)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=85273&action=view)
gst-parse.diff

Ok, here's a patch that does the required magic...
The HAVE_MT_SAVE_FLEX define is obsolete now as the parser sources that are
used are always generated by a new enough flex, thus the #ifdefs are removed.

The pre-generated sources are always updated if the flex/bison sources are
modified, thus every developer who intents to change the parser needs a new
enough flex and bison installed. The pre-generated sources should always be
committed after a change, otherwise the build bot will complain.

Can someone please review this patch?