Michel Dänzer wrote:
> - if (dstFormat == &_mesa_texformat_argb8888) {
> + if (!littleEndian || dstFormat == &_mesa_texformat_argb8888) {
In cases like this, wouldn't you want the exclusive or of the two
values, ie
((littleEndian && dstFormat == &_mesa_texformat_argb8888) ||
(!littleEndian && dstFormat == &_mesa_texformat_argb8888_rev))
I don't think it makes sense otherwise - you'll always end up using one
translation regardless of the texformat.
Keith
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
