-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Worn gloves count as unarmed weapons when no weapon is held #41209
Conversation
Does this break martial arts that limit the unarmed weapons you're allowed to use, like Aikido and Judo? |
I don't understand the point of this? |
After reading this, I had the suspicion that it did and had a fix in mind. But, upon testing, it did not break them -so all good on this case.
The point of the flag is that it treats all gloves as temporary unarmed weapons. This means the following: |
So that means that I can wield studded glove and attack with them as if it was a normal weapon? Isn't that a bit hacky? Why not give a damage entry to armor like maybe |
Sure, you can -but just wielding it won't count it as an unarmed weapon (maybe this is the root of misunderstanding). You'll just be smacking them with the gloves, therefore not subject to anything unarmed related. Having it worn would make it count as unarmed.
Because the unarmed flag is required for everything unarmed related. Its not about just applying the damage from gloves but also martial arts, "gloves" taking damage, unarmed exp/damage calculations, and actual wearable unarmed weapons as an outer layer (which I'm planning to add to existing unarmed weapons if this gets merged). |
Have you considered how this affects Taekwondo specifically? Ideally, the stats of one's shoes should apply. Barring that, Taekwondo becomes the only unarmed martial art that is--unrealistically--not affected by armor. |
I am not interested in adding the edge case of using shoes as weapons just to appease Taekwondo. That would be a seldom visited feature and adds unnecessary complexity. |
What about one-piece armor suits that have gloves/gauntlets incorporated like rm13, phase immersion suits, power armors etc? |
If it covers hands, then it can be added to them. |
Added some unarmed weapons to be worn on hands as the outer layer to give the PR more relevance if merged. |
Great for the cestus, not so good for brass/steel/nail knuckles, which also fill the hand with a grip, preventing you from holding much of anything else along with them. |
I erroneously thought they just wrapped around your fingers this whole time haha. I have updated it so only the cestus is allowed to be worn. |
Summary
SUMMARY: Features "Worn gloves count as unarmed weapons when no weapon is held"
Purpose of change
To facilitate the use of worn gloves as weapons when nothing is wielded since we are technically punching the enemies with them. This allows a unique quality to unarmed combat as we can now be "wielding" our weapons as soon as we drop whatever it is we are holding.
This PR is to facilitate this ability. Glove weapons and/or updates to existing gloves to have damage still need to happen. For the sake of not having dead code, and to prove out the feature, studded gloves have also been added.
Describe the solution
If the selected weapon is null (we aren't holding anything), then use the highest layer of gloves as our weapon instead. If it does not inherently have the unarmed combat flag, add it, then remove it later down the line.
This also converts cur_weapon from a reference to a pointer since reference variables cannot be reassigned. If not done this way, the gloves will be duped and held as a weapon instead.
Allowed the cestus to be able to be worn as an outer layer on hands, and still be used as an unarmed weapon.
Testing
Spawn light gloves, and punch debug monster. Make sure it trains unarmed, and the gloves can take damage.
Turn on force unarmed while wielding a cudgel, and repeat above.
Attack with a cudgel to make sure it still attacks with a held weapon.