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

Drawing damaged or reinforced knifes from survivor utility belt causes crash #13453

Closed
chaosvolt opened this issue Aug 30, 2015 · 12 comments

Comments

Projects
None yet
5 participants
@chaosvolt
Copy link
Contributor

commented Aug 30, 2015

Did some science of this after Thwap gave some more helpful info in the thread they posted about this ( http://smf.cataclysmdda.com/index.php?topic=11124.0 ).

Drawing a knife from the survivor utility belt will cause a crash if the knife is damage OR reinforced. Basically any condition besides the norm. Sheath, boots, and normal survivor belt all act normal.

Screenshots for the screenshot god, PNGs for the PNG throne:

the crashening

the revenge of the crashing

@Griffinhart

This comment has been minimized.

Copy link
Contributor

commented Sep 1, 2015

Unable to reproduce on Cygwin x86 ncurses build version 0.C-5853-g1cad147.

Methodology:

  1. Reset testing world.
  2. New character.
  3. Debug menu spawned survivor utility belt, steak knife, soldering iron, scrap metal.
  4. Wore || survivor utility belt.
  5. activated || soldering iron and made the || steak knife extra sturdy.
  6. Sheathed the ++ steak knife in the || survivor utility belt.
  7. Drew the ++ steak knife from the || survivor utlity belt.
  8. Worked just fine.
@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

Not sure if that means it's fixed or if it's a specific build issue.

@Griffinhart

This comment has been minimized.

Copy link
Contributor

commented Sep 1, 2015

I'm inclined to think the latter.
On Sep 1, 2015 08:05, "Chaosvolt" notifications@github.com wrote:

Not sure if that means it's fixed or if it's a specific build issue.


Reply to this email directly or view it on GitHub
#13453 (comment)
.

@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

I would assume so. I'm assuming Cygwin is a compiler? As both the OP in the thread and I used pre-compiled experimentals from the main site.

@Griffinhart

This comment has been minimized.

Copy link
Contributor

commented Sep 1, 2015

Cygwin is a Unix-like environment for Windows (strictly speaking, it's a
DLL and an API layer) that includes many compilers. You can kind of think
of it like a Unix emulator, but that's entirely incorrect (and Red Hat
themselves explicitly note that Cygwin is not an emulator).

I compiled with Cygwin gcc (I want to say 4.9.3, but it might be 4.9.2; I'm
not at the machine to check).

e:

$ g++ --version
g++ (GCC) 4.9.3
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

On Sep 1, 2015 12:19 PM, "Chaosvolt" notifications@github.com wrote:

I would assume so. I'm assuming Cygwin is a compiler? As both the OP in
the thread and I used pre-compiled experimentals from the main site.


Reply to this email directly or view it on GitHub
#13453 (comment)
.

@Coolthulhu

This comment has been minimized.

Copy link
Contributor

commented Sep 1, 2015

Couldn't reproduce.

The only explanation I have for this goes like this:

  • The fact that only damaged/reinforced items cause the crash suggests heavily that it has something to do with drawing of the colored health bars next to them. If I'm right, turning off health bars in options->interface should stop the crash
  • The crash is probably related to something with pointers, hence why it doesn't reproduce reliably and why compiler choice affects it
  • Different versions of the game may not have the crash, even though it wasn't actually fixed
  • May be different in tiles/non-tiles and x32/x64. When you get a version that crashes, report the entire version. Tileset choice shouldn't affect it (should happen even with tiles off).
@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

Hmm. Very odd. Um...dafuq?

To quote Thwap in the thread:

Confirmed: Turning off item health bars in the interface settings prevents the crash.

@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 1, 2015

As for why different versions do this...no idea. I might go download the latest Windows console experimental to see if it works.

EDIT: attempting to test whether the interface option affects this led to hilarity ensuring. Load errors because the last version I had close at hand was one I'd been using to test my changes to Tank mod, loaded an older save I had for testing this issue... XD

EDIT 2: In any case, doubly confirmed. No idea yet of the cause or what makes it build-specific, but disabling item health display in Windows, tiles mode, build 3626 (a3a5eb2) prevented a crash.

@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 2, 2015

Well. Tested the ASCII version, latest experimental build 3646 (1cad147).

more borks

@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 2, 2015

To quote Thwap again:

So I messed with the jsons a bit more and tested it a couple different ways, just to rule anything else out, and here's pretty much the final word on this bug in concise form:

IF an item is wearable
AND the item has a reinforced/damaged knife sheathed in it
AND the item has more than one use_action, thus requiring the menu
AND item health bars are enabled
AND you're using the pre-compiled experimental from the website

Then the game crashes as soon as the item is activated, without displaying the menu. If any one of these conditions aren't met then there's no crash, and no other conditions seem to apply. Maybe someone can paste this to the Git entry for the bug.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Sep 2, 2015

@chaosvolt

This comment has been minimized.

Copy link
Contributor Author

commented Sep 2, 2015

Hmm. Anyone got more info on what builds do and do not exhibit this issue?

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.