Skip unoccupied tags

[stefur]

This adds the possbility to skip unoccupied tags when using cycle-tags as requested in #1. Append the flag -o or --occupied to the command in order move the tagmask over only the occupied tags.

Note to self: update readme and help.

[justinlovinger]

--occupied cycles to unoccupied tags when cycling previous on the first occupied tag or cycling next on the last occupied tag. For example, calling flow cycle-tags next 32 --occupied on tag 8 when 1 and 8 are occupied should cycle to 1. Instead, it cycles to 9, then 10, then 13, etc. Other than that minor issue, it works.

P.S. I suggest using Clap to parse arguments, as a separate pull-request of course, https://docs.rs/clap/latest/clap/. It also automatically generates help.

[stefur]

--occupied cycles to unoccupied tags when cycling previous on the first occupied tag or cycling next on the last occupied tag. For example, calling flow cycle-tags next 32 --occupied on tag 8 when 1 and 8 are occupied should cycle to 1. Instead, it cycles to 9, then 10, then 13, etc. Other than that minor issue, it works.

You're right. Admittedly I only use 6 tags so I never caught that mistake. Hopefully got it working properly now.

P.S. I suggest using Clap to parse arguments, as a separate pull-request of course, https://docs.rs/clap/latest/clap/. It also automatically generates help.

Yeah, I opted for picoargs for the simplicity. Always trade-offs. :)

[justinlovinger]

You're right. Admittedly I only use 6 tags so I never caught that mistake. Hopefully got it working properly now.

It appears to work now from my end.

Yeah, I opted for picoargs for the simplicity. Always trade-offs. :)

The utility may be complex enough now to warrant something more complex, such as allowing flow cycle-tags -o ... in addition to flow cycle-tags ... -o, but you do you.