The current one seems to have some unnecessary code, though I could just be misunderstanding it.
Current function:
SCAR Code:
{*******************************************************************************
function BankScreen: Boolean;
by: SRL Dev Team
Description: Finds Bankscreen. Returns true if Found.
*******************************************************************************}
function BankScreen: Boolean;
var
c: Integer;
begin
if (CountColor(0, MSX1, MSY1, MSX2, MSY2) > 113000) then
begin
MouseBox(26, 138, 489, 157, 1);
c := GetTimeRunning + 45000;
while (not BankScreen) and (GetTimeRunning < c) do
Wait(50);
Wait(RandomRange(100, 150));
end;
Result := FindTextTPA(4106994, 20, 20, 22, 400, 45, 'Bank', upchars, Nothing);
end;
My version:
SCAR Code:
function BankScreen: Boolean;
var
i: Integer;
begin
if not LoggedIn then Exit;
for i := 0 to 1 do
begin
Result := CountColor(4106994, 194, 25, 322, 44) = 423;
if Result then
Exit
else
MouseBox(35, 67, 449, 271, 3);
end;
end;
They both work at the same speed so it really doesn't matter, just posting for opinions..