Lance's Notes
Warning! This is the old Hercules bugtracker archive, and may not reflect the current state of Hercules. The current bugtracker is on GitHub Issues.
Issue information
Issue ID
#896
Status
Working as Intended
Severity
None
Started
Hercules Elf Bot
Jan 30, 2008 14:29
Last Post
Hercules Elf Bot
Jan 30, 2008 14:29
Confirmation
Hercules Elf Bot - Jan 30, 2008 14:29
Originally posted by [b]theultramage[/b]
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=896
posting it so I don't forget about it ('still got his updated sentry script queued after this)
EDIT: attached the related files
This post has been edited by theultramage: Jan 31 2008, 06:12 AM
http://www.eathena.ws/board/index.php?autocom=bugtracker&showbug=896
QUOTE
< [Lance]> ultramage: identified the problem i told you earlier. Somebody forgot to document you have to call s
cript_removetop(st, -1, 0); after a get_val2
< [Lance]> ultramage: now, another problem - mob_ai_sub_hard_activesearch is called after tbl is defined.. means the target is most likely unlocked first after locking.. aint that a problem?
< [Lance]> ultramage: for the mob command script to work (unitattack), if(!tbl) must be changed to if(!(tbl=map_id2bl(md->target_id))) (~ line 1271)
cript_removetop(st, -1, 0); after a get_val2
< [Lance]> ultramage: now, another problem - mob_ai_sub_hard_activesearch is called after tbl is defined.. means the target is most likely unlocked first after locking.. aint that a problem?
< [Lance]> ultramage: for the mob command script to work (unitattack), if(!tbl) must be changed to if(!(tbl=map_id2bl(md->target_id))) (~ line 1271)
QUOTE
<@ultramage> [Lance]: I suggest correcting the function instead, because id2bl(md->target_id) is already being
done 2 pages above your fix
10:04 <@ultramage> if you need to execute the same code twice, there must be a problem somewhere
done 2 pages above your fix
10:04 <@ultramage> if you need to execute the same code twice, there must be a problem somewhere
posting it so I don't forget about it ('still got his updated sentry script queued after this)
EDIT: attached the related files
This post has been edited by theultramage: Jan 31 2008, 06:12 AM
Attached File(s)
