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
|