« first day (3236 days earlier)      last day (1705 days later) » 

2:30 AM
str_getcsv parses the CSV error – #78457
 
 
4 hours later…
6:37 AM
> β€œIt’s all gone quiet,” said Roobarb....
 
7:15 AM
Morning room
 
7:25 AM
morns
 
Morning
 
8:00 AM
wangle to resort to trickery
 
8:59 AM
 
@Jimbo it sure is.
how'd your di talk to Go people go?
 
Good thanks! :) They were all really nice, supportive, told me they really liked it etc. Even though I fucked up on the live demo :D First positive thing I had from the Go community
So now I just need to find some Go conferences that aren't in November
 
hmm is that a bug? 3v4l.org/bT3cH
does a space needs a backslash even if its inside qoutes? The mailsevers i tested agaist dont need that backslash.
 
9:21 AM
> FILTER_VALIDATE_EMAIL
@Jimbo you should record these........
@user3655829 it's a complete mess. If you need to validate it, google have some code that does the job 'properly'. Otherwise.....validate it by sending an email to that address.
 
@Danack Yeah it was only a group Meetup, I'll record it for a major conf
 
I did a live demo at my last talk......it went so well people didn't believe it was live.
 
@Jimbo will video be on youtube??
 
@Linus If they record it at the next major con it will be
 
Hello hello there
 
9:36 AM
@Danack Haha, nice. What was it? I built an "mvc framework" foundation in 3 minutes with a routes.yml and auto-wiring.
With a "shared" db connection injected further down in a repository
 
@Danack I thought about that this method is not fixable. But still someone tried to use such a mail in a system i maintain. Clearly the solution is to filter all emails with spaces :)
 
Nice, RabbitMQ stuff. Also your code only shows as black in those slides
 
@Jimbo are you on a good or terrible computer?
aka works for me.
(thanks though, I will fix that at some point)
 
yomin
 
9:41 AM
o/
 
> the __HALT_COMPILER(); token
without checking, that implies that the whole thing is parsed as a single token, rather than the normal string and brackets and semi-colon.
 
Even Chrome cant validate emails properly :(
 
10:01 AM
@SebastianBergmann oh laud. It's not even in normal PHP, it's part of the phar extraction...
 
10:22 AM
Nice :-/, I guess.
 
@Danack no it isn't
__halt_compiler is the token
The issue is that phar only supports the upper-case variant
 
10:54 AM
would valgrind php-fpm also look at the children handling requests? the log output is so quiet I was expecting more :->
man page says valgrind goes into fork automatically
 
@beberlei --trace-children=yes
 
11:08 AM
@NikiC thank you
 
Good afternoon all.
 
11:24 AM
Good morning.
 
A glorious British bank holiday... too hot to do anything, too busy to go anywhere, and too short to feel like a break.
For those that are similarly lacking in productive outlets today, I'd appreciate any feedback on the discussion for an error reporting API
 
whatchoo talkin bout ?
oh I see it ...
it's really long ...
 
tl;dr: Should we have an error reporting helper to format errors that relate to a specific parameter
 
probably, there are already routines for throwing type errors for param that take an idx and arginfo (I'm pretty sure of that), I would see if they cannot be extended so that all messages are always consistent across all usages ... in general I don't love the idea of adding a new API and it would probably be better since this is targeting 8 to change an existing one ... right at the bottom of the list of things php needs is more API variations or magic macros ...
if you're going to be fetching arginfo, try to use cache slots where applicable, if applicable (I've no idea if it is applicable)
 
11:42 AM
I wasn't planning that far ahead, I've been passing the parameter index to the function and just formatting the error as "Parameter %d is invalid: %s", grabbing the param name would be nice, but not essential
 
@nikic I think you forgot to update the following test ext/standard/tests/array/extract_error.phpt (Test extract() function (error conditions)) when you comited 'Avoid duplicate "non well-formed" warning' or is it from something else?
 
@Mark
> Fatal error: Uncaught TypeError: Argument 1 passed to param() must be of the type %s, %s given, called in %s on line %d and defined in /opt/src/parallel/%s:%d
that's the normal type error, and I sorta think, it should read something like
> Fatal error: Uncaught TypeError: Argument 1 passed to param() must be %s, %s, called in %s on line %d and defined in /opt/src/parallel/%s:%d
so then possibly you could reuse the routines that produce that message ...
these are just ideas, they may be wrong ... and it's always safe to ignore me :)
 
Nah that seems like a good idea tbh
 
that's part of zend_verify_arg_error from the looks of it, all of them are just formatted in that function
Unless I'm missing a global somewhere, I don't think most of that information is passed through using internal_function_params
 
well it couldn't be ...
show me some code, an example of the problem you see ...
 
11:54 AM
Morngins o/
 
I think a majority of the time, you can determine the index from the zval* ...
 
github.com/php/php-src/pull/4559/files This is the PR that probably needs to be broken up. But as an example:
 
I see
then you'll need an api that mirrors zend_verify_arg_type (or whatever it's called) and produces that similar output ... it's just a shame there has to be new API ...
it might not be a mirror, you'll want the frame pointer ... (to do called in %s etc)
at least keep it somewhere close (same file), so that anyone doing CTRL+F for string in the original message will hit the new one too possibly ... and name it with zend_ prefix obviously ...
 
Assuming the executor globals are available inside a PHP_FUNCTION, I would think it should be possible to get c lose to that error style
Then again, these are not user functions so don't appear to carry the "called in line xxx" bit
 
12:15 PM
Morning!
 
12:33 PM
@NikiC Any chance this could be fixed?
 
@MarkRandall imho the parameter number doesn't really add much
I don't typically include parameter numbers when throwing exception in php code either ... do you?
The semantic name is what is useful
 
TypeError does, which is where I was approaching it from for consistency
 
TypeError does because it's a very generic error that doesn't know things about specific functions
This feels like a solution in search of a problem to me
 
I more considered it a single point that could enforce formatting and potentially be expanded with internal references to the argument info to translate the name, but I'm fine with zend_throw_error if there's no need for that
I didn't want to get into a situation where an implied parameter name given in an error message might be different from the one given in docs or reflection.
 
cmb
12:51 PM
@MarkRandall, ACK. Also there are some messages which do not even include the param name.
 
Okie dokie. Well if there's no need to enforce consistency on them, i'll get them resubmitted as just plain Errors
 
1:07 PM
I did have one other Q, which was stripping the false return type. How should we handle that? If this is partly to clean up the return types I'd think just returning null would make sense along with the exception, but obviously that poses somewhat of a problem
 
What is the best micro-framework for php? any suggestion? I am thinking about zend-expressive, but if someone has a better suggestion, please share.
 
@mega6382 if you are into all the psr stuff go with slim
 
@PeeHaa well, expressive is the same way, psr-15 middleware, psr-7 http etc.
 
In particular I'm thinking of the case when using @ to suppress errors
 
Yeah
 
1:12 PM
@MarkRandall if it's converted to exceptions, the false return goes away with it, right? Or do you mean the case where we keep a warning?
 
@mega6382 Hence I said if you are into kinky shit like that go with slim :)
 
@PeeHaa A bit of context, I am trying to build an app using the "Hexagonal Architecture", and since it requires that the domain layer be independent of the framework, so I thought that it would be easier to use a micro-framework. There are only 4 use cases, so it is very small app.
And I don't have any experience or knowledge of "hexagonal architecture", nor have I had to use any micro-framework in the past(for php, that is).
 
It's general best practice either way to decouple you domain from whatever framework you are using
Some frameworks make it just really hard to do :D
Pretty sure both slim and ze and even symfony allow you to do it
But if you are looking for micro I would go with slim as seems much more maintained
 
Thanks <3
 
cmb
@NikiC, just occurred to me: should we add @throws to the stubs to aid updating the docs?
I mean except for TypeError
 
1:18 PM
Perhaps I'm misunderstanding how the error suppression operator works. If I call a function that throws an error, but I suppress it using @, the exception is thrown but the script just terminates without an error message. If there is no suppression operator the error is printed as per E_ALL
 
How the stfu operator works: you use it, god kills a kitten
:P
 
@MarkRandall @ does not suppress exceptions
or the "uncaught exception" error for that matter
 
I'd best go dig into the INI, using Docker and 7.4beta2 when I run gist.github.com/marandall/7739d16ddbc3cbff5e5723d12756dd87 I get no output and exit code 255, if I remove the @ I get the expected uncaught error message
However both of them throw, so it answers the Q about the return values.
 
@MarkRandall I always get the error message, regardless of php version
 
I see it displaying the error on 3v4l too, which does somewhat raise more questions than it answers.
 
1:37 PM
@MarkRandall any questions I can answer? Possibly because display_errors is on by default on 3v4l
 
I just recompiled the docker container to have display_errors=on in the INI, same effect. 255 exit no message when suppressing with @
 
do you have xdebug?
 
I disabled it for the last run, same effect
 
weird
 
O rly, this is interesting.
It's opcache. If I remove zend_extension=opcache.so from the INI it works as intended.
 
1:55 PM
@MarkRandall Well congrats
That looks like a legit bug in 7.4
It works in 7.3 and master, but not 7.4
It looks like we're not reconstructing the silencing live range...
 
Cool, well at least I can stop worrying that I'm missing something obvious.
 
2:13 PM
@MarkRandall Should be fixed with github.com/php/php-src/commit/…
 
/me waves
 
/me surfs on the wave
 
2:30 PM
extension_dir="./ext" in php.ini is not working – #78458
 
 
2 hours later…
4:15 PM
Please Provide The Feture Of AM/PM Comparission. – #78459
 
@Jeeves Reading that ticket gave me a headache bruv
 
Now for your feture presentation
Spelled like a drunk would say "comparison"
 
I get people use different languages, but spellcheckers are built in nowadays
 
I am guessing something similar is built into bugs?
 
@StatikStasis I mean all browsers have it
Look at the red line under your search term ;)
 
4:25 PM
Yeah- sorry it is browser. I'm not thinking. I just had a conversation with someone at Slack about their app having a redline under it when it is actually a word and they said it uses a google library and I was just mixing up their app and my browser up in my mind. I wasn't thinking before I typed that. @PeeHaa
 
:-)
 
4:40 PM
@nikic about the array_push warning2error conversion, the AST and JIT use the same error massage which means that const FOO = [PHP_INT_MAX => 42, "foo"]; isn't handle by zend_cannot_add_element
Should I modify does at the same time or not? Because if I do I'm not sure this doesn't also impact unpack and if it does only array_fill would still use the warning
So should I promote all those warnings in one go or still have them split ?
Okay for the JIT bit there is only this section which I would need to edit lxr.room11.org/xref/php-src%40master/ext/opcache/jit/…
(from what I understand)
Well okay maybe not for the JIT
I'm just confused :( I didn't sign up for assembly :(
 
5:21 PM
github invented time travel
 
lol, nice
or like, very good predictive models
 
When your profiling performance is so fast you break causality.
 
6:36 PM
Thing about the PHP future debate is, some people want to go one direction, some people want to go completely the opposite direction.
So for naming, I propose the current version of PHP sticks with "PHP" and those that want to go the other way, create a fork, and use the acronym in reverse.
 
I disagree. I think 95% of the direction is actually the same, and where people really disagree is in the 5% corridor (and make it feel like it's 95% disagree)
 
Right, a thing most people don't understand is that discussions are not representative of actual opinion at all
 
though, the fork solution sounds good
 
Discussions are always dominated by dissenting voices, even if there are only few of them
 
But even that dissent is often in a narrow band of functionality and feature set relative to the overall progress that's been done to the language
when I think about 7, there were really only a few contentious RFCs, and those that were contentious tended to be about details not concepts (even if those details were major)
 
6:40 PM
Namespaced declares will help a lot IMHO.
 
I think whatever is done, as long as a clear and easy migration path, that's like 90% of the battle
 
Delete wordpress. That's half the problem solved ;)
 
@MarkRandall I have yet to meet anyone who is concerned about breaking too many things too fast. Everyone I talk to wants more stuff to be cleaned up.
 
I'm certainly one of them Danack. I think there's a lot of potential for PHP in the next 10, 15 years, provided the tools are there. That we're having to go through the language and add extra type-errors because functions are quasi-overloaded is proof that there's improvements to be made.
For what it's worth, I'm willing to put in the time to help make those improvements.
 
6:58 PM
btw - if anyone looking for a remote developer, due to events unfolding not necessarily to my advantage, I'm going to be available from...tomorrow.
3
Or if anyone can recommend a recruitment person/agency for remote people, I'd appreciate having some recommendations.
6
/looking for a job, just in time for the recession is great timing.
 
@Danack =(
 
@Danack do you C or Java?
 
@Gordon I look after the Imagick extension, so I know more than a little C. I've done Java.....this millenia. But it might have changed a bit since I last touched it. (also most of my experience was j2me rather than j2ee).
> I know more than a little C
fyi, my current CVE / year rate is less than 0.2
 
7:16 PM
@Danack shot you a message on LinkedIn
 
cool.
 
Damn Danack, sorry to hear that
 
7:33 PM
Kinda disappointed no-one pointed out my proposed naming scheme for the opposite version of PHP was still "PHP" :P
 
Pls someone help me with docker
2
Say I have a git repo. And I need it to build the container
How do I use that as a build context without it doing something stupid like copying .git and without me letting docker crap all over the place (aka add a .dockerignore file into the directory I need)
 
Something preventing just cloning the single branch and then deleting the .git before running the build stage?
 
I don't want to touch the directory in any way
 
7:48 PM
Hmm, I've not heard of docker adding a .dockerignore file, I thought that had to be part of the repo.
 
@NikiC can you share a repo and say what you want done? I've also never heard of docker modifying git repos.
 
I think I'm asking the wrong question here
Say I'm inside a php-src checkout that I don't want touched and don't want to touch. How do I build PHP inside a docker container from there?
 
My suggestion would be to copy most the build process from the official docker PHP files. You can keep your Dockerfile in a different directory and then change the build context to point to your src dir.
 
If I point the build context to the source dir, won't it include the .git directory as part of the build context?
 
ah right. I think I have deja vu - github.com/Danack/php-compiler/commit/…
That's the same thing right? (sorry just eating).
 
8:02 PM
You could potentially use your own .dockerignore in the parent to remove the .git from the build context.
But it wouldn't be much different than rm'ing the .git directory beforehand
 
8:17 PM
@NikiC you need that .dockerignore file in the root directory...
@NikiC Can you give a bit more detail around what you're doing? Are you trying to make images of compiled versions of PHP? Via a script or something?
 
8:37 PM
The important thing is you copy the source code into the container first (so you can actually compile it), so when you RM it inside the container, you don't touch your original copy.
... and I posted that about 30 minutes late xD
 
 
2 hours later…
10:41 PM
hash_equals I have no idea why these are doing ZPP on zval only to check they're both strings.
 

« first day (3236 days earlier)      last day (1705 days later) »