Personally, I prefer DTM's to BMP's. Your choice.
SCAR Code:
if FindBitmapIn(log, x, y, 546, 203, 752, 465) then
You have SRL in your script, so you can just do this:
SCAR Code:
if FindBitmapIn(log, x, y, MIX1, MIY1, MIX2, MIY2) then
Coordinates are not suggested, because they can change for different people. What are you trying to click using this line?
SCAR Code:
Mouse(627, 85, 20, 20, true);
Instead of this:
SCAR Code:
FindObj(x, y, 'hop down Tree', TreeColor, 30)then
I suggest this:
SCAR Code:
FindObjCustom(x, y, ['hop', 'own', 'ree'], [TreeColor1, TreeColor2, TreeColor3], 10)then
So that it can use three colors(or more!) instead of one.
I just did a quick look over, there's probably more that can be improved.
~Sandstorm