A minor fix that you could throw in:
I noticed that whenI recieved a piece of the Varrock Museum Statue when FMing it thought it was a log and I had to manually bank it to fix it.
Its not much just saying. I will be posting a proggy soon =]
Edit:
In lines 1167 - 1171 is where it calls to walk to the Banker once your invetory is empty or a specified time has gone by:
Code:
until (((GetSystemTime - T) > (5 * 60 * 1000)) or ((InvCount - Length(Exc)) < 2));
WriteLn('Back to bank...');
if (FindBankersMM(P)) then
Walk(P.X, P.Y, 0, 0)
else
Version that works w.o that minor error when clicking to the bank:
Code:
until (((GetSystemTime - T) > (5 * 60 * 1000)) or ((InvCount - Length(Exc)) < 2));
WriteLn('Back to bank...');
wait(1000);
if (FindBankersMM(P)) then
Walk(P.X, P.Y, 0, 0)
else
Yes it was pathetic amount to add to the script but it helps. The script will sometimes click the map too quickly to move to the bank when a fire was just being made that the flag will disappear leaving the player to glitch out.
Maybe adding a delay would help?
Also, when it says "You can't light a fire here." The script only moves 2 tiles up or down randomly. A problem I saw was that it would either get stuck in the middle of the GE trying to light a fire by the bankers and make it look a little suspicious, OR it will move Up and then Down repeatedly over perviously detected fires so youre running back and forth trying to light.
Maybe instead have it go in the up direction if the player is on the upper half of the GE and down in the lower half. Just a suggestion