« first day (26 days earlier)      last day (23 days later) » 

9:44 AM
haha
@kbok what's "rebase" supposed to mean?
 
Rebase instead of merging
 
Can I do it from web UI?
hm, I guess not.
crap, I have to leave.
I'll rebase the changes at the evening, I guess :/
damn uni
 
Like I said, if you're OK with the changes but don't have a git console handy, just tell me and I'll do it
 
If you want to have these on master asap, no problem, I can always make issues later
if not, leave it so I can practice :)
 
Hi guyes
 
9:53 AM
hi
 
Any one in Android here
 
how on earth the topic of the room is related to Android?
 
may be on earth right ?
 
Hahahaha are you for real
The room's title is "Lua C++ API project"
 
heeheeeeee no realy.
ok ok friends leave it
 
10:04 AM
Take your time then. I'll be busy for the rest of the week anyway.
 
 
2 hours later…
11:47 AM
There's a lot of hate on std::function, and the guys asked us to consider not using it. It might to worth to dig into since we could probably remove one level of indirection in the lua_Cfunction wrapper
 
 
7 hours later…
6:29 PM
@kbok what
you mean Lounge said we shouldn't use function?
 
Yeah.
 
because?
 
Extra level of indirection, superfluous type erasure are valid points
There was a lot of invalid points as well
 
lol
umh, I wonder about that type erasure
we have to hold these function objects somehow, no?
 
Yes, but we can hold the function objects instead of std::functions
It will likely be more complex though. Totally not the priority IMO
 
6:37 PM
...
we really gain little
so yeah, laterz
ok reading your prs now
 
 
1 hour later…
7:54 PM
awww dude, you merged it
 
8:11 PM
is it bad or good lol
it was only one commit
 
Yeah, we could've just rebased it into master
 
the big green button was too tempting :(
 
hahah yeah
I just don't like those stupid "merge pull request XYZ" commits and if you don't watch out your history becomes riddles with those things
 
@kbok fail with void return
i think.
 
Yeah
I'm just waiting for the lambda PR to be merged to take care of that
 
8:28 PM
hm?
what else needs to be merged?
@kbok i mean apart from stream
 
9:11 PM
There's another PR
Oh yeah nevermind
 
um
so, void return doesn't work
is this by design?
i guess not.
variant result = invoke(func, params); <- this assumes that invoke returns
what if it doesn't
we have to take this as a special case
and don't try to push the result
soooo SFINAE
 
9:32 PM
right? @kbok
 
I'm not sure. Should we push nothing or nil?
Anyway we might as well implement support for returning tuples
brb
 
Hm, now that I think about it
It doesn't really matter whether we push a nil or not
Aside of performance loss of course
But yeah, since we need tuple support anyway, this has to be investigated
 

« first day (26 days earlier)      last day (23 days later) »