[bfbf7ff9c]Crash on start-up
Moderator: GZDoom Developers
Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
- Major Cooke
- Posts: 8176
- Joined: Sun Jan 28, 2007 3:55 pm
- Preferred Pronouns: He/Him
- Location: QZDoom Maintenance Team
Re: [bfbf7ff9c]Crash on start-up
Confirmed, I upgraded to VS2017 and ran CMake targeted for VS2017. It all works now.
-
-
- Posts: 3144
- Joined: Wed Nov 24, 2004 12:59 pm
- Graphics Processor: ATI/AMD with Vulkan/Metal Support
- Contact:
Re: [bfbf7ff9c]Crash on start-up
This is more or less fine except right now the build machine is stuck with 2015. Hoping to resolve before the end of this month. So far Microsoft partners (since Microsoft's site points to contacting partners for it) are just ignoring my inquiries for volume licensing, so going to try just calling Microsoft soon.Graf Zahl wrote:Or we may simply unsupport VC++ 2015 for being buggy.
Re: [bfbf7ff9c]Crash on start-up
I just tried to build g73fd072d1 and I'm getting crash on startup too.
VS2017 Community 15.9.3
Yes, hard-deleted and re-generated build directory with CMake
Nothing, still crashes.
VS2017 Community 15.9.3
Yes, hard-deleted and re-generated build directory with CMake
Nothing, still crashes.
Re: [bfbf7ff9c]Crash on start-up
Are you sure it's the same crash? At the moment it crashes for me in S_UpdateSounds() with listenactor being null.
This is completely unrelated to a potential code generation bug in VS2015.
This is completely unrelated to a potential code generation bug in VS2015.
Re: [bfbf7ff9c]Crash on start-up
Oh, I'm unsure about that. In that case I'll make a separate bug report with a crash log, to rule out bad compiler crash.
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49073
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: [bfbf7ff9c]Crash on start-up
_mental_ wrote:Are you sure it's the same crash? At the moment it crashes for me in S_UpdateSounds() with listenactor being null.
This is completely unrelated to a potential code generation bug in VS2015.
I pushed a quick fix for this. Since the global timer needs to be moved out of the level struct anyway I'm not bothering with a complete fix for now.
Re: [bfbf7ff9c]Crash on start-up
I wanted to submit the same change, but it was not enough. Listener can be null here as well.
This one is a bit tricky. Am I right that if type is not SOURCE_None, listener and level should not be null?
This one is a bit tricky. Am I right that if type is not SOURCE_None, listener and level should not be null?
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49073
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: [bfbf7ff9c]Crash on start-up
Inside a level the listener cannot be null because the camera cannot be null, so if it is null, the sound cannot really be a level sound. Yes, you are correct about that assumption, such a sound should not really play when there is no listener.
-
- Posts: 83
- Joined: Thu Jan 25, 2018 1:37 pm
Re: [bfbf7ff9c]Crash on start-up
Crashes for me too.
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49073
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: [bfbf7ff9c]Crash on start-up
That post was rather pointless. We have already confirmed that this is a compiler bug in Visual C++ 2015, but do not know a workaround yet how to escape it.
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49073
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: [bfbf7ff9c]Crash on start-up
In that case I vote for disabling optimization for VS2015. That can later easily be removed when we really want to move on to a more modern C++ standard.