On Saturday 17 January 2009 00:02:09 ext Luiz Augusto von Dentz wrote: > Hi Siarhei, > > It seems gstreamer does have G_BYTE_ORDER, so you probably don't need > those #ifdef. I also notice some unnecessary white spaces in the of a > line. Hi, I did not verify that patch thoroughly (I only checked that it compiles) and also seems like I somehow forgot to pass it through 'checkpatch.pl'. Thanks for a good catch. Regarding G_BYTE_ORDER, you are probably right. I'm not very familiar with gstreamer yet, so somebody else could probably fix this stuff much faster than me. In any case, it probably makes sense to relay all the endian conversion related things to gstreamer and ALSA, and use only native byte order in SBC. This will simplify the code a bit and will make optimizing it easier. I'm quite worried about big endian systems, but I have plans to buy a PS3 home in about a month timeframe unless something extraordinary happens :) Best regards, Siarhei Siamashka -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
