Bugzilla Bug 83996

>>> media-video/sswf-1.7.1 merged.
Please add the ~amd64 keyword to ebuild

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
media-video/sswf-1.7.1

Well, it builds, and it runs, but it seems to hardloop for me.  I don't know
how long these things are *suppose* to take, but I gave up after 10 minutes of
it taking 99.7% of my cpu to try and build one of the samples (new-anim).  I
can't mark tested under these circumstances.

*** Bug 83998 has been marked as a duplicate of this bug. ***

Adam: what did you do to test this package?

Here are the fixed functions for 64-bit that go into 
src/lib/libsswf_tags.c++


long TagBase::SIBitSize(long value)
{
        long    cnt;

        if(value < 0) {
                // a special case
                value =~ value;
        }
        cnt = 1;
        while(value > 0) {
                cnt++;
                value=value/2;
        }
        return cnt;
}


long TagBase::UIBitSize(unsigned long value)
{
        long    cnt;

        cnt = 0;
        do {
            cnt++;
            value=value/2;
        } while(value > 0);
        return cnt;
}

...and the code has been forwarded upstream. the same fix applies to both 1.7.1
and 1.7.2.

Created an attachment (id=52683) [details]
patch for sswf-1.7.1 and 1.7.2

Above patch fixed the "hardloop", I've wrote it, I've tested (examples
worked)it on both 32 and 64 bit. I can provide ebuilds if necessary. Please let
me know.

Created an attachment (id=53351) [details]
this fixes a quirk in a struct

long on 64bit platform is 8 bytes, int is 4, however on 32bit it's long is 4
bytes as is int. This fixes the struct so that it is using the proper size on
64bit platform.

Created an attachment (id=53353) [details]
sswf-1.7.1-r1_ebuild.patch

Patch to the 1.7.1-r1 ebuild attached. Can somebody mark it ~amd64, now that
I've fixed it up, and forwarded the changes to the author?!

Created an attachment (id=53908) [details]
sswf-1.7.1-r1.ebuild

This works for me now.	Attached is the acutal ebuild I used for this.	I also
used the two patches from this thread, referenced in the ebuild.

Thanks for the patches, they're in CVS now!