Laytan comments

Results 212 comments of


                                            Laytan

add crypto.rand_bytes for Darwin and BSD

Rebased this to fix conflicts, still ready for review

[runtime]: Add os_specific_linux, that calls the syscall directly.

I think the idea with runtime is to import nothing (other than intrinsics) and copy implementations. I also had a look and it seems writing is the same syscall (number...

[runtime]: Add os_specific_linux, that calls the syscall directly.

Ah I had the wrong assumption, I looked at a couple syscall tables for the mentioned platforms and it was 4 everywhere. Now I did another search and see they...

New, different error (than #2817 was generating) using -target:wasi_wasm32 or -target:wasi_wasm64p32

Do you have wasm-ld in that location? If you do, are you sure there isn't more output given with the failure? You can also try running the command yourself on...

Compiler produces bad code

We did a few ABI fixes in the last few weeks which has fixed this for me, could you confirm/close the issue?

core/container/avl: Add an AVL tree implementation

LGTM. The only thing I might like to see is a rename of the `insert` procedure, the behaviour is more like a `find_or_insert` because it does not insert if it...

Add xinput bindings to core:sys/windows

Just want to mention there was some talk in the discord saying it is a bad idea to link to xinput at compile time, not sure if that is true,...

Make MacOS releases self contained / bundle LLVM

It actually went down to 60MB because we are zipping it now. Here is a run I did on my fork: https://github.com/laytan/Odin/actions/runs/8010221628 if you want to check out the artifacts.

Stream read returns No_Progress, instead of EOF

Fixed in #2998

Improper bool assignment in successive calls to proc with three return values

Probably the same underlying issue as #3159