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
#6748
Status
Fixed
Severity
None
Started
Hercules Elf Bot
Oct 2, 2012 22:31
Last Post
Hercules Elf Bot
Oct 3, 2012 1:27
Confirmation
N/A
Hercules Elf Bot - Oct 2, 2012 22:31
Originally posted by [b]Poseid0n[/b] Some infinite loop in dialog.
The test with the 50 novus don't work.
Hercules Elf Bot - Oct 2, 2012 23:46
Originally posted by [b]Euphy[/b] Fixed the loop in [rev=16810]. The Novus test appears to work correctly (though correct me if I'm wrong, I only tested that part of the script).
This post has been edited by
Euphy
on Oct 2, 2012 23:47
Hercules Elf Bot - Oct 3, 2012 1:13
Originally posted by [b]Vianna[/b] Isn't a
[code]do { ... } while (!.@loop);[/code]
more appropriate for this case?
Hercules Elf Bot - Oct 3, 2012 1:27
Originally posted by [b]Euphy[/b] I think [b]do{}while()[/b] looks ugly in scripts, but you're right, I should've used [b]while(!.@loop)[/b]. It's minor though, I'll correct it on my next commit. Thanks!
Edit: On second thought, it really doesn't matter, it looks better the way it is. xD
This post has been edited by
Euphy
on Oct 3, 2012 1:49