the-real-herowl
  • Joined on 2023-08-07
the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-26 02:54:24 +02:00
Wither finishing, effect fixes

Fixed in #3951.

the-real-herowl created pull request VoxeLibre/VoxeLibre#3951 2023-09-26 02:50:51 +02:00
WIP: Potions API redo
the-real-herowl created branch potions_api_redo in the-real-herowl/MineClone2 2023-09-26 02:44:23 +02:00
the-real-herowl pushed to potions_api_redo at the-real-herowl/MineClone2 2023-09-26 02:44:23 +02:00
f5df2c8f78 General effects API overhaul
71d41cbe95 Added register_effect()
Compare 2 commits »
the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-24 15:23:10 +02:00
Wither finishing, effect fixes

Fine, but what size should it be exactly? I set it at what felt right to me. It's like one number, it's not a problem to change it, or even make it into a setting.

All you ever wanted…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 20:38:14 +02:00
Wither finishing, effect fixes

Removed.

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 20:37:19 +02:00
Wither finishing, effect fixes

Removed searching for nodes, moved mob affecting to the mcl_mobs (I made it similar to magma blocks damage). Now it only looks through players, and only checks a single node per player. Should be…

the-real-herowl pushed to master at the-real-herowl/MineClone2 2023-09-23 20:34:00 +02:00
6e60fb1715 Optimisation and cleanup
the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 03:34:24 +02:00
Wither finishing, effect fixes

Yeah, so I tested ABMs, and they are very unreliable. They mostly don't work when running through the rose. We could of course add a check for the effect before searching for the node, but it…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 02:19:14 +02:00
Wither finishing, effect fixes

Yeah, this warrants a redo.

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 02:17:57 +02:00
Wither finishing, effect fixes

Yeah, I think the potions API is due for a rewrite. What I have in mind is getting rid of those specific functions and making it possible to register effects (in mods too) by name, and dealing…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-23 02:12:12 +02:00
Wither finishing, effect fixes

This is dealing effect at attack, when hitting with a melee attack. This is for wither skeletons, because since I was doing wither effect, I wanted to put it wherever it should be (and it required…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 23:08:12 +02:00
Wither finishing, effect fixes

I don't think I am the right person to review this as I never really touched mob related stuff, but here are some minor things I've found after skimming through the code...

Also,…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:51:11 +02:00
Wither finishing, effect fixes

Yeah, so I figured that all "wither" sort of mobs are supposed to be resistant. Wither boss is already resistant due to being boss. This leaves only the wither skeleton, this is true, but it makes…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:48:26 +02:00
Wither finishing, effect fixes

True, I'm removing it. Not commiting yet, since this is "just" a pointless if.

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:46:17 +02:00
Wither finishing, effect fixes

Well, look at the other effects. I kept it in the same style. The whole potions API could be redone though, just of course not in this PR.

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:44:23 +02:00
Wither finishing, effect fixes

Firstly, it makes it compatible with any ghasts variant a mod could add (or we could add down the line, which may not happen). Secondly, I had coded the spider poison resistance by using a…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:39:09 +02:00
Wither finishing, effect fixes

If it is a player, entity is not present... Yeah, seems like the condition should be expanded.

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:36:11 +02:00
Wither finishing, effect fixes

Because I consider it cheap enough that timer only convolutes things. I think the function call staying on every step and an if and the timer would take a similar time to setting a bunch of…

the-real-herowl commented on pull request VoxeLibre/VoxeLibre#3878 2023-09-22 22:33:48 +02:00
Wither finishing, effect fixes

That's why I have wither despawning code: wither follows the spawner, and despawns after a set time from him going offline (if somebody is fighting the wither in this time, this turns back the…