[gstreamer-bugs] [Bug 584174] New: Problems with tvtime assembler

[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=584174

  GStreamer | gst-plugins-good | Ver: 0.10.26
           Summary: Problems with tvtime assembler
           Product: GStreamer
           Version: 0.10.26
          Platform: Other
        OS/Version: opensolaris
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: brian.cameron at sun.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: 2.25/2.26
   GNOME milestone: Unspecified


In the new 0.10.15 release of gst-plugins-good, the deinterlace plugin was
added.
The mmx.h assembler code fails to compile because some "movl" statements
include a ":" character.  In talking with the Solaris compiler team this is an
error, and the code compiles okay if the ":" characters are removed.

I also needed to change the function mm_support to be "inline int" instead of
"inline extern int" otherwise I get these errors:

        "/tmp/ube_sGAA.28938.ekaqH4", line 3809 : Multiply defined label:
"TryAMD"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3818 : Multiply defined label:
"TryCyrix"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3839 : Multiply defined label:
"EMMXSupported"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3843 : Multiply defined label: "AMD"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3858 : Multiply defined label:
"ThreeDNowSupported"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3862 : Multiply defined label:
"Intel"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3863 : Multiply defined label:
"MMXtest"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3872 : Multiply defined label:
"NotSupported1"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3876 : Multiply defined label:
"NotSupported2"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3880 : Multiply defined label:
"NotSupported3"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3884 : Multiply defined label:
"NotSupported4"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3888 : Multiply defined label:
"NotSupported5"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3892 : Multiply defined label:
"NotSupported6"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3896 : Multiply defined label:
"NotSupported7"
        "/tmp/ube_sGAA.28938.ekaqH4", line 3901 : Multiply defined label:
"Return"

Looking at the code, I think that simply changing it to "inline int" should be
reasonable.

Adding a patch which fixes these compile issues.


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