|
00:20
vrurg left,
vrurg_ joined
05:54
guifa left
06:32
camelia left
06:34
camelia joined
08:51
sena_kun joined
10:13
sena_kun left
|
|||
| lizmat | and yet another Rakudo Weekly News hits the Net: rakudoweekly.blog/2025/02/10/2025-06-its-a-bot/ | 14:44 | |
|
16:10
vrurg joined,
vrurg_ left
16:23
vrurg left
16:25
vrurg joined
18:22
sena_kun joined
|
|||
| Geth | MoarVM/main: 9a1af71f3d | MasterDuke17++ (committed using GitHub Web editor) | src/strings/utf8.c Some simplifications to MVM_string_utf8_decode (#1910) * Remove always-false checks The size of the normalized results of decoding will always be less-than-or-equal to the number of input bytes, so these checks aren't needed. ... (8 more lines) |
18:50 | |
| lizmat | MasterDuke17: with this patch, 2 spectests fail for me | 19:17 | |
| tellable6 | lizmat, I'll pass your message to MasterDuke | ||
| lizmat | t/spec/S32-str/encode.rakudo.moar (Wstat: 512 Tests: 41 Failed: 2) | ||
| Failed tests: 12-13 | |||
| the first one isn't too bad: it apparently loses some information in the error message | 19:18 | ||
| the second one is more serious: "ssß".encode("latin1").decode("utf8") doesn't die anymore, but encodes to "ss" | 19:19 | ||
| I've already bumped NQP, but will not bump Rakudo | |||
|
21:00
vrurg left
21:01
vrurg joined
21:30
scovit joined
|
|||
| scovit | hello | 21:31 | |
| tellable6 | 2020-10-15T21:39:20Z #raku-dev <patrickb> scovit: Thanks for your input! I think I'll keep the current solution for now. Then we can discuss how to best do it in the future. I don't think there is much of a backwards compatibility issue here. | ||
| 2020-10-15T21:39:00Z #raku-dev <patrickb> scovit: Thanks for your input! I think I'll keep the current solution for now. Then we can discuss how to best do it in the future. I don't think there is much of a backwards compatibility issue here. | |||
| scovit | dirops.c defienes PATH_MAX that is a security issue, would you accept a patch to get rid of it? | 21:32 | |
| basically replacing MVM_dir_cwd with a unix and a windows safe versions | 21:33 | ||
| patrickb | scovit: I have no idea what my above comment was about... :-P | ||
| scovit | patrickb: me neither, long time I don't come here | ||
| patrickb | scovit: That sounds like a pretty straight forward patch and a no brainier to merge. Please go ahead! | 21:34 | |
|
21:36
scovit left
22:16
guifa joined
23:07
guifa left
23:09
guifa joined
23:16
scovit joined
|
|||
| scovit | Can someone review this patch? github.com/scovit/MoarVM/commit/37...d78801dc66 | 23:17 | |
| also, I couldn't test if it builds on windows | |||
| further, it could be that the linux code path works on win32, not sure how moarvm deals with unicode filenames | 23:19 | ||
| my guess is that those are the only difference | |||
|
23:25
sena_kun left
23:32
guifa left
23:35
scovit left
23:53
guifa joined
|
|||