« first day (2274 days earlier)      last day (2671 days later) » 

1:00 PM
@Wes I don't think I have seen another opninion
 
Wes
sherif seems to have a different opinion :P
 
No he does not
Which is exactly why I asked the question I asked. To make it clear to you :P
> I mean those methods which I defined in my interface, which can only be public.
yesterday, by PeeHaa
I don't care about the private stuff, because I know when I throw a beer, throw 0 beers, throw 999999999 beers, throw a lizard, throw -1 beers in it the outcome is what I expect it to be
 
Wes
but it is a contradiction, because you are testing how code executes but not really, because you do it through something else that might also be broken / add subtle problems that aren't immediately noticed
 
> add subtle problems that aren't immediately noticed
 
Just think of public methods as third party APIs. Would you ever check if the DB of an API you're consuming is up? No, that's their own implementation. You don't even know if they use a DB.
 
1:03 PM
That means I am missing a test (on the public method)
 
@pmmaga There you go. You get it now.
The entire purpose of encapsulation is that the consumer of that interface does not care how public foo(Int $int): Int does what it does. Just that it does exactly what it says it will do.
Changing the implementation details, should be completely transparent to the consumer.
 
@Sherif if you have a complete public interface and it is tested, then obviously that is all that is required, but during dev other devs may need to know if the suppositions I made in my black box hold true ... if when the interface is complete, tests overlap, they are obviously removable and should be removed ... I come back to writing private code that isn't used yet, but exists for the purposes of making collab easier for other devs ... none of this is what I aim for ...
 
Wes
Jan 2 at 20:15, by Wes
@Danack an example could be... if i have a function A that does something, should i be aware that the function uses another function B for it to work, or should i test A's signature regardless of how it's implemented? because if i'm aware that A uses B, i can assume that B works/is tested and just test what A adds to B
 
@JoeWatkins Not sure what you're trying to say. All I am saying is that there is no point in unit testing private methods under the principle of encapsulation. This is pretty well documented stuff.
 
ok good, I'll go read documentation ...
 
1:09 PM
Not trying to be dismissive, I just don't understand why you believe there is a benefit to testing private methods. I've not found any evidence of that (in practice).
 
I don't have time to argue the toss about it to be honest, I do see value in it, tried to explain it ... you were dismissive ... I'm done ...
 
@Wes IMHO you should test A's signature regardless of how it's implemented. Testing B or not shouldn't influence at all the test you're writing for A.
Unless perhaps on the example from Joe. If A is not finished yet and other people will use B to finish it, it may make sense.
 
Wes
@pmmaga now multiply for like ten layers. would you still test the outermost layer's result only, or make assumptions about the other 9 layers?
 
@JoeWatkins Sorry you feel that way. I really didn't understand how that adds value in a pragmatic sense (vis a vis: collaboration). I'd just be breaking tests that don't tell me anything useful (i.e. I would already expect the test to break when I change implementation details). As long as I have full knowledge of the interface spec, though, that doesn't matter, because it changes nothing of the outcome for the interface.
 
@Wes Would you test if MySQL will actually SELECT the way it should? If PHP mapped $_GET correctly?
 
Wes
1:16 PM
imho pmmaga it's not a good idea, because 1- they would be mostly redundant tests (assuming you tested the other layers separately) 2- things go easily mad
@pmmaga no. it's exactly what i'm saying. i need to make assumptions about third party stuff
 
@Wes as you should about privates
 
Wes
no. that's my own stuff
 
While on the subject and somewhat related: did you have time to look into chat.stackoverflow.com/transcript/message/34965048#34965048 yet by any chance?
@Wes ^
 
Wes
not yet
 
@Wes It doesn't matter who wrote it, it matters who'll use it
 
Wes
1:25 PM
testing code that uses it means testing the code that uses it. why should i be interested in being informed that the "code who uses it" is broken but just because it uses an untested private method? clearly that private method should've been tested on its own...
i don't see how that is different from a third party lib. i'm delegating something to something else, i don't care what it is, and i am told that it works. if it doesn't it's not my problem... someone (me, since it's private stuff) should've tested it
it's not that your option makes the testing less effective @pmmaga just a bit redundant
because if you have two public methods using the same private method, chances are you are testing the private method twice, if doing your way
makes sense?
 
@PeeHaa I tried, but I don't know how to install pecl to install xdebug. ^^
 
Wes
@JoeWatkins are you reading documentation? :B
 
@PeeHaa I'll definetly look into it
I love when my code gets shot at, burned, hanged, drowned, quarried, drowned again and cremated and still passes tests
 
@kelunik Can't you just grab a proper so from somewhere somehow?
 
Wes
programmers are a strange species. i don't understand why they can't discuss stuff without getting emotionally attached to the topic.... :D
@PeeHaa please read ^
 
1:34 PM
I would argue that testing the private methods makes it redundant.
What I meant with who uses it is that your private method can only be used by your public ones (even if more than one). If your private method is wrong, that will be shown by the tests on your public method. Conversely, if your private method being wrong doesn't make the public one fail, either your tests don't have enough coverage or the private method is useless.. :X
 
evening room
 
Wes
but why the public method should care of the private method's coverage?
 
@PeeHaa It has to work somehow.
 
o/ @Linus
 
\o
 
1:37 PM
@kelunik Have you tried black magic yet?
 
Wes
@JoeWatkins for what it is worth, i think you are great <3
 
@Wes Is there any specific reason you are pinging me specifically? :P
 
Wes
@PeeHaa because you kind of participated to the discussion and i value your opinion? :B
 
@Wes Because we're passionate about our job
 
Stupid example: https://3v4l.org/2COhV
If you test the private _add and the private _subtract independently, they may both be correct. If you test the public add without ever passing the $flag as FALSE, your tests will tell you that everything is fine even though it isn't
(And that's why coverage is important)
 
Wes
1:43 PM
"without ever passing the $flag as FALSE" i'm not saying that you shouldn't test public methods :P i'm saying that each method should have its test, regardless of its visibility
 
Wes
that doesn't really answer it :P if you rely on something else, that something else must (should ideally, imho) be tested separately
 
hmm... but in the example I gave above, what exactly would be the goal of testing the privates?
(Assuming you have tests for the public method pass the flag as TRUE and as FALSE)
 
So just for an experiment, I converted the manual to markdown
8
 
Wes
there is nothing to test here because math operators are already tested by php. but if your private methods were more complex, there would be stuff to test
unrelated: i'm using finally{} for the second time ever \o/
@Leigh HERO
wait.
 
1:55 PM
@Leigh Great! I did it for the wiki.
 
@Leigh you...
 
Wes
:'(
 
@Wes That precisely where we disagree. :) But let's just agree to disagree, it's friday! Everything will be fine :P
 
@Wes I thought you told me I got emotional, but I don't do emotion. Robots don't work like that
 
@JoeWatkins (@bwoebi you can probably answer this too) add_assoc_string doesn't copy the string right, it literally adds a string typed zval to the hashtable with with char* you supplied?
 
Wes
2:10 PM
@PeeHaa aaaah no. i meant, read the stuff above that :P
 
:-P
 
Having a problem with a memleak, the char buffer added to the hashtable is never freed, but if I free it just after I add it to the hashtable, then it's a use-after-free
(it's allocated by OpenSSL)
 
Wes
is ML back up?
 
How do I make that string refcounted, create a new zend_string?
 
TIL there's a HangDrum app that also allows recording, fun stuff
 
2:16 PM
@Leigh there are no char* in hashtables anymore, it allocates zend_string and inserts that
 
So it's fine for me to free the char* buffer I used for add_assoc_string?
 
yes
 
cheers
 
is an SLK 300 a bit girly ?
 
is it free? :)
 
2:18 PM
no :D
9k '07 plate
 
@JoeWatkins eeeeeew
get a beamer instead
 
I wanted porsche but it sold this morning apparently ... been looking at it every day for couple of weeks and thinking "I should go and test drive that" ... just went there and it's sold :(
 
@PeeHaa Can you suggest me something. When a user uploads a file on a rest end point, We are scanning the file before moving it from tmp directory. In case if a virus found in the file, What http error code shall we send to the user. Do you think we should make our own custom codes in such case.
 
Oh :( porsche would also have had my stamp of approval
@Tarun Not a custom code. Just send more info in the response body
 
yeah black, boxter s ... he reckons I should come back on saturday in case sale falls through ... nobody is going to let the sale fall through ... I'd sell one of my children ...
 
2:22 PM
Have you told him that?
Also which one?
:P
 
@PeeHaa Thanks. Is it okay if we send message with 400 HTTP code.
 
Sure
 
Okay. Thank You
 
np
 
Anonymous
!!wotd
 
2:24 PM
etiolate: to cause to become weakened or sickly; drain of color or vigor.
 
@PeeHaa either/both && wife
 
:D
 
Anonymous
> to cause to become
 
Anonymous
Is that propa engrish ?
 
@JayIsTooCommon Looks like some symbol got lost?
Actually it is english
Although weird
 
2:25 PM
Damn. The black boxster looks really good
 
Anonymous
Ah yeah with a symbol it makes sense, but without I don't think?
 
if something causes you to become ... ?
Not sure. I think you can do that in english
If only we had somebody who understand english in here
Are there any non brits in here?
 
Anonymous
Yeah that's right. I just mean the actual sentence "to cause to become blah blah" is weird
 
Anonymous
Ask a German, they'll know :D
 
yeah, I'm very annoyed ... I should have gone in there weeks ago ... I literally saw it about 6 times a day ...
 
2:27 PM
heheheh
 
@kelunik Well, it is a dependency though? Also why should that conflict with a phar?? Just use php vendor/bin/phpunit like everyone else?!
 
Anonymous
@JoeWatkins I'm selling my car soon if you want it. Top of the range.
 
@JayIsTooCommon Are you getting another one?
 
Anonymous
I want to get a frenchy ds3 I think
 
@JayIsTooCommon now isn't the time to fuck with me Jay ...
 
2:28 PM
I remember a red corsa talk here :P
 
Anonymous
@JoeWatkins Hey, the offer's always there ;P
 
@JayIsTooCommon No argument there. It's weird as fuck
 
@JayIsTooCommon that's the one that looks like a training shoe, right ?
 
@JoeWatkins How would you know what a training shoe looks like? :P
 
Wes
2:30 PM
lol
 
I've seen them in films and on the interweb @PeeHaa
 
hehehe :D
 
Anonymous
I want @Gordon's car ideally
 
Wes
@JayIsTooCommon french are bad, especially peugeot and except renault :B
 
@Wes YOU WOT MATE?
In fact, you're right :(
 
Wes
2:32 PM
it's true. peugeots are terrible :B
 
Peugeot are cheap
Citroën sales are terrible
But Renault are good cars
 
just buy a skoda :)
 
Anonymous
@Wes I know, damn French. /cc @FélixGagnon-Grenier
2
 
Wes
indeed renault are ok
 
I've got a dacia (which is basically a renault) ... was cheap and hasn't broken yet ...
 
2:32 PM
Before:
==32605== LEAK SUMMARY:
==32605== definitely lost: 170 bytes in 11 blocks

After:
==9889== LEAK SUMMARY:
==9889== definitely lost: 0 bytes in 0 blocks
 
Wes
@JayIsTooCommon ahahah
 
\o/
Now for still reachable...
 
is also so boring that I sometimes consider driving it into a wall
@Leigh excellent
 
==9889== still reachable: 744 bytes in 17 blocks
 
My mum used to have a Renault 5 - with a massive 900cc engine. I think it got like 60mpg.
 
2:33 PM
@Leigh best feeling in the world
 
Anonymous
I'd sell my body for a merc tbh
 
they've always been good at that ... and actually pugs/citroens can be very comfortable ...
 
@JayIsTooCommon You'd need more than your body
 
Anonymous
@Leigh not with my hips
 
the big citroen they do now, I test drove that ... with weird suspension and everything, uber comfortable ...
 
2:34 PM
Child bearing hips...
 
Anonymous
lol
 
@JayIsTooCommon that might scratch the itch, how much do you want, and is finance available ?
 
Anonymous
my street name will hips n nips
 
Wes
@JayIsTooCommon boo. mercs are so unappealing and bad at everything
i'd get bmw, at least they are good to drive, but mercs bleah :B
 
Anonymous
@JoeWatkins Sorry I don't travel to remote islands, that's where all the murderers hide
 
2:35 PM
@Wes you look like a douche in both
 
@bwoebi Not everyone uses that.
 
Wes
my next car, i think
 
Anonymous
You're such an Italian
 
@bwoebi Because PHPUnit dependencies add up quite a lot and might conflict with your actual dependencies.
 
Wes
it's japanese @JayIsTooCommon it's a fiat 124 based on a mazda mx5 :B
 
2:36 PM
@JayIsTooCommon there's only been one murder in the last few weeks ...
 
Anonymous
@JoeWatkins Alright fine, i'll come
 
@Wes sounds you are a wealthy person ..! what's your job?
 
@JayIsTooCommon /me undresses
 
Anonymous
:P
 
Wes
@Shafizadeh lol. i'm poor
 
2:40 PM
@Wes lol italian miata
 
@Wes your next car doesn't seem meekly ;-)
 
Wes
it's a cheap small car actually
don't need more than that. as long it's fun to drive and cheap to maintain :B
 
really!? good
 
@JayIsTooCommon get a proper car...
 
Wes
@PeeHaa it's the best thing. japanese engineering with italian styling
 
2:41 PM
@kelunik I've never seen that being an issue
 
Anonymous
@PeeHaa what do you class as a proper car?
 
A bayerische one
 
a bmw @Jay
 
Anonymous
Do they even have roads in the Netherlands?
 
Wes
japs should really hire us more often, because they can't do styling
seems made of tinfoil
 
2:43 PM
that is so ugly
 
Anonymous
I mean, i'd happily have an i8 if it was free
 
@Wes Kill it with fire
 
Wes
ikr...
 
they should stick to bikes
 
Anonymous
@Leigh FACK
 
2:44 PM
wait
 
Wes
ahahaha
 
Anonymous
that looks ace tbh :P
 
Can't say dutch without at least a couple of windmills
 
lol
 
Wes
@JoeWatkins italian design: http://media.caranddriver.com/images/08q3/267371/2009-toyota-yaris-review-car-and-driver-photo-225692-s-450x274.jpg
japs design: http://o.aolcdn.com/dims-global/dims3/GLOB/legacy_thumbnail/430x242/quality/85/http://o.aolcdn.com/commerce/autodata/images/USC50TOC193C021001.jpg
at some point here like 50% of cars were yaris. perfect for our narrow roads. now they have completely disappeared because of that hideous look
it's basically the same car, just with a different body... for some reason they wanted to ruin it...
 
2:49 PM
that's not so bad from the japs there ... the problem with the lexus is they just didn't stop designing the thing ... at some point, I'll bet it wasn't painful to look at ...
 
Anonymous
@PeeHaa get logic so we can collab please.
 
Wes
@PeeHaa that is so beautiful tho :P
 
collaborating is really hard even if I installed it because of the vsts
 
@pee
@PeeHaa
need your help
 
Anonymous
lol @pee is like a vamp indicator
 
Anonymous
2:53 PM
@PeeHaa the wat?
 
0
Q: How to know total number of views for an article just like youtube

beginnerWe know youtube can tell us how many different users has seen this video ,just like that how can i calculate how many different users has visited my article ,Should i use ip addresses of user to know that from how many different ip addresses the page has been accessed , and save those differ...

 
@kelunik uh, what then? A custom installation of phpunit? or what?
 
@JayIsTooCommon vsts / au's
e.k.a. plugins
 
Anonymous
@PeeHaa You won't need em with Logic :D
 
Anonymous
logic does all the things.
 
Anonymous
2:54 PM
or you mean logic plugins?
 
Anonymous
If so, yes. But we could manage...
 
Anonymous
I'll try and render a vocal track I mixed, this weekend, and send it to you
 
Omnisphere ftw
If you haven't played with it, and if you can, do it
 
@JayIsTooCommon Yes please
@JayIsTooCommon Don't you use an AU / VST plugins?
At the very least install the sylenth1 because it's great
@Ekin Don't know that one. Let me see if I can find a torrent
 
@bwoebi A phar, as said and as @JoeWatkins also mentioned.
 
3:05 PM
@kelunik … why would you use a phar, when it's in vendor??? And anyway, we depend on these specific phpunit versions, so let's specify that in our composer.json … that's what it's for.
 
Anonymous
@PeeHaa I haven't done any music prod for years (Was probably the most enjoyable thing i've ever done apart from theatre tech). I didn't use my own studio with logic so no idea what I used. It just worked :D
 
@PeeHaa I'd give you a full legit copy if I was still in Uruguay :(
 
@bwoebi As said, there might be conflicts. And people might want to run it as phar.
 
*** Error in sapi/cli/php': double free or corruption (!prev): 0x0000000001b0cc80 ***`
changed my mind, I'm cool with there being memleaks
 
@kelunik well, if there's a conflict, then you cannot use our tests, because they then wouldn't run anyway??
 
3:06 PM
@JayIsTooCommon You did theatre tech. Such a drama queen\
 
Anonymous
@PeeHaa I was a fucking boss. All the girls loved me.. promise.
 
:P
 
@bwoebi Depends whether they're loaded at the same time.
 
@kelunik ??
 
in JavaScript, 33 mins ago, by Danack
Does anyone have any links to articles for 'gotchas' or difficulties that people coming from PHP experience when trying to get their first Node web application into production? For example...in PHP uncaught exceptions only affect a single request, but my understanding is that they would affect all of the requests being processed in Node. That's the type of difficulty I'd like to have a list of.
 
3:12 PM
@bwoebi For PHPT tests for example, PHPUnit's dependencies aren't loaded during the tests, dunno whether they're all loaded in the normal tests.
@Danack Exceptions thrown in Aerys also affect only the current request mostly.
 
@kelunik but our tests aren't phpt tests??
 
@Leigh you can do it ...
 
@JoeWatkins That should do it: github.com/php/php-src/pull/2283
At least the test suite passes now without complaining about double frees, and valgrind says no leaks :p
 
3:29 PM
osum, good job
does it merge up to master clean ?
 
When trying to compile xdebug directly, I end up in
+ phpize
Configuring for:
PHP Api Version:         20160731
Zend Module Api No:      20160303
Zend Extension Api No:   320160731
/usr/bin/m4:aclocal.m4:3223: cannot open `m4/ax_check_compile_flag.m4': No such file or directory
autom4te: /usr/bin/m4 failed with exit status: 1
 
@JoeWatkins It does, waiting for travis to pass, would like eyes on it which is why I didn't just merge myselfl
 
cool
@kelunik branch ?
@Leigh is the line that nulls what was just free'd left from debugging ?
(and the line beneath that that zeros something)
 
@JoeWatkins 2.5.0 release
 
I meant php version ?
 
3:36 PM
Oh, a broken one...
[16:35:50][4390][~] $ php -v
Failed loading /home/kelunik/.phpenv/versions/master/lib/php/extensions/debug-non-zts-20160303/opcache.so:  /home/kelunik/.phpenv/versions/master/lib/php/extensions/debug-non-zts-20160303/opcache.so: cannot open shared object file: No such file or directory
PHP 7.2.0-dev (cli) (built: Jan  6 2017 13:15:13) ( NTS DEBUG )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.2.0-dev, Copyright (c) 1998-2017 Zend Technologies
 
right, master has a patch in it that this looks related too
lemme find out what to do
(checkout 7.1 if this is blocking you on something)
 
That one works, yes, thanks!
 
wait, it was merged into all of them, so don't fetch that branch
 
@trowski @bwoebi Could you have a look at github.com/WyriHaximus/reactphp-async-interop-loop/issues/16?
 
@JoeWatkins Nope, it's in a for loop, and I read some OpenSSL funcs will alloc new buffers if passed NULL, and re-use if not. Because it's a loop and I freed it, I wanted to make sure it was a null ptr again
 
3:43 PM
Morning.
 
@Leigh k cool, I'll read it properly (expanded) when I've got a minute
 
Mutation Testing is commencing on 22 files...
(.: killed, M: escaped, S: uncovered, E: fatal error, T: timed out)

...S......M...M.......MMMTETM...........MMMMMMMMMMMMMMMMMMMM |   60 (14/22)
MMMMMMMMMMMMMMMMMMMMMMMMMMMMSMMMMMMMMMMMMMMMMMMMMMMMM...MMM. |  120 (16/22)
M..M.SSSSSS

131 mutations were generated:
      37 mutants were killed
       8 mutants were not covered by tests
      83 covered mutants were not detected
       1 fatal errors were encountered
       2 time outs were encountered
It started out looking good... ^ Running Humbug on Amp.
 
excellent ... I don't have to hate you ...
 
oh that's not bad at all! Still not bad! ooooooooh damn...
:P
 
@kelunik It's invoking the onReadable watcher 3 times because a defer is added to stop the loop when tick() is invoked here: github.com/WyriHaximus/reactphp-async-interop-loop/blob/master/…
I'll add that to the issue.
 
4:00 PM
@bwoebi The issue is that their loop isn't spec compliant.
 
Your loop is bad and you should feel bad
 
@PeeHaa haha
 
I would have created a proper meme image if I weren't lazy :P
 
@bwoebi Actually I may have spoken too soon… it might be spec compliant, but since they expose a tick() method, they can execute only 2 ticks… so of course the read watcher will only be invoked twice.
 
@Trowski Adding the defer watcher before run should immediately run it, and thus only run one tick, no?
 
4:02 PM
@kelunik But watchers are only activated at the end of a tick.
 
@Trowski the should be activated before the next tick actually
At least that's what the spec says
 
@bwoebi Then we'll need a test for that. Simple fix.
 
@Trowski Good, adding the test for that then
 
@Trowski Registering before the first tick should run immediately IMO.
 
Guys, I'm facing a very annoying issue for a long time, perhaps someone could shed a light on this matter...
 
4:07 PM
Goddamit @Leigh
 
@kelunik The spec is a little open to interpretation, we should update that to make it clearer.
 
All my Wordpress websites at shared enrivonment at Hostinger all have the same bug: Random redirects to HTTPS
Some times it tries to load all resources through HTTPS, seems randomly
most of the time the redirect occurs on wp-admin, while saving posts, navigating, etc
I already added this to .htaccess, for example:
RewriteCond %{HTTPS} on
RewriteRule (.*) http://%{HTTP_HOST}%{REQUEST_URI}
 
@Trowski at least the docblock for defer() should be clear with next tick. But it's lacking on the other functions
 
But nothing seems to stop these redirects, unless I add a SSL certificate. Hostinger is aware of the issue and says they will fix it - but it's been more than 6 months
 
> Watchers (enabling or new watchers) MUST immediately be marked as enabled, but only be activated (i.e. callbacks can be called) right before the next tick.
@Trowski See Loop::enable() docblock
right before the next tick … where is that open to interpretation?
 
4:10 PM
@Trowski Does the spec currently cover the "end of tick"?
 
@bwoebi Doing it at the end of the tick seemed sufficient.
 
@Trowski That's what tests are for … to show that it isn't.
 
@bwoebi Yep, which I'm glad we'll cover that.
In a running loop, doing it at the end of the tick is equivalent to doing it at the beginning of the next tick.
 
yep
 
@bwoebi Does the spec currently define what a tick is?
 
4:19 PM
@kelunik think fixed master, can you fetch and try another build for my sanity please
 
@JoeWatkins Sure.
 
@kelunik no, I already noticed that … do you have a good wording to describe it?
 
[17:20:04][4404][~] $ time (./php.sh master && ./xdebug.sh 2.5.0)
real	2m19.158s
user	3m40.344s
sys	0m29.436s
Works fine @JoeWatkins, thanks.
 
4:25 PM
@bwoebi I'm wondering that myself since it passed locally.
Might just be reverting that one.
 
@Trowski Perhaps because I pushed that test like 1 min before you pushed the last change?
I think NativeLoop has an issue with interval = 0
 
@bwoebi Yeah, that change has nothing to do with the failures.
 
yeah
I was just liek wtf, why was that callback invoked a 137 times?!
 
All the impl fail locally for me.
 
Well, have fun fixing that
 
4:28 PM
@bwoebi Well… I can't.
Having a 0 timeout will cause weird behavior.
 
Looks like
 
It is executed a random number of times in the other impl too, where I'm not in control of that.
 
So… shall we forbid that (in spec) … or work around it in impl?
 
0 = execute me as fast as you possibly can.
Which I don't know why you'd do, but I'm not sure if we have to forbid it either.
 
4:30 PM
@Trowski yeah, that's what I'd expect
 
But lets not use it in a test :-P
 
@Trowski hahahah
If it were just our own interface, I'd say fine… but it's a spec, so, no way around either saying 0 and below are explicitly unspecified or making it work
 
@tpunt well wtf
 
@bwoebi I vote for the former.
Making it work will add a lot of overhead for something that shouldn't matter.
 
@Trowski I would too - after all you could just defer recursively
 
4:34 PM
I hate everything and everyone ...
 
@JoeWatkins Better hate your git which didn't warn…
 
everything is a blur, I need a rest ...
 
@bwoebi Does uv not work with streams created with stream_socket_pair?
 
@Trowski it should
why?
 
@Trowski We can work around that by making 0 timeouts defers internally.
 
4:40 PM
@bwoebi I was going to update the test to this: gist.github.com/trowski/a2b755e109391353fa4adc7f51b741f5
 
@bwoebi Why unspecified instead of an error?!
 
It works with all the impl except uv, where it executes it 0 times.
@kelunik That is another option: Make a timeout of 0 an error.
 
@Trowski just use onWritable(STDERR) … we're anyway using that already in tests
@Trowski Oh, in that case I'd just add an exception for uv
 
@Trowski I'm explicitly talking about error instead of unspecified, see github.com/async-interop/event-loop/issues
 
I thought it'd be errors everywhere
 
4:42 PM
@bwoebi Hmm... still fails. Now I wonder if something else isn't going on there.
 
@bwoebi why won't it let me cherry-pick c50f61b9b06d65ed2391e7751a2adbdf8db91754 from 7.1 for master ?
 
@kelunik Ah, I see. I really wish I could turn on email notifications for just that repo.
 
@JoeWatkins because that's a merge commit? … "but no -m option was given"
so, give it an -m option?
 
nvm, it looks like the commit I wanted is in all branches ... sorry, go back to what you were doing :)
(thanks for coming to rescue also)
 
@bwoebi Ah, it's because I called uv_stop() in UvLoop::stop().
 
4:47 PM
@Trowski instead of setting the flag?
so, it actually works fine?
 
@bwoebi In addition to setting the flag.
If I don't call uv_stop() then it works.
 
so, we can easily support repeat(0)?
 
@bwoebi No no, different issue.
It wasn't calling the write watcher in that test I showed you.
Apparently uv will actually stop immediately, even within a tick. libevent and libev don't do that.
 
oh
 
it looks like links are gettin to internals now, I spot one in my inbox with links to wiki
 
4:53 PM
@Wes now you can try re-announcing
@kelunik I do not see much value here in having an extra well-defined exception within Loop for that
 
Wes
@bwoebi thanks. what's the duration of the voting?
1 week? i read it somewhere..
 
@Wes 1-2 weeks as you think it's appropriate
 
15 hours ago, by Levi Morrison
@Wes Why don't we just fix PDO again?
 
Wes
do you think that's possible? :B
 
4:57 PM
We are doing "language design by least resistance"? :D
 
Wes
@bwoebi should i go with 1 week or 2? your opinion, i have no clue...
 
@Wes meh, that RFC is very small and pretty uncontroversial IMHO, go with 1 week
 

« first day (2274 days earlier)      last day (2671 days later) »