Remove spurious string copy.

Ping! 🔔

Friendly ping! 🔔

Fast exit on unknown device

Improve message for wrong pcm device numbers

Remove error msg about unit number being negative

In D56845#1352108, @christos wrote:

In D56845#1349251, @fernape wrote:

I would argue that the user might not know it. "Invalid argument" is barely descriptive IMHO. If mixer knows the problem is that the supplied device number is negative, why not tell the user the exact problem instead of trusting the user "should know" something?

The reason I'm pushing back against this is because, apart from the fact that the -d option is already documented in the man page, the logic of the option is that you supply the name of the device (e.g., pcm0) or just its unit number. The error messages you introduce are accurate in the sense that they are more fine-grained in an input sense, but the point is that the user is trying to reference a device that does not exist, and these messages in my opinion make the issue more about the fact that the numbers are negative/out of range/etc, and not about the fact that the device simply does not exist. That's why I think "Invalid argument" for example is enough, because if someone deliberately tries to use the -d option, they should know what arguments it expects also, they wouldn't use an option randomly.

In D56845#1348954, @christos wrote:

To be fair, I don't think these changes are meaningful. mixer(8) will fail for all the cases you are trying to catch anyway, and I think the messages are clear enough already.

Negative dunit:

$ mixer -d -1
mixer: cannot set default unit to -1: Invalid argument

dunit larger than the total number of mixers:

$ mixer -d10
mixer: cannot set default unit to 10: Invalid argument

Too large number:

$ mixer -d9999999999999999999999
mixer: strtol(9999999999999999999999): Result too large

Invalid input:

$ mixer -dfoobar
mixer: strtol(foobar): Invalid argument

The current scheme is uniform, and if a user wants to use the -d option in the first place, they should know that the argument is supposed to be a unit number (or pcm<unit_number>), so the current messages point to what the error is pretty well IMHO.

Friendly ping 🔔 !!

LGTM!

LGTM!

LGTM, thanks!

LGTM!

Terriffic. Thanks! (with ports-secteam@ hat) that for some reason doesn't show in Phab.

LGTM!

Ya que decimos "porter" probablemente debamos dejar "ports".

In D53910#1304417, @ziaee wrote:

Another possibility might be "For over 30 years, this grassroots project has been powered by volunteers, research institutions, and corporate supporters."