« first day (4934 days earlier)   

7:45 AM
@IluTov Small correction: That changed a little over the course of the PR. In the merged version it's: {closure:<parent_function>:<start_line>}, with <parent_function> being the filename for closures defined at the top level.
@Girgias For the shortName this is correct. When comparing the full name the name before the change included the namespace. With the change it consistently starts with {closure:.
@Derick See two messages above: It's not actually the filename in the majority of cases. I don't feel strongly about including the end line, but I'm also not sure if there is a benefit in doing so. It would still be ambiguous for cases line: $foo = function () { }; $bar = function () { }; in a single line.
If including the end line would make things easier for you, then I'm happy to make the adjustments.
FWIW: I've intentionally pinged you in the PR for your opinion, but you might've missed the email.
@Trowski This is about shortName incorrectly truncating? Do you have a simple stand-alone reproducer? I'll look into that nevertheless, but having a clear test case would make stuff easier.
 
 
2 hours later…
10:01 AM
@TimWolla Right, I forgor
 
@Trowski Looking at the php-src implementation, a more reliable check would be: ReflectionFunction::isAnonymous(). It returns true for actual closures, and false for first class callables.
 
 
1 hour later…
11:18 AM
@Trowski Amphp still fails in nightly (now with amp 3.0.1), but I can't reproduce the issue locally. In my case, the websocket-client never actually call the weakClosure. Were you able to reproduce this?
 
 
3 hours later…
2:04 PM
@TimWolla 8.2+, so I'd need to keep the current code for 8.1 anyway. I'll keep that in mind for the future, thank you. Also thanks for jumping on that issue right away!
@IluTov Was the nightly build before @TimWolla's PR was merged? Looks like it.
@IluTov The tests for amphp/amp will have still failed before the PR, but should be passing now.
The callback in websocket-client is on a timer, so may or may not be run depending on how fast the tests complete. CI tends to be slower than local, so is more likely to trigger such timers.
 
2:19 PM
@Trowski Nightly is at 01:00 UTC, so yes the latest nightly is older than my fix.
And sure thing for the fix. Was an obvious bug, I broke it, so I fix it :-)
 

« first day (4934 days earlier)