Sorry, the << was a typo, that's now how the code looks. :) However, this
DID help me quite a bit! Thanks! :)
>Two things: that 'break' there will only make it load one mobile. The
>other thing is you want "i < invasion_size" not "i << invasion_size".
>The latter--that you have--is a mathematical operation (even in C++,
>but certain classes [istream/ostream] use operator overloading to
>define new behavior for the operator), and indicates that you should
>left shift 'i' by 'invasion_size' bits. However, the code as you
>presented it would not cause the mobiles to load forever. It would
>only load one mobile, even if you took out the 'break;' statement
>because of the fact that when i = 0, i << invasion_size will also
>equal 0 and thus, "i << invasion_size" is false (0) and the loop ends.
>So, either you've changed the code since it loaded mobiles repeatedly,
>or you have posted the wrong code, or you just didn't identify the
>behavior correctly.
>
>In addition, it's not desirable to do "char_to_room(mob, location);"
>more than once with the same mobile. You probably want code
>equivalent to,
>
> mob = read_mobile(mob_rnum, REAL);
> char_to_room(mob, room_rnum);
>
>Of course, using your variables in place of mine. How you have it,
>you are doing char_to_room() for the same one mobile over and over
>again, which I doubt is what you want to do (that means that the
>character ends up in the list invasion_size amount of times, but it's
>still only one character, and will probably cause a crash).
[ Tadrith Skandrakae, Implementor of Dreamscape ]
[ - telnet://mud.dreamscape.org:1234/ - ]
["I lay my dreams before your feet; ]
[ tread softly, for you tread on my dreams. ]
+------------------------------------------------------------+
| Ensure that you have read the CircleMUD Mailing List FAQ: |
| http://democracy.queensu.ca/~fletcher/Circle/list-faq.html |
+------------------------------------------------------------+
This archive was generated by hypermail 2b30 : 12/15/00 PST