|
00:33
MasterDuke joined
02:00
lizmat joined
02:56
ilbot3 joined
07:05
domidumont joined
07:10
geospeck joined,
brrt joined
07:12
domidumont joined
07:36
lizmat joined
07:46
brrt joined
07:56
Ven joined
09:25
robertle joined
09:58
xi- joined
10:04
lizmat joined
10:46
Ven joined
10:48
Geth joined
11:06
Ven joined
11:38
bisectable6 joined
11:52
brrt joined
12:16
Ven joined
12:53
zakharyas joined
13:50
Ven joined
14:14
geospeck joined
14:22
geospeck joined
14:25
Ven joined
14:39
Ven joined
14:53
Ven joined
14:59
geospeck joined
15:09
Ven_ joined
15:34
Ven_ joined
15:54
Ven joined
17:26
lizmat joined
17:43
geospeck joined
17:59
domidumont joined
18:21
geospeck joined
18:26
geospeck_ joined
18:31
geospeck joined
18:38
robertle joined
19:12
MasterDuke joined
19:14
MasterDuke_ joined
|
|||
| MasterDuke_ | anyone have an objection to github.com/MoarVM/moarvm.org/pull/8 ? | 19:15 | |
|
20:15
committable6 joined
20:21
zakharyas joined
|
|||
| dogbert17 | shouldn't there be parentheses around this MVM_BIGINT thingy or are they really optional? github.com/MoarVM/MoarVM/blob/mast...ops.c#L720 | 20:23 | |
| i.e. 'if (MVM_BIGINT_IS_BIG(ba)) {' instead of 'if MVM_BIGINT_IS_BIG(ba) {' | 20:26 | ||
|
20:36
zakharyas joined
20:38
zakharyas joined
|
|||
| jnthn | All's fair in macros and war... | 21:13 | |
| yoleaux | 18:13Z <lizmat> jnthn: I have an almost working prototype of .toggle, but first will have some LPW socializing and dinner | ||
| jnthn | dogbert17: I likely compiles 'cus the macro puts parens around what it expands to, which is probably wise, but yeah, it looks a bit odd :) | 21:14 | |
| dogbert17 | it causes the program cppcheck to generate a syntax error and then I noticed that parens had been used elsewhere in the file | 21:21 | |
| I might PR that | |||
| jnthn | Feel free :) | 21:28 | |
|
22:04
japhb joined
22:32
japhb joined
22:41
geospeck joined
|
|||
| Geth | MoarVM: dogbert17++ created pull request #755: Fix missing parens |
22:41 | |
|
23:11
lizmat joined
|
|||