I'm confused, this just puts a CVar around existing code right? How does this fix the issue it's supposed to fix?

I'm confused, this just puts a CVar around existing code right? How does this fix the issue it's supposed to fix?

if i'm understanding this correctly, we already have a fix implemented for an authentic bug, this makes it so that fix is wrapped in a cvar

Actually I think this is a fix for the GC versions. See https://github.com/zeldaret/oot/blob/c3dc2994485277e3ff17f63f89335ab5c1cca7c6/src/overlays/actors/ovl_En_Ge3/z_en_ge3.c#L241 in the decomp. Regardless, we generally target the N64 version when it comes to what bugs to keep, so it makes sense to move this to a CVar.

Actually I think this is a fix for the GC versions. See https://github.com/zeldaret/oot/blob/c3dc2994485277e3ff17f63f89335ab5c1cca7c6/src/overlays/actors/ovl_En_Ge3/z_en_ge3.c#L241 in the decomp. Regardless, we generally target the N64 version when it comes to what bugs to keep, so it makes sense to move this to a CVar.

iirc it was fixed in 1.1 (maybe only PAL 1.1)

@xoascf could you point this at zhora?

Could you also add this to the default enhancement preset zeroing it out?