----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/74/ -----------------------------------------------------------
Review request for Viewer. Summary ------- Current code composites RGBA PNG images that contain a bKGD chunk down to RGB, discarding the alpha channel. This patch removes that code, since it contradicts purpose of the bKGD chunk as described in the PNG spec and as commonly used. This addresses bug VWR-24420. http://jira.secondlife.com/browse/VWR-24420 Diffs ----- doc/contributions.txt UNKNOWN indra/llimage/llpngwrapper.h UNKNOWN indra/llimage/llpngwrapper.cpp UNKNOWN Diff: http://codereview.secondlife.com/r/74/diff Testing ------- Tested uploading the 2 images attached to VWR-24420 with and without the patch. Before patch, "bad alpha.png" was uploaded as RGB, after patch, both images were uploaded as RGBA. Thanks, Thickbrick
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges