move allowed_lengths for music macros out

This moves the allowed_lengths setting out of the preprocessor. Special
cases should not be hardcoded in the preprocessor.
This commit is contained in:
Bryan Bishop 2012-11-30 09:54:02 -06:00
parent bdad293da3
commit a1a9fa5560
2 changed files with 5 additions and 9 deletions

View File

@ -2990,8 +2990,6 @@ music_commands_new = {
music_command_enders = [0xEA, 0xEB, 0xEE, 0xFC, 0xFF,]
# special case for 0xFD (if loopchannel.count = 0, break)
def create_music_command_classes(debug=False):
klasses = [GivePoke]
for (byte, cmd) in music_commands_new.items():
@ -3010,6 +3008,10 @@ def create_music_command_classes(debug=False):
klass_name = cmd_name+"Command"
klass = classobj(klass_name, (Command,), params)
globals()[klass_name] = klass
if klass.macro_name == "notetype":
klass.allowed_lengths = [1, 2]
elif klass.macro_name in ["togglenoise", "sfxtogglenoise"]:
klass.allowed_lengths = [0, 1]
klasses.append(klass)
# later an individual klass will be instantiated to handle something
return klasses

View File

@ -490,12 +490,6 @@ def macro_translator(macro, token, line):
else:
allowed_lengths = [allowed_length]
if macro.macro_name == "notetype":
allowed_lengths = [1,2]
elif macro.macro_name == "togglenoise" \
or macro.macro_name == "sfxtogglenoise":
allowed_lengths = [0,1]
assert len(params) in allowed_lengths, \
"mismatched number of parameters on this line: " + \
original_line
@ -589,4 +583,4 @@ def read_line(l):
sys.stdout.write(comment)
for l in sys.stdin:
read_line(l)
read_line(l)