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

Asking NPCs to teach martial arts causes segmentation faults #1831

Closed
3t3rn4lfl4m3 opened this issue Jun 29, 2013 · 4 comments

Comments

Projects
None yet
5 participants
@3t3rn4lfl4m3
Copy link

commented Jun 29, 2013

Every time I ask an NPC to teach me a martial art they know, I get a segmentation fault. Asking them to teach skills works without problems.

I am playing "Version: 0.6-116-gd1d6525".

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@oddbjol

This comment has been minimized.

Copy link
Contributor

commented Jun 29, 2013

Martial arts aren't skills, they're weirdly behaving items. The training code for npcs assigns an ACT_TRAIN task to the player, with a nonexisting skill (the martial art).

I'm sure there's an easy fix for this, but there are issues like this one all over the code. Martial art styles should probably be implemented differently.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Jun 30, 2013

Huh. It worked fine for me back in the 02Feb day.

@BevapDin

This comment has been minimized.

Copy link
Contributor

commented Jan 18, 2014

No, but that crash might be removed by #5606 (which does not affect this issue).

@mugling

This comment has been minimized.

Copy link
Contributor

commented Mar 13, 2016

Much of the NPC code has been rewritten since this was reported nearly 3 years ago. @3t3rn4lfl4m3 please re-open if you can reproduce in any recent version of master

@mugling mugling closed this Mar 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.