|
github.com/moarvm/moarvm | IRC logs at irclog.perlgeek.de/moarvm/today Set by moderator on 25 July 2013. |
|||
|
00:32
benabik joined
|
|||
| dalek | arVM: a00adad | jimmy++ | src/core/exceptions.c: fixed Win32 build |
02:17 | |
| arVM: 8ecae01 | jimmy++ | src/core/ (3 files): some cleanups |
03:25 | ||
| diakopter | JimmyZ: why'd you take out the root add | 03:46 | |
| JimmyZ | diakopter: I don't understand why it needs, could you explain, please? | 03:52 | |
| diakopter | oh; I hadn't questioned that; lemme look | ||
| dunno. | 03:54 | ||
| yeah I guess leave it out | |||
| JimmyZ | hehe | ||
| diakopter | oh | ||
| I know | |||
| I think I see why | |||
| JimmyZ | why? | 03:55 | |
| diakopter | JimmyZ: well, it's possible FROGGS was attempting to follow jnthn's advice about grabbing a ref to the frame? | 04:05 | |
| eh, nah | |||
| JimmyZ doesn't know :-) | 04:07 | ||
|
05:00
crab2313 joined
07:09
FROGGS joined
|
|||
| FROGGS | diakopter: thanks | 07:10 | |
| yoleaux | 27 Jul 2013 22:29Z <diakopter> FROGGS: ((MVMExceptionBody *)OBJECT_BODY(exObj)) is unnecessary; all you need is ex.body | ||
| 27 Jul 2013 22:29Z <diakopter> FROGGS: it's in there twice in cb52130 | |||
| 27 Jul 2013 22:30Z <diakopter> FROGGS: er, ex->body.origin ergh | |||
| diakopter | FROGGS: well, I dunno | 07:14 | |
| I couldn't think of a reason | |||
| FROGGS | JimmyZ: can you commit whitespace-cleanup separately please? It makes it easier to read your changes in a commit | ||
| diakopter: reason? what reason? | |||
| diakopter: I was talking about ex.body->origin... | 07:15 | ||
| diakopter | ah, yes, just trying to speculate why the add root thing | ||
| JimmyZ | FROGGS: sorry, my editor does trim_trailing_white_space_on_save: true | 07:21 | |
| FROGGS | JimmyZ: yeah, I guesses that | ||
| diakopter | FROGGS: did you have a reason for adding the root? | 07:22 | |
| JimmyZ | I use sublime text editor | ||
| FROGGS | I got it from here: github.com/MoarVM/MoarVM/blob/mast...ops.c#L367 | ||
| JimmyZ: maybe just save before changing? and commit if it was modified? | |||
| diakopter | ah; that one's there because the pointer is cached on instance | ||
| FROGGS | ahh | ||
| oaky | |||
| diakopter | FROGGS: though actually to be more precise | 07:23 | |
| FROGGS: actually yeah that's a bug | |||
| JimmyZ | diakopter: root? I thought you say 8ecae0129f | ||
| diakopter | it should add &instance->clargs instead, after it's assigned | 07:24 | |
| oops | |||
| JimmyZ | I removed MVM_gc_root_temp_push there | ||
| diakopter | actualy it was permanent | ||
| FROGGS: could you fix it in procops.c? | |||
| I'm glad to too | 07:25 | ||
| er | |||
| FROGGS | I can later, must go off now, see ya | ||
| diakopter | oops, gotta MVMROOT clargs too. | ||
|
11:44
colomon joined
|
|||
| dalek | arVM: 236e8c1 | jimmy++ | src/ (3 files): small fixes |
11:51 | |
|
12:02
crab2313 joined
12:35
birdwindupbird joined
13:20
crab2313 joined
13:25
colomon joined
14:18
benabik joined
|
|||
| dalek | arVM: 827d256 | jimmy++ | / (8 files): added pushcompsc, popcompsc op |
14:36 | |
| arVM: 140422f | benabik++ | / (138 files): Kill all whitespace at end of line |
14:37 | ||
| benabik | There. That should cause less noise in JimmyZ++'s commits. | 14:38 | |
| JimmyZ | benabik++ | 14:43 | |
| benabik | ack and sed. :-) | 14:44 | |
| dalek | arVM: 2262765 | jimmy++ | src/core/interp.c: removed needless MVMROOT |
14:49 | |
| arVM: 6cc86a2 | jimmy++ | / (6 files): small fixes to pushcompsc |
15:11 | ||
|
15:12
benabik joined
16:38
colomon joined
|
|||
| diakopter | benabik: heh. | 18:12 | |
| well, surely I can make some good progress on dyncall today.. | |||
| benabik: only problem is.. all my editors add the indents | 18:16 | ||
| yet another topic: well, no one's laughed too hard at my ucd2c.pl | 18:17 | ||
| publicly yet, anyway | 18:18 | ||
| benabik | diakopter: Add a pre-commit hook that rejects commits with whitespace-at-eol? | ||
| diakopter | heh. | ||
| (I keep waiting for someone to wander or stumble into ucd2c.pl and do a jaw-dropping triple-take at the .. terribleness. | 18:19 | ||
| () | |||
| ) | |||
| benabik | It did make my version of sed explode. | 18:20 | |
| diakopter | .oO( what do you call fishing for anti-compliments ) |
||
| bwhaha | |||
| explode how? | 18:21 | ||
| benabik | Something like "re error: invalid byte sequence" or something. | ||
| OS X see is often kinda special. | |||
| *sed | |||
| diakopter | well, it's not inconceivable that it's an invalid utf8-level encoding | 18:22 | |
| my editor's free software after all. ;) | |||
| so.. since we're already doing template replacement for Makefile.in, why not substitute every $() variable that's not dynamic anyway | 18:24 | ||
| slow down configure; speed up make infinitesimally? | |||
| I guess that does tie it more to p5 | 18:25 | ||
| eh, slightly. | |||
| well I can always ask ingy to convert it to shell only | |||
|
18:27
colomon joined
18:35
Alpha64 joined
|
|||
| diakopter | I wonder if jnthn would be ok with me adding loops/lists to the template substitution in configure for the makefile | 18:36 | |
| diakopter guesses so | |||
| masak | as long as it still builds on Windows, I guess he's fine with it... :) | 18:59 | |
| diakopter | :P heh this'll definitely only build on windows first | 19:00 | |
|
20:06
crab2313 joined
20:43
colomon joined
22:21
colomon joined
23:13
colomon joined
|
|||