Re: [PHP-DEV] Inconsistencies with constructors in SPL

2011-05-17 Thread Peter Cowburn
Hi folks, On 16 May 2011 22:25, Etienne Kneuss wrote: > Hi, > > On May 16 16:52:08, Andrew Curioso wrote: >> Well, that wasn't where I was expecting that thread to go, but to wrap it up >> what do you think... >> Is it too late to put this on the 5.4 roadmap for consideration? >> >> I'm assuming

Re: [PHP-DEV] [RFC] Improved parser error message

2011-05-17 Thread Julien Pauli
I remember error messages have been improved since last decade, but they were engine ones. Parser ones have never been touched, so to improve them, I +1 your idea Felipe :) Julien On Mon, May 16, 2011 at 7:52 PM, Florian Anderiasch wrote: > On 16.05.2011 14:15, Felipe Pena wrote: >> Other exampl

Re: [PHP-DEV] Inconsistencies with constructors in SPL

2011-05-17 Thread Andrew Curioso
>From what I can tell, whatever changes made to fix this bug, are independent of #54384. Bug #54384 happens when the parent ctor is not called. This bug is triggered only when it is called. I did a quick check of the code and that seems to be the case. But the existence of that bug makes me think

[PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Alexey Shein
I need karma to commit test fixes: 1) session_encode_basic - added serialize_precision=100 ini setting 2) fix for http://bugs.php.net/48203: there's a segfault when CURLOPT_STDERR file pointer is closed before calling curl_exec, i.e. like this: $fp = fopen(dirname(__FILE__) . '/bug48203.tmp', 'w'

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Pierre Joye
hi, On Tue, May 17, 2011 at 3:40 PM, Alexey Shein wrote: > I need karma to commit test fixes: > 1) session_encode_basic - added serialize_precision=100 ini setting Is it not fixed already? > 2) fix for http://bugs.php.net/48203: there's a segfault when Pls open a bug report and attach the patc

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Alexey Shein
2011/5/17 Pierre Joye : > hi, > > On Tue, May 17, 2011 at 3:40 PM, Alexey Shein wrote: >> I need karma to commit test fixes: >> 1) session_encode_basic - added serialize_precision=100 ini setting > > Is it not fixed already? > As Tyrael already said this one was missed. >> 2) fix for http://bugs

[PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Felipe Pena
2011/5/16 Felipe Pena > Hi all, > As I have proposed previously in an old thread... What about we name all > the tokens to have an improved parser error message? (i.e. anymore > T_PAAMAYIM_NEKUDOTAYIM, T_DOLLAR_OPEN_CURLY_BRACES in the messages etc) > > [...] > > Other examples and patch at: >

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Alexey Shein
Created http://bugs.php.net/bug.php?id=54798. -- Regards, Shein Alexey -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Gustavo Lopes
Em Tue, 17 May 2011 14:40:53 +0100, Alexey Shein escreveu: 2) fix for http://bugs.php.net/48203: there's a segfault when CURLOPT_STDERR file pointer is closed before calling curl_exec, i.e. like this: $fp = fopen(dirname(__FILE__) . '/bug48203.tmp', 'w'); $ch = curl_init(); curl_setopt($ch

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Ferenc Kovacs
On Tue, May 17, 2011 at 3:54 PM, Pierre Joye wrote: > hi, > > On Tue, May 17, 2011 at 3:40 PM, Alexey Shein wrote: > > I need karma to commit test fixes: > > 1) session_encode_basic - added serialize_precision=100 ini setting > > Is it not fixed already? > > there were other tests with missing s

Re: [PHP-DEV] Need karma for committing test patches

2011-05-17 Thread Alexey Shein
> A few remarks: > > * Isn't this supposed to be fixed? Was there something that triggered a > regression? As far as I can see Jani checked only case when already closed file descriptor was passed to curl_setopt. Here's the check is performed in curl_exec, i.e. in curl_setopt file descriptor was v

RE: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Andi Gutmans
Hi Felipe, Read the archives. We discussed T_PAAMAYIM_NEKUDOTAYIM extensively and agreed to keep it as a historical landmark. Let's not have the discussion yet again within the same year. Google it just to see the # of mentions. What we could do as a compromise between history and improving ma

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Gustavo Lopes
Em Tue, 17 May 2011 17:01:26 +0100, Andi Gutmans escreveu: Read the archives. We discussed T_PAAMAYIM_NEKUDOTAYIM extensively and agreed to keep it as a historical landmark. Let's not have the discussion yet again within the same year. Google it just to see the # of mentions. What we cou

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Derick Rethans
On Tue, 17 May 2011, Felipe Pena wrote: > 2011/5/16 Felipe Pena > > > Hi all, > > As I have proposed previously in an old thread... What about we name all > > the tokens to have an improved parser error message? (i.e. anymore > > T_PAAMAYIM_NEKUDOTAYIM, T_DOLLAR_OPEN_CURLY_BRACES in the messages

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Ferenc Kovacs
On Tue, May 17, 2011 at 6:01 PM, Andi Gutmans wrote: > Hi Felipe, > > Read the archives. We discussed T_PAAMAYIM_NEKUDOTAYIM extensively and > agreed to keep it as a historical landmark. Let's not have the discussion > yet again within the same year. Google it just to see the # of mentions. > > M

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Gustavo Lopes
Em Tue, 17 May 2011 17:07:09 +0100, Gustavo Lopes escreveu: [...] Anyway, +1 to the change from you. Sorry, it should read "from me". -- Gustavo Lopes -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

RE: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Zeev Suraski
+1 with Andi's improvement. Gustavo - I realize it's not about changing the token's name, but nobody (=very few) looks at the token names. The point is to keep the tradition of also exposing this specific token name to the user, but still making it clear that what was expected was :: - without

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Stas Malyshev
Hi! I think we need to keep token name in the message, since it makes it easier to understand what parser expected if you need to debug the parser (as opposed to your code). So I think we need to have both the human-readable name and the token name, as Andi suggested. -- Stanislav Malyshev, S

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Ferenc Kovacs
On Tue, May 17, 2011 at 6:49 PM, Stas Malyshev wrote: > Hi! > > I think we need to keep token name in the message, since it makes it easier > to understand what parser expected if you need to debug the parser (as > opposed to your code). So I think we need to have both the human-readable > name an

Re: [PHP-DEV] Re: [RFC] Improved parser error message

2011-05-17 Thread Rasmus Lerdorf
On 05/17/2011 09:58 AM, Ferenc Kovacs wrote: On Tue, May 17, 2011 at 6:49 PM, Stas Malyshevwrote: Hi! I think we need to keep token name in the message, since it makes it easier to understand what parser expected if you need to debug the parser (as opposed to your code). So I think we need to

Re: [PHP-DEV] Inconsistencies with constructors in SPL

2011-05-17 Thread Martin Scotta
Martin Scotta On Mon, May 16, 2011 at 4:10 PM, Anthony Ferrara wrote: > Personally, I really don't care for something like this. Would it be > caught by a __call declaration if one existed (since it is a call to > an undefined method)? Would you expect it to? > Although current PHP implement