May I suggest a format here?
For PowerSkill.mac, it should be run with one parameter, the name of the skill: /macro powerskill brewing, etc.
Use an INI file for the recipes.
Don't worry about zone checks in the macro, it is up to the user to check the ini file to make sure he is starting in the right zone, in front of the right merchant.
So, you check the files, make sure you are in SH in front of the gnome in the tent, you type /macro powerskill brewing. The macro would then perform like HeroBrew.mac.
For fletching just use PowerArrows. It's a special case, since you don't have to move at all, just sit in front of an arrow merchant. Or turn powerarrows into an inc file and have PowerSkill call it.
There should be a skill level check. The macro should not take you above 200 in any skill without being specifically told to do so. A parameter that accepts either "max" or a target skill level would be good.
Also, don't use 0|1 as your toggles. It gets confusing! :) If you want to destroy everything produced you should specify "destroy"... so brewing would be "/macro powerskill brewing destroy", etc.
Add all that together and what you get is a macro that is called something like this:
Code: Select all
/macro powerskill brewing 200 destroy
That macro call would essentially run HeroBrew.mac till the user skill level reached 200, destroying all returns from the combine.