[gstreamer-bugs] [Bug 517985] [lame] compile error on Solaris, broken GST_ELEMENT_ERROR macro use

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=517985

  GStreamer | gst-plugins-ugly | Ver: 0.10.17

Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEEDINFO
            Summary|compile error with gstlame.c|[lame] compile error on
                   |on Solaris                  |Solaris, broken
                   |                            |GST_ELEMENT_ERROR macro use




------- Comment #1 from Tim-Philipp Müller  2008-02-22 15:06 UTC -------
Ouch, I guess you can tell a macro is really broken if even the creator of said
macro can't use it properly :)


Should be fixed in CVS now:

  2008-02-22  Tim-Philipp Müller  <tim at centricular dot net>

        * ext/lame/gstlame.c: (gst_lame_sink_setcaps):
          Fix broken GST_ELEMENT_ERROR macro, fixes compile with the Sun
          Workshop 12 compiler, but probably also crashes (#517985).

This is the change:

http://webcvs.freedesktop.org/gstreamer/gst-plugins-ugly/ext/lame/gstlame.c?r1=1.114&r2=1.115


I have no idea what causes those gettext warnings though. Any chance you could
have a look at that? (Maybe using #include <foo.h> helps? Although it shouldn't
really make a difference, I think)


-- 
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.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=517985.