« first day (2365 days earlier)      last day (2583 days later) » 

Wes
3:03 PM
 
@Wes too much git, not enough unit testing
 
@bwoebi @kelunik @Trowski ping, github.com/amphp/amp/issues/118
 
Wes
i hate the "i" in logos. it is always hard to get right
and it is the only letter that is hard to get right
 
@kivlara Setting the deferred to null will destroy the deferred faster, but it doesn't really solve any problem.
 
in my aesthetic impaired opinion, you could try removing the dot from the i
 
Wes
3:09 PM
maybe @pmmaga i thought it would look good but it didn't
 
@kivlara The same is true for the change I made to Coroutine, so I'll probably add it, but it would be collected by the cyclic garbage collector eventually.
 
@Trowski At the moment I am not familiar with all internals of AMP but, working with it, so this changes save me from leak, after talking with you yesterday I try to fix leak in my code, but only this really save me
 
@kivlara It's not really a leak though, eventually memory should be freed.
 
@Trowski I tried, as you say to call gc, but it does not help me
 
@kivlara Then you're holding onto a reference somewhere.
 
3:14 PM
@Trowski 1 min, i will push an example, and send you link
 
@kivlara Your patch also doesn't account for already fulfilled or failed promises.
 
@Trowski @Trowski I realised what you mean, but there are cases when you can't do it without strong ref
 
@kivlara Well then you can't expect it to be destructed…
 
gc don't help
@Trowski ups, 1 min ... i miss something
 
It wouldn't because you're holding onto a reference to the promise through the deferred: gist.github.com/umbri/…
 
3:30 PM
@Trowski updated ...
 
@kivlara So you're holding onto a reference to a promise for a really long time that isn't resolved… there's nothing I can do about that.
:36528913 Which deferred?
 
@Trowski sorry, I read my question one more time and ... no sense :)
 
You realized you have a reference to it :-D
 
@Wes Yes. Internals is awful at actually reading...
 
@Trowski Yep, doing 10 things async, looks like humans are not asynchronous
:)
 
3:42 PM
I'm having a conflict in my head... should I use SplEnum or an array and throw exception if value isn't in array?
 
Hi all guys
I have a big problem
i didn't able to solve :(
 
@Trowski Verified if resolved before timeout: gist.github.com/umbri/55d0d3d9c0f26843e16c46808359d468
 
please guys I'm desperate with CodeIgniter and PHP
 
That's kinda what you signed up for by choosing CI :P
 
Wes
@pmmaga @JayIsTooCommon
 
3:53 PM
@kivlara Try with this commit. I refactored things to remove some circular refs.
 
Wes
occupation: does oss logos. doesn't finish them
 
huh another phpstorm update
 
Wes
adds to curriculum
 
I have to fix a site already present.. if I could change CI....
 
@Wes I still love the gitilicious one
 
Wes
3:54 PM
yeah that was great considering i did it in 5 mins :P
 
phpstorm + google gave birth to this...
 
@Trowski Especially the cases where you null the deferred before resolving
 
Maybe you should just limit it to 5 mins always
 
@Wes it looks like christmass tree
there is so much colour on it
 
Wes
they are the colors of code coverage @brzuchal
plus the php violet/blue
 
3:56 PM
nice, but I'm getting nystagmus
 
Wes
lol
 
@Trowski revert that … the deferreds will anyway be destroyed with destruction of the callback, which promise impls are supposed to release immediately after invocation
well, it may make sense for the cases where we may resolve the deferred before all promises are resolved
but not for any other case
will refactor
 
@bwoebi There's checks for null.
It actually eliminated a by-ref boolean to do it that way, so it's probably better.
 
Anonymous
@Wes nice!, how does it look without the circles?
 
@bwoebi With the deferred reference removed, promises will be destroyed immediately if there are no remaining references rather than having to wait for the cycle collector.
 
4:04 PM
Can someone give me an assertion error from phpunit? I don't have php set up on this machine and I need an example error and can't seem to find one by searching on Google lol
 
@Trowski yeah, I'm refactoring
 
@bwoebi I see no reason to change it: I didn't affect the normal case, and I improved the cases on error or where promises don't resolve.
 
@LeviMorrison Anything specific?
 
Something along the lines of "expected X saw Y"
 
1 sec
There was 1 failure:

1) PeeHaa\AsyncTwitterTest\Api\Request\Status\RetweetTest::testTrimUserSetsCorrectly
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-xtrim_user
+trim_user

D:\Web\AsyncTwitter\tests\Unit\Api\Request\Status\RetweetTest.php:26

FAILURES!
Tests: 303, Assertions: 608, Failures: 1.
 
4:05 PM
@bwoebi Of course if you have an even better method…
 
@Trowski I just pull your commit, and start testing ...
 
@PeeHaa Thanks.
 
anytime
 
What does this mean?
Array
(
    [] => stdClass Object
(Except from print_r())
 
4:14 PM
@Trowski the Artax PRs for amp_v2 are fine to merge, right?
 
It means you are using the wrong dump tool for one :P
 
I'd be interested to hear why and what I should be doing after I fix this bug.
But for now, I have no idea what that means.
 
var_dump() is superior in every way
 
I am confused between templates and views, any help?
 
@Andrea hikari no yume?
 
4:19 PM
@CoderDudeTwodee A view can be anything. In my application it's JSON. Templates are just ways to keep views DRY and seperate.
 
@NikiC is her new public account apparently … that previously was the private one, now I do not know of any private account of her… [I think she has yet another one in use, but I don't know which one…]
 
@bwoebi k, just wanted to check
 
@Allenph can you give me a link to a nice tutorial
 
No because I haven't read such a tutorial in years.
I'
ll give you a better explanation in a minute if I remember.
 
posted on April 07, 2017 by bwoebi

See #156.

 
4:30 PM
@bwoebi Sure we can skip using the temps? There is a possibility that resolving a promise would trigger resolution of the next promise in an array of promises.
@bwoebi Yep.
 
Wes
rhino illustrator ps phpstorm sourcetree chrome all open... running out of memory (16gb) lol
 
@Trowski oh, that's why… I'm sorry :-/
Fixing again
 
@bwoebi It would be silly code… but it's not out of the realm of reasonability. Thanks. :-D
 
@Trowski It may just happen coincidentally through a series of when()'s being resolved through multiple levels of Generators or such
 
@bwoebi @Trowski
1. I tested with @Trowski commit and all was fine for 15 min
2. Pull @bwoebi commit and after 10 sec get a Promise was already resolved!
3. revert to @Trowski commit
Just in case this will help
 
Wes
4:44 PM
ok i might have done one that i actually like :D
 
github.com/zeit/next.js looks interesting, README says, it's kinda inspired by PHP
 
@Wes The i is weird
 
Wes
the i is hard
 
it isn't delineated sufficiently clearly
 
@Wes What is phpu art?
 
Wes
4:45 PM
ARGHdfgsdjsdfg
 
:P
 
Wes
/me quits everything
 
@Wes Danube font ?
 
Wes
no
 
Trolling @Wes is the best way to start the friday night
 
Wes
4:45 PM
you ginger bastard
:B
 
hahahaha
You should see my current "beard" right now
I should shave, but cannot be arsed
 
no one posted rebecca black .. incomplete friday
 
You sure about that? :P
 
@PeeHaa any idea why this is failing?
 
@iroegbu I blame scrutinizer instead of you I think
 
4:48 PM
@Trowski github.com/amphp/amp/commit/… okayokay \cc @kivlara
 
Maybe, no blank line before EOF?
 
That is an issue that it should just report. It should not blow up because of that
Does travis pass?
 
yes
 
Yeah blame scrutinizer
lemme try restarting it
Crashes again :(
Could you fix the two EOF newlines please and the extra space here? <3
Rest looks mighty fine tnx
 
@PeeHaa Sorry. Boss came in.
Remember that array I was talking about?
 
4:55 PM
@bwoebi But now you reverted what I was trying to accomplish…
 
var_dump doesn't show that weird index at all but it's for sure what's messing everything up.
 
@Trowski no, I didn't
 
Wes
 
@bwoebi Yes, now there's circular references to deferreds again.
 
@Trowski where?
 
4:57 PM
all() for example, the promises have a reference to the deferred.
So the cyclic garbage collector will have to run to free them.
 
@Allenph messing what up?
 
My framework's router.
I have been looking for this bug for like 2 days.
It's really weird.
 
Why is your router doing with an instance of stdclass?
 
@bwoebi There it probably doesn't matter as much, because you'd only gain freeing of the Deferred.
But that's all I gained in timeout() as well, so meh…
 
@Trowski ugh, I removed the $deferred = null; too -.-
That was an accident
 
5:03 PM
@PeeHaa It's a configuration object that did not warrant a class.
 
So it's an array?
 
No. I'm weird. I like to use stdClass even though it's an extra line because it makes multidimensionals easier to read.
 
@bwoebi Ah, I see though with the way you wrote it, I can safely null the deferred after resolving, since $pending === 0 will stop double resolution.
 
@Trowski yes
 
@bwoebi Basically the same, but a little cleaner :-)
 
5:08 PM
yep :-)
 
@Allenph Yes that is weird
 
Not neccesarily wrong though, right?
 
If I would review that code I would not let it pass
 
Oh. I lied also. I just put the var_dump in the wrong place.
It is an empty string as an index.
Why?
 
Because you build it with an empty string? :P
I am going to remove the phpunit run from scrutinizer @iroegbu and let travis report the coverage
No idea why it is borked
 
5:12 PM
I seems okay now
And I haven't updated
 
@PeeHaa Sorry. I meant why would you not let the code pass?
I think it's worth the extra line.
 
Because it's weird for one. And secondly because the stdclass is limited in functionality and adds nothing
 
Can I type cast an array and assign that to a variable?
(IE don't have to type cast it every time.)
 
Why would you cast it?
 
5:16 PM
Because I like using configuration OBJECTS not ARRAYS.
Bi-product of working full-stack, probably.
 
wat
 
Fail.
Byproduct*
 
even with the fixed typo: wat
 
Everyone uses objects for configuration in JS.
 
There are only objects in js
 
5:17 PM
I'm just used to thinking of config that way and it's easier to read to me, as well as less to write in use.
 
@NikiC I got fed up of having a bland and corporate public persona and a more expressive private persona, so I jettisoned both of them in favour of a new pseudonym. :p
 
@PeeHaa That doesn't change what I said.
 
And they are actually flexible
Unlike PHP's fake stdclass
 
(haven't touched StackOverflow or GitHub yet though. will At Some Point™)
 
@Allenph It does change the fact that if you use that as an argument to use it in php it makes no sense
const config = {
    foo: 'bar',
    baz: {
    }
}
 
5:20 PM
@bwoebi I have more Twitter accounts than can be understood by mere mortals
 
Now do that in php
 
(or at least I can't keep track of them…)
 
@Andrea I … noticed. :o
 
@Andrea lol
 
@bwoebi So we can stop playing git-tag, does this look correct?
 
5:21 PM
I imaging you at some point losing track yourself and trolling your own account
 
though as of yesterday I have 3 less than before in active use, so :p
@PeeHaa hah
 
@Trowski patch line 56/57 is unnecessary as well as 79/80
@Andrea but you do still have a private account in active use that I do not follow?
 
@bwoebi several
 
@Andrea I mean with similar activity than @hikari_no_yume did while private
 
@bwoebi nah. not likely to end up with one either
 
5:26 PM
fine :-)
 
@PeeHaa I see what you mean.
I still prefer to do it that way. Configuration object may be the wrong word.
 
Anonymous
@Wes what about without the circles? They don't add much, simplicity might be better
 
In this case, it is an object representing a registered route...however I didn't create a class for it because it by nature will only ever be created by the router.
If you looked at the code, you'd still probably tear it to pieces, but.
 
Wes
felt too bland @JayIsTooCommon but i will submit one without circles
circles actually balance he amount of whitespace
 
(In fact if any of you senior guys would tear my framework to pieces I would appreciate it.)
 
Wes
5:29 PM
there is a lot of white space within u and n, but not so much around i and t
so i've added circles to balance
 
@Allenph You may link it. I am pretty busy myself right now. But I may look at it later and/or somebody else may want to look at it.
Hope you have a thick skin :-)
 
The documentation is 100% outdated.
I'm more interested in code review and as I've learned more the purpose of the framework has changed entirely.
 
/me sees Something extends Model well that smells... /me looks further oooooh class Eloquent extends \Illuminate\Database\Eloquent\Model {};
God I hate that Eloquent crap
 
Anonymous
I prefer that
 
5:39 PM
really don't have time for a review now, but it looks inspired by laravel
 
Anonymous
 
Anonymous
But each to their own
 
meaning statics everywhere
At the very least use braces instead of using faith driven statement / control structure bodies
@JayIsTooCommon I like that one
Also start writing tests
 
Anonymous
Yeah I need to crack on with that
 
Anonymous
mobile.twitter.com/nihylum/status/850117786626084864 this one looks like the phant just shat out a unit
 
5:42 PM
@JayIsTooCommon Wasn't meant for you :P
 
Anonymous
Oh :p still counts
 
yeah :D
@JayIsTooCommon hehehehe
 
Wes
@JayIsTooCommon sketchtoy.com/68009164
 
@Andrea You're certainly very active :P
 
@NikiC are you saying I tweet excessively? :p
 
Anonymous
5:45 PM
@Wes What a lovely bird
 
@Wes the penis looks so happy
 
Wes
ahah
 
@bwoebi Yeah, since that's an error anyway, no use cleaning up for it.
 
Wes
that's supposed to be a fish @JayIsTooCommon
 
@Trowski right
 
Anonymous
5:47 PM
@Wes oh.... awkward
 
@Andrea I need more birb photos *-*
 
@Andrea Yes ^^ Also, how is your Japanese?
 
@bwoebi unfortunately teaching is over for this semester, so I'm in transit less and thus seeing less birbs
 
FWIW @Wes yours is one of the best
 
@NikiC not very good
 
5:48 PM
Your logo on the gallery not your penis
 
in 2010 I learned the kana and a little grammar, but never got much further. most everything else has been picked up from osmosis over time.
 
Wes
lol
i'm not sure. i think i prefer the simple one more
 
yes that's the best so far
But there are a couple that are goodish too
 
@NikiC I think the single thing that's improved it most is installing the rikaichan mouseover dictionary Firefox extension
 
@Andrea Recently learned the kana as well, now working on kanji based on heisig
 
5:51 PM
@NikiC Ah! I have that book on my shelf. I should return to it; this summer would be a good time.
 
Wes
@PeeHaa the logo there is very much ok, font is terrible
 
yes I think I agree
eeeeeeew
I agree with @Wes
 
evening
 
Wes
never happened before
 
nope it's a first
 
5:52 PM
@Andrea that does look useful, will try
 
Kinda scary
@Linus yo
 
o/
 
6:07 PM
@PeeHaa do I need to squash or just go ahead and push with all the corrections?
 
If you can squash that would be great <3
 
> to squash or not to squash
sup o/
 
sup @FélixGagnon-Grenier
 
having a concert tonight.
 
Good boy
 
6:09 PM
oh, right
 
3 messages moved to Trash
 
:P
 
I realize that maybe I could be a bit less forthcoming, sometimes
 
meh
 
heh. aight :)
 
6:10 PM
Just take your xtc and go wild
 
alrighty m8 gr8
 
I am such a good influence
 
:)
does this qualify as haiku?
 
lol
 
> if db doesn't
> exist then please create for
> me a test db
it has the hard to read motion, but it just feels wrong (ofc I always being so very productive while working)
 
6:13 PM
hehehe
What concert are you going to?
 
*playing in
 
oooooh cool
 
If you Google pierre mercure concert philou you should see something. It's for a center the takes care of handicapped kids
 
cc @JayIsTooCommon ^
 
We even get to play some contemporary tigan music from a presently living composer. Normally all repertoire is from dead people so that's exciting
@PeeHaa lol
 
6:17 PM
Yeah you do that classical stuff right?
 
That's right
erm
 
:P
Ugh french sucks
 
yeah we don't translate much
so egocentrist. we got that from the french
 
> 500 - Internal Server Error
EVENT DETAILS
CONCERTOS POUR PHILOU
lol
 
I... see it correctly?
maybe some dns stuff
 
oh it's gone now for me too
Was afraid I missed a non french text :D
 
:D
oh well screw it I will follow @Jeeves advice
 
@FélixGagnon-Grenier Are you green?
 
6:24 PM
lolwat
 
@Jeeves I will be very soon, and you?
 
hahahaha
 
@FélixGagnon-Grenier What if they are very nice?
 
this is scary. hahaha
 
it is :P
 
6:25 PM
hehe!
dem eyes
 
I was not dissapoint by that image search at all
 
green elephant?
 
stoned
 
hahha
 
lolya. pretty un-disappointing
 
Wes
6:27 PM
imagine it with colors :B
 
the dot over the i draws my stare
 
@Wes that reminds me a little of a Jenga puzzle.
…WordPress Jenga, anyone?
 
Wes
:P
what's wordpress jenga?
 
WordPress Jenga: a game where you have to remove random functions from WordPress and not break the site.
6
(an idea I had just now)
 
Wes
ahahaha
 
6:29 PM
I love it
this is called "post production optimization" rite?
 
Wes
isn't all software like that, a jenga game :D
 
hahahaha
@FélixGagnon-Grenier optimisation in prod :p
 
6:34 PM
WordPress Jenga: a game where you have to remove random functions from WordPress and not break the site.
 
Anonymous
@PeeHaa hahaha
 
Anonymous
@FélixGagnon-Grenier what do you play?
 
the violin :)
 
Anonymous
That's awesome
 
indeed :) it's a marvelous instrument, I find
 
Anonymous
6:37 PM
Are you any good ?
 
@JayIsTooCommon :-)
 
@JayIsTooCommon not sure how to measure that, but I've been playing for 21 years now, and people pay me to play in concerts with two rehearsals in three days.
Not yet of the level of international orchestras such as the Orchestre Symphonique de Montreal, but not completely amateur either
 
Anonymous
Any vids?
 
I have nothing to play again
 
6:52 PM
@kivlara If you pull the latest master of amp it should destruct things a little faster.
 
@PeeHaa if this makes sense I can generate pull request again github.com/iroegbu/AsyncTwitter/commit/…
moved some examples
 
Oh nice
Yeah makes sense
Was on my list to fix that
tnx
 

« first day (2365 days earlier)      last day (2583 days later) »