[PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-12-04 Thread Ilia Alshanetsky
iliaa Wed Dec 4 12:16:19 2002 EDT Modified files: /php4/ext/gd/libgd gd_topal.c Log: Fixed a memory leak inside gdImageTrueColorToPalette(). Index: php4/ext/gd/libgd/gd_topal.c diff -u php4/ext/gd/libgd/gd_topal.c:1.9 php4/ext/gd/libgd/gd_topal.c:1.10 ---

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Markus Fischer
On Thu, Oct 17, 2002 at 12:06:30AM -0700, Rasmus Lerdorf wrote : > > > So you are saying you want to make it optional? Because as it is there is > > > no way to not give a value. > > > > Exactly. > > I don't see any really compelling reason to do that. When you convert a > truecolor image

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Rasmus Lerdorf
> > So you are saying you want to make it optional? Because as it is there is > > no way to not give a value. > > Exactly. I don't see any really compelling reason to do that. When you convert a truecolor image to a palette-based one, not providing the size of this palette doesn't make any

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Markus Fischer
On Wed, Oct 16, 2002 at 11:25:14PM -0700, Rasmus Lerdorf wrote : > > On Wed, Oct 16, 2002 at 11:12:25PM -0700, Rasmus Lerdorf wrote : > > > It's not an optional arg though. So by leaving it off, do you mean if an > > > invalid value is given? I don't like changing args like that. > > > > Ye

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Rasmus Lerdorf
> On Wed, Oct 16, 2002 at 11:12:25PM -0700, Rasmus Lerdorf wrote : > > It's not an optional arg though. So by leaving it off, do you mean if an > > invalid value is given? I don't like changing args like that. > > Yes I know it's not optional. But having it default to 256 > colors makes

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Markus Fischer
On Wed, Oct 16, 2002 at 11:12:25PM -0700, Rasmus Lerdorf wrote : > It's not an optional arg though. So by leaving it off, do you mean if an > invalid value is given? I don't like changing args like that. Yes I know it's not optional. But having it default to 256 colors makes sense if t

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Rasmus Lerdorf
It's not an optional arg though. So by leaving it off, do you mean if an invalid value is given? I don't like changing args like that. -Rasmus On Thu, 17 Oct 2002, Markus Fischer wrote: > On Wed, Oct 16, 2002 at 09:52:53PM -0700, Rasmus Lerdorf wrote : > > It's not an issue in the external gd

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Markus Fischer
On Wed, Oct 16, 2002 at 09:52:53PM -0700, Rasmus Lerdorf wrote : > It's not an issue in the external gd since this function doesn't work at > all there. They had colorsWanted hardcoded to 256. Ah! Ok, another suggestion. Wouldn't it make sense to have the last parameter default to

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Rasmus Lerdorf
It's not an issue in the external gd since this function doesn't work at all there. They had colorsWanted hardcoded to 256. -Rasmus On Thu, 17 Oct 2002, Markus Fischer wrote: > Just a sidenote: I think if this would also be fixed in > gd/gd.c then we could prevent a segfault with the e

Re: [PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Markus Fischer
Just a sidenote: I think if this would also be fixed in gd/gd.c then we could prevent a segfault with the external gd library too. On Wed, Oct 16, 2002 at 10:34:45PM -, Rasmus Lerdorf wrote : > rasmusWed Oct 16 18:34:45 2002 EDT > > Modified files:

[PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-16 Thread Rasmus Lerdorf
rasmus Wed Oct 16 18:34:45 2002 EDT Modified files: /php4/ext/gd/libgd gd_topal.c Log: Fix for bug #19941 Index: php4/ext/gd/libgd/gd_topal.c diff -u php4/ext/gd/libgd/gd_topal.c:1.7 php4/ext/gd/libgd/gd_topal.c:1.8 --- php4/ext/gd/libgd/gd_topal.c:1.7

[PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-06 Thread Rasmus Lerdorf
rasmus Sun Oct 6 13:47:51 2002 EDT Modified files: /php4/ext/gd/libgd gd_topal.c Log: Fix for bug 19781 Index: php4/ext/gd/libgd/gd_topal.c diff -u php4/ext/gd/libgd/gd_topal.c:1.6 php4/ext/gd/libgd/gd_topal.c:1.7 --- php4/ext/gd/libgd/gd_topal.c:1.6S

[PHP-CVS] cvs: php4 /ext/gd/libgd gd_topal.c

2002-10-05 Thread Rasmus Lerdorf
rasmus Sun Oct 6 03:19:08 2002 EDT Modified files: /php4/ext/gd/libgd gd_topal.c Log: Ok, perhaps it's too late at night, but I see absolutely no reason for this chunk of code here. Why is it writing a palette.map file to disk in the middle of this functio