« first day (3634 days earlier)      last day (1297 days later) » 

12:43 AM
@NikiC Both files have no instances of $result_type that I can see. Unless I'm blind.
 
1:18 AM
@Tiffany Reply from MR with 1 line of text: ttl=1 day
 
2:08 AM
Anyone here? How to include a php file which is in another directory?
 
include 'path/to/dir'
AH YES I FIGURED OUT SVN
I needed to pass ` --include-externals` with my commit comand
 
Hey, its not working....include 'include/footer.php';
sorry
ya thats the code not working
 
2:32 AM
You know, without the error message we're not gonna go far
Because that's how you do it:
And because it's 3:30AM here, I'm off to bed
 
@RifkyNiyas if you're trying to use a relative path like you would in HTML...don't... it doesn't work
also
probably better doing require __DIR__ . '/../path/to/dir/footer.php';
where /../ is the level of depth of the file you want to include from the originating file
@MarkR had a question for you, can I ask you on discord?
 
@Girgias Good night dude! We just woke up here
@Tiffany Thank you. It worked
 
 
2 hours later…
4:38 AM
Never thought I'd look forward to winter but knowing that my new neighbors are less likely to be outside making noise when I'm trying to sleep is something to look forward to...
 
 
1 hour later…
Wes
5:58 AM
what's the alternative name for C arrays? i am trying to write "arrays" but i don't want to specify "not php arrays" every time :P
 
 
2 hours later…
8:02 AM
@Crell You need to rebase over github.com/php/php-src/commit/…, which changes those tests
 
8:29 AM
php-fpm segfaults for an unknown reason ・ opcache ・ #80154
 
8:54 AM
@NikiC would it make sense for 8.1 to deprecate userland methods which override internal methods, but doesn't declare a return type?
 
9:08 AM
@MateKocsis I don't think so
 
9:28 AM
@MateKocsis See also externals.io/message/106539
Esp the part about implicitly inheriting return types if they aren't specified
 
 
2 hours later…
11:44 AM
basename("/tmp","tmp") fails ・ *Directory/Filesystem functions ・ #80155
 
12:17 PM
Incomplete records may be written ・ dbase ・ #80156
 
12:59 PM
@Wes C arrays are more a concept than an actual thing - they are just a pointer to some memory typically allocated as multiple of the base size of its underlying type
just calling it a pointer to some allocated memory is probably fine too :-P
 
Wes
aw :P
 
@NikiC thanks for the link! I'm not a fan of implicit return type inheritance. At least not as a general feature. I could imagine though to make this happen in case of internal methods for 8.1, then we could emit a deprecation notice when a return type is omitted from overriding methods a few years later, and finally, return type declarations could be made required in 9.0, so implicit return type inheritance wouldn't be needed anymore. Or this idea still doesn't make much sense?
 
@MateKocsis Problem would be performance impact
I though maybe with attributes we could find a way to nudge people
 
@Girgias Do you mean the notice? AFAIK the notice would be emitted compile-time, so it shouldn't affect performance much.
 
@MateKocsis I'm probably missing something but how would you do the implicit return type inheritance? You can't necessarely do all of that at compile time
E.g. a conditional class definition which is only known at Runtime
 
1:17 PM
@Girgias OK, then I think we're talking about different things :) Return type inheritance is certainly a runtime stuff (and Nikita also wrote in the linked thread that it's not easy to implement), but signature validation happens at compile-time (otherwise I am horribly mistaken), so we could emit notices during this process without performance problems.
 
Right, but how would you validate the signature of a class which is defined only at RunTime?
The approach seems sound, but it's PHP we are talking about, there are a bazillion edge cases D:
 
@Girgias The difference IMO is that the notice is only thrown once per request, and not as many times as the method is called.
 
I'm not exactly sure how the parser works but would your stategy cope with:
`
if ($someRuntimeCondition) {
class Foo { /* ... */ }
} else {
class Foo { /* different semantics */ }
}
`
Where you imagine Foo to inherit an internal class
 
1:33 PM
@Girgias Yes, it would. As soon as Foo with an incompatible method return type is compiled, a notice would be emitted. It's the same when other aspects (param types etc) are validated, only the error level is different.
 
Right :)
 
I think the main problem is BC. I don't think it's a huge problem, but e.g. DateTime is quite frequently extended in user-land (e.g. Carbon), so there should be some grace period for people to update the signatures.
 
@Girgias Should be no problem, zend_compile.c will compile both classes, no matter which one ends up being selected at runtime. 3v4l.org/BGUYr/vld#output
 
Obviously
@IluTov Good to know, I have no knowledge about the compile step :D
Nor the VM actually, except where to find OP handler defs
 
2:03 PM
@Girgias Thanks for volunteering. Much appreciated.
 
@salathe Currently on updating the French translation
What was needed again at this point?
Wasn't it freezing SVN?
 
2:18 PM
I thought the SVN to GIT issue was mainly about languages?
 
Removing the list of formatting strings from php.net/date is annoying ・ Documentation problem ・ #80157
 
@Jeeves wat
 
cmb
2:45 PM
blame @Girgias :)
 
:(
What did I do now :(
 
@Girgias Look at @Jeeves :)
 
I didn't even do that one :(
 
Well you're gitting blamed for it anyway... or should that be svning for it
 
@Girgias Hmm, then @cmb wants to divert suspicion from himself? 🕵️ :P
 
2:50 PM
I think Derick changed that
or it might be me but I don't think so
 
cmb
@Girgias oops, sorry, that was actually @Derick's patch (which I had committed). I was confused with your change to printf stuff. :)
 
Yeah printf was me, but I put it in an entity so it's on every page now :D
 
cmb
we might do the same for the date formats
 
3:12 PM
Hey guys! THis thing worked on morning. Now its showing an error
require DIR . '/../include/config.php';
require(): Failed opening. THis thing comes up
 
Did you delete your config file?
 
No. I made it only now
 
Check it's a) saved and b) that PHP has read access to it
 
How can the second one be checked
 
is_readable($path)
 
3:20 PM
What does it mean exactly that you are removing PHP 5 from the manual?
Does it mean that I will no longer be able to find information about PHP only functionality?
e.g. mysql_* API?
 
It will be archived, PHP 5 will be 2 major versions out of support
 
100% effective SQL injection prevention.
 
I prevent SQL injection by keeping all my users in an unencrypted CSV file in the web root.
3
 
Ok, then next question what does it mean to be archived? Will this link still work? php.net/manual/en/book.mysql.php
 
3:26 PM
@NikiC Can ecalloc fail e.g. spl_fixedarray_init? Maybe if ZEND_MM_CUSTOM is used?
 
@2dsharp I have questions. Is this some kind of a new feature in web browser/servers? Have you been blocked by the site that allows SQLi for using SQLi? What is that message supposed to tell you as a user?
 
@Dharman No, that is a poor website "feature" designed by an idiot in the name of security.
 
lol
I went to the site and I got that error too
what a stupid site
 
You specifically went to the banned url.
 
and it's a project Project Security
no, I went to the product page and used id 105 OR 1=1
 
3:30 PM
Right.
 
awesome!
 
@LeviMorrison depends on what you mean by fail. It can't return null, but it can bailout
 
Yes, that's exactly what I mean, thank you.
I'm noticing that get_gc uses a plain int n. SplFixedArray stores its size as zend_long. Do you think get_gc should use a zend_long or should I convert SplFixedArray to use a plain int?
 
Why did SO never fully generate my 24x24 thumbnail? :(
 
@LeviMorrison get_gc should be using size_t (but that is not an invitation to change it)
 
3:40 PM
@LeviMorrison Pretty sure I did a PR related to that and got shut down
 
@NikiC What's the rationale for leaving it alone?
Seems a object handlers were left alone for the s/int/bool/ and s/int/zend_result/ changes?
 
@2dsharp if it's any comfort, it's fully generated on my phone
but not on my PC
 
@Tiffany Maybe that's another variant on the phone?
I will just get a new picture.
 
or just give a day :P
usually it corrects itself
 
@Tiffany it's been like this ever since I put the picture up. Has been more than 3 years now I think.
I guess the new avatar will propagate in a while soon.
 
4:00 PM
@NikiC ARGH! Merge conflicts again!
 
cmb
4:14 PM
@Dharman this link still works, because this extension has not (yet) been removed. So far we removed only completely out-dated and most likely unimportant extensions. Anyhow, old docs can be downloaded from doc.php.net/archives for offline viewing.
 
Does anybody have among us and up for a round?
 
4:32 PM
Hm, well, Taylor likes the short-functions RFC at least...
 
@Crell Otwell?
 
Yes.
Someone mentioned it on Twitter and responded with a droll emoji. shrug
Does it meet with your approval?
 
@Crell dunno ^^
 
I figure I'll put it on the list after Tuesday; RC1 seems like a good time to wait for.
 
Part of the problem are the completely braindead PSR formatting guidelines
 
4:37 PM
You mean the brace-on-own-line stuff?
 
In particular the really unfathomably stupid brace placement rules
@Crell Yes
 
I voted against both PSR-2 and PSR-12, don't look at me. :-)
 
@PeeHaa you and Ekin?
 
@Crell I saw the PR, but I haven't seen anything on the mailing list; was there a message?
 
Not yet. Just a PR and RFC that I've linked here informally.
 
4:47 PM
I support => on all functions. I had it in my RFC for short closures at one point. I think it was Nikita that suggested I remove it, can't remember. Maybe it was ML feedback. Curse this memory.
 
There's some PR comments asking for using fn instead, which I'm unsure about. If it could be done 100% in the lexer I'd be OK with it if that's consensus, but overall I think I prefer sticking with function.
 
@StatikStasis I just have an easier time working with Google Sheets than Excel :p You can write your own script to transform data too, if you wanted. It's pretty convenient
 
@Alesana so does Excel... but it requires learning VB Script
 
@Tiffany Oh I didn't know that
I still would rather just use Google Sheets :p
 
VB Script is bleh
 
4:54 PM
I can't remember the last time I wrote anything other than IntelliJ, KWrite, or Google Docs.
 
I used to have to write classic ASP, which also used VB Script... it was not fun
 
I want to write some IntelliJ plugins but I don't have time to learn Java right now :p
 
@Alesana Amen on both counts. :-)
 
I am wondering if it'd be better to learn Kotlin
 
Considering that JetBrains wrote it, for the purposes of writing plugins, there's a good argument to be made there.
 
4:59 PM
That was my thought! But I meant as a programming language to know in general haha
 
@Crell IMO, it should stay function. fn should stay for auto-using closures. Or something like that...
 
That's my thinking.
 
5:29 PM
@NikiC I can't find an existing portability macro for restrict; is that right? Do we really not use restrict anywhere?
 
5:44 PM
@LeviMorrison We don't, I haven't seen it once in the source code
As it's a C99 feature IIRC
 
That doesn't stop us from using all sorts of C99 features? lol
 
I mean php-src is now C99 so you can use them regardless
But I suppose nobody took care of using it conditionally before?
 
Has anyone ever used mysqli prepared statement cursors?
 
explicit cursor for loops, probably my favorite feature of PL/SQL
 
No I mean in mysqli
I am talking about this feature: ioncannon.net/programming/998/…
 
5:58 PM
@Dharman I know, but it reminded me of cursors in PL/SQL when I used to write code in PL/SQL
new job is driving up a lot of old memories of when I first learned PL/SQL
 
6:17 PM
@StatikStasis check yo steam
 
are you guys playing it?
 
yep right noaw
 
may I join? :P
 
Yes please
Ekin started the server
 
7:14 PM
What are you guys playing? :(
 
7:26 PM
Okay, so I can't push to git@git.php.net:/php-src.git ; I get a permission denied warning. If I push to https://git.php.net/push/php-src.git it works. O.o
I'm using an ssh-ed25519 key; wonders if it matters.
 
@LeviMorrison yes it does, git.php.net doesn't support elliptic curves keys
We had that same discussion last year, sent an email to the void (aka systems) and never got a reply :(
If you use an RSA key you should be fine
 
7:56 PM
@2dsharp Among Us
 
@Tiffany Is this the imposter thing everyone's talking about?
 
yes
on Steam for 5 USD
 
That's pretty cheap :D I will get it.
Is the Android version free?
 
Looks like it
If you install it we can give you the server code
 
On it
 
8:05 PM
\o/
 
@PeeHaa code please
 
@2dsharp VFLQQQ
EU server
@2dsharp Twodee?
 
Yep! I am in
 
8:22 PM
New server coming
@Ekin keeps crashing her machine
 
@2dsharp TQUIXQ
 
^
 
\o/
 
8:33 PM
Can it be watched?
 
uhmmmm not sure
Don't think so unless somebody has obs running
 
if @StatikStasis was playing he could stream it
I guess I could stream it as well, but I haven't streamed anything in a couple years
 
9:06 PM
@Alesana are you playing on steam?
 
@PeeHaa Yeah
 
What's your name on that apparatus?
We can add you to chat
 
Ah okay cool
It's shawn.james.c I think
There's like an account name and public name I'm not sure which one is used
I haven't opened up Steam in years
 
I think Christ is being born in Knutsford
 
sent invite
 
9:10 PM
I have a job there in the morning, and I'm fresh out of Frankincense and Myrh, I think I have a spare Gold-plated HDMI lead knocking about somewhere though
 
@DaveRandom Pretty sure god could afford a better night camera though
:P
 
Galaxy S7
still haven't found the energy to think about replacing it
 
Okay I accepted
 
@PeeHaa Np, just sent 24 screenshots/s I'll make playback.
 
60/sec or gtfo
 
9:17 PM
Watch it on 480p.
 
@Alesana Do you see me in your list?
 
Let me see how to find it
What's your name on there?
 
***** :P
 
Haha okay yeah I see you
I was wondering what those stars were
 
Let me restart steam I got kicked out of the game anyway
 
9:23 PM
Okay sounds good
We can do a new one
 
You still in the game?
 
Yeah
It's still confusing me haha
I think I need to watch a tutorial on how to play
 
At the start of the game in the lobby you can see how may imposters there are
The last games always had two imposters
The point of the crewmembers (non imposters) is doing their tasks (click the map icon on the left)
If all tasks are done. The crewmembers win
Imposters just try to sabotage and murder all crewmembers without being ousted as an imposter
 
okay that makes sense
I'm moreso confused on sabatoging
I thought you could kill two people at once too
Whoops
 
Sabotaging just keeps crew members from their tasks and/or gives you (le imposter) better cover to murder people
Ohai FBI! o/
Can you send a steam message to me?
I still do not see you in my list
nvm I just added you to chat
 
9:35 PM
Memory leak when xml nodes are stored into local array ・ SimpleXML related ・ #80158
 
Well at least I figured out how to kick
 
@PeeHaa What'S going on? Have not been on. Heading home now/ will be streaming later.
 
10:00 PM
@StatikStasis we're playing Among Us
on steam
 
10:43 PM
@NikiC Yay, thanks. The PDO issue is also showing green: github.com/php/php-src/pull/6220
 
@StatikStasis @Ekin and me got you a game
We have been playing among us with 5 people from the room @StatikStasis
We need 5 more somewhere next week for extra annoyance
 
11:33 PM
@PeeHaa !!
Are y'all still playing now?
@PeeHaa @Ekin Thank you both!
Downloading it now.
@PeeHaa Think I will stream this tonight now! Let me know how to join you guys if you have an opening.
 
11:59 PM
@StatikStasis I think they're done for the night :(
it was fun though
 
Just got in my first game now
 

« first day (3634 days earlier)      last day (1297 days later) »