Is this patch possible to get in php 5.x? What should I improve so my
patch would get reviewed?
I would need it in php 5.0 my self. It would come to comercial webserver
so there is required _stable_ build which won't self build and patched
version. Specialy because this was first time I ever lo
On Wed, 6 Apr 2005, Steph wrote:
> erm - when was the decision to no longer maintain the .dsp files made?
Nobody did, but why maintain something outdated? Feel free to fix them
though...
Derick
--
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org
--
PHP Internals - P
Hi All,
I have come across a double free because of improper refcount
manipulation.
smileys;
}
}
$myts = new MyTextSanitizer();
$smiles =& $myts->getSmileys(); //calling by ref alone causes improper
refcount
$smiles = $myts->getSmileys(); //this does not cause improper refcount
?>
What is ha
On Wed, 6 Apr 2005, Kamesh Jayachandran wrote:
> Hi All,
> I have come across a double free because of improper refcount
> manipulation.
> class MyTextSanitizer
> {
> var $smileys=array()
> function MyTextSanitizer() {}
> function getSmileys()
> {
> return $this->smileys;
It happens in php-5.0.4 also.
With regards
Kamesh Jayachandran
On Wed, 6 Apr 2005 09:16:34 +0200 (CEST), "Derick Rethans"
<[EMAIL PROTECTED]> said:
> On Wed, 6 Apr 2005, Kamesh Jayachandran wrote:
>
> > Hi All,
> > I have come across a double free because of improper refcount
> > manipulation.
>
That was what I meant :). In the past I had more success when compiling PHP
on windows when using the new build system, which is *better* maintained.
At 09:10 06.04.2005, you wrote:
On Wed, 6 Apr 2005, Steph wrote:
> erm - when was the decision to no longer maintain the .dsp files made?
Nobody di
I seem to recall this discussion from days of yore. And the word was,
'nope, don't let the things wallow in historical mire, for many do use
them'.
I'll add it to the TODO (which is starting to look almost possible again).
> On Wed, 6 Apr 2005, Steph wrote:
>
> > erm - when was the decision to
Derick Rethans wrote:
On Wed, 6 Apr 2005, Steph wrote:
erm - when was the decision to no longer maintain the .dsp files made?
Nobody did, but why maintain something outdated? Feel free to fix them
though...
Derick
I think it is better to give up with different build system for windows.
It so m
Hi All,
Few weeks back I have checked in the configure.in and few m4 patches so
that NetWare cross compile goes through. I tested the same by checking
out a sources from php5_0 tree and ./buildconf and ./configure with
approproiate --host and --target switches.
When I do the same with whatever the
A colleague noticed that in an upgrade from 4.3.10 to 4.3.11 the
pear/DB extension was no longer installed. The ChangeLog does not
mention this. Should it be added to the changelog, and why was it
dropped? I didn't completely follow the discussion here...
Thanks for any hints,
Martin
--
<[EMA
On Tue, 5 Apr 2005, Davey wrote:
As Wez knows, there are issues with OSX/PHP 5.0.x/PDO, so on
What issues?
--Jani
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
It doesn't compile with 5.0.x
But thats nothing to do with the memleaks.
- Davey
Jani Taskinen wrote:
On Tue, 5 Apr 2005, Davey wrote:
As Wez knows, there are issues with OSX/PHP 5.0.x/PDO, so on
What issues?
--Jani
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, vi
Rubbish.
http://netevil.org/node.php?nid=202 shows how to make it compile.
On Apr 6, 2005 6:49 AM, Davey <[EMAIL PROTECTED]> wrote:
> It doesn't compile with 5.0.x
>
> But thats nothing to do with the memleaks.
>
> - Davey
>
> Jani Taskinen wrote:
> > On Tue, 5 Apr 2005, Davey wrote:
> >
> >> A
You know, I could have sworn that I only looked for a : as the separator.
I seem to remember a discussion about this in the past, but don't
recall the details.
Would you mind searching the archives for the old php-dev mailing list
to see if you can find anything else on this matter?
--Wez.
On Apr
You're not making much sense here; first you say that you think the
errors are because he is not linking against version 8, but then you
say that only version 7.4 or higher is required.
--Wez.
On Apr 5, 2005 9:24 PM, Christopher Kings-Lynne
<[EMAIL PROTECTED]> wrote:
> >> I would suggest to fix t
Please send it to me.
--Wez.
On Apr 2, 2005 11:02 PM, Christopher Kings-Lynne
<[EMAIL PROTECTED]> wrote:
> > I'd like to roll PHP 5.1 Beta 1 very soon.
> > Wez, I've been waiting for PDO in order to get this going. Do you feel
> > it's mature enough or should I hold off some more time?
>
> Well
Hey Andi,
I've been a busy with work (and house hunting), so I've been a bit out of it.
Let me catch up and take stock on PDO again; I'd like to push a PECL
release of the various packages this weekend, before we go with Beta
1.
I can't think of any show-stoppers in PDO right now, so the time to
You're not making much sense here; first you say that you think the
errors are because he is not linking against version 8, but then you
say that only version 7.4 or higher is required.
As in I suspect (1) he's linking against 7.3 or (2) for some reason the
libpq 8.0 windows DLL is specifically mi
OK, I will upgrade the version of libpq on the snaps box. It probably didn't
get updated the last time i upgraded to 8.0.0.
Edin
- Original Message -
From: "Christopher Kings-Lynne" <[EMAIL PROTECTED]>
To: "Wez Furlong" <[EMAIL PROTECTED]>
Cc: "Edin Kadribasic" <[EMAIL PROTECTED]>; "Deri
Thats right, theres just bugs in the currect PECL releases and I
couldn't get CVS to compile with 5.0.x. Thinking on it now, CVS HEAD
is probably 5.1 only right? and theres a 5.0.x branch? ::looks::
Regardless, are these memory leak reports of any use?
- Davey
Wez Furlong wrote:
Rubbish.
http://n
Hi,
it would be very useful if class_implements()/class_parents() would also
handle a string containing a class name instead of requiring an object
instance.
regards,
Lukas
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
Hello Lukas,
Wednesday, April 6, 2005, 3:46:08 PM, you wrote:
> Hi,
> it would be very useful if class_implements()/class_parents() would also
> handle a string containing a class name instead of requiring an object
> instance.
Class as it says does not need an instance. Can you send a patch?
Hello Wez,
Wednesday, April 6, 2005, 2:48:22 PM, you wrote:
> Hey Andi,
> I've been a busy with work (and house hunting), so I've been a bit out of it.
> Let me catch up and take stock on PDO again; I'd like to push a PECL
> release of the various packages this weekend, before we go with Beta
>
The plan is to have only one mysql driver that can be linked against
both version 3 and 4 of the client libraries.
--Wez.
On Apr 6, 2005 9:56 AM, Marcus Boerger <[EMAIL PROTECTED]> wrote:
> Hello Wez,
>
> Wednesday, April 6, 2005, 2:48:22 PM, you wrote:
>
> > Hey Andi,
>
> > I've been a busy w
Hello Wez,
i very much dislike that idea because a mysql3 will never be working as
expected. It can never support multiple statements at a time or transactions
or whatnot. Thus i pretty much want something that keeps me from using it
until i am pretty sure i really can live with that. A distingu
But neither can most other drivers.
Since the majority of the API is the same, what sense is there in
duplicating the code?
--Wez.
On Apr 6, 2005 10:22 AM, Marcus Boerger <[EMAIL PROTECTED]> wrote:
> Hello Wez,
>
> i very much dislike that idea because a mysql3 will never be working as
> expec
Marcus Boerger wrote:
Hello Lukas,
Wednesday, April 6, 2005, 3:46:08 PM, you wrote:
Hi,
it would be very useful if class_implements()/class_parents() would also
handle a string containing a class name instead of requiring an object
instance.
Class as it says does not need an instance. Can you
Andrey Hristov wrote:
Marcus Boerger wrote:
Hello Lukas,
Wednesday, April 6, 2005, 3:46:08 PM, you wrote:
Hi,
it would be very useful if class_implements()/class_parents() would
also handle a string containing a class name instead of requiring an
object instance.
Class as it says does not nee
Marcus Boerger wrote:
i very much dislike that idea because a mysql3 will never be working as
expected. It can never support multiple statements at a time or transactions
or whatnot. Thus i pretty much want something that keeps me from using it
until i am pretty sure i really can live with that.
You know, I could have sworn that I only looked for a : as the separator.
Not according to php_stream_locate_url_wrapper():
for (p = path; isalnum((int)*p) || *p == '+' || *p == '-' || *p == '.';
p++) n++;
if ((*p == ':') && (n > 1) && !strncmp("://", p, 3)) {
protocol = path;
} e
Hi all,
I have cooked up a patch that gives the possibility to use not
only an object but a string value with class_implements() and
class_parents() which are part of ext/spl.
Unified diff:
http://andrey.hristov.com/projects/php_stuff/patches/class_parents_implements.patch.txt
Source snippet:
http
Hello Andrey,
just three tiny changes were needed before applying. If onyl other parts
could be done that fast.
Wednesday, April 6, 2005, 5:03:08 PM, you wrote:
> Hi all,
> I have cooked up a patch that gives the possibility to use not
> only an object but a string value with class_implement
It's not 'nix you want to worry about, but systems where : identifies
a drive or device; win32, vms (?), (and Amiga, if that still counts :)
Could be more trouble than its worth, BC wise.
--Wez.
On Apr 6, 2005 11:02 AM, Sara Golemon <[EMAIL PROTECTED]> wrote:
> The only thing that positively lea
On Wed, Apr 06, 2005 at 10:03:42AM +0200, Martin Kraemer wrote:
> A colleague noticed that in an upgrade from 4.3.10 to 4.3.11 the
> pear/DB extension was no longer installed. The ChangeLog does not
> mention this. Should it be added to the changelog, and why was it
> dropped? I didn't completely f
I think it would work just fine as shared too with HEAD..
Someone with Macosx should try it out.
(first you need to remove the silly AC_MSG_ERROR() from
ext/pdo/config.m4..)
--Jani
On Wed, 6 Apr 2005, Wez Furlong wrote:
Rubbish.
http://netevil.org/node.php?nid=202 shows how to
On Wednesday 06 April 2005 17:56, Jani Taskinen wrote:
> I think it would work just fine as shared too with HEAD..
> Someone with Macosx should try it out.
> (first you need to remove the silly AC_MSG_ERROR() from
> ext/pdo/config.m4..)
Wrong, pdo drivers need to link against p
Greg might be able to answer this,
--Wez.
On Apr 6, 2005 11:37 AM, Joe Orton <[EMAIL PROTECTED]> wrote:
> On Wed, Apr 06, 2005 at 10:03:42AM +0200, Martin Kraemer wrote:
> > A colleague noticed that in an upgrade from 4.3.10 to 4.3.11 the
> > pear/DB extension was no longer installed. The ChangeL
> It's not 'nix you want to worry about, but systems where : identifies
> a drive or device; win32, vms (?), (and Amiga, if that still counts :)
>
Doi, drive letters
So yeah that can be worked around with a:
#ifdef PHP_WIN32
if (p = path+1 && isalpha(*path) && (*p == '/' || *p == '\\')) { /*
Wez Furlong wrote:
Greg might be able to answer this,
--Wez.
On Apr 6, 2005 11:37 AM, Joe Orton <[EMAIL PROTECTED]> wrote:
On Wed, Apr 06, 2005 at 10:03:42AM +0200, Martin Kraemer wrote:
A colleague noticed that in an upgrade from 4.3.10 to 4.3.11 the
pear/DB extension was no longer install
On Apr 4, 2005, at 12:35 AM, Jon Parise wrote:
Apologies if this got lost amongst the effort to prepare the recent
set of releases, but I'm curious whether or not people find my second
patch acceptable, based on the results of the prior discussion. If
not, I'll know to abandon the issue.
Here is a
Ok, lets try this:
http://www.brianfrance.com/patch.build
Brian
On Apr 6, 2005, at 7:19 PM, Brian J. France wrote:
On Apr 4, 2005, at 12:35 AM, Jon Parise wrote:
Apologies if this got lost amongst the effort to prepare the recent
set of releases, but I'm curious whether or not people find my second
OK so keep me posted in the beginning of next week.
I'll also look at some of the other mentioned issues.
At 08:48 AM 4/6/2005 -0400, Wez Furlong wrote:
Hey Andi,
I've been a busy with work (and house hunting), so I've been a bit out of it.
Let me catch up and take stock on PDO again; I'd like to p
You got quite a few typos there. For example:
- fifeth_arg_force_ref (fifth)
- PHP_SEM_NOT_AVAILEBLE (AVAILABLE)
- allready
Following constants are against coding standards:
- "ENOENT",
- EEXIST
zval_dtor(error_code); should be zval_ptr_dtor(&error_code) to make sure
you protect reference counting
OK, I will upgrade the version of libpq on the snaps box. It probably didn't
get updated the last time i upgraded to 8.0.0.
The thing is the 7.4 version _should_ work just fine...
Chris
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
At 09:10 AM 4/6/2005 +0200, Derick Rethans wrote:
On Wed, 6 Apr 2005, Steph wrote:
> erm - when was the decision to no longer maintain the .dsp files made?
Nobody did, but why maintain something outdated? Feel free to fix them
though...
It's not outdated and should be kept in sync by the people who
If there's something that looks like a scheme (i.e., a well-formed
sequence of
characters followed by ':'),
see if it's registered;
if it is,
the appropriate wrapper should be used.
Otherwise,
on platforms where ':' has significance,
try it again as a file path.
Otherwise, it fails due t
I made some changes to phpize now. But not using your
patch since that would not have worked everywhere.
Searching for these tools is _still_ a big no no.
Just make sure they're in PATH.
--Jani
On Wed, 6 Apr 2005, Brian J. France wrote:
Ok, lets try this:
http://www.brianfrance.
47 matches
Mail list logo
