« first day (4034 days earlier)      last day (905 days later) » 

12:00 AM
@MarkR I'll maybe have a look at it, but I don't think the approach I'm doing is going to make this any simpler, as it's more or less the current system, just being stored in the executable globals
 
That's what I did in my RFC.
Added the HashTable* to the global struct and added an extra ref
 
I'll have a think about it
But I really should go to bed lol
 
g'nite
beware vampires etc etc
 
 
3 hours later…
3:19 AM
@MarkR tonight I am going to suuuuuuuck
...your blood
 
 
4 hours later…
7:32 AM
\o
 
 
1 hour later…
8:33 AM
o/
 
9:11 AM
\o
 
9:23 AM
o/
 
10:06 AM
\o
 
10:18 AM
o/
 
10:40 AM
\o
 
Wish you all a killer week
 
11:03 AM
/murders @ln-s
 
Well that was an interesting twist of events
 
11:21 AM
Don't forget to bury the body
 
Everyone knows that to bury a body you have to do it properly and get that calcium oxide
rookies
 
 
1 hour later…
12:51 PM
@ramsey, @JoeWatkins: bugs.php.net/bug.php?id=81390 has been fixed and is part of 8.1RC5, but bug is still open and has not been mentioned in NEWS file. Should we close and adapt the NEWS file under "28 Oct 2021, PHP 8.1.0RC5" retroactively?
 
1:18 PM
@PatrickAllaert yes, you will also have to add it to changelog in php-web I think ... note the bug was at one point closed, and got changed from closed to assigned, which is odd ...
ah there's nothing in changelog yet, so no problem ...
 
1:36 PM
o/
 
\o
 
2:00 PM
Ooooh.... 8.1.0 GA coming out this month... exciting!
8.0.x shall no longer be the belle of the ball....
 
She'll be a madam, not a madmoiselle?
Ugh, I would kill for a generic Result type right now. :-(
 
2:31 PM
@Crell what for?
like PDO result or something?
 
No, a result type is a monad type. Rust's Result type.
 
oh
 
It's of type Success or Error(and here's an error object).
 
is type Success similar to a bool "true" in functionality?
 
Depends on the implementation.
You can either have an OK/Error version, (in which case the success path has no associated data), or a Result/Error version (in which case both carry some additional data).
So, eg, if you have a findUser() method, it can have a return type of Result<User, Err>. Meaning (approximately, in PHP terms) it would return EITHER a Result object with a ->value property that is a User object, OR an Error object with an ->err property that is an Err object.
This is why Ilija and I were trying to work on pattern matching and tagged unions (enums with constructors). Because it gives you effectively that.
 
2:41 PM
reason I ask is PDO does something similar but not quite
I'll paste a redacted code snippet
PDOStatement::execute() returns a bool, which can be determined how to proceed before fetching results
 
Rreplied with what a Result object would look like instead. Ish.
 
2:57 PM
interesting
 
It's not super helpful in that particular case, frankly.
It's mostly useful when there are multiple possible error cases, that are genuinely errors and not "oh, the array happens to be empty."
 
3:53 PM
I take exception to that.
laugh damnit, I need validation :O cries
 
@Crell We've got a type like that in MongoDB. github.com/mongodb/mongo/blob/master/src/mongo/base/…
 
Well that doesn't help me much, now does it. :-P
 
StatusWith<int> numRows() {
  if (shitHappened) {
    return Status(ErrorCodes::OhFuck, "Shit happened.");
  } else {
    return 42;
  }
}
Just sharing for others' benefit to understand the design pattern
 
thumbsup.gif
 
But like... you could make something like that in PHP. Obvs it'd be better with generics, but you could still make it.
 
4:06 PM
Technically one could write it in PHP today, using just classes. It's just a lot more annoyingly verbose and wouldn't have type safety.
Jinx!
 
heh, yp
 
The forbidden feature! hiss =P
 
Psalm can help with it and there are libraries which do it
 
I thought Niki still had it on his todo pile...
 
4:07 PM
That's a point, who's on board with writing an RFC for writing no-op'd generics?
 
Generics? I think it's still at the "oh crap, this is a shit ton of work" stage.
 
I'm having a memory leak when the callback is a closure, but no idea why this happens
@Crell Nikita did have a working prototype, the issue, as always, is speed
 
@Crell It would just be adding the syntax, it would be stripped at compile time. So give psalm something to work with that wasn't spamming comments
 
@Girgias looking...
 
It came up in a discussion with Joe (I think?) that it might be interesting to see if there was interest.
 
4:08 PM
Elided Generics have been mentioned several times in the last few years.
 
@Sara Thanks :D also I was doing some triage and found github.com/php/php-src/pull/4132, are you going to continue this or should this be closed?
 
Yep, there was a huge thread on reddit on runtime erased generics
 
I think the resistance on going ahead with complete type erasure is:
1/ What if we want real generics later and the syntax we picked is borked?
2/ Users are dumb and will assume the types are enforced when they aren't and won't use a static analyzer, and up will become down and black will become white and lo the seventh seal will break and Gabriel's horn will sound and....
@Girgias Honestly can't recall why I abandoned this, I'm assuming it was because: "Ooooh, shiny!"
Feel free to take it and run with it.
 
@Sara I think 1) is reasonable, but at this point I think the grammar would be fixed more by what other platforms do, which is a bit tail wagging the dog, but this dog is slow.
 
@MarkR Agreed on both counts there. yes.
@Girgias What's the memory leak say? Like, which allocation does it point at?
 
4:16 PM
@Sara So I'm at uni doing some Quantum Mech rn, but the allocation leaking is this one: heap.space/xref/php-src/Zend/zend_closures.c?r=04cbd841#505
 
... and that's how you mic drop your education :P
 
Okay, so the actual closure itself, not some sub-component. All I really needed there.
 
@PeeHaa Listening now!
 
Also, the quote "Busy doing Quantum Mechanics atm, ask me about this web scripting thing later" is priceless.
 
@PeeHaa You really like hard house beats don't you. =P
 
4:22 PM
@MarkR It's surprisingly the thing I find low-key the easiest in my degree and I only chose it because I needed to fill some modules for this term, so I don't know if this an indication of the general wrong perception of QM, or that pure math is way to fricking hard lol
 
@StatikStasis It'sa me
 
@Girgias As someone who spent a chunk of time trying to understand QFT and failing completely, that hurts my soul :P
 
@PeeHaa There was an old house song called Join Me that I cannot seem to find that I think you would like. It came out in 2000 I think. I'll have to find it later.
 
@MarkR tbh, in the 4weeks we've been doing it, we've barely done anything substantiable, like we're just starting to learn the 5 principles, I think most of it comes in the second term module, which basically everyone says to not take lol
 
@PeeHaa Actually here it is... but there is another version of it where it goes into a hard house beat. Still a dope song to this day. youtube.com/watch?v=xHTjfJLTXsk
 
4:26 PM
@MarkR Eh, QFT is a bit of a different category
At my university QFT wasn't taught at all in two semesters of theoretical quantum mechanics
 
For some reason, my university scheduled my two hour quantum mechanics lecture to start at 8am monday mornings.......which should have been illegal.
 
Oh I know that one!
Always reminds me of the piano of robert miles' children
 
For me it was trying to learn from Leonard Susskinds lectures but they were way over my head.
 
@PeeHaa Check this one out... not the version I was trying to find but a little harder hitting. youtube.com/watch?v=rNMIUjQUkLc
It's like it goes into a totally different song.
 
ooooh yas \o/ that is exactly how I remember the birth of the harder styles :D
 
4:36 PM
=D
 
@Girgias All looks pretty reasonable to me. The only thing that make me wonder is zend_hash_destroy(&EG(autoloaders.class_autoload_functions)); should probably be called before zend_objects_store_destroy(&EG(objects_store));. Otherwise, when you get to hashtable cleanup that would free the reference to the closure, the actual object has already been orphaned by the store going away.
 
@Sara Oh, maybe, I'll try this when I get back home
@Danack That should indeed be illegal
 
4:56 PM
what change in 8.1 resulted in this? 🤔 3v4l.org/ivTl8
 
@SaifEddinGmati If the change had gone in the other direction, I would blame an update to the mbstring locale files, but an increase feel more like an outright bug.
 
this is the only change entry for mbstring
 
@SaifEddinGmati MBString has gotten a shit load of refactoring during this year
What's likely is that because you didn't provide an encoding, it previously detected another encoding
i.e. always specify with which encoding you're working with mbstring funcs
 
here with utf8 3v4l.org/D84Jm
i found couple other cases as well.

// [3, '🥇🥈🥉'], // FAIL - new value: 6
// [12, '♈♉♊♋♌♍♎♏♐♑♒♓'], // FAIL - new value: 24

should i report this as a bug?
 
Maybe then
 
5:45 PM
@SaifEddinGmati I realize today that bugs.php.net/bug.php?id=81390 was actually fixed as part of 8.1.0RC5
and no NEWS entry was written
I'll fix that retroactively
The amount of recent changes in MBstring worries me TBH
 
i have almost 1000 test cases for functions that wrap mb extension, this is the only failure i encountered.

i feel bad for not testing with 8.1 earlier ^^"

https://github.com/azjezz/psl/pull/246
 
Oh, THAT is pleasing to know
 
@Crell Thanks for plugging Xdebug the other day. However, it did absolutely do nothing :-/
 
6:52 PM
@Derick the only thing I've found that was effective was refusing to release a new version that supports the latest version of PHP...
 
@Derick Damn. :-(
 
@Danack I am not a fan of blackmail :-)
 
Have you considered... muuurrrderrr? youtube.com/watch?v=1o7SM68oES0
 
Joe's timing is impeccable...
 
7:01 PM
oh is it, what's going on ?
 
something about murder, channel has been kind of grim today
post halloween effect maybe
 
@Girgias it's what Sara said, (shutdown) order of operations ... also, make zend_autoload_startup and zend_autoload_shutdown, you'll always want to shutdown the autoloader (like refs/fibers, and in the same place roughly) ...
I haven't looked closely but this looks pretty messy right now ...
 
@JoeWatkins I mean, I don't know what I'm doing, but thanks for the pointers
 
I guess WIP, so when ready for review, holla
@Girgias there is only one way to learn ;)
 
Yeah, it's still very much a WIP, I've got a couple of things which still need to be done
While you're here, how can I create a FCI/FCC combination for an internal function?
I need to forward the call from spl_autoload_register to autoloader_register_class() and I can't just alias because of the weird 2nd param it has
 
7:13 PM
zero an fci and fcc pointer, set fci.size = sizeof(zend_fcall_info), set params and object on fci, set fcc.object to fci.object
 
Okay, that's simpler than expected
 
set fcc.function_handler
 
that's a zend_function * right?
 
yeah
 
how do I get that out of just the function name
 
7:15 PM
zend_hash_find_ptr()
on function table
with the internal function name as key
 
:o
Ok thanks!
 
7:27 PM
@Danack Awesome, enabled
@Danack Interested in how github.com/github/roadmap/issues/289 will look like, that one is relevant to us
Whether there's actually going to be a separate state for invalid or some kind of label-based thing
 
7:47 PM
Well... I found partially the issue, me not including the zend_autoload.h header in php_spl.c might be a reason why shit broke
Also, turns out I'm overflowing the stack allocation now :D
Or maybe I just something else wrong
 
@Derick Have you had the chance to look at bugs.php.net/bug.php?id=81458 ?
 
8:46 PM
@Girgias What's the purpose of the zend_autoloader thing you are working on? Prep work for autoloading functions too?
 
@LeviMorrison Yeah
Just trying to get classes working to get familiar with the different areas needed
 
9:07 PM
Well, I'm at the point trying to understand why I'm getting different behaviour for spl_autoload_register() which seem very strange, and I need to determine if it was broken and now correct or the other way round .-.
 
9:21 PM
Got a new office on Friday! Twice as big as my previous one... room for activities!!
Not going to get more done though... same amount... maybe less.
 
this is whelming
also, gratz :P
 
Okay wtf
Why does spl_autoload_unregister('spl_autoload_call'); flush the autoloading stack
 
Apr 25 '15 at 1:58, by Danack
@Worf I'm not going to say that drugs are bad, I'm just going to say that some of the SPL was designed when people were discovering Mescaline, and that the results were bad.
 
9:37 PM
Thanks for this reminder
 
btw that was allegedly fixed bugs.php.net/bug.php?id=48493
 
@Danack Cannot be ext/spl/tests/spl_autoload_002.phpt tests for that
I was trying to understand what why this test was failing
 
10:39 PM
work was migrated to Jira cloud over the weekend from self-hosted... the UI is slick, but I can't turn off the WYSIWYG functionality ... it's like being forced to fight with Word 😫
just let me type markdown dammit
 

« first day (4034 days earlier)      last day (905 days later) »