-
Joined on
2024年08月09日
I've updated ziglang/zig#31542's title so it includes all "less common" filesystems, i.e. not covered by CI.
If this logic is accepted, then it should be implemented in Io.Uring too and at the very least tested manually, because at the time of writing this, unit tests don't cover Io.Evented...
If openat reveals the file exists, I think it's better to return error.PathAlreadyExists to simulate what happens with NOREPLACE instead of panic.
Wouldn't it be better to just call Threaded.renameat here instead of complicating control flow by mutating the flags?
This is always true, so the branch is unnecessary.
I've updated this follow up issue here: ziglang/zig#31542
Thanks, though I prefer to keep it as OpenError. I think it's clear enough from the namespace, it mirrors std.Io.Threaded, and it's nicer for when you grep. Otherwise, OpenAtError would be...
Rebased now and cleaned up the commit history, which I should have done in the first place. Again, thank you both.
switch statements on large types' (#31557) from justusk/zig:cbe-switch-128 into master
libzigc: Implement tanhf, tanh, modfl, modff, modf, acoshf, asin' (#31536) from mihael/zig:libzigc/more-math-functions into master
Thanks for the tip!
While it's not necessary for Io.Threaded implementation on its own, and that's how this PR started, I've added this error code to File.OpenError because of the Io.Uring implementation.
Thanks. Updated both std.Io.Threaded and std.Io.Uring implementations.
std.fs test failures on NFS on linux