hmm not sure I would go that way.
Me personally i rarely use mount in my usual WP as most of them are farming items or things like malatina's. So checking for possible mounting in the main loop probably wouldn't benefit me much. I am only 1 person though.
So basically if done the way you suguest you would use
in the waypoint and then it will mount and also remount if for some reason it gets dismounted.
At the moment you would just use
but doing this won't remount if dismounted for some reason.
So main difference is that it would remount.
So maybe look at reasons that would get you dismounted.
1. Cast a skill or use a potion
2. Get attacked but you get hit enough to cause the dismount, a single attack usually won't dismount you.
3. You gather an item, harvest or just quest item either will dismount you.
4. Hmm I can't think of another, there must be more though.
Discussions
1.
is taken care of at the moment by the profile option i just added.
2.
I think you would want to fight back in most cases if knocked off, it takes 2 seconds to remount? if a mob can knock you off your mount that 2 seconds might be enough to kill you maybe?
3.
In this case I think you would deffinately want to remount but in order to harvest or gather an item you also need to have code to tell it to do so, so not a big stretch to add player:mount() after the code.
I think what you propose is deffinately neater and may be more useful if other things are added like harvesting on the fly without having the harvest code at coords.
Can also make harvest/quest WP easier as wouldn't need to constantly use player:mount(). Generally though quests requiring multiple items gathered are usually in same area so mounting between each gather wouldn't help.
I am kind of in the middle on this 1, could go either way and still be happy lol