Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the Tokimeki / old Juiced 2 method for Breath of Fire III #18653

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

hrydgard
Copy link
Owner

See #18650 / #6379

Merges the two framebuffers into one, instead of complex logic to pick the right texture from the coordinates.

Allows us to delete a piece of complex code from the renderers, but for now at least, requires us to enable the compat flag.

I don't think there are any other known games that rely on this?

@hrydgard hrydgard added the Code Cleanup Cleanup to make future work easier. Needs to be done sometimes. label Dec 30, 2023
@hrydgard hrydgard added this to the v1.17.0 milestone Dec 30, 2023
@hrydgard
Copy link
Owner Author

hrydgard commented Jan 6, 2024

@unknownbrackets summoning for opinion since I'm deleting your old workaround here :) It's always a bit iffy to go from general hack to compat.ini option, but I still would argue that in this case it's cleaner this way.

Allows us to delete a bunch of complex code.
@hrydgard
Copy link
Owner Author

hrydgard commented Jan 11, 2024

I'm going to get this in, it'll make some things I want to do like #18646 easier.

Hopefully we can make the framebuffer-merge heuristic safe enough in the future to enable without a flag - the only known breakage when doing so is that it slows down Juiced 2 drastically since it does self-texturing within the merged range, which isn't an issue in BOF.

@hrydgard hrydgard merged commit fa2cb91 into master Jan 11, 2024
18 checks passed
@hrydgard hrydgard deleted the simplify-bof-rendering branch January 11, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Cleanup Cleanup to make future work easier. Needs to be done sometimes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant