mdonoughe / sbz-switch

Utility for switching Sound Blaster outputs on Windows
Apache License 2.0
43 stars 5 forks source link

Entering negative float values trigger an error. #16

Open Galenmacil opened 3 weeks ago

Galenmacil commented 3 weeks ago

Consider the following command:

SBZ-Switch.exe set -f "Speaker Calibrator" "FR Level" -1.5

It reports:

error: Found argument '-1.5' which wasn't expected, or isn't valid in this context

Changing the entered value to 1.5 instead works fine. The "Speaker Calibrator" levels are decibel offsets which can range from -20.0 to +20.0 in the "Sound Blaster Speaker" setup program. I use a Creative Sound Blaster Z card.

mdonoughe commented 2 weeks ago

Can you try this build? https://github.com/mdonoughe/sbz-switch/actions/runs/11375913501/artifacts/2066408460

Galenmacil commented 2 weeks ago

It work with the EXE you provided. However, and I do not mean to be finicky, it generates and error now with parameter 'list-devices'

2024-10-18T03:08:29.090336Z ERROR sbz_switch: Unexpected error error=unsupported rust type

Thanks though, this is not a big issue. I also liked the fact that it was reporting, via the console, each setting being applied when using the 'apply' command. It doesn't anymore but I would assume this is a temporary thing since this updated EXE is still in debug.

EDIT: The -v option no longer work. --version still does.

mdonoughe commented 2 weeks ago

This one should be better: https://github.com/mdonoughe/sbz-switch/actions/runs/11413661793/artifacts/2076800116

There is no -v option. It's -V for version.

Galenmacil commented 1 week ago

That is what happen when you are an older guy like me who learned console command in DOS where case sensitivity was non-existent... Sorry for reporting that as a problem.

The newer binaries works fine in every situation I could think of on my system.

Thanks for the update!