Re: [PHP-DEV] [PATCH] Constification of some PHP API calls

2013-07-31 Thread Andrey Hristov
On 07/31/2013 04:56 PM, Sara Golemon wrote: Ah, yeah, I can see that clearly now. Looks universally cool to me. Do you just need someone with engine karma to push it? right. I have no Zend karma, or at least didn't have at CVS/SVN times. On Wed, Jul 31, 2013 at 1:35 AM, Andrey Hristov wr

Re: [PHP-DEV] [PATCH] Constification of some PHP API calls

2013-07-31 Thread Sara Golemon
Ah, yeah, I can see that clearly now. Looks universally cool to me. Do you just need someone with engine karma to push it? On Wed, Jul 31, 2013 at 1:35 AM, Andrey Hristov wrote: > Hi Sara, > > On 07/31/2013 12:55 AM, Sara Golemon wrote: > >> At a glance, this looks awesome. There are a few

Re: [PHP-DEV] [PATCH] Constification of some PHP API calls

2013-07-31 Thread Andrey Hristov
Hi Sara, On 07/31/2013 12:55 AM, Sara Golemon wrote: At a glance, this looks awesome. There are a few changes where you introduced #ifdef WIN32 checks that I want to spend a little more time looking at... In the WIN32 code there was an additional variable, declared always, but switched from

Re: [PHP-DEV] [PATCH] Constification of some PHP API calls

2013-07-30 Thread Sara Golemon
At a glance, this looks awesome. There are a few changes where you introduced #ifdef WIN32 checks that I want to spend a little more time looking at... -Sara P.S. - Could I convince you to put it on a github fork to make it easier to read? No big deal if not... I just like that web view. :) O

Re: [PHP-DEV] [PATCH] Constification of some PHP API calls

2013-07-29 Thread Julien Pauli
On Mon, Jul 29, 2013 at 12:01 PM, Andrey Hristov wrote: > Hi, > after experiencing again warnings about conversion from const char * to > char * when calling some PHP API functions I decided to spend some time and > constify a few of these. The result is a patch that constifies the Streams > API