you haven't added a repeat for when its doing next player i think so it just does that then leaves the next player logged inuntil it logs out you need to add this instead of your submainloop although i haven't tested its jus what springs to mind the script looks good actually
SCAR Code:
Procedure SubMainLoop;
begin
repeat
repeat
begin
if not(LoggedIn) then exit;
begin
wait(300+random(25));
OpenUpBank;
WalkPathTo('furnace');
SmeltItUp;
WalkPathTo('bank');
Proggy;
Sleepy;
end;
end;
until (barsleft<=0) or not(loggedin);
Writeln('Player Done');
//Logout; isnt needed
NextPlayer(False);
until AllPlayersInactive;
end;
oh but in your smelting people dont trust coord clicking especially as there are no failsafes id make colorclicking as the main attempt and then coordclicking eg
SCAR Code:
'silver': begin
Wait(1000+random(200));
if findcolortolerance(x, y, 00000, xs, ys, xe, ye, 20) then
begin
mouse(x, y, 5, 5, false)
end else
begin
Mouse(200,435,2,2,False);
end;
Wait(200+random(20));
ChooseOption('X');
Wait(1000+random(200));
TypeSend(IntToStr(28+random(500)));//<whats this btw? itd just send a number
end;
Edit:i didn't test either of them its just what i think :P