Laytan
Laytan
I feel like we are talking past each other here. When using the default heap allocator in Odin, yes, we don't need to know the size and everything works already....
Alright, I will keep this open for a maintainer to close if they agree with you, thanks for looking into this though!
Could you elaborate on trying something similar with C and Zig? In my tests they also don't export the variable by default. I would like to see the code snippet...
To me, a define is something that is defined, and a defineable something you can define. I do agree it is not a very ergonomic name though.
I've rebased and improved the PR. Now it also collects doc comments tied to the declaration if available, this also needed a change to the way it is shown because...
There is also an issue with the current builds on Linux, it uses part of llvm-14 and part of llvm-11 to build/distribute. I confirmed that this is not the case...
Made this PR (https://github.com/odin-lang/Odin/pull/3329) to fix the last issue that came up while doing this one. Once that is merged we should be all green here, the crypto test that...
You probably also want `os.O_TRUNC` for the destination file, so it truncates the file if it already exists, otherwise it would just add to it
I am running into problems with it too, it is also bleeding into the text functionality of the `vendor:nanovg` package, text seems to render upside down and at the completely...
@gingerBill Just rebased and updated to be in line with the updates on master, can I get a review on this at some point?