« first day (3228 days earlier)      last day (1711 days later) » 

3:04 AM
Unable to parse supported time format – #78426
 
3:38 AM
\o
 
4:11 AM
o/
mornus
 
 
2 hours later…
5:43 AM
@Girgias your error prototype looks great!
 
6:33 AM
hm, getting /usr/bin/ld: ext/standard/.libs/cyr_convert.o: in function `php_convert_cyr_string':
/home/benny/code/projects/php/php-src/ext/standard/cyr_convert.c:251: undefined reference to `php_error_docref0' and more in master
(probably forgot to make distclean, trying again)
 
6:45 AM
mornin
 
7:13 AM
Can someone tell me or show a motivational video/article on why I should pick mongodb (or any other nosql document store) over mysql to store user threads of user posts and comments?
 
Wes
though, things might be different right now :P
 
7:28 AM
@Wes I just finished watching the video and asked this question. XD
 
Morngins
 
7:43 AM
@JoeWatkins @LeviMorrison @Derick up early and was motiviated to try myself at the error notification callback API we discussed: github.com/php/php-src/pull/4555
 
Will have a look later today.
@2dsharp I used to work for MongoDB, and I would still thoroughly recommend MongoDB. It's IMO a lot easier to work with, and for a user threads/posts/comments thing going to be really fast; as long as you design your schema right.
 
@2dsharp the best video of mongodb and why you should use it is: youtube.com/watch?v=0vPt7GI-2kc
 
7:59 AM
@beberlei looks okay at a glance, just one question ...
 
hiatus a break in or as if in a material object : gap
 
not sure if maybe, the notify callbacks should be able to stop propagation, and my brain can make an argument for and against it ... so do words about that somewhere ...
fuzzy, sunday morning, awake for 20 minutes brain ... which may be asking a stupid question ...
 
good point!
 
also, strdup that name in the list and free on dtor
(probably use interned zend strings) ...
 
This test error is probably due to me not putting this into extern section? ext/intl/common/.libs/common_enum.o:(.bss+0xe0): multiple definition of `zend_error_notify_callbacks'
ext/intl/.libs/intl_convertcpp.o:(.bss+0x0): first defined here
 
8:02 AM
and try to sprinkle some assertions in there to force correct usage
> ZEND_API zend_llist zend_error_notify_callbacks;
don't export that, that should be a local symbol ...
 
right
 
8:16 AM
if this targets php8, maybe something nicer is possible ?
@beberlei some thoughts ... when you say that "some extensions don't forward the call" that's because it doesn't make sense, it doesn't make sense for xdebug to print out a pretty error, and then forward the call to another handler ...
 
i know
 
well I'm sorta thinking, maybe you develop this into a mechanism that not only notifies but enumerates, do you get where I'm going ?
 
enumerates?
i have ideas to extract the last error repated logic for example, just wanted to work in small steps :)
 
like if a notification callback returns null, you enter the next notification callback, but if a notification callback returns an error_cb function, you stop iterating and use that as zend_error_cb, something along those lines ?
 
ah
i don't want it to stop iterating to be honest
its should guarantee that they get called
 
8:22 AM
yeah I thought that too, well that's also possible ...
 
i do think we can improve the whole error code even more
 
upon registration you could take a notification callback and an error callback, while iterating, select error callback based on retval of notification callback, and pass currently set callback to all notification callbacks, so they can create (or destroy) the chain they can create (or destroy) today ?
 
this kind of stuff here for example is scary :) github.com/php/php-src/blob/master/main/main.c#L1333-L1335
hm interned string would be nice, but i need error callbacks list registered before interned strings get initialized, because dtrace is registered so early
 
IPv6 brackets – #78427
 
@beberlei ah, in that case probably just a simple dup of the string, if you're displaying those names then you're relying on those addresses, and some extension may do something strange with the address it gave you on registration, so I'd eliminate that as a possible source of bugs ...
 
8:33 AM
yep i fiddled with this a bit, its safer this way you are right
 
and on second thoughts, I revoke the idea I just had to merge the api's into one ... there is value in the guarantee this is trying to provide and it's simple to provide it, so keeping it simple is good/best ...
I can't words very well this morning ...
 
no problem :) this whole thing is also very complicated, so its always hard to remember all the constraints and possibilities. if this gets merged I will work on more improvements of the whole error handling
 
cool, it's very messy ...
 
 
1 hour later…
9:46 AM
what is the ini setting report_zend_debug actually being used for? i would assume its for testing, but not a single test uses it.
2002, "Make it possible to test language features like newly introduced 'abstract' independantly from debug mode."
heh, until then when you had enable debug activated, the error handler would always print all errors regardless of display_errors setting :-D
 
10:10 AM
@Derick Gotcha, I was thinking that this could be an appropriate use-case for giving mongo a try, but my rdbms-based brain needed some motivation. Thanks.
 
 
1 hour later…
11:11 AM
@beberlei thanks :D Seems like a bunch are going to be promoted to Errors in any case.
Also for php_error_docref0, remove the 0 at the end and it'll work as I think the renaming of that function from docref0 to docref and removing the define landed in PHP 7.4
 
@Girgias docref0 really was me not make distclean'ing after months of not compiling, so my fault :)
 
Ah fair :p
 
@Girgias as for exception class, i would make that dependent on if its useful. fopen() could use some special extepsions, but many other cases are probably fine with runtimeexception or the likes of
 
Sure, but from what Nikita told me a bunch of the current warnings should be Errors in PHP 8 regardless
So probably working on that first before expanding the PoC (which apparently leaks a global currently D: )
Can I delete the test for 73646? Because this can't really happen if an invalid length throws an Error. (in regards to PR github.com/php/php-src/pull/4554) bugs.php.net/bug.php?id=73646
 
 
1 hour later…
cmb
12:30 PM
@Girgias, the str_repeat() in that test is superfluous; it's only about the NULL return in this case
so simply var_dump(mb_ereg_search_init(NULL)); should serve as good regression test
ideally should be tested on an old PHP version without the actual patch
 
If I use separate in Z_PARAM_ARRAY_EX then I need to dtor the zval, right?
 
12:56 PM
@cmb Okay, Is there a good way to test multiple version or do I just need to checkout an old branch, compile, and run the test?
 
cmb
you need to checkout an old branch; you can use git worktree to have multiple branches checked out
 
Oh, new command of Git that I didn't know about
 
1:34 PM
arf Why does appveyor not cancel a CI when new changes have been pushed to the PR >->
 
@beberlei huh... even after reading the code I still don't understand what this does
@LeviMorrison nope
 
@NikiC i think in historic perspective, this code must have been there when no tests existed, because it changes the output of errors when --enable-debug is active. At some point this started to cause problems and the INI flag was introduced to control this instead of removing it alltogether
specificially it calls an otherwise seldomly used error output function that includes code checking for active debuggers and calls a breakpoint (at least on windows)
i think we can remove this completly, but I would ask anatol if this is used on windows for debugging purposes
 
1:54 PM
Let's ask @cmb ^^
 
cmb
@beberlei, I'm not sure about that. I can see some value on Windows,
since OutputDebugString() prints to the debug console, not stderr.
 
@nikic just to be sure for strr(i)pos when the needle is "" and the offset is negative, it should return length-of_haystring + offset ?
 
cmb
@beberlei, if in doubt, I suggest to raise this issue on internals, or to provide a PR.
 
2:47 PM
hey all
 
Oh crap, how do I install oniguruma for Ubuntu on WSL ?
 
Step 1, install docker, probably.
 
yeah Docker
the thing which kinda doesn't work out of the box in WSL
:(
 
3:08 PM
apt get?
 
No package with that name
Oh it's called libonig2
Now just need to add the xenial depot i hope
 
3:25 PM
Well after more research it is called libonig4 for Ubuntu 18.04
 
because nobody wants to (mis)spell oniguruma
 
Well sure, but why is it called libonig4 for Bionic (i.e. Ubuntu 18.04) instead of libonig2 for Xenial (i.e. Ubuntu 16.04)
 
@Girgias sounds right
 
That's the bit which doesn't make sense to me
Well I got a patch ready but didn't fix the strpos tests because of the nul byte problem (and I'm too lazy to split the test again)
Seems I'm finally having a green CI for the Warning conversion PR, can you have a look at it again @nikic ?
Gosh and now the configure step doesn't find Oniguruma, I really should install Linux one of these days and stop using WSL
 
4:01 PM
error_page 503 /var/home/DockerHost/config/nginx/error_pages/503_maintenance.html;

    location / {
        return 503;
    }
Is that not the appropriate incantation to configure a custom error page for 503 responses? It gives a 503....just with not the right html.
"Very few directive in nginx take a filesystem path." - ugh. no apparently
 
4:16 PM
nginx config is random. Just try
 
4:40 PM
@cmb i am totally fine with keeping the report_zend_debug thing. I would say its easy to refactor into a zend_error_notify_cb (see github.com/php/php-src/pull/4555). that would remove it from the critical path, so less need to worry about it anyway
 
5:12 PM
@nikic
Removing the explicit return values doesn't seem to work
Still getting FALSE returns for cases like var_dump( @strpos("a", "") );
 
@Girgias likely needs a check in zend_memnstr
 
Okay, will do that
 
 
1 hour later…
6:30 PM
@nikic var_dump(strrpos("a", "")); should be 0 or 1 ?
(the double R version, because I applied the same check in zend_memnrstr than I did in zend_memnstr but not sure that it displays the correct behaviour)
 
6:47 PM
@Girgias 1
 
Also, is it actually wise to accept an empty needle and not return false ?
Had that conversation just now with someone
Okay, so the same check does the job
 
@Girgias I don't see why not
 
¯\_(ツ)_/¯
 
I mean, returning false is outright wrong
Throwing a warning isn't wrong, but it also seems unnecessary, because there is a well-defined behavior here
 
Fair enough
Also what do I do with the checks that I added into strpos from stripos ?
Just remove them because it's covered by zend_memnstr ?
 
6:52 PM
@Girgias yes
And by remove I mean don't add, but don't remove either
 
So I leave them in stripos ?
 
yes
 
Should a removal of these checks go into a seperate PR ?
 
stripos performs string lowering operations, which are expensive, so it's worthwhile to exclude impossible cases early. For strpos on the other hand, this would just be a redundant check, because it will be checked later anyway.
 
Ah ok so that's the reason, but what about those in strrpos then ?
 
7:46 PM
Good evening, Roomates!
@NikiC The Z? Oh yeah, nasty bug. Pretty sure it's chronic.
@Wes Yeah- there is a ton of crap out there. You can spend a lot of money. I only use a pre-workout formula along with some creatine and protein afterward.
@PeeHaa So... how do you know the flavor of that? =P
3
 
cmb
8:30 PM
@beberlei, +1 for keeping report_zend_debug (unless proven that it's basically useless)
 
8:57 PM
@cmb side benefit, it shows how value my refactoring with the notification cbs is :)
 
cmb
:)
 
So the current version of Github actions is deprecated and it's all changing to Yaml. This is clearly the fault of @Ocramius .
 
lol
Poor Ocrmius
He didn't ask for anything
 
@Danack hahahaha :) i think they cloned azure pipelines syntax, because its now built on top of it
 
pretty much explains why a whole load of people from github that had only recently joined quit like the week after github actions were launched.
> That thing you built and just launched? Yeah we're killing it.
 
9:04 PM
motivation!
 
 
2 hours later…
11:04 PM
Okay also added strstr into the strpos PR as it is pretty similar
Everything should work
 

« first day (3228 days earlier)      last day (1711 days later) »