New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleaned up custom battleground scripts #3025
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sry I should've use single line comment, not request change
|
||
// delay before match begins | ||
sleep 6000; | ||
mapannounce "bat_a01", "The rules are simple. The first team to break the opponent's Emperium wins!", bc_map; | ||
sleep 3000; | ||
for ( .@i = 5; .@i > 0; .@i-- ) { | ||
for ( .@i = 5; .@i > 0; --.@i ) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just want you guys to know why I did this
HerculesWS/Hercules#872
By the way, on an unrelated note, you might want to replace
.@i++
with++.@i
, since it's more efficient (the first one needs to cache the previous value of.@i
to return it, while the second one doesn't)
* Turn the GvG flag off before setting the Battleground flag to avoid stack warning.
Anything left to be done? |
I thought I look through the script over 5 times before I press the open issue button now this is interesting .... delwall should throw an error if the wall has never been set
|
See Hercules/Hercules#2017
Added a warning to script command delwall
This reverts commit 8c0b52f.
Addressed Issue(s): update my battleground script templates #3024
Server Mode: Both
Description of Pull Request:
Cleaned up AnnieRuru's BG scripts.