Severed bunny head
Register | User Profile | Member List | F.A.Q | Privacy Policy | New Blog | Search Forums | Forums Home
Doomworld Forums : Powered by vBulletin version 2.2.5 Doomworld Forums > Special Interest > Eternity > S_NULL no longer supported?
 
Author
All times are GMT. The time now is 17:00. Post New Thread    Post A Reply
printz
CRAZY DUMB ZEALOT


Posts: 6845
Registered: 06-06
I excitedly downloaded r1289 to see what has been newly done to stuff like the portal code, but I can no longer run my mod because at startup it says: 
code:
E_ThingFrame: thing 'BruiserDemon': invalid raisestate 's_null'
. Understandably, I created that monster who has a nullified raise state (no raise). Is it a bug that dummy sprite name "S_NULL" is not accepted?
Old Post 08-01-10 21:37 #
printz is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Gez
Why don't I have a custom title by now?!


Posts: 7043
Registered: 07-07
printz said:
I excitedly downloaded r1289 to see what has been newly done to stuff like the portal code

Sorry to disappoint you, but the portal code changes are at the moment still in the portal-clip branch. Which means they are not in the trunk. Which means they are not in the SVN builds hosted by DRD Team.
Old Post 08-01-10 21:45 #
Gez is online now Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Quasar
Moderator


Posts: 4615
Registered: 08-00
printz said:
I excitedly downloaded r1289 to see what has been newly done to stuff like the portal code, but I can no longer run my mod because at startup it says: 
code:
E_ThingFrame: thing 'BruiserDemon': invalid raisestate 's_null'
. Understandably, I created that monster who has a nullified raise state (no raise). Is it a bug that dummy sprite name "S_NULL" is not accepted?

I need to see your EDF to determine what is actually going on here.
Old Post 08-02-10 19:10 #
Quasar is offline Profile || Blog || PM || Email || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
Mordeth
Administrator


Posts: 1883
Registered: 05-00
Instead of defining the raise-state, just leave the raise-state out of your new definition of BruiserDemon..? Not providing a raise-state will result in your monster not being able to be raised. In your case it can still be raised, but attempting to do so will result in it trying to revive a monster while the sprite is being removed.
Old Post 08-03-10 00:16 #
Mordeth is offline Profile || Blog || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
Quasar
Moderator


Posts: 4615
Registered: 08-00
Mordeth said:
Instead of defining the raise-state, just leave the raise-state out of your new definition of BruiserDemon..? Not providing a raise-state will result in your monster not being able to be raised. In your case it can still be raised, but attempting to do so will result in it trying to revive a monster while the sprite is being removed.

Regardless, S_NULL should not cause an error. It's the first state defined and should be known by name. Please note that S_NULL is not a *sprite* name however, it's a frame mnemonic.
Old Post 08-03-10 01:10 #
Quasar is offline Profile || Blog || PM || Email || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
printz
CRAZY DUMB ZEALOT


Posts: 6845
Registered: 06-06
In my case, the BruiserDemon inherits properties from another custom monster within the wad, the WarlordOfHell, who can be resurrected. As I remember (another computer), this S_NULL is the first occurence in the custom EDF.

Last edited by printz on 08-03-10 at 08:26
Old Post 08-03-10 08:17 #
printz is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Quasar
Moderator


Posts: 4615
Registered: 08-00
printz said:
In my case, the BruiserDemon inherits properties from another custom monster within the wad, the WarlordOfHell, who can be resurrected. As I remember (another computer), this S_NULL is the first occurence in the custom EDF.

Please pastebin the entire EDF with an expiration of 1 month or longer.

Also, before we go any further please be sure you are doing the obvious:
code:

stdinclude("root.edf")

This is *required* now, because "Default Fallback" processing had to be removed to enable the new single-parsing-pass EDF architecture created in edf-branch to support DECORATE states, unless you intend to redefine everything in root.edf yourself, which for forward compatibility reasons, is *not* recommended.

Last edited by Quasar on 08-03-10 at 14:24
Old Post 08-03-10 14:13 #
Quasar is offline Profile || Blog || PM || Email || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
printz
CRAZY DUMB ZEALOT


Posts: 6845
Registered: 06-06
Relax, I can just send it to you thru PM when I'm back. Understandably I'm NOT going to post the whole EDF here.
Old Post 08-03-10 14:52 #
printz is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Quasar
Moderator


Posts: 4615
Registered: 08-00
printz said:
Relax, I can just send it to you thru PM when I'm back. Understandably I'm NOT going to post the whole EDF here.

That's why I asked you to use pastebin.com :P
Old Post 08-04-10 01:16 #
Quasar is offline Profile || Blog || PM || Email || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
printz
CRAZY DUMB ZEALOT


Posts: 6845
Registered: 06-06
I'm sending it right now.

Actually, I've changed my mind, because now it works, but it breaks an EDF rule: it's case sensitive. I changed this line from 

code:
  raisestate             s_null


to

code:
  raisestate             S_NULL


and it stopped giving the error.

Last edited by printz on 08-06-10 at 16:49
Old Post 08-06-10 16:37 #
printz is offline Profile || Blog || PM || Search || Add Buddy IP || Edit/Delete || Quote
Quasar
Moderator


Posts: 4615
Registered: 08-00
printz said:
I'm sending it right now.

Actually, I've changed my mind, because now it works, but it breaks an EDF rule: it's case sensitive. I changed this line from 

code:
  raisestate             s_null


to

code:
  raisestate             S_NULL


and it stopped giving the error.

Well, you're right, EDF is *not* supposed to be case-sensitive. So this shouldn't be happening >_> Maybe I made a slipup somewhere; I will go look for it right now.

EDIT: Fixed in r1292. I used the wrong ehash_t specialization, which resulted in use of strcmp/D_HashTableKeyCase rather than the appropriate strcasecmp/D_HashTableKey routines. Never caught it during edf-branch testing because all my stock EDFs happen to refer to states using same-case names.

Last edited by Quasar on 08-10-10 at 04:29
Old Post 08-10-10 04:23 #
Quasar is offline Profile || Blog || PM || Email || Homepage || Search || Add Buddy IP || Edit/Delete || Quote
All times are GMT. The time now is 17:00. Post New Thread    Post A Reply
 
Doomworld Forums : Powered by vBulletin version 2.2.5 Doomworld Forums > Special Interest > Eternity > S_NULL no longer supported?

Show Printable Version | Email this Page | Subscribe to this Thread

 

Forum Rules:
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts 		HTML code is OFF
vB code is ON
Smilies are OFF
[IMG] code is ON
 

< Contact Us - Doomworld >

Powered by: vBulletin Version 2.2.5
Copyright ©2000, 2001, Jelsoft Enterprises Limited.

Forums Directory