remove unused return values
This commit is contained in:
parent
6c5a8f484e
commit
3247352fd0
|
@ -75,15 +75,13 @@ class OscCommentMacroListener(CommentMacroListener):
|
||||||
logger.info(beautify_lisp_string(lisp_tree_str))
|
logger.info(beautify_lisp_string(lisp_tree_str))
|
||||||
|
|
||||||
# execute macro from name
|
# execute macro from name
|
||||||
ret = -1
|
|
||||||
name = ctx.children[0].getText()
|
name = ctx.children[0].getText()
|
||||||
try:
|
try:
|
||||||
ret = self.osc.command[name](self, ctx)
|
self.osc.command[name](self, ctx)
|
||||||
except KeyError:
|
except KeyError:
|
||||||
print(name + " macro is not compatable with OSC.")
|
print(name + " macro is not compatable with OSC.")
|
||||||
return -1
|
return -1
|
||||||
logger.debug("Exiting Macro")
|
logger.debug("Exiting Macro")
|
||||||
return ret
|
|
||||||
|
|
||||||
def enterMaster(self, ctx: CommentMacroParser.MasterContext):
|
def enterMaster(self, ctx: CommentMacroParser.MasterContext):
|
||||||
ctx.targets = []
|
ctx.targets = []
|
||||||
|
|
|
@ -37,16 +37,15 @@ class HogDevice():
|
||||||
if (device.type.getText() != 'h'):
|
if (device.type.getText() != 'h'):
|
||||||
logger.warn("Only Hog type devices are curently supported.")
|
logger.warn("Only Hog type devices are curently supported.")
|
||||||
logger.error("ERROR: macro discarded!")
|
logger.error("ERROR: macro discarded!")
|
||||||
return -1
|
return
|
||||||
else:
|
else:
|
||||||
try:
|
try:
|
||||||
osc = self.servers[device.number.value]
|
osc = self.servers[device.number.value]
|
||||||
except KeyError:
|
except KeyError:
|
||||||
logger.warn("Net# "+str(device.number.value)+" not found.")
|
logger.warn("Net# "+str(device.number.value)+" not found.")
|
||||||
logger.error("ERROR: macro discarded!")
|
logger.error("ERROR: macro discarded!")
|
||||||
return -1
|
return
|
||||||
osc.send_message(path, arg)
|
osc.send_message(path, arg)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _master_go(self, ctx):
|
def _master_go(self, ctx):
|
||||||
if ctx.number is not None:
|
if ctx.number is not None:
|
||||||
|
@ -55,7 +54,7 @@ class HogDevice():
|
||||||
if (len(ctx.master.targets) == 0):
|
if (len(ctx.master.targets) == 0):
|
||||||
logger.info("Main GO")
|
logger.info("Main GO")
|
||||||
self.osc.button_press(ctx.device, "/hog/hardware/maingo")
|
self.osc.button_press(ctx.device, "/hog/hardware/maingo")
|
||||||
return 1
|
return
|
||||||
else:
|
else:
|
||||||
for i in ctx.master.targets:
|
for i in ctx.master.targets:
|
||||||
if isinstance(i, int) is not True:
|
if isinstance(i, int) is not True:
|
||||||
|
@ -68,13 +67,12 @@ class HogDevice():
|
||||||
master = str(i)
|
master = str(i)
|
||||||
logger.info("GO on master " + master)
|
logger.info("GO on master " + master)
|
||||||
self.osc.button_press(ctx.device, "/hog/hardware/go/" + master)
|
self.osc.button_press(ctx.device, "/hog/hardware/go/" + master)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _master_halt(self, ctx):
|
def _master_halt(self, ctx):
|
||||||
if (len(ctx.master.targets) == 0):
|
if (len(ctx.master.targets) == 0):
|
||||||
logger.info("Main HALT")
|
logger.info("Main HALT")
|
||||||
self.osc.button_press(ctx.device, "/hog/hardware/mainhalt")
|
self.osc.button_press(ctx.device, "/hog/hardware/mainhalt")
|
||||||
return 1
|
return
|
||||||
else:
|
else:
|
||||||
for i in ctx.master.targets:
|
for i in ctx.master.targets:
|
||||||
if isinstance(i, int) is not True:
|
if isinstance(i, int) is not True:
|
||||||
|
@ -88,20 +86,19 @@ class HogDevice():
|
||||||
logger.info("HALT on master " + master)
|
logger.info("HALT on master " + master)
|
||||||
self.osc.button_press(ctx.device,
|
self.osc.button_press(ctx.device,
|
||||||
"/hog/hardware/pause/" + master)
|
"/hog/hardware/pause/" + master)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _master_fade(self, ctx):
|
def _master_fade(self, ctx):
|
||||||
if (ctx.number) is None:
|
if (ctx.number) is None:
|
||||||
logger.error("ERROR: Missing required argument for LEVEL")
|
logger.error("ERROR: Missing required argument for LEVEL")
|
||||||
return -1
|
return
|
||||||
else:
|
else:
|
||||||
level = ctx.number.value
|
level = ctx.number.value
|
||||||
if (level < 0 or level > 100):
|
if (level < 0 or level > 100):
|
||||||
logger.error("Level must be between 0 and 100.")
|
logger.error("Level must be between 0 and 100.")
|
||||||
return -1
|
return
|
||||||
if (len(ctx.master.targets) == 0):
|
if (len(ctx.master.targets) == 0):
|
||||||
logger.error("MASTER FADE doesn't support * current master.")
|
logger.error("MASTER FADE doesn't support * current master.")
|
||||||
return -1
|
return
|
||||||
else:
|
else:
|
||||||
for i in ctx.master.targets:
|
for i in ctx.master.targets:
|
||||||
if isinstance(i, int) is not True:
|
if isinstance(i, int) is not True:
|
||||||
|
@ -117,26 +114,23 @@ class HogDevice():
|
||||||
self.osc.send_message(ctx.device,
|
self.osc.send_message(ctx.device,
|
||||||
"/hog/hardware/fader/" + master,
|
"/hog/hardware/fader/" + master,
|
||||||
level)
|
level)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _master_fade_grand(self, ctx):
|
def _master_fade_grand(self, ctx):
|
||||||
level = ctx.number.value
|
level = ctx.number.value
|
||||||
if (level < 0 or level > 100):
|
if (level < 0 or level > 100):
|
||||||
logger.error("Level must be between 0 and 100.")
|
logger.error("Level must be between 0 and 100.")
|
||||||
return -1
|
return
|
||||||
logger.info("Fading Grand Master to " + str(level) + "%")
|
logger.info("Fading Grand Master to " + str(level) + "%")
|
||||||
level *= 255 / 100 # percent in Macro, 0>255 in OSC
|
level *= 255 / 100 # percent in Macro, 0>255 in OSC
|
||||||
self.osc.send_message(ctx.device, "/hog/hardware/fader/0", level)
|
self.osc.send_message(ctx.device, "/hog/hardware/fader/0", level)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _master_choose(self, ctx):
|
def _master_choose(self, ctx):
|
||||||
if (ctx.number.value < 0):
|
if (ctx.number.value < 0):
|
||||||
logger.error("Master must be greater than 0.")
|
logger.error("Master must be greater than 0.")
|
||||||
return -1
|
return
|
||||||
master = str(ctx.number.value)
|
master = str(ctx.number.value)
|
||||||
logger.info("Choose Master " + master)
|
logger.info("Choose Master " + master)
|
||||||
self.osc.button_press(ctx.device, "/hog/hardware/choose/" + master)
|
self.osc.button_press(ctx.device, "/hog/hardware/choose/" + master)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _list_go(self, ctx):
|
def _list_go(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
|
@ -145,37 +139,31 @@ class HogDevice():
|
||||||
list += "." + str(ctx.number.value)
|
list += "." + str(ctx.number.value)
|
||||||
logger.info("Go on List " + list)
|
logger.info("Go on List " + list)
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/go/0", list)
|
self.osc.send_message(ctx.device, "/hog/playback/go/0", list)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _list_halt(self, ctx):
|
def _list_halt(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
logger.info("Halting List " + str(i))
|
logger.info("Halting List " + str(i))
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/halt/0", i)
|
self.osc.send_message(ctx.device, "/hog/playback/halt/0", i)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _list_release(self, ctx):
|
def _list_release(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
logger.info("Releasing List " + str(i))
|
logger.info("Releasing List " + str(i))
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/release/0", i)
|
self.osc.send_message(ctx.device, "/hog/playback/release/0", i)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _scene_go(self, ctx):
|
def _scene_go(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
logger.info("Go on Scene " + str(i))
|
logger.info("Go on Scene " + str(i))
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/go/1", i)
|
self.osc.send_message(ctx.device, "/hog/playback/go/1", i)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _scene_halt(self, ctx):
|
def _scene_halt(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
logger.info("Halt Scene " + str(i))
|
logger.info("Halt Scene " + str(i))
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/halt/1", i)
|
self.osc.send_message(ctx.device, "/hog/playback/halt/1", i)
|
||||||
return 1
|
|
||||||
|
|
||||||
def _scene_release(self, ctx):
|
def _scene_release(self, ctx):
|
||||||
for i in ctx.targets:
|
for i in ctx.targets:
|
||||||
logger.info("Release Scene " + str(i))
|
logger.info("Release Scene " + str(i))
|
||||||
self.osc.send_message(ctx.device, "/hog/playback/release/1", i)
|
self.osc.send_message(ctx.device, "/hog/playback/release/1", i)
|
||||||
return 1
|
|
||||||
|
|
||||||
command = {"GM": _master_go,
|
command = {"GM": _master_go,
|
||||||
"HM": _master_halt,
|
"HM": _master_halt,
|
||||||
|
|
|
@ -58,6 +58,11 @@ def comment(text):
|
||||||
walker.walk(oscMacro, tree)
|
walker.walk(oscMacro, tree)
|
||||||
|
|
||||||
|
|
||||||
|
def orderly_exit():
|
||||||
|
logger.info('Goodbye.')
|
||||||
|
sys.exit(0)
|
||||||
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
signal.signal(signal.SIGINT, orderly_exit)
|
signal.signal(signal.SIGINT, orderly_exit)
|
||||||
if len(sys.argv) > 1:
|
if len(sys.argv) > 1:
|
||||||
|
@ -67,7 +72,6 @@ if __name__ == '__main__':
|
||||||
while True:
|
while True:
|
||||||
text = input("comment# ")
|
text = input("comment# ")
|
||||||
if text == 'exit':
|
if text == 'exit':
|
||||||
logger.info('Goodbye.')
|
orderly_exit()
|
||||||
sys.exit(0)
|
|
||||||
else:
|
else:
|
||||||
comment(text)
|
comment(text)
|
||||||
|
|
Loading…
Reference in New Issue