Laytan
Laytan
The posix_spawn bindings should go into `core:sys/posix`. When I did that package I thought this header was not useful for Odin because of this quote: > The posix_spawn() and posix_spawnp()...
Probably not a good resolution which is why this isn't a PR, but something like this should work, essentially we need to hash something more than just the ast pointer:...
Not sure really, I think there is no precedent for this in the compiler currently. Maybe hashing the type and expression (as a string) is already enough, maybe not. Would...
The compiler has this piece of code to determine if it should do colors, is this a good idea to port over and use here? https://github.com/odin-lang/Odin/blob/f49ebae9562257effe014e3c175496915041d5f2/src/main.cpp#L2860-L2905
A possible solution is only showing vet errors when there are 0 actual compiler errors. Is that a good solution? Maybe.
> There is a compiler error though, which is the whole premise of the issue. Which is why a solution is to not show vet errors when there are compiler...
I don't think there's an (actionable) solution to this then
@gingerBill seems to have explicitly disabled this in 3fc60930 , assigning to him for clarity
I realise now this might need to return -1 when it went out of bounds to let callers deal with the fact it did (instead of truncating). Will look at...
Kind off fixed in https://github.com/odin-lang/Odin/commit/a219da14ce8bbd15fec4ea962fd0ec175da0f0aa