mm icon indicating copy to clipboard operation
mm copied to clipboard

Remove global dependencies (1)

Open mzxrules opened this issue 8 months ago • 4 comments

One thing of note: a pair of actor headers written in a way that made them dependent on GreatFairyType enum z_bg_dy_yoseizo.h. I rewrote them to not require this dependency because z_bg_dy_yoseizo.h itself imports the dependency of object_dy_obj.h, which isn't really valid for either actor.

mzxrules avatar Mar 24 '25 06:03 mzxrules

Changes made in this pr are licensed under CC0.

mzxrules avatar Mar 25 '25 09:03 mzxrules

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 you think about moving them to a new PR? We can probably get them merged them sooner.

AngheloAlf avatar Mar 25 '25 11:03 AngheloAlf

About the current PR, as you have probably noted there are still a few stuff that haven't been moved to a better place and stuff still depend on global.h.

I think it would be easier to do cleanup in other headers and get rid of other stuff, like, for example, getting rid of functions.h and variables.h

AngheloAlf avatar Mar 25 '25 11:03 AngheloAlf

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 you think about moving them to a new PR? We can probably get them merged them sooner.

I would agree with the sentiment. It would be nice to separate those two headers into there own PR.

hensldm avatar Apr 19 '25 22:04 hensldm