Much better implementation. As for the failed tests, I would double check and make sure you don't have any stray debug statements (e.g. printf).
Sent from my iPhone On Dec 11, 2011, at 2:27 PM, Clint M Priest <cpri...@zerocue.com> wrote: > https://bugs.php.net/patch-display.php?bug_id=49526&patch=accessor_v2.diff&revision=latest > > I rewrote part of how it handles the job due to abysmal performance of > version 1. This one is near the same performance as without the code changes > using __get() and is actually faster in some cases (public getter). > > I also checked all the tests and fixed the issues I found related. > > There is however some strange behavior that I do not understand. For some > reason if I run all of the tests, 100+ or so will fail. If I have the test > script write those failed tests to a file and just run those tests, all but > four work fine. It appears to be something to do with line endings, in other > cases the output does not match expected because of the addition of 'unicode' > to the output. I've no idea what's going on here but after 3 days of mucking > around, I can't find a solution as to why this is occurring. > > These failures do not occur on a fresh checkout but nothing I've changed > should be causing any of the issues, just no idea here, any help would be > appreciated. > > Felipe: I could not find any segmentation fault tests from before when you > posted, nor now. Are you still seeing segmentation faults? > > Thoughts? > > -Clint > -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
