« first day (4117 days earlier)      last day (1052 days later) » 

03:31
 
6 hours later…
09:49
Morngins
 
1 hour later…
11:04
@MarkR what about only forbidding redefining constants with a different value? i.e. make define() to the same already defined value not error at all, and throw the exception in the other case
I thought about it but const FOO = 1; const FOO = 1; makes about as much sense as redeclaring the exact same function twice and that already throws.
11:25
@MarkR this mainly throws because nobody bothered to compare functions for identity yet, which is quite a bit of comparisons, I suppose
@bwoebi Because there's no value in doing it.
11:40
@kelunik yep, it's typically solved by include_once. however constants sometimes clash with the same name and value and need ugly defined() wrappers around
@bwoebi Should have scoped your constants then...
 
1 hour later…
12:59
Would you consider this a bug or is there a rationale behind why array_diff_uassoc() should only filter on the first qualifying key? 3v4l.org/GWoNH
 
2 hours later…
14:52
@mickmackusa Interestingly this behavior was changed from PHP 7 on. I think PHP 5 was right in returning an empty array and should be fixed.
(probably as part of an "optimization" as with default comparison rules, no two array keys can be equal … unless you have a custom comparator which does)
@mickmackusa btw. array_udiff_uassoc should probably exhibit the same misbehaviour?
cmb
cmb
15:26
15:42
@cmb isn't that quick fix missing a bounds check?
cmb
cmb
@bwoebi I think that's covered by Z_TYPE(ptr->val) != IS_UNDEF; IS_UNDEF is used as sentinel)
@cmb oh, is it? wasn't aware
@cmb I always thought we were supposed to check nNumUsed?
cmb
cmb
There is a loop over the 2nd/3rd etc. array which stops if the keys are identical, and then doesn't check following elements; the patch just goes to that loop, in case the following data comparison fails.
@bwoebi these lists are build manually; they're not HashTables
I haven't checked that patch thoroughly, though; therefore "quick fix" :)
Yeah, missed that, you're right : 5289 ZVAL_UNDEF(&list->val);
 
2 hours later…
cmb
cmb
17:40
If it only was possible to write readable code with PHP. ;)
cmb
cmb
:D
18:25
Man the logic of some ppl
@Girgias I don't mind if people are ignorant. But ignorant and arrogant is a pretty nasty combination.
yes
Like, saying the docs are not clear, fine
Wanting the behaviour to be changed, because the documentation isn't clear is...
A take
19:24
@IluTov This almost certainly a waste of your time btw.
19:54
@Danack I know...
Nov 14 '20 at 19:02, by IluTov
@Danack Might be a good idea. Speaking of, any progress on evaluating a better communication tool? I know I keep asking, but you keep dodging :P
The same answer applies btw:
Nov 14 '20 at 20:17, by Danack
@IluTov No progress, but my cronic pain is almost getting better. It feels like I have one huge crack left to do right at the top of my neck....and I'm hoping to actually get stuff done when I'm not feeling quite so shite all the time.
I'm not smart when it comes to not responding.
Except now a way of moderating github issues should be part of that tool....probably.
20:43
@IluTov if you're gonna continue to argue with said person, point to JS embedded in HTML, I would assume that an end script tag in a comment ain't gonna stop the JS context
21:02
@Girgias I'm not, but pretty much the same situation.
21:23
that's why I pointed that as an argument :D
cmb
cmb
21:38
You can argue whatever you want, reply will always be that the language doesn't behave as documented. So better not reply at all. :)
Yeah, so fix the docs so it does behave like the docs :D
22:28

« first day (4117 days earlier)      last day (1052 days later) »