← Back to team overview

phatch-dev team mailing list archive

Re: [Merge] lp:~nadiana/phatch/bug_fix_487435 into lp:phatch

 

Review: Needs Information
Shouldn't this line:
del image.info['transparency']

be:
del img.info['transparency']

Is the same bug also not present in remove_alpha? I think functions which returns copies should not modify the original image, but maybe I've misunderstood something.

What is the reason that in put_alpha 1 and F images are converted to RGBA instead of LA? Is LA support broken in PIL? Just curious.

For the rest it seems good to me. 

Thanks for your work!

-- 
https://code.launchpad.net/~nadiana/phatch/bug_fix_487435/+merge/20311
Your team Phatch Developers is subscribed to branch lp:phatch.



References