« first day (2051 days earlier)      last day (2897 days later) » 

2:02 PM
> This RFC permits the ''null'' type to be used in type unions such as ''Foo | null''. While it isn't very helpful it doesn't cause any harm to permit only ''null'': <php>function foo(): null;</php> However, some people feel it may be redundant with ''void''. In addition, should we some day introduce covariant return types, we need to be able (given that we want to be as specific as possible) to specify a return type of ''null'', where the parent has a signature like ''Foo | null''.

Thus this RFC proposes another vote whether ''void'' shall be replaced with ''null'' in general.
this is not a good idea
 
sec
2 days ago, by Levi Morrison
@bwoebi I think voting null instead of void should be a totally different RFC and vote.
this section needs to be removed here
I had misunderstood Levi…
 
> Long Type Expressions
also should be removed ?
 
That's future scope?
Not sure why these should be removed?
 
2:08 PM
okay
I think
> Hence this RFC proposes a vote on whether ''?'' shall be replaced by ''| null'' in general.
also a bad idea
all the reasoning that ?Type | OtherType doesn't make sense is good, but there's no logical pathway from there, to abolishing something that existed for five minutes
that section should just conclude with "? will not be supported by unions"
agree ?
 
Nah, this is intentional
At least I personally don't want to see ?Foo in PHP
just Foo | null
It's just an additional vote
 
> * on abolishing ''?QuestionMarkNullables'' in favor of union | null,
that's not a 50% vote
> * the removal of void in favor of standalone null,
nor is that
I finished editing anyway
 
@JoeWatkins It should be a 50% vote. People choose whatever they like. If it were a BC break, I'd agree
 
I can't really see anything missing ... I really don't think its a good idea to put four votes on ...
 
@JoeWatkins the reasoning is aesthetics and redundancy
 
2:14 PM
it's a language change
they both are ...
 
@JoeWatkins The language was already changed. It's now just deciding on the form of the change
 
I don't think so
the form was decided for nullables
 
@JoeWatkins I do. That actually was the whole plan
The vote was rather intended to be on the concept and a fallback if unions do not pass
 
the vote for what ?
 
nullables
If Levi hadn't waited so long the unions RFC would introduce nullables at all
 
2:16 PM
I don't think that was clear to voters ...
well he did, and it's done, it looks like a language change to me ...
I think you should do a section on inheritance
(just mention they are invariant, because I been called on that, even though it's obvious)
actually, are they invariant ?
 
yes, they are
I believe at least
 
I may remember a conversation about being able to remove types from a union on inheritance, did that go anywhere ?
if it's as I done it, then they are invariant ...
 
@JoeWatkins I've seen, you have a todo there ;o) github.com/php/php-src/pull/1887/…
@JoeWatkins no. That should be introduced whenever we introduce co/contravariance in general
 
right okay
 
@JoeWatkins a section is too much, a single sentence is enough ^^
 
2:22 PM
yeah, probably so
 
@JoeWatkins I consider it like a secondary vote on nullables. 2/3 vote is on "Introduce nullables" and the secondary vote is basically "Allow ?Foo or only Foo|null"
The only issue is that nullables were voted on first, which is why we couldn't place that vote there
 
nearly everyone voted in favour of ?Types, you think they are going to be ready to reverse that decision already ?
 
@JoeWatkins yes. They mainly voted on the concept of nullables.
If I am wrong though, the vote will show it.
 
it's reasonable to prohibit ? in unions, you can justify that ... I can't see justification for removing ? ...
 
And we can only reverse it now. In 7.2 it's a major BC break
@JoeWatkins The main point is redundancy
 
2:27 PM
but there are good arguments in favour of keeping it ...
brevity for one ...
 
My nightmare is of ?Foo and Foo|null mixed in code…
 
that's not really your problem ...
 
@JoeWatkins which is, in my eyes, a reason to remove it. It's too concise.
 
haha
 
You easily miss it.
 
2:28 PM
oh you were serious
 
yes :-D
 
that doesn't feel like a real argument, I think if people had wanted a different syntax, they would have voted no already, everyone knew unions were coming ...
 
@JoeWatkins coming, yes. If they pass, dunno.
People don't like to speculate here
 
I wish this had all been organized a bit better
 
They possibly wanted to be rather safe than sorry and include it
@JoeWatkins yes.
The main issue is Levi taking so damn long
 
2:31 PM
you can say that, I can agree ... but you can also say that we allowed him to take that long ...
 
@JoeWatkins He asked for it and we granted him the favor
But Dmitry sometime pushed it…
 
I think in future cases, we have to take action in these circumstances
nobody is to blame for having a real life ... but, we cannot hold things up for any one person ...
 
eventually. Eventually you should not be able to have a RFC on hold for more than 4 months except for very big RFCs (i.e. bigints, phpng and such things)
 
even 4 months is pretty long
we do things in hours, why do you need four months ?
 
I have no idea…
We can do large patches in about 2 weeks (including review etc.)…
 
2:34 PM
yeah
even with real life, I think 6 weeks is a reasonable time frame, if you announce something, and don't take it to voting within 6 weeks, anyone should be able to take over the rfc, compete with it, do whatever ...
 
@JoeWatkins 6 weeks is quite short TBH … it's reasonable, but still short.
 
it's considered, it's difficult to improve the focus of the collective ... one of the things that makes it difficult is repetition and drift ... if you start an rfc and there is healthy discussion about it, and then you take it nowhere, you have wasted everyone's time, asked for their opinions on something and then by convention, nothing will get done about it if you don't do it yourself ... that needs fixing I think
 
greetings
 
@LeviMorrison I implore you to take a few minutes, read over the RFC and eventually start the vote. … In case you don't until Monday evening, I'm going to go to vote with the RFC as is. … We respect you, but we're going to be short on time and I don't want to leave it until the penultimate minute. Thanks.
Just to be sure, I think I'll write this also per mail…
 
people are reluctant to make competing RFC's, and never take over RFC's ... putting a shortish shelf life on them encourages people to do that ... I think it needs encouraging ...
 
2:46 PM
@JoeWatkins You're not wrong … That's actually why I'm now taking the initiative. If Levi doesn't, I'm going to.
 
[10:44:21] critical FastRoute\BadRouteException: Regex "(/.*)?" for parameter "path" contains a capturing group in /git/vendor/nikic/fast-route/src/DataGenerator/RegexBasedAbstract.php:108
^ @NikiC Can't you just rewrite them to non-capturing groups automatically? :P
 
@kelunik and then get bug reports why the (?1) regex group repeating does not work?
 
@Fabor You complete bitch
 
@Jimbo context?
 
@bwoebi :-) See thing on right --->
 
2:50 PM
> wow @Gordon, you met Justin Beiber! - 5h ago by Fabor
this?
 
Yep :D
 
but it's true
Fun Fact: when @Jimbo say GUI, I hear gooey, which makes a lot of sense after we !!urban'ed Jimbo
 
@Gordon LOL. Man, don't set a precedent for the humour level tonight or it's going to be crazy :P
 
oh, before I forget, we will likely not go the to Tapas bar tonight since the Trivago office is just around the corner
 
@Gordon Gaylord mentioned that it wasn't the best last time, plus I'm not particularly a football fan. No Tapas tonight is okay, but perhaps we can go to a cocktail bar or something later on?
That one by the beach was good, I can't remember too much about that though as I was pretty smashed
 
2:58 PM
not sure what the plans are. I suspect we'll end up in the place downstairs in the little harbor. they have tapas, too. but let's see. I also got a recommendation for another cocktail bar in Palma
 
Nice. Decided I'll get a taxi back, so shenanigans shall ensue. German V England.
 
Yay!
 
i stuck on a using ajax to get my chatroom id very long ago anyone can help me to slove this?
 
@nonstop328 Have you tried jQuery?
 
@nonstop328 ask Jimbo about websockets
 
3:00 PM
D:
 
i was using pure php js ajax to do chatroom
so i was using sql statement to call all chatroom from my database
here is the 1st page which is chatrooms.php
 
pastebin please
 
1 message moved to bin
 
 
3:04 PM
how to paste in bin? i was quite new here
 
@nonstop328 pastebin.com
 
"down" or "done" is the word which describes such a website doesn't load ?
 
Ah ok thx
 
Wes
!!coffee
@Jeeves you suck as butler
 
3:11 PM
@Wes Not as good as @PeeHaa's mum, though
3
 
It's ok I'm a belieber
 
lol, @PeeHaa Why aren't you here?
Another English guy came and he's already heard of my drinking... don't know if that's funny or sad...
 
You didn't tell me enough times
 
:-(
 
Yea sucks. Still need to find out whether NL or UK is better at the drinking game :(
 
Wes
3:13 PM
lol
 
http://pastebin.com/JSScpQB3 < let user to select chatrooms
http://pastebin.com/iVGJRyXm < a chatroom gui page
http://pastebin.com/U4J1pd89 < insert message call by ajax
http://pastebin.com/kQJ07eXq < load message call by ajax
http://pastebin.com/gF3PdE9p < ajax with poll

my question is how i get the chatroom id? now i was unable to get chatroom id
when i insert into database the chatroom_id is 0
Anyone can help me? i have a hard time to do this i have no idea to get the id
 
evenin
 
This guy is going places
 
O/
He is sleeping
 
@AnmolRaghuvanshiVersion2.0 oh really? I thought it is always on ...!
 
3:23 PM
@JoeWatkins is the typed properties branch rebased?
or at least in sync with 7.0.7 changes?
 
Wes
@kelunik will finish the logo in the next days. we have holidays up to 2 june :D do we have a decision for the trunk?
 
@Wes I don't think so. :-/
 
@Wes As popular vote is typically wrong when votes are very close, we use the straight trunk, obviously.
 
Wes
typically i don't know. but could be wrong indeed
 
hmm, ok, nvm, may rebase myself
 
Wes
3:31 PM
i don't like the other trunk either, but it can be improved compared to the straight rolled up trunk, at least
 
@Ocramius not rebased, but you can merge it in easily
(we've merged master into it, multiple times)
 
yeh, I basically need to try it with the fixes that @NikiC made around reflection :)
lol, doesn't rebase cleanly
 
As said, we've merged master into it.
 
yeh
 
it's like a 150 commits, have fun rebasing that
 
3:33 PM
lemme try that
 
also, last merge is from 2 days ago anyway
 
/me slaps @JoeWatkins for those commit messages >.<
 
haha
 
Applying: remove todo
Applying: ...
Applying: betterness
Applying: some mo tests
Applying: fixes and more stuff
Applying: woops
GRRRR
 
...
 
3:35 PM
oh well, he's RM now, he decides these things :P
hrhr
 
That's what happens when multiple people work on a feature branch… @Ocramius
If just one person works on it, force push
but if not, it gets messed up
 
yeh, I'm usually very nazi on my own branches :D
 
@Ocramius not sure how you mean that?
 
@SQB Go spam your mother
 
SQB
@PeeHaa nah, she's already following the proposal.
 
3:37 PM
That;s odd. I don't see blowing in the description of the proposal
 
@bwoebi my branch is my branch - all the rest are merged in things (by me)
 
SQB
She supports everything I do; I'm mama's good boy.
 
anyway, branch compiled fine, now writing tests for my shite
 
Now I'm wondering what mama's bad boy looks like
 
@Ocramius There are already a bunch of tests for the magic
but feel free to cover not yet covered things…
 
3:38 PM
Yes, I am writing integration tests for D2 and the other things I wrote
if I discover new things, I'll send tests against that branch
 
oh, you mean tests for your repos… misunderstood.
 
@bwoebi we already support partial non-invariance. This seems like just the thing we may want to support right away
 
@NikiC what, where?
 
@kelunik So, this is the regex I use to find capturing groups: github.com/nikic/FastRoute/blob/master/src/DataGenerator/… ... are you sure I should try automatically rewriting them? :P
@bwoebi e.g. for nullables
Also, you can add a return type that did not previously exist
 
the latter, yes. … But forgot we have support for nullables…
 
3:42 PM
So the whole "add or remove types" thing is usually supported (though you can't completely remove a parameter types)
@bwoebi especially if you want to replace nullables completely it's important to keep the existing variance semantics ... and support them for everything else for consistency would be nice as well
 
@NikiC yeah, I totally agree
I just forgot about nullable variance… so, in this case it's only natural to support it too here
 
Hah, this one will be fun to fix:
 
@Ocramius you're not returning from the function and $properties is not a reference, what's the point of that anon. function?
 
$properties is an array of references
 
@Ocramius what's the problem here?
 
3:53 PM
@Ocramius oh god
 
ah... right
 
yep, IKR
 
@Ocramius do you have nightmares?
 
this was done for performance... I'll write some tests and see if I can work around it by not using references for typed properties
the idea is to avoid read, then write to array, then write back to object. Mostly related with hydration performance
 
@Ocramius Also, this test makes no sense … $properties is never returned in the function and it isn't by-ref??
 
3:54 PM
@bwoebi properties is an array of references
it doesn't need to be passed by ref
 
oh god
 
Wes
@Ocramius do you ever write something that doesn't require prayers? :B
 
1 min ago, by NikiC
@Ocramius oh god
 
@Wes @NikiC the answer to both questions is basically "yes, I do have nightmares"
 
@NikiC haha :-D
 
3:55 PM
o/
 
the simplest solution would be to have something like "mass-assign-to-object-properties" kinda solution in php-src
which basically replaces all this hydration and scope bullshit :P
I should probably write an extension for that, then write a very slow userland solution for it -.-
oh god... I just said I'd write an extension... I must be drunk.
 
@Ocramius well, I'm all in favor of supporting references … if you can solve the technical problems…
 
@Ocramius so ... what does that code do? why doesn't it directly assign to rebound this properties?
 
@NikiC you really ask that? It obviously does magic.
 
@NikiC this code basically creates a shadow copy of the object into an array, then allows the closure to overwrite those properties
 
3:59 PM
@Ocramius I get that part ... but I'm missing out on the "why" of it
 
the why is mostly performance. If I don't do this, I have twice the method calls to initialize a proxy
 
@Ocramius why?
 
ARGH
fucking macbook touchpad
 
turtles all the way down :P
 
because I have $props = $someClosureWithPrivatePropertyReadAccess(); $newProps = $initialize($props); $someClosureWithPrivatePropertyWriteAccess($newProps);
 
4:01 PM
I hate it when processes end up in stuck state and nothing works anymore :-( … the whole UI is frozen, only mouse works :-/ … Inspecting via ssh :x
 
instead, this approach removes the second call
in addition to that, I don't want to write to properties that weren't really initialized
but anyway, let me finish writing some tests, then I'll see if I can just push it onto my lib at the cost of performance
 
@Ocramius I think my question is why $initialize doesn't get private property access through $this
 
Fatal error: Typed property ProxyManagerTestAsset\ClassWithMixedTypedProperties::$publicStaticProperty must not be static in /Users/ocramius/Documents/Projects/ProxyManager/tests/ProxyManagerTestAsset/Clas‌​sWithMixedTypedProperties.php on line 31
wut... did I miss something or does the spec not consider statics?
 
you didn't miss something
 
@NikiC because $this is multiple levels of inheritance
you may have 3 parent classes defining private properties
I'm abstracting the initialization code away, both for ease of use and for perf, again
 
4:05 PM
doin ?
 
plus I have to re-initialize to object defaults anyway (de-unset, basically)
 
@NikiC oh … why no statics support?
 
ok, removing statics for now... a bit of a WTF for me though - that is already an ugly inconsistency
 
It's true, but I've never consciously noticed it…
 
@Ocramius uuuuh right
 
4:07 PM
Is there any technical reason why typed props do not support statics? @JoeWatkins
 
@bwoebi I think it comes down to "static have none of the machinery of instance properties"
Handy stuff like ... an actual assignment opcode
Static assignment works using fetch_static_prop + assign
 
@NikiC this … can be changed
 
hmm
 
so before we can even start thinking about type checks gotta refactor that
 
error message a bit un-descriptive here
Fatal error: Type of ProxyManagerGeneratedProxy\__PM__\ProxyManagerTestAsset\ClassWithMixedTypedPropert‌​ies\Generated543a09ac208c3efb120be6f0a8b3fa51::$publicProperty0 must be string (as in class ProxyManagerTestAsset\ClassWithMixedTypedProperties) in /Users/ocramius/Documents/Projects/ProxyManager/src/ProxyManager/GeneratorStrate‌​gy/EvaluatingGeneratorStrategy.php(68) : eval()'d code on line 3
specifically... wtf was assigned? :D
 
4:09 PM
@Ocramius That's an inheritance error
 
hmmm
ok, gimme a sec, trying with a different file writer strategy
 
it's basically that, it's a thing that could quickly follow as a standalone thing, or preferably we get them for free with typed zvals ... is mentioned in rfc ...
 
@JoeWatkins I think that typed zvals is super far away...
if we want to support static props, it's not going to be through typed zvals
 
ok, I think this one is a legit bug
 
I think I have a faulty bit in the physical memory region currently mapped to video memory… That looks super weird when chunks of the page are rendered at bad places for splitseconds…
 
@JoeWatkins typed zvals are in general far too expensive to be worth it.
 
@NikiC agree ... I would just rather get them that way ... or not at all ...
 
@Ocramius that causes the fatal error?
 
yes
 
@Ocramius type is not inherited and then failed?
 
4:13 PM
@Ocramius BUG
 
/me sighs
 
At least I don't see a redeclaration of that property in there
 
I also am not seeing traces atm, btw
 
@Ocramius because it's a compile-time fatal error
Those aren't exceptions
 
yeh, I still don't know whether it is caused by just evaluation or an action though
lemme try it... may be an actual bug tho, yes
if you strip the proxymanager cruft from it and include both files, you should be able to reproduce it though
 
4:17 PM
Yes, I can repro it
 
cool
 
sapi/cli/php -r 'class A { public string $a; } class B extends A {}'

Fatal error: Type of B::$a must be string (as in class A) in Command line code on line 1
 
@bwoebi lol
One would've thought that someone had tested this already :D
 
yeah, too simple … just nobody tested that case
:-P
we have like a 60 tests
but not this^^
 
bugger
 
4:19 PM
haha
 
do I have to figure this out, or is someone on it already ?
 
Wes
can i test it on 3v4l again?
 
Not sure, but if someone could even push a quick hotfix, I can re-compile and keep finding more bugs
I wish my C-fu was better, but I'd rather not.
 
@JoeWatkins I am, and in wtf mode… not sure what you did there with ZEND_ACC_CHANGED
 
dude, me neither ...
 
4:23 PM
:D
 
HAHAHHAHHA
 
Don't worry, nobody really understands what SHADOW and CHANGED are about
 
ok, I'm gonna get beer
 
Wes
lol
 
I do not understand why the if (UNEXPECTED(child)) branch is entered at all…
 
4:32 PM
it isn't
there's an else
if (UNEXPECTED(parent_info->type && !(parent_info->flags & ZEND_ACC_PRIVATE))) {
			if (parent_info->type == IS_OBJECT) {
				zend_error_noreturn(E_COMPILE_ERROR,
					"Type of %s::$%s must be %s%s (as in class %s)",
					ZSTR_VAL(ce->name),
					ZSTR_VAL(key),
					parent_info->allow_null ? "?" : "",
					ZSTR_VAL(zend_resolve_property_type(parent_info->type_name, parent_info->ce)),
					ZSTR_VAL(ce->parent->name));
			} else {
				zend_error_noreturn(E_COMPILE_ERROR,
					"Type of %s::$%s must be %s%s (as in class %s)",
hits last zend_error_noreturn
 
@JoeWatkins yeah ... that whole else doesn't make sense right?
 
it really doesn't
 
if it's not in the child class it will simply be inherited
 
Ugh, the else is of that branch
too much if nesting
 
so ... kill it with fire!
 
4:34 PM
I literally have no idea why that is there
 
I'll kill it with fire as Nikita proposed and add a test…
 
yeah, tests pass with it removed
must have been a special kind of high that night ...
I wonder what I thought I was doing ...
 
git commit -am 'Joe seems to have been high....'
 
lol
 
@JoeWatkins pushed with a test^^
 
4:40 PM
cool
 
@Ocramius you can continue (lol)
 
probably ...
 
4:53 PM
3 hours ago, by bwoebi
@JoeWatkins I've seen, you have a todo there ;o) https://github.com/php/php-src/pull/1887/files#diff-9760ee109b1c5922071fac1e19d1‌​17dfR367
^ @JoeWatkins may you please finish this todo? :-)
 
Wes
it should be illegal to make 60s unkippable ads
 
@Wes ADBLOCK THAT SHIT
 
Wes
nevar
 
Seriously. Screw my business. They are all assholes
 
5:00 PM
yeah, I will do it, tomorrow if not this evening ...
 
Especially those pre-rolls
 
there's another todo you could have a go at bob also
or maybe it's not required ... const defaults
 
@JoeWatkins you mean constants in defaults?
 
yeah
 
mornin'
 
5:03 PM
moring
 
'nin
 
@Saitama thanks, dude, I needed some music
 
yw :)
 
It's weird, I just listed to 21 pilots for the first time yesterday, and ladybaby last week
 
lol, 21 pilots is quite nice imo
 
5:16 PM
@JoeWatkins but you are taking care of it…? github.com/php/php-src/pull/1797/… … or am I missing something?
 
and this song is cool too: youtube.com/watch?v=foE1mO2yM04
 
5:47 PM
!!github
 
good: Everything operating normally as of 2016-05-25T18:44:58Z
 
hmmm
 
what are the other args for !!github?
 
!!plugin github
 
@PeeHaa Syntax: plugin [list|disable|enable] [plugin-name]
 
5:48 PM
errr
I don't know how it works :P @DaveRandom
!!github room-11
 
Organization Stack Overflow - PHP Chat: 16 public repos
 
And
!!github room-11/jeeves
 
Room-11/Jeeves Chatbot attempt - Watchers: 14, Forks: 15, Last Push: 2016-05-27T16:49:49Z
 
And status which is the same as paramless github
 
\o/
got it!
!!github gourabNagDev/Leloutama
 
5:52 PM
gourabnagDev/Leloutama A multi-threaded webserver written completely in PHP. - Watchers: 2, Forks: 0, Last Push: 2016-05-16T08:50:43Z
 
!!github amphp/loop
 
amphp/loop - Watchers: 1, Forks: 0, Last Push: 2016-05-27T20:07:12Z
 
Could somebody tell me, who is this Lester guy?
 
@Saitama Have you ever had a super annoying itch on your back you couldn't reach?
 
yeah...
 
5:56 PM
@Saitama then why do you ask? ;-P
 
:D
 
is there an easy way to import an sql file in php?
 
@PeeHaa shell_exec("mysql -ublah -ppass < $dumpfile") :-D
 
^ that
 
@bwoebi That's pretty much what I am about to do
 

« first day (2051 days earlier)      last day (2897 days later) »