cli: fix relative volume adjustment #12

danio · 2024-05-26T00:10:07+02:00

danio commented

2024-05-26 00:10:07 +02:00

oops

danio added 1 commit 2024-05-26 00:10:08 +02:00

cli: fix relative volume adjustment 047d09ffb1

pederbs reviewed 2024-05-26 00:15:06 +02:00

grzegorz_clients/cli.py

						
				@ -170,10 +170,11 @@ def set_volume(

				    if volume.startswith("+") or volume.startswith("-"):

				        current_volume = api.get_volume()

				        new_volume = max(0, min(100, current_volume + int(volume)))

				        new_volume = int(new_volume)

pederbs commented

2024-05-26 00:15:02 +02:00

all inputs to max are int, this line should be a noop?

danio commented

2024-05-26 00:52:02 +02:00

api.get_volume() returns a float, api.set_volume() requires an int

current-volume + int(volume) is a float

could just fix one of them instead of this though

api.get_volume() returns a float, api.set_volume() requires an int current-volume + int(volume) is a float could just fix one of them instead of this though