« first day (3597 days earlier)      last day (1575 days later) » 
00:00 - 22:0022:00 - 00:00

00:58
Oh, well, it apparently takes longer than the Azure pipeline allows for. How nice.
Probably locked up somewhere
I see other tests on appveyor with debug enabled being about 1hr 15
huh, actually dev.azure.com/phpazuredevops/PHP/_build/… comes in at 1hr 14 so looks like the normal jobs are now bumping up against the limit
Oh good. So it's not entirely just me breaking everything.
01:50
As long as I'm here, what are some other functions that have terrible parameter names we should fix before it's too late?
php.net/manual/en/function.array-map.php - Can we do better than $array1?
Have you made other changes to the page already that would force retranslation?
And what would you suggest?
A few, but they've not been merged yet.
Not saying it's a great name, but I can't think of anything better (at this moment, granted, I have a migraine)
array would be better in that 99% of the time you only use the one.
$input?
You could suggest it in the same pull request?
02:04
It would need to be updated in the source first, then docs. I already got told off for suggesting a doc change without updating the code. :-)
The XML?
huh?
Or the parameter is named $array1 in php-src?
I think it depends on the situation, but at first glance, without context, it seems odd for documentation wording to be dependent directly on the source code... in this context it seems odd, but I'm sure there are other contexts where the dependency is reasonable
Processing information and word formation is difficult for me at the moment :/
As of PHP 8 named parameters mean the param name is part of the API; hence the docs need to match the source.
And we should make sure the param names are sensible before they become part of the API for realsies.
I'll get back to this tomorrow. I need to get away from the glowing rectangle.
02:47
@Crell Imo it should be dropped; no required arrays.
array_map($callable, ...$args) where $args can be empty would be legit; it would just return [].
$callback could use some work, maybe.
Transformer? array_map(transformer: $fn, ...$arrays)
Although, based on the RFC I'm not sure you can actually use variadics and named functions parameters together like this.
function map(callable $transformer, iterable ... $inputs): iterable {
    // for now I just care how named parameters work
    return [];
}
map(transformer: fn ($x, $y) => [$x, $y], [1, 2], [3, 4]);
> Fatal error: Cannot use positional argument after named argument in /in/bQD6r on line 9
Since variadics always "positional" and must always go at the end, this basically means the features can't be used together, right? /cc @NikiC
> It is possible to use positional and named arguments in the same call, however the named arguments must come after the positional arguments[.]
As far as I can tell, the answer is "yes".
I think someone will need to convince me that this is not a bug: 3v4l.org/sSVTZ.
> Functions declared as variadic using the ...$args syntax will also collect unknown named arguments into $args.
Unless I've mis-spelled something then all those parameters are known, so it shouldn't be collected into the "unknown" parameters.
@Crell Briefly skimming the other functions like array_filter, array_reduce, etc, this should be named array; this way people can say array_reduce(array: $a, callback: $fn) and those same named parameters work for every array function of this type (but not sorting ones).
I would prefer input over array, though.
 
1 hour later…
04:44
@LeviMorrison I've hit similar issue yestarday with varargs I've expected to be able to use splat operator to pass an arra of args, when I couldn't since the positional arguments where unpacked after named ones.
function foo(string $foo, ...$bars): void{}
$args = ['bar' => 'bar', 'bar1' => 'bar1', 'bar2'];
foo('Foo', ...$args);
This causes an Fatal error: Uncaught Error: Cannot use positional argument after named argument during unpacking
in cases where we expect input array from unknown source we cannot expect the named ones to be at the end and a solution with sorting by key is also not gonna work since when unpacking arguments indexes of unnamed arguments will change
A solution to that is only use of foo('Foo', bars: $args);
Which shows that named arguments with variadics can and should be used only without splat operator - the unpacking argument operator.
IMHO splat operator ... should allow unpacking arguments from array and keep all keys regardles of varargs position in function signature and don't complain about positionals being placed after named arguments.
 
1 hour later…
06:08
Integrated PAM support in PHP 8 ・ PAM ・ #80004
06:19
@Jeeves Yeah, we'll just slip that right in there for ya, pal!
06:34
> I fear this workaround will become a dead end for PHP8. This will be a major issue because OMV is requiring this feature. And i don’t think OMV is the only one that wants to use PAM for authentication. IMO this is an essential requirement for a programming language that is used in professional applications.
I didn't know such thing like OMV even exists
 
2 hours later…
cmb
cmb
08:07
@LeviMorrison thanks for volunteering as maintainer of that package :p
Hey, 81 more bugs until some lucky person gets to be ticket 80085
08:18
@MarkR Why 85?
You're '85?
Ohhh some lucky poerson, but still why 85?
Because I'm secretly 7 years old and it spells BOOBS
08:33
@MarkR Yeah, you're 7yrs old. Can't argue that :D
Bore da
09:19
Morning Derick.
cmb
cmb
Would anyone used to the ZEND_MAP_PTR_*() API like to review github.com/krakjoe/uopz/pull/131 ?
Tha e cho gaothach an-dràsta fhèin.
09:35
@Derick I heard they make ointments to help that now.
heh. for once I have a useful gaelic sentence..
I usually paste them into Google translate but that last one it just went "nope"
cmb
cmb
It's so windy right now.
Must be reading my mind O_O apparently windy.com/?51.152,-6.943,4 is about to hit
cmb
cmb
That's the translation of Derick's sentence. No wind here.
09:42
Oh :p okay then
Google Translate definitely understands it
Galican to English?
@Crell Besides this, (exception) error messages now almost always include the param name, so that's another reason why the doc need to be in sync
Probably dumb question, but why do we keep the smart_string_* API? Sure there are a couple of usages still but the smart_str_* is a drop in replacement, or am I mistaken here?
cmb
cmb
One is working with char*s and the other with zend_string*s
@Crell see also github.com/php/php-src/pull/5895; would be happy if someone would take that over
09:55
gaelic, to english, yes sure
cmb
cmb
anyway, big issue is the missing union type support for docs/PhD
galican is not gaelic :-)
Right, I know that, but what is the advantage of it working with a char* instead of a zend_string?
cmb
cmb
I think mostly for legacy reasons; a lot of stuff has not been ported to use zend_string.
Ah, fair then
10:04
There is no advantage of using char*, just the opposite. See the difference between "baseline" and the first (and second) zend_string lines: derickrethans.nl/…
Ah ha, well that would explain why translate fell on it's fade... operator error :)
@Derick Good to know :D also nice graphs :D
11:02
morns
11:31
@Derick :D you interviewed yourself. Did you do it in one recording, it do it in separate recordings then stitch them together?
Or*
Just one take. Very little editing required too :-)
I did pan my two voices between the two tracks (for left and right ear)
Neat
And I did some effects for the sample snippet
12:05
@Derick <3 :D
Tha e cho gaothach an-dràsta fhèin. ^
@Derick I need some guidance for xdebug. Where's the best place to speak to you about this.
I think PHP 8.1 could be a version where we added meaningful OO APIs. So I had a thought experiment.. of adding one to ext/json: gist.github.com/kocsismate/3936b27b2e445d8492ad395e8cb83ea7. What do you guys think about something like this?
@MateKocsis 100% for it so long as you make them all throw by default :)
although I'm not sure on why it would need stdobject / item wrapping?
@MarkR Yes, that would be my preference, too. And this is just an initial version of course, so if there's support behind it, we could bikeshed the details :)
@MarkR Do you mean that you'd go with static methods?
@MateKocsis Yup. JS does it right IMO, it has a JSON.parse and a JSON.stringify and accepts / returns native objects. Makes a lot of sense because the return value of JSON.parse might be legitimately a number, a single string etc
Hi, I have a question about OAuth2. Say for example I get access token from Auth0 Authorization Server. How does one save data related to this user on the Resource Server? Would you store the UserId that's in the token into a database?
12:20
@MarkR I agree, that's another sensible option. The reason why I started with non-static methods is that this way, it's comfortable to do multiple things with the same input (isValid(), keyExists() etc. methods). Althopugh, it might not be a major use-case.
That said, I'm ok with using static methods :)
I don't think something like keyExists etc makes sense if half the data can't be represented with such a method.
Wes
Wes
13:02
anybody knows a "community wishlist" kind of service, when people can propose suggestions and other people can vote them?
That's easy:

1. Generics
2 through 10. Something that's probably impossible
@Wes Would you want to pre-validate the wishlist items?
@Wes From a quick google search: upvoty.com
Wes
Wes
yes, ideally
I'm sure there are free / open-source alternatives.
Honestly, they're probably gonna tell us what we already know they want. Generics, enums, drop the $, make -> to . :P
Wes
Wes
method overloading!!!!111!
13:09
@Wes No, but thinking about making one....but one problem is that just curating that would be a lot of work. And telling people, "no, that's a stupid idea" would be frustrating for everyone.
@Danack Shouldn't we look if there are existing solutions first?
@IluTov haven't actually done anything but have been thinking about it. I still think the biggest problem is how to restrict discussions to only people who have enough done work to be able to actually have a positive contribution to make, rather than those who just want their voice heard.
@Danack This one for example is open source: getfider.com
@Danack Discussions, or voting?
@IluTov have you actually used that? It looks more like a bug reporting tool than a thing that could be used for helping people have focused discussions.
13:14
@Danack "You can choose to control access to your site by making it Private." Sounds like it should cover that, we could give out user accounts based on how well you behave on the ML.
@Danack Nope, literally just found this right now.
@MarkR Discussions, but more in the sense of "lets work on this thing" rather than "I want generics, why haven't they been done yet".
@IluTov we already have an auth system for people.php.net. We could really do with exposing that as an api...
so that it can be used as auth on other things.
@Sara Did you make the announcement yesterday of beta2?
@Danack Yup. I really want that for my prototype redesign
@Danack But do we want to restrict discussion to people with a php.net account? I thought the point was to get feedback from the community.
The problem isn't necessarily feedback, it's that certain people monopolise the feedback... usually by a factor of 20 or so
13:17
@MarkR If I wrote some words on how I think it could work...would you have any time to write some code for it? Or something equivalent...
@Danack You mean the API from master.php.net? OAuth would probably be the way to go
But i've no idea who / if that infrastructure is maintained.
@IluTov yes. first, fuck the community feedback. The vast majority of PHP users just aren't that clever. And many of the ones who are clever focus solely on their own little use-case for PHP. Second, discussions need to be under a code of conduct where we are able to kick people out if they show that they are a negative contributor.
@MarkR ...I should probably know this by now, but does Oauth expose identity as well as permissions?
@Danack Usually the token that is returned contains permissions to query the identity
@Danack Do you expect this tool to (partially) replace the ML? If so, is there any way left for the community to give feedback?
I mean it doesn't have to be oauth, a simple JWT is plenty to verify a login.
But I'd still suggest OAuth, existing libs and much less chance of introducing a gaping security hole
13:25
when I run php -v I get a confirmation that my mac is using 7.4..version
I went to php.ini and set the memory limit to -1, but when I check again though the console it still tells me 128M.
Any idea how else I do change the PHP memory limit on my mac?
cmb
cmb
@useragent check that you changed the proper php.ini: php --ini
@cmb I get this after running php --ini

Configuration File (php.ini) Path: /usr/local/etc/php/7.4
Loaded Configuration File: /usr/local/etc/php/7.4/php.ini
Scan for additional .ini files in: /usr/local/etc/php/7.4/conf.d
Additional .ini files parsed: /usr/local/etc/php/7.4/conf.d/ext-opcache.ini,
/usr/local/etc/php/7.4/conf.d/php-memory-limits.ini
@Derick wow... what a crazy time we're living in.
cmb
cmb
13:29
@useragent then you'll likely want to have a look at /usr/local/etc/php/7.4/conf.d/php-memory-limits.ini
@IluTov Not directly.....my general view is that each 'community' should have their own discussion, and work to produce a single output to send "upstream" to other communities. i.e. people in the symfony community could work on giving feedback on an rfc and saying the result of that discussion as a single voice. But realistically....people just don't want to do the work to do that as writing clear feedback is more work than people want to do.
I just did that and changed the limit to -1 as well....testing @cmb
at least I already get -1 when I run this: php -r "echo ini_get('memory_limit').PHP_EOL;"
the point at which community feedback becomes more useful is when we have enough spare developers who are looking around for what things should be worked on next, which is almost certainly not the case currently...
Problem solved @cmb :) :) :)
Thank you
cmb
cmb
yw
13:36
I should write more thoughts down, but to set expectations again:
@Danack If we at least partially move the discussion to a private platform, would you lower the barrier to get a php.net account so that people like @Crell and @IMSoP can still partake in the discussion? Honestly, I think there are quite a few people with voting rights who know less about how the language works than some with no voting rights.
@IluTov Crell got approved :D
@MarkR Oh did he? Nice :)
@MarkR This could also be useful to detach where the vote is held, for example on a tiny website vote.php.net where we don't need to deal with docuwiki (also makes it easier to move RFC around...)
@Girgias Yeah if master API gets exposed, a proper voting system is first thing on my hit-list
13:41
Also, how do we know what people qualify for internal discussions without them having partaken in any major discussion? Quite obvious for those who contribute code but not those who don't.
@IluTov The problem is that because we are tied to docuwiki and there are no restriction so anyone who has a php.net account can vote, which is over 1000 people, which is pretty bonkers...
@IluTov yes creating an account should be easier. and to be clear, the platform can be public readable, it's just the posting requires authentication.....and we don't need to restrict it to just people.php.net. If the symfony community wanted to gather their thoughts, and push a summary of their thoughts to PHP internals that would be useful.
@Girgias I should work on my voting platform some more....which is actually the thing I want the identity/auth thing for first...
@Danack I mean I would expect the "hard" part is to tie into the current php.net infrastructure, then initially it's just a "dummy" system which allow anyone with a php.net account to vote
Because restricting voting power is going to be a bikeshed on its own which needs to go through the RFC process
@MateKocsis i don't like it being an instance of Json, because in JSON the difference between [] and {} is important, and you wrap it into an object now, so you don't know if its a list or an object
@Danack Makes sense. To be clear: I'm absolutely in favor of restricting who can post. This is probably the only realistic way to keep the platform from decaying into chaos. But there needs to be a way for them to join discussion without getting any other privileges to the php.net services.
13:47
Unable to load dynamic library .... undefined symbol ・ Dynamic loading ・ #80005
laters - I need to go crack my neck. in the meantime have a video that explains how people view colour: youtube.com/watch?v=FTKP0Y9MVus
@Girgias Yeah that is bonkers. Proposing to revoke accounts after a certain time of inactivity would probably not go down well.
wasting 5 mins because PHP truncates error messages at NUL-bytes ...
@IluTov I think it would be "reasonable" to say 10 years of no activity on list/commits to any PHP repo
@GabrielCaruso fwiw I'm pretty sure @NikiC mentioned this was expected and correct, I also noticed that change
@MateKocsis I'd love to have a built-in implementation that takes an iterable and outputs JSON to a stream. This should save a lot of memory in some use cases.
@GabrielCaruso ah, thx didn't see that
@Girgias We probably also need to take contributions into account. If Andi, Zeev or Rasmus don't contribute for 10 years, they should probably keep their accounts. Which makes it hard to come up with a strict rule.
Also hard to define contributions, does writing to the mailing list count?
^^ totally not a Friday link :)
That's why I said activity, not contribution. It's not about losing the PHP account it's about having voting rights removed as they should not be tied together.
I am pretty sure even Z did say that at one point
13:59
@IluTov i was also thinking about proposing the same thing (although, my limit was 5 years)
probably during the whole short tag deprecation mess
And what do you think about revoking commit access from inactive members? I think it's also some kind of security threat that 1000+ people can push anything to php-src.
@MateKocsis Absolutely.
I know that it's OS, so commits usually get reviewed... But I can imagine that some might slip through any meaningful review (e.g. when Nikita is on vacation 😃)
cmb
cmb
I'm afraid there is no solution to the voting rights issue. A time limit is useless, since those who are interested in voting would just push a harmless commit from time to time. And getting a php.net account is also pretty easy (just apply as translator, and provide a couple of translations).
14:07
@TheodoreBrown oh, that's a really good idea! As far as I see, it would work pretty well with psr-7!
@MateKocsis That's not true tho
Not everyone has php-src karma
Ah, that's a bit better then 🙂
@Girgias But still probably hundreds, right?
@Girgias somewhere ~250 with php-src karma and maybe another 100 with restricted karma somewhere in that repo
14:09
Until last year there were two different karma tiers as well, Zend/ had special karma rules
@salathe right, and we should probably revoke access from time to time, the question is how do you decide upon it
Active users of that karma... probably you can count them with your fingers (maybe toes).
I can't seem to find the file on git.php.net which lists all the people which have specific karma :(
Uh pretty sure there was meant to be a flags param in that stringify() somewhere
@Girgias The thing is also, it would probably take 1 e-mail to request the access back if you want to contribute something. If you've already been approved before that shouldn't be a huge problem.
@Girgias svn.php.net/viewvc/SVNROOT/global_avail?view=log although it's erroring out for me
14:12
Or rather, if you want to get back to contributing regularly. If it's just once a year someone else can merge it.
@IluTov sure, but there are certainly people who won't do it
@MarkR Ah yes it's on SVN...
@MarkR yeah don't try and view the commits, you'll make the whole server grind to a halt
@IluTov Or you wiuld have to push an empty commit every year 😃
@MateKocsis If you do that on more than a few occasions you'll probably lose access anyway :P
@MateKocsis git commit -m 'bump copyright year' :D
14:14
@salathe Well, that's pretty much exactly what people are suggesting. Probably didn't work out well if it was discontinued.
Haha 😃
@GabrielCaruso Uhhhhhhh shite
@IluTov It was fine (for some definition of fine). It was only used during the website changes, and when it had served its purpose we closed it.
@salathe Ohhh I see. If the moderation effort is not too high maybe that might be worth reconsidering. We could use that tool to gather feedback from the community and a different tool for internal discussion.
Speaking of which... any appetite for using docs.php.net for docs rather than having a million pages on php.net?
cmb
cmb
14:19
@Sjon, are you planning to pursue github.com/php/php-src/pull/5937? The compile failure appears to be resolved in github.com/php/php-src/pull/5959 (which looks pretty identical to your PR), but there are still some test failures. Point is, I'd like to close either PR. :)
Because if I end up backing the main website with a DB it's going to be next to useful for rsyncing
@MarkR overall, I like this interface very much! I only don't like the stringify name. JsonEncoder::encode() would be more straightforward for me. Or maybe JsonSerializer::serialize()? What would you think about the 2 other methods I used: toArray() and toObject()? So that we can have more accurate types, and the $assoc optional parameter is not needed
@MateKocsis Would lead to some measure of confusion IMO as $assoc really just determines if object types are converted to assoc arrays during parsing. Would also need to throw a different kind of json error (or add new error flags for unexpected type) if the parsed result wasn't either but was still legit json
Yes, that's exactly what I had in mind! Nevertheless, there's plenty of time to bikeshred the details. But first, let's release 8.0 🙂
That sounds like a Sara and Gabriel problem now tbh :P
If anyone has a write up on how to compile PECL modules for 8.0 on docker that would be handy, I've been meaning to try it before public beta on Nov 26th
cmb
cmb
14:32
pecl install … ?
> The name Zend is a portmanteau of their forenames, Zeev and Andi.

TIL
@PaulDragoonis here is fine
@IluTov Being on wikipedia?
@Girgias Yeah. en.wikipedia.org/wiki/Andi_Gutmans Andi is also Swiss apparently (or at least born here) ^^
@cmb That's odd... I was missreading the dockerfile logs, it's actually pecl itself that's not found, not redis
14:36
@IluTov His page is way more detailed than Zeev's, and I added some info on his cause it was lowkey depressing
Nobody has ever written mine :-/
Which is surprising considering you've been an MP
I've ... not been an MP
Why did I think that then...
WOuldn't you think I'd be using that title? :-)
14:39
True, you were a candidate however :') sorry for putting you in power accidentally :p
> Did you mean: device returns
o_O
@Girgias For councillor, not MP.
@Tiffany Algorithmic cruelty :-)
@Derick Aaaaa, ok
@Derick okay. I have a kubernetes/docker container running remotely on a server. I appreciate you may not know the answer in this context so I hope we can just pretend we have a remote shell on a remote SSH server. This remote shell we're running "phpunit" command line. Once we run this command then it should trigger a breakpoint in my IDE on local machine. Is there an online guide covering how to deal with remote CLI executors (versus remote WEB UI ones)
If you're already ssh-ing into it, just set up an SSH tunnel and forward remote port 9000 to local port 9000, something with ssh -R 9000:localhost:9000 yourhost — then set xdebug.remote_host = 9000 and make your IDE listen on port 9000 too. You can start phpunit with:
XDEBUG_CONFIG="idekey=dr remote_host=localhost" path/to/phpunit
14:50
Todays hunt: somehow deleting dead code increases my memory used by 1.5MiB.
@Derick @Derick good news! the project here is already using "ngrok" to forward port 9000 to localhost. So that's done. Perhaps it's a matter of .ini configuration and IDE config.
you can make the xdebug.remote_host setting in php.ini too, and if you always want to debug, set xdebug.remote_autostart=1 and then you don't need the XDEBUG_CONFIG env var
isn't ngrok used for something else?
Maybe you're right, and I'm saying the wrong tool name, but they already have it working for Web UI, and can't get it to work for CLI. So here I am
@Derick @Derick what's your advice for me ? (it's for a customer project, so it should just be simple) - settings side.
@Derick DERICK FOR MP! DERICK FOR PM!
2
cmb
cmb
@Sara, is it too late for further resource to object conversions in PHP 8?
14:57
Sorry for the double name ping, this stack overflow tool is a bit quirky ;)
If web already works, and the php.ini's are the same, just running it like:
XDEBUG_CONFIG="idekey=dr remote_host=localhost" path/to/phpunit
should work
@Sara You know how much work for how little pay that is?
@Derick how's it compare to open source contributor?
contributor? :D
MPs make £82k, excluding expenses
and bribes, can't forget the bribes
I'm not a Tory.
cmb
cmb
15:11
@IluTov, you may want to fix the merge conflicts of github.com/php/php-src/pull/5922 on WE or next week, and ping Dmitry for review. :)
@cmb Yeah I'll do that. It also fails on Windows, I was hoping I wouldn't have to set it up :P
I'll also need help with nullsafe JIT. I'm utterly confused how jumps are implemented in the JIT. I can only find a few referenced of some opcodes (like ZEND_COALESCE) but no real implementation of those. And I can make JMP_NULL (the opcode for nullsafe) work by adding it to the same places, but I have no idea how :D
cmb
cmb
@IluTov Ugh, JIT issues; maybe these are already resolved? Otherwise I can have a look, but likely I won't be of much help there. :(
@cmb Probably not, it's my PR that doesn't work. I was hoping it would be obvious to Dmitry but I'll just have to set up a Windows machine.
cmb
cmb
you likely want to install VisualStudio instead of the MSBuildTools, though
@cmb Thanks, that looks like a manageable installation ^^ I'll try it tomorrow.
cmb
cmb
15:24
Doesn't quite work with Visual Studio, though. But the manual install isn't really hard (you only need the "Desktopentwicklung mit C++" Workload with default settings).
15:37
ohai again room
@PeeHaa o/
@PeeHaa \o
cmb
cmb
Anyone seen news-web.php.net/php.pecl.dev/16898? Not sure what to do there. Add Jesse as lead?
IMO, it should move to PECL?
unless it already is. Otherwise, I think we can add him as lead
@LeviMorrison The problem with dropping all explicit arrays on array_map() is that you then can't use an explicit param name to specify the input array. At least, I don't think so... Also, changing the behavior of array_map is out of scope (and my skillset at the moment).

Standardizing on "input" and "callback" works for me if it works for everyone else.
15:48
@Crell Fair.
@Crell There is an issue on the task repo: github.com/php/php-tasks/issues/16
fileName O.o
/closes tab and flips table
@Girgias Oh! Exciting.
@MateKocsis Lots to bikeshed, but I'm generally on board with the JSON parser object direction.
16:06
@cmb The batch script calls phpsdk_deps but that doesn't seem to be available at that point, do I need to add C:\php-sdk\bin to the path variable? When I call it directly it directly I get this error:
> INFORMATION: Es konnten keine Dateien mit dem angegebenen
> Muster gefunden werden.
>
> Fatal error: Uncaught SDK\Exception: Couldn't execute cl.exe. in C:\php-sdk\lib\php\libsdk\SDK\Config.php:66
> Stack trace:
> #0 C:\php-sdk\bin\phpsdk_deps.php(120): SDK\Config::getCurrentArchName()
> #1 {main}
> thrown in C:\php-sdk\lib\php\libsdk\SDK\Config.php on line 66
probably missing something obvious
I seem to remember CL being part of visual studio
Oh sorry, I thought that was an internal tool.
I did check "Desktopentwicklung mit C++", let's see why it's not installed...
@MarkR not exacly, there's a lot of things to finish 🙂. E.g. warning to error promotions, fixing default value handling for named params, improving param names, or syncing the manual with the stubs
or any ABI breaks
16:29
@Girgias surely that needs to get sent to internals and reddit, and definitely at least 1 RFC...
@bwoebi (or anybody else), what am I doing here wrong wrt to arginfo (PHP 7.4)? gist.github.com/derickr/7370f19e19346d01bd6976c7e20e5ff2
Windows is harassing me. God I hate that platform.
@salathe Obviously it needs to be sent to The Whimsy International Typing and Topography Earthling Representatives AKA Twitter
@Derick where's the issue? there are 3 names, you try to access the fourth?
ah well
@Derick access these as char*, not as zend_string*, cast them first, it's an internal function
16:44
Hello can someone help with Slim3 problem?
i cant access to the files in Uploads folder.
@Derick If you access zdata->func.internal_function.arg_info[0].name that should work too
@bwoebi So they're not zend strings? :-)
@Derick Correct, they're statically defined rodata
ah, okay, that does make sense. Although, it's confusing
just bare C NULL-delimited strings
16:46
now I need to turn them into zend_strings :-/
@Girgias oh yes there too; got to make sure to poll the twits!
cmb
cmb
@IluTov ugh, I think that can't work (thought I checked that, but probably not). Point is after you installed the SDK, you have to start phpsdk-vs16-x64.bat (or -x86.bat) in the php-sdk installation directory. Then you cd to php-src, and are good to go.
@cmb Ohhh, that was it, it's doing something now ^^ Thanks :)
Why does VS make it so damn hard to add the executables to the path??
cmb
cmb
and one particular gotcha (for those used to building PHP on other systems): name test does not rebuilt anything; need to nmake first
@IluTov because on typical Win dev setups there are several VS versions and multiple Win SDKs, and the script tries to setup the correct ones for the PHP version.
That stuff is mostly written for the PHP Windows maintainers.
17:13
@cmb It works :O Thank you!!
cmb
cmb
17:24
yw
17:53
doc question regarding null coalesce assignment operator, should documentation for it go - php.net/manual/en/language.operators.assignment.php?
18:11
there's documentation for the null coalesce operator here: php.net/manual/en/… but I was asked about the assignment operator specifically (??=)
It seems like those should go together?
Oh, I see what you mean. Hm.
@Tiffany My gut feeling at the moment is to put it with coalesce, and then link to it from assignment. But I could probably make a good argument the other way around, too.
@Crell yeah, I think other way around would work best
As long as it cross-links I think either place could work.
Oh, yay, this time the PR tests passed. Why? No idea. Timing is everything. github.com/php/php-src/pull/6027
@Crell yeah, I added a readme to ext/mysqli/tests that was just a markdown file, and one or two of my commits "failed," I'm guessing because I happened to push them in between failing builds, then the final commit passed
:sigh:
@Crell You need to check the CI failures, because we have some flaky tests due to networking or FPM, and sometimes the CI pipelines can't update APT and just dies
Yeah, it timed out before. This time I renamed a variable and it passed. So, meh.
18:55
@TheodoreBrown What does "Seats" represent on your results page for the vote?
19:09
@StatikStasis the options, I think
Great, so that was all for nothing, tests pass on my Windows machine xD (With thread safety and AVX intrinsics as in the build)
20:11
@StatikStasis The number of candidates to elect. E.g. in the release manager election there are two seats. For attribute syntax only one candidate can be elected, so one seat.
Ah, gotcha!
You can actually override it with &seats=2 in the URL and see that the quota decreases and both #[] and @[] would be elected.
nice!
@TheodoreBrown I like your propaganda link you've added. =P theodorejb.me/2020/08/21/…
And there's another PR to rename more parameters. Low hanging fruit FTW.
@StatikStasis Haha, yep :D
20:22
(We'll see if this one times out.)
20:52
Why does zend_get_type_by_const return "number" for some zvals on Windows? When can that happen?
Only place it showed up that I can see is casting an object
@MarkR I can't reproduce it locally, it just happens in the CI :/
which test?
There's no explicit casting, maybe it has something to do with the var args? I have no clue. Not being able to reproduce it locally makes it pretty much impossible to solve.
Or maybe the underlying value is just matching by chance.
21:17
I'm not sure :S I'd probably start by dumping the entire zval data in the error message and then hammering CI again
Maybe try without the short closure as well
@MarkR It might be an existing JIT bug that was not triggered in any of the existing tests. I'll ask Dmitry to take a look. Thanks!
Not unlikely it seems :(
cmb
cmb
@Tiffany it only deserves special mention on that page, if it is special. I'm not sure if it is; the long discussion on the final implementation is more about internal details, but there might be some actual edge-case that might be worthwhile to document.
@IluTov I did a typical debug build (configure --enable-snapshot-build --enable-debug --disable-debug-pack --disable-zts), and when I'm running the test (nmake test TESTS=ext\opcache\tests\match\005.phpt), I hit an abort. Quick debugging shows that zend_get_type_by_const() is called with type=236. That happened already several times, and I think it always was due to erroneous register usage (it's different on Windows and Linux).
21:37
@cmb Interesting. I wonder if that's in my code or in existing code. That must be an absolute pain to debug.
@cmb friend of mine messaged me asking why it wasn't documented
problems(?) of having RL friends who're PHP devs
cmb
cmb
@IluTov if you want to debug, see github.com/microsoft/…
What I was also wondering, Dmitry switched to tracing JIT by default, which means lots of code will never be jitted in CI. Should we also add a build with opcache.jit=1205? I'm afraid this could allow bugs to sneak in.
cmb
cmb
and yes, a PITA, if I even knew the basics of the JIT implementation to start real debugging ;)
and for Dmitry that means switching to Windows, what he doesn't like
not sure if we should have a 1205 built; maybe it was just temporary, but when we switched to tracing JIT there were issues on Windows (and likely on Linux, but these had been already fixed)
@Tiffany well, none of the combined assignment operators are explicitly documented (there's an example showing +=, but that's about it)
I'll find out what he's wanting documented, not sure if it's implementation, how it works, examples, or what...
cmb
cmb
21:44
good idea!
and thinking about it, it's probably a good idea to document the exact semantics of combined assignment in general, namely that $a +=$b is sugar for $a = $a + $b with the exception that $a is evaluated only once
that's true, and would get rid of the top comment (I think)
I think it would be good to have a table with all of the assignment operators
cmb
cmb
I'm fine with that. :)
@cmb If we don't it's very hard to write tests that actually test the JIT. You never know what part of a function the JIT will actually decide to optimize. I found a bug that only gets triggered with the 1205 flag.
argh, pushing my brain too much, not fully recovered from migraine :/
cmb
cmb
@IluTov Well, we could add tests for any newly found bugs as they are noticed (that's called bug driven developement or BDD). ;) But yeah, may make sense to add another configuration, if we have the capacities. Not sure we have.
00:00 - 22:0022:00 - 00:00

« first day (3597 days earlier)      last day (1575 days later) »