search

D00Med / vehicles

minetest mod that adds vehicles. Written for minetest 0.4.??
Other
17 stars 27 forks source link

Bunch of fixes #63

Closed rollerozxa closed 2 years ago

rollerozxa commented 2 years ago

Bunch of fixes, including turning default, tnt and dye into optional dependencies making it more game-agnostic (works in build game now, along with others), and fixing a lot of deprecation warnings.

D00Med commented 2 years ago

Removing the "unused source" is a bad thing if anyone ever wants to clean up / fix this mod.

Good work though

rollerozxa commented 2 years ago

Oh, is the newer API somehow slower? I guess I could give optimising it a go.

rollerozxa commented 2 years ago

Ok so I did some tiny optimisations. Every vehicle step is now sitting on an even more stable <2ms rate (it was pretty fine previously) and it keeps up well when playing with a significantly lowered dedicated_server_step (all this on my Phenom, mind you), so I don't know what other performance issues there is. If it turns out to really be an issue then the Git history always exists.

D00Med commented 2 years ago

Oh, is the newer API somehow slower? I guess I could give optimising it a go. It doesn't work well to use the same function for boats, planes, and ground vehicles. To improve any one, it becomes very difficult compared to when they were split. I suppose the old API should still just go.

Thankyou for working on this ^^

rollerozxa commented 2 years ago

and thank you for merging! :smile: I also went ahead and made a new release on ContentDB with the new changes included.