« first day (3168 days earlier)      last day (1780 days later) » 
02:00 - 19:0019:00 - 00:00

Wes
7:06 PM
@NikiC does copy on write on array reset the next auto increment index now?
since 7.0 wtf 3v4l.org/0gQl8
 
He passed! Time to see how much our insurance is going up...
 
@Wes possibly only for empty arrays
At least that's what I see looking at code
 
@StatikStasis congrats to your son
 
7:22 PM
@Danack no more varchar(255) in pg, all things are text
sleep(28800);
 
Wes
@NikiC can i rfc something that normalizes this behavior, with your feedback, obviously?
in particular if $array1 === $array2, then doing $array1[] = 123; $array2[] = 123; must result in the same array
that's not guaranteed currently
 
I don't think we'd want to guarantee that
 
Wes
on copy on write?
 
Including nNextFreeElement and nInternalPointer in ==/=== is a no-go probably
 
Wes
i mean if you make a copy, you might as well reset the next auto increment to the "real" one, without holes
 
7:34 PM
that would make more sense to me, yes
We could also try to do a better job of updating it on unset
I.e. if you unset at nNextFreeElement-1 then decrement it. What array_pop does
 
Wes
ok, i'm on it. btw, there should be a warning on php.net that says that one should not use unset() with integer keys, but use array_splice()
unset into a copy you mean?
 
not a copy
 
Wes
ok i see... not sure about that
i suppose it's ok
@NikiC correct me if i am wrong
if i create an array $a and fill it with 1000 entries, then i unset() all the entries on $a
then i do $b = $a;
is $b["add"] = 123; creating a buffer with size 1000+ or just 1?
i use words i don't know. i mean the C array, is it size 1000 or whatever minimum size hash tables have?
i think we had this conversation already lol
 
7:52 PM
@Wes just one
empty arrays are special cased
 
Wes
so if it's 2, $b will have size of 1024 buckets, right? despite it contains just 2 elements
 
@Wes yeah
 
Wes
that might be improved too, unless constraining the hash code is slower than copying as-is, right?
again i use words i don't know
is 7.4 feature freeze? means i need to target 8 now?
@NikiC would that cover "pop" done in non-pop order? i.e. 3v4l.org/psRt6
 
@Wes no still about ~1month
@Wes I'd love if it did, but don't see how to do that technically
 
8:12 PM
Instead of just --, search backwards for a non-hole?
 
@LeviMorrison search how though?
it's a ht
 
You can search bawards within a bucket, right?
 
you can't, but even if you could, what you're looking for is not in the same bucket
 
Hmm, the assoc array vs numeric array stuff bites again.
 
8:14 PM
this would degenerate to full table scan at least for some cases
 
before I open an issue on github, is it expected that building php 7.4 with --with-password-argon2 doesn't link against argon2?
 
@FlorianMargaine alpha1 or git?
 
Wes
9:12 PM
@NikiC wiki.php.net/rfc/… very very first draft, how does it sound so far?
ugh, engrish
 
@Wes you should explain why it's a problem.
 
File cache no longer works – #78185
 
Wes
@Danack well, i kinda did already. let me retry. hold on
 
9:32 PM
> Next PHP minor version
I think you misspelled '8'.
 
Wes
niki said 7.4 is okay still :B
bleah
so much engrish
vikings is nice, but the battles are pathetic
i think they were lacking enough money for the extras in the first seasons :B
 
you should lead with, i.e have the introduction start with:
> If $a and $b are identical arrays, after doing the same array_push(…, $val) operation on them, they should still be identical identical arrays. Currently they are not, instead blah blah blah
 
Wes
9:52 PM
yeah that sounds like a good idea :P
 
 
1 hour later…
Wes
10:56 PM
 
" – but it should be, at least in some dangerous cases." - can be deleted to make it sound less like an opinion. in fact all the words of that section can be left with just the code and "example 1", "example 2".
 
Wes
well, that can't be done everywhere, presumably for performance
that's why it happens only on copy on write
i'm trying to reword that bit, hold on pls
thanks btw
check now :B
 
11:16 PM
I still say delete the words of "When an array is assigned to a new reference, and ...". If it the RFC can't fix it for everywhere, put that as a 'known limitation' at the end of the proposal section.
 
Wes
but that's not the intended change
i mean i can pretend that it is but....
think i am going to announce it like it is now
consider that this is extremely rare to happen
just the shitty code i am dealing with....
returns me arrays with fucked up auto increments...
 
if you're not going to delete it, you should rearrange "This behavior happens, unfortunately, also between different scopes." to be more readable.
 
Wes
11:54 PM
i changed it a bit, but ran out of strength, i announced it on the ml, hopefully it will be good enough
 
02:00 - 19:0019:00 - 00:00

« first day (3168 days earlier)      last day (1780 days later) »