Problem with /who

A forum for reporting bugs NOT related to custom plugins.

Moderator: MacroQuest Developers

s16z
a ghoul
a ghoul
Posts: 97
Joined: Thu Apr 01, 2004 12:03 pm

Problem with /who

Post by s16z » Sat May 15, 2004 10:28 am

Using May 14 2004 19:23:36 source, V6 SP6.

When I type /who, I get

Code: Select all

List of matching spawns
-----------------------------------
(0-100) pc was not found.
Using /who with an option just gives the same results, but replace pc with whatever option.

Oid
a snow griffon
a snow griffon
Posts: 416
Joined: Thu Oct 17, 2002 3:26 am
Contact:

Post by Oid » Sat May 15, 2004 11:01 am

Hmm... already a topic about this, search is your friend
Smokey the Lax says only you can prevent reproduction.

Amadeus
The Maestro
The Maestro
Posts: 2036
Joined: Sat Jun 29, 2002 3:51 pm

Post by Amadeus » Sat May 15, 2004 1:34 pm

Just out of curiosity, what did you search for to get that thread you mentioned? I searched for 'who' and '/who' and got nothing (which is strange since I would expect SOMETHING to come up).

Either that or search is too complicated for me ..lol (I usually read everythin and never use search...but I don't see this thread you mentioned in the bug forum).

Amadeus
The Maestro
The Maestro
Posts: 2036
Joined: Sat Jun 29, 2002 3:51 pm

Post by Amadeus » Sat May 15, 2004 1:37 pm

Well, I guess I found it ...IN THE WRONG DAMN FORUM.

s16z was not the one that deserved the flame (albeit it was a rather tame flame).

Either way, this discussion will now move to this thread ...where it belongs.

Lax
We're not worthy!
We're not worthy!
Posts: 3524
Joined: Thu Oct 17, 2002 1:01 pm
Location: ISBoxer
Contact:

Post by Lax » Sat May 15, 2004 2:27 pm

ok, yes other people have this bug, and no I have no idea why (it does seem to be only with VS6 and even then, DKAA tested with VS6 and said it works 100%). The bug isn't actually with /who, according to some tests I had someone do last night. /who and the standard spawn search (aka part of /who that isnt /who, and part of /target, ${Spawn[search]}, ${NearestSpawn[n,search]}, /maphide, /mapshow, /mapfilter custom, etc) both changed.

Okay, that in mind. Does /target work correctly? Does /echo ${Spawn[npc name]} work correctly? If you answered NO to both of these questions, the problem isnt even /who

I'm going to do some testing since I dont really expect anyone reporting this problem to be able to handle it since it doesnt crash... but... if I dont find it you'll have to figure out how to look at debug spew
Lax Lacks
Master of MQ2 Disaster
Purveyor of premium, EULA-safe MMORPG Multiboxing Software
* Multiboxing with ISBoxer: Quick Start Video
* EQPlayNice, WinEQ 2.0

Jerle69
a hill giant
a hill giant
Posts: 263
Joined: Wed Apr 28, 2004 3:26 pm

Post by Jerle69 » Sat May 15, 2004 2:42 pm

I use VS6SP6; I'll see if it works for me, if not I'll tell you what's up. If it does work, I won't report back that there's a problem :P
--Jerle

Lax
We're not worthy!
We're not worthy!
Posts: 3524
Joined: Thu Oct 17, 2002 1:01 pm
Location: ISBoxer
Contact:

Post by Lax » Sat May 15, 2004 3:27 pm

I'm testing with VS6, and i dont think i even have SP6 installed on it.. I took it right from the zip, in a clean directory, no custom plugins, no changes to ini files, nothing.. and /who works perfectly.
Lax Lacks
Master of MQ2 Disaster
Purveyor of premium, EULA-safe MMORPG Multiboxing Software
* Multiboxing with ISBoxer: Quick Start Video
* EQPlayNice, WinEQ 2.0

JGC84
a ghoul
a ghoul
Posts: 116
Joined: Sat Nov 29, 2003 2:51 am

Post by JGC84 » Sat May 15, 2004 3:34 pm

i'm having the same who issue, compiling w/ VS 6, SP 5 (maybe), only thing i've changed is the hud ini file... so unless thats effecting it.. ok: just unloaded the hud plugin, didnt change anything
ACK! I think I blew it up....

I dont think your allowed to talk about this outside of a "free speech zone" anymore.

Oid
a snow griffon
a snow griffon
Posts: 416
Joined: Thu Oct 17, 2002 3:26 am
Contact:

Post by Oid » Sat May 15, 2004 4:06 pm

Bah! that wasn't even a flame, a polite suggestion! I didnt even say fuck!
Smokey the Lax says only you can prevent reproduction.

Amadeus
The Maestro
The Maestro
Posts: 2036
Joined: Sat Jun 29, 2002 3:51 pm

Post by Amadeus » Sat May 15, 2004 4:44 pm

Someone posted that it works when debugspew is on, and doesn't when it's off.

Right now '/who <class>' isn't working for me (ie, 'who druid'); however, I'm raiding and can't debug right now.

Will fix tonight though if Lax hasn't got it by then since it's not working for me and can debug it.

Amadeus
The Maestro
The Maestro
Posts: 2036
Joined: Sat Jun 29, 2002 3:51 pm

Post by Amadeus » Sat May 15, 2004 4:48 pm

ok...nevermind, obviously this was changed when I wasn't looking. I guess you have to do 'who class druid' now.

I might put an if statement in there sometime to catch 3rd arguments that are class names later.

Lax
We're not worthy!
We're not worthy!
Posts: 3524
Joined: Thu Oct 17, 2002 1:01 pm
Location: ISBoxer
Contact:

Post by Lax » Sat May 15, 2004 5:40 pm

I'll modify the spawn search parser to take player classes and player races without the "class" or "race" keywords. Post on another thread about that if you need to, leave this one for people who do /who and have it not display spawns ;)

Someone having this problem PLEASE do a vanilla install of MQ and test it. Make a completely new directory, unzip and compile without making ANY changes (dont change the ini, dont add custom plugins, dont add custom code, NO changes). Then go into the game, and see if it works. If it DOES work, then the issue has to do with customizations (not necessarily your fault). If it DOES NOT work, then the issue has to do with the compiler. PLEASE do this test.

Dont post another post on this thread unless it is the outcome of this test.
Lax Lacks
Master of MQ2 Disaster
Purveyor of premium, EULA-safe MMORPG Multiboxing Software
* Multiboxing with ISBoxer: Quick Start Video
* EQPlayNice, WinEQ 2.0

SwiftyMUSE
Developer
Developer
Posts: 1205
Joined: Tue Sep 23, 2003 10:52 pm

Post by SwiftyMUSE » Sat May 15, 2004 6:25 pm

Ok, here is some results from testing...

There is a spawn list.
If the spawn fails to pass the spawntype it exits the search.
If it passes the spawntype, it failes on:

Code: Select all

	if (pSearchSpawn->ZRadius<10000.0f &&  (pSpawn->Z > pChar->Z + pSearchSpawn->ZRadius) ||(pSpawn->Z < pChar->Z - pSearchSpawn->ZRadius))
		return FALSE;
I am checking the specific values next and will update the post with more information.

SwiftyMUSE
Developer
Developer
Posts: 1205
Joined: Tue Sep 23, 2003 10:52 pm

Fix for /who issue

Post by SwiftyMUSE » Sat May 15, 2004 7:02 pm

in MQ2Commands.cpp change:

Code: Select all

 if (pSearchSpawn->ZRadius<10000.0f &&  (pSpawn->Z > pChar->Z + pSearchSpawn->ZRadius) ||(pSpawn->Z < pChar->Z - pSearchSpawn->ZRadius)) 
      return FALSE; 
to:

Code: Select all

 if (pSearchSpawn->ZRadius<10000.0f &&  ((pSpawn->Z > pChar->Z + pSearchSpawn->ZRadius) ||(pSpawn->Z < pChar->Z - pSearchSpawn->ZRadius))) 
      return FALSE; 
in the BOOL SpawnMatchesSearch(PSEARCHSPAWN pSearchSpawn, PSPAWNINFO pChar, PSPAWNINFO pSpawn) function.

Lax, can you get the fix into dev cvs?

Thanks,

Lax
We're not worthy!
We're not worthy!
Posts: 3524
Joined: Thu Oct 17, 2002 1:01 pm
Location: ISBoxer
Contact:

Post by Lax » Sat May 15, 2004 7:59 pm

Fixed for next zip, thank you
Lax Lacks
Master of MQ2 Disaster
Purveyor of premium, EULA-safe MMORPG Multiboxing Software
* Multiboxing with ISBoxer: Quick Start Video
* EQPlayNice, WinEQ 2.0