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