Very good try.
SCAR Code:
procedure login;
begin
Mouse(457, 290, 0, 0, true);
wait(500 + random(100));
TypeSend(uname);
wait(300);
TypeSend(upass);
wait(500 + random(100));
Mouse(303, 322, 0, 0, true);
wait(5000 + random(500));
Mouse(388, 339, 0, 0, true);
end;
This is to login your player. Instead, just make a procedure called DeclarePlayers, stick it in your mainloop, then after it write LoginPlayer. It will automatically log you in.
SCAR Code:
procedure ppocket;
begin
If(Findcolor(x, y, victcol, 1, 1, 516, 338)) then
picknum:= picknum + 1;
Mouse(x, y, 0, 0, false);
wait(200 + random(50));
Mouse(x - 80, y + 40, 0, 0, true);
end;
As for this, you use coordinates to click on the option. However, you can just use ClickOption.
ClickOption('ickpocket', 1) means that the script will click on the option pickpocket, and it will click it in the mainscreen, signified by the 1.
SCAR Code:
procedure setup;
begin
Mouse(685, 483, 0, 0, true);
wait(200 + random(50));
Mouse(647, 430, 0, 0, true);
end;
As for this, I'm not too sure what it's for as I haven't tried it on Runescape. However, I do believe it might have something with turning run to true. Instead, you can just do: SetRun(True).
Otherwise, instead of using coordinates to click on the gametabs, then just use GameTab(1), GameTab(2) and so forth.
All in all, it's an excellent try. I hope to see you advancing in SCAR, one day to become a superior scripter