[Latest]DUMP v2 no longer loads

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.

Post a reply

Smilies
:D :) :( :o :shock: :? 8-) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :geek: :ugeek: :!: :?: :idea: :arrow: :| :mrgreen: :3: :wub: >:( :blergh:
View more smilies

BBCode is OFF
Smilies are ON

Topic review
   

Expand view Topic review: [Latest]DUMP v2 no longer loads

Re: [Latest]DUMP v2 no longer loads

by Major Cooke » Wed Oct 26, 2016 6:46 am

Also two warnings in textures.cpp, unused h and w variables.

Code: Select all

3>C:\Games\doom 2\zdoom\src\textures\texture.cpp(619): warning C4101: 'h': unreferenced local variable
3>C:\Games\doom 2\zdoom\src\textures\texture.cpp(618): warning C4101: 'w': unreferenced local variable

Re: [Latest]DUMP v2 no longer loads

by Graf Zahl » Wed Oct 26, 2016 6:27 am

Yep. Seems I mised the linefeed.
Those messages are only on in developer mode, btw.

Re: [Latest]DUMP v2 no longer loads

by Major Cooke » Wed Oct 26, 2016 6:20 am

Oh shit, I recompiled the wrong branch. My mistake.

However...
Spoiler: Image
I think this could be a bit more orderly.

Re: [Latest]DUMP v2 no longer loads

by _mental_ » Wed Oct 26, 2016 6:02 am

So where did it crash?

Re: [Latest]DUMP v2 no longer loads

by Major Cooke » Wed Oct 26, 2016 6:01 am

It still doesn't load. Instantly exits out, and I updated my code.

I'm on a 64-bit version, using windows 10, set as RelWithDebugInfo.

Re: [Latest]DUMP v2 no longer loads

by Graf Zahl » Wed Oct 26, 2016 3:52 am

Thanks. Although this mod shows another problem, thanks to some bogus texture definitions (Jeez, people, the texture namespaces exist for a reason, not to be ignored at will!)

Re: [Latest]DUMP v2 no longer loads

by _mental_ » Wed Oct 26, 2016 3:17 am

Most likely a call to FMultiPatchTexture::CheckForHacks() must be deferred, i.e. moved to recently added ResolvePatches() function like I did in this PR.

Re: [Latest]DUMP v2 no longer loads

by Major Cooke » Tue Oct 25, 2016 4:21 pm

Major Cooke wrote:Using this commit
Nope. I compiled the very latest where you added the self reference warning.

Re: [Latest]DUMP v2 no longer loads

by Graf Zahl » Tue Oct 25, 2016 4:08 pm

I suspect the same thing you already reported.Does it work with my fix from today?

[Latest]DUMP v2 no longer loads

by Major Cooke » Tue Oct 25, 2016 3:57 pm

http://forum.zdoom.org/viewtopic.php?t=51072

Using this commit

Just trying to boot this up immediately shuts down zdoom.

Top