Results 41 comments of Derek Hensley

While we still have the separate FileName attribute, I feel like we should still use `snake_case` for the filenames.

Just want to note, since merging the last PR, I have noticed there are 2 actors that have PARAMS getters named after TARGETMODE (which was renamed in the last PR)....

> Just want to note, since merging the last PR, I have noticed there are 2 actors that have PARAMS getters named after TARGETMODE (which was renamed in the last...

>z_olib.c needs to include z64actor.h because of PosRot, which feels kinda dumb, we may want to move that struct somewhere else, since it isn't actor specific. Perhaps just in `z64math.h`...

> As a small fyi I ended up removing `graph.h` since only actually two symbols were being used outside of `graph.c` I moved those two declarations to `gfx.h` Looking at...

Personally I think I would still prefer `AVOID_UB` though perhaps there is someplace we could document the define, and how it is meant to preserve what the UB does in...

Looks like a format is needed :)

> There are some parts of this PR that still need a bit more discussion, but I think the new `z_en_a_obj.h` and `z_en_item00.h` headers are good to have. What do...

Held for documentation and https://github.com/zeldaret/oot/pull/2267

> Few things i noticed while testing this: > > * I also have a temp-file problem, every compile after a "setup" run creates exactly 4030 empty files in /tmp/...