« first day (3855 days earlier)      last day (1084 days later) » 
00:00 - 21:0021:00 - 00:00

9:01 PM
o/
 
> Psalm has a large number of code issues.
(quote from their own docs)
 
cmb
@IMSoP best of luck with the former; I had to give up :)
@DaveRandom but that is a Good Thing, cause you can keep your memory_limit low :p
 
@NikiC Fixed it, I need to swap the EG(bailout) when switching fibers.
 
@cmb comments and cdata elements aren't displayed in the debug output. and perhaps not even XML entities - wondering why users don't use the to-xml functionality to get a serialized form of the element for display. maybe b/c they would need to wrap it in <pre> and htmlspecialchars' it?
 
cmb
I prefer var_dump() because it is generic; well, should be.
 
9:21 PM
@NikiC Pushed a new, complete commit: github.com/trowski/php-src/commit/…
Let me know if that should be pushed to master.
Oh, forgot to remove EG(fiber_error). Well, I can do that if you think it looks good otherwise.
 
@Trowski Could you separate out just the bailout changes without the sanitizer part. I don't think these are related (or are they?)
 
@NikiC No, the sanitizer sort of just proves it's correct.
So yes, I can make those separate commits.
Give me a few minutes.
 
So, what happens now if a fatal error is thrown. Is that exit(1) actually hit?
 
Yes
 
And what does exit do in a fiber?
 
9:27 PM
Exits the program as usual.
 
we don't want to exit the program on fatal error though
 
Where do we want it to go?
 
Do you see failures if you pass --repeat 2 to run-tests?
 
@cmb ...low enough to forget that it's a bad idea? :-P
intelligence_limit
 
@cmb It's not broken though :"P
 
cmb
9:30 PM
heh
guess libxml2 needs something like https://github.com/libgd/libgd/pull/692
 
@NikiC Just to be sure I'm doing it right, make test -j6 TESTS="--repeat 2 Zend/tests/fibers" yes?
 
@Trowski yes
Well, that -j6 is useless :)
 
@NikiC No errors from that
 
You need to pass it to TESTS
 
@NikiC The -j6 is passed to make. Seems to work fine.
 
9:33 PM
@Trowski yes, but it's not passed to the test runner
 
@NikiC Oh, that works for the test runner too, cool. TIL
 
@Trowski Ah yes, I think --repeat mode is being a bit too clever there to avoid some false positives
Anyway, imagine you're in a web server context
And you have a fatal error on one request. You don't want to kill the whole process because of that
 
@NikiC Eh… yeah, I'm thinking too much about cli.
 
9:47 PM
@NikiC Ok, I replaced exit(1) with a zend_bailout to the previous EG(bailout).
 
9:58 PM
EG(fiber_bailout) is set to true to simply repeat zend_bailout() when switching to the previous fiber (or {main}).
If in a nested fiber, the prior zend_try in zend_fiber_execute will catch it and repeat.
I probably should have set EG(fiber_bailout) to false before calling zend_bailout in case it's caught, oops
That seems to make sense… rather than pushing a fatal error up the tree, now we're pushing any zend_bailout up the tree.
 
10:30 PM
Pushed a couple of tests that fail on master with asan but pass in that branch.
 
10:48 PM
 
 
1 hour later…
11:59 PM
what do php-src devs gain from the langspec?
 
00:00 - 21:0021:00 - 00:00

« first day (3855 days earlier)      last day (1084 days later) »