« first day (2326 days earlier)      last day (2621 days later) » 

5:02 PM
@JoeWatkins PHP 7.1.3-dev segfaults on TravisCI when running the php-code-coverage testsuite with PHPDBG: travis-ci.org/sebastianbergmann/php-code-coverage/jobs/…
 
/cc @bwoebi
I'll have a look
 
@JoeWatkins Thanks.
 
@JoeWatkins okay; I'm not aware of any recent changes to phpdbg?
 
Talking about Travis, github.com/travis-ci/travis-ci/issues/… is super annoying.
 
@kelunik well c38c c288 is utf-8 for ̈ ... so my current theory is something is starting with utf-8 ä, thinking it's latin-1, then re-encoding that as utf-8... as for the hexdump of the value to be printed, i'm not sure i want to hazard a guess there yet
 
5:08 PM
@SebastianBergmann well, there's no coverage with phpdbg for 5.6… coverage is a php 7 thing?
 
If anyone can help with following will be much appreciated. My brain hurts already. gist.github.com/jeserkin/1d420e7930b4a6a627576a7f4ae9a90b It is really difficult to find correct flow between Java and PHP :(
 
@bwoebi Right. Stupid me.
 
not aware of changes either
php-code-coverage should declare xmlwriter as ext dep
 
So how do I tell Travis in github.com/sebastianbergmann/php-code-coverage/blob/4.0/… to not run PHPDBG when PHP 5.6 is used?
Current PHP 7.1.3-dev does not segfault for me locally.
 
should require it, can't really run the tests without it
looks opcache related
 
5:14 PM
Maybe I should add it to require-dev and keep the suggest.
 
@PaulCrovella Yes, but it's not supposed to be there. When I hexdump the output in the script, it doesn't appear, but it appears in the output of curl.
 
@bwoebi it's aborting in phpdbg shutdown when destroying file sources table, when opcache is loaded, without opcache is fine
 
@JoeWatkins phpdbg with opcache … ugh
 
yeah, agree ... does that give you any hints ? something to do with interning possibly
@SebastianBergmann sounds okay
although, not really sure, you require it whatever, not just when developing ...
 
5:17 PM
@JoeWatkins there's possibly an issue with compile_file/init_compile_file
 
@JoeWatkins I would love to. Thing is, every time I fix a dependency like that the whole world cries "BC BREAK!!! THIS WAS NOT NEEDED BEFORE!!! OMG? HOW DO INSTALL A PHP EXTENSION!1!"
 
@kelunik okay, so c38c c288 is what you're getting from curl then? can you dump the content-type response header as well?
 
@SebastianBergmann anyone running the tests successfully already has the extension though, right ?
 
@PaulCrovella Content-Type: text/html; charset=utf-8
 
5:24 PM
Security is just a facade?
9
 
morngins
 
yo PeeHaa
 
@bwoebi shared memory is getting into file sources table
as key
 
@JoeWatkins yeah, but if shared mem gets in, it shouldn't have a refcount
where exactly does it crash?
 
Anyone? :(
 
5:32 PM
opcache shuts down before phpdbg, so it may not have a refcount, but will have been released by opcache before phpdbg can check that
ZSTR_IS_INTERNED
 
@JoeWatkins oh …
the file_sources should be freed before mshutdown
okay I see it's happening in mshutdown
yeah, must be executed before
 
I'll let you fix it :)
 
I just don't know right now how to execute something just before
 
		zend_try {
			php_request_shutdown(NULL);
		} zend_end_try();
before that possibly
 
It must happen between user data cleanup and zend_destroy_modules in zend_shutdown()
@JoeWatkins that's request shutdown (?!)
@JoeWatkins wait … is it released in rshutdown or mshutdown?
 
5:39 PM
before trying module shutdown then
mshutdown
but wait, I'm bit confused, I can't hack my way out of the fault
even when init'ing new string for key that is definitely not in shm
 
@PaulCrovella Finally got the "ä" to display correctly, just all other UTF-8 characters are invalid now. :-D
 
is this table returned to user at all ? or keys from it ?
 
@MadaraUchiha, is this the type of canonical answer you so graciously support?
 
@kelunik I... okay. :D I mean I want to take a look at all this directly, 'cause I'm only getting bits and pieces, but I'm also pretty sure I don't want to see what's actually happening.
 
@PaulCrovella The reason is fconfigure stdout -translation crlf -encoding [iana2tcl $config(output_charset)] which converts the encoding automatically. Internally, Tcl always uses Unicode.
 
5:45 PM
well of course, why didn't I think of that
 
I searched for something like that before, but couldn't find it in the code.
 
diff --git a/sapi/phpdbg/phpdbg.c b/sapi/phpdbg/phpdbg.c
index 320c8afac6..121f1edb17 100644
--- a/sapi/phpdbg/phpdbg.c
+++ b/sapi/phpdbg/phpdbg.c
@@ -231,7 +231,6 @@ static PHP_MSHUTDOWN_FUNCTION(phpdbg) /* {{{ */
        zend_hash_destroy(&PHPDBG_G(bp)[PHPDBG_BREAK_METHOD]);
        zend_hash_destroy(&PHPDBG_G(bp)[PHPDBG_BREAK_COND]);
        zend_hash_destroy(&PHPDBG_G(bp)[PHPDBG_BREAK_MAP]);
-       zend_hash_destroy(&PHPDBG_G(file_sources));
        zend_hash_destroy(&PHPDBG_G(seek));
        zend_hash_destroy(&PHPDBG_G(registered));
@bwoebi that works whatever ...
my hack must have been too hacky to work, or voodoo ... that seems to be the logical thing to do anyway ... possibly others table dtors should be moved also
 
this may work
 
do I sense a "but ..." ?
 
no
At least no but I'm aware of right now
 
5:50 PM
okay good, ship it, guess all branches need it ?
I wonder why this is only showing itself now ...
 
@JoeWatkins because nobody runs phpdbg with opcache active!?!
or people do have enable_cli off
 
guess that's default for travis on 7 though, or else why it is loaded for 7.1?
 
dunno
 
think we should move all table dtors to same place (and wp dtors) ?
 
no, wp dtors must be after
 
5:56 PM
ok pushing just that change, we'll wait for other kinds of the same bug before making more changes, right ?
I have a feeling other kinds of the same bug are possible
 
@PaulCrovella Completely fixed now...stupid mistake...
 
@SebastianBergmann fixed in git
 
@PaulCrovella I applied set html [encoding convertfrom "utf-8" $html] correctly, but after $html has been split into lines instead of before, so it didn't have any effect…
 
success \o/
 
/o\
 
6:05 PM
@JoeWatkins yep
 
6:24 PM
@PeeHaa Today. The code base isn't too bad, but could really do with some tests. Or types. Or comments. Or a few other things.
 
have fun :)
 
7:07 PM
@Danack So you have some work to do :)
 
sounds like he's working on jeeves
 
Jeeves has types
He's an elephant
 
Hello
if i have a .mp4 file on a server, and i make a link (<a href="/path/to/video>download</a>"). How do i make the mp4 file download
as its just opening it up in the browser
 
Try the google thing
It may come as a shock but people have been forcing downloads before
 
evening
 
7:14 PM
o/
 
\o
 
oh it's monday ...
TEST JEEVES @PeeHaa
 
I'm sleep deprived and just explained someone's code golf. could I get a proof read on an answer?
 
!!uptime
 
╔══════════════════════════════════════════════════════════╗
║ [5 hours, 13 minutes and 32 seconds] without an accident ║
║               since [2017-02-27 14:02:02]                ║
╚══════════════════════════════════════════════════════════╝
 
7:15 PM
Done @JoeWatkins
It works
 
oh okay ...
however ... TEST JEEVES @PeeHaa
@bwoebi I totally broke phpdbg
 
@JoeWatkins You should have added tests
 
haha ... I ... didn't run them
 
:P
 
!!alias test say ¿Pero por qué? Estoy aquí, ¿no?
 
7:21 PM
@JoeWatkins @bwoebi What do you think about reviving gist.github.com/krakjoe/5275773 + wiki.php.net/rfc/instance_counter? Could be helpful for long running applications finding leaks.
 
Command '!!test' aliased to '!!say ¿Pero por qué? Estoy aquí, ¿no?'
 
!!test
 
¿Pero por qué? Estoy aquí, ¿no?
 
@kelunik was widely hated
 
@JoeWatkins I saw the voting results, yes, but why?
 
7:23 PM
I don't remember any details
 
Shouldn't it be get_class_objects_count($class) : int and get_objects_count() : array for consistent returnType ?
 
it's so old that it targets php5
 
I hope there would be annotations in PHP someday, and async, wait and enums structs and type aliases, modules, real namespaces and binary code or precompiled codes :/
So many things in one sentence, it's like never gonna happen or at least not in next 20yrs :(
 
user895378
7:42 PM
morning
 
@rdlowrey o/
 
user895378
o/
 
7:52 PM
@JoeWatkins So it basically only lacked motivation.
 
@JoeWatkins git.php.net/… says it's reverted?
 
3 hours ago, by Eugene
If anyone can help with following will be much appreciated. My brain hurts already. https://gist.github.com/jeserkin/1d420e7930b4a6a627576a7f4ae9a90b It is really difficult to find correct flow between Java and PHP :(
 
@Eugene Where are you stuck?
 
8:12 PM
@bwoebi hi, just confirmed ResultSet::fetch() is not equivalent to PDO::FETCH_ASSOC mode. It is equivalent to FETCH_BOTH, though. I couldn't find the exact equivalent to the aforementioned PDO assoc behavior in amphp/mysql. Could you confirm if it's missing or intentionally not supported?
 
@marcio yes, it's fetch BOTH
@marcio what do you need it for actually? (array) fetchObject() is doing to too… should be easy to add though
 
@bwoebi yes, that's what I'm doing. I just noticed FETCH_ASSOC could be missing :)
 
@marcio I mean, what's the benefit of ASSOC vs BOTH?
if you want to access keys, you can do it as well in both mode?
 
@bwoebi fetch both can be bad because you need to check if the key is numeric when doing map() |> reduce() stuff (or array_keys) + casting (array) $object is extra work on userland side too. Users can live with it, it's not the end of the world but...
 
@SebastianBergmann only in 7.0
 
8:22 PM
@Eugene What that says is basically unpack('n*',substr(hash('sha256','Hello World!',true), -2))[1] % 10000, but that would never be 7712 as that gist purports.
 
@Sherif that is because instead of 'Hello World!' it gets hash of 'Hello World!'
So hash from hash so to say
But let me try.
 
wat
 
If that so, then I will probably kiss you
totally not gay
 
@PeeHaa When do I use Reject and then Delete for comments on php.net?
 
@Eugene You mean unpack('n*',substr(hash('sha256',hash('sha256','Hello World!',true), true), -2))[1] % 10000?
 
8:24 PM
Yes
 
OK, that was definitely not what that gist said though.
 
Anonymous
@JoeWatkins your dog is unreal <3
 
@Sherif damn you beatiful you
it works
I've really perverted all the logic. Woun't even show.
 
@kelunik Someone did try to explain it to me at some point but I just randomly click things
 
@kelunik Just delete. All reject does is attempt to email the op. It's pointless.
Because half the time they don't give us a real email anyway. Or it's 35 years old and the email is invalid.
 
8:28 PM
Ok, fine, it's gone. \o/
master.php.net/manage/user-notes.php?view=notes&type=4 provides also delete reasons, the normal interface doesn't.
 
Should probably delete all these votes: master.php.net/manage/…
 
@kelunik Yea enjoy doing that 25 items at a time.
I'd just log into mysql and do it in one query.
 
@kelunik That's all I care about
 
@Sherif Yup.
 
8:36 PM
Pay no mind to the fact that I'm the guy who wrote that interface
I wonder why on earth anyone would invite me to a MariaDB conference. I don't even use MySQL anymore. I'm not even a db guy ffs!
 
9:14 PM
There are missing docs for pcntl_async_signals pl1.php.net/… which is added in 7.1 php.net/manual/en/…
 
9:28 PM
docs are missing for domnamespacenode too, and that went in like 15 years ago
 
10:09 PM
@kelunik it's for work
there are several projects and they all need a messaging system
hell, two of them have some they of "system" but it is made from string and bubble gum
and that bubble gum is not compatible with php 5.5+
 
10:23 PM
@JoeWatkins Per my comment on github.com/krakjoe/apcu-bc/pull/23 you should feel free to ignore that if the "fix" is too complex to justify.
 
10:44 PM
Awesome... it's segfaulting on travis, but not locally. Love it when that happens.
 
11:11 PM
Blame Ubuntu and move on.
 
11:57 PM
Sara what book are you reading.
 

« first day (2326 days earlier)      last day (2621 days later) »