GNOME Bugzilla – Bug 568468
gstgsmdec issue
Last modified: 2009-01-20 20:32:58 UTC
On Solaris we are applying the attached patch to gstgsmdec. Should this go upstream?
Created attachment 126857 [details] [review] patch
Looking at this patch, I'm guessing this is not right, and shouldn't be applied. I'm not sure why this patch was added to our builds but I think we should be using the system byte order rather than hardcoding to a specific endianness, right?
The patch is obviously wrong: - if the gsm decoder outputs big-endian always, then it needs changing in the template caps as well - if it outputs host-endian, then it's already doing the right thing. I don't know what the gsm decoder actually does, so I don't know which of these options is correct - I suspect we don't need to change anything though.
Thanks for the heads up. I removed this patch since it was clearly wrong. Closing this bug.