In message <[EMAIL PROTECTED]> you wrote:

> Author: adamblokus
> Date: Thu Jun  5 07:20:32 2008
> New Revision: 4263
> 
> URL: http://source.netsurf-browser.org?rev=4263&view=rev
> Log:
> Added hadling images other than png and jpeg - with transparency.
> 
> +             rgb_buffer = (char*)malloc(3*img_width*img_height);
> +             alpha_buffer = (char*)malloc(img_width*img_height);
> +             if(!(rgb_buffer&&alpha_buffer)){
> +                     LOG(("Not enough memory?"));
> +                     return false;
> +             }

Minor comment: the if() body needs "free(rgb_buffer); free(alpha_buffer);"
as one of the two calls can fail but not necessary the other one.

John.
-- 
John Tytgat
[EMAIL PROTECTED]

Reply via email to