Re: RayShade change

[James W. <ja...@fr...>]

Roger Holmes wrote:
> Could one of you please check in this change for me:
> 
> In RayShade, in RS_Rasterize.cpp, in RSRasterizer_Rasterize_RGB_Span_PM
> 
> Please change:
> 
> 			case kQ3PixelTypeRGB32:
> 			case kQ3PixelTypeARGB32:
> 				{
> 					unsigned int pixel;
> 					pixel = 	(0xFF << 24) |
> 								(rgbPixels[i][0] << 16) |
> 								(rgbPixels[i][1] << 8)  |
> 								(rgbPixels[i][2] << 0);
> 					*((unsigned int *)rowAddr) = pixel;
> 				}
> 				break;
> 
> 
> 
> 
> To:
> 
> 
> 
> 			case kQ3PixelTypeRGB32:
> 			case kQ3PixelTypeARGB32:
> 				{
> 					unsigned int pixel;
> #if QUESA_HOST_IS_BIG_ENDIAN
> 					pixel = 	(0xFF << 24) |
> 								(rgbPixels[i][0] << 16) |
> 								(rgbPixels[i][1] << 8)  |
> 								(rgbPixels[i][2] << 0);
> 					*((unsigned int *)rowAddr) = pixel;
> #else
> 					pixel = 	(0xFF << 0) |
> 								(rgbPixels[i][0] << 8)  |
> 								(rgbPixels[i][1] << 16) |
> 								(rgbPixels[i][2] << 24);
> 					*((unsigned int *)rowAddr) = pixel;
> #endif
> 				}
> 				break;
> 
> 
> 
> Many thanks.

Done.  I wonder if similar fixes are needed for 16-bit color?

-- 
   James W. Walker, Innoventive Software LLC
   <http://www.frameforge3d.com/>