« first day (4933 days earlier)      last day (12 days later) » 

1:24 PM
Congrats :)
 
 
1 hour later…
2:27 PM
@Trowski We get errors in amphp/websocket-client in nightly: github.com/php/php-src/actions/runs/8730974321/job/23955672566 Apparently it's because of this error: "Fatal error: Uncaught FiberError: Cannot switch fibers in current execution context". It seems to happen only on master. Do you know if this is expected?
To clarify, php-src's master.
 
@IluTov No, definitely not. Those throws appear to be happening after the tests have run, is that right?
If I scroll up a bit I see messages like Call to undefined method Amp\Websocket\PeriodicHeartbeatQueue::PeriodicHeartbeatQueue::__construct():47}()
 
@Trowski It appears so. Maybe they are related to the previous failures.
@Trowski Ohh... I believe this may be related to github.com/php/php-src/pull/13550. /cc @TimWolla Maybe some code recognizes closures by their class names.
 
PeriodicHeartbeatQueue definitely has a constructor. The duplication of the class name looks a bit sus.
 
Right, searching for {closure} reveals some usages.
 
Ah, ok. Did the closure get destroyed too soon?
 
2:43 PM
@Trowski The name of closures was changed from "{closure}" to "{closure:file:line}" to improve the stack trace.
 
weakClosure does some magic to remove the reference to $this. Perhaps some recent changes broke it?
 
@IluTov in 8.4? I need to fix Xdebug to not also do this then
 
And some code seems to rely on things like $method !== '{closure}'.
@Derick Yes.
 
recently, right?
(cause I haven't seen failing tests)
 
@IluTov "Call to undefined method" is rather confusing though.
 
2:44 PM
@Derick Less than a week
 
@Trowski I think the error comes from the fact that amp misses that this is a closure, and tries to call it by name.
 
that's recent :-)
 
@IluTov That doesn't sound right. Amp doesn't invoke callbacks by name.
Ah, I think it's because the change from {closure} broke the code in weakClosure.
 
@Trowski Yeah, that's probably it.
 
@IluTov The function isn't being invoked by name directly. Maybe that's happening due to bindTo? github.com/amphp/amp/blob/… How can I detect that a closure was generated from a FCC?
 
2:52 PM
@Trowski You could use str_starts_with("{closure")
That should be cross compat
 
@Trowski github.com/amphp/amp/blob/… Just below it.
 
@IluTov Oh so it is, thank you.
 
hmm, annoying PHP does {closure:file:linestart} where Xdebug has {closure:file:linestart-lineend}
it would have been better if PHP picked the latter too...
 
@Derick Hmm... I'll leave that up to @TimWolla.
 
@TimWolla pokey poke :-)
 
2:56 PM
@Girgias Sounds good. The leading {closure: must be being eaten by the error message.
 
github.com/symfony/symfony/pull/54614 Symfony will need to adjust again in that case, just fyi.
@Trowski Yeah, that's confusing. Not sure what's happening there.
 
Is it possible that eval() allocates pointers on the arena which are allocated via malloc (and not our emalloc?)
 
3:29 PM
@Girgias ReflectionFunction::getShortName() is returning the name without the leading "{closure:" github.com/amphp/amp/actions/runs/8740641427/job/…
 
blerg
 
Appears to be due to calling a parent method from a child object.
 
Ah so it's a "fake closure"
 
@Girgias Not sure what you mean. The failure is coming from this line, which is calling this method on the parent class which returns a closure.
 
@Trowski Fake closures basically refer to "proper" functions/methods and not an anonymous function
So you'll get the name of the underlying method/function
 
3:44 PM
That doesn't appear to be the case here.
A fake closure is tested on the next line.
 
I'm confused by the unedfined call
 
The path using the re-bound closure which invokes the method by name is being used because the method name doesn't start with "{closure". See this line which I just updated.
Seems the error message was not truncating "{closure" from the message. That is happening somewhere else and is returned by ReflectionFunction::getShortName()
 
huuuuuuum
 
4:47 PM
o/
 
5:08 PM
@Girgias What are you referring to?
 
@IluTov Still my compile time resolution hell, as I don't think my issue is immediately with the type in question as far as I can tell :(
 
5:25 PM
@Girgias If you ask a more specific question, I can try to answer :) Do you have a bug report?
 
5:54 PM
Arnaud is having a look, and I seems to have pinpointed the issue
Without eval the class is freed first and then the trait, with eval the trait is freed first which leads to trying to free the same thing twice
 
JRL
6:45 PM
my wife and i are making our first ever offer on purchasing a house today...
 
In which country?
 
JRL
US
 
Good luck!
 
JRL
I guess a part of me thought that i would be renting forever because the housing market was so fucked by boomers for my generation
so im a little caught off guard that we're actually doing it
even though i obviously took the active steps to make it happen
 
I mean, I'm confused by housing prices, people near where my parents live are selling. And it's in the middle of no where and the house is selling way above what I would have guessed the market was
 
JRL
6:48 PM
interest rates are insane right now
 
And then I'm staring at London prices :|
 
JRL
it's literally the worst time to buy a place in the last 50 years
 
7:27 PM
@JRL I said the same thing 1 year ago, and it's actually worse today. Nobody really ever knows when it will actually correct so... as long as you aren't causing yourself financial stress, I wouldn't sweat it too much.
 
JRL
@LeviMorrison yeah, that's why we're offering anyway. no sense in trying to time the housing market.
but definitely a good thing to keep in mind.
 
7:44 PM
You can always refinance in the future when interest rates drop. Painful in the meantime though.
 
JRL
@Trowski that's the plan actually. i did some numbers, and if rates drop to even like 3.5%, we could refinance into a 15 year instead of 30 and STILL drop our payment
rates have been below 3.5% for the majority of this century, which guarantees nothing, but was sort of my line for a "return to average"
 
@Girgias @IluTov I tagged a new version of Amp, since I believe the change I made will fix weakClosure for 8.4 once the closure naming is fixed in php-src.
 
@Trowski Thank you for taking care of this so quickly!
 
github.com/php/php-src/actions/runs/8730974321/job/… Several tests were failing before. Now only the last one is still failing which I pointed out earlier.
 
8:35 PM
@JRL I doubt we'll see it that low. But should be lower than what we have today... eventually. What rate did you get offered? A year ago I got a great rate at 6%. My prior two houses were 2.85% and 2.65% respectively.
 
JRL
@LeviMorrison i was initially offer 6.875%, but rates went up by 0.5% industry wide in just the last two weeks, so current quote is 7.375%
 
Saw this CNBC article when looking at the news, that checks out.
 
JRL
i highly, highly doubt there will be a worse time in the next 20 years to get a loan than this year and possibly next year
 
9:21 PM
Fuck me, 7%?! And you didn't even have Liz Truss.
 
"Hilariously" two of my friends are doing math PhD's in Leeds, so they have met John Truss a couple of times
 
We went from 1.54% to 4.28% this month, after our fixed term was up
Who's John Truss?
luckily, it only adds about 75 quid to the payment each month
 
Liz Truss's father, who's an emeritus professor of pure maths at the University of Leeds, and a staunch Labour supporter
 
ah
I did not know that
then dear old Lizzie was more sensible in the past, but now as bat shit crazy as Taylor Greene
 
Are you looking forward to the London elections where we have the tremendous right to now choose only one candidate
 
JRL
9:35 PM
@Derick you have to go back to 2000 to get a rate as high as the ones we have now
 
@Girgias FPTP is a digression. I now feel I almost need to pick Kahn, even though he wouldn't be my first choice. Anything besides that idiot of a tory
 
@Derick I'm not even sure Susan Hall is the most batshit insane candidate on the ballot, somehow that's an achievement
But yeah
 
Only one that has a chance
I really just want ranked voting
and she'd end up below Lord Buckethead
 
I mean same, but we both know why the Tories changed the voting system back to stupid old FPTP
 
yes
anyhow. New Disco episode.
tata
 
10:35 PM
RCV FTW.
 

« first day (4933 days earlier)      last day (12 days later) »