« first day (4934 days earlier)      last day (28 days later) » 
07:00 - 19:0019:00 - 20:00

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 :-)
 
 
2 hours later…
3:49 PM
@Trowski Ah, I see, thanks!
 
 
2 hours later…
JRL
5:38 PM
rofl
well, our offer on the house was rejected without a counter
because the seller's parents co-signed on the house when they bought it and won't let them sell at a price the market can support
their agent didn't try to convince us that the asking was reasonable, he tried to explain to us why the sellers were stubborn and unreasonable, lol
 
5:55 PM
@JRL I wouldn't expect the first offer to just be accepted in most cases. You can explain why your offer is fair and make the same (or slightly higher) offer.
Often people fish to see if you'll pay more, even if they'd accept your initial offer.
 
JRL
yeah, we could re-offer, but their agent basically told us that the parents want 100k over market or they will just take the house off the market
 
Oh well in that case, best of luck to them lol
 
JRL
that was my response, lol
they aren't even going to respond to our offer with a rejection
their agent reached out to tell us that the sellers plan to just let our offer expire without replying
i gathered that the seller's agent is not happy with them
because it seemed like the only reason he reached out to us was to vent about his clients
 
Was your offer 100k below the asking price?
 
JRL
no, not that far below
 
6:02 PM
I've heard houses were selling for over asking for the last few years. I don't expect that to be the case anymore with interest rates where they are.
 
JRL
6:16 PM
in my particular area, my wife and i have a 90th percentile credit score, 90th percentile assets, 95th percentile income, and zero other debt, and they were asking for the very top of our range
interest rates are just that high
our purchasing budget would be $400k higher if we could get an interest rate of around 4% even, so the interest rates are going to make selling those houses very difficult i think
 
6:37 PM
That is essentially the point of higher interest rates. Slow down economic activity to avoid inflation.

It's a crude tool, but it's all the Fed's got.
 
07:00 - 19:0019:00 - 20:00

« first day (4934 days earlier)      last day (28 days later) »