[Hamlib-developer] [Hamlib/Hamlib] 0317e6: Remove double call to rig_token_lookup() and rot_t...

[dforsi <no...@gi...>]

  Branch: refs/heads/master
  Home:   https://github.com/Hamlib/Hamlib
  Commit: 0317e64dc83f9b0b3999d2c98df67608e4fa960e
      https://github.com/Hamlib/Hamlib/commit/0317e64dc83f9b0b3999d2c98df67608e4fa960e
  Author: Daniele Forsi IU5HKX <iu...@gm...>
  Date:   2025-08-20 (Wed, 20 Aug 2025)

  Changed paths:
    M tests/rigctl_parse.c
    M tests/rotctl_parse.c

  Log Message:
  -----------
  Remove double call to rig_token_lookup() and rot_token_lookup()

The needed value is already in the variable.

Test case /to check that the changes don't break the code):
tests/rigctl --set-conf=rig_pathname=test,write_delay=1,timeout=2 --show-conf Q | grep --no-group-separator -A1 -E "(rig_pathname|^write_delay|^timeout):"
tests/rotctl --set-conf=rot_pathname=test,write_delay=1,timeout=2 --show-conf Q | grep --no-group-separator -A1 -E "(rot_pathname|^write_delay|^timeout):"

The output before and after this patch is:
rig_pathname: "Path name to the device file of the rig"
        Default: /dev/rig, Value: test
write_delay: "Delay in ms between each byte sent out"
        Default: 0, Value: 1
timeout: "Timeout in ms"
        Default: 0, Value: 2
Command 'Q' not found!
rot_pathname: "Path name to the device file of the rotator"
        Default: /dev/rotator, Value: test
write_delay: "Delay in ms between each byte sent out"
        Default: 0, Value: 1
timeout: "Timeout in ms"
        Default: 0, Value: 2



To unsubscribe from these emails, change your notification settings at https://github.com/Hamlib/Hamlib/settings/notifications