0001126: Crash if calling "togglemap" during demo loading

Notes
(0005234) Torr Samaho (administrator) 2012-10-28 09:01	Doesn't crash for me. Neither in 1.0 nor in the latest testing build.

(0005239) unknownna (updater) 2012-10-28 11:52	It always crashes for me.

(0005240) Torr Samaho (administrator) 2012-10-28 12:16	Strange, it can't make this crash on my end. Your crash log gave a hint though on what could be causing this. Please test if this is any different.

(0005241) unknownna (updater) 2012-10-28 12:26	It still crashes. I uploaded a new crash report.

(0005242) Torr Samaho (administrator) 2012-10-28 12:39	I now managed to reproduce the crash by staring to hit "togglemap" before the demo starts to play.

(0005243) unknownna (updater) 2012-10-28 12:52	Yes, that's what I meant by repeatedly pressing "togglemap" in a fast manner during demo loading.

(0005248) Torr Samaho (administrator) 2012-10-28 19:53	Alright, how is this?

(0005249) unknownna (updater) 2012-10-28 21:04 edited on: 2012-10-28 21:04	That took care of it, but I noticed something else: If you call togglemap before the player joins the game, it will display the overlayed automap once the player joins the game, even if am_overlay is 0. 1. zandronum -playdemo 2012.09.24_08.47.13_doom2.cld 2. Press your bound "togglemap" key once when you see the status bar appear in the console during demo loading. It seems that you can view the map earlier by entering the automap and then exiting it.

(0005250) Torr Samaho (administrator) 2012-10-29 20:36	Ok, how is this?

(0005257) unknownna (updater) 2012-10-30 08:52 edited on: 2012-10-30 10:14	It partially fixed the issue. It still happens if you press your bound togglemap key when the status bar mugshot appears. 1. zandronum -playdemo 2012.09.24_08.47.13_doom2.cld 2. Press your bound "togglemap" key once when you see the status bar mugshot appear in the console during demo loading.

(0005268) Torr Samaho (administrator) 2012-11-01 18:59	Alright, and this?

(0005270) unknownna (updater) 2012-11-01 19:14 edited on: 2012-11-01 19:31	That took care of it. Well done. Edit: Separate issue: "changemap" is disabled when watching demos, but "map" isn't. If "map" is called, the client will use the demo dmflags/compatflags.

(0005277) Dusk (developer) 2012-11-01 19:48	I don't think that's a problem, since "map" starts a new game. The dmflags thing is something that could be looked into and also happens if you "map" out of client mode while connected to a server into singleplayer.

(0005278) unknownna (updater) 2012-11-01 19:52 edited on: 2012-11-01 20:01	Indeed, the dmflags/compatflags are reset when you start a new game from the menu, so the current behavior feels inconsistent. Edit: Actually, it seems that the compatflags aren't reset when you start a new game from the menu. And some other things aren't reset either, e.g., alwaysapplydmflags.

(0005370) unknownna (updater) 2012-11-11 19:45	I think that we should mark this ticket as resolved. If we wish to alter the "map" behavior, we should create a new ticket and discuss it there.

(0005371) Torr Samaho (administrator) 2012-11-11 19:47	I agree.

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue Community Support

This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.

Supporters:

No one explicitly supports this issue yet.

Opponents:

No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2012-10-14 15:34	unknownna	New Issue
2012-10-14 15:35	unknownna	Status	new => confirmed
2012-10-28 09:01	Torr Samaho	Note Added: 0005234
2012-10-28 11:52	unknownna	File Added: CrashReport.zip
2012-10-28 11:52	unknownna	Note Added: 0005239
2012-10-28 12:16	Torr Samaho	Note Added: 0005240
2012-10-28 12:26	unknownna	File Added: CrashReport_02.zip
2012-10-28 12:26	unknownna	Note Added: 0005241
2012-10-28 12:39	Torr Samaho	Note Added: 0005242
2012-10-28 12:52	unknownna	Note Added: 0005243
2012-10-28 19:53	Torr Samaho	Note Added: 0005248
2012-10-28 19:53	Torr Samaho	Assigned To	=> Torr Samaho
2012-10-28 19:53	Torr Samaho	Status	confirmed => needs testing
2012-10-28 21:04	unknownna	Note Added: 0005249
2012-10-28 21:04	unknownna	Note Edited: 0005249	View Revisions
2012-10-29 20:36	Torr Samaho	Note Added: 0005250
2012-10-30 08:52	unknownna	Note Added: 0005257
2012-10-30 10:14	unknownna	Note Edited: 0005257	View Revisions
2012-11-01 18:59	Torr Samaho	Note Added: 0005268
2012-11-01 19:14	unknownna	Note Added: 0005270
2012-11-01 19:31	unknownna	Note Edited: 0005270	View Revisions
2012-11-01 19:48	Dusk	Note Added: 0005277
2012-11-01 19:52	unknownna	Note Added: 0005278
2012-11-01 20:01	unknownna	Note Edited: 0005278	View Revisions
2012-11-11 19:45	unknownna	Note Added: 0005370
2012-11-11 19:47	Torr Samaho	Note Added: 0005371
2012-11-11 19:47	Torr Samaho	Status	needs testing => resolved
2012-11-11 19:47	Torr Samaho	Fixed in Version	=> 1.1
2012-11-11 19:47	Torr Samaho	Resolution	open => fixed
2018-09-30 21:08	Blzut3	Status	resolved => closed

Relationships