MarcoFalke fa2b529f92 refactor: Remove redundant call to IsArgSet ...

Checking for IsArgSet before calling GetArg while providing an arbitrary
default value as fallback is both confusing and fragile.

It is confusing, because the provided fallback is dead code. So it would
be better to just call GetArg without a fallback.

Even better would be to provide the true fallback value and sanitize it
as if it were user-input, but this can be done in a follow-up.

Removing the redundant call to IsArgSet will have to be done either way,
so do it now.

2025-03-25 10:38:00 +01:00

4.2 KiB

Raw Blame History

View Raw