Re: [pve-devel] [PATCH proxmox-firewall 09/37] config: firewall: add types for rules

[Stefan Hanreich <s.hanreich@proxmox.com>]

On 4/3/24 12:46, Max Carrara wrote:
> Hmm, since this is only used below, IMO it's fine that this returns a
> tuple like that on `Ok` - but should functions like that be used in
> multiple places, it might be beneficial to use a type alias or even a
> tuple struct for readability's sake.

Yes, I figured it was alright in this case, since it's mostly its own
function so the FromStr implementation is less noisy

> Should maaaybe note that it must be sorted for binary search, not just
> for any reason. :P

Will do!