Issues with Neshke's Extendable Spawning System (NESS) v8.1.3 in NWN:EE

Nic_Mercy

Has anyone encountered issues using this in NWN:EE? I am experiencing some issues with it such as zones with waypoints using SA_ crashing the client upon entering, and spawns continuously spawning without limit when using SD05M01 but not if I use not variance (such as SD05). I have also seen the number spawned when using a variable number SN03M01 being more than the max, but seemed to respect non-variable tags like SN03. Have I done something wrong or is there some sort of incompatibility with NWN:EE and NESS?

To note, for anyone who tries to test. Make sure you do a build of the module before testing. I imported the NESS erf into a fresh module just to test and I crash upon entering if I use SA on the waypoints. If anyone has a solution I'd love to hear it cause I really like how the system works and would prefer not to switch to something different if possible.

ForSerious

I downloaded the 8.1.4 version that no one else seems to have downloaded.
They declared an integer outside of the scope of where it gets used. I guess this lets it pass compilation, but when it tries to increment it, nothing happens. This makes a memory leak, in that: it spawns until your RAM is full.

in spawn_main line 239, copy and delete that whole line. "int jCount;"
On line 1407 press enter and paste that line.
Should look something like:

if (nSpawnAllAtOnce == FALSE)
                    {
                        // Spawn another Creature
                        DoSpawn(oSpawn, nTimeNow);
                    }
                    else
this is line 1407   {
                    int jCount; // This is the added line
                        if (nSpawnNumberAtOnce > 0)
                        {
                            if (nSpawnNumberAtOnceMin == 0 || nEmptyChildSlots >= nSpawnNumberAtOnceMin)
                            {
                                // Spawn Sets of Creatures
                                for (jCount = 1; ((jCount <= nEmptyChildSlots) && (jCount <= nSpawnNumberAtOnce)); jCount++)
                                {
                                    DelayCommand(0.0, DoSpawn(oSpawn, nTimeNow));
                                }
                            }
                        }
                        else
                        {
                            // Spawn All Creatures
                            for(jCount = 1; jCount <= nEmptyChildSlots; jCount++)
                            {
                                DoSpawn(oSpawn, nTimeNow);
                            }
                        }
                    }

Compile all the scripts, and try it again.

ForSerious

That was specifically for SA crashing. I didn't look at the other issues you mentioned, but they're probably from the same type of incorrect code placement. I can look into them later.

meaglyn

That int jCount declaration should have been fine where it was. It does not need to be in that else clause. I believe OP's issues to be due to the broken compiler in the latest release. Nothing wrong with that code.

ForSerious

It was declared outside of the function it was used in. Since there's no main function, that's probably how it slipped past the compiler—and maybe that was possible to do in older versions, but it is still a really bad way to code.
Obviously there is something wrong with that code since moving it to an expected scope fixed the issue.

ForSerious

Also, broken compiler? I've heard no mention of this before nor seen anything on my 100+ scripts.