Conversation started Apr 12, 2014 at 15:22.
user895378
Apr 12, 2014 15:22
Ah found the problem.
user895378
@bwoebi and so did you
user895378
It's the pecl parser
user895378
I'm actually moving away from that.
user895378
I'm eventually going to remove the pecl/http parser altogether
Does the pecl parser have a bug?
or is it the interaction with it? @rdlowrey
user895378
Apr 12, 2014 15:24
I think my wrapper around it seems to have a bug. But I've found that my userland parser is more or less equivalent with the pecl version I've created (speed-wise) so I'm doing away with it.
user895378
So you won't need pecl/http going forward with aerys.
user895378
I'm going to commit a couple of lines of changes that will fix the bug (I've already tested it on your box).
user895378
I'll ping you after I push those so you can pull them in.
user895378
I've been working without pecl/http on my local dev box for awhile now so I never noticed the issue.
user895378
@bwoebi okay, those changes are pushed to the abstractions branch
Apr 12, 2014 15:30
@rdlowrey ah you just removed it completely
user895378
Yeah, I'm going to eventually remove it altogether.
user895378
And there won't be a separate Aerys\Parse subnamespace at all.
user895378
One parser to rule them all.
:-)
user895378
And in any case, if I ever use a compiled http parser it will be one I write myself. So the pecl thing isn't very useful. When you throw in that it's not significantly faster than my userland version it doesn't make a ton of sense to maintain it and add complexity to the repo or tell people to pull in another pecl extension.
 
Conversation ended Apr 12, 2014 at 15:33.