Skip to content

Conversation

@Goober5000
Copy link
Contributor

@Goober5000 Goober5000 commented Nov 12, 2025

This dates back to commit 300d5fc2c1. It probably did something at some point, but with various upgrades over the years, it no longer has any effect.

It looks like a lot of the model interp code (e.g. model_interp_defpoints, which isn't called anywhere) can be removed as well, but that's a bit more involved.

@Goober5000 Goober5000 added cleanup A modification or rewrite of code to make it more understandable or easier to maintain. Waiting for Stable Marks a pull request that is to be merged after the next stable release, due to a release cycle labels Nov 12, 2025
This dates back to commit `300d5fc2c1`.  It probably did something at some point, but with various upgrades over the years, it no longer has any effect.

It looks like a lot of the model interp code (e.g. `model_interp_defpoints`, which isn't called anywhere) can be removed as well, but that's a bit more involved.
Copy link
Member

@Baezon Baezon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn, this is going to ruin the bit wrt to the number of 'totally for real final death time' timestamps ships have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup A modification or rewrite of code to make it more understandable or easier to maintain. Waiting for Stable Marks a pull request that is to be merged after the next stable release, due to a release cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants